Skip to content

clean up AWS resources for DB migration#106

Merged
AnJuHyppolite merged 1 commit intodevfrom
clean-up-DB-migration-AWS-resources-#733
Nov 7, 2025
Merged

clean up AWS resources for DB migration#106
AnJuHyppolite merged 1 commit intodevfrom
clean-up-DB-migration-AWS-resources-#733

Conversation

@AnJuHyppolite
Copy link
Copy Markdown
Contributor

@AnJuHyppolite AnJuHyppolite commented Nov 3, 2025 •
edited
Loading

Description

This PR removes the AWS CDK Infrastructure as Code for the serverless Aurora database, including the logic for dynamically selecting the VPC and configuring scaling capacity based on environment.

Ticket

Resolves the following items under Clean up DB migration AWS resources:

  • FeedbackDbStack is deleted in the code
  • FeedbackDbStack is deleted from the Innov-Platform-Dev AWS account
  • Database PRs are documented in Feedback Widget Database Migration Technical Exploration

Approach

  • Deleted code for serverless Aurora CDK database setup
  • Deleted code for dynamic VPC selection and scaling capacity configuration

Steps to Test

N/A

Notes

  • The dev FeedbackDBStack in AWS has been deleted; its status is DELETE_COMPLETE.
  • The PRs that implemented the database work are documented in Feedback Widget Database Migration Technical Exploration under Feedback Widget Database Pull Requests.

ezhangy
ezhangy approved these changes Nov 5, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@ezhangy ezhangy 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!! let me know how following the deployment process goes!

@AnJuHyppolite AnJuHyppolite merged commit 4e3f09f into dev Nov 7, 2025
1 check passed
@AnJuHyppolite AnJuHyppolite deleted the clean-up-DB-migration-AWS-resources-#733 branch November 7, 2025 17:35
@AnJuHyppolite AnJuHyppolite mentioned this pull request Nov 7, 2025
Merged
AnJuHyppolite added a commit that referenced this pull request Nov 7, 2025
@AnJuHyppolite
11-07-2025 Release (Fix API CloudWatch alarms) (#108)
3c6c813 
<!-- Please complete the following sections as necessary. -->

### Description

<!-- Summary of the changes, related issue, relevant motivation, and
context -->
Merges the following PRs to main: 
- #101
- #103
- #104
-  #107
- #106
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ezhangy ezhangy ezhangy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AnJuHyppolite @ezhangy