Skip to content

Switch to Go 1.25+#5205

Merged
rata merged 3 commits into
opencontainers:mainfrom
kolyshkin:carry-5169
Mar 31, 2026
Merged

Switch to Go 1.25+#5205
rata merged 3 commits into
opencontainers:mainfrom
kolyshkin:carry-5169

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

Go 1.24 is no longer supported.

(this is a carry of #5169 with the additional test fix)

kolyshkin and others added 3 commits March 30, 2026 13:13
Since switching to Go 1.25 in go.mod, the "detect fd leaks" test fails
like this:

> not ok 57 runc create[detect fd leak as comprehensively as possible]
> # (in test file tests/integration/create.bats, line 76)
> #   `[ "$violation_found" -eq 0 ]' failed
> ...
> # Violation: FD 9 -> '/system.slice/runc-test_busybox.scope/cpu.cfs_quota_us'
> # Violation: FD 10 -> '/system.slice/runc-test_busybox.scope/cpu.cfs_period_us'
> ...

This happens because Go 1.25 adds a feature to dynamically set GOMAXPROC
based on current CPU quota values. This feature can be disabled by setting

	GODEBUG=containermaxprocs=0,updatemaxprocs=0

but it is harmless to keep it (except for the above test failure).

Add an exception to the test case.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
@kolyshkin kolyshkin marked this pull request as ready for review March 30, 2026 21:35
@kolyshkin kolyshkin added this to the 1.5.0-rc.2 milestone Mar 30, 2026
@kolyshkin kolyshkin added the backport/1.5-todo A PR in main branch which needs to be backported to release-1.5 label Mar 30, 2026
Copy link
Copy Markdown
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@rata rata merged commit 348d92f into opencontainers:main Mar 31, 2026
55 checks passed
@kolyshkin kolyshkin added backport/1.5-done A PR in main branch which has been backported to release-1.5 and removed backport/1.5-todo A PR in main branch which needs to be backported to release-1.5 labels Apr 1, 2026
@kolyshkin kolyshkin mentioned this pull request Apr 3, 2026
@kolyshkin kolyshkin removed this from the 1.5.0-rc.2 milestone Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.5-done A PR in main branch which has been backported to release-1.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants