Skip to content

Frontend/prod deploy pipeline #591

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Feb 25, 2025
Merged

Frontend/prod deploy pipeline #591

merged 7 commits into from
Feb 25, 2025

Conversation

jsandoval81
Copy link
Collaborator

@jsandoval81 jsandoval81 commented Feb 21, 2025 •
edited
Loading

Requirements List

  • Environment variable coordination

Description List

  • Added frontend deployment pipeline config for Prod

Testing List

  • Review pipeline file for any obvious misconfigs

Notes

After merging to CSG development:

  • If not already present, create branch in CSG repo: frontend/prod-deploy-pipeline (@jlkravitz)
  • Make insignificant frontend code change on CSG branch frontend/prod-deploy-pipeline (@jlkravitz)
    • Should trigger the workflow in CSG repo
  • Evaluate workflow run for any errors (@jlkravitz)
  • Once successfully deployed, manually smoke test the deployed Prod UI (@jsandoval81)
  • After confirming Prod UI, open small PR to remove trigger branch frontend/prod-deploy-pipeline from workflow (@jsandoval81)

ChiefStief
ChiefStief reviewed Feb 25, 2025
View reviewed changes
ChiefStief
ChiefStief reviewed Feb 25, 2025
View reviewed changes
@jsandoval81
Copy link
Collaborator Author

@jlkravitz This is ready for your review / merge / testing. I've outlined the steps in the Notes above. If any of that is unclear please reach out on slack. Thanks!

jlkravitz
jlkravitz previously approved these changes Feb 25, 2025
View reviewed changes
Copy link
Collaborator

@jlkravitz jlkravitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one typo but otherwise looks good!

jlkravitz
jlkravitz approved these changes Feb 25, 2025
View reviewed changes
Copy link
Collaborator

@jlkravitz jlkravitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isabeleliassen Good to merge. Once you do so, I'll test per the PR description.

@isabeleliassen isabeleliassen merged commit de6100f into csg-org:development Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlkravitz jlkravitz jlkravitz approved these changes

@ChiefStief ChiefStief ChiefStief approved these changes

@isabeleliassen isabeleliassen isabeleliassen approved these changes

@carlsims carlsims Awaiting requested review from carlsims

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jsandoval81 @jlkravitz @ChiefStief @isabeleliassen