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
release(executors): update ami naming #62579
Conversation
@@ -25,7 +25,7 @@ export AWS_SECRET_ACCESS_KEY="${AWS_EXECUTOR_AMI_SECRET_KEY}" | |||
|
|||
# Point to GCP boot disk image/AMI built by //cmd/executor/vm-image:ami.build | |||
NAME="${IMAGE_FAMILY}" | |||
if [ "${RELEASE_INTERNAL:-}" != "true" ]; then | |||
if [ "${RELEASE_INTERNAL:-}" == "true" ]; 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.
Need a similar one for docker mirror?
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.
Hmmm does ami.push only get triggered during releases?
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.
I was going to ask that, I assumed it is but not 100% certain.
We can revisit this PR next week when everyone is back btw.
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.
d03bc32
to
1652b48
Compare
* update naming * update docker mirror image family name (cherry picked from commit 894a951)
Context
Test plan
We found out during the 5.4.0 release that the amis weren't named properly. Thus PR fixes that.
Below is the matrix for naming AMIs.
Matrix