Drop kubevirt infraClusterKubeconfig API fields as it's not neeeded (covered by the credentials) (#3675)

Signed-off-by: Artiom Diomin <[email protected]>
Co-authored-by: Artiom Diomin <[email protected]>

Author: kubermatic-bot

docs/api_reference/v1beta2.en.md

@@ -1,6 +1,6 @@
 +++
 title = "v1beta2 API Reference"
-date = 2025-06-11T12:48:35+02:00
+date = 2025-06-12T15:55:08+03:00
 weight = 11
 +++
 ## v1beta2
@@ -572,7 +572,6 @@ KubevirtSpec defines the Kubevirt provider
 | Field | Description | Scheme | Required |
 | ----- | ----------- | ------ | -------- |
 | infraNamespace | InfraNamespace is the namespace that KubeVirt provider will use to create and manage resources in the infra cluster, such as VirtualMachines, VirtualMachineInstances, etc... | string | true |
-| infraClusterKubeconfig | InfraClusterKubeconfig is a base64-encoded kubeconfig file that points to a KubeVirt infra cluster (a cluster where KubeVirt is installed). This Kubeconfig will be used to create and manage KubeVirt specific resources such as DataVolumes. | string | true |
 | zoneAndRegionEnabled | ZoneAndRegionEnabled indicates if need to get Region and zone labels from the cloud provider | bool | false |
 | loadBalancerEnabled | LoadBalancerEnabled indicates if the ccm should create and manage the clusters load balancers. | bool | false |
 

docs/api_reference/v1beta3.en.md

@@ -1,6 +1,6 @@
 +++
 title = "v1beta3 API Reference"
-date = 2025-06-11T12:48:35+02:00
+date = 2025-06-12T15:55:08+03:00
 weight = 11
 +++
 ## v1beta3
@@ -574,7 +574,6 @@ KubevirtSpec defines the Kubevirt provider
 | Field | Description | Scheme | Required |
 | ----- | ----------- | ------ | -------- |
 | infraNamespace | InfraNamespace is the namespace that KubeVirt provider will use to create and manage resources in the infra cluster, such as VirtualMachines, VirtualMachineInstances, etc... | string | true |
-| infraClusterKubeconfig | InfraClusterKubeconfig is a base64-encoded kubeconfig file that points to a KubeVirt infra cluster (a cluster where KubeVirt is installed). This Kubeconfig will be used to create and manage KubeVirt specific resources such as DataVolumes. | string | true |
 | zoneAndRegionEnabled | ZoneAndRegionEnabled indicates if need to get Region and zone labels from the cloud provider | bool | false |
 | loadBalancerEnabled | LoadBalancerEnabled indicates if the ccm should create and manage the clusters load balancers. | bool | false |
 

pkg/apis/kubeone/types.go

@@ -446,9 +446,6 @@ type KubevirtSpec struct {
 	// InfraNamespace is the namespace that KubeVirt provider will use to create and manage resources in the infra cluster,
 	// such as VirtualMachines, VirtualMachineInstances, etc...
 	InfraNamespace string `json:"infraNamespace"`
-	// InfraClusterKubeconfig is a base64-encoded kubeconfig file that points to a KubeVirt infra cluster (a cluster where KubeVirt is installed).
-	// This Kubeconfig will be used to create and manage KubeVirt specific resources such as DataVolumes.
-	InfraClusterKubeconfig string `json:"infraClusterKubeconfig"`
 	// ZoneAndRegionEnabled indicates if need to get Region and zone labels from the cloud provider
 	ZoneAndRegionEnabled bool `json:"zoneAndRegionEnabled,omitempty"`
 	// LoadBalancerEnabled indicates if the ccm should create and manage the clusters load balancers.

pkg/apis/kubeone/v1beta2/types.go

@@ -451,9 +451,6 @@ type KubevirtSpec struct {
 	// InfraNamespace is the namespace that KubeVirt provider will use to create and manage resources in the infra cluster,
 	// such as VirtualMachines, VirtualMachineInstances, etc...
 	InfraNamespace string `json:"infraNamespace"`
-	// InfraClusterKubeconfig is a base64-encoded kubeconfig file that points to a KubeVirt infra cluster (a cluster where KubeVirt is installed).
-	// This Kubeconfig will be used to create and manage KubeVirt specific resources such as DataVolumes.
-	InfraClusterKubeconfig string `json:"infraClusterKubeconfig"`
 	// ZoneAndRegionEnabled indicates if need to get Region and zone labels from the cloud provider
 	ZoneAndRegionEnabled bool `json:"zoneAndRegionEnabled,omitempty"`
 	// LoadBalancerEnabled indicates if the ccm should create and manage the clusters load balancers.

pkg/apis/kubeone/v1beta2/zz_generated.conversion.go

@@ -443,9 +443,6 @@ type KubevirtSpec struct {
 	// InfraNamespace is the namespace that KubeVirt provider will use to create and manage resources in the infra cluster,
 	// such as VirtualMachines, VirtualMachineInstances, etc...
 	InfraNamespace string `json:"infraNamespace"`
-	// InfraClusterKubeconfig is a base64-encoded kubeconfig file that points to a KubeVirt infra cluster (a cluster where KubeVirt is installed).
-	// This Kubeconfig will be used to create and manage KubeVirt specific resources such as DataVolumes.
-	InfraClusterKubeconfig string `json:"infraClusterKubeconfig"`
 	// ZoneAndRegionEnabled indicates if need to get Region and zone labels from the cloud provider
 	ZoneAndRegionEnabled bool `json:"zoneAndRegionEnabled,omitempty"`
 	// LoadBalancerEnabled indicates if the ccm should create and manage the clusters load balancers.

pkg/apis/kubeone/v1beta3/types.go

@@ -19,7 +19,6 @@ package validation
 import (
 	"bytes"
 	"crypto/x509"
-	"encoding/base64"
 	"fmt"
 	"net"
 	"os"
@@ -209,15 +208,6 @@ func ValidateCloudProviderSpec(cluster kubeoneapi.KubeOneCluster, fldPath *field
 			allErrs = append(allErrs, field.Forbidden(kubevirtFld, "only one provider can be used at the same time"))
 		}
 		providerFound = true
-		switch providerSpec.Kubevirt.InfraClusterKubeconfig {
-		case "":
-			allErrs = append(allErrs, field.Required(kubevirtFld.Child("infraClusterKubeconfig"), "is required for kubevirt provider"))
-		default:
-			_, err := base64.StdEncoding.DecodeString(providerSpec.Kubevirt.InfraClusterKubeconfig)
-			if err != nil {
-				allErrs = append(allErrs, field.Forbidden(kubevirtFld.Child("infraClusterKubeconfig"), "must be base64-encoded"))
-			}
-		}
 		if providerSpec.Kubevirt.InfraNamespace == "" {
 			allErrs = append(allErrs, field.Required(kubevirtFld.Child("infraNamespace"), "is required for kubevirt provider"))
 		}

pkg/apis/kubeone/v1beta3/zz_generated.conversion.go

@@ -513,29 +513,17 @@ func TestValidateCloudProviderSpec(t *testing.T) {
 		},
 		{
 			name: "valid Kubevirt provider config",
-			providerConfig: kubeoneapi.CloudProviderSpec{
-				Kubevirt: &kubeoneapi.KubevirtSpec{
-					InfraClusterKubeconfig: "YXBpVmVyc2lvbjogdjEKY2x1c3RlcnM6Ci0gY2x1c3RlcjoKICAgIGNlcnRpZmljYXRlLWF1dGhvcml0eS1kYXRhOiBMU1hZWgogICAgc2VydmVyOiBodHRwczovL3h5ei5leGFtcGxlLmNvbTo2NDQzCiAgbmFtZTogeHl6CmNvbnRleHRzOgotIGNvbnRleHQ6CiAgICBjbHVzdGVyOiB4eXoKICAgIHVzZXI6IGRlZmF1bHQKICBuYW1lOiB4eXoKY3VycmVudC1jb250ZXh0OiB4eXoKa2luZDogQ29uZmlnCnByZWZlcmVuY2VzOiB7fQp1c2VyczoKLSBuYW1lOiBkZWZhdWx0CiAgdXNlcjoKICAgIHRva2VuOiB4eXphdzI1Lnh5ego=",
-					InfraNamespace:         "tenant-xyz",
-				},
-			},
-			expectedError: false,
-		},
-		{
-			name: "Kubevirt provider config missing InfraClusterKubeconfig",
 			providerConfig: kubeoneapi.CloudProviderSpec{
 				Kubevirt: &kubeoneapi.KubevirtSpec{
 					InfraNamespace: "tenant-xyz",
 				},
 			},
-			expectedError: true,
+			expectedError: false,
 		},
 		{
 			name: "Kubevirt provider config missing InfraNamespace",
 			providerConfig: kubeoneapi.CloudProviderSpec{
-				Kubevirt: &kubeoneapi.KubevirtSpec{
-					InfraClusterKubeconfig: "YXBpVmVyc2lvbjogdjEKY2x1c3RlcnM6Ci0gY2x1c3RlcjoKICAgIGNlcnRpZmljYXRlLWF1dGhvcml0eS1kYXRhOiBMU1hZWgogICAgc2VydmVyOiBodHRwczovL3h5ei5leGFtcGxlLmNvbTo2NDQzCiAgbmFtZTogeHl6CmNvbnRleHRzOgotIGNvbnRleHQ6CiAgICBjbHVzdGVyOiB4eXoKICAgIHVzZXI6IGRlZmF1bHQKICBuYW1lOiB4eXoKY3VycmVudC1jb250ZXh0OiB4eXoKa2luZDogQ29uZmlnCnByZWZlcmVuY2VzOiB7fQp1c2VyczoKLSBuYW1lOiBkZWZhdWx0CiAgdXNlcjoKICAgIHRva2VuOiB4eXphdzI1Lnh5ego=",
-				},
+				Kubevirt: &kubeoneapi.KubevirtSpec{},
 			},
 			expectedError: true,
 		},