Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: bump go version to 1.21.7 #357

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

BbolroC
Copy link
Member

@BbolroC BbolroC commented Mar 26, 2024

Please check out the issue below for details.

It is to update a go version to 1.21.7 which makes the CI go green again.

Fixes: #356

Signed-off-by: Hyounggyu Choi [email protected]

Copy link
Contributor

@ldoktor ldoktor left a comment

Choose a reason for hiding this comment

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

Ack to restore the CI but I'd be nice to investigate why is it failing with the latest go...

It is to update a go version to 1.21.7 which makes the CI go green again.

Fixes: confidential-containers#356

Signed-off-by: Hyounggyu Choi <[email protected]>
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

Thanks for the investigation and fix. It feels like us doing:

go install sigs.k8s.io/controller-runtime/tools/setup-envtest@latest

rather than having a pinned version of our dependencies means things like this are inevitable, so maybe we can discuss a longer-term strategy on the CI calls cc @wainersm

@BbolroC
Copy link
Member Author

BbolroC commented Mar 26, 2024

Ack to restore the CI but I'd be nice to investigate why is it failing with the latest go...

Hi, Lukas. I think the upstream is aware of this at kubernetes-sigs/controller-tools#888. Hopefully, they will resolve it soon.

@BbolroC BbolroC merged commit 61d4540 into confidential-containers:main Mar 26, 2024
10 checks passed
@BbolroC BbolroC deleted the bump-go-version branch March 26, 2024 17:44
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.

CI errors on go version while installing setup-envtest
3 participants