re-read the sts for checking the annotation fixing both pvc resize flows

Author: gkech

pkg/controller/perconaservermongodb/statefulset.go

@@ -62,9 +62,15 @@ func (r *ReconcilePerconaServerMongoDB) reconcileStatefulSet(ctx context.Context
 		return nil, errors.Wrapf(err, "reconcile PVCs for %s", sfs.Name)
 	}
 
-	if _, ok := sfs.Annotations[api.AnnotationPVCResizeInProgress]; ok {
-		log.V(1).Info("PVC resize in progress, skipping reconciliation of statefulset", "name", sfs.Name)
-		return sfs, nil
+	// Re-read the StatefulSet from the cluster to check for the PVC resize
+	// annotation, as it may have been set during reconcilePVCs and the local
+	// sfs object would be stale.
+	currentSts := new(appsv1.StatefulSet)
+	if err := r.client.Get(ctx, types.NamespacedName{Name: sfs.Name, Namespace: sfs.Namespace}, currentSts); err == nil {
+		if _, ok := currentSts.Annotations[api.AnnotationPVCResizeInProgress]; ok {
+			log.V(1).Info("PVC resize in progress, skipping reconciliation of statefulset", "name", sfs.Name)
+			return sfs, nil
+		}
 	}
 
 	err = r.createOrUpdate(ctx, sfs)