Skip to content
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

ci: docker login to right registry when promoting #5899

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

phisco
Copy link
Contributor

@phisco phisco commented Aug 20, 2024

Description of your changes

Promoting a release following the release process errored out complaining about the login failing when trying to push to xpkg.upbound.io, it was missing explicitly setting the registry to log in to, therefore defaulting to dockerhub.

I have:

  • Read and followed Crossplane's contribution process.
  • Run earthly +reviewable to ensure this PR is ready for review.
  • Added or updated unit tests.
  • Added or updated e2e tests.
  • Linked a PR or a docs tracking issue to document this change.
  • Added backport release-x.y labels to auto-backport this PR.

Need help with this checklist? See the cheat sheet.

Signed-off-by: Philippe Scorsolini <p.scorsolini@gmail.com>
@phisco phisco requested a review from a team as a code owner August 20, 2024 10:43
@phisco phisco requested a review from turkenh August 20, 2024 10:43
@phisco phisco merged commit f4247d8 into crossplane:master Aug 20, 2024
17 checks passed
@phisco
Copy link
Contributor Author

phisco commented Aug 20, 2024

/backport

Copy link

Backport failed for release-1.16, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-1.16
git worktree add -d .worktree/backport-5899-to-release-1.16 origin/release-1.16
cd .worktree/backport-5899-to-release-1.16
git switch --create backport-5899-to-release-1.16
git cherry-pick -x 3f906fbfbd9dff991649319629267608871e7c9c

Copy link

Successfully created backport PR for release-1.17:

@jbw976 jbw976 added this to the v1.17 milestone Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants