Bug 1937663 - [DOCs][CNV] Live migration limits and timeouts
Summary: [DOCs][CNV] Live migration limits and timeouts
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Documentation
Version: 2.5.4
Hardware: All
OS: All
unspecified
medium
Target Milestone: ---
: ---
Assignee: Sabrina Jess
QA Contact: Oded Ramraz
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-03-11 09:48 UTC by Mustafa Aydın
Modified: 2021-05-06 20:42 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-05-06 20:42:26 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Mustafa Aydın 2021-03-11 09:48:12 UTC
Document URL: 
https://docs.openshift.com/container-platform/4.6/virt/live_migration/virt-live-migration-limits.html
Section Number and Name: 
Configuring live migration limits and timeouts
Describe the issue: 
The example references to the kubevirt namespace which is not available and also references to the livemigration as the featuregate. 

Suggestions for improvement: 
Please test this configuration example and put the correct example

Additional information:

Comment 1 Mustafa Aydın 2021-03-27 07:58:17 UTC
Please ignore the "livemigration as featuregate" part, seems like it is supposed to be at the configmap.

Comment 3 Sabrina Jess 2021-04-16 17:31:15 UTC
@jhopper You recently assisted with other edits on this page, any chance you can help with updating the code sample on this page?

Comment 4 Jenifer Abrams 2021-04-16 18:13:47 UTC
(In reply to Sabrina Jess from comment #3)
> @jhopper You recently assisted with other edits on this page, any
> chance you can help with updating the code sample on this page?

Hi, 
Ah yes good catch on the namespace. The default Openshift Virt configmap does look a bit different, this example is from a v2.6 install -- users would then add the migration params as the docs describe: 

#  oc edit configmap kubevirt-config -n openshift-cnv

apiVersion: v1
data:
  default-network-interface: masquerade
  feature-gates: DataVolumes,SRIOV,LiveMigration,CPUManager,CPUNodeDiscovery,Sidecar,Snapshot
  machine-type: pc-q35-rhel8.3.0
  selinuxLauncherType: virt_launcher.process
  smbios: |-
    Family: Red Hat
    Product: Container-native virtualization
    Manufacturer: Red Hat
    Sku: 2.6.0
    Version: 2.6.0
kind: ConfigMap
metadata:
  creationTimestamp: "2021-03-26T18:01:04Z"
  labels:
    app: kubevirt-hyperconverged
  name: kubevirt-config
  namespace: openshift-cnv
  resourceVersion: "15371295"
  selfLink: /api/v1/namespaces/openshift-cnv/configmaps/kubevirt-config
  uid: <uuid>

Comment 6 Sabrina Jess 2021-04-19 17:56:51 UTC
@jhopper Thank you! Please take a look at the preview here: https://github.com/openshift/openshift-docs/pull/31680 and update BZ Status to VERIFIED if it looks good to go.

Comment 7 Jenifer Abrams 2021-04-20 14:22:47 UTC
The default configmap values all look good now, but the example migration section was removed. You can add the sample migration block under the existing feature-gates line:

  feature-gates: DataVolumes,SRIOV,LiveMigration,CPUManager,CPUNodeDiscovery,Sidecar,Snapshot
  migrations: |-
    parallelMigrationsPerCluster: "5"
    parallelOutboundMigrationsPerNode: "2"
    bandwidthPerMigration: "64Mi"
    completionTimeoutPerGiB: "800"
    progressTimeout: "150"

Comment 8 Jenifer Abrams 2021-04-20 18:58:06 UTC
Latest change looks good, setting to VERIFIED.


Note You need to log in before you can comment on or make changes to this bug.