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

Gen2 migrations fixes #14045

Merged
merged 7 commits into from
Dec 18, 2024
Merged

Gen2 migrations fixes #14045

merged 7 commits into from
Dec 18, 2024

Conversation

abhi7cr
Copy link
Contributor

@abhi7cr abhi7cr commented Dec 18, 2024

Description of changes

  • Fix Userpool client provider casing (needs to be uppercase) in CDK
  • Fix Data table mapping output key name
  • Check for absence of auth in gen1 definition fetcher
  • Add comment for RemovalPolicy statements to force a CI/CD or sandbox deploy post Refactor by instructing customers to remove the comment.

Description of how you validated changes

yarn test, manual testing of running gen2 codegen

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)
  • New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
  • Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@abhi7cr abhi7cr requested a review from a team as a code owner December 18, 2024 19:57
@abhi7cr abhi7cr changed the title Gen2 migrations execute Gen2 migrations fixes Dec 18, 2024
@abhi7cr abhi7cr force-pushed the gen2-migrations-execute branch from 1e3c57b to b1ca1b1 Compare December 18, 2024 21:38
@@ -179,13 +180,17 @@ const unsupportedCategories = (): Map<string, string> => {
Object.keys(apiList).forEach((api) => {
const apiObj = apiList[api];
if (apiObj.service == 'API Gateway') {
unsupportedCategoriesList.set('rest api', unsupportedCategories.get('rest api')!);
const restAPIDocsLink = unsupportedCategories.get(restAPIKey);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

part of lint fix

}
});
}
} else {
if (unsupportedCategories.has(category) && Object.entries(meta[category]).length > 0) {
unsupportedCategoriesList.set(category, unsupportedCategories.get(category)!);
const unsupportedCategoryDocLink = unsupportedCategories.get(category);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

part of lint fix

@abhi7cr abhi7cr requested a review from aws-aemilia December 18, 2024 23:24
@abhi7cr abhi7cr merged commit 990a0a3 into migrations Dec 18, 2024
5 checks passed
@abhi7cr abhi7cr deleted the gen2-migrations-execute branch December 18, 2024 23:26
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.

3 participants