Skip to content

feat(instance): add tags to private nics#2793

Merged
Codelax merged 6 commits intoscaleway:masterfrom
scaleway-bot:v1.3237.0
Feb 28, 2023
Merged

feat(instance): add tags to private nics#2793
Codelax merged 6 commits intoscaleway:masterfrom
scaleway-bot:v1.3237.0

Conversation

@scaleway-bot
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 22, 2023

Codecov Report

Merging #2793 (f358ddc) into master (064b924) will increase coverage by 0.00%.
The diff coverage is 84.37%.

@@           Coverage Diff           @@
##           master    #2793   +/-   ##
=======================================
  Coverage   75.18%   75.18%           
=======================================
  Files         137      137           
  Lines       27805    27833   +28     
=======================================
+ Hits        20904    20926   +22     
- Misses       6186     6192    +6     
  Partials      715      715           
Impacted Files Coverage Δ
internal/namespaces/instance/v1/instance_cli.go 89.84% <84.37%> (-0.10%) ⬇️
...ernal/namespaces/instance/v1/custom_privatenics.go 81.08% <0.00%> (-2.71%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codelax
Codelax previously approved these changes Feb 24, 2023
scaleway-sdk-go: *instance.ListPrivateNICsResponse does not support pagination
🟥🟥🟥 JSON STDERR 🟥🟥🟥
{
"error": "scaleway-sdk-go: *instance.ListPrivateNICsResponse does not support pagination"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Codelax it seems strange

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll take a look at the sdk

@Codelax Codelax added this pull request to the merge queue Feb 24, 2023
@Codelax Codelax removed this pull request from the merge queue due to a manual request Feb 24, 2023
@Codelax Codelax self-requested a review February 24, 2023 10:17
@Codelax Codelax dismissed their stale review February 24, 2023 10:18

issue with paging

@Codelax Codelax added this pull request to the merge queue Feb 28, 2023
Merged via the queue into scaleway:master with commit aa12c16 Feb 28, 2023
@quantumsheep quantumsheep deleted the v1.3237.0 branch June 8, 2023 13:52
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.

4 participants