Skip to content

ci: Make node buildbox runtime-architecture specific#53943

Merged
camscale merged 1 commit intomasterfrom
camh/node-buildbox-arch
Jun 13, 2025
Merged

ci: Make node buildbox runtime-architecture specific#53943
camscale merged 1 commit intomasterfrom
camh/node-buildbox-arch

Conversation

@camscale
Copy link
Copy Markdown
Contributor

@camscale camscale commented Apr 11, 2025

Add a runtime architecture suffix to the node buildbox OCI image name so
that we do not keep rebuilding it when doing an ARM64 release. ARM64
releases are built on ARM64 hosts, but the node buildbox is for AMD64 so
the ARM64 build always has to rebuild it. Adding a runtime architecture
suffix should prevent this rebuild and make the release faster.

Test-run: https://github.com/gravitational/teleport.e/actions/runs/14305406062
Companion: https://github.com/gravitational/teleport.e/pull/6360


I have manually triggered the workflow to build the buildboxes on master with
this PR, so the images with the architecture suffix are present in ghcr:

The workflow run was triggered with:

$ gh workflow run --repo gravitational/teleport.e build-buildboxes.yaml \
    --ref camh/node-buildbox-arch \
    --raw-field branch=camh/node-buildbox-arch

@camscale camscale added the no-changelog Indicates that a PR does not require a changelog entry label Apr 11, 2025
@github-actions github-actions bot requested review from atburke and hugoShaka April 11, 2025 04:59
@camscale camscale force-pushed the camh/node-buildbox-arch branch from 451101f to 490936c Compare June 13, 2025 06:42
Add a runtime architecture suffix to the node buildbox OCI image name so
that we do not keep rebuilding it when doing an ARM64 release. ARM64
releases are built on ARM64 hosts, but the node buildbox is for AMD64 so
the ARM64 build always has to rebuild it. Adding a runtime architecture
suffix should prevent this rebuild and make the release faster.
@camscale camscale force-pushed the camh/node-buildbox-arch branch from 490936c to d4cd6c9 Compare June 13, 2025 06:45
@camscale camscale added this pull request to the merge queue Jun 13, 2025
Merged via the queue into master with commit 416c099 Jun 13, 2025
41 checks passed
@camscale camscale deleted the camh/node-buildbox-arch branch June 13, 2025 06:59
@backport-bot-workflows
Copy link
Copy Markdown
Contributor

@camscale See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR
branch/v17 Create PR
branch/v18 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/branch/v17 backport/branch/v18 no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants