Skip to content
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

POL-1406 AWS Rightsize RDS Instances Fix #2787

Merged
merged 187 commits into from
Oct 29, 2024
Merged

Conversation

XOmniverse
Copy link
Contributor

Description

Fixes issue where policy would fail if a "db.serverless" (or any other instance type not present in the mapping file) instance was found.

Also makes it so that the policy template does not bother to gather metrics for "db.serverless" instances, since we won't be making recommendations for these regardless, as they are not ordinary RDS instances.

Link to Example Applied Policy

https://app.flexera.com/orgs/6/automation/applied-policies/projects/7954?policyId=671f96e992caab6c320f8449

Contribution Check List

  • New functionality includes testing.
  • New functionality has been documented in the README if applicable
  • New functionality has been documented in CHANGELOG.MD

Copy link
Contributor

1 Warning
⚠️

cost/aws/rightsize_rds_instances/aws_rightsize_rds_instances.pt

Policy template updated but associated README.md file has not been. Please verify that any necessary changes have been made to the README.

Generated by 🚫 Danger

@nia-vf1 nia-vf1 self-assigned this Oct 29, 2024
Copy link
Contributor

@nia-vf1 nia-vf1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nia-vf1 nia-vf1 assigned XOmniverse and unassigned nia-vf1 Oct 29, 2024
@XOmniverse XOmniverse merged commit cbb55c5 into master Oct 29, 2024
11 checks passed
@XOmniverse XOmniverse deleted the POL-1406-rightsize-rds branch October 29, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants