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

Pin Go version to 1.21 #976

Merged
merged 2 commits into from
Oct 17, 2024
Merged

Pin Go version to 1.21 #976

merged 2 commits into from
Oct 17, 2024

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Oct 11, 2024

The library is now dependency in image builder and we agreed on Slack that although it is just two functions from the crypt package, it makes sense to share this code (and from otk), therefore we must do the effort of not upgrading the Go version to latest and greatest but keep the version in check with UBI containers (1.21.x currently).

All dependant bot updates which will fail the build must be closed or kept in a queue until Go can be upgraded few months later.

@lzap
Copy link
Contributor Author

lzap commented Oct 11, 2024

SBOM support for DNF5 is not implemented

Is this relevant?

@lzap
Copy link
Contributor Author

lzap commented Oct 16, 2024

SBOM failures again.

go.mod Outdated Show resolved Hide resolved
@achilleas-k
Copy link
Member

SBOM failures again.

Not required to merge.

@lzap lzap force-pushed the go-downgrade branch 2 times, most recently from e7b162b to 2252b89 Compare October 17, 2024 07:31
@lzap
Copy link
Contributor Author

lzap commented Oct 17, 2024

So I wrote a small utility which allows bumping of dependencies while keeping the Go requirement. It allowed me to bump all dependencies which can be updated and keep Go 1.21.

@lzap
Copy link
Contributor Author

lzap commented Oct 17, 2024

@achilleas-k ready for re-review

@lzap lzap changed the title Go downgrade Pin Go version to 1.21 Oct 17, 2024
@achilleas-k achilleas-k enabled auto-merge October 17, 2024 08:29
@achilleas-k achilleas-k added this pull request to the merge queue Oct 17, 2024
Merged via the queue into osbuild:main with commit b0ca45d Oct 17, 2024
18 of 19 checks passed
@lzap lzap deleted the go-downgrade branch October 17, 2024 09:08
@lzap
Copy link
Contributor Author

lzap commented Oct 17, 2024

Thank you!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants