Skip to content

[release/1.6] Downgrade MinGW to version 10.2.0#7133

Merged
mikebrow merged 1 commit intocontainerd:release/1.6from
kzys:backport/1.6/7106
Jul 5, 2022
Merged

[release/1.6] Downgrade MinGW to version 10.2.0#7133
mikebrow merged 1 commit intocontainerd:release/1.6from
kzys:backport/1.6/7106

Conversation

@kzys
Copy link
Copy Markdown
Member

@kzys kzys commented Jul 5, 2022

There is currently an issue in the race detector in Go on Windows when
used with a newer version of GCC. The issue was first reported here:

golang/go#46099

Fixes #7104

Signed-off-by: Gabriel Adrian Samfira [email protected]
(cherry picked from commit 1ef4bda)

Signed-off-by: Kazuyoshi Kato [email protected]

There is currently an issue in the race detector in Go on Windows when
used with a newer version of GCC. The issue was first reported here:

golang/go#46099

Fixes containerd#7104

Signed-off-by: Gabriel Adrian Samfira <[email protected]>
(cherry picked from commit 1ef4bda)

Signed-off-by: Kazuyoshi Kato <[email protected]>
@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kzys
Copy link
Copy Markdown
Member Author

kzys commented Jul 5, 2022

#7131 was having the following issue.

https://github.com/containerd/containerd/runs/7199569926?check_suite_focus=true#step:9:220

=== Failed
=== FAIL: pkg/cri/server  (0.00s)
==6156==ERROR: ThreadSanitizer failed to allocate 0x000004299000 (69832704) bytes at 0x200ded1054000 (error code: 87)
FAIL	github.com/containerd/containerd/pkg/cri/server	0.263s
DONE 516 tests, 9 skipped, 1 failure in 242.336s
mingw32-make: *** [makefile:206: test] Error 1
Error: Process completed with exit code 2.

@gabriel-samfira does it make sense to backport the change to 1.5 branch as well?

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kzys kzys marked this pull request as ready for review July 5, 2022 17:12
@gabriel-samfira
Copy link
Copy Markdown
Contributor

@kzys this will happen in any environment with a mingw version above 10.2.0 so it will need to be backported to all supported branches. Sadly, the default GitHub runners have an updated version of MinGW.

@kzys
Copy link
Copy Markdown
Member Author

kzys commented Jul 5, 2022

@gabriel-samfira Thanks! Opening #7134 for 1.5.x

Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mikebrow mikebrow merged commit 288440a into containerd:release/1.6 Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants