[UX] Fix GCP show-gpus order after combining VM types #3083
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.
The current price order for GCP in
sky show-gpus
is incorrect, because:It also fixes #2953.
Tested (run the relevant ones):
bash format.sh
sky show-gpus a100
sky show-gpus a100 --all-regions
sky show-gpus
sky show-gpus v100:1 --all-regions
Before this PR
Note: for example,
asia-east1
has different hourly price, because before this PR we use the same VM price for all different regions which is not correct.pytest tests/test_smoke.py
pytest tests/test_smoke.py::test_fill_in_the_name
bash tests/backward_comaptibility_tests.sh