Fix race caused by Scheme update

Signed-off-by: Andrei Pavlov <[email protected]>
Mirantis · Oct 30, 2024 · dc2991c · dc2991c
1 parent 225edee
commit dc2991c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/internal/certmanager/verifier.go b/internal/certmanager/verifier.go
@@ -22,7 +22,8 @@ import (
 	"k8s.io/client-go/rest"
 )
 
-func VerifyAPI(ctx context.Context, restcfg *rest.Config, scheme *runtime.Scheme, namespace string) error {
+func VerifyAPI(ctx context.Context, restcfg *rest.Config, namespace string) error {
+	scheme := runtime.NewScheme()
 	checker, err := cmapichecker.New(restcfg, scheme, namespace)
 	if err != nil {
 		return err

diff --git a/internal/controller/management_controller.go b/internal/controller/management_controller.go
@@ -384,7 +384,7 @@ func (r *ManagementReconciler) enableAdditionalComponents(ctx context.Context, m
 		capiOperatorValues = v
 	}
 
-	err := certmanager.VerifyAPI(ctx, r.Config, r.Scheme, r.SystemNamespace)
+	err := certmanager.VerifyAPI(ctx, r.Config, r.SystemNamespace)
 	if err != nil {
 		return fmt.Errorf("failed to check in the cert-manager API is installed: %v", err)
 	}