[octane] upgrade-db with-graph should apply workaround for floating IPs

Bug #1623048 reported by Roman Sokolkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Pavel Chechetin

Bug Description

Octane upgrade-db executes fix_neutron_migrations function that does workaround for floating ips.

Octane upgrade-db --with-graph should also handle this.

Steps to reproduce:
 Deploy an original env with Fuel 7.0
 Assign floating IPs
 Start upgrade procedure
 Switchover control plane (from old to new)

Expected results:
 All floating IPs should be still pingable from outside world

Actual result:
 All floating IPs are disconnected from outside world (but according to Neutron DB should be connected)

Reproducibility:
 Always

Workaround:
 Apply to SQL queries from:
 https://github.com/openstack/fuel-octane/blob/master/octane/util/db.py#L100-L114

Description of the environment:
 Source Fuel version: 7.0
 Destination Fuel version: 9.x

tags: added: team-upgrades
removed: team-bugfix
Changed in fuel:
assignee: nobody → Pavel Chechetin (paulche)
Revision history for this message
Dmitry Klenov (dklenov) wrote :

@Roman, please give more detailed description. It would be hard to reproduce / verify this issue. Please also specify between which versions you perform the upgrade.

Changed in fuel:
status: New → Incomplete
tags: added: area-python
summary: - [octane] upgrade-db with-graph should handle workaround for floating IPs
+ [octane] upgrade-db with-graph should apply workaround for floating IPs
Revision history for this message
Pavel Chechetin (paulche) wrote :

@Dmitry, I just updated the description and gonna get it done.

description: updated
Changed in fuel:
status: Incomplete → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-octane (master)

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

Dmitry Klenov (dklenov)
Changed in fuel:
milestone: none → 7.0-updates
importance: Undecided → Medium
Revision history for this message
Ilya Kharin (akscram) wrote :

This report contains a wrong milestone, it should be 9.2.

Changed in fuel:
milestone: 7.0-updates → 9.2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-octane (master)

Reviewed: https://review.openstack.org/370669
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=3c0e78bc864c9f4b21a5dae90408f0f6f1f25812
Submitter: Jenkins
Branch: master

commit 3c0e78bc864c9f4b21a5dae90408f0f6f1f25812
Author: Pavel Chechetin <email address hidden>
Date: Sat Sep 24 09:37:23 2016 +0300

    Add fix for floating IPs

    Depends-On: Ic5eb4b4308feba55cbb97ebc63c1a41b4ed14de8
    Change-Id: I2b3eb8b283c6294bae73e8765da44aefef1a1561
    Closes-Bug: 1623048

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

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

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

Reviewed: https://review.openstack.org/389203
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=9a803f36cfba7100063e7a9c97c30f85a5700a50
Submitter: Jenkins
Branch: stable/mitaka

commit 9a803f36cfba7100063e7a9c97c30f85a5700a50
Author: Pavel Chechetin <email address hidden>
Date: Sat Sep 24 09:37:23 2016 +0300

    Add fix for floating IPs

    Depends-On: Ic5eb4b4308feba55cbb97ebc63c1a41b4ed14de8
    Change-Id: I2b3eb8b283c6294bae73e8765da44aefef1a1561
    Closes-Bug: 1623048

tags: added: in-stable-mitaka
Revision history for this message
Vladimir Khlyunev (vkhlyunev) wrote :

Verified on 596

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-octane ocata-eol

This issue was fixed in the openstack/fuel-octane ocata-eol release.

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.