Merge pull request RHEcosystemAppEng#77 from tchughesiv/fixMerge

fix owned PR merge
rewantsoni · Dec 16, 2021 · b94cd5b · b94cd5b
2 parents df31fad + 5ab1094
commit b94cd5b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/controllers/dbaastenant_authorization.go b/controllers/dbaastenant_authorization.go
@@ -195,7 +195,7 @@ func (r *DBaaSTenantReconciler) reconcileTenantRbacObjs(ctx context.Context, ten
 			!reflect.DeepEqual(clusterRolebinding.Subjects, clusterRoleBindingObj.Subjects) {
 			clusterRoleBindingObj.RoleRef = clusterRolebinding.RoleRef
 			clusterRoleBindingObj.Subjects = clusterRolebinding.Subjects
-			if err := r.updateIfOwned(ctx, &tenant, &clusterRoleObj); err != nil {
+			if err := r.updateIfOwned(ctx, &tenant, &clusterRoleBindingObj); err != nil {
 				return err
 			}
 		}
@@ -226,7 +226,7 @@ func (r *DBaaSTenantReconciler) reconcileInventoryRbacObjs(ctx context.Context,
 			!reflect.DeepEqual(rolebinding.Subjects, roleBindingObj.Subjects) {
 			roleBindingObj.RoleRef = rolebinding.RoleRef
 			roleBindingObj.Subjects = rolebinding.Subjects
-			if err := r.updateIfOwned(ctx, &inventory, &roleObj); err != nil {
+			if err := r.updateIfOwned(ctx, &inventory, &roleBindingObj); err != nil {
 				return err
 			}
 		}