-
Epic
-
Resolution: Done
-
Major
-
None
Goal
Allow administrators to create new Network Attachment Definitions for OVN Kubernetes secondary localnet networks.
Non-Requirements
- <List of things not included in this epic, to alleviate any doubt raised during the grooming process.>
Notes
- This should be a new item in the dropdown menu of NetworkAttachmentDefinition dialog. When selecting "OVN Kubernetes secondary localnet network" two additional fields should show up: "Bridge mapping", MTU (optional) and "VLAN (optional)". The "Bridge mapping" attribute should have a tooltip explaining what it is. That should render into:
apiVersion: k8s.cni.cncf.io/v1 kind: NetworkAttachmentDefinition metadata: name: <name> namespace: <namespace> spec: config: |2 { "cniVersion": "0.4.0", "name": "<bridge mapping>", "type": "ovn-k8s-cni-overlay", "topology":"localnet", "vlanID": <VLAN>, # set only if passed from the user "mtu": <MTU>, # set only if passed from the user "netAttachDefName": "<namespace>/<name>" }
- Tooltip text: "Reference to a physical network name. A bridge mapping must be configured on cluster nodes to map between physical network names and Open vSwitch bridges."
- Draft of the localnet documentation https://github.com/openshift/openshift-docs/blob/0fe4ec0bcc31afd4ba5060fb87dc9c347da34603/modules/configuring-localnet-switched-topology.adoc#L92
Done Checklist
- DEV - Upstream code and tests merged: https://github.com/openshift/console/pull/13293
- QE - Test plan: https://polarion.engineering.redhat.com/polarion/#/project/CNV/workitem?id=CNV-10548
- QE - Test automation: https://gitlab.cee.redhat.com/cnv-qe/kubevirt-ui/-/merge_requests/539
1.
|
upstream roadmap issue | Closed | Unassigned | ||
2.
|
upstream documentation | Closed | Unassigned | ||
3.
|
upgrade consideration | Closed | Unassigned | ||
4.
|
CEE/PX summary presentation | Closed | Unassigned | ||
5.
|
test plans in polarion | Closed | Guohua Ouyang | ||
6.
|
automated tests | Closed | Guohua Ouyang | ||
7.
|
[docs] Review UX design | Closed | Avital Pinnick |