-
Notifications
You must be signed in to change notification settings - Fork 300
Publish test images in a multiarch manifest #3883
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
base: main
Are you sure you want to change the base?
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
5a1ce8a to
4aea4c1
Compare
4aea4c1 to
67fa95f
Compare
67fa95f to
9a9219d
Compare
9a9219d to
487fc7d
Compare
|
/test pull-containerized-data-importer-e2e-hpp-latest |
fd24749 to
e971db9
Compare
|
/retest |
|
/cc @akalenyu |
e971db9 to
01ef6fd
Compare
|
A few unsupported images were pushed for arm/s390x, so I added a filter |
The test images weren't pushed correctly, so I'd basically just push them in the same manner as the operators. Signed-off-by: Thomas-David Griedel [email protected]
01ef6fd to
65b3882
Compare
|
For Validation: |
|
Found a bug where |
Signed-off-by: Thomas-David Griedel [email protected]
What this PR does / why we need it:
The test images weren't manifested correctly, so this pushes them in the same manner as the operators.
Push logic now filters per-image supported architectures to avoid publishing broken manifests.
Adds per-image Bazel
container_push targets.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #3688
Special notes for your reviewer:
Release note: