Some minor fixes

- Add comment to predicate function.
- chenge the order of an if case.

pkg/liqo-controller-manager/namespaceOffloading-controller/manage_liqo_scheduling_label.go

@@ -18,10 +18,11 @@ func addLiqoSchedulingLabel(ctx context.Context, c client.Client, namespaceName
 		return err
 	}
 
+	if namespace.Labels == nil {
+		namespace.Labels = map[string]string{}
+	}
+
 	if value, ok := namespace.Labels[liqoconst.SchedulingLiqoLabel]; !ok || value != liqoconst.SchedulingLiqoLabelValue {
-		if namespace.Labels == nil {
-			namespace.Labels = map[string]string{}
-		}
 		namespace.Labels[liqoconst.SchedulingLiqoLabel] = liqoconst.SchedulingLiqoLabelValue
 		if err := c.Update(ctx, namespace); err != nil {
 			klog.Errorf(" %s --> Unable to add liqo scheduling label to the namespace '%s'", err, namespace.GetName())

pkg/liqo-controller-manager/namespaceOffloading-controller/namespaceoffloading_controller.go

@@ -108,11 +108,15 @@ func (r *NamespaceOffloadingReconciler) Reconcile(ctx context.Context, req ctrl.
 }
 
 // Todo: how to awake this controller for every NamespaceOffloading when a new NamespaceMap is created (or recreated).
+// The name of all NamespaceOffloading resources must be always equal to "offloading", resources with a different
+// name are not considered.
+// At the moment the content of NamespaceOffloading.Spec is assumed to be immutable so we have to monitor only the deletion
+// phase and not all updates.
 func namespaceOffloadingPredicate() predicate.Predicate {
 	return predicate.Funcs{
 		UpdateFunc: func(e event.UpdateEvent) bool {
 			if !(e.ObjectNew.GetDeletionTimestamp().IsZero()) && ctrlutils.ContainsFinalizer(e.ObjectNew,
-				namespaceOffloadingControllerFinalizer) {
+				namespaceOffloadingControllerFinalizer) && e.ObjectNew.GetName() == liqoconst.DefaultNamespaceOffloadingName {
 				return true
 			}
 			return false