Merge pull request #2126 from AllenZMC/automated-cherry-pick-of-#2117-upstream-release-1.1

Automated cherry pick of #2117: Fix nil problem in updateConfiguration

Merge pull request #2126 from AllenZMC/automated-cherry-pick-of-#2117-upstream-release-1.1
Automated cherry pick of #2117: Fix nil problem in updateConfiguration
20a8e72c · karmada-bot · GitHub · b24573ee · e90a02a6 · 20a8e72c
Unverified Commit 20a8e72c authored 2 years ago by karmada-bot Committed by GitHub 2 years ago
Hide whitespace changes
Inline Side-by-side

Showing

with 2 additions and 1 deletion
+2 -1
--- a/pkg/resourceinterpreter/customizedinterpreter/configmanager/manager.go
+++ b/pkg/resourceinterpreter/customizedinterpreter/configmanager/manager.go
@@ -95,7 +95,8 @@ func (m *interpreterConfigManager) updateConfiguration() {

 		config, err := helper.ConvertToResourceExploringWebhookConfiguration(unstructuredConfig)
 		if err != nil {
-			klog.Errorf("Failed to convert object(%s), err", config.GroupVersionKind().String(), err)
+			gvk := unstructuredConfig.GroupVersionKind().String()
+			klog.Errorf("Failed to convert object(%s), err: %v", gvk, err)
 			return
 		}
 		configs = append(configs, config)