-
Notifications
You must be signed in to change notification settings - Fork 188
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
awslimitchecker unable to find TrustedAdvisor checks #466
Comments
@xRokco I'll look into this, thanks for reporting it. What region are you running in? Unless you're in China or GovCloud, Trusted Advisor will provide no value whatsoever, as every limit that it supports is also provided by either Service Quotas or the individual service's own API. At this point the Trusted Advisor code really only exists to support users in GovCloud and China until those partitions catch up with Service Quotas (which, for all I know, may have happened already). |
Yeah we're just in regular region (us-east-2, eu-west-1, etc). |
@xRokco The Trusted Advisor checks are still done, but appear to essentially be ignored. Looking at the Supported Limits table in the docs, every limit outside of China and GovCloud (which still have the per-instance-size EC2 limits) that's reported by Trusted Advisor is also reported by either Service Quotas or the service's own API... and limit values from Service Quotas or API take precedence over Trusted Advisor. So yeah, Trusted Advisor is not only not providing any additional data, but the data it does provide ends up getting ignored in normal regions. I'm going to leave this issue open, though, as we should really be either not even attempting TA polling outside of China/Gov, or we should at least get rid of the now-midleading warning messages. |
Hi, we're using regular AWS and China AWS. Trusted Advisor support will be very good for us as Service Quotas isn't available in China and there are some checks that available via Trusted Advisor but not via Service API. |
@jantman I just went through a long rabbit hole with support (Enterprise) and can confirm that this check has been removed in February.
|
Thanks, @jacobisaliveandwell. Apologies for any confusion you encountered with this. I'll do my best to get this cleaned up soon. |
Fixes #466 - Disable TA by default except in China and GovCloud
This will be fixed (well, really, worked around) in the next release, by #505. |
A fix for this issue has been merged to the develop branch and is slated for release in 10.0.0 on Monday, December 7, 2020. Anyone who is able to test the fix is strongly encouraged to do so, and comment on this issue with their results. Instructions for installing awslimitchecker from a git branch are available in the Installing for Development documentation. |
Bug Report
When reporting a bug in awslimitchecker, please provide all of the following information,
as well as any additional details that may be useful in reproducing or fixing
the issue:
Version
Python, 8.0.2
Installation Method
Using pip, with awslimitchecker in a requirement.txt file
pip install -r requirements.txt --target .
Supporting Software Versions
➜ python --version
Python 3.7.5
➜ virtualenv --version
16.7.9
Actual Output
Expected Output
Looking at the code, the _get_limit_check_id() function in trustedadvisor.py seems to poll Trusted Advisor for its set of checks and then loop through it, looking for a check with category "performance" and name "Service Limits", but does not find any.
Doing some research into Trusted Advisor, I can see that there are five separate categories of checks in my account - Cost Optimization, Performance, Security, Fault Tolerance and Service Limits.
However, looking at how Trusted Advisor used to be laid out (at least many years ago) from this AWS blog post it appears that previously "Service Limits" fell under the "Performance" category, as the awslimitchecker code seems to indicate.
Perhaps I'm missing something but I don't seem to be able to find a way to use Trusted Advisor currently.
TrustedAdvisor
Business support
Testing Assistance
Yes
The text was updated successfully, but these errors were encountered: