← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1949081] Re: [OVN] check_for_mcast_flood_reports maintenance task not accounting for localnet "mcast_flood" changes

 

Reviewed:  https://review.opendev.org/c/openstack/neutron/+/815843
Committed: https://opendev.org/openstack/neutron/commit/d02fb560aa35c4931d40000e8ca4c0e4105299b7
Submitter: "Zuul (22348)"
Branch:    master

commit d02fb560aa35c4931d40000e8ca4c0e4105299b7
Author: Lucas Alvares Gomes <lucasagomes@xxxxxxxxx>
Date:   Thu Oct 28 14:17:20 2021 +0100

    [OVN] Update check_for_mcast_flood_reports() to check for mcast_flood
    
    The patch [0] changed the value of the "mcast_flood" option for the
    localnet ports in OVN to "false" to avoid multicast packet duplication
    but, the maintenance task check_for_mcast_flood_reports() was never
    updated to account for that change.
    
    So, localnet ports created prior to [0] would continue to have the
    "mcast_flood" value as "true" and new ports would have it as "false".
    This patch fixes the maintenance task.
    
    [0] https://review.opendev.org/c/openstack/neutron/+/797418
    
    Change-Id: Ib90704a85c9ed659d71ac8ed42b1b542bd2496a2
    Closes-Bug: #1949081
    Signed-off-by: Lucas Alvares Gomes <lucasagomes@xxxxxxxxx>


** Changed in: neutron
       Status: In Progress => Fix Released

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

Title:
  [OVN] check_for_mcast_flood_reports maintenance task not accounting
  for localnet "mcast_flood" changes

Status in neutron:
  Fix Released

Bug description:
  The patch [0] changed the value of the "mcast_flood" option for the
  localnet ports in OVN to "false" to avoid multicast packet duplication
  but, the maintenance task check_for_mcast_flood_reports was never
  updated to account for that change.

  Current the maintenance task only checks if the mcast_flood option is
  set and if so it moves on to the next port. This means that localnet
  ports created prior to [1] will continue to have the mcast_flood value
  as "true".

  We need to update check_for_mcast_flood_reports to change the value of
  mcast_flood to "false" for the localnet ports in case they are unset
  or set to "true")

  [1] https://review.opendev.org/c/openstack/neutron/+/797418

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



References