Task state DELAYED is not taken into account when finishing a workflow

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

Bug Description

Steps to reproduce:

wf:
  task1:
    action: std.noop
    policies:
      wait-before: 10

  task2:
    action: std.noop

If we run such workflow then as soon as task2 is completed the entire workflow will complete too and task1 will be ignored (because it will be in DELAYED state)

Changed in mistral:
importance: Undecided → High
status: New → Confirmed
milestone: none → kilo-3
Changed in mistral:
assignee: nobody → Nikolay Makhotkin (nmakhotkin)
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/160380

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

Reviewed: https://review.openstack.org/160380
Committed: https://git.openstack.org/cgit/stackforge/mistral/commit/?id=4d08db0f4201c2c95f6c0491c3c9c8845e4033c3
Submitter: Jenkins
Branch: master

commit 4d08db0f4201c2c95f6c0491c3c9c8845e4033c3
Author: Nikolay Mahotkin <email address hidden>
Date: Mon Mar 2 18:30:06 2015 +0300

    Fixing finishing workflow in case DELAYED task state

     * If we have 2 parallel tasks and one of these in the
       DELAYED state, workflow incorrectly finishes and DELAYED state is
       not taken to account.

    Closes-Bug: #1422621

    Change-Id: I187a61117f5ad94c6f4ab33fbc750350f8adef0b

Changed in mistral:
status: In Progress → Fix Committed
Changed in mistral:
status: Fix Committed → Fix Released
Changed in mistral:
milestone: kilo-3 → 2015.1
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.