Public bug reported:

when restart ovs-agent, br-int lose flows ephemerally due to unnecessary
flow operation.

In _add_port_tag_info[1],type of lvm.vlan is int and
other_config.get('tag') is a str, they can never be equal. We should do
type conversion for lvm.vlan before comparing to avoid unnecessary
operation of ovsdb and flows.

[1]
https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1063-L1070

** Affects: neutron
     Importance: Undecided
         Status: New

** Description changed:

  when restart ovs-agent, br-int lose flows ephemerally due to unnecessary
  flow operation.
  
  In _add_port_tag_info[1],type of lvm.vlan is int and
  other_config.get('tag') is a str, they can never be equal. We should do
  type conversion for lvm.vlan before comparing to avoid unnecessary
  operation of ovsdb and flows.
  
  [1]
- 
https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1063
+ 
https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1063-L1070

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1843425

Title:
  br-int lose flows ephemerally due to unnecessary flow operation

Status in neutron:
  New

Bug description:
  when restart ovs-agent, br-int lose flows ephemerally due to
  unnecessary flow operation.

  In _add_port_tag_info[1],type of lvm.vlan is int and
  other_config.get('tag') is a str, they can never be equal. We should
  do type conversion for lvm.vlan before comparing to avoid unnecessary
  operation of ovsdb and flows.

  [1]
  
https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1063-L1070

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1843425/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to