Skip to content

Commit 587e814

Browse files
committed
pkg/infrastructure: use private generateSecret
capi v1.11 removed support for v1beta1 clusters in the GenerateSecret function. This institutes a simple workaround of creating a private copy of the function until we are ready to upgrade all of our usage to v1beta2.
1 parent 5cc7726 commit 587e814

File tree

1 file changed

+15
-1
lines changed

1 file changed

+15
-1
lines changed

pkg/infrastructure/clusterapi/clusterapi.go

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import (
1919
"k8s.io/apimachinery/pkg/util/wait"
2020
"k8s.io/utils/ptr"
2121
clusterv1 "sigs.k8s.io/cluster-api/api/core/v1beta1"
22+
"sigs.k8s.io/cluster-api/util"
2223
utilkubeconfig "sigs.k8s.io/cluster-api/util/kubeconfig"
2324
"sigs.k8s.io/controller-runtime/pkg/client"
2425

@@ -202,7 +203,7 @@ func (i *InfraProvider) Provision(ctx context.Context, dir string, parents asset
202203
}
203204
// Create the secret.
204205
clusterKubeconfig := clusterKubeconfigAsset.Files()[0].Data
205-
secret := utilkubeconfig.GenerateSecret(cluster, clusterKubeconfig)
206+
secret := generateSecret(cluster, clusterKubeconfig)
206207
if err := cl.Create(ctx, secret); err != nil {
207208
return fileList, err
208209
}
@@ -731,3 +732,16 @@ func warnIfFalsyInfraConditions(ctx context.Context, objRef *corev1.ObjectRefere
731732
}
732733
logrus.Infof("Done checking conditions for %s", objInfo)
733734
}
735+
736+
// generateSecret returns a Kubernetes secret for the given Cluster and kubeconfig data.
737+
// TODO: upgrade our usage from the deprecated capi cluster v1beta1 -> v1beta2
738+
// and remove this function in favor of Generate secret in capi util/kubeconfig.
739+
func generateSecret(cluster *clusterv1.Cluster, data []byte) *corev1.Secret {
740+
name := util.ObjectKey(cluster)
741+
return utilkubeconfig.GenerateSecretWithOwner(name, data, metav1.OwnerReference{
742+
APIVersion: clusterv1.GroupVersion.String(),
743+
Kind: "Cluster",
744+
Name: cluster.Name,
745+
UID: cluster.UID,
746+
})
747+
}

0 commit comments

Comments
 (0)