br-ex should not be used in br-mappings configuration

Bug #1357298 reported by Aleksey Kasatkin
44
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Ryan Moe
5.1.x
Fix Released
Critical
Ihor Kalnytskyi
6.0.x
Fix Released
Critical
Ryan Moe

Bug Description

/etc/neutron/plugin.ini :

....

[ovs]
enable_tunneling=True
integration_bridge=br-int
local_ip=192.168.0.3
tunnel_bridge=br-tun
bridge_mappings = physnet1:br-ex

br-ex should not be used here

Changed in fuel:
status: New → Triaged
importance: Undecided → High
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/114591

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Andrew Woodward (xarses) wrote :

Bug and commit message are insufficient to understand why this is needed let alone justify priority

Changed in fuel:
assignee: Sergey Vasilenko (xenolog) → Mike Scherbakov (mihgen)
Mike Scherbakov (mihgen)
Changed in fuel:
assignee: Mike Scherbakov (mihgen) → Sergey Vasilenko (xenolog)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

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

commit edab551d9d9fc955e4f8d04a13e501a1afc42830
Author: Sergey Vasilenko <email address hidden>
Date: Fri Aug 15 18:49:21 2014 +0400

    make external network type as 'local'

    and remove it from bridge mappings.
    Network, which is used as a source of Floating-IP
    should not be associated with any external interface.
    It's not needed since Grizzly release.

    Partial-Bug: #1357298
    Related-bug: #1272349
    Closes-bug: #1260051
    Change-Id: I62c7ebaa099c3580ddb68c2a8906e1cd2321ba58

Mike Scherbakov (mihgen)
Changed in fuel:
status: In Progress → Fix Committed
status: Fix Committed → In Progress
Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Dima Shulyak (dshulyak) wrote :
Revision history for this message
Sergey Vasilenko (xenolog) wrote :

IMHO, this should be fixed by Nailgun API versioning.

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

This part must be moved from network_manager into deployment_serializer, we have versioning in serializer already.

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

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

Changed in fuel:
assignee: Aleksey Kasatkin (alekseyk-ru) → Igor Kalnitsky (ikalnitsky)
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Raised to critical due its blocker condition for current master branch

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in fuel:
assignee: Igor Kalnitsky (ikalnitsky) → Ryan Moe (rmoe)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/125391
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=069849284a7c50cf3cb957c9b9392c45c6fc6675
Submitter: Jenkins
Branch: master

commit 069849284a7c50cf3cb957c9b9392c45c6fc6675
Author: Igor Kalnitsky <email address hidden>
Date: Tue Sep 30 18:39:18 2014 +0300

    Introduce NeutronNetworkDeploymentSerializer

    Neutron attributes serialization is moved from NetworkManager to
    NeutronNetworkDeploymentSerializer.
    Versioning is introduced in NeutronNetworkDeploymentSerializer.

    The previous code in network/neutron.py checked specifically for 5.1
    environments. This caused 6.0 environments to fall back to the
    old behavior and set network_type to 'flat' and configure 'physnet'.

    Change-Id: I3940d0910c888794f82dd2159727ef5073367916
    Closes-Bug: #1357298

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Aleksey Kasatkin (<email address hidden>) on branch: master
Review: https://review.openstack.org/125100
Reason: https://review.openstack.org/#/c/125391/ was merged that done the same thing.

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

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/126280

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

Reviewed: https://review.openstack.org/126280
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=42e784e0ce6738dee122f1f6942baee08c4e149e
Submitter: Jenkins
Branch: stable/5.1

commit 42e784e0ce6738dee122f1f6942baee08c4e149e
Author: Igor Kalnitsky <email address hidden>
Date: Tue Sep 30 18:39:18 2014 +0300

    Introduce NeutronNetworkDeploymentSerializer

    Neutron attributes serialization is moved from NetworkManager to
    NeutronNetworkDeploymentSerializer.
    Versioning is introduced in NeutronNetworkDeploymentSerializer.

    The previous code in network/neutron.py checked specifically for 5.1
    environments. This caused 5.1.1 environments to fall back to the
    old behavior and set network_type to 'flat' and configure 'physnet'.

    (cherry picked from commit 069849284a7c50cf3cb957c9b9392c45c6fc6675)

    Change-Id: I3940d0910c888794f82dd2159727ef5073367916
    Closes-Bug: #1357298

Revision history for this message
Dennis Dmitriev (ddmitriev) wrote :

Fix released on 5.1.1; on 'neutron' network, private network is "network_type": "gre", and public network is "network_type": "local".

{"build_id": "2014-11-20_13-27-34", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "27", "auth_required": true, "api": "1.0", "nailgun_sha": "7580f6341a726c2019f880ae23ff3f1c581fd850", "production": "docker", "fuelmain_sha": "eac9e2704424d1cb3f183c9f74567fd42a1fa6f3", "astute_sha": "51087c92a50be982071a074ff2bea01f1a5ddb76", "feature_groups": ["mirantis"], "release": "5.1.1", "release_versions": {"2014.1.3-5.1.1": {"VERSION": {"build_id": "2014-11-20_13-27-34", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "27", "api": "1.0", "nailgun_sha": "7580f6341a726c2019f880ae23ff3f1c581fd850", "production": "docker", "fuelmain_sha": "eac9e2704424d1cb3f183c9f74567fd42a1fa6f3", "astute_sha": "51087c92a50be982071a074ff2bea01f1a5ddb76", "feature_groups": ["mirantis"], "release": "5.1.1", "fuellib_sha": "c6eb418e6273007e6563c4377978c1ba7b2ba34e"}}}, "fuellib_sha": "c6eb418e6273007e6563c4377978c1ba7b2ba34e"}

Revision history for this message
Mykhaylo Slobodyan (mykhaylo-slobodyan) wrote :

[root@node-1 ~]# cat /etc/neutron/plugin.ini | tail -n5
[ovs]
enable_tunneling=True
integration_bridge=br-int
local_ip=10.108.2.3
tunnel_bridge=br-tun

5.1.1 ISO #18

Revision history for this message
Dennis Dmitriev (ddmitriev) wrote :

api: '1.0'
astute_sha: 7720992a297a478638b9623c53356e0b355abc24
auth_required: true
build_id: 2014-11-20_09-52-02
build_number: '119'
feature_groups:
- mirantis
fuellib_sha: 6852c960a805b7a84cb6ce4b79b0e729404c4f41
fuelmain_sha: 30a3062406b1bb32c0b981b89224ca92bf9141af
nailgun_sha: 5a10371b1f74f15bfdd3156cf08a6a77729b4731
ostf_sha: 85272980659a977582e5b2dc67479fe9d7c1bbe7
production: docker
release: '6.0'
release_versions:
  2014.2-6.0:
    VERSION:
      api: '1.0'
      astute_sha: 7720992a297a478638b9623c53356e0b355abc24
      build_id: 2014-11-20_09-52-02
      build_number: '119'
      feature_groups:
      - mirantis
      fuellib_sha: 6852c960a805b7a84cb6ce4b79b0e729404c4f41
      fuelmain_sha: 30a3062406b1bb32c0b981b89224ca92bf9141af
      nailgun_sha: 5a10371b1f74f15bfdd3156cf08a6a77729b4731
      ostf_sha: 85272980659a977582e5b2dc67479fe9d7c1bbe7
      production: docker
      release: '6.0'

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.