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

Fix promote-images-prod after splitting it out #10292

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

jcgruenhage
Copy link
Member

Problem

promote-images was split into promote-images-dev and promote-images-prod in #10267.

dev credentials were loaded in promote-images-dev and prod credentials were loaded in promote-images-prod, but promote-images-prod needs dev credentials as well to access the dev images to replicate them from dev to prod.

Summary of changes

Load dev credentials in promote-images-prod as well.

@jcgruenhage jcgruenhage requested a review from a team as a code owner January 7, 2025 12:13
.github/workflows/build_and_test.yml Outdated Show resolved Hide resolved
@jcgruenhage jcgruenhage force-pushed the jcgruenhage/fix-promote-images-prod branch from 43a6703 to 5ed9d87 Compare January 7, 2025 12:53
@jcgruenhage jcgruenhage enabled auto-merge January 7, 2025 13:13
@jcgruenhage jcgruenhage added this pull request to the merge queue Jan 7, 2025
Merged via the queue into main with commit 4aa9786 Jan 7, 2025
82 checks passed
@jcgruenhage jcgruenhage deleted the jcgruenhage/fix-promote-images-prod branch January 7, 2025 13:46
Copy link

github-actions bot commented Jan 7, 2025

7227 tests run: 6875 passed, 0 failed, 352 skipped (full report)


Code coverage* (full report)

  • functions: 31.2% (8412 of 26962 functions)
  • lines: 48.0% (66788 of 139234 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
5ed9d87 at 2025-01-07T13:53:17.619Z :recycle:

bayandin pushed a commit that referenced this pull request Jan 7, 2025
## Problem
`promote-images` was split into `promote-images-dev` and
`promote-images-prod` in
#10267.

`dev` credentials were loaded in `promote-images-dev` and `prod`
credentials were loaded in `promote-images-prod`, but
`promote-images-prod` needs `dev` credentials as well to access the
`dev` images to replicate them from `dev` to `prod`.

## Summary of changes
Load `dev` credentials in `promote-images-prod` as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants