ephemeral_mb is not properly set in the agent driver

Bug #1592895 reported by Vladyslav Drok
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
High
Vladyslav Drok

Bug Description

When deploy_utils.parse_instance_info is called in the agent driver, it returns a dictionary with ephemeral_gb set, but IPA and ironic-lib expect ephemeral_mb, as it is done in iscsi_deploy.get_deploy_info. This causes the ephemeral partition not being created by the IPA.

A relevant log - http://logs.openstack.org/25/329625/2/check/gate-tempest-dsvm-ironic-agent_ssh-partition-image-nv/e7115a3/logs/ironic-bm-logs/node-0_console_14-06-2016-20:28:43.txt.gz#_2016-06-14_20_28_05_387, you can see that ephemeral_mb is none.

Vladyslav Drok (vdrok)
description: updated
Changed in ironic:
assignee: nobody → Vladyslav Drok (vdrok)
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/330089

Changed in ironic:
status: New → In Progress
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → High
tags: added: liberty-backport-potential mitaka-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/330089
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=2e0716720a91030d9600d40dc949c27ff62bc43b
Submitter: Jenkins
Branch: master

commit 2e0716720a91030d9600d40dc949c27ff62bc43b
Author: Vladyslav Drok <email address hidden>
Date: Wed Jun 15 19:45:20 2016 +0300

    Properly set ephemeral size in agent drivers

    This commit ensures that ephemeral_mb will be passed to an IPA
    ramdisk, as it is what is expected by both IPA and ironic-lib.

    This change also deduplicates some of the logic between
    iscsi_deploy.build_instance_info_for_deploy and
    deploy_utils.parse_instance_info methods.

    Closes-bug: #1592895
    Change-Id: I0c0ea7a887f9aeb9fc4280aa86098d59ee8cad20

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/ironic 6.1.0

This issue was fixed in the openstack/ironic 6.1.0 release.

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.