rollouts: reconcile fleet rollouts periodically (#3841)

* rollouts: reconcile fleet rollouts periodically

Author: droot

rollouts/controllers/rollout_controller.go

@@ -25,6 +25,7 @@ import (
 	"sort"
 	"strings"
 	"sync"
+	"time"
 
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -156,6 +157,18 @@ func (r *RolloutReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ct
 	if err != nil {
 		return ctrl.Result{}, err
 	}
+	if rollout.Spec.Clusters.SourceType == gitopsv1alpha1.GCPFleet &&
+		(rollout.Status.Overall == "Completed" || rollout.Status.Overall == "Stalled") {
+		// TODO (droot): The rollouts in completed/stalled state will not be reconciled
+		// whenever fleet memberships change, so scheduling a periodic reconcile
+		// until we fix https://github.com/GoogleContainerTools/kpt/issues/3835
+		// This can be safely removed once we start monitoring fleet changes.
+		// Note: we watch containercluster types, so this problem doesn't exist for the
+		// KCC clusters.
+		return ctrl.Result{
+			RequeueAfter: 30 * time.Second,
+		}, nil
+	}
 
 	return ctrl.Result{}, nil
 }