fix: Let the service updater patch the spec instead of rewriting it

This change may help overcoming infinite rewrite loops where STUNner tries to overwrite the LB
Service with its own version and then another controller (e.g., MetalLB) doing modifications that
we just try to blindly overwrite again.

Author: rg0now

internal/updater/client_util.go

@@ -131,7 +131,7 @@ func (u *Updater) upsertService(svc *corev1.Service, gen int) (ctrlutil.Operatio
 		Namespace: svc.GetNamespace(),
 	}}
 
-	op, err := ctrlutil.CreateOrUpdate(u.ctx, client, current, func() error {
+	op, err := ctrlutil.CreateOrPatch(u.ctx, client, current, func() error {
 		if err := setMetadata(current, svc); err != nil {
 			return nil
 		}