Change the default value of "propagate_uplink_status" to True

[Rodolfo Alonso]

RFE [1] implemented a new port extension for SR-IOV ports. When this extension is enabled, the user can set this new flag "propagate_uplink_status" to True. That will set the VF link-state associated to this SR-IOV to "auto".

By default, without this extension, the link-state will be set to "enable" or "disable".

With this extension and "propagate_uplink_status" to False, the behavior will be the same ("enable"/"disable").

This bug is proposing to, by default, if this extension is enabled, to set "propagate_uplink_status" to True. That will set the link-state of the VF to "auto". That means the VF state will follow the PF state

NOTE: in some devices, the VF set to "enable" can transmit traffic when the PF is "down". That means if the VF is set to "auto", the VF won't be able to transmit. But, IMO, this behavior is more reasonable that allowing the VF to use the line when the PF is down.

[1]https://bugs.launchpad.net/neutron/+bug/1722720

[Slawek Kaplonski (slaweq)]

Lets treat this as new RFE and discuss on the drivers team meeting.

[Slawek Kaplonski (slaweq)]

This RFE was discussed and approved on today's drivers meeting. Thx for the proposal.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/744210

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/744208
Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=1f06863e2b906ea5f12620ad1ba49719b2d97f7e
Submitter: Zuul
Branch: master

commit 1f06863e2b906ea5f12620ad1ba49719b2d97f7e
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Fri Jul 31 15:58:20 2020 +0000

    Change "propagate_uplink_status" default value to True

    The API extension ``uplink-status-propagation`` has changed the default
    value of ``propagate_uplink_status`` from False to True. Now, when this
    extension is enabled, the VF link state will be set, by default, to
    "auto", following the PF link state.

    Change-Id: I594198741d0ff9df71b5c0053387d229c66154c1
    Partial-Bug: #1888487

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/744210
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d4ae3f75a542112c92130828f992eec17d5bf0ba
Submitter: Zuul
Branch: master

commit d4ae3f75a542112c92130828f992eec17d5bf0ba
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Fri Jul 31 15:33:47 2020 +0000

    Change "propagate_uplink_status" default value to True

    When "uplink-status-propagation" extension is enabled, new ports
    created will default the value of "propagate_uplink_status" to True.

    Closes-Bug: #1888487

    Change-Id: If1e533a61aeebbb4761d669c516fe86a4381765c

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/836656

[OpenStack Infra (hudson-openstack)]

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

commit 3a81b051d4a8e6f63f8da211816d0dbda62aee17
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Apr 5 13:48:05 2022 +0000

    [SR-IOV] Default "propagate_uplink_status" flag to True

    Extension "uplink-status-propagation" does not allow to modify existing
    ports. This extension only enables the creation of new ports with
    this new flag.

    Similar to [1], this patch changes the default behaviour of the
    exiting ports: if no "propagate_uplink_status" flag is present, "True"
    is returned now. The aim of this change is to enable this feature for
    all existing ports, that is usually the aim of an administrator when
    enables this extension.

    [1]https://bugs.launchpad.net/neutron/+bug/1888487

    Closes-Bug: #1967881
    Related-Bug: #1888487

    Change-Id: Ica5b76e0a9a5ae12f764c66be259d7f3cd5b248b