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

awslimitchecker checkID returned none value #441

Closed
naorc12 opened this issue Oct 30, 2019 · 5 comments
Closed

awslimitchecker checkID returned none value #441

naorc12 opened this issue Oct 30, 2019 · 5 comments

Comments

@naorc12
Copy link

naorc12 commented Oct 30, 2019

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
awslimitchecker 4.0.2 / awslimitchecker 7.1.0 (tested on 2 different versions)

Installation Method
pip install awslimitchecker
Not virtualenv

Supporting Software Versions
Python 2.7.6
Description: Ubuntu 14.04.5 LTS

Actual Output
Our script use awslimitchecker, we are getting No data and error message since the last few hours. the problem is that the CheckID value returned is none(this is happened because _get_limit_check_id function that could not find {check['category'] == 'performance' and check['name'] == 'Service Limits'}).
Tried to check it manually to validate, run:
"aws support describe-trusted-advisor-checks --language en"
and i didn not see any checks that under "performance" category and the name is "Service Limits", maybe i misunderstood something here or maybe amazon changed something...

Expected Output
The real limit value.

TrustedAdvisor
Business support.

And here is the output for running "awslimitchecker -r ap-northeast-1 -l -vv":
limitcheckerDebug.log

@naorc12 naorc12 changed the title Bug Report awslimitchecker checkID returned none value Oct 30, 2019
@jantman
Copy link
Owner

jantman commented Oct 30, 2019

@naorc12 Yes, that check likely does not exist for your account anymore. There are a few things going on here at once:

  1. The EC2 service limits are drastically changing over this week and the next week, according to this announcement from AWS. I'm currently working on a fix for this in EC2 vCPU-based On-Demand Instance Limits #432, and hope to release it this week.
  2. It seems that the Trusted Advisor check we've been using, category==performance name=='Service Limits' is going away... it's not present on new accounts and may be disappearing on existing accounts. There is a new entire category for "Service Limits" in Trusted Advisor. However, there is also the new Service Quotas service/API, which is supposed to provide detailed and up-to-the-minute limit information. Issue Retrieve Current Limits from Service Quotas #413 is open for implementing Service Quotas, and discontinuing use of Trusted Advisor where there's overlap. I plan on implementing this as soon as possible, hopefully this week or this weekend.

I'll keep this issue open to fix this particular bug, and will get the fix in the next release.

Until I get that fix out, you can completely skip using Trusted Advisor for this account using the --skip-ta command line option or, if using the Python API, by passing use_ta=False to AwsLimitChecker.check_thresholds().

PS - I see that you're running under Python 2.7.6. According to PEP 373, Python 2.7 is going to be end-of-life / unsupported on January 1, 2020. Accordingly, per #438, this next release will announce the deprecation of support for Python versions below 3.5, and the release after that will stop testing or officially supporting Python versions below 3.5.

jantman added a commit that referenced this issue Oct 31, 2019
fixes #441 - unhandled exception during TA polling
@jantman
Copy link
Owner

jantman commented Oct 31, 2019

A fix for this has been merged to the develop branch if you'd like to test. It will be included in the next release.

@jantman
Copy link
Owner

jantman commented Oct 31, 2019

And since apparently I was too busy earlier today to remember to be a good person... thank you so much for reporting this, and apologies for the issues!

@naorc12
Copy link
Author

naorc12 commented Oct 31, 2019

@jantman Thanks, I'll check it out and of course waiting for the new release

@jantman
Copy link
Owner

jantman commented Nov 4, 2019

This has been fixed in 8.0.0 which is now live on PyPI and Docker Hub. Apologies for any delays in getting this released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants