-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat(spaces-artifacts): add option to push in parallel image to spacess-artifacts #132
Conversation
…s-artifacts Signed-off-by: Christopher Haar <[email protected]>
@turkenh think we need to backport this in the other active release branches too - wdyt ? |
@@ -51,7 +51,7 @@ KIND_VERSION = v0.21.0 | |||
# Due to the way that the shared build logic works, images should | |||
# all be in folders at the same level (no additional levels of nesting). | |||
|
|||
REGISTRY_ORGS ?= docker.io/upbound xpkg.upbound.io/upbound | |||
REGISTRY_ORGS ?= docker.io/upbound xpkg.upbound.io/upbound xpkg.upbound.io/spaces-artifacts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I think we will need to translate this to earthly way 🤔 Let me check what it means.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same for upbound/universal-crossplane - or what we want todo here ? - i will wait to open the PR then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I think we should stick to the build submodule there.
Git push to origin failed for release-1.15 with exitcode 1 |
Git push to origin failed for release-1.16 with exitcode 1 |
Signed-off-by: Hasan Turken <[email protected]>
Description of your changes
push images to xpkg.upbound.io/spaces-artifacts in parallel
we need to add the new ENV Variables:
Fixes #
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR.Need help with this checklist? See the cheat sheet.