Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fixing the issue with RDS storage # #201

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

qburst-praven
Copy link

What type of PR is this?
Bug

What this PR does / why we need it:
At present if you run the docker compose it breaks for RDS with the error

"Could not get rds storage price" error="Price list response should be equal only to 1 product" storage_filters="{\n Filters: [\n {\n Field: "volumeType",\n Type: "TERM_MATCH",\n Value: "General Purpose"\n },\n {\n Field: "productFamily",\n Type: "TERM_MATCH",\n Value: "Database Storage"\n },\n {\n Field: "termType",\n Type: "TERM_MATCH",\n Value: "OnDemand"\n },\n {\n Field: "deploymentOption",\n Type: "TERM_MATCH",\n Value: "Single-AZ"\n }\n ],\n ServiceCode: "AmazonRDS"\n}"

This is because the current filter does not add databaseEngine. This PR adds the same and fixes the issue.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant