yahoo-eng-team team mailing list archive

[Rodolfo Alonso <1967881@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

Port extension "uplink_status_propagation" enables the creation of a
port child register "portuplinkstatuspropagation". This register
contains the flag "propagate_uplink_status". This flag is set by default
as True when a port is created.

The problem this bug presents is with the existing ports before enabling
the extension. Those ports don't have any "portuplinkstatuspropagation"
register associated. The extension will populate the port dictionary
with the value of "propagate_uplink_status"; in case of not having this
flag, the value will be "False".

This bug proposes something similar to [2]. This patch set the default
value of the flag to "True" for new ports. In this case, the proposal
will affect existing ports **without** "portuplinkstatuspropagation"
registers: if the Neutron DB port does not have this flag, return a
"True" value. The aim of this proposal aligns with [2][3] goal, if an
admin enables this feature, it is because he/she wants to have this
feature enabled in all ports.


[1]https://github.com/openstack/neutron/blob/3615cd85a4cc6aeecf7f066c4eb21c3cdca71d4c/neutron/db/uplink_status_propagation_db.py#L31-L33
[2]https://review.opendev.org/c/openstack/neutron-lib/+/744208
[3]https://bugs.launchpad.net/neutron/+bug/1888487


Related bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2070629

** Affects: neutron
     Importance: Low
     Assignee: Rodolfo Alonso (rodolfo-alonso-hernandez)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => Rodolfo Alonso (rodolfo-alonso-hernandez)

** Changed in: neutron
   Importance: Undecided => Low

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

Title:
  [SR-IOV] Before enabling "uplink_status_propagation", ports are not
  created with this flag

Status in neutron:
  New

Bug description:
  Port extension "uplink_status_propagation" enables the creation of a
  port child register "portuplinkstatuspropagation". This register
  contains the flag "propagate_uplink_status". This flag is set by
  default as True when a port is created.

  The problem this bug presents is with the existing ports before
  enabling the extension. Those ports don't have any
  "portuplinkstatuspropagation" register associated. The extension will
  populate the port dictionary with the value of
  "propagate_uplink_status"; in case of not having this flag, the value
  will be "False".

  This bug proposes something similar to [2]. This patch set the default
  value of the flag to "True" for new ports. In this case, the proposal
  will affect existing ports **without** "portuplinkstatuspropagation"
  registers: if the Neutron DB port does not have this flag, return a
  "True" value. The aim of this proposal aligns with [2][3] goal, if an
  admin enables this feature, it is because he/she wants to have this
  feature enabled in all ports.

  
  [1]https://github.com/openstack/neutron/blob/3615cd85a4cc6aeecf7f066c4eb21c3cdca71d4c/neutron/db/uplink_status_propagation_db.py#L31-L33
  [2]https://review.opendev.org/c/openstack/neutron-lib/+/744208
  [3]https://bugs.launchpad.net/neutron/+bug/1888487

  
  Related bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2070629

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