[chore] fix prefix

Signed-off-by: Stepan Paksashvili <stepan.paksashvili@flant.com>

Author: ipaqsa

internal/objectpatch/patch.go

@@ -2,6 +2,7 @@ package objectpatch
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/deckhouse/module-sdk/pkg"
 )
@@ -27,46 +28,22 @@ func (p *Patch) Description() string {
 	return fmt.Sprintf("%v", op)
 }
 
-// GetName returns the name of the object to patch.
-func (p *Patch) GetName() string {
-	name, ok := p.patchValues["name"]
-	if !ok {
-		return ""
+// SetObjectPrefix sets prefix for object name.
+func (p *Patch) SetObjectPrefix(prefix string) {
+	if p.patchValues == nil {
+		return
 	}
 
-	// Handle both string and typed names
-	return fmt.Sprintf("%v", name)
-}
-
-// GetNamespace returns the namespace of the object to patch.
-func (p *Patch) GetNamespace() string {
-	ns, ok := p.patchValues["namespace"]
+	name, ok := p.patchValues["name"]
 	if !ok {
-		return ""
+		return
 	}
 
-	// Handle both string and typed namespaces
-	return fmt.Sprintf("%v", ns)
-}
-
-// SetPrifixName sets the name for the patch operation with a prefix.
-func (p *Patch) SetNamePrefix(prefix string) {
-	// Set the name for the patch operation with a prefix.
-	// This is used to identify the target object in Kubernetes.
-	if p.patchValues == nil {
-		p.patchValues = make(map[string]any)
+	if strings.HasPrefix(name.(string), prefix+"-") {
+		return
 	}
-	p.patchValues["name"] = fmt.Sprintf("%s-%s", prefix, p.GetName())
-}
 
-// SetName sets the name for the patch operation.
-func (p *Patch) SetName(name string) {
-	// Set the name for the patch operation.
-	// This is used to identify the target object in Kubernetes.
-	if p.patchValues == nil {
-		p.patchValues = make(map[string]any)
-	}
-	p.patchValues["name"] = name
+	p.patchValues["name"] = fmt.Sprintf("%s-%s", prefix, name.(string))
 }
 
 // WithSubresource sets the subresource to patch (e.g., "status", "scale").

pkg/patch.go

@@ -93,10 +93,7 @@ type NamespacedPatchCollector interface {
 // - filterOperation to modify object via Get-filter-Update process
 type PatchCollectorOperation interface {
 	Description() string
-	GetName() string
-	SetName(name string)
-	SetNamePrefix(prefix string)
-	GetNamespace() string
+	SetObjectPrefix(prefix string)
 }
 
 type PatchCollectorOption interface {