feat: HA tailnet coordinator #4170
Conversation
| "github.com/coder/coder/testutil" | ||
| ) | ||
|
|
||
| func TestCoordinatorSingle(t *testing.T) { |
There was a problem hiding this comment.
I think writing a benchmark would be helpful before merge. @johnstcn ran into some oddities with PostgreSQL pubsub performance, so we should ensure those limits aren't applicable here.
There was a problem hiding this comment.
Since this already works fine for v1 I think it would be good to find the limits for this later and recommend scaling in the docs.
|
|
||
| t.Run("AgentWithoutClients", func(t *testing.T) { | ||
| t.Parallel() | ||
| coordinator, err := tailnet.NewHACoordinator(slogtest.Make(t, nil), database.NewPubsubInMemory()) |
There was a problem hiding this comment.
We should probably test this against multiple coordinators working together, otherwise it's not really replicating a deployment scenario.
There was a problem hiding this comment.
I have the HA tests below
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
a7f1727 to
5c43d63
Compare
|
I have read the CLA Document and I hereby sign the CLA |
|
recheck |
|
This was merged in #3227 |
No description provided.