-
Notifications
You must be signed in to change notification settings - Fork 106
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
Added support for checking last_status for a config recorder #750
Conversation
Signed-off-by: Varun Sharma <varun.sharma@progress.com>
Signed-off-by: Varun Sharma <varun.sharma@progress.com>
@varunsharma5 : Please don't add ReadyForDoc & Documentation label till Inspec-cloud-dev doesn't approve it. |
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.
Please check my comments.
Other Comments:
-
Please update the docs link on line no 41 to "https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-config-configurationrecorder.html".
-
Please update the text of line no 41 from "[AWS documentation on Configuration]" to "[AWS documentation on AWS Config Configuration Recorder]".
-
Do we need the plural resource also or singular resource is fine?
Incorporated the review comments Co-authored-by: Soumyodeep Karmakar <63713087+soumyo13@users.noreply.github.com>
Signed-off-by: Varun Sharma <varun.sharma@progress.com>
…s into aws_config_delivery_log Signed-off-by: Varun Sharma <varun.sharma@progress.com>
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.
Added the "Do not merge" label.
Signed-off-by: Varun Sharma <varun.sharma@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
1718707
to
aed7a5b
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Signed-off-by: Varun Sharma varun.sharma@progress.com
Description
Added support for checking last_status for a config recorder
Issues Resolved
#749
Check List
Please fill box or appropriate ([x]) or mark N/A.
rake lint
passes