Master node installation should not delete existing partitions on target disk silently

Bug #1325068 reported by Miroslav Anashkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Aleksandr Didenko
4.1.x
Won't Fix
Medium
Aleksandr Didenko
5.0.x
Fix Released
Medium
Aleksandr Didenko

Bug Description

In case already installed master node is occasionally booted from Fuel installation media, it's installation process will automatically redeploy the existing master node with complete disk wipe.
It is quite dangerous.
Let master node installation asks user to confirm target disk wipe, but only in case some partitions already exist on target disk.
If there are no partitions - let it proceed with installation silently.

Changed in fuel:
milestone: none → 5.0.1
Revision history for this message
Mike Scherbakov (mihgen) wrote :

This will become an issue for automated testing on bare metal. Let's think how we can avoid it.

Changed in fuel:
milestone: 5.0.1 → 5.1
Revision history for this message
Aleksandr Didenko (adidenko) wrote :

We can add new variable (via kernel params) to force formatting without asking for confirmation.

Changed in fuel:
assignee: nobody → Alexander Didenko (adidenko)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

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

Changed in fuel:
status: New → In Progress
Changed in fuel:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/97227
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=12113261e20992ebf31d259ced4ee3358befd977
Submitter: Jenkins
Branch: master

commit 12113261e20992ebf31d259ced4ee3358befd977
Author: Aleksandr Didenko <email address hidden>
Date: Mon Jun 2 15:21:12 2014 +0300

    Confirm disk re-partitioning before proceeding

    Ask for a confirmation before Fuel installation on a drive with
    existing partition table.

    New kickstart kernel parameter:
      - "forceformat" - (yes/no) defaul is "no". Set to "yes" if you
                        want to format chosen drive automatically.

    For example, if you add "installdrive=sda forceformat=yes" to the
    kernel parameters line in Grub menu, then "sda" drive will be
    formatted automatically and Fuel will be installed on it.

    Change-Id: I0dfcb1d461748fe6b0698f125201b730d63e41d4
    Closes-bug: #1325068

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

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/98417

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

Fix proposed to branch: stable/4.1
Review: https://review.openstack.org/98430

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

This is a feature, not a bug. What is it doing in stable releases?

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

Reviewed: https://review.openstack.org/98417
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=860c40df85ad07b4d6782db3d6714acdef8fc8fb
Submitter: Jenkins
Branch: stable/5.0

commit 860c40df85ad07b4d6782db3d6714acdef8fc8fb
Author: Aleksandr Didenko <email address hidden>
Date: Mon Jun 2 15:21:12 2014 +0300

    Confirm disk re-partitioning before proceeding

    Ask for a confirmation before Fuel installation on a drive with
    existing partition table.

    New kickstart kernel parameter:
      - "forceformat" - (yes/no) defaul is "no". Set to "yes" if you
                        want to format chosen drive automatically.

    For example, if you add "installdrive=sda forceformat=yes" to the
    kernel parameters line in Grub menu, then "sda" drive will be
    formatted automatically and Fuel will be installed on it.

    Change-Id: I0dfcb1d461748fe6b0698f125201b730d63e41d4
    Closes-bug: #1325068
    (cherry picked from commit 12113261e20992ebf31d259ced4ee3358befd977)

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (stable/4.1)

Reviewed: https://review.openstack.org/98430
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=180f6397eceb3e1b1b96cea419e80d4909d73ca9
Submitter: Jenkins
Branch: stable/4.1

commit 180f6397eceb3e1b1b96cea419e80d4909d73ca9
Author: Aleksandr Didenko <email address hidden>
Date: Mon Jun 2 15:21:12 2014 +0300

    Confirm disk re-partitioning before proceeding

    Ask for a confirmation before Fuel installation on a drive with
    existing partition table.

    New kickstart kernel parameter:
      - "forceformat" - (yes/no) defaul is "no". Set to "yes" if you
                        want to format chosen drive automatically.

    For example, if you add "installdrive=sda forceformat=yes" to the
    kernel parameters line in Grub menu, then "sda" drive will be
    formatted automatically and Fuel will be installed on it.

    Change-Id: I0dfcb1d461748fe6b0698f125201b730d63e41d4
    Closes-bug: #1325068

Revision history for this message
Meg McRoberts (dreidellhasa) wrote :

Added to "Fixed Issues" in the 5.0.1 Release Notes.

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.