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

CHI-1260: Deploy all prod workflow #503

Merged
merged 27 commits into from
Aug 29, 2023
Merged

Conversation

stephenhand
Copy link
Collaborator

@stephenhand stephenhand commented Aug 28, 2023

Description

  • Add a generic 'deploy multiple helplines for multiple environments workflow'
  • Refactor 'Deploy All Staging Accounts' workflow to use above workflow
  • Add a 'Deploy All Production Accounts' workflow using the same workflow
  • Updated deploy workflow to only need the environment specifying once
  • Updated single helpline deploy to use multiple deploy as well

Checklist

  • Corresponding issue has been opened

Related Issues

CHI-1260

Verification steps

  • Test the new 'deploy all prod' workflow & regression test the 'deploy all staging' and single helpline deploy workflows (particularly dev)

…at uses the generic one. Removed need to specify environment twice in a custom workflow
@stephenhand
Copy link
Collaborator Author

@robert-bo-davis @GPaoloni FYI

@stephenhand stephenhand marked this pull request as ready for review August 29, 2023 12:36
Copy link
Collaborator

@murilovmachado murilovmachado 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 to me. I'm going to borrow the same idea for the flex-plugins repo

Comment on lines +1 to +3
[
"AS"
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

You have only AS here for testing, but eventually, we'll list them all, correct?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, sorry, should have explained in the PR it's the old chicken & egg problem, shouldn't merge the PR before it's tested but can't test the new workflow without it being merged

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, sorry should have noted that on the PR

The old 'chicken & egg' problem that we shouldn't merge before we test but the new workflow can't be tested before it's merged

@stephenhand stephenhand merged commit 4c37341 into master Aug 29, 2023
8 checks passed
@stephenhand stephenhand deleted the CHI-1260-deploy_all_prod_wf branch August 29, 2023 13:13
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