Software Deployment DELETE action does not work for CFN_SIGNAL

Bug #1488058 reported by Rabi Mishra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Undecided
Rabi Mishra

Bug Description

For software deployments with 'DELETE' actions, deployment signaling fails when used with CFN_SIGNAL. This happens because the 'deploy_signal_id' passed as part of deployment metadata is NULL.

"deploy_signal_id", "value": null,

We return None from '_get_ec2_signed_url' when action==DELETE

https://github.com/openstack/heat/blob/master/heat/engine/resources/signal_responder.py#L95

Rabi Mishra (rabi)
Changed in tripleo:
assignee: nobody → Rabi Mishra (rabi)
affects: tripleo → heat
Revision history for this message
Steven Hardy (shardy) wrote :
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/216223

Changed in heat:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/216223
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=6c29d7a842d499227bcb16f68a4ff228ed6721b3
Submitter: Jenkins
Branch: master

commit 6c29d7a842d499227bcb16f68a4ff228ed6721b3
Author: Rabi Mishra <email address hidden>
Date: Mon Aug 24 17:19:13 2015 +0530

    Fix Software Deployment DELETE action for CFN_SIGNAL transport

    This is a regression from https://review.openstack.org/#/c/198183.

    Change-Id: I76702f12130e18c9e5f036e9e771e8b44fa8da78
    Closes-Bug: #1488058

Changed in heat:
status: In Progress → Fix Committed
Changed in heat:
milestone: none → liberty-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: liberty-3 → 5.0.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.