std.mistral_http action should send a signal containing url for callback

Bug #1500833 reported by Nikolay Makhotkin
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
High
Nikolay Makhotkin
Mitaka
Fix Released
High
Nikolay Makhotkin

Bug Description

From the user perspective, I don't want to see those values in headers: execution-id, task-id, action-id. Need to send in headers certain callback url, e.g.:

{
  "callback_url": "/v2/action_executions/<action-execution-id>"
}

Mistral sends a signal to third party service and provides url for the callback. Then, the service sends an answer to that url. No need to pass extra parameters, need to pass only link where the answer is expected.

Changed in mistral:
status: New → Triaged
Revision history for this message
Moshe Elisha (melisha) wrote :

It would be great if the std.mistral_http will have a boolean property "presigned" so the callback_url can be used without authentication.

Changed in mistral:
milestone: liberty-rc2 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (master)

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

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

Reviewed: https://review.openstack.org/245772
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=dde634d8b9d8134685c785c012a594080bf9b05e
Submitter: Jenkins
Branch: master

commit dde634d8b9d8134685c785c012a594080bf9b05e
Author: Nikolay Mahotkin <email address hidden>
Date: Mon Nov 16 16:27:06 2015 +0300

    Adding callback url to action context

     * Callback URL is now provided by std.mistral_http
     action to easy understand where the result should be sent

    Closes-Bug: #1500833

    Change-Id: I347edc5848bde4262a2d57f1787db5f16e218f0a

Changed in mistral:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/250187

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (stable/liberty)

Reviewed: https://review.openstack.org/250187
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=9775b399bef707ce8926a4dc24a121f6b55ae7f4
Submitter: Jenkins
Branch: stable/liberty

commit 9775b399bef707ce8926a4dc24a121f6b55ae7f4
Author: Nikolay Mahotkin <email address hidden>
Date: Mon Nov 16 16:27:06 2015 +0300

    Adding callback url to action context

     * Callback URL is now provided by std.mistral_http
     action to easy understand where the result should be sent

    Closes-Bug: #1500833

    Change-Id: I347edc5848bde4262a2d57f1787db5f16e218f0a
    (cherry picked from commit dde634d8b9d8134685c785c012a594080bf9b05e)

tags: added: in-stable-liberty
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/mistral 2.0.0.0b1

This issue was fixed in the openstack/mistral 2.0.0.0b1 development milestone.

Changed in mistral:
status: Fix Committed → Fix Released
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/mistral 1.0.1

This issue was fixed in the openstack/mistral 1.0.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

This issue was fixed in the openstack/mistral 1.0.1 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.