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

make: add "run-builder" target #28587

Merged
merged 1 commit into from Oct 30, 2023
Merged

Conversation

jrajahalme
Copy link
Member

@jrajahalme jrajahalme commented Oct 13, 2023

Add make target for running bash in the cilium builder container. This provides quick access for the correct build environment, with the correct go version for example. If backported to releases, this makes it easy to to work with go vendoring with different releases, which each require a different Go version to be able to run 'go mod' commands.

@jrajahalme jrajahalme added release-note/misc This PR makes changes that have no direct user impact. area/build Anything to do with the build, more general then area/CI labels Oct 13, 2023
@jrajahalme jrajahalme requested a review from a team as a code owner October 13, 2023 10:02
@jrajahalme jrajahalme requested a review from jibi October 13, 2023 10:02
Add make target for running bash in the cilium builder container.  This
provides quick access for the correct build environment, with the correct
go version for example. If backported to releases, this makes it easy to
to work with go vendoring with different releases, which each require a
different Go version to be able to run 'go mod' commands.

Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
@jrajahalme jrajahalme added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 30, 2023
@aanm aanm merged commit b257fd5 into cilium:main Oct 30, 2023
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Anything to do with the build, more general then area/CI 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.

None yet

3 participants