-
Notifications
You must be signed in to change notification settings - Fork 1
New: [AEA-6028] - Added cleanup scripts for old stacks and proxygen instances #490
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
base: main
Are you sure you want to change the base?
Conversation
|
This PR is linked to a ticket in an NHS Digital JIRA Project. Here's a handy link to the ticket: AEA-6028 |
| export async function deleteUnusedStacks(baseStackName: string, repoName: string, basePath: string): Promise<void> { | ||
| const cloudFormationClient = new CloudFormationClient({}) | ||
| const route53Client = new Route53Client({}) | ||
| const hostedZoneId = await getHostedZoneId(route53Client) |
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.
what if this errors or returns nothing (eg running in the wrong environment)
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.
If it errors this will bubble up to the pipeline, if it returns nothing it will just not cleanup the CNAMES which is probably ok
|


Summary