Remove the sleep and add a waiter to the test_start_workflow cron test

Bug #1778757 reported by Dougal Matthews
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
High
Adriano Petrich

Bug Description

A recent patch has a workaround for CI failures. However, we should tidy it up and add a better solution.

workround: https://review.openstack.org/#/c/578115
Original bug: https://bugs.launchpad.net/mistral/+bug/1778736

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/578359

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

Reviewed: https://review.openstack.org/578359
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=dc8b50e036d4a1eeb6979332d8f92479be819f0d
Submitter: Zuul
Branch: master

commit dc8b50e036d4a1eeb6979332d8f92479be819f0d
Author: Adriano Petrich <email address hidden>
Date: Wed Jun 27 12:24:59 2018 +0100

    Remove time.sleep from tests

    Use the await method instead of time.sleep to improve the tests.

    Closes-Bug: 1778757
    Change-Id: I084b8acfa01e80ac287983667e3285d7fc0d58b9

Changed in mistral:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 7.0.0.0b3

This issue was fixed in the openstack/mistral 7.0.0.0b3 development milestone.

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.