Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(instance): use sbs images from marketplace on server create #4466

Merged
merged 9 commits into from
Feb 4, 2025

Conversation

Codelax
Copy link
Member

@Codelax Codelax commented Jan 29, 2025

Newly created server should now be using sbs volumes (ex: scw instance server create type=PLAY2-PICO).

While updating tests, I encountered an issue with scw instance server update, using a SBS volume will create an invalid request.

@Codelax Codelax added enhancement instance Instance issues, bugs and feature requests labels Jan 29, 2025
Some cassettes still need an update, will first fix server update with sbs volumes
@Codelax Codelax force-pushed the feat/instance-default-image-sbs branch from be6d85e to ac92ac5 Compare January 29, 2025 15:18
@Codelax Codelax force-pushed the feat/instance-default-image-sbs branch from d1f981d to b2871e1 Compare January 30, 2025 15:50
@github-actions github-actions bot added the load-balancer Load-balancer issues, bugs and feature requests label Jan 30, 2025
@github-actions github-actions bot added the vpc Virtual Private Cloud (VPC) issues, bugs and feature requests label Jan 30, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.28%. Comparing base (5bf53e0) to head (92028fa).
Report is 466 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (5bf53e0) and HEAD (92028fa). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (5bf53e0) HEAD (92028fa)
2 1
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4466      +/-   ##
==========================================
- Coverage   75.45%   69.28%   -6.17%     
==========================================
  Files         202      300      +98     
  Lines       44323    67775   +23452     
==========================================
+ Hits        33444    46961   +13517     
- Misses       9653    19163    +9510     
- Partials     1226     1651     +425     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Codelax Codelax marked this pull request as ready for review January 30, 2025 17:43
@remyleone remyleone added this pull request to the merge queue Feb 4, 2025
Merged via the queue into scaleway:master with commit 4bcc0e3 Feb 4, 2025
13 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement instance Instance issues, bugs and feature requests load-balancer Load-balancer issues, bugs and feature requests vpc Virtual Private Cloud (VPC) issues, bugs and feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants