disk utils creates an ext2 configdrive, only compatible with linux

[Jay Faulkner]

ironic-lib, when it creates a configdrive, is creating it as ext2 (https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L759). This is bad as the standard (http://cloudinit.readthedocs.io/en/latest/topics/datasources/configdrive.html#version-2) requires ISO or VFAT format.

We later, in most cases, have a nova configdrive to image over this, making the bug significantly less impactful.

[OpenStack Infra (hudson-openstack)]

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

[Jay Faulkner (jason-oldos)]

Because this created partition is imaged with the configdrive image created by nova, this bug is less severe than it seems at first glance. We should still use a compatible filesystem for consistency with what nova will put down, but it's not a major bug, and would not impact windows users.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/418458
Committed: https://git.openstack.org/cgit/openstack/ironic-lib/commit/?id=2f0ea84488d2944d2e0f22b6b6d29508d061b8f8
Submitter: Jenkins
Branch: master

commit 2f0ea84488d2944d2e0f22b6b6d29508d061b8f8
Author: Jay Faulkner <email address hidden>
Date: Tue Jan 10 08:07:49 2017 -0800

    Create configdrive partitions as vfat

    This fixes a bug where ironic-lib would create configdrive partitions as
    ext2, which is incompatible with any OS but Linux and goes against the
    configdrive standard as written in cloud-init docs.

    Change-Id: Ib4ad8e0b4d6dbef9d441d9ae13fd82f8f51957e6
    Closes-bug: #1655389

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/ironic-lib 2.5.0 release.

[Dmitry Tantsur (divius)]

I guess this issue was fixed by the ironic-lib change