Watch owned objects to react to OOB changes

It's expected (and useful) behaviour to recreate a controller-created
object that is deleted out of band (e.g., by a pesky human). To make
it work this way, it's usually only necessary for the controller to
watch the objects in question, since a deletion will then trigger a
reconciliation of the owner object.

Signed-off-by: Michael Bridgen <mikeb@squaremobius.net>

module/controllers/bootstrapmodule_controller.go

@@ -70,6 +70,7 @@ func (r *BootstrapModuleReconciler) Reconcile(ctx context.Context, req ctrl.Requ
 		}
 		// This is a hack to work around https://github.com/fluxcd/source-controller/issues/315
 		source.Spec.Reference.Branch = "main"
+		// the resulting source is "controller owned" by the bootstrap module
 		return controllerutil.SetControllerReference(&mod, &source, r.Scheme)
 	})
 	if err != nil {
@@ -236,6 +237,8 @@ func (r *BootstrapModuleReconciler) SetupWithManager(mgr ctrl.Manager) error {
 
 	return ctrl.NewControllerManagedBy(mgr).
 		For(&fleetv1.BootstrapModule{}).
+		Owns(&sourcev1.GitRepository{}).
+		// These are not "controller-owned" by the bootstrap modules, so this cannot use `.Owns`
 		Watches(
 			&source.Kind{Type: &fleetv1.RemoteAssemblage{}},
 			&handler.EnqueueRequestForOwner{

module/controllers/remoteassemblage_controller.go

@@ -197,6 +197,7 @@ func (r *RemoteAssemblageReconciler) SetupWithManager(mgr ctrl.Manager) error {
 			&source.Kind{Type: &corev1.Secret{}},
 			handler.EnqueueRequestsFromMapFunc(r.assemblagesForSecret),
 			builder.WithPredicates(predicate.ResourceVersionChangedPredicate{})).
+		Owns(&kustomv1.Kustomization{}).
 		Complete(r)
 }