fix issue-314 by checking zookeeperCluster resource version before up…

…dating sts ADDENDUM (#355) Fix owner-rv check by checking the existing STS object
pravega · Jul 1, 2021 · b18f163 · b18f163
1 parent bd0d4ef
commit b18f163
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/zookeepercluster/zookeepercluster_controller.go b/pkg/controller/zookeepercluster/zookeepercluster_controller.go
@@ -234,8 +234,8 @@ func (r *ReconcileZookeeperCluster) reconcileStatefulSet(instance *zookeeperv1be
 		return err
 	} else {
 		// check whether zookeeperCluster is updated before updating the sts
-		if !zookeeperClusterFresherThanSts(instance, sts) {
-			return fmt.Errorf("Staleness: cr.ResourceVersion %s is smaller than labeledRV %s", instance.ResourceVersion, sts.Labels["owner-rv"])
+		if !zookeeperClusterFresherThanSts(instance, foundSts) {
+			return fmt.Errorf("Staleness: cr.ResourceVersion %s is smaller than labeledRV %s", instance.ResourceVersion, foundSts.Labels["owner-rv"])
 		}
 		foundSTSSize := *foundSts.Spec.Replicas
 		newSTSSize := *sts.Spec.Replicas