Rebase and fix comments

kubernetes-sigs · Dec 12, 2023 · 60a02dd · 60a02dd
1 parent 64bdfec
commit 60a02dd
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/pkg/services/govmomi/clustermodules/provider.go b/pkg/services/govmomi/clustermodules/provider.go
@@ -19,6 +19,8 @@ package clustermodules
 
 import (
 	"context"
+	"fmt"
+	"net/http"
 	"strings"
 
 	"github.com/vmware/govmomi/vapi/cluster"
@@ -91,11 +93,12 @@ func (cm *provider) DoesModuleExist(ctx context.Context, moduleUUID string, clus
 
 	_, err := cm.manager.ListModuleMembers(ctx, moduleUUID)
 	if err == nil {
+		log.V(4).Info("Cluster module exists", "moduleUUID", moduleUUID)
 		return true, nil
 	}
 
 	// Is there a better way of unwrap vAPI errors?
-	if strings.Contains(err.Error(), "404 Not Found") {
+	if strings.Contains(err.Error(), fmt.Sprintf("%d %s", http.StatusNotFound, http.StatusText(http.StatusNotFound))) {
 		log.V(4).Info("Cluster module doesn't exist", "moduleUUID", moduleUUID)
 		return false, nil
 	}