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

CNV-33890: Swap feature in CNV RN 4.16

Open sbeskin-redhat opened this issue 1 year ago • 14 comments

Resolves: https://issues.redhat.com/browse/CNV-33888

OCP 4.16 CNV 4.16

Preview: https://77535--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/release_notes/virt-4-16-release-notes.html#virt-4-16-technology-preview

sbeskin-redhat avatar Jun 17 '24 10:06 sbeskin-redhat

@sbeskin-redhat: This pull request references CNV-33890 which is a valid jira issue.

In response to this:

Resolves: https://issues.redhat.com/browse/CNV-33888

OCP 4.16 CNV 4.16

Preview:

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Jun 17 '24 10:06 openshift-ci-robot

🤖 Fri Jul 05 14:27:54 - Prow CI generated the docs preview:

https://77535--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/release_notes/virt-4-16-release-notes.html

ocpdocs-previewbot avatar Jun 17 '24 10:06 ocpdocs-previewbot

/retest

sbeskin-redhat avatar Jun 17 '24 13:06 sbeskin-redhat

@sbeskin-redhat: This pull request references CNV-33890 which is a valid jira issue.

In response to this:

Resolves: https://issues.redhat.com/browse/CNV-33888

OCP 4.16 CNV 4.16

Preview: https://77535--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/release_notes/virt-4-16-release-notes.html#virt-4-16-technology-preview

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Jun 25 '24 10:06 openshift-ci-robot

@fabiand Sorry, I forgot to mention that this PR is only about the release notes: virt/release_notes/virt-4-16-release-notes.adoc The other files relating to the swap feature (virt-configuring-higher-vm-workload-density.adoc and virt-using-wasp-agent-to-configure-higher-vm-workload-density.adoc) are updated in another PR: https://github.com/openshift/openshift-docs/pull/76344

sbeskin-redhat avatar Jun 26 '24 14:06 sbeskin-redhat

/label peer-review-needed

sbeskin-redhat avatar Jul 03 '24 20:07 sbeskin-redhat

/label peer-review-in-progress

max-cx avatar Jul 04 '24 12:07 max-cx

/remove-label peer-review-needed

max-cx avatar Jul 04 '24 12:07 max-cx

@sbeskin-redhat, :slightly_smiling_face: I'm looking at the preview and it seems that you already added the prerequisites but still need to add a procedure. Also, I see the previous reviewer's comments that have not been addressed or responded to. I also recommend consulting the repo rules files such as how to document TP features.

:slightly_smiling_face: AFAIK, we normally get peer reviews when we feel our authoring work on the PR is completed, so I suggest that you add the missing procedure, get this PR to a point where you believe that it's ready to be merged, and ask the QE to review and approve it. Also, please remember to squash the commits. And then add the peer-review-needed label.

Since I'm only onboarding for the peer review squad, let's ask for a second opinion or additional feedback from @dfitzmau.

max-cx avatar Jul 05 '24 11:07 max-cx

/remove-label peer-review-in-progress

max-cx avatar Jul 05 '24 11:07 max-cx

Sorry, I forgot to mention that this PR is only about the release notes: virt/release_notes/virt-4-16-release-notes.adoc The other files relating to the swap feature (virt-configuring-higher-vm-workload-density.adoc and virt-using-wasp-agent-to-configure-higher-vm-workload-density.adoc) are updated in another PR: #76344

@sbeskin-redhat, :slightly_smiling_face: in such a case, please remove the changes that should not be merged with this PR.

max-cx avatar Jul 05 '24 11:07 max-cx

/label peer-review-in-progress

dfitzmau avatar Jul 05 '24 11:07 dfitzmau

Hi @sbeskin-redhat . I agree with Max. The commits need to be squashed, non-intended files should be reverted, and the merge conflict(s) need to be addressed.

dfitzmau avatar Jul 05 '24 11:07 dfitzmau

/remove-label peer-review-in-progress

dfitzmau avatar Jul 05 '24 11:07 dfitzmau

@dfitzmau Hi, Squashed, rebased, resolved

sbeskin-redhat avatar Jul 05 '24 12:07 sbeskin-redhat

@sbeskin-redhat: 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 05 '24 14:07 openshift-ci[bot]

/label merge-review-needed

sbeskin-redhat avatar Jul 05 '24 14:07 sbeskin-redhat

@sbeskin-redhat: This pull request references CNV-33890 which is a valid jira issue.

In response to this:

Resolves: https://issues.redhat.com/browse/CNV-33890

OCP 4.16 CNV 4.16

Preview: https://77535--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/release_notes/virt-4-16-release-notes.html#virt-4-16-technology-preview

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 openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Jul 07 '24 17:07 openshift-ci-robot