Skip to content

feat(k8s): create PN automatically with cluster if none given #3152

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

Merged
merged 4 commits into from
Jun 14, 2023

Conversation

Mia-Cross
Copy link
Contributor

This PR adds a custom clusterCreate function that creates a Private Network with the cluster if no private_network_id is given.

@remyleone remyleone added the k8s Kubernetes Kapsule issues, bugs and feature requests label Jun 7, 2023
@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch from bb37714 to 58d29dd Compare June 8, 2023 14:50
@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2023

Codecov Report

Merging #3152 (ca0b265) into master (ff8cc42) will decrease coverage by 0.03%.
The diff coverage is 73.49%.

@@            Coverage Diff             @@
##           master    #3152      +/-   ##
==========================================
- Coverage   75.61%   75.58%   -0.03%     
==========================================
  Files         177      177              
  Lines       38287    38367      +80     
==========================================
+ Hits        28951    29001      +50     
- Misses       8272     8299      +27     
- Partials     1064     1067       +3     
Impacted Files Coverage Δ
internal/namespaces/k8s/v1/custom_cluster.go 79.57% <73.49%> (-2.78%) ⬇️

... and 1 file with indirect coverage changes

@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch from c223f96 to 08527a1 Compare June 8, 2023 15:55
@Mia-Cross Mia-Cross marked this pull request as ready for review June 8, 2023 16:38
@jtherin
Copy link
Member

jtherin commented Jun 8, 2023

Beware, that will break kosmos!

remyleone
remyleone previously approved these changes Jun 9, 2023
@remyleone remyleone dismissed their stale review June 9, 2023 16:14

kosmos

@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch from 08527a1 to 0c6ca1c Compare June 13, 2023 09:57
@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch 4 times, most recently from eb80b8b to 6a35975 Compare June 14, 2023 09:59
@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch from 6a35975 to 5b2af5f Compare June 14, 2023 12:28
@Mia-Cross Mia-Cross force-pushed the k8s_pn_id_required branch from 3b4bdec to f37aa1f Compare June 14, 2023 13:07
@remyleone remyleone added this pull request to the merge queue Jun 14, 2023
Merged via the queue into scaleway:master with commit ae816d9 Jun 14, 2023
@Mia-Cross Mia-Cross deleted the k8s_pn_id_required branch June 14, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
k8s Kubernetes Kapsule issues, bugs and feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants