enabling uplink_status_propagation ML2 extension is not supported

Bug #1949491 reported by Przemyslaw Lal
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Charm Guide
Fix Released
Undecided
Diko Parvanov
OpenStack Neutron API Charm
Fix Released
Undecided
Diko Parvanov
Train
In Progress
Undecided
Danny Cocks
Ussuri
In Progress
Undecided
Danny Cocks
Victoria
In Progress
Undecided
Danny Cocks
Wallaby
In Progress
Undecided
Danny Cocks
Xena
In Progress
Undecided
Danny Cocks

Bug Description

The ML2 extension "uplink_status_propagation" allows to configure SR-IOV VFs in a way that makes them to follow their PFs link state instead of enforcing their link state to be always UP.

Currently, this config option isn't exposed in any way in OVN or neutron charms.

Feature that we'd like to see configurable through charm config was developed as part of [0].

Example use case: bonding of VF interfaces inside a VM. When both VFs have a "link-state: enable" (the only possible configuration with current limitations), one will not get the benefit of bonding when the PF link's state goes down (e.g. network switch failure). The instance won't be aware of that fact as both VFs will be reported as UP, and will send 50% of network traffic into a black hole.

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

Revision history for this message
Przemyslaw Lal (przemeklal) wrote :

Tests confirmed that neutron-api config change is enough to make this work cluster-wide.

no longer affects: charm-ovn-chassis
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (master)
Changed in charm-neutron-api:
status: New → In Progress
Revision history for this message
Aurelien Lourot (aurelien-lourot) wrote :

Added charm-guide so we write an entry to the release notes

Changed in charm-neutron-api:
status: In Progress → Fix Committed
milestone: none → 22.04
Changed in charm-guide:
status: New → In Progress
assignee: nobody → Diko Parvanov (dparv)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-neutron-api (master)

Reviewed: https://review.opendev.org/c/openstack/charm-neutron-api/+/818750
Committed: https://opendev.org/openstack/charm-neutron-api/commit/3e387d591869b714be1c323f240fcaff289b3c10
Submitter: "Zuul (22348)"
Branch: master

commit 3e387d591869b714be1c323f240fcaff289b3c10
Author: Diko Parvanov <email address hidden>
Date: Mon Nov 22 12:24:53 2021 +0200

    Adding uplink_status_propagation option

    Add charm option uplink-status-propagation, which for releases >= stein
    will add the ml2 plugin uplink_status_propagation, that enabled ports to
    have the propagate_uplink_status attribute to allow instance VF to see the PF
    link state change for SR-IOV ports.

    Change-Id: I4f1ace630c74363057b0bbdbc759c482125ad015
    Closes-Bug: 1949491

Diko Parvanov (dparv)
Changed in charm-guide:
status: In Progress → Fix Committed
Changed in charm-neutron-api:
assignee: nobody → Diko Parvanov (dparv)
Changed in charm-neutron-api:
status: Fix Committed → Fix Released
Changed in charm-guide:
status: Fix Committed → Fix Released
Revision history for this message
Felipe Reyes (freyes) wrote :

Targeting this bug to the stable releases based on the need expressed at https://bugs.launchpad.net/charm-neutron-api/+bug/2045215

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902340

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902341

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902342

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (stable/victoria)

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902343

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-neutron-api (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902344

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-neutron-api (stable/train)

Change abandoned by "Elod Illes <email address hidden>" on branch: stable/train
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902341
Reason: Train is about to transition to End of Life. Open patches needs to be abandoned before branch deletion.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-neutron-api (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/charm-neutron-api/+/902340
Committed: https://opendev.org/openstack/charm-neutron-api/commit/6f303256517588d9e1ac530150d419c7f8573fd8
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 6f303256517588d9e1ac530150d419c7f8573fd8
Author: Diko Parvanov <email address hidden>
Date: Mon Nov 22 12:24:53 2021 +0200

    Adding uplink_status_propagation option

    Add charm option uplink-status-propagation, which for releases >= stein
    will add the ml2 plugin uplink_status_propagation, that enabled ports to
    have the propagate_uplink_status attribute to allow instance VF to see the PF
    link state change for SR-IOV ports.

    Change-Id: I4f1ace630c74363057b0bbdbc759c482125ad015
    Partial-Bug: 1949491
    (cherry picked from commit 3e387d591869b714be1c323f240fcaff289b3c10)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-neutron-api (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/charm-neutron-api/+/902344
Committed: https://opendev.org/openstack/charm-neutron-api/commit/11a8d64774b81b358211792e11a62ca66aaed1ab
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 11a8d64774b81b358211792e11a62ca66aaed1ab
Author: Diko Parvanov <email address hidden>
Date: Mon Nov 22 12:24:53 2021 +0200

    Adding uplink_status_propagation option

    Add charm option uplink-status-propagation, which for releases >= stein
    will add the ml2 plugin uplink_status_propagation, that enabled ports to
    have the propagate_uplink_status attribute to allow instance VF to see the PF
    link state change for SR-IOV ports.

    Change-Id: I4f1ace630c74363057b0bbdbc759c482125ad015
    Partial-Bug: 1949491
    (cherry picked from commit 6f303256517588d9e1ac530150d419c7f8573fd8)

tags: added: in-stable-wallaby
tags: added: in-stable-victoria
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-neutron-api (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/charm-neutron-api/+/902343
Committed: https://opendev.org/openstack/charm-neutron-api/commit/6243b39c150cb0d386730b61db5b5c3d15423a47
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 6243b39c150cb0d386730b61db5b5c3d15423a47
Author: Diko Parvanov <email address hidden>
Date: Mon Nov 22 12:24:53 2021 +0200

    Adding uplink_status_propagation option

    Add charm option uplink-status-propagation, which for releases >= stein
    will add the ml2 plugin uplink_status_propagation, that enabled ports to
    have the propagate_uplink_status attribute to allow instance VF to see the PF
    link state change for SR-IOV ports.

    Change-Id: I4f1ace630c74363057b0bbdbc759c482125ad015
    Partial-Bug: 1949491
    (cherry picked from commit 3e387d591869b714be1c323f240fcaff289b3c10)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-neutron-api (stable/ussuri)

Reviewed: https://review.opendev.org/c/openstack/charm-neutron-api/+/902342
Committed: https://opendev.org/openstack/charm-neutron-api/commit/7dca5dd22a664e2c161c0d4f6e5dbef8b1d10893
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 7dca5dd22a664e2c161c0d4f6e5dbef8b1d10893
Author: Diko Parvanov <email address hidden>
Date: Mon Nov 22 12:24:53 2021 +0200

    Adding uplink_status_propagation option

    Add charm option uplink-status-propagation, which for releases >= stein
    will add the ml2 plugin uplink_status_propagation, that enabled ports to
    have the propagate_uplink_status attribute to allow instance VF to see the PF
    link state change for SR-IOV ports.

    Change-Id: I4f1ace630c74363057b0bbdbc759c482125ad015
    Partial-Bug: 1949491
    (cherry picked from commit 3e387d591869b714be1c323f240fcaff289b3c10)

tags: added: in-stable-ussuri
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-neutron-api (stable/train)

Change abandoned by "Danny Cocks <email address hidden>" on branch: stable/train
Review: https://review.opendev.org/c/openstack/charm-neutron-api/+/902341
Reason: See comment above - backport is not needed for train and it is now EoL

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.