Implement validation of workflow graph integrity (check that all mentioned tasks in wf exist)

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

Bug Description

Example:

wf:
  task1:
    on-success: task3

  task2:
   ...

This workflow must not pass validation check because task3 doesn't exist.

Changed in mistral:
milestone: none → liberty-rc1
importance: Undecided → Medium
Changed in mistral:
assignee: nobody → Nikolay Makhotkin (nmakhotkin)
Changed in mistral:
status: New → Triaged
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/223172

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/223172
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=002949a8048c83955f76dfb8694060799fa65095
Submitter: Jenkins
Branch: master

commit 002949a8048c83955f76dfb8694060799fa65095
Author: Nikolay Mahotkin <email address hidden>
Date: Mon Sep 14 18:01:19 2015 +0300

    Adding validation of workflow graph

     * To avoid circular dependency between mistral.workbook.parser
     and mistral.workflow.commands reserved task names were moved to
     workbook.tasks module.

    Closes-Bug: #1494635

    Change-Id: I21337a9b616f55c3bbdc957538b25d36246b3903

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