utils.mkfs() does not check whether formatting to the specified filesystem

Bug #1286244 reported by Roman Prykhodchenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Unassigned

Bug Description

Currently there's no check whether mkfs is able to format the partition to the specified filesystem. Specifying an unsupported one will lead to a generic exception.

Changed in ironic:
assignee: nobody → Roman Prykhodchenko (romcheg)
Revision history for this message
Roman Prykhodchenko (romcheg) wrote :

The major problem is that the only reliable way to fix this bug is to parse the error message returned by mkfs. That requires specifying LANG=C environment variable for running mkfs. However, passing environment variables is not supported in oslo yet.

This patch https://review.openstack.org/#/c/86021/ allows passing environment variables to processutils.execute() so we should wait until it gets merged.

Changed in ironic:
status: New → Triaged
Revision history for this message
Dmitry Tantsur (divius) wrote :

Hi Roman! Are you still working on this patch? Patch you mention in your comment got landed.

Changed in ironic:
assignee: Roman Prykhodchenko (romcheg) → nobody
Changed in ironic:
assignee: nobody → Mikhail Durnosvistov (mdurnosvistov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: Triaged → In Progress
Changed in ironic:
assignee: Mikhail Durnosvistov (mdurnosvistov) → Yuriy Zveryanskyy (yzveryanskyy)
Changed in ironic:
assignee: Yuriy Zveryanskyy (yzveryanskyy) → Mikhail Durnosvistov (mdurnosvistov)
Changed in ironic:
milestone: none → juno-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/98102
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=0d0c31af7a992fbb46e41076240a000923e81a83
Submitter: Jenkins
Branch: master

commit 0d0c31af7a992fbb46e41076240a000923e81a83
Author: Mikhail Durnosvistov <email address hidden>
Date: Thu Jun 5 15:50:17 2014 +0300

    Check whether specified FS is supported

    Added to `utils.mkfs()` checking that not supported file systems.

    Change-Id: If3cff21b8b4ffb969e7bcfc0d9769a36f23061dd
    Closes-Bug: #1286244

Changed in ironic:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ironic:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: juno-rc1 → 2014.2
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.