Skip to content

test(session-net): investigate five-minute fenced-status compaction timeout #856

Description

@VerifiedOrganic

A required CI run at commit 594f4e11f55658204dbc5fd7ef6e6194c17b0a05 exceeded the existing five-minute workload guard in persistent_three_voter_fenced_status_converges_after_response_loss_and_compaction. The test reported all three consensus engines as Running, no storage error, and no snapshot/purge progress at failure. The failing run does not establish why progress stopped.

Evidence:

The one unchanged-head retry passed: miscellaneous shard, attempt 2. Its checkout matches the original tested merge tree. All 45 current checks passed, but the original failure remains unexplained. Preserve both attempts; this retry is not a fix or closure criterion. This is distinct from #838's 100 ms protected-transition case and #820's replacement-readiness interruption case.

Acceptance:

  1. Reproduce or causally explain the missing workload/snapshot progress, preserving the original failure and exact base/head, test profile and command. Use bounded synthetic diagnostics to distinguish leader/admission state, committed/applied progress, snapshot scheduling and transport behavior.
  2. Correct the demonstrated cause at its owning boundary, with a detector that fails on removal of the correction and an adversarial negative case. Do not infer a fix from an unrelated passing retry.
  3. Preserve the existing five-minute workload guard, other deadlines, durability/readback assertions and feat(session-store): add native WAL and explicit persistence modes #776 native WAL plus Durable/Async semantics. Do not weaken checks, force a leadership change, add sleeps, or replace the actual recovery path merely to obtain a pass.
  4. Qualify the isolated case and the complete current CI profile, including restart/response-loss/compaction behavior, and obtain independent review. State any remaining uncertainty explicitly.

This issue records an observed test failure and does not authorize a performance claim, a waiver of required checks, or a deployment claim.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions