added coment explaninig package cloning changes

return err for all config controllers when cluster not found

adds check of cluster ownership

addons/controllers/antrea/antreaconfig_controller.go

@@ -80,7 +80,7 @@ func (r *AntreaConfigReconciler) Reconcile(ctx context.Context, req ctrl.Request
 		if apierrors.IsNotFound(err) && cluster != nil {
 			log.Info(fmt.Sprintf("'%s/%s' is listed as owner reference but could not be found",
 				cluster.Namespace, cluster.Name))
-			return ctrl.Result{}, nil
+			return ctrl.Result{}, err
 		}
 		log.Error(err, "could not determine owner cluster")
 		return ctrl.Result{}, err

addons/controllers/calico/calicoconfig_controller.go

@@ -78,7 +78,7 @@ func (r *CalicoConfigReconciler) Reconcile(ctx context.Context, req ctrl.Request
 		if apierrors.IsNotFound(err) && cluster != nil {
 			log.Info(fmt.Sprintf("'%s/%s' is listed as owner reference but could not be found",
 				cluster.Namespace, cluster.Name))
-			return ctrl.Result{}, nil
+			return ctrl.Result{}, err
 		}
 		log.Error(err, "could not determine owner cluster")
 		return ctrl.Result{}, err

addons/controllers/kapp-controller/kappcontrollerconfig_controller.go

@@ -73,7 +73,7 @@ func (r *KappControllerConfigReconciler) Reconcile(ctx context.Context, req ctrl
 		if apierrors.IsNotFound(err) && cluster != nil {
 			r.Log.Info(fmt.Sprintf("'%s/%s' is listed as owner reference but could not be found",
 				cluster.Namespace, cluster.Name))
-			return ctrl.Result{}, nil
+			return ctrl.Result{}, err
 		}
 		r.Log.Info("could not determine owner cluster")
 		return ctrl.Result{}, err

addons/pkg/util/clusterbootstrapclone/clusterbootstrapclone.go

@@ -380,7 +380,16 @@ func (h *Helper) HandleExistingClusterBootstrap(clusterBootstrap *runtanzuv1alph
 			packages = packagesToBeCloned
 		}
 	} else {
-		// packages with values from inline need to be cloned.
+		// packages with values from inline need to be cloned for two cases:
+		//
+		// 1. When no Clusterbootstrap is given by user and clusterbootstraptemplate
+		//    is cloned as clusterbootstrap but resolved tkr is not set - In this
+		//   case the providers are already cloned so providers have to be cloned again
+		//
+		// 2. When full clusterbootstrap without tkr annotation is given by user.
+		//    In this case we only want to handle inline packages for creating
+		//    secret. The providers are expected to be present in the
+		//   clusternamespace, created by user before clusterbootstrap is created.
 		var nonEmptyInlinePackages []*runtanzuv1alpha3.ClusterBootstrapPackage
 		for _, cbPackage := range packages {
 			if cbPackage != nil && cbPackage.ValuesFrom != nil && cbPackage.ValuesFrom.Inline != nil {