Skip to content

[FIXED] Fix TestNRGSwitchStateClearsQueues#7230

Merged
neilalexander merged 1 commit intomainfrom
fix-TestNRGSwitchStateClearsQueue
Aug 28, 2025
Merged

[FIXED] Fix TestNRGSwitchStateClearsQueues#7230
neilalexander merged 1 commit intomainfrom
fix-TestNRGSwitchStateClearsQueue

Conversation

@sciascid
Copy link
Copy Markdown
Contributor

@sciascid sciascid commented Aug 28, 2025

The test leaves behind file server/tav.idx. Setting the raft's store to use a temporary directory. This gets automatically removed at the end of test.

Signed-off-by: Daniele Sciascia daniele@nats.io

@sciascid sciascid requested a review from a team as a code owner August 28, 2025 13:09
The test leaves behind file server/tav.idx. Setting the raft's
store to use a temporary directory. This gets automatically
removed at the end of test.

Signed-off-by: Daniele Sciascia <daniele@nats.io>
@sciascid sciascid force-pushed the fix-TestNRGSwitchStateClearsQueue branch from fd2b6fb to c907a7f Compare August 28, 2025 13:13
Copy link
Copy Markdown
Member

@MauriceVanVeen MauriceVanVeen left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

LGTM!

@neilalexander neilalexander merged commit bd0b623 into main Aug 28, 2025
48 checks passed
@neilalexander neilalexander deleted the fix-TestNRGSwitchStateClearsQueue branch August 28, 2025 14:19
MauriceVanVeen pushed a commit that referenced this pull request Sep 3, 2025
The test leaves behind file server/tav.idx. Setting the raft's store to
use a temporary directory. This gets automatically removed at the end of
test.

Signed-off-by: Daniele Sciascia <daniele@nats.io>
@wallyqs wallyqs changed the title Fix TestNRGSwitchStateClearsQueues [FIXED] Fix TestNRGSwitchStateClearsQueues Sep 3, 2025
neilalexander added a commit that referenced this pull request Sep 8, 2025
Includes the following:
- #7200
- #7201
- #7202
- #7209
- #7210
- #7211
- #7213
- #7212
- #7216
- #7217
- #7230
- #7239
- #7246
- #7248
-
8241a15,
specifically delayed errors that are not JS API errors
- #7158 (not containing
2.12-specific changes)
- #7233
- #7255
- #7249
- #7259
- #7265
- #7273 (not including Go
1.25.x)
- #7258
- #7222

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
Signed-off-by: Neil Twigg <neil@nats.io>
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.

3 participants