Skip to content

[release/1.7] backport windows runner and golang toolchain updates#11972

Merged
AkihiroSuda merged 5 commits intocontainerd:release/1.7from
austinvazquez:cherry-pick-golang-06052025-updates-to-1.7
Jul 15, 2025
Merged

[release/1.7] backport windows runner and golang toolchain updates#11972
AkihiroSuda merged 5 commits intocontainerd:release/1.7from
austinvazquez:cherry-pick-golang-06052025-updates-to-1.7

Conversation

@austinvazquez
Copy link
Member

@austinvazquez austinvazquez commented Jun 10, 2025

This change backports a few CI updates for to release/1.7 branch.

  1. Updates Windows runners to run on Windows 2022 and Windows 2025 (Windows 2019 runners are deprecated)
  2. Updates Golang toolchain in build and release CI to latest Go 1.23.x and Go 1.24.x.

Note: cherry-picks were not clean as 1) unlike the release branches main no longer supports Go 1.23 2) the integration test help function for skipping tests on Windows 2025 runners moved from integration/client to integration/platform due to integration/client being a module in release/1.7 branch 3) the pre- 0.13.0 hcsshim library for osversion required the application to be manifested with the OS version, so this change copies the version code added to 0.13.0 which no longer requires the manifest.

@k8s-ci-robot
Copy link

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

@github-project-automation github-project-automation bot moved this to Needs Triage in Pull Request Review Jun 10, 2025
@austinvazquez austinvazquez force-pushed the cherry-pick-golang-06052025-updates-to-1.7 branch 2 times, most recently from 16c2bff to b9bcac1 Compare June 11, 2025 00:38
@austinvazquez austinvazquez force-pushed the cherry-pick-golang-06052025-updates-to-1.7 branch 3 times, most recently from d2eb354 to 41244eb Compare June 11, 2025 15:14
@austinvazquez austinvazquez marked this pull request as ready for review June 11, 2025 15:21
@dosubot dosubot bot added area/toolchain Build and Release Toolchain github_actions Pull requests that update GitHub Actions code platform/windows Windows labels Jun 11, 2025
@austinvazquez austinvazquez marked this pull request as draft June 19, 2025 16:13
@austinvazquez
Copy link
Member Author

Debugging why image load tests are not being skipped on Windows 2025.

@austinvazquez austinvazquez force-pushed the cherry-pick-golang-06052025-updates-to-1.7 branch from dec5cbf to a6d949a Compare June 19, 2025 17:21
@austinvazquez
Copy link
Member Author

Okay the challenge with skipping the tests on Windows 2025 for release/1.7 branch is the build number returned by hcsshim is slightly different. It seems to be related to this change in hcsshim.

So I am thinking the cri test binary here is not manifested which is causing the skip logic to fail. Adding some debug information to a test CI run (I don't have a Windows machine to test locally) to see if we can workaround it without having to update hcsshim in the release branches.

kiashok and others added 5 commits June 19, 2025 12:16
Signed-off-by: Kirtana Ashok <kiashok@microsoft.com>
(cherry picked from commit 2f1948a)
Signed-off-by: Austin Vazquez <austin.vazquez.dev@gmail.com>
- go1.23.8 (released 2025-04-01) includes security fixes to the net/http
  package, as well as bug fixes to the runtime and the go command.
  Ref: https://github.com/golang/go/issues?q=milestone%3AGo1.23.8+label%3ACherryPickApproved
- go1.24.2 (released 2025-04-01) includes security fixes to the net/http
  package, as well as bug fixes to the compiler, the runtime, the go
  command, and the crypto/tls, go/types, net/http, and testing packages.
  Ref: https://github.com/golang/go/issues?q=milestone%3AGo1.24.2+label%3ACherryPickApproved

Signed-off-by: Akhil Mohan <akhilerm@gmail.com>
(cherry picked from commit 5629e9f)
Signed-off-by: Austin Vazquez <austin.vazquez.dev@gmail.com>
use go1.23.8 as the default go version for running in CI and making
release binaries.

NOTE: This is done so that when release/2.1 branch cut happens the
2.1 release uses 1.23.x for building release binaries rather than
1.24.x. Once the 2.1 release is out, this will be reverted and
go1.24.x will be made the default in the main branch

Signed-off-by: Akhil Mohan <akhilerm@gmail.com>
(cherry picked from commit 6f93c65)
Signed-off-by: Austin Vazquez <austin.vazquez.dev@gmail.com>
Signed-off-by: Austin Vazquez <austin.vazquez.dev@gmail.com>
(cherry picked from commit fea77e1)
Signed-off-by: Austin Vazquez <austin.vazquez.dev@gmail.com>
(cherry picked from commit 415df38)
@austinvazquez austinvazquez force-pushed the cherry-pick-golang-06052025-updates-to-1.7 branch from a6d949a to a35978f Compare June 19, 2025 19:16
@austinvazquez austinvazquez marked this pull request as ready for review June 19, 2025 21:23
@austinvazquez
Copy link
Member Author

@kiashok , @akhilerm , @estesp this PR is similar to #11970 and #11969 but for the release/1.7 branch. PTAL.

@austinvazquez austinvazquez requested a review from estesp June 19, 2025 21:27
@austinvazquez
Copy link
Member Author

I have root caused the fuzz failures to be unrelated to this change and opened #12007 with the findings. Can those be considered that non-blocking for now?

@austinvazquez
Copy link
Member Author

Needs one more pair of eyes. Unblocks v1.7.28 release.

@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Jul 1, 2025
@AkihiroSuda AkihiroSuda merged commit a8a4d95 into containerd:release/1.7 Jul 15, 2025
92 of 96 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Jul 15, 2025
@austinvazquez austinvazquez deleted the cherry-pick-golang-06052025-updates-to-1.7 branch July 17, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/toolchain Build and Release Toolchain github_actions Pull requests that update GitHub Actions code platform/windows Windows size/L

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants