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

AWS::Config::ConfigurationRecorder.ResourceTypes AllowedValues expansion #1492

Merged
merged 4 commits into from
Apr 28, 2020

Conversation

alexmlee
Copy link
Contributor

@alexmlee alexmlee commented Apr 24, 2020

fixes #1491

Description of changes:
Add new resource types

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov

This comment has been minimized.

Copy link
Contributor

@PatMyron PatMyron left a comment

Choose a reason for hiding this comment

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

@alexmlee alexmlee force-pushed the missing-allowable-types branch from 96925a1 to 15efe4a Compare April 24, 2020 22:37
@alexmlee
Copy link
Contributor Author

Thanks for the contribution! Left a note on the issue if you're interesting in getting these fixes into all regions

I updated the PR to have change the file the autogeneration stems from. Do I need to issue the command to auto-generate the region files and commit those as well?

Thanks!

@PatMyron PatMyron changed the title added resource types AWS::Config::ConfigurationRecorder.ResourceTypes AllowedValues Apr 24, 2020
Comment on lines 8 to 12
"AWS::ApiGateway::DomainName",
"AWS::ApiGateway::Method",
"AWS::ApiGateway::RestApi",
"AWS::ApiGateway::Stage",
"AWS::ApiGatewayV2::DomainName",
Copy link
Contributor

Choose a reason for hiding this comment

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

not seeing these AWS::ApiGateway additions documented here or in the Config console

Do you know if they are recent additions or if there is somewhere else to find them?

Screen Shot 2020-04-24 at 3 57 11 PM

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🤔 Huh! I swear they were in here last week.

"AWS::ApiGatewayV2::Api",
"AWS::ApiGatewayV2::Stage",
"AWS::AutoScaling::AutoScalingGroup",
"AWS::AutoScaling::LaunchConfiguration",
"AWS::AutoScaling::ScalingPolicy",
"AWS::AutoScaling::ScheduledAction",
"AWS::CertificateManager::Certificate",
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you know if this one should be AWS::ACM::Certificate or AWS::CertificateManager::Certificate?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This one has also disappeared in the last week as an option. I think it was replaced by AWS::ACM::Certificate.

@PatMyron
Copy link
Contributor

I updated the PR to have change the file the autogeneration stems from. Do I need to issue the command to auto-generate the region files and commit those as well?

Source file should be enough, we can handle the autogeneration!

@alexmlee alexmlee force-pushed the missing-allowable-types branch from c215e9e to db4e144 Compare April 28, 2020 20:03
@PatMyron
Copy link
Contributor

@alexmlee should we merge these in? Or do you want to keep looking into the other resource types?

@alexmlee
Copy link
Contributor Author

@alexmlee should we merge these in? Or do you want to keep looking into the other resource types?

I think this is ready to go!

@PatMyron PatMyron merged commit fc41b0a into aws-cloudformation:master Apr 28, 2020
@PatMyron
Copy link
Contributor

PatMyron commented Aug 13, 2020

@PatMyron PatMyron mentioned this pull request Aug 13, 2020
73 tasks
@PatMyron PatMyron changed the title AWS::Config::ConfigurationRecorder.ResourceTypes AllowedValues AWS::Config::ConfigurationRecorder.ResourceTypes AllowedValues expansion Sep 5, 2020
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.

Missing AWS::Config::ConfigurationRecorder.ResourceTypes
2 participants