build:v51 anthoturc/amr-multiarch-fork - #94
Merged
Merged
Conversation
Member
|
@anthoturc I looked why the build is failing. The A second layer down, the current state of https://update.k3s.io/v1-release/channels is an invalid tls certificate. Any re-run of this step (after the diff is updated) will likely also fail. I think we still need are the version number from |
raykroeker
force-pushed
the
anthoturc/amr-multiarch-fork
branch
from
September 16, 2026 17:49
aea8be8 to
680195e
Compare
Member
Mitigated by bae78f1 |
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Previously the devcontainer build infrastructure would install docker-compose in the container via scripts, but that script ended up requiring global changes to Python code in order to install docker-compose. Now we just install it here to avoid that problem. Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
…commands Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
This is set to never with podman since I'm still experimenting. Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
This is useful to avoid locking the cache, which is desirable if using buildah, since it has had bugs around this feature. The tradeoff is that you could potentially find locking issues if running multiple parallel apt commands (such as in CI). Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
This reverts commit 27ab1a5333cc5c5af743c6878f42da23aa5050bf. Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Use the latest stable version of debian (trixie) resulting in some cascading changes. Add llvm's repo key without using the removed 'add-key' binary. This *also* bumps the version of llvm from bookworm-19 to trixie-22. Drop j5j from a curl install to using 'node-json5' available from the debian repo. It is a drop-in replacement for j5j that strips json5 and outputs base json. Install just and yq from the deb repo vs from a url. Install git from default deb repo vs backports. Install docker using devcontainer/features versus from legacy microsoft repo. 'origin/amr/multiarch-and-podman-support'. Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Include node-json5, just and yq packages in devcontainer. Previously the stages were orphaned. True up all of the cache mount options. Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
…s, packages) Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
anthoturc
force-pushed
the
anthoturc/amr-multiarch-fork
branch
from
September 17, 2026 16:26
bae78f1 to
488c9ba
Compare
Signed-off-by: Anthony Turcios (anthoturc) <anthony@buoyant.io>
anthoturc
changed the title
raykroeker
requested changes
Sep 17, 2026
raykroeker
left a comment
raykroeker
left a comment
Member
There was a problem hiding this comment.
1 minor change to the Dockerfile and I think we're good.
raykroeker
approved these changes
Sep 17, 2026
alpeb
approved these changes
Sep 17, 2026
prior to this commit, the arm64 and amd64 artifacts depended on the arch of the host machine. going forward, dev images that are being published should build both by default.
j5j appears to only have artifacts for amd64 so it is being built manually in the docker image. just and yq are being copied over since they were missing in the last commit
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.