openshift-docs icon indicating copy to clipboard operation
openshift-docs copied to clipboard

#CNV-60399 - Clarify the supported Guest

Open Vaishali-gif-rh opened this issue 4 months ago • 2 comments
trafficstars

Version(s): 4.16+

Issue: https://issues.redhat.com/browse/CNV-60399

Link to docs preview: https://95208--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/managing_vms/virt-edit-vms.html

QE review:

  • [ ] QE has approved this change.

Additional information:

Vaishali-gif-rh avatar Jun 25 '25 12:06 Vaishali-gif-rh

🤖 Wed Jul 16 10:37:03 - Prow CI generated the docs preview:

https://95208--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/managing_vms/virt-edit-vms.html https://95208--ocpdocs-pr.netlify.app/openshift-rosa-hcp/latest/virt/managing_vms/virt-edit-vms.html https://95208--ocpdocs-pr.netlify.app/openshift-rosa/latest/virt/managing_vms/virt-edit-vms.html

ocpdocs-previewbot avatar Jun 25 '25 12:06 ocpdocs-previewbot

/label peer-review-needed

Vaishali-gif-rh avatar Jul 01 '25 15:07 Vaishali-gif-rh

/label peer-review-in-progress

aspauldi avatar Jul 02 '25 17:07 aspauldi

Hi Vaishali, just a few comments and a request that you provide a preview link to the procedure you've updated (in this case, the preview link would be https://95208--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/managing_vms/virt-edit-vms.html#virt-hot-plugging-memory_virt-edit-vms). Thanks!

aspauldi avatar Jul 02 '25 18:07 aspauldi

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

aspauldi avatar Jul 02 '25 18:07 aspauldi

/label merge-review-needed

Vaishali-gif-rh avatar Jul 03 '25 08:07 Vaishali-gif-rh

@vsibirsk, Perhaps this link is more appropriate?

cc: @Vaishali-gif-rh, @ctomasko

aspauldi avatar Jul 07 '25 12:07 aspauldi

@vsibirsk, Perhaps this link is more appropriate?

cc: @Vaishali-gif-rh, @ctomasko

unfortunately no.

this one is best i could find: https://access.redhat.com/articles/7090862 the requirements section there is what we need here

vsibirsk avatar Jul 07 '25 18:07 vsibirsk

Hi @vsibirsk, thanks for sharing the link. It looks like it’s for internal use and not accessible to customers. Just checking --do we need to include a link here? If yes, could you please point me to one that’s customer-accessible?

image

Vaishali-gif-rh avatar Jul 09 '25 11:07 Vaishali-gif-rh

@Vaishali-gif-rh: all tests passed!

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Jul 16 '25 10:07 openshift-ci[bot]

@vsibirsk - Thank you for your suggestion. I've incorporated the matrix from internal doc, let me know if that looks good.

Vaishali-gif-rh avatar Jul 16 '25 13:07 Vaishali-gif-rh

/label merge-review-needed

Vaishali-gif-rh avatar Jul 17 '25 14:07 Vaishali-gif-rh

/cherrypick enterprise-4.16

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz

@ShaunaDiaz: once the present PR merges, I will cherry-pick it on top of enterprise-4.16 in a new PR and assign it to you.

In response to this:

/cherrypick enterprise-4.16

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-sigs/prow repository.

/cherrypick enterprise-4.17

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz

@ShaunaDiaz: once the present PR merges, I will cherry-pick it on top of enterprise-4.17 in a new PR and assign it to you.

In response to this:

/cherrypick enterprise-4.17

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-sigs/prow repository.

/cherrypick enterprise-4.18

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz

@ShaunaDiaz: once the present PR merges, I will cherry-pick it on top of enterprise-4.18 in a new PR and assign it to you.

In response to this:

/cherrypick enterprise-4.18

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-sigs/prow repository.

/cherrypick enterprise-4.19

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz

@ShaunaDiaz: once the present PR merges, I will cherry-pick it on top of enterprise-4.19 in a new PR and assign it to you.

In response to this:

/cherrypick enterprise-4.19

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-sigs/prow repository.

/cherrypick enterprise-4.20

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz

@ShaunaDiaz: #95208 failed to apply on top of branch "enterprise-4.16":

Applying: added code
error: patch failed: modules/virt-hot-plugging-memory.adoc:20
error: modules/virt-hot-plugging-memory.adoc: patch does not apply
error: Did you hand edit your patch?
It does not apply to blobs recorded in its index.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Using index info to reconstruct a base tree...
A	modules/virt-hot-plugging-memory.adoc
Patch failed at 0001 added code

In response to this:

/cherrypick enterprise-4.16

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-sigs/prow repository.

@ShaunaDiaz: new pull request created: #96392

In response to this:

/cherrypick enterprise-4.20

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-sigs/prow repository.

@ShaunaDiaz: new pull request created: #96393

In response to this:

/cherrypick enterprise-4.17

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-sigs/prow repository.

@ShaunaDiaz: #95208 failed to apply on top of branch "enterprise-4.18":

Applying: added code
error: patch failed: modules/virt-hot-plugging-memory.adoc:20
error: modules/virt-hot-plugging-memory.adoc: patch does not apply
error: Did you hand edit your patch?
It does not apply to blobs recorded in its index.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Using index info to reconstruct a base tree...
M	modules/virt-hot-plugging-memory.adoc
Patch failed at 0001 added code

In response to this:

/cherrypick enterprise-4.18

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-sigs/prow repository.

@ShaunaDiaz: new pull request created: #96394

In response to this:

/cherrypick enterprise-4.19

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-sigs/prow repository.

@Vaishali-gif-rh You can create manual CPs for the failed picks as needed and resubmit, referencing this PR.

ShaunaDiaz avatar Jul 21 '25 12:07 ShaunaDiaz