-
Notifications
You must be signed in to change notification settings - Fork 923
Add AWS region me-central-1 #2030
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
Conversation
|
Is this PR ready for review or someone can investigate the problem? I also tried running the tox, scrape-ec2-sizes.py downloads a 6GB ec.json file, which looks okay. But this script prints this error. thanks. |
Also use larger ijson buffer for faster parsing.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## trunk #2030 +/- ##
=======================================
Coverage 83.40% 83.40%
=======================================
Files 353 353
Lines 81685 81685
Branches 8632 8632
=======================================
Hits 68124 68124
Misses 10738 10738
Partials 2823 2823
🚀 New features to boost your workflow:
|
|
Sorry for the delay. The PR (with somewhat related updates to AWS EC2 sizes and prices) has been merged into trunk. Thanks for the contribution. |
Add AWS region me-central-1
Description
We need support for opt in AWS region
me-central-1.Resources:
endpoint: https://docs.aws.amazon.com/general/latest/gr/ec2-service.htmlcountryandsignature_version: https://docs.aws.amazon.com/general/latest/gr/s3.htmlStatus
We tried to follow the documentation (see [1]) to update theses files:
contrib/scrape-ec2-prices.pycontrib/scrape-ec2-sizes.pySadly when running
tox -e scrape-ec2-sizes,scrape-ec2-prices,scrape-ec2-sizesfailed:Afterwards we retried with trunk and its also failing:
Also the mentioned example (see [2]) in the documentation is outdated. The list
EC2_REGIONSdoes not exist anymore incontrib/scrape-ec2-prices.py.The change we did seems to be sufficient to delete a VM, are there other tests to run?
[1] https://libcloud.readthedocs.io/en/latest/development.html#updating-ec2-sizing-and-supported-regions-data
[2] 762f0e5
Checklist (tick everything that applies)