Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-59629

Localnet UI dialog should set an explicit MTU

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • CNV v4.21.0
    • None
    • CNV User Interface
    • None
    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • False
    • None
    • Important
    • None

      Description of problem:

      The implicit default for MTU with OVN localnet is 1400. That is counterintuitive for network-savvy users who expect the default to be 1500. That leads to situations when a customer observes packet loss due to fragmentation, but is unable to spot an invalid configuration.

      Version-Release number of selected component (if applicable):

      4.15+ (not entirely sure when was the localnet UI introduced)

      How reproducible:

      Always

      Steps to Reproduce:

      1. Create a new NAD with the localnet topology, without setting any MTU
      2. Look at the YAML

      Actual results:

      ...
      spec:
        config: |-
          {
              "cniVersion": "0.4.0",
              "name": "physname",
              "type": "ovn-k8s-cni-overlay",
              "netAttachDefName": "phoracek/net1",
              "topology": "localnet"
          }
      

      Expected results:

      ...
      spec:
        config: |-
          {
              "cniVersion": "0.4.0",
              "name": "physname",
              "type": "ovn-k8s-cni-overlay",
              "netAttachDefName": "phoracek/net1",
              "mtu": 1500,
              "topology": "localnet"
          }

      Additional info:

      The UI should require the MTU attribute to be set explicitly, defaulting it text field to 1500.

              mschatzm@redhat.com Matan Schatzman
              phoracek@redhat.com Petr Horacek
              Leon Kladnitsky Leon Kladnitsky
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated: