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

Ato 469/update deploy dev resources #272

Merged
merged 4 commits into from
Jun 19, 2024

Conversation

isaac-GDS
Copy link
Contributor

@isaac-GDS isaac-GDS commented May 10, 2024

What?

  • switched sandpit to dev

  • Add ECR for Dev in the gds-di-development account. Added script to deploy custom named dev-platform resource due to the account already having a dev-platform ECR.

This should be temporary while the dev environment

  • remove sandpit script

@isaac-GDS isaac-GDS requested a review from a team as a code owner May 10, 2024 06:56
@isaac-GDS isaac-GDS marked this pull request as draft May 10, 2024 07:02
@isaac-GDS isaac-GDS marked this pull request as ready for review May 10, 2024 07:53
@isaac-GDS isaac-GDS force-pushed the ATO-469/update-deploy-dev-resources branch from 74f87c6 to b973bd9 Compare May 10, 2024 14:53
billrdunn
billrdunn previously approved these changes Jun 18, 2024
Copy link
Contributor

@billrdunn billrdunn left a comment

Choose a reason for hiding this comment

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

LGTM. I've noticed a couple of things I'd like to tidy up in the infrastructure README - will raise a PR once this one is merged.

 - added new provision_dev script
 - excluded dev specific ECR stack in provision_all script as it will cause error from naming
 - provision_dev script should be a temporary script until we move dev into our own account.
Copy link

sonarcloud bot commented Jun 19, 2024

@isaac-GDS isaac-GDS requested a review from billrdunn June 19, 2024 11:06
@isaac-GDS isaac-GDS added this pull request to the merge queue Jun 19, 2024
Merged via the queue into main with commit 1103ad9 Jun 19, 2024
6 checks passed
@isaac-GDS isaac-GDS deleted the ATO-469/update-deploy-dev-resources branch June 19, 2024 11:32
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.

None yet

4 participants