Skip to content

[Re-land] Update to call List Supported Virtual Machine and Cloud Service SKUs#2456

Merged
cRui861 merged 1 commit intomasterfrom
rechen/return-2400
Mar 1, 2022
Merged

[Re-land] Update to call List Supported Virtual Machine and Cloud Service SKUs#2456
cRui861 merged 1 commit intomasterfrom
rechen/return-2400

Conversation

@cRui861
Copy link
Member

@cRui861 cRui861 commented Mar 1, 2022

Bringing back VM SKU API support.

  • Switch to using Service VM SKUs API from pulling json information from GitHub

  • Fix 'filter' typo

  • Update repo to contain vm size category list and fix unit tests

…2400)

* Switch to using Service VM SKUs API from pulling json information from GitHub

* Fix 'filter' typo

* Update repo to contain vm size category list and fix unit tests
@codecov
Copy link

codecov bot commented Mar 1, 2022

Codecov Report

Merging #2456 (ae88a7d) into master (3fed00a) will decrease coverage by 0.01%.
The diff coverage is 92.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2456      +/-   ##
==========================================
- Coverage   65.00%   64.98%   -0.02%     
==========================================
  Files         914      914              
  Lines       26361    26345      -16     
  Branches     5179     5178       -1     
==========================================
- Hits        17136    17121      -15     
+ Misses       9225     9224       -1     
Impacted Files Coverage Δ
src/app/services/compute/vm-size.service.ts 87.23% <91.30%> (+1.29%) ⬆️
...rc/app/services/compute/vmsize_sample_responses.ts 100.00% <100.00%> (ø)
src/app/services/compute.service.ts 11.86% <0.00%> (-3.39%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3fed00a...ae88a7d. Read the comment docs.

@cRui861 cRui861 merged commit 71c3bd3 into master Mar 1, 2022
@cRui861 cRui861 deleted the rechen/return-2400 branch March 1, 2022 19:29
@cRui861
Copy link
Member Author

cRui861 commented Mar 1, 2022

Fixes #2462

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants