Add EC2 RI Offerings #1742
Closed
Add EC2 RI Offerings #1742
Conversation
This file contains 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
* Added describe ri offering end point * added package to unit test * added offering class convertible unit test * added numpy to setup.py * removed unnecessary print statement * added unit test for empty offering and put csv files in regional folders * added folders for regions * added folders for regions * added csv files to regional folders * add all csv files to regional folders * fixed minor syntax issue * Fixed unit test by adding a variable * Added regional files individually * add back offering table hash * can find the file name from offering id * succesfully returns offering info from id * Split up 20 mb hash table to increase speed * fixed hash function * Add for all regions and instance types * add hash table offering files * fixed hash table functions within moto/models * added ri offering id unit tests * fixed white space2 * fixed unit test for duration int still need to fix underlying issue with instance type files * fixed blank line * Revert "fixed white space2" This reverts commit b9b37dd. * Revert "fixed blank line" This reverts commit 5758e2c. * fixed blank line2 * reset RI csv files to remove non-existent RI's * added more logging * add improved ri info csv tables * add improved csv tables * fixed white space * supports multiple offering ids in input * fixed white space * supports unspecified instance tenancy, offering class, etc * fixed white space * fixed multiple ri unit test * removed unnecessary print statements * fixed instance type in unit test * Added describe ri offering end point * added offering class convertible unit test * added unit test for empty offering and put csv files in regional folders * added folders for regions * added folders for regions * added csv files to regional folders * add all csv files to regional folders * Added regional files individually * add back offering table hash * can find the file name from offering id * succesfully returns offering info from id * Split up 20 mb hash table to increase speed * fixed hash function * Add for all regions and instance types * fixed hash table functions within moto/models * fixed white space2 * fixed unit test for duration int still need to fix underlying issue with instance type files * fixed blank line * Revert "fixed white space2" This reverts commit b9b37dd. * Revert "fixed blank line" This reverts commit 5758e2c. * fixed blank line2 * add improved ri info csv tables * fixed white space * supports multiple offering ids in input * fixed white space * supports unspecified instance tenancy, offering class, etc * fixed white space * fixed multiple ri unit test * removed unnecessary print statements * fixed instance type in unit test * adjusted hash table algorithm to group be region * Added adjustments for hash table by region * added new hash tables by regions * added hash tables by regions * made hash table length smaller * fixed syntax * removed unnecessary function * removed duplicate functions * fixed hash table in ec2/models * added additional unit test to check for offering id in wrong region * fixed typo * fixed region used in unit test * fixed offerings to remove false offering ris * made slight adjustment to unit test * fixed manifest file * removed duplicate unit test * removed duplicate unit tests2 * fixed unit test to count for no instance type in region * removed unnecessary comment
changed it to fix unit test to make it return an invalid value
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.
Fixes #1741