Fix kubebuilder topological markers: use "=" instead of ":" between t…

…okens

- Do not add topological markers for sensitive fields

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>

pkg/types/field.go

@@ -152,7 +152,9 @@ func NewField(g *Builder, cfg *config.Resource, r *resource, sch *schema.Schema,
 	f.FieldType = fieldType
 	f.InitType = initType
 
-	AddServerSideApplyMarkers(f)
+	if !sch.Sensitive {
+		AddServerSideApplyMarkers(f)
+	}
 
 	return f, nil
 }
@@ -173,7 +175,7 @@ func AddServerSideApplyMarkers(f *Field) {
 	case schema.TypeSet:
 		if es, ok := f.Schema.Elem.(*schema.Schema); ok {
 			switch es.Type { //nolint:exhaustive
-			// We assume scalar types can be granular maps.
+			// We assume scalar types can be granular sets.
 			case schema.TypeString, schema.TypeBool, schema.TypeInt, schema.TypeFloat:
 				f.Comment.ServerSideApplyOptions.ListType = ptr.To[markers.ListType](markers.ListTypeSet)
 			}

pkg/types/markers/ssa.go

@@ -20,7 +20,7 @@ const (
 	// elements.
 	ListTypeSet ListType = "set"
 
-	// ListTypeSet can be granularly merged, and different managers can own
+	// ListTypeMap can be granularly merged, and different managers can own
 	// different elements in the list. The list can include only nested types
 	// (i.e. objects).
 	ListTypeMap ListType = "map"
@@ -68,19 +68,19 @@ func (o ServerSideApplyOptions) String() string {
 	m := ""
 
 	if o.ListType != nil {
-		m += fmt.Sprintf("+listType:%s\n", *o.ListType)
+		m += fmt.Sprintf("+listType=%s\n", *o.ListType)
 	}
 
 	for _, k := range o.ListMapKey {
-		m += fmt.Sprintf("+listMapKey:%s\n", k)
+		m += fmt.Sprintf("+listMapKey=%s\n", k)
 	}
 
 	if o.MapType != nil {
-		m += fmt.Sprintf("+mapType:%s\n", *o.MapType)
+		m += fmt.Sprintf("+mapType=%s\n", *o.MapType)
 	}
 
 	if o.StructType != nil {
-		m += fmt.Sprintf("+structType:%s\n", *o.StructType)
+		m += fmt.Sprintf("+structType=%s\n", *o.StructType)
 	}
 
 	return m

pkg/types/markers/ssa_test.go

@@ -20,20 +20,20 @@ func TestServerSideApplyOptions(t *testing.T) {
 			o: ServerSideApplyOptions{
 				MapType: ptr.To[MapType](MapTypeAtomic),
 			},
-			want: "+mapType:atomic\n",
+			want: "+mapType=atomic\n",
 		},
 		"StructType": {
 			o: ServerSideApplyOptions{
 				StructType: ptr.To[StructType](StructTypeAtomic),
 			},
-			want: "+structType:atomic\n",
+			want: "+structType=atomic\n",
 		},
 		"ListType": {
 			o: ServerSideApplyOptions{
 				ListType:   ptr.To[ListType](ListTypeMap),
 				ListMapKey: []string{"name", "coolness"},
 			},
-			want: "+listType:map\n+listMapKey:name\n+listMapKey:coolness\n",
+			want: "+listType=map\n+listMapKey=name\n+listMapKey=coolness\n",
 		},
 	}