-
Notifications
You must be signed in to change notification settings - Fork 25
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
Attempt to fix multiarch workflow #342
Conversation
Shortlived images are now built from scratch, not from a base. (Same can be done later for bundle & catalog) It simplifies the workflows especially with multiarch Also make multiarch configurable, and only for merged commits, not per PR
Codecov Report
@@ Coverage Diff @@
## main #342 +/- ##
=======================================
Coverage 51.42% 51.42%
=======================================
Files 43 43
Lines 5186 5186
=======================================
Hits 2667 2667
Misses 2323 2323
Partials 196 196
Flags with carried forward coverage won't be shown. Click here to find out more. |
per-PR workflow tested here: https://github.com/netobserv/network-observability-operator/actions/runs/5059748527/jobs/9081725300?pr=343 |
If we agree on that we should make similar changes in the other repos |
hold on ... there's something fishy ; the generated bundle seems wrong, its containerImage is odd |
...to avoid unexpected collisions
next try: #344 |
LGTM |
Hey @msherif1234 - added two more commits:
So I think I'm done with this PR, although (as said on slack) I've found new issues related to multi-arch - but that's not related to this PR, I think it can still be merged |
thanks @jpinsonneau @msherif1234 ! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jotak The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Shortlived images are now built from scratch, not from a base. (Same can be done later for bundle & catalog)
It simplifies the workflows especially with multiarch. We don't need anymore the "ci*" make targets.
Restore the SHA-builds on merged commits
Also make multiarch configurable, and only for merged commits, not per PR