Document URL: https://cnv_setup--ocpdocs.netlify.com/openshift-enterprise/latest/cnv_install/cnv_install.html Section Number and Name: "Installing CNV" Describe the issue: There should be a new section to create the kubevirt config to open required feature gates. Suggestions for improvement: Rename "Installing the Components Manifest" to "Installing KubeVirt manifests" Add a new step (as step 1) to the procedure: """ 1. Create the KubeVirt configuration $ oc create configmap -n kube-system kubevirt-config --from-literal="feature-gates=DataVolumes, CPUManager" configmap "kubevirt-config" created """ Additional information:
Thanks Fabian, here's the commit where I added this: https://github.com/openshift/openshift-docs/pull/12518/commits/e8ec2b69dd913302c3aa774fa6f001ee6df4113d Please verify the change, thanks again!
Looks good!
per discussion with fabian & kedar, cpumanager shouldnt be enabled Pan, could you please remove it from the doc?
Thus instead of: """ 1. Create the KubeVirt configuration $ oc create configmap -n kube-system kubevirt-config --from-literal="feature-gates=DataVolumes, CPUManager" configmap "kubevirt-config" created """ This: """ 1. Create the KubeVirt configuration $ oc create configmap -n kube-system kubevirt-config --from-literal="feature-gates=DataVolumes" configmap "kubevirt-config" created """
https://github.com/openshift/openshift-docs/pull/12518/commits/7659c6ebf97638ce40345ab0fc54e7175d0a9f6b I made Fabian's change in the above commit. Moving back to ON_QA - thanks all!
LGTM
i dont see it on the official doc though
Fabian, please confirm that this BZ is not applicable to APB, and we can close it then. Thank you.
this is part of the APB deployment, we dont need to manually enable it
(In reply to Nelly Credi from comment #7) > i dont see it on the official doc though Yes, I ended up removing that section when we moved away from manifests. :) Thanks!