cloud-provider-vsphere

cloud-provider-vsphere copied to clipboard

Clean up and refactor the logic for CCM entrypoint.

4

**What this PR does / why we need it**: Clean up the entrypoint logic for the `vsphere-cloud-controller-manager` command. **Which issue this PR fixes** *(optional, in `fixes #(, fixes #, ...)`...

HanFa

vSphere CPI should be configurable with a server in the format of an IPv6 address

2

### What happened? When attempting to create a Kubernetes Secret in the format the vSphere CPI expects, with an IPv6 address, causes Kubernetes to error. The vSphere CPI expects the...

tylerschultz

Node initialization fails with multiple IPs and node IP is not first

4

### What happened? Deployed vSphere CPI into a new cluster, when it goes to initialize the node, it is seemingly failing because it only gets the first IP in vSphere...

bnason

Replace cloud-provider-vsphere staticcheck with golangci-lint

2

### Which jobs are failing? cloud-provider-vsphere ### Which tests are failing? pull-cloud-provider-vsphere-verify-staticcheck ### Since when has it been failing? After base image was bumped ### Testgrid link https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/cloud-provider-vsphere/607/pull-cloud-provider-vsphere-verify-staticcheck/1506423298420903936 ### Reason...

lubronzhan

Bump govmomi to 0.27.3

2

### What would you like to be added? Bump the govmomi version ### Why is this needed? Our GOVC is too old. We should pick up the recent one. Also...

lubronzhan

chart: use of plaintext password in config map leaks credentials

4

https://github.com/kubernetes/cloud-provider-vsphere/blob/ce447aa6b3d277a0fec371b1a648a26b3f3e7d63/charts/vsphere-cpi/templates/configmap.yaml#L26-L27 why don't these reference a secret like this: ```yaml secretName: vsphere-cloud-secret secretNamespace: kube-system ```

addyess

Helm install/upgrade fails

1

### What happened? Attempted to install the latest version of the CPI Helm chart and received the following error from Helm: `Error: UPGRADE FAILED: template: vsphere-cpi/templates/service.yaml:1:14: executing \"vsphere-cpi/templates/service.yaml\" at :...

mstrent

https://github.com/kubernetes/cloud-provider-vsphere/blob/1650759df957b8d22d4f2fa32fc331e250163fe5/charts/vsphere-cpi/templates/daemonset.yaml#L43 The nodeSelector in the provided vSphere CPI Helm chart will result in pods never scheduling on RKE2 clusters. Instead, the nodeSelector should be: `node-role.kubernetes.io/control-plane: "true"` Adding `--set-string daemonset.nodeSelector."node-role\.kubernetes\.io/control-plane"=true` to...

mstrent

Document and validate the least needed vSphere permission

1

### What would you like to be added? We should have documented the least needed permission. Also verify that CPV still works as expected. In theory, it needs permission described...

lubronzhan

Document a support policy/commitment for Kubernetes versions

20

**Is this a BUG REPORT or FEATURE REQUEST?**: > Uncomment only one, leave it on its own line: > /kind bug > /kind feature **What happened**: The documentation does not...

ibauersachs