Merge pull request #600 from andrewsykim/default-insecure

default insecure: true in the cloud provider since CAPV doesn't manage vCenter certificates yet

Author: k8s-ci-robot

examples/default/cluster/cluster.yaml

@@ -26,6 +26,7 @@ spec:
     global:
       secretName: "cloud-provider-vsphere-credentials"
       secretNamespace: "kube-system"
+      insecure: true
     virtualCenter:
       "${VSPHERE_SERVER}":
         datacenters: "${VSPHERE_DATACENTER}"

pkg/cloud/vsphere/services/cloudprovider/csi.go

@@ -565,7 +565,7 @@ func ConfigForCSI(ctx *context.ClusterContext) *cloudprovider.Config {
 	config := &cloudprovider.Config{}
 
 	config.Global.ClusterID = fmt.Sprintf("%s/%s", ctx.Cluster.Namespace, ctx.Cluster.Name)
-	config.Global.Insecure = false
+	config.Global.Insecure = ctx.VSphereCluster.Spec.CloudProviderConfiguration.Global.Insecure
 	config.Network.Name = ctx.VSphereCluster.Spec.CloudProviderConfiguration.Network.Name
 
 	config.VCenter = map[string]cloudprovider.VCenterConfig{}