Skip to content

Flake fix investigation - #1430

Open
Sushisource wants to merge 8 commits into
mainfrom
sj/flake-fix
Open

Flake fix investigation#1430
Sushisource wants to merge 8 commits into
mainfrom
sj/flake-fix

Conversation

@Sushisource

Copy link
Copy Markdown
Member

No description provided.

@Sushisource
Sushisource force-pushed the sj/flake-fix branch 2 times, most recently from 2b2748d to 7312729 Compare July 29, 2026 21:53
A failed history fetch could arrive after zero-cache eviction removed the run, dropping the task
token and blocking
subsequent polls. Preserve the token and report unstored WFT failures through a stream-level action.
@Sushisource
Sushisource marked this pull request as ready for review July 30, 2026 22:30
@Sushisource
Sushisource requested a review from a team as a code owner July 30, 2026 22:30
@Sushisource Sushisource changed the title [DRAFT] Flake fix investigation Flake fix investigation Jul 30, 2026
Comment thread crates/sdk-core/src/core_tests/workflow_tasks.rs
Comment thread crates/sdk-core/src/worker/activities/local_activities.rs Outdated
Comment on lines +198 to +202
actions.extend(
activations
.into_iter()
.map(WorkflowStreamAction::Activation),
);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be worth a comment explaining importance of ordering activations after FailUnstoredWft variants.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It actually doesn't matter really - it's just incidentally the most convenient way to combine the different types into the enum

@Sushisource
Sushisource enabled auto-merge (squash) July 31, 2026 21:59
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