Record correct OperatorGroup in status

It seems like this was a typo in the original implementation - the
controller correctly updates the resource to match the "merged" version,
but was then using the "desired" version of the resource (which only
specifies what parts the user was interested in, and does not reflect
the full live state) when updating the status of the policy.

The updated section in `musthaveOpGroup` now more closely matches the
analogous section in `musthaveSubscription`.

Signed-off-by: Justin Kulikauskas <[email protected]>

Author: JustinKuli

controllers/operatorpolicy_controller.go

@@ -1183,18 +1183,16 @@ func (r *OperatorPolicyReconciler) musthaveOpGroup(
 			earlyConds = append(earlyConds, calculateComplianceCondition(policy))
 		}
 
-		desiredOpGroup.ResourceVersion = opGroup.GetResourceVersion()
-
 		opLog.Info("Updating OperatorGroup to match desired state", "opGroupName", opGroup.GetName())
 
 		err = r.TargetClient.Update(ctx, &opGroup)
 		if err != nil {
 			return false, nil, changed, fmt.Errorf("error updating the OperatorGroup: %w", err)
 		}
 
-		desiredOpGroup.SetGroupVersionKind(operatorGroupGVK) // Update stripped this information
+		opGroup.SetGroupVersionKind(operatorGroupGVK) // Update stripped this information
 
-		updateStatus(policy, updatedCond("OperatorGroup"), updatedObj(desiredOpGroup))
+		updateStatus(policy, updatedCond("OperatorGroup"), updatedObj(&opGroup))
 
 		return true, earlyConds, true, nil
 	default: