fix(dbinstance): correct return values

Signed-off-by: Charel Baum <[email protected]>
crossplane-contrib · Dec 20, 2022 · 9fc98b3 · 9fc98b3
1 parent 226faec
commit 9fc98b3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/rds/dbinstance/setup.go b/pkg/controller/rds/dbinstance/setup.go
@@ -484,13 +484,13 @@ func areVPCSecurityGroupIDsUpToDate(cr *svcapitypes.DBInstance, out *svcsdk.DBIn
 	// if user is fine with default SG or lets DBCluster manage it
 	// (removing all SGs is not possible, AWS will keep last set SGs)
 	if len(desiredIDs) == 0 {
-		return false
+		return true
 	}
 
 	actualGroups := out.VpcSecurityGroups
 
 	if len(desiredIDs) != len(actualGroups) {
-		return true
+		return false
 	}
 
 	actualIDs := make([]string, 0, len(actualGroups))