Skip to content

renovate: fix API files generation using renovate #35676

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

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

aanm
Copy link
Member

@aanm aanm commented Oct 31, 2024

When trying to use renovate to generate the API files, every time protobuf is updated for example, as renovate GitHub action didn't have Golang installed, the generation of files simply fail. Thus we will pass GO as the golang version available on the cilium builder image.

Fixes: c6fb9e4 ("Makefile: generate API with protocolbuffers/protobuf update")

When trying to use renovate to generate the API files, every time
protobuf is updated for example, as renovate GitHub action didn't have
Golang installed, the generation of files simply fail. Thus we will pass
GO as the golang version available on the cilium builder image.

Fixes: c6fb9e4 ("Makefile: generate API with protocolbuffers/protobuf update")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested review from a team as code owners October 31, 2024 13:44
@aanm aanm requested a review from brlbil October 31, 2024 13:44
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 31, 2024
@aanm aanm added release-note/misc This PR makes changes that have no direct user impact. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 31, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 31, 2024
@aanm
Copy link
Member Author

aanm commented Oct 31, 2024

/test

@aanm aanm enabled auto-merge October 31, 2024 13:44
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Curious how did you notice this? Was it failing loudly or silently?

@aanm
Copy link
Member Author

aanm commented Oct 31, 2024

Curious how did you notice this? Was it failing loudly or silently?

@joestringer It failed here

@aanm aanm added this pull request to the merge queue Nov 4, 2024
Merged via the queue into main with commit add0679 Nov 4, 2024
276 checks passed
@aanm aanm deleted the pr/fix-renovate-golang branch November 4, 2024 12:12
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants