Skip to content

Commit

Permalink
Merge #105555
Browse files Browse the repository at this point in the history
105555: roachtest: add delay after adding constraints. r=kvoli a=andrewbaptist

Fixes: #105506.
Fixes: #105505.

Add a delay after adding the constraints before recommissioning the
node. This allows the constraints to propagate to all nodes and avoids
replicas being added back to the node.

Epic: none

Release note: None

Co-authored-by: Andrew Baptist <[email protected]>
  • Loading branch information
craig[bot] and andrewbaptist committed Jun 27, 2023
2 parents 5fdd740 + c858088 commit 0b0a212
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/cmd/roachtest/tests/replicagc.go
Original file line number Diff line number Diff line change
Expand Up @@ -271,6 +271,8 @@ func (h *replicagcTestHelper) isolateDeadNodes(ctx context.Context, runNode int)
h.t.Fatal(err)
}
}
// Wait to make sure the new constraints are read by all nodes.
time.Sleep(time.Minute)
}

func waitForZeroReplicasOnN3(ctx context.Context, t test.Test, db *gosql.DB) {
Expand Down

0 comments on commit 0b0a212

Please sign in to comment.