Migrations do not match correspondent yaml structures

Bug #1653061 reported by Julia Aranovich
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Fedor Zhadaev
Mitaka
Fix Released
High
Fedor Zhadaev
Newton
Fix Committed
High
Fedor Zhadaev
Ocata
Fix Committed
High
Fedor Zhadaev

Bug Description

These changes https://review.openstack.org/#/c/342714/22/nailgun/nailgun/fixtures/openstack.yaml
in NIC/bond are absent in Alembic migrations.

For example, we have different labels here:
https://github.com/openstack/fuel-web/blob/stable/mitaka/nailgun/nailgun/fixtures/openstack.yaml#L2047
and there:
https://github.com/openstack/fuel-web/blob/stable/mitaka/nailgun/nailgun/db/migration/alembic_migrations/versions/fuel_9_2.py#L186

Actual for stable/mitaka and master branches (nic-plugin-feature was not merged to newton yet).

Changed in fuel:
importance: Undecided → High
Fedor Zhadaev (fzhadaev)
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Fedor Zhadaev (fzhadaev)
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/415810

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

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

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

Reviewed: https://review.openstack.org/415810
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=e0c836ceb9e1679292d1f4a0d018aa61a1dbd724
Submitter: Jenkins
Branch: master

commit e0c836ceb9e1679292d1f4a0d018aa61a1dbd724
Author: Fedor Zhadaev <email address hidden>
Date: Fri Dec 30 11:14:29 2016 +0400

    Make migration file for 9.2 consistent with openstack.yaml

    Change-Id: I9bcabbcdbb3f22422a709146303af57bd594bd79
    Closes-bug: #1653061

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/newton)

Change abandoned by Fedor Zhadaev (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/420684

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (stable/mitaka)

Change abandoned by Fedor Zhadaev (<email address hidden>) on branch: stable/mitaka
Review: https://review.openstack.org/420661

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/420778

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

Reviewed: https://review.openstack.org/420778
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=2c9ab75d88ff681a96a71d443dc51bba9468b57d
Submitter: Jenkins
Branch: master

commit 2c9ab75d88ff681a96a71d443dc51bba9468b57d
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 16 18:57:20 2017 +0400

    Make migration file for 9.2 consistent with openstack.yaml

    Change-Id: Iaa93d5463952f0e4a0599fef8630ce9d47753ad6
    Closes-bug: #1653061
    Closes-bug: #1652076

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

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

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

Reviewed: https://review.openstack.org/421349
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=d1088badd9e7245a91d5a3d141685600c7a4730f
Submitter: Jenkins
Branch: stable/mitaka

commit d1088badd9e7245a91d5a3d141685600c7a4730f
Author: Fedor Zhadaev <email address hidden>
Date: Tue Jan 17 20:18:08 2017 +0400

    Make migration file for 9.2 consistent with openstack.yaml

    Change-Id: Iaa93d5463952f0e4a0599fef8630ce9d47753ad6
    Closes-bug: #1653061
    Closes-bug: #1652076

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

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

tags: added: on-verification
Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Part of this bug was verified on 9.2 snapshot #798.

Steps to verify:
Compare interface settings in https://github.com/openstack/fuel-web/blob/stable/mitaka/nailgun/nailgun/fixtures/openstack.yaml with interface settings in environment in Fuel UI: dpdk, mtu.

For full verification need to compare also sriov settings.

tags: removed: on-verification
Revision history for this message
Ekaterina Khomyakova (ekhomyakova) wrote :

Verified on Fuel 9.2 #809

Steps to verify:
Compare `nic_attributes` section from openstack.yaml with `attributes` from api/nodes/<id>/interfaces.

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

Reviewed: https://review.openstack.org/422432
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=7555b7a822f6906579a3e6cce19d42024782ac69
Submitter: Jenkins
Branch: stable/newton

commit 7555b7a822f6906579a3e6cce19d42024782ac69
Author: Fedor Zhadaev <email address hidden>
Date: Mon Jan 16 18:57:20 2017 +0400

    Make migration file for 9.2 consistent with openstack.yaml

    Change-Id: Iaa93d5463952f0e4a0599fef8630ce9d47753ad6
    Closes-bug: #1653061
    Closes-bug: #1652076
    (cherry picked from commit 2c9ab75)

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

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

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.