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

images: Use cilium-builder image instead of golang to build hubble #35697

Merged

Conversation

learnitall
Copy link
Contributor

The hubble image currently uses the golang docker image within the build stage. This commit modifies the build stage to use the cilium-builder image instead which contains the necessary tooling to allow for arm64 builds in CI workflows that run on amd64 machines.

See #35351 and #35324 for more context.

@learnitall learnitall added release-note/misc This PR makes changes that have no direct user impact. area/build Anything to do with the build, more general than area/CI affects/v1.14 This issue affects v1.14 branch labels Nov 1, 2024
@learnitall learnitall requested a review from a team as a code owner November 1, 2024 21:13
@learnitall learnitall requested a review from rolinh November 1, 2024 21:13
@learnitall learnitall added affects/v1.15 This issue affects v1.15 branch affects/v1.16 This issue affects v1.16 branch needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch and removed affects/v1.14 This issue affects v1.14 branch labels Nov 1, 2024
@learnitall
Copy link
Contributor Author

/test

1 similar comment
@learnitall
Copy link
Contributor Author

/test

@learnitall learnitall force-pushed the pr/learnitall/cilium-hubble-arm branch 2 times, most recently from d158baf to f095a30 Compare November 20, 2024 18:33
@learnitall
Copy link
Contributor Author

/test

@learnitall learnitall force-pushed the pr/learnitall/cilium-hubble-arm branch from f095a30 to 33de2ab Compare November 22, 2024 16:04
@learnitall
Copy link
Contributor Author

/test

1 similar comment
@michi-covalent
Copy link
Contributor

/test

@michi-covalent
Copy link
Contributor

hmmm eks test keeps failing. @learnitall do you wanna try rebasing and see if it helps? 💭

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The hubble image currently uses the golang docker image within the build
stage. This commit modifies the build stage to use the cilium-builder
image instead which contains the necessary tooling to allow for arm64
builds in CI workflows that run on amd64 machines.

See cilium#35351 and
cilium#35324 for more context.

Signed-off-by: Ryan Drew <ryan.drew@isovalent.com>
@learnitall learnitall force-pushed the pr/learnitall/cilium-hubble-arm branch from 33de2ab to eb80c98 Compare December 2, 2024 18:19
@learnitall
Copy link
Contributor Author

Just pushed a rebase @michi-covalent, let's see if that helps.

@michi-covalent
Copy link
Contributor

/test

@michi-covalent michi-covalent added this pull request to the merge queue Dec 3, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 3, 2024
Merged via the queue into cilium:main with commit f0d3f93 Dec 3, 2024
64 checks passed
@jschwinger233 jschwinger233 mentioned this pull request Dec 3, 2024
2 tasks
@jschwinger233 jschwinger233 added backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. and removed needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Dec 3, 2024
@jschwinger233 jschwinger233 mentioned this pull request Dec 3, 2024
1 task
@jschwinger233 jschwinger233 added backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. and removed needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Dec 3, 2024
@github-actions github-actions bot added backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. and removed backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. labels Dec 3, 2024
@joestringer joestringer added backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. and removed backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. labels Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/v1.15 This issue affects v1.15 branch affects/v1.16 This issue affects v1.16 branch area/build Anything to do with the build, more general than area/CI backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. 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

6 participants