Uploaded image for project: 'Fast Datapath Product'
  1. Fast Datapath Product
  2. FDP-38

[OVN] setting tag_request to 0 is causing wrong calculation for check_pkt_larger

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • None
    • ovn23.09
    • False
    • Hide

      None

      Show
      None
    • False

      OVNK currently sets vlan ID to 0 on br-ex switch ports:

      Example:

      _uuid               : 8586d9ff-f40e-43ae-a2d6-b0d08f09467e                                                                                                                   
      addresses           : [unknown]
      dhcpv4_options      : []
      dhcpv6_options      : []
      dynamic_addresses   : []
      enabled             : []
      external_ids        : {}
      ha_chassis_group    : []
      mirror_rules        : []
      name                : br-ex_master-0.sfdc03602291.lab.upshift.rdu2.redhat.com                                                                                                
      options             : {network_name=physnet}                                                                                                                                 
      parent_name         : []
      port_security       : []
      tag                 : []
      tag_request         : 0
      type                : localnet
      up                  : false
       

      This causes OVN to do funny things:

      sh-4.4# ovn-sbctl --no-leader-only lflow-list | grep chk_pkt_                                                                                                                
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoe-GR_master-0.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1418); ne
      xt;)                                                                                                                                                                         
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoj-GR_master-0.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1414); ne
      xt;)                                                                                                                                                                         
        table=18(lr_in_chk_pkt_len  ), priority=0    , match=(1), action=(next;)                                                                                                   
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoe-GR_master-1.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1418); ne
      xt;)                                                                                                                                                                         
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoj-GR_master-1.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1414); ne
      xt;)                                                                                                                                                                         
        table=18(lr_in_chk_pkt_len  ), priority=0    , match=(1), action=(next;)                                                                                                   
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoe-GR_master-2.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1418); ne
      xt;)                                                                                                                                                                         
        table=18(lr_in_chk_pkt_len  ), priority=50   , match=(outport == "rtoj-GR_master-2.sfdc03602291.lab.upshift.rdu2.redhat.com"), action=(reg9[1] = check_pkt_larger(1414); ne
      xt;)                                                                           

      OVN should be smarter to check the value and not auto-calculate things based on whether value is empty list or not.

            lorenzobianconi lorenzo bianconi
            sseethar Surya Seetharaman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: