Skip to content

[release/1.6] Update builds to go 1.21.8#9945

Merged
estesp merged 1 commit intocontainerd:release/1.6from
dmcgowan:go-1.21.8-1.6
Mar 8, 2024
Merged

[release/1.6] Update builds to go 1.21.8#9945
estesp merged 1 commit intocontainerd:release/1.6from
dmcgowan:go-1.21.8-1.6

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented Mar 7, 2024

@AkihiroSuda AkihiroSuda changed the title update to go 1.21.8, 1.22.1 [release/1.6] update to go 1.21.8, 1.22.1 Mar 7, 2024
Copy link
Copy Markdown
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

Comment thread .github/workflows/ci.yml
# Go version we currently use to build containerd across all CI.
# Note: don't forget to update `Binaries` step, as it contains the matrix of all supported Go versions.
GO_VERSION: "1.20.13"
GO_VERSION: "1.21.8"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we missed one change in #9890. Because after setting up composite action, we should not be setting GO_VERSION env in the workflow file.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Included the missing change in the action now, but this file doesn't seem to be updated to use the action

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Have pushed the changes here dmcgowan#3

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have added one more commit to fix the linter issues

@austinvazquez
Copy link
Copy Markdown
Member

austinvazquez commented Mar 8, 2024

Looks like the changes to use the composite action were dropped again in the push.

@dmcgowan
Copy link
Copy Markdown
Member Author

dmcgowan commented Mar 8, 2024

@austinvazquez I removed it, CI was failing. We can update that in a follow up.

@estesp estesp merged commit 187d5b9 into containerd:release/1.6 Mar 8, 2024
@dmcgowan dmcgowan changed the title [release/1.6] update to go 1.21.8, 1.22.1 [release/1.6] Update builds to go 1.21.8 Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants