Skip to content

night: deploy-gate flake: test_frontdoor TestSourceSideSequencing.test_stalled_member_drop_does_not_wedge_the_group fails under hub load - #2113

Draft
harmoniqs-amico-agent[bot] wants to merge 1 commit into
lane/0.4.1from
night/amicode-1997-2026-11-03
Draft

night: deploy-gate flake: test_frontdoor TestSourceSideSequencing.test_stalled_member_drop_does_not_wedge_the_group fails under hub load#2113
harmoniqs-amico-agent[bot] wants to merge 1 commit into
lane/0.4.1from
night/amicode-1997-2026-11-03

Conversation

@harmoniqs-amico-agent

@harmoniqs-amico-agent harmoniqs-amico-agent Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Closes #1997

Night-runner draft (notturno, amicissimo#531): implemented headlessly in a Modal sandbox, gates .notturno/gates green. Ready for review — never auto-merged by this slice.

@harmoniqs-amico-agent
test(hub): target the stalled-member flood at /event (#1997)
03616d8 
The untargeted raw flood was drained by whichever shard-0 stream ticked
first; sometimes /global/event, which no member reads, so the slow member
never filled its queue and the drop never happened. Target the flood at
the member's /event path (the tuple form other tests use). The drop-path
claim is unchanged: the group lock must stay free during the stall close.
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 33511b21-6c7d-4287-b88a-afdc703fa143

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

0 participants