-
Notifications
You must be signed in to change notification settings - Fork 2k
feat(aws): add check rds_instance_extended_support #9865
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
base: master
Are you sure you want to change the base?
Conversation
|
✅ Conflict Markers Resolved All conflict markers have been successfully resolved in this pull request. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #9865 +/- ##
==========================================
- Coverage 92.07% 8.67% -83.40%
==========================================
Files 180 832 +652
Lines 24833 23496 -1337
==========================================
- Hits 22864 2039 -20825
- Misses 1969 21457 +19488
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
fd57551 to
c66fc61
Compare
|
The large drop in In particular, the API workflow generates I believe the project coverage regression should be fixed in the CI/Codecov setup rather than in this PR. |
danibarranqueroo
left a comment
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.
Thanks for this great contribution! Please review this requested changes.
| { | ||
| "Provider": "aws", | ||
| "CheckID": "rds_instance_extended_support", | ||
| "CheckTitle": "RDS instances are not enrolled in RDS Extended Support", |
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.
| "CheckTitle": "RDS instances are not enrolled in RDS Extended Support", | |
| "CheckTitle": "RDS instance is not enrolled in RDS Extended Support", |
| "Severity": "medium", | ||
| "ResourceType": "AwsRdsDbInstance", | ||
| "ResourceGroup": "database", | ||
| "Description": "Checks whether Amazon RDS DB instances are enrolled in Amazon RDS Extended Support. If the instance reports `EngineLifecycleSupport` as `open-source-rds-extended-support`, it is enrolled and the check fails. Otherwise, the check passes.", |
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.
| "Description": "Checks whether Amazon RDS DB instances are enrolled in Amazon RDS Extended Support. If the instance reports `EngineLifecycleSupport` as `open-source-rds-extended-support`, it is enrolled and the check fails. Otherwise, the check passes.", | |
| "Description": "**RDS DB instances** are evaluated for enrollment in Amazon RDS Extended Support. The check fails if `EngineLifecycleSupportis` set to `open-source-rds-extended-support`, indicating the instance will incur additional charges after standard support ends.", |
| "ResourceGroup": "database", | ||
| "Description": "Checks whether Amazon RDS DB instances are enrolled in Amazon RDS Extended Support. If the instance reports `EngineLifecycleSupport` as `open-source-rds-extended-support`, it is enrolled and the check fails. Otherwise, the check passes.", | ||
| "Risk": "DB instances enrolled in RDS Extended Support can incur additional charges after the end of standard support for the running database major version. Remaining on older major versions can also delay necessary upgrades, increasing operational and security risk.", | ||
| "RelatedUrl": "https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/extended-support.html", |
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.
| "RelatedUrl": "https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/extended-support.html", | |
| "RelatedUrl": "", |
| }, | ||
| "Recommendation": { | ||
| "Text": "Upgrade enrolled DB instances to an engine version covered under standard support to stop Extended Support charges. For new DB instances and restores created via automation, explicitly set the engine lifecycle support option to avoid unintended enrollment in RDS Extended Support when that is your policy.", | ||
| "Url": "https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/extended-support-charges.html" |
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.
| "Url": "https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/extended-support-charges.html" | |
| "Url": "https://hub.prowler.com/check/rds_instance_extended_support" |
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, move it to the current UNRELEASED version and follow the same style as other new checks in the Added section
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, remove this changes. Don't worry about codecov since it's not a blocker for merging PRs, I'll talk with the team about this to see if there is a problem with the current settings.

Context
This feature request offers a new AWS check
rds_instance_extended_support.It checks whether Amazon RDS DB instances are enrolled in Amazon RDS Extended Support. If the instance reports
EngineLifecycleSupportasopen-source-rds-extended-support, it is enrolled and the check fails. Otherwise, the check passes.Description
DB instances enrolled in RDS Extended Support can incur additional charges after the end of standard support for the running database major version. Remaining on older major versions can also delay necessary upgrades, increasing operational and security risk.
The check is covered by unit-tests.
Checklist
SDK/CLI
UI
API
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.