Avoid conflicts in updateSnapshotContentStatus

This patch replace the usage of `UpdateStatus` with a patch against the
`status` subresource in the `updateSnapshotContentStatus`.

The reason behind this change is to avoid conflicts that could
potentially arise, as they pose a risk within the context of the calling
function `createGroupSnapshotWrapper`.

If `createGroupSnapshotWrapper` is called multiple times for the same
GroupSnapshot, this will lead to multiple stale `VolumeSnapshot` and
`VolumeSnapshotContent` being created.

Related to: #1050

Author: leonardoce

pkg/sidecar-controller/snapshot_controller.go

@@ -484,8 +484,16 @@ func (ctrl *csiSnapshotSideCarController) updateSnapshotContentStatus(
 
 	if updated {
 		contentClone := contentObj.DeepCopy()
-		contentClone.Status = newStatus
-		newContent, err := ctrl.clientset.SnapshotV1().VolumeSnapshotContents().UpdateStatus(context.TODO(), contentClone, metav1.UpdateOptions{})
+
+		patches := []utils.PatchOp{
+			{
+				Op:    "replace",
+				Path:  "/status",
+				Value: newStatus,
+			},
+		}
+
+		newContent, err := utils.PatchVolumeSnapshotContent(contentClone, patches, ctrl.clientset, "status")
 		if err != nil {
 			return contentObj, newControllerUpdateError(content.Name, err.Error())
 		}