Bug 1874084 - List only a subset of NIC models
Summary: List only a subset of NIC models
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Documentation
Version: 2.4.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.5.0
Assignee: Andrew Burden
QA Contact: Yossi Segev
URL:
Whiteboard:
Depends On: 1869620
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-08-31 14:16 UTC by Petr Horáček
Modified: 2020-12-07 09:51 UTC (History)
3 users (show)

Fixed In Version: 2.5.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-12-07 09:51:46 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

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.
Note You need to log in before you can comment on or make changes to this bug.