l3 connectivity port config parsed in reverse order to other charms

Bug #1850956 reported by Edward Hope-Morley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
charm-ovn-chassis
Fix Released
Critical
Frode Nordahl

Bug Description

The neutron-openvswitch and neutron-gateway charms parse data-port as <bridge>:<port> but ovn-chassis does <port>:<bridge>. If there is no explicit reason for this can we please change it to be the same as the other charms? There is actually common code to do this @ [1].

[1] https://github.com/juju/charm-helpers/blob/master/charmhelpers/contrib/openstack/neutron.py#L275

Changed in charm-ovn-chassis:
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Edward Hope-Morley (hopem) wrote :
Changed in charm-ovn-chassis:
milestone: none → 20.02
assignee: nobody → Frode Nordahl (fnordahl)
status: Triaged → In Progress
Frode Nordahl (fnordahl)
Changed in charm-ovn-chassis:
importance: Wishlist → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-ovn-chassis (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-ovn-chassis (master)

Reviewed: https://review.opendev.org/705963
Committed: https://git.openstack.org/cgit/x/charm-ovn-chassis/commit/?id=8712a3d029271fdb58c5842bf19f60f360bef1bd
Submitter: Zuul
Branch: master

commit 8712a3d029271fdb58c5842bf19f60f360bef1bd
Author: Frode Nordahl <email address hidden>
Date: Wed Feb 5 14:35:45 2020 +0100

    Change how bridge interface mappings are configured

    Pick up fixes from openstack-charmers/charm-layer-ovn#6

    Change-Id: Ic4b5e83f443e2f8fc34dcacbf8eb419aacc3943a
    Closes-Bug: #1850956

Changed in charm-ovn-chassis:
status: In Progress → Fix Committed
Liam Young (gnuoy)
Changed in charm-ovn-chassis:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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