Fix handling of coco / hub replicas restart

Hub upgrade was restarting the existing instance, which is fine for cases where
the user didn't set any replica for it. Coco's upgrade was scaling to
the number of replicas passed as parameters, which doesn't work if the
user didn't set any and has running instances.

This merges both: if no replica is set, just restart the enabled ones.
Otherwise scale to whatever the user provided.

mgradm/shared/coco/coco.go

@@ -34,6 +34,9 @@ func Upgrade(
 		return err
 	}
 
+	if !cocoFlags.IsChanged {
+		return systemd.RestartInstantiated(podman.ServerAttestationService)
+	}
 	return systemd.ScaleService(cocoFlags.Replicas, podman.ServerAttestationService)
 }
 

mgradm/shared/hub/xmlrpcapi.go

@@ -72,8 +72,6 @@ func EnableHubXmlrpc(systemd podman.Systemd, replicas int) error {
 		if err := systemd.ScaleService(replicas, podman.HubXmlrpcService); err != nil {
 			return utils.Errorf(err, L("cannot enable service"))
 		}
-	} else {
-		log.Info().Msg(L("Not starting Hub XML-RPC API service"))
 	}
 	return nil
 }
@@ -95,7 +93,10 @@ func Upgrade(
 		return err
 	}
 
-	return systemd.RestartInstantiated(podman.HubXmlrpcService)
+	if !hubXmlrpcFlags.IsChanged {
+		return systemd.RestartInstantiated(podman.HubXmlrpcService)
+	}
+	return systemd.ScaleService(hubXmlrpcFlags.Replicas, podman.HubXmlrpcService)
 }
 
 // generateHubXmlrpcSystemdService creates the Hub XMLRPC systemd files.