Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat(k8s): integrate public_ip_disabled field #2157

Merged
merged 3 commits into from
Oct 16, 2023

Conversation

Mia-Cross
Copy link
Contributor

The k8s API just got a new field public_ip_disabled which allows users to choose if they want their pools to be given a public IP or not.
It should be integrated to the scaleway_k8s_pool resource.

@Mia-Cross Mia-Cross force-pushed the k8s_public_ip_disabled branch from 890cace to b3c8f7e Compare October 13, 2023 17:04
@Mia-Cross Mia-Cross force-pushed the k8s_public_ip_disabled branch from b3c8f7e to 6e2cfd1 Compare October 13, 2023 17:26
@Mia-Cross Mia-Cross marked this pull request as ready for review October 13, 2023 17:27
@codecov-commenter
Copy link

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (6bd7d5d) 73.24% compared to head (6e2cfd1) 73.25%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2157   +/-   ##
=======================================
  Coverage   73.24%   73.25%           
=======================================
  Files         203      203           
  Lines       32146    32155    +9     
=======================================
+ Hits        23546    23555    +9     
  Misses       6573     6573           
  Partials     2027     2027           
Files Coverage Δ
scaleway/resource_k8s_pool.go 83.52% <100.00%> (+0.35%) ⬆️

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

@remyleone remyleone added this pull request to the merge queue Oct 16, 2023
Merged via the queue into scaleway:master with commit 618ebe7 Oct 16, 2023
33 checks passed
@Mia-Cross Mia-Cross deleted the k8s_public_ip_disabled branch October 16, 2023 09: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.

3 participants