Skip to content

Use the Go toolchain in CI matrix to build binaries#9946

Merged
estesp merged 1 commit intocontainerd:mainfrom
austinvazquez:use-go-version-from-matrix
Mar 8, 2024
Merged

Use the Go toolchain in CI matrix to build binaries#9946
estesp merged 1 commit intocontainerd:mainfrom
austinvazquez:use-go-version-from-matrix

Conversation

@austinvazquez
Copy link
Copy Markdown
Member

@austinvazquez austinvazquez commented Mar 8, 2024

Issue

The binaries job in CI is not respecting the Go version specified in the matrix.

image

Description

This change updates the binaries job in CI to use the Go version specified in the matrix.

Testing

image

Signed-off-by: Austin Vazquez <macedonv@amazon.com>
@k8s-ci-robot
Copy link
Copy Markdown

Hi @austinvazquez. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@austinvazquez austinvazquez marked this pull request as ready for review March 8, 2024 01:00
@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Mar 8, 2024

Good find

Copy link
Copy Markdown
Member

@dims dims left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp added this pull request to the merge queue Mar 8, 2024
Merged via the queue into containerd:main with commit 5102387 Mar 8, 2024
@akhilerm akhilerm added cherry-pick/1.6.x cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels Mar 8, 2024
@austinvazquez austinvazquez deleted the use-go-version-from-matrix branch March 8, 2024 03:49
@akhilerm akhilerm added cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch and removed cherry-pick/1.6.x cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-picked/1.6.x PR commits are cherry-picked into release/1.6 branch cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch needs-ok-to-test size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants