-
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(elbv2): add new check elbv2_is_in_multiple_az
#4800
feat(elbv2): add new check elbv2_is_in_multiple_az
#4800
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4800 +/- ##
==========================================
+ Coverage 89.08% 89.14% +0.05%
==========================================
Files 923 931 +8
Lines 28274 28504 +230
==========================================
+ Hits 25189 25409 +220
- Misses 3085 3095 +10 ☔ View full report in Codecov by Sentry. |
elbv2_is_in_multiple_az
# AWS ELBv2 Configuration | ||
# aws.elbv2_is_in_multiple_az | ||
# Minimum number of Availability Zones that an ELBv2 must be in | ||
elbv2_min_azs: 2 | ||
|
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, add this to the documentation.
You can check the documentation for this PR here -> Prowler Documentation |
...r/providers/aws/services/elbv2/elbv2_is_in_multiple_az/elbv2_is_in_multiple_az.metadata.json
Outdated
Show resolved
Hide resolved
You can check the documentation for this PR here -> Prowler Documentation |
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.
👏🏼 👏🏼
Context
Checks whether Elastic Load Balancer V2 (Application, Network, or Gateway Load Balancers) is configured with registered instances from at least the specified number of Availability Zones (AZs). The control fails if the load balancer does not span the minimum number of AZs specified.
Description
Checklist
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.