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

Fix #85 and #154 #158

Merged
merged 11 commits into from
Mar 12, 2016
Merged

Fix #85 and #154 #158

merged 11 commits into from
Mar 12, 2016

Conversation

jantman
Copy link
Owner

@jantman jantman commented Mar 12, 2016

  • add support for RDS 'DB Clusters' and 'DB Cluster Parameter Groups' limits
  • use API to retrieve RDS limits
  • switch RDS from calculating usage to using the DescribeAccountAttributes usage information, for all limits other than those which are per-resource and need resource IDs (Max auths per security group, Read replicas per master, Subnets per Subnet Group)

jantman added 9 commits March 11, 2016 20:59
…instances, but was actually counting the number of reservations not instances
…escribeAccountAttributes usage information, for all limits other than those which are per-resource and need resource IDs (Max auths per security group, Read replicas per master, Subnets per Subnet Group)
@codecov-io
Copy link

Current coverage is 100.00%

Merging #158 into master will not affect coverage as of 3f5093d

@@            master    #158   diff @@
======================================
  Files           17      17       
  Stmts         1391    1355    -36
  Branches       228     216    -12
  Methods          0       0       
======================================
- Hit           1391    1355    -36
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of 3f5093d

Powered by Codecov. Updated on successful CI builds.

jantman added a commit that referenced this pull request Mar 12, 2016
@jantman jantman merged commit 89db702 into master Mar 12, 2016
@jantman jantman deleted the issues/85 branch March 12, 2016 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants