image_source can break deployment if IPA is used

Bug #1400156 reported by Vladyslav Drok
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
High
Vladyslav Drok

Bug Description

If image_source in instance_info dictionary specified in 'glance://image-uuid' format, it will break deployment, as IPA will try to write this image to /tmp/{image_source} and will fail to create such file, raising IOError: [Errno 2] No such file or directory.

Vladyslav Drok (vdrok)
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/139988

Changed in ironic:
status: New → In Progress
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → High
milestone: none → kilo-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/139988
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=5b05cab7a648bb4bb9363ba45bff3ce8292ca5b4
Submitter: Jenkins
Branch: master

commit 5b05cab7a648bb4bb9363ba45bff3ce8292ca5b4
Author: Vladyslav Drok <email address hidden>
Date: Mon Dec 8 14:30:09 2014 +0200

    Fix image_info passed to IPA for image download

    It's possible to specify instance_info['image_source'] to be a value
    like 'glance://uuid'. When using AgentDeploy, image_source is passed
    to IPA in image_info dictionary in 'id' field without modification,
    and IPA tries to download image to /tmp/{id} location. Deployment
    fails as /tmp/glance://uuid is invalid location.

    In this change image_source is split to ensure that uuid will be used
    to name the file.

    Closes-bug: #1400156
    Change-Id: Id3658d2a32c87867d828bc3b3fecc94e514a4526

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: kilo-1 → 2015.1.0
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.