Use GB instead of MB for the swap part size in Ironic

[Lucas Alvares Gomes]

Having different unit data for the parameters is not ideal, as we expect the size of the root and ephemeral partition in GB we also should expect the size of the swap partition to be in GB. In Nova they still using root_gb and swap_mb but this is something that the Nova Ironic driver should translate before sending the request to the Ironic api.

[OpenStack Infra (hudson-openstack)]

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

[aeva black (tenbrae)]

Bumping to J3 since the patch is blocked while we try to land the nova driver

[Lucas Alvares Gomes (lucasagomes)]

As this is going to affect the nova driver and we are already in J3, I'm bumping it to Kilo

[Dmitry Tantsur (divius)]

Hi Lucas! Could you give a status update to this bug? Also adding Nova as ironic driver has to be fixed as well. Also status -> low, as it's barely a bug.

[Lucas Alvares Gomes (lucasagomes)]

Hi Dmitry,

So the status is that we don't have an agreement about how it should look like. People seems to agree with having consistent values for all size parameters, but we didn't decide whether use MB for everything, or GB with fractial sizes etc... As it was modifying the Nova Driver and is just a medium/low priority bug getting it fix in Juno was a bit more complicated due the Driver Freeze, but I will tackle it once Kilo opens.

[Lucas Alvares Gomes (lucasagomes)]

We haven't come with a decision about how to solve this problem yet. So I'm unassigning myself for now

[OpenStack Infra (hudson-openstack)]

Change abandoned by Lucas Alvares Gomes (<email address hidden>) on branch: master
Review: https://review.openstack.org/83788

[Michael Davies (mrda)]

Quoting from IRC: "devananda | mrda: that feels like a discussion on the color of the trim of the bikeshed we already store farm equipment in"