Conversation
A tag alone lets the builder and the runtime drift under a rebuild; the digest of each multi-platform index fixes every architecture at once. The header now carries the docker run recipe the image expects. Signed-off-by: xgreenx <xgreenx9999@gmail.com> Assisted-by: Claude Fable 5.1
One tag must pull on GitHub runners, EKS nodes of either architecture and Apple Silicon. Each platform compiles natively on a runner of its own architecture and pushes by digest; a final job joins the digests under the tag. QEMU is not used: an emulated tlsn/mpz build costs hours, while ubuntu-24.04-arm is native and free for a public repo. Same shape as libid-org/notary#10. Signed-off-by: xgreenx <xgreenx9999@gmail.com> Assisted-by: Claude Fable 5.1
A missing notary_url or gas signer used to surface only when a rotation was due, weeks after the deployment looked healthy. `run` and `once` without --dry-run now check both at startup, and a config file that does not exist says where its path came from. Signed-off-by: xgreenx <xgreenx9999@gmail.com> Assisted-by: Claude Fable 5.1
The image needs Cargo.toml, Cargo.lock and src/ and nothing else. Without a .dockerignore a local build ships target/ to the daemon as well -- 3.6 GB against 265 kB here -- for files no stage copies. Assisted-by: Claude Opus 5 Signed-off-by: xgreenx <xgreenx9999@gmail.com>
Contributor
Author
|
Multi-arch proof for 0afb69c: Custom Docker image run →
|
Contributor
Author
|
Superseded by #9, which is now a single PR against |
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
1 participant
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.
One
ghcr.io/libid-org/keeper:<tag>must run on GitHub runners, EKS nodes of either architecture and Apple Silicon, as libid-org/notary#10 does. Stacked on #7.Changes
docker runrecipe..dockerignoreallowlists the three paths the build copies; the local context drops from 3.6 GB to 265 kB.run/oncewithout--dry-runrefuse a config missingnotary_urlor a gassigner, instead of failing at the first rotation weeks later.ubuntu-24.04,ubuntu-24.04-arm), push by digest, join both under one manifest list. No QEMU.Verified
cargo test --all: 31 unit + 4 integration pass, 4 of them new. Clippy-D warningsand nightly fmt clean.actionlintclean on all four workflows.linux/arm64built natively on an M-series Mac: 150 MB, uid 10001.custom-0afb69cas one manifest list,linux/amd64+linux/arm64, both built natively. Digests in the comment below.notary_url, nosignereach exit 1 with the intended message, andonce --dry-runpolls Google's live JWKS for its two real kids.Not verified / known issues
LIBID_DEPLOY_URLnames chain-configurations v0.7.0, not released yet.