OADP-935: Incorrect Backup apiVersion in "Using Data Mover for CSI snapshots"

[RichardHoch]

OADP 1.0.6; OCP 4.9+

Bug reported by QE; PR approived by QE.

Resolves https://issues.redhat.com/browse/OADP-935 by changing the Backup apiVersion in the Back GR codeblock in "Using Data Mover for CSI snapshots."

Preview: See step 3a in https://53198--docspreview.netlify.app/openshift-enterprise/latest/backup_and_restore/application_backup_and_restore/backing_up_and_restoring/backing-up-applications.html#oadp-using-data-mover-for-csi-snapshots_backing-up-applications

[ocpdocs-previewbot]

🤖 Build preview is available at:
https://53198--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/4135

[PrasadJoshi12]

/lgtm

[openshift-ci]

@PrasadJoshi12: changing LGTM is restricted to collaborators

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[RichardHoch]

/label OADP
/label peer-review-needed

[Wiharris]

/label peer-review-in-progress

[Wiharris]

/LGTM

[Wiharris]

/remove-label peer-review-in-progress
/remove-label peer-review-needed
/label peer-review-done

[RichardHoch]

/oadp
/merge-review-needed [I am double-checking on when to merge when this is good to go]

[RichardHoch]

@weshayutin Please merge on Monday. Tuesday is OK, too.

[weshayutin]

@RichardHoch I do not have merge rights here.

[mjpytlak]

@RichardHoch Handling this now. Just a reminder. We are no longer updating documentation for 4.6 and 4.7. Those versions are EOL. I will merge this into 4.8+. I updated the first comment in the PR to reflect that.

[mjpytlak]

/cherrypick enterprise-4.12

[mjpytlak]

/cherrypick enterprise-4.11

[mjpytlak]

/cherrypick enterprise-4.10

[mjpytlak]

/cherrypick enterprise-4.9

[mjpytlak]

/cherrypick enterprise-4.8

[openshift-cherrypick-robot]

@mjpytlak: new pull request created: #53467

In response to this:

/cherrypick enterprise-4.12

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@mjpytlak: new pull request created: #53468

In response to this:

/cherrypick enterprise-4.11

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@mjpytlak: new pull request created: #53469

In response to this:

/cherrypick enterprise-4.10

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@mjpytlak: new pull request created: #53470

In response to this:

/cherrypick enterprise-4.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-cherrypick-robot]

@mjpytlak: #53198 failed to apply on top of branch "enterprise-4.8":

Applying: OADP-935: Incorrect Backup apiVersion in Using Data Mover for CSI snapshots
Using index info to reconstruct a base tree...
A	modules/oadp-using-data-mover-for-csi-snapshots.adoc
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): modules/oadp-using-data-mover-for-csi-snapshots.adoc deleted in HEAD and modified in OADP-935: Incorrect Backup apiVersion in Using Data Mover for CSI snapshots. Version OADP-935: Incorrect Backup apiVersion in Using Data Mover for CSI snapshots of modules/oadp-using-data-mover-for-csi-snapshots.adoc left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 OADP-935: Incorrect Backup apiVersion in Using Data Mover for CSI snapshots
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick enterprise-4.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[mjpytlak]

@RichardHoch I believe the CP back to 4.8 failed because the module you updated does not exist in 4.8. I believe this is what CONFLICT (modify/delete): modules/oadp-using-data-mover-for-csi-snapshots.adoc deleted in HEAD and modified in [OADP-935](https://issues.redhat.com//browse/OADP-935): Incorrect Backup apiVersion in Using Data Mover for CSI snapshots. Version [OADP-935](https://issues.redhat.com//browse/OADP-935): Incorrect Backup apiVersion in Using Data Mover for CSI snapshots of modules/oadp-using-data-mover-for-csi-snapshots.adoc left in tree. error: Failed to merge in the changes. means.

Please verify that the module does exist in 4.8.

[RichardHoch]

@mjpytlak Thanks! Double-checking with the SME that we do want this for 4.8 (4.9+ may be enough). If so, will check on the module. If needed, I'll CP and submit the PR. If not, I'll let you know.

[mjpytlak]

Confirmed that this CP to 4.8 failed because when the original author of the module added to the doc doc, they added it to 4.9+ only. [1]

I have spoken to @RichardHoch and he will double check with the SME and the original author to determine if this content should live in 4.8, as well. If so, Richard will open a separate PR to track this work.

I have removed the enterprise-4.8 label and updated the first comment of the PR to indicate that this update applies to 4.9+ only.

[1] #48619

[RichardHoch]

@mjpytlak Thanks for the great advice and support!