-
Notifications
You must be signed in to change notification settings - Fork 2
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
PLT-655 Add workflows to promote images and release app #1397
Conversation
.github/workflows/promote.yml
Outdated
|
||
jobs: | ||
promote: | ||
runs-on: self-hosted |
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.
This can run on ubuntu-latest since it doesn't rely on anything in our internal network.
.github/workflows/release.yml
Outdated
id-token: write | ||
|
||
jobs: | ||
# Promote and Deploy to prod-test. We do not have api job deploy job for prod-test because we only deploy to worker for prod-test |
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.
You can still drop promote-prod-test-api below. And this line can be simplified
# Promote and Deploy to prod-test. We do not have api job deploy job for prod-test because we only deploy to worker for prod-test | |
# Promote and Deploy to prod-test, which only includes worker |
Co-authored-by: Sean Fern <[email protected]>
.github/workflows/promote.yml
Outdated
options: | ||
- api | ||
- worker | ||
#These are the job permissions |
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 comment needed here, just a blank line to visually separate from triggers
.github/workflows/release.yml
Outdated
with: | ||
environment: prod-test | ||
module: api | ||
secrets: inherit |
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.
The promote-prod-test-api job is not needed
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.
Looks good! Thanks for working through all the comments.
🎫 Ticket
https://jira.cms.gov/browse/PLT-655
🛠 Changes
A new workflow to promote images up environments
ℹ️ Context
This is to push out and release our images to all our environments in github actions so we can move away from jenkins.
🧪 Validation
See run at https://github.com/CMSgov/ab2d/actions/runs/11150069080 for successful promote and deploy.