From 60a02ddc059878ef7ce6ad2cceca7efd1a858134 Mon Sep 17 00:00:00 2001 From: Ricardo Katz Date: Tue, 12 Dec 2023 17:01:36 -0300 Subject: [PATCH] Rebase and fix comments --- pkg/services/govmomi/clustermodules/provider.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pkg/services/govmomi/clustermodules/provider.go b/pkg/services/govmomi/clustermodules/provider.go index 6025059c7a..5a163c026b 100644 --- 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 }