Fuel for OpenStack

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

Bug #1647557 reported by Michael Kraynov on 2016-12-06

This bug affects 2 people

	Status	Importance	Assigned to	Milestone
Fuel for OpenStack	Fix Committed	High	Vladimir Sharshov	Fuel for OpenStack 11.0
Mitaka	Fix Released	High	Vladimir Sharshov	Fuel for OpenStack 9.2-mu-1
Newton	Fix Released	High	Vladimir Sharshov	Fuel for OpenStack 10.1
Ocata	Fix Committed	High	Vladimir Sharshov	Fuel for OpenStack 11.0

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

Tags:

Revision history for this message

Michael Kraynov (mkraynov) wrote on 2016-12-06:

network_template_2.yaml Edit (2.7 KiB, text/plain)

tags:

added: customer-found

Revision history for this message

Sergey Vasilenko (xenolog) wrote on 2016-12-06:

full puppet log required

Changed in fuel:
status:	New → Incomplete

Revision history for this message

Michael Kraynov (mkraynov) wrote on 2016-12-06:

puppet-apply.log Edit (310.0 KiB, text/plain)

Puppet log file from the node in the attachment

Dmitry Pyzhov (dpyzhov) on 2016-12-06

Changed in fuel:
milestone:	none → 9.2

Alexey Shtokolov (ashtokolov) on 2016-12-06

Changed in fuel:
status:	Incomplete → New

Oleksiy Molchanov (omolchanov) on 2016-12-07

Changed in fuel:
assignee:	nobody → Fuel Sustaining (fuel-sustaining-team)
importance:	Undecided → High

Oleksiy Molchanov (omolchanov) on 2016-12-07

information type:

Public → Private

Revision history for this message

Michael Kraynov (mkraynov) wrote on 2016-12-07:

host_nw.yaml Edit (12.0 KiB, text/plain)

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

Oleksiy Molchanov (omolchanov) on 2016-12-07

Changed in fuel:
assignee:	Fuel Sustaining (fuel-sustaining-team) → Sergey Vasilenko (xenolog)
status:	New → Confirmed

Michael Kraynov (mkraynov) on 2016-12-14

information type:

Private → Public

Sergey Vasilenko (xenolog) on 2016-12-15

Changed in fuel:
assignee:	Sergey Vasilenko (xenolog) → Vladimir Sharshov (vsharshov)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2016-12-15: Fix proposed to fuel-library (master)

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

Changed in fuel:
status:	Confirmed → In Progress

OpenStack Infra (hudson-openstack) on 2016-12-16

Changed in fuel:
assignee:	Vladimir Sharshov (vsharshov) → Sergey Vasilenko (xenolog)

Sergey Vasilenko (xenolog) on 2016-12-20

Changed in fuel:
assignee:	Sergey Vasilenko (xenolog) → Vladimir Sharshov (vsharshov)

Alexey Shtokolov (ashtokolov) on 2016-12-27

tags:

added: feature

Revision history for this message

Dmitry Pyzhov (dpyzhov) wrote on 2016-12-27:

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 on 2017-01-13:

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-18: 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 on 2017-01-18: 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 on 2017-01-18: Fix proposed to fuel-library (stable/newton)

#10

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

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-24: Fix merged to fuel-library (stable/newton)

#11

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 on 2017-02-07: Fix merged to fuel-library (stable/mitaka)

#12

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 on 2017-02-27: Fix included in openstack/fuel-library 11.0.0.0rc1

#13

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

Revision history for this message

Ekaterina Shutova (eshutova) wrote on 2017-03-10:

#14

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.