Skip to content

Stability fixes#2860

Merged
vigoo merged 22 commits intomainfrom
stability-fixes
Mar 3, 2026
Merged

Stability fixes#2860
vigoo merged 22 commits intomainfrom
stability-fixes

Conversation

@vigoo
Copy link
Contributor

@vigoo vigoo commented Mar 2, 2026

Trying to stabilize the integration tests

  • adds request timeout configuration support but only enables it on a single edge (registry svc calls from executor)
  • fixes a race condition in our Cache type and adds lot of tests to it
  • removes an async recursion from HttpWorkerLogEventStream that I've seen causing stack overflow
  • make log streaming in the test framework not panic if the connection is dropped from it
  • using a deterministic idempotency key for initialize to make it impossible to double-initialize (even if some more race condition is not fixed yet)
  • updated test-r so it should now better print failures on CI when using CTRF reporting
  • fixes an oplog opening race condition that was probably the cause of many issues in the "executor is overloaded" case (not necessarily during tests on CI)
  • investigated a lot about a weird flaky deserialization error. It turned out to be a problem in the oplog query, which used a wrong agent-type metadata when enriching some of the oplog entries. Whether it ran into the wrong metadata or not was depending on the runner (hash-based, probably). Fixed now
  • Found and fixed an issue in the wasi:http durability wrapper that could lead to "unexpected oplog entry" during recovery - but not the same that we've seen on the effect.ai live demo. (None of our wasi:http libraries did this usage pattern in practice)

/// causing premature drop, the Weak becoming un-upgradeable, cache eviction,
/// and creation of a **second** oplog instance for the same worker. Two instances
/// means two independent `last_committed_idx` counters, leading to duplicate
/// INSERT attempts and a unique key violation in SQLite.
Copy link
Contributor

Choose a reason for hiding this comment

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

👏

@vigoo vigoo merged commit 6d2fb11 into main Mar 3, 2026
26 checks passed
@vigoo vigoo deleted the stability-fixes branch March 3, 2026 12:52
@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants