connect an ovs bridge with two or more bridges using network template

Bug #1647557 reported by Michael Kraynov
20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Vladimir Sharshov
Mitaka
Fix Released
High
Vladimir Sharshov
Newton
Fix Released
High
Vladimir Sharshov
Ocata
Fix Committed
High
Vladimir Sharshov

Bug Description

At this moment there is no way to connect an ovs bridge with multiple bridges using network templates.
Network template example in the attachment.

Steps to reproduce:
Create new environment, upload the network template. Click deploy.
Expected results:
Environment should be deployed.
Actual result:
2016-12-05 10:54:41 ERR bridge br-prv can not be included into more then one element, elements: [{"action"=>"add-patch", "bridges"=>["br-prv", "br-fw-admin"], "mtu"=>65000, "provider"=>"ovs"}, {"action"=>"add-patch", "bridges"=>["br-prv", "bridge1"], "mtu"=>65000, "provider"=>"ovs"}, {"action"=>"add-patch", "bridges"=>["br-prv", "bridge2"], "mtu"=>65000, "provider"=>"ovs"}] at /etc/puppet/modules/openstack_tasks/manifests/openstack_network/server_config.pp:134 on node node-6.domain.tld
Reproducibility:
100%
Workaround:
Not known
Description of the environment:
 Operation system: Ubuntu
 Versions of components: MOS9.0

Revision history for this message
Michael Kraynov (mkraynov) wrote :
tags: added: customer-found
Revision history for this message
Sergey Vasilenko (xenolog) wrote :

full puppet log required

Changed in fuel:
status: New → Incomplete
Revision history for this message
Michael Kraynov (mkraynov) wrote :

Puppet log file from the node in the attachment

Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: none → 9.2
Changed in fuel:
status: Incomplete → New
Changed in fuel:
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
importance: Undecided → High
information type: Public → Private
Revision history for this message
Michael Kraynov (mkraynov) wrote :

There is a customer's network template in the attachment.

Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Sergey Vasilenko (xenolog)
status: New → Confirmed
information type: Private → Public
Changed in fuel:
assignee: Sergey Vasilenko (xenolog) → Vladimir Sharshov (vsharshov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/411417

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Vladimir Sharshov (vsharshov) → Sergey Vasilenko (xenolog)
Changed in fuel:
assignee: Sergey Vasilenko (xenolog) → Vladimir Sharshov (vsharshov)
tags: added: feature
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

Your request is too far from our reference architecture. Our code doesn’t support it yet. We are going to extend our functionality with workaround for your use case, but please keep in mind that it is an extension and not a bug. Also we cannot guarantee that this case will not be broken in future because there is no described use case and there are no automated tests for it. In order to include your case in list of supported you can propose an enhancement according our workflow https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Enhancements

Revision history for this message
Alexey Shtokolov (ashtokolov) wrote :

This a feature request. And should be delivered as a part of 9.3

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/411417
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=b9c7f17994a3c03ade4cfddb7b7873d57131f727
Submitter: Jenkins
Branch: master

commit b9c7f17994a3c03ade4cfddb7b7873d57131f727
Author: Vladimir Sharshov (warpc) <email address hidden>
Date: Thu Dec 15 19:54:47 2016 +0300

    Allow OVS bridge with two or more bridges

    Change way to find minimal MTU for every paths

    Change-Id: I1f77a6f034c00df4e119fba3cb6ebf3475036275
    Closes-Bug: #1647557

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/421839

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/421840

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/newton)

Reviewed: https://review.openstack.org/421840
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=f13ccdc2a676fde145b7441aef6565b9e60dc307
Submitter: Jenkins
Branch: stable/newton

commit f13ccdc2a676fde145b7441aef6565b9e60dc307
Author: Vladimir Sharshov (warpc) <email address hidden>
Date: Thu Dec 15 19:54:47 2016 +0300

    Allow OVS bridge with two or more bridges

    Change way to find minimal MTU for every paths

    Change-Id: I1f77a6f034c00df4e119fba3cb6ebf3475036275
    Closes-Bug: #1647557
    (cherry picked from commit b9c7f17994a3c03ade4cfddb7b7873d57131f727)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/mitaka)

Reviewed: https://review.openstack.org/421839
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=0833a50366d647beadc3b4726ba21c76805a562b
Submitter: Jenkins
Branch: stable/mitaka

commit 0833a50366d647beadc3b4726ba21c76805a562b
Author: Vladimir Sharshov (warpc) <email address hidden>
Date: Thu Dec 15 19:54:47 2016 +0300

    Allow OVS bridge with two or more bridges

    Change way to find minimal MTU for every paths

    Change-Id: I1f77a6f034c00df4e119fba3cb6ebf3475036275
    Closes-Bug: #1647557
    (cherry picked from commit b9c7f17994a3c03ade4cfddb7b7873d57131f727)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 11.0.0.0rc1

This issue was fixed in the openstack/fuel-library 11.0.0.0rc1 release candidate.

Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Verified on MOS 9.2 mu1 updates.
Errors in puppet log before:
2017-03-10 11:01:41 +0000 Puppet (err): bridge br-prv can not be included into more then one element, elements: [{"action"=>"add-patch", "bridges"=>["br-prv", "br-fw-admin"], "mtu"=>65000, "provider"=>"ovs"}, {"action"=>"add-patch", "bridges"=>["br-prv", "bridge1"], "mtu"=>65000, "provider"=>"ovs"}, {"action"=>"add-patch", "bridges"=>["br-prv", "bridge2"], "mtu"=>65000, "provider"=>"ovs"}] at /etc/puppet/modules/openstack_tasks/manifests/openstack_network/server_config.pp:121 on node node-1.test.domain.local
Deploy was finished w/o errors with updates.

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.