Set TypeMeta data for agent cluster-manifests

zaneb · zaneb · commit 3dc204e4ee97 · 2024-04-08T11:10:25.000+12:00
diff --git a/cmd/openshift-install/testdata/agent/configimage/manifests/default_manifests.txt b/cmd/openshift-install/testdata/agent/configimage/manifests/default_manifests.txt
@@ -43,6 +43,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   creationTimestamp: null
   name: ostest
@@ -73,7 +75,7 @@ status:
   progress:
     totalPercentage: 0
 -- expected/cluster-deployment.yaml --
-apiVersion: v1
+apiVersion: hive.openshift.io/v1
 kind: ClusterDeployment
 metadata:
   creationTimestamp: null
@@ -95,6 +97,8 @@ spec:
     name: ostest-pull-secret
 status: {}
 -- expected/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   creationTimestamp: null
   name: openshift-was not built correctly
diff --git a/cmd/openshift-install/testdata/agent/image/configurations/baremetal_host_bmc_config.txt b/cmd/openshift-install/testdata/agent/image/configurations/baremetal_host_bmc_config.txt
@@ -129,6 +129,8 @@ metadata:
 rendezvousIP: 192.168.111.80
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   annotations:
     agent-install.openshift.io/install-config-overrides: '{"platform":{"baremetal":{"hosts":[{"name":"master-0","bmc":{"username":"foo","password":"pwd1","address":"http://172.22.0.10:8000/redfish/v1/Systems/d3fe1eb9-f04c-465d-b5d6-d267f0a0059b","disableCertificateVerification":false},"role":"","bootMACAddress":"00:00:00:aa:bb:01","hardwareProfile":""},{"name":"master-1","bmc":{"username":"foo","password":"pwd2","address":"http://172.22.0.11:8000/redfish/v1/Systems/65bf6806-b634-4494-bccf-032ebdecc65d","disableCertificateVerification":false},"role":"","bootMACAddress":"00:00:00:aa:bb:02","hardwareProfile":""},{"name":"master-2","bmc":{"username":"foo","password":"pwd3","address":"http://172.22.0.12:8000/redfish/v1/Systems/ff67e706-28c4-42f6-b9a2-ce714195300a","disableCertificateVerification":false},"role":"","bootMACAddress":"00:00:00:aa:bb:03","hardwareProfile":""}],"clusterProvisioningIP":"172.22.0.3","provisioningNetwork":"Managed","provisioningNetworkInterface":"enp1s0","provisioningNetworkCIDR":"172.22.0.0/24","provisioningDHCPRange":"172.22.0.10,172.22.0.254"}}}'
diff --git a/cmd/openshift-install/testdata/agent/image/configurations/external_compact.txt b/cmd/openshift-install/testdata/agent/image/configurations/external_compact.txt
@@ -44,6 +44,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   annotations:
     agent-install.openshift.io/install-config-overrides: '{"platform":{"external":{"platformName":"some-cloud-provider"}}}'
diff --git a/cmd/openshift-install/testdata/agent/image/configurations/vsphere_no-credentials.txt b/cmd/openshift-install/testdata/agent/image/configurations/vsphere_no-credentials.txt
@@ -47,6 +47,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   creationTimestamp: null
   name: ostest
diff --git a/cmd/openshift-install/testdata/agent/image/configurations/vsphere_with-credentials.txt b/cmd/openshift-install/testdata/agent/image/configurations/vsphere_with-credentials.txt
@@ -65,6 +65,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   annotations:
     agent-install.openshift.io/install-config-overrides: '{"platform":{"vsphere":{"vcenters":[{"server":"vcenter.openshift.com","user":"testUser","password":"testPassword","datacenters":["testDatacenter"]}],"failureDomains":[{"name":"testfailureDomain","region":"testRegion","zone":"testZone","server":"vcenter.openshift.com","topology":{"datacenter":"testDatacenter","computeCluster":"/testDatacenter/host/testComputecluster","networks":["testNetwork"],"datastore":"/testDatacenter/datastore/testDatastore","resourcePool":"/testDatacenter/host/testComputecluster/Resources","folder":"/testDatacenter/vm/testFolder"}}]}}}'
diff --git a/cmd/openshift-install/testdata/agent/image/manifests/default_manifests.txt b/cmd/openshift-install/testdata/agent/image/manifests/default_manifests.txt
@@ -44,6 +44,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/agent-cluster-install.yaml --
+apiVersion: extensions.hive.openshift.io/v1beta1
+kind: AgentClusterInstall
 metadata:
   creationTimestamp: null
   name: ostest
@@ -74,7 +76,7 @@ status:
   progress:
     totalPercentage: 0
 -- expected/cluster-deployment.yaml --
-apiVersion: v1
+apiVersion: hive.openshift.io/v1
 kind: ClusterDeployment
 metadata:
   creationTimestamp: null
@@ -96,6 +98,8 @@ spec:
     name: ostest-pull-secret
 status: {}
 -- expected/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   creationTimestamp: null
   name: openshift-was not built correctly
@@ -104,6 +108,8 @@ spec:
   releaseImage: $RELEASE_IMAGE
 status: {}
 -- expected/infraenv.yaml --
+apiVersion: agent-install.openshift.io/v1beta1
+kind: InfraEnv
 metadata:
   creationTimestamp: null
   name: ostest
diff --git a/cmd/openshift-install/testdata/agent/unconfigured-ignition/configurations/from-ztp-manifests.txt b/cmd/openshift-install/testdata/agent/unconfigured-ignition/configurations/from-ztp-manifests.txt
@@ -43,20 +43,26 @@ stringData:
     }
 
 -- cluster-manifests/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   name: cluster0-image-set
   namespace: cluster0
 spec:
   releaseImage: $RELEASE_IMAGE
 
 -- expected/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   name: cluster0-image-set
   namespace: cluster0
 spec:
   releaseImage: $RELEASE_IMAGE
 
 -- expected/infraenv.yaml --
+apiVersion: agent-install.openshift.io/v1beta1
+kind: InfraEnv
 metadata:
   creationTimestamp: null
   name: ostest
diff --git a/cmd/openshift-install/testdata/agent/unconfigured-ignition/manifests/default_manifests.txt b/cmd/openshift-install/testdata/agent/unconfigured-ignition/manifests/default_manifests.txt
@@ -48,6 +48,8 @@ metadata:
 rendezvousIP: 192.168.111.20
 
 -- expected/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   creationTimestamp: null
   name: openshift-was not built correctly
@@ -56,6 +58,8 @@ spec:
   releaseImage: $RELEASE_IMAGE
 status: {}
 -- expected/infraenv.yaml --
+apiVersion: agent-install.openshift.io/v1beta1
+kind: InfraEnv
 metadata:
   creationTimestamp: null
   name: ostest
@@ -96,4 +100,4 @@ stringData:
           "auth": "c3VwZXItc2VjcmV0Cg=="
         }
       }
-    }
+    }
diff --git a/cmd/openshift-install/testdata/agent/unconfigured-ignition/validations/missing-cluster-image-set-allowed.txt b/cmd/openshift-install/testdata/agent/unconfigured-ignition/validations/missing-cluster-image-set-allowed.txt
@@ -40,9 +40,11 @@ stringData:
     }
 
 -- expected/cluster-image-set.yaml --
+apiVersion: hive.openshift.io/v1
+kind: ClusterImageSet
 metadata:
   creationTimestamp: null
   name: openshift-was not built correctly
 spec:
   releaseImage: $RELEASE_IMAGE
-status: {}
+status: {}
diff --git a/pkg/asset/agent/manifests/agentclusterinstall.go b/pkg/asset/agent/manifests/agentclusterinstall.go
@@ -173,6 +173,10 @@ func (a *AgentClusterInstall) Generate(dependencies asset.Parents) error {
 		}
 
 		agentClusterInstall := &hiveext.AgentClusterInstall{
+			TypeMeta: metav1.TypeMeta{
+				Kind:       "AgentClusterInstall",
+				APIVersion: hiveext.GroupVersion.String(),
+			},
 			ObjectMeta: metav1.ObjectMeta{
 				Name:      getAgentClusterInstallName(installConfig),
 				Namespace: installConfig.ClusterNamespace(),
diff --git a/pkg/asset/agent/manifests/clusterdeployment.go b/pkg/asset/agent/manifests/clusterdeployment.go
@@ -57,7 +57,7 @@ func (cd *ClusterDeployment) Generate(dependencies asset.Parents) error {
 		clusterDeployment := &hivev1.ClusterDeployment{
 			TypeMeta: metav1.TypeMeta{
 				Kind:       "ClusterDeployment",
-				APIVersion: "v1",
+				APIVersion: hivev1.SchemeGroupVersion.String(),
 			},
 			ObjectMeta: metav1.ObjectMeta{
 				Name:      getClusterDeploymentName(installConfig),
diff --git a/pkg/asset/agent/manifests/clusterdeployment_test.go b/pkg/asset/agent/manifests/clusterdeployment_test.go
@@ -44,7 +44,7 @@ func TestClusterDeployment_Generate(t *testing.T) {
 			expectedConfig: &hivev1.ClusterDeployment{
 				TypeMeta: metav1.TypeMeta{
 					Kind:       "ClusterDeployment",
-					APIVersion: "v1",
+					APIVersion: "hive.openshift.io/v1",
 				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      getClusterDeploymentName(getValidOptionalInstallConfig()),
diff --git a/pkg/asset/agent/manifests/clusterimageset.go b/pkg/asset/agent/manifests/clusterimageset.go
@@ -82,6 +82,10 @@ func (a *ClusterImageSet) Generate(dependencies asset.Parents) error {
 
 func (a *ClusterImageSet) generateManifest(version, releaseImage, clusterNamespace string) error {
 	clusterImageSet := &hivev1.ClusterImageSet{
+		TypeMeta: metav1.TypeMeta{
+			Kind:       "ClusterImageSet",
+			APIVersion: hivev1.SchemeGroupVersion.String(),
+		},
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      fmt.Sprintf("openshift-%s", version),
 			Namespace: clusterNamespace,
diff --git a/pkg/asset/agent/manifests/clusterimageset_test.go b/pkg/asset/agent/manifests/clusterimageset_test.go
@@ -41,6 +41,10 @@ func TestClusterImageSet_Generate(t *testing.T) {
 				},
 			},
 			expectedConfig: &hivev1.ClusterImageSet{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "ClusterImageSet",
+					APIVersion: "hive.openshift.io/v1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name: "openshift-was not built correctly",
 				},
@@ -70,6 +74,10 @@ func TestClusterImageSet_Generate(t *testing.T) {
 				},
 			},
 			expectedConfig: &hivev1.ClusterImageSet{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "ClusterImageSet",
+					APIVersion: "hive.openshift.io/v1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      "openshift-was not built correctly",
 					Namespace: getValidOptionalInstallConfig().ClusterNamespace(),
@@ -92,6 +100,10 @@ func TestClusterImageSet_Generate(t *testing.T) {
 				&releaseimage.Image{},
 			},
 			expectedConfig: &hivev1.ClusterImageSet{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "ClusterImageSet",
+					APIVersion: "hive.openshift.io/v1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      "openshift-4.16.0",
 					Namespace: "agent-cluster",
diff --git a/pkg/asset/agent/manifests/infraenv.go b/pkg/asset/agent/manifests/infraenv.go
@@ -86,6 +86,10 @@ func (i *InfraEnv) Generate(dependencies asset.Parents) error {
 
 func (i *InfraEnv) generateManifest(clusterName, clusterNamespace, sshKey, additionalTrustBundle string, proxy *types.Proxy, architecture string) error {
 	infraEnv := &aiv1beta1.InfraEnv{
+		TypeMeta: metav1.TypeMeta{
+			Kind:       "InfraEnv",
+			APIVersion: aiv1beta1.GroupVersion.String(),
+		},
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      clusterName,
 			Namespace: clusterNamespace,
diff --git a/pkg/asset/agent/manifests/infraenv_test.go b/pkg/asset/agent/manifests/infraenv_test.go
@@ -49,6 +49,10 @@ func TestInfraEnv_Generate(t *testing.T) {
 				getValidAgentConfig(),
 			},
 			expectedConfig: &aiv1beta1.InfraEnv{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "InfraEnv",
+					APIVersion: "agent-install.openshift.io/v1beta1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      getValidOptionalInstallConfig().ClusterName(),
 					Namespace: getValidOptionalInstallConfig().ClusterNamespace(),
@@ -77,6 +81,10 @@ func TestInfraEnv_Generate(t *testing.T) {
 				getValidAgentConfig(),
 			},
 			expectedConfig: &aiv1beta1.InfraEnv{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "InfraEnv",
+					APIVersion: "agent-install.openshift.io/v1beta1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      getClusterDeploymentName(getProxyValidOptionalInstallConfig()),
 					Namespace: getProxyValidOptionalInstallConfig().ClusterNamespace(),
@@ -106,6 +114,10 @@ func TestInfraEnv_Generate(t *testing.T) {
 				getValidAgentConfigWithAdditionalNTPSources(),
 			},
 			expectedConfig: &aiv1beta1.InfraEnv{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "InfraEnv",
+					APIVersion: "agent-install.openshift.io/v1beta1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      getClusterDeploymentName(getProxyValidOptionalInstallConfig()),
 					Namespace: getProxyValidOptionalInstallConfig().ClusterNamespace(),
@@ -136,6 +148,10 @@ func TestInfraEnv_Generate(t *testing.T) {
 				getValidAgentConfig(),
 			},
 			expectedConfig: &aiv1beta1.InfraEnv{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "InfraEnv",
+					APIVersion: "agent-install.openshift.io/v1beta1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      getClusterDeploymentName(getProxyValidOptionalInstallConfig()),
 					Namespace: getProxyValidOptionalInstallConfig().ClusterNamespace(),
@@ -173,6 +189,10 @@ func TestInfraEnv_Generate(t *testing.T) {
 				&agentconfig.AgentConfig{},
 			},
 			expectedConfig: &aiv1beta1.InfraEnv{
+				TypeMeta: metav1.TypeMeta{
+					Kind:       "InfraEnv",
+					APIVersion: "agent-install.openshift.io/v1beta1",
+				},
 				ObjectMeta: metav1.ObjectMeta{
 					Name:      "agent-cluster",
 					Namespace: "agent-ns",
diff --git a/pkg/asset/agent/manifests/util_test.go b/pkg/asset/agent/manifests/util_test.go
@@ -452,6 +452,10 @@ func getAgentHostsWithBMCConfig() *agentconfig.AgentHosts {
 
 func getGoodACI() *hiveext.AgentClusterInstall {
 	goodACI := &hiveext.AgentClusterInstall{
+		TypeMeta: metav1.TypeMeta{
+			Kind:       "AgentClusterInstall",
+			APIVersion: "extensions.hive.openshift.io/v1beta1",
+		},
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      getAgentClusterInstallName(getValidOptionalInstallConfig()),
 			Namespace: getValidOptionalInstallConfig().ClusterNamespace(),
