cameri / nats-server
83%

DEFAULT BRANCH: main
Repo Added
Token 2QKcehgSsljsL9v8rBel0tjHUQKpp0kYJ regen
Build 2 Last
Files 66
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 82.561% (-0.04%) from 82.602%
39098288153

push

github

web-flow 
[FIXED] Remove stale JetStream account usage after peer removal (#8570)

Fixes #8563.

When a JetStream server is permanently removed from the cluster, its
last reported account usage can remain in `jsAccount.rusage`. Because
that value is still included in the account totals, replacing a stream
replica on another server can cause storage usage to be counted twice.

I reproduced this with a 4-node cluster and an R3 stream. After removing
one of the replica-holding servers and letting the stream move to the
spare node, the stream was still correctly running with 3 replicas, but
the account storage usage increased as if there were 4 copies.

This change removes the departed server's usage when its meta peer
removal is committed. It also ignores a late usage update from the
removed node so that the stale entry cannot be added back after cleanup.

I added a regression test that:

* creates an R3 stream on a 4-node cluster,
* verifies the initial account usage,
* removes one of the servers hosting a replica,
* waits for the replica to move to the spare server,
* verifies the account usage stays unchanged,
* and checks that a late usage update from the removed peer does not
recreate the stale entry.

Testing:

* new regression test passes repeatedly
* regression test passes with `-race`
* related peer-removal and account-usage tests pass
* `go vet ./server`
* `git diff --check`

71360 of 86433 relevant lines covered (82.56%)

354504.55 hits per line

Relevant lines Covered
86433 RELEVANT LINES 71360 COVERED LINES
354504.55 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
39098288153 main [FIXED] Remove stale JetStream account usage after peer removal (#8570) Fixes #8563. When a JetStream server is permanently removed from the cluster, its last reported account usage can remain in `jsAccount.rusage`. Because that value is still i... push web-flow github
82.56
38852520035 main [FIXED] Remove stale JetStream account usage after peer removal (#8570) Fixes #8563. When a JetStream server is permanently removed from the cluster, its last reported account usage can remain in `jsAccount.rusage`. Because that value is still i... push web-flow github
82.6
See All Builds (2)

Badge your Repo: nats-server

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh