search

nats-io / nats-server

High-Performance server for NATS.io, the cloud and edge native messaging system.
https://nats.io
Apache License 2.0
16.02k stars 1.41k forks source link

[FIXED] Panic on nil sysAcc #6075

Closed MauriceVanVeen closed 2 weeks ago

MauriceVanVeen commented 2 weeks ago

Signed-off-by: Maurice van Veen github@mauricevanveen.com

derekcollison commented 2 weeks ago

This test is flapping all of the sudden. @MauriceVanVeen can you take a look?

TestNoRaceJetStreamClusterGhostConsumers

image
MauriceVanVeen commented 2 weeks ago

That test was already a flapper before, there are also other flappers, like TestNoRaceJetStreamClusterCheckInterestStatePerformanceWQ

=== RUN   TestNoRaceJetStreamClusterCheckInterestStatePerformanceWQ
    norace_test.go:11062: require true, but got false
--- FAIL: TestNoRaceJetStreamClusterCheckInterestStatePerformanceWQ (2.91s)

We need to take a look at them either way, but just wanted to say it's not because of this PR/change.