Fix CI race condition

Bug #2051040 reported by Arnaud Morin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
Undecided
Arnaud Morin

Bug Description

We recently had two issues in CI:

mistral.tests.unit.engine.test_subworkflows_pause_resume.SubworkflowPauseResumeTest.test_pause_resume_cascade_down_to_with_items_subworkflowstesttools.testresult.real._StringException: Traceback (most recent call last):
  File "/home/zuul/src/opendev.org/openstack/mistral/mistral/tests/unit/engine/test_subworkflows_pause_resume.py", line 975, in test_pause_resume_cascade_down_to_with_items_subworkflows
    self.assertEqual(states.RUNNING, wf_2_ex_1_task_1_ex.state)
  File "/home/zuul/src/opendev.org/openstack/mistral/.tox/unit-postgresql/lib/python3.10/site-packages/testtools/testcase.py", line 395, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/home/zuul/src/opendev.org/openstack/mistral/.tox/unit-postgresql/lib/python3.10/site-packages/testtools/testcase.py", line 482, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: 'RUNNING' != 'IDLE'

And:

mistral.tests.unit.engine.test_policies.PoliciesTest.test_timeout_policy_success_after_timeouttesttools.testresult.real._StringException: Traceback (most recent call last):
  File "/home/zuul/src/opendev.org/openstack/mistral/mistral/tests/unit/engine/test_policies.py", line 1505, in test_timeout_policy_success_after_timeout
    self.assertEqual(states.IDLE, task_ex.state)
  File "/home/zuul/src/opendev.org/openstack/mistral/.tox/unit-mysql/lib/python3.10/site-packages/testtools/testcase.py", line 395, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/home/zuul/src/opendev.org/openstack/mistral/.tox/unit-mysql/lib/python3.10/site-packages/testtools/testcase.py", line 482, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: 'IDLE' != 'RUNNING'

We should try to fix that

Changed in mistral:
assignee: nobody → Arnaud Morin (arnaud-morin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/mistral/+/906437

Changed in mistral:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/mistral/+/906595

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on mistral (master)

Change abandoned by "Arnaud Morin <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/mistral/+/906437
Reason: dup https://review.opendev.org/c/openstack/mistral/+/906595

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

Reviewed: https://review.opendev.org/c/openstack/mistral/+/906595
Committed: https://opendev.org/openstack/mistral/commit/a3e8f5b0437c2497b1f5e80e50fb873df38f9f53
Submitter: "Zuul (22348)"
Branch: master

commit a3e8f5b0437c2497b1f5e80e50fb873df38f9f53
Author: Arnaud M <email address hidden>
Date: Wed Jan 24 00:37:47 2024 +0100

    Fix some race conditions in mistral CI

    We recently had two issues in CI, this commit tries to fix them.

    Closes-Bug: #2051040

    Change-Id: I05d850344c653bc858b4565673606531da803342
    Signed-off-by: Arnaud M <email address hidden>

    BonneNuitLesPetits

    Change-Id: Ife23f809674dfaaa83fb752c22d77bcc30aa044e

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