Issue #4156 (part2) - Bug: On restart, agbot doesn't update a node ev…

…en though a new service was added and a deployment policy update occurred

Signed-off-by: Le Zhang <zhangl@us.ibm.com>

agreementbot/consumer_protocol_handler.go

@@ -498,10 +498,9 @@ func (b *BaseConsumerProtocolHandler) HandlePolicyChangeForAgreement(ag persiste
 			t_comp, consumerNamespace, t_reason := compcheck.CheckClusterNamespaceCompatibility(dev.NodeType, dev.ClusterNamespace, dev.IsNamespaceScoped, busPol.ClusterNamespace, wl.ClusterDeployment, ag.Pattern, false, msgPrinter)
 			if !t_comp {
 				glog.V(5).Infof(BCPHlogstring(b.Name(), fmt.Sprintf("cluster namespace %v is not longer compatible for agreement %v. Reason is: %v", consumerNamespace, ag.CurrentAgreementId, t_reason)))
-
+				return true, true, false
 			}
 			// new cluster namespace is still compatible
-			return true, true, false
 		}
 	}