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.
Kami
approved these changes
Mar 2, 2025
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:
|
Member
|
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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)