Skip to content

Commit

Permalink
fix TestJetStreamJWTMove flapper (#6115)
Browse files Browse the repository at this point in the history
test(flapper): fix TestJetStreamJWTMove - by removing a check on stream
update for the configuration, likely the server was able to accomplish
the move, right after, the test will assert that C2 is the stream was
placed in the correct cluster



Signed-off-by: Alberto Ricart <alberto@synadia.com>
  • Loading branch information
derekcollison authored Nov 13, 2024
2 parents d727886 + 6dd76cf commit e1b2f5d
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions server/jetstream_jwt_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -345,10 +345,9 @@ func TestJetStreamJWTMove(t *testing.T) {
require_NoError(t, err)

// Perform actual move
ci, err = js.UpdateStream(&nats.StreamConfig{Name: "MOVE-ME", Replicas: replicas,
_, err = js.UpdateStream(&nats.StreamConfig{Name: "MOVE-ME", Replicas: replicas,
Placement: &nats.Placement{Tags: []string{"cloud:C2-tag"}}})
require_NoError(t, err)
require_Equal(t, ci.Cluster.Name, "C1")

sc.clusterForName("C2").waitOnStreamLeader(aExpPub, "MOVE-ME")

Expand Down

0 comments on commit e1b2f5d

Please sign in to comment.