[Upgrade] Octane fuel-restore command changes nailgun password

Bug #1606613 reported by Alisa Tselovalnikova
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Nikita Zubkov
8.0.x
Fix Released
High
Nikita Zubkov
Mitaka
Fix Released
High
Nikita Zubkov

Bug Description

Detailed bug description:
 The command "octane fuel-restore --from {data_backup} --admin-password {nailgun_password}" changes nailgun password.

Steps to reproduce:
 1. Deploy 8.0 Fuel
 2. Create data backup using:
    octane fuel-backup --to /root/data_backup.tar.gz
 3. Replace 8.0 fuel with 9.0 Fuel
 4. Restore data backup with incorrect using:
    octane fuel-restore --from {data_backup} --admin-password {nailgun_password}
After provided steps nailgun password is changed.

Actual result:
 Nailgun password is changed.

Reproducibility:
 100%

Description of the environment:
 8.0 - http://paste.openstack.org/show/539091/
 9.0 - http://paste.openstack.org/show/539092/
 octane 9.0.0-1.mos1207

Changed in fuel:
milestone: none → 9.1
assignee: nobody → Fuel Octane (fuel-octane-team)
Ilya Kharin (akscram)
Changed in fuel:
status: New → Confirmed
importance: Undecided → Medium
Nikita Zubkov (zubchick)
Changed in fuel:
assignee: Fuel Octane (fuel-octane-team) → Nikita Zubkov (zubchick)
Changed in fuel:
importance: Medium → High
Ilya Kharin (akscram)
Changed in fuel:
milestone: 9.1 → 10.0
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/366846

Changed in fuel:
status: Confirmed → In Progress
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/370662

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

Reviewed: https://review.openstack.org/370662
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=7618ca3ac2d172df4bcb36b26516c38cd311ed83
Submitter: Jenkins
Branch: stable/mitaka

commit 7618ca3ac2d172df4bcb36b26516c38cd311ed83
Author: Nikita Zubkov <email address hidden>
Date: Wed Sep 7 18:45:10 2016 +0300

    Check that astute.yaml password is valid before backup

    This commit adds astute.yaml password validation in backup procedure. If
    we have a valid password we don't need ask a user about it and could use
    astute.yaml "as is" without fixing password there.

    Change-Id: I0dd79ec624159dd1132ca06a86b35d80941c8c39
    Partial-Bug: #1606613

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

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

commit d678e1453582336664b06c66a22f00041d021105
Author: Nikita Zubkov <email address hidden>
Date: Wed Sep 7 18:45:10 2016 +0300

    Check that astute.yaml password is valid before backup

    This commit adds astute.yaml password validation in backup procedure. If
    we have a valid password we don't need ask a user about it and could use
    astute.yaml "as is" without fixing password there.

    Change-Id: I0dd79ec624159dd1132ca06a86b35d80941c8c39
    Partial-Bug: #1606613

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

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/373326

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

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/373353

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

Reviewed: https://review.openstack.org/373326
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=58faa33624aed93bf8d8032ce0b2386e06e76dd8
Submitter: Jenkins
Branch: stable/8.0

commit 58faa33624aed93bf8d8032ce0b2386e06e76dd8
Author: Nikita Zubkov <email address hidden>
Date: Tue Sep 20 16:29:03 2016 +0300

    Check that astute.yaml password is valid before backup

    This commit adds astute.yaml password validation in backup procedure. If
    we have a valid password we don't need ask a user about it and could use
    astute.yaml "as is" without fixing password there.

    Change-Id: I0dd79ec624159dd1132ca06a86b35d80941c8c39
    Partial-Bug: #1606613

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

Reviewed: https://review.openstack.org/373353
Committed: https://git.openstack.org/cgit/openstack/fuel-octane/commit/?id=502c8adf1bd9cb3f2fd3fdd5f98cf1def09b41db
Submitter: Jenkins
Branch: stable/7.0

commit 502c8adf1bd9cb3f2fd3fdd5f98cf1def09b41db
Author: Nikita Zubkov <email address hidden>
Date: Tue Sep 20 16:29:03 2016 +0300

    Check that astute.yaml password is valid before backup

    This commit adds astute.yaml password validation in backup procedure. If
    we have a valid password we don't need ask a user about it and could use
    astute.yaml "as is" without fixing password there.

    Change-Id: I0dd79ec624159dd1132ca06a86b35d80941c8c39
    Partial-Bug: #1606613

Revision history for this message
Vladimir Khlyunev (vkhlyunev) wrote :

Fixed by checking state of password stored in astute.yaml - #299

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

Verified on 8.0 + MU4 updates.

Actual results:
Password stored in astute.yaml stays the same.

tags: removed: on-verification
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.