Heat software orchestration ignores software config if user_data_format is missing

Bug #1295536 reported by Thomas Spatzier
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Triaged
Low
JunJie Nan

Bug Description

The current design/implementation of HOT software orchestration applies software configs to servers when those software configs are associated to a server via a SoftwareDeployment resource. However, it is also required that the user_data_format of the server is set to 'SOFTWARE_CONFIG' - otherwise, the software configs will be ignored.

This is unintuitive for template writers. As a template writer, I would expect that the fact that I associate a software config via a software deployment to a server is enough. So it should not be required to also set the user_data_format attribute.

JunJie Nan (nanjj)
Changed in heat:
assignee: nobody → JunJie Nan (nanjj)
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Agreed. Starting to use it and this just seems like a missed detail. Easily corrected though.

Changed in heat:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Steve Baker (steve-stevebaker) wrote :

This could only be validated by looking up the server resource from the deployment validate method. Its messy but I suppose its the only option.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
status: Triaged → In Progress
tags: added: icehouse-rc-potential
Changed in heat:
milestone: none → icehouse-rc2
tags: removed: icehouse-rc-potential
Changed in heat:
milestone: icehouse-rc2 → juno-1
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/82992
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=09a7d80b31e989b272c7a3d5991c84688ad6017b
Submitter: Jenkins
Branch: master

commit 09a7d80b31e989b272c7a3d5991c84688ad6017b
Author: JUNJIE NAN <email address hidden>
Date: Wed Mar 26 13:32:54 2014 +0800

    Validate user_data_format for server with deployment

    Validation error will be raised if user_data_format is not set
    correctly.

    Partial-Bug: #1295536
    Change-Id: I0509c9c3a71d09fa8e2113710d561ffd3289d814

Thierry Carrez (ttx)
Changed in heat:
milestone: juno-1 → juno-2
JunJie Nan (nanjj)
summary: - Heat software orchestration ignores software config is user_data_format
+ Heat software orchestration ignores software config if user_data_format
is missing
Revision history for this message
Steven Hardy (shardy) wrote :

Jun Jie, can you confirm, is the fix above the full fix for this bug, or are we expecting additional patches?

If it's completed, please you you update the status to fix committed (or fix released and remove the milestone target if it's already been released)

Steven Hardy (shardy)
Changed in heat:
milestone: juno-2 → juno-3
Revision history for this message
Thomas Spatzier (thomas-spatzier) wrote :

IIRC, Jun Jie's patch only raise a validation error when the user_data_format property was missing or did not match the SOFTWARE_CONFIG value even though SoftwareDeployments were associated to a server. A more convenient solution IMO would be to just assume that value whenever SotwareDeployments are associated.

Revision history for this message
Steve Baker (steve-stevebaker) wrote :

This check has to happen via REST API calls rather than by calling stack.resource_by_refid in the validate method. I'm not sure what you could set on the nova server to store the user_data_format - maybe you could set/get a nova meta key/value

Thierry Carrez (ttx)
Changed in heat:
milestone: juno-3 → juno-rc1
Changed in heat:
milestone: juno-rc1 → kilo-1
Angus Salkeld (asalkeld)
Changed in heat:
milestone: kilo-1 → kilo-2
Angus Salkeld (asalkeld)
Changed in heat:
milestone: kilo-2 → next
Changed in heat:
status: In Progress → Fix Released
status: Fix Released → Triaged
importance: Medium → Low
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.