controllers/backupconfiguration_controller_helpers.go

@@ -290,7 +290,7 @@ func (r *BackupConfigurationReconciler) addSidecar(backupConf formolv1alpha1.Bac
 
 // Delete the sidecar role is there is no more sidecar container in the namespace
 func (r *BackupConfigurationReconciler) deleteRBAC() error {
-	for _, roleBindingName := range []string{FORMOL_BS_CREATOR_ROLEBINDING, FORMOL_SIDECAR_ROLEBINDING} {
+	for _, roleBindingName := range []string{FORMOL_BS_CREATOR_ROLEBINDING, FORMOL_SIDECAR_CLUSTERROLEBINDING} {
 		roleBinding := rbacv1.RoleBinding{}
 		if err := r.Get(r.Context, client.ObjectKey{
 			Namespace: r.Namespace,
@@ -301,7 +301,7 @@ func (r *BackupConfigurationReconciler) deleteRBAC() error {
 			}
 		}
 	}
-	for _, roleName := range []string{FORMOL_BS_CREATOR_ROLE, FORMOL_SIDECAR_ROLE} {
+	for _, roleName := range []string{FORMOL_BS_CREATOR_ROLE, FORMOL_SIDECAR_CLUSTERROLE} {
 		role := rbacv1.Role{}
 		if err := r.Get(r.Context, client.ObjectKey{
 			Namespace: r.Namespace,

controllers/backupsession_controller.go

@@ -62,9 +62,7 @@ func (r *BackupSessionReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 		Namespace: backupSession.Spec.Ref.Namespace,
 		Name:      backupSession.Spec.Ref.Name,
 	}, &backupConf); err != nil {
-		if !errors.IsNotFound(err) {
 		r.Log.Error(err, "unable to get BackupConfiguration")
-		}
 		return ctrl.Result{}, err
 	}
 	if !backupSession.ObjectMeta.DeletionTimestamp.IsZero() {