Wrong execution state with conditional transitions

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

Bug Description

The following workflow should fail if we do not pass variable 'var' but it succeed.

Something wrong with final workflow execution state analysis.

---
version: '2.0'

wfwf:
  input:
    - var: null

  tasks:
    task1:
      action: std.fail
      on-success:
        - task2
      on-error:
        - task3: <% $.var != null %>

Revision history for this message
Syed Ahsan Shamim Zaidi (ahsanmohsin04) wrote :

Hi,

Can you kindly provide more detail so that i can look it up.

Thanks

Changed in mistral:
status: New → Triaged
Revision history for this message
Nikolay Makhotkin (nmakhotkin) wrote :

Hi,

It is kind of very specific bug in mistral engine, I can't provide more details on that.

Lingxian Kong (kong)
Changed in mistral:
assignee: nobody → Nikolay Makhotkin (nmakhotkin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (master)

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

commit 6662bec3140de91cf2329060e4e3867f2c2859ce
Author: Nikolay Mahotkin <email address hidden>
Date: Fri Nov 6 13:29:54 2015 +0300

    Fixing workflow execution state calculation

     * Fixed workflow execution state calculation
     when it didn't take to account conditional
     transitions (Error was considered handled even
     if transition doesn't trigger a task)

    Closes-Bug: #1510936

    Change-Id: Ia694b6dba9ab2ce9bff2da1fe05f5988864619d9

Changed in mistral:
status: Triaged → Fix Committed
tags: added: liberty-backport-potential
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/246839

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

Reviewed: https://review.openstack.org/246839
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=99a804d21769bd756b29b80c973d2d919f5aff1d
Submitter: Jenkins
Branch: stable/liberty

commit 99a804d21769bd756b29b80c973d2d919f5aff1d
Author: Nikolay Mahotkin <email address hidden>
Date: Fri Nov 6 13:29:54 2015 +0300

    Fixing workflow execution state calculation

     * Fixed workflow execution state calculation
     when it didn't take to account conditional
     transitions (Error was considered handled even
     if transition doesn't trigger a task)

    Closes-Bug: #1510936

    Change-Id: Ia694b6dba9ab2ce9bff2da1fe05f5988864619d9
    (cherry picked from commit 6662bec3140de91cf2329060e4e3867f2c2859ce)

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.

Changed in mistral:
milestone: mitaka-1 → 2.0.0
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.