Skip to content

Conversation

@sjhala-ccs
Copy link
Contributor

@sjhala-ccs sjhala-ccs commented Jul 24, 2020

BZ-1857788 https://bugzilla.redhat.com/show_bug.cgi?id=1857788

Added a note to remove evictionStrategy: LiveMigrate and use runStrategy: Always for VMs using hostpath-provisioner or SR-IOV network interfaces as a workaround for cluster upgrade issue

Preview build: https://bz-1857788--ocpdocs.netlify.app/openshift-enterprise/latest/virt/upgrading-virt.html#how-openshift-virtualization-upgrades-affect-your-cluster

Tagging @aglitke for SME review
Tagging @rnetser for QE review

Merge/CP to enterprise-4.5 and enterprise-4.6

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 24, 2020
@openshift-docs-preview-bot

The preview will be available shortly at:

@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 27, 2020
@sjhala-ccs sjhala-ccs force-pushed the BZ-1857788 branch 2 times, most recently from bfc7fd2 to a9f574d Compare July 28, 2020 15:39
@rnetser
Copy link

rnetser commented Aug 12, 2020

/lgtm

@openshift-ci-robot
Copy link

@rnetser: changing LGTM is restricted to collaborators

Details

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.

Copy link
Member

@ousleyp ousleyp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This mostly looks great but I have one very minor fix and a couple of nitpicky suggestions. Take 'em or leave 'em. :) Thanks, Shikha!

@ousleyp ousleyp added branch/enterprise-4.5 branch/enterprise-4.6 CNV Label for all CNV PRs peer-review-done Signifies that the peer review team has reviewed this PR labels Aug 12, 2020
@ousleyp ousleyp added this to the Next Release milestone Aug 12, 2020
@openshift-ci-robot openshift-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 13, 2020
@ousleyp ousleyp merged commit f5db616 into openshift:master Aug 13, 2020
@ousleyp
Copy link
Member

ousleyp commented Aug 13, 2020

/cherrypick enterprise-4.5

@ousleyp
Copy link
Member

ousleyp commented Aug 13, 2020

/cherrypick enterprise-4.6

@openshift-cherrypick-robot

@ousleyp: new pull request created: #24801

Details

In response to this:

/cherrypick enterprise-4.5

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

@ousleyp: new pull request created: #24802

Details

In response to this:

/cherrypick enterprise-4.6

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.

@sjhala-ccs sjhala-ccs deleted the BZ-1857788 branch August 13, 2020 18:01
wking added a commit to wking/openshift-docs that referenced this pull request Oct 8, 2020
xrefs aren't allowed in modules:

  $ grep -A1 'You must not include xrefs' modules/mod-docs-ocp-conventions.adoc
  You must not include xrefs in modules or create an xref to a module. You can
  only use xrefs to link from one assembly to another.

But this module only has a single consumer:

  $ git --no-pager grep 'include.*virt-about-upgrading-virt' origin/master
  origin/master:virt/upgrading-virt.adoc:include::modules/virt-about-upgrading-virt.adoc[leveloffset=+1]

So shift it into the virt/ assembly to allow the xref added in
87623ba (BZ-1857788 Added note about upgrade workaround for VMs
using HPP, 2020-07-24, openshift#24172).
This includes virtual machines that use hostpath provisioner storage or SR-IOV network interfaces.
As a workaround, you can reconfigure the virtual machines so that they can be powered off automatically during a cluster upgrade. Remove the `evictionStrategy: LiveMigrate` field and
set the `runStrategy` field to `Always`. Learn more about xref:../virt/live_migration/virt-configuring-vmi-eviction-strategy.adoc#virt-configuring-vmi-eviction-strategy[configuring virtual machine eviction strategy].
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ousleyp hey - reminder to check for xrefs in modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.5 branch/enterprise-4.6 CNV Label for all CNV PRs peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants