-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(elasticbeanstalk): add new check elasticbeanstalk_enhanced_health_reporting_enabled
#5348
feat(elasticbeanstalk): add new check elasticbeanstalk_enhanced_health_reporting_enabled
#5348
Conversation
… attributes and fix test to adjust to the changes
…d list_tags_for_resource
elasticbeanstalk_enhanced_health_reporting_enabled
…should-have-enhanced-health-reporting-enabled
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5348 +/- ##
=======================================
Coverage 89.36% 89.36%
=======================================
Files 1022 1023 +1
Lines 31382 31434 +52
=======================================
+ Hits 28044 28091 +47
- Misses 3338 3343 +5 ☔ View full report in Codecov by Sentry. |
if ( | ||
environment.health_reporting is None | ||
or environment.health_reporting != "enhanced" | ||
): |
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 ( | |
environment.health_reporting is None | |
or environment.health_reporting != "enhanced" | |
): | |
if environment.health_reporting != "enhanced": |
@@ -0,0 +1,34 @@ | |||
{ | |||
"Provider": "aws", | |||
"CheckID": "elasticbeanstalk_enhanced_health_reporting_enabled", |
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.
"CheckID": "elasticbeanstalk_enhanced_health_reporting_enabled", | |
"CheckID": "elasticbeanstalk_environment_enhanced_health_reporting", |
Context
Add a new check to make sure enhanced health reporting is enabled for AWS Elastic Beanstalk environments. The control fails if enhanced health reporting is not enabled.
Enhanced health reporting provides a more detailed view of the health of the underlying infrastructure, allowing for quicker responses to changes that could affect application availability. It helps gauge the severity of issues and pinpoint possible causes by evaluating logs and metrics from environment EC2 instances.
Description
Added a new check
elasticbeanstalk_enhanced_health_reporting_enabled
with respective unit tests and metadata.Checklist
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.