[SDK Sentinel] Stabilize immediate schedule trigger test - #1802
Merged
Conversation
tconley1428
approved these changes
Sep 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Caution
This PR contains untrusted AI-generated code. Do not approve or run CI until a maintainer has reviewed the diff. SDK Sentinel verified that GitHub Actions remained approval-gated with zero executable jobs when this PR was opened.
Summary
Stabilize the Cloud integration test after the [first captured Cloud failure](https://github.com/temporalio/sdk-python/actions/runs/33407614934/job/99538982979) observed zero actions immediately after creating a schedule with an immediate trigger. The bounded snapshot contains no earlier equivalent passing Cloud run.
Root cause
create_schedulecan return before the triggered action is visible throughdescribein Cloud's eventually consistent schedule state. The test assumed synchronous visibility after it became Cloud-eligible in the [Cloud suite expansion](f6594f1).Fix
Poll
describewith the repository's existing eventual-consistency helper until one action is visible, then reuse that successful description for the workflow assertions.Validation
The pre-change local test passed once, so natural reproduction did not occur. The candidate passed 10 focused stress repetitions and the canonical repository validator. The actual Cloud path remains untested locally and requires the
cloud-testenvconfig CI job.Validation status:
ci-validation-requiredhighhighmediumstandard— 1 changed file; 847 Bnot-reproduced— The pre-change test passed once against the local server; the external Cloud environment was unavailable under the investigation network boundary.pass-after-only— The adjusted test passed 10 focused post-change stress repetitions; no deterministic pre-change failure was available locally.Poe lint, bridge lint, and editable build) — passedPoe lint, bridge lint, and editable build) — passedInvestigator-run checks
poe test -s -k test_schedule_trigger_immediately— passed; 1 attempt(s). Pre-change local-server run passed; the Cloud timing window was not reproduced.poe test -s -k test_schedule_trigger_immediately --flake-finder --flake-runs=10— passed; 10 attempt(s). Post-change local-server stress run passed all 10 repetitions..ci-flake-runtime/input/validate.sh— failed; 1 attempt(s). Python checks passed, but the investigation HOME pointed rustup at an empty toolchain cache.RUSTUP_HOME=/home/runner/.rustup CARGO_HOME=/home/runner/work/sdk-sentinel/sdk-sentinel/target/.ci-flake-runtime/cache/cargo .ci-flake-runtime/input/validate.sh— passed; 1 attempt(s). The unchanged canonical validator passed using the preloaded offline Rust toolchain and Cargo cache.Required target CI
Residual risks
Automation provenance
b6ee53cf748cfc98d9e32eee2d136ca8abb9a6adIf this finding should not be fixed, apply the
sdk-sentinel:false-positivelabel and close the PR. Sentinel will suppress the finding until its affected source changes.SDK Sentinel recurrence history
2026-08-31 15:28 UTC2026-08-31 15:28 UTC