Conversation
Move teh Configure AWS Credentials step to before the Run pre-build steps in the ecr-multi-arch-deploy_* scripts
|
@ehanson8 I tagged you as this PR should fix something you're waiting on in the alma-patronload repo. Christopher said we should add a (temporary) commit to that MITLibraries/alma-patronload#158 to update the dev-build.yml workflow to reference this branch (IN-1786) instead of the main branch. When that commit gets pushed to GitHub, it should automatically run the new workflow since it's part of an open PR to the main branch. Do you prefer I make this (temporary) change or would you like to? |
I can take care of this and I'll reach out if anything goes haywire, thanks for your work on this! |
ehanson8
left a comment
There was a problem hiding this comment.
Spoke too soon, CI passed but dev-build didn't run anything https://github.com/MITLibraries/alma-patronload/actions/runs/33549429731
@ehanson8 this is the error message for the failed dev-build run: I think that the issue is that the name of the feature branch here is |
Absolutely my bad, apologies! All good now |
Move the Configure AWS Credentials step to before the Run pre-build steps in the ecr-multi-arch-deploy_* scripts
Why these changes are being introduced
A bug was discovered in alma-patronload that prevented access to shared files in the pre-build sequence.
How this addresses that need
Moved the
Configure AWS credentialsstep to before theRun pre-build stepssection inSide effects of this change
No other changes
Relevant ticket(s)
IN-1786