add cli-stack for binary distribution#3195
Open
sampras343 wants to merge 2 commits intoconforma:release-v0.8from
Open
add cli-stack for binary distribution#3195sampras343 wants to merge 2 commits intoconforma:release-v0.8from
sampras343 wants to merge 2 commits intoconforma:release-v0.8from
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan Review Summary by QodoAdd CLI stack for multi-architecture binary distribution
WalkthroughsDescription• Add multi-architecture CLI binary distribution image • Create Tekton pipeline configurations for PR and push events • Aggregate pre-built binaries from multiple architectures into single image • Support Linux, Darwin, and Windows platforms across architectures Diagramflowchart LR
A["Per-arch CLI builds<br/>amd64, arm64, ppc64le, s390x"] -->|"Extract binaries"| B["Packager stage<br/>decompress and repackage"]
B -->|"Collect all binaries"| C["Minimal UBI9 image<br/>with all tar.gz archives"]
D["Tekton PR Pipeline<br/>conforma-cli-stack-pull-request.yaml"] -->|"Builds on PR"| C
E["Tekton Push Pipeline<br/>conforma-cli-stack-push.yaml"] -->|"Builds on push"| C
C -->|"Distributes to"| F["quay.io registry<br/>for developer access"]
File Changes1. Dockerfile.cli-stack.rh
|
Contributor
Code Review by Qodo
1. Unbuildable digest placeholders
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Contributor
ⓘ You are approaching your monthly quota for Qodo. Upgrade your plan Review Summary by QodoAdd CLI stack for multi-platform binary distribution
WalkthroughsDescription• Add multi-architecture CLI binary aggregation image • Create Tekton pipeline configurations for pull requests and push events • Bundle CLI binaries for Linux, Darwin, and Windows platforms • Enable binary distribution via container registry Diagramflowchart LR
A["Multi-arch CLI builds<br/>amd64, arm64, ppc64le, s390x"] -->|"Aggregate binaries"| B["Dockerfile.cli-stack.rh<br/>Packager stage"]
B -->|"Extract & compress"| C["Binary archives<br/>Linux, Darwin, Windows"]
C -->|"Final minimal image"| D["quay.io/securesign/<br/>conforma-cli-stack"]
E["Pull Request trigger"] -->|"Build on PR"| F["conforma-cli-stack-pull-request.yaml"]
G["Push to release-v0.8"] -->|"Build on push"| H["conforma-cli-stack-push.yaml"]
F -->|"Uses pipeline"| D
H -->|"Uses pipeline"| D
File Changes1. Dockerfile.cli-stack.rh
|
Contributor
Code Review by Qodo
1. Unbuildable digest placeholders
|
Author
simonbaird
reviewed
Mar 25, 2026
simonbaird
reviewed
Mar 25, 2026
simonbaird
reviewed
Mar 25, 2026
simonbaird
reviewed
Mar 25, 2026
sampras343
force-pushed
the
cli-stack
branch
3 times, most recently
from
6ac7a03 to
b75a324
Compare
Member
|
/ok-to-test |
Member
|
The acceptance test failure here is unexpected. Created #3197 to (hopefully) fix it. If that works and gets merged, I'll rebase this one and see if we can get it green. |
Signed-off-by: Sachin Sampras M <sampras343@gmail.com>
Signed-off-by: Sachin Sampras M <sampras343@gmail.com>
Member
|
Rebased on fresh upstream/release-v0.8, hopefully this will get the tests passing. |
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Member
|
/ok-to-test |
Member
|
Should be ready to merge, but will wait for @sampras343 to give the go-ahead. |
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.
Add cli-stack for binary aggregation into a single image and distribution to developers portal