-
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
aws - quotas - include aws default service quotas #7572
aws - quotas - include aws default service quotas #7572
Conversation
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.
lgtm, albeit not super familiar with the quota details.
q['QuotaCode']: q | ||
for q in _get_quotas(client, s, 'list_service_quotas') | ||
} | ||
quotas.update(dquotas) |
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.
@thisisshi
Please reconsider using the other way around. This morning I received some alerts from my accounts, and I thought our business team must be working on a big project. 🤣 It uses the default value rather than the applied quota value. See below for an example. The Quota
is a default one, the applied one is 1000+.
Service Quota - service usage > 80% - [my-awsome-account - ap-southeast-2]
ServiceName: Amazon Elastic Compute Cloud (Amazon EC2)
QuotaName: Running On-Demand Standard (A, C, D, H, I, M, R, T, Z) instances
Quota: 5.0
Usage: 325.0
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.
Here is my version, so far so good.
newscorp-ghfb#127
Need to rerecord the tests, but this policy before would not match but now does with this change: