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
Feature/9 ec2 findings #191
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #191 +/- ##
===========================================
- Coverage 29.43% 29.42% -0.02%
===========================================
Files 69 69
Lines 4546 4551 +5
===========================================
+ Hits 1338 1339 +1
- Misses 3208 3212 +4
Continue to review full report at Codecov.
|
@@ -115,6 +116,10 @@ def _add_security_group_name_to_ec2_grants(self): | |||
self.add_security_group_name_to_ec2_grants_callback, | |||
{'AWSAccountId': self.aws_account_id}) | |||
|
|||
def _check_ec2_zone_distribution(self): |
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.
👏 👏 👏
This is part of a proprietary feature (#9). It adds a check for the number of availability zones having one or more EC2 instances deployed and the monitoring state.
Please also review the proprietary PR.