Bug 1874084

Summary:	List only a subset of NIC models
Product:	Container Native Virtualization (CNV)	Reporter:	Petr Horáček <phoracek>
Component:	Documentation	Assignee:	Andrew Burden <aburden>
Status:	CLOSED ERRATA	QA Contact:	Yossi Segev <ysegev>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	2.4.0	CC:	aburden, cnv-qe-bugs, ncredi
Target Milestone:	---
Target Release:	2.5.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	2.5.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-12-07 09:51:46 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1869620
Bug Blocks:

Description Petr Horáček 2020-08-31 14:16:53 UTC

Document URL: 
https://docs.openshift.com/container-platform/4.5/virt/virtual_machines/vm_networking/virt-attaching-vm-multiple-networks.html#virt-networking-wizard-fields-web_virt-attaching-multiple-networks

Section Number and Name: 
Networking fields

Describe the issue: 
We are listing all the possible options for NIC models, while we should we limit it to a subset that reflects what we recommend and will to support.

Suggestions for improvement: 
Complement https://bugzilla.redhat.com/show_bug.cgi?id=1869620 by limiting the list to e1000e and virtio.

Additional information:

Comment 1 Andrew Burden 2020-09-03 16:09:10 UTC

Removed the unsupported NIC types from the Networking wizard table

PR: https://github.com/openshift/openshift-docs/pull/25255

Comment 3 Andrew Burden 2020-09-07 09:45:26 UTC

Sorry Nelly, I updated the PR to say enterprise-4.6 only but not the bug.
The UI bug (now added to 'Depends on') is targeted for 4.6

Comment 4 Yossi Segev 2020-11-05 19:41:50 UTC

Reviewed and added one change request in the PR https://github.com/openshift/openshift-docs/pull/25255#pullrequestreview-524605615.

Comment 5 Andrew Burden 2020-11-06 09:54:05 UTC

Thanks Yossi, updated and merged.

Comment 6 Yossi Segev 2020-11-08 07:49:06 UTC

Verified by CNV Network QE.