Skip to content

Remove the stale RabbitMQ first-round-failures handoff (GH-3726) - #3731

Merged
jeremydmiller merged 1 commit into
mainfrom
gh-3726/remove-stale-handoff
Jul 30, 2026
Merged

Remove the stale RabbitMQ first-round-failures handoff (GH-3726)#3731
jeremydmiller merged 1 commit into
mainfrom
gh-3726/remove-stale-handoff

Conversation

@jeremydmiller

Copy link
Copy Markdown
Member

The handoff doc landed in #3727 to carry evidence between sessions. That work is done, and leaving the file at the repo root now actively misleads a reader:

  • Its headline itemmulti_node_exclusive_listener_failover.listener_fails_over_when_the_leader_running_it_crashes — is fixed in Reset listener agent status to Running on restart (GH-3604) #3728. Its central claim, that the listener "kept flapping," was wrong: the failover always worked and only the agent's reported status was stuck. The doc tells the next reader to go chase a flap that does not exist.
  • Its Bug_1594 section describes a deterministic failure that isn't one. That test uses Servers.SqlServerConnectionString, and the three failing modes were a missing SQL Server container, not a product defect.

#3726 carries the corrected account of all four tests, and the two that are still unexplained stay tracked there and on #3729. Nothing is lost by deleting the file.

🤖 Generated with Claude Code

The handoff landed in #3727 to carry evidence between sessions, and that work is done. Leaving
it at the repo root now actively misleads:

* its headline item — the exclusive-listener failover test — is fixed in #3728, and its central
  claim that the listener "kept flapping" was wrong. The failover always worked; only the agent's
  reported status was stuck.
* its Bug_1594 section describes a deterministic failure that does not exist. That test uses
  Servers.SqlServerConnectionString, and the three failing modes were a missing SQL Server
  container, not a product defect.

GH-3726 carries the corrected account of all four tests, and the two still-unexplained ones stay
tracked there and on #3729. Nothing is lost by deleting the file.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@jeremydmiller
jeremydmiller merged commit 90b987d into main Jul 30, 2026
31 checks passed
@jeremydmiller jeremydmiller mentioned this pull request Jul 30, 2026
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.

1 participant