-
Notifications
You must be signed in to change notification settings - Fork 4
NRL-1595 account wide infra deploy pipeline #1098
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
Conversation
|
🚀 PR environment successfully deployed. |
|
🚀 PR environment successfully deployed. |
b218e85 to
21014f9
Compare
|
🚀 PR environment successfully deployed. |
- NRL-1595 Minor readme amendments - NRL-1595 Rough deploy account wide infra workflow - NRL-1595 Create script to pull truststore certs for all environments in a given account - NRL-1595 pre-feedback pipeline. Different loop attempts - NRL-1595 made changes based on feedback and account-specific envs approach. Ready for test runs once workflow envs created - NRL-1595 pull account id from env secrets rather than variables - NRL-1595 set trigger to on push to perform first workflow run - NRL-1595 push only on this branch - NRL-1595 revert trigger now that we have initial run to manual - NRL-1595 Fix exit codes - NRL-1595 Hardcode env choice and branch name for now - NRL-1595 Attempt to resolve bad account id retrieval - NRL-1595 Fix tfplan output - NRL-1595 check deployed version written to state correctly - NRL-1595 fix bad apply - NRL-1595 tidy up + remove hardcoded references. This commit's run will fail because no manual inputs. Will need a test running manually once merged into develop but hopefully ready for PR - NRL-1595 fix bad apply - NRL-1595 tidy up + remove hardcoded references. This commit's run will fail because no manual inputs. Will need a test running manually once merged into develop but hopefully ready for PR - NRL-1595 attempt to fix bad apply command - NRL-1595 Pull target branch and environment from inputs - needs testing once merged - NRL-1595 Bot suggestions + amendments from self-review
ffebbaf to
c99e926
Compare
|
🚀 PR environment successfully deployed. |
sandyforresternhs
left a comment
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 great! Couple of q's / comments :)
… to all environments - review feedback.
…count-wide-infra-deploy
|
|
🚀 PR environment successfully deployed. |
|
🚀 PR environment successfully deployed. |



Created a github workflow that deployed account-wide infrastructure based on the account-wide deployment readme