Skip to content

[python] test_events: ignore Unavailable status events - #6799

Merged
gz merged 1 commit into
mainfrom
events-test-tolerate-unavailable
Aug 4, 2026
Merged

[python] test_events: ignore Unavailable status events#6799
gz merged 1 commit into
mainfrom
events-test-tolerate-unavailable

Conversation

@gz

@gz gz commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

test_events asserts an exact status-event sequence, so it fails whenever the runner
happens to record Unavailable. That status means the runner momentarily could not reach
the pipeline, which reflects the cluster rather than the lifecycle under test.

Describe Manual Test Plan

Seen in CI a compiler-server outage crash-looped the pipeline container, the pod left service, its DNS
record was withdrawn, and the runner recorded one Unavailable event between Initializing
and Running:

At index 5 diff: ('Provisioned', 'Provisioned', 'Unavailable', 'Unavailable', False, 'Success', 'InUse')
             != ('Provisioned', 'Provisioned', 'Initializing', 'Running', False, 'Success', 'InUse')

Replayed the 47 events that run recorded (from its support bundle) through the test's
mapping code:

Result
Before this change FAIL, reproducing the CI failure exactly
After this change PASS
After this change, with a genuine Paused transition deleted still FAIL

The last row matters: the filter drops only Unavailable, and the test still catches a
lifecycle transition that goes missing.

Checklist

  • Unit tests added/updated
  • Integration tests added/updated
  • Documentation updated
  • Changelog updated

Breaking Changes?

None.

The runner records Unavailable whenever it momentarily cannot reach the pipeline, which reflects the cluster rather than the lifecycle under test. One such event, caused by a compiler-server outage that crash-looped the pipeline container, broke the exact-sequence assertion in cloud CI.

Signed-off-by: Gerd Zellweger <mail@gerdzellweger.com>
@gz
gz requested a review from mihaibudiu August 4, 2026 05:16
@gz
gz enabled auto-merge August 4, 2026 05:17
@gz
gz added this pull request to the merge queue Aug 4, 2026
Merged via the queue into main with commit 6e99d28 Aug 4, 2026
1 check passed
@gz
gz deleted the events-test-tolerate-unavailable branch August 4, 2026 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants