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

PLT-655 Add workflows to promote images and release app #1397

Merged
merged 38 commits into from
Oct 4, 2024
Merged

Conversation

oluwolenpbc
Copy link
Contributor

@oluwolenpbc oluwolenpbc commented Sep 24, 2024

🎫 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.

@oluwolenpbc oluwolenpbc requested a review from a team as a code owner September 24, 2024 03:31
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Show resolved Hide resolved

jobs:
promote:
runs-on: self-hosted
Copy link
Member

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.

@oluwolenpbc oluwolenpbc requested a review from gsf September 26, 2024 15:29
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@oluwolenpbc oluwolenpbc requested a review from gsf October 3, 2024 17:34
@gsf gsf changed the title adding workflow to promote images PLT-655 Add workflows to promote images and release app Oct 3, 2024
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Show resolved Hide resolved
.github/workflows/promote.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/promote.yml Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@oluwolenpbc oluwolenpbc requested a review from gsf October 3, 2024 18:30
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
Copy link
Member

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

Suggested change
# 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

options:
- api
- worker
#These are the job permissions
Copy link
Member

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

with:
environment: prod-test
module: api
secrets: inherit
Copy link
Member

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

Copy link
Member

@gsf gsf left a 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.

@oluwolenpbc oluwolenpbc merged commit 7da005a into main Oct 4, 2024
8 checks passed
@oluwolenpbc oluwolenpbc deleted the PLT-655 branch October 4, 2024 07:16
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.

2 participants