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

Makefile: Run generate-k8s-api in builder image #32063

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Apr 18, 2024

Requires: #31687

By default, run the make generate-k8s-api target inside the Cilium builder image to ensure that the correct version of protoc is used to generate the code.

Suggested-by: @rolinh

@joestringer joestringer requested review from a team as code owners April 18, 2024 19:32
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Apr 18, 2024
@joestringer
Copy link
Member Author

/test

@aanm aanm added the dont-merge/blocked Another PR must be merged before this one. label Apr 18, 2024
@aanm
Copy link
Member

aanm commented Apr 18, 2024

Conflicts with #31687

@joestringer joestringer marked this pull request as draft April 18, 2024 21:51
@joestringer joestringer changed the base branch from main to pr/christarazi/k8s-1.30 April 18, 2024 21:51
@joestringer
Copy link
Member Author

/test

@christarazi christarazi force-pushed the pr/christarazi/k8s-1.30 branch 2 times, most recently from 69100fe to 7c0faf9 Compare April 23, 2024 18:49
Base automatically changed from pr/christarazi/k8s-1.30 to main April 24, 2024 08:12
@joestringer joestringer removed the dont-merge/blocked Another PR must be merged before this one. label Apr 24, 2024
@joestringer joestringer marked this pull request as ready for review April 24, 2024 14:51
By default, run the 'make generate-k8s-api' target inside the Cilium
builder image to ensure that the correct version of protoc is used to
generate the code.

Suggested-by: Robin Hahling <robin.hahling@gw-computing.net>
Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer
Copy link
Member Author

/test

@joestringer joestringer added this pull request to the merge queue Apr 29, 2024
Merged via the queue into main with commit 00df179 Apr 29, 2024
261 checks passed
@joestringer joestringer deleted the pr/joe/dockerize-k8s-api-gen branch April 29, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

3 participants