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

Bug #1297871 reported by Lucas Alvares Gomes on 2014-03-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Won't Fix
Low
Unassigned
OpenStack Compute (nova)
Low
Unassigned

Bug Description

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.

Changed in ironic:
importance: Undecided → Medium
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
aeva black (tenbrae) on 2014-03-26
Changed in ironic:
status: New → Triaged

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

Changed in ironic:
status: Triaged → In Progress
Changed in ironic:
milestone: none → juno-1
aeva black (tenbrae) on 2014-06-11
Changed in ironic:
milestone: juno-1 → juno-2

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

Changed in ironic:
milestone: juno-2 → juno-3

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

Changed in ironic:
milestone: juno-3 → next
Dmitry Tantsur (divius) wrote :

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.

summary: - Use GB instead of MB for the swap part size
+ Use GB instead of MB for the swap part size in Ironic
Changed in ironic:
importance: Medium → Low
tags: added: ironic

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.

Changed in nova:
status: New → Confirmed
importance: Undecided → Low

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

Changed in ironic:
assignee: Lucas Alvares Gomes (lucasagomes) → nobody
status: In Progress → Confirmed

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

Michael Davies (mrda) wrote :

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"

Changed in ironic:
status: Confirmed → Won't Fix
Changed in nova:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers