Skip to content

chore: replace wsconncache with a single tailnet #8176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Jul 12, 2023
Merged

chore: replace wsconncache with a single tailnet #8176

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
7222135
chore: replace wsconncache with a single tailnet
coadler Jun 26, 2023
d4181b0
remove duplicate AwaitReachable
coadler Jun 26, 2023
2a133d1
properly release net.Conn
coadler Jun 26, 2023
aa6bcb6
tailnetTransport
coadler Jun 26, 2023
f9040fc
link to issue in deprecation notice
coadler Jun 26, 2023
a988fef
address review comments
coadler Jun 28, 2023
2e201dc
fixup! address review comments
coadler Jun 28, 2023
5901f68
Merge branch 'main' into colin/rm-wsconncache2
coadler Jun 28, 2023
311ea2b
second round
coadler Jun 28, 2023
30aefcb
fixup! second round
coadler Jun 28, 2023
e55e146
support swapping coordinators
coadler Jun 29, 2023
457470d
use multiagents for all clients
coadler Jun 29, 2023
2c32434
Merge branch 'main' into colin/rm-wsconncache2
coadler Jun 29, 2023
11f2805
fixups
coadler Jun 30, 2023
a88be66
fixup! fixups
coadler Jun 30, 2023
49a8364
Merge branch 'main' into colin/rm-wsconncache2
coadler Jun 30, 2023
5e4b631
fixes
coadler Jun 30, 2023
8ef2fb7
Merge branch 'main' into colin/rm-wsconncache2
coadler Jun 30, 2023
9864d89
Merge branch 'main' into colin/rm-wsconncache2
coadler Jul 7, 2023
dd3cc15
comment
coadler Jul 7, 2023
8896ae4
fixup! comment
coadler Jul 7, 2023
be4db71
fixup! comment
coadler Jul 7, 2023
7bfac9b
add feature flag
coadler Jul 11, 2023
10d44fe
Merge branch 'main' into colin/rm-wsconncache2
coadler Jul 11, 2023
ef44092
fixes
coadler Jul 12, 2023
9a29d85
Merge branch 'main' into colin/rm-wsconncache2
coadler Jul 12, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into colin/rm-wsconncache2
  • Loading branch information
@coadler
coadler committed Jun 28, 2023
commit 5901f686b2cc95e86709b5d832cb3d5c92d1d857
26 changes: 12 additions & 14 deletions agent/agent_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ import (
"github.com/stretchr/testify/require"
"go.uber.org/goleak"
"golang.org/x/crypto/ssh"
"golang.org/x/exp/slices"
"golang.org/x/xerrors"
"tailscale.com/net/speedtest"
"tailscale.com/tailcfg"
Expand Down Expand Up @@ -1109,7 +1110,7 @@ func TestAgent_Lifecycle(t *testing.T) {
var got []codersdk.WorkspaceAgentLifecycle
assert.Eventually(t, func() bool {
got = client.GetLifecycleStates()
return len(got) > 0 && got[len(got)-1] == want[len(want)-1]
return slices.Contains(got, want[len(want)-1])
}, testutil.WaitShort, testutil.IntervalMedium)

require.Equal(t, want, got[:len(want)])
Expand All @@ -1131,7 +1132,7 @@ func TestAgent_Lifecycle(t *testing.T) {
var got []codersdk.WorkspaceAgentLifecycle
assert.Eventually(t, func() bool {
got = client.GetLifecycleStates()
return len(got) > 0 && got[len(got)-1] == want[len(want)-1]
return slices.Contains(got, want[len(want)-1])
}, testutil.WaitShort, testutil.IntervalMedium)

require.Equal(t, want, got[:len(want)])
Expand Down Expand Up @@ -1168,8 +1169,7 @@ func TestAgent_Lifecycle(t *testing.T) {
}, 0)

assert.Eventually(t, func() bool {
ready = client.GetLifecycleStates()
return len(ready) > 0 && ready[len(ready)-1] == codersdk.WorkspaceAgentLifecycleReady
return slices.Contains(client.GetLifecycleStates(), codersdk.WorkspaceAgentLifecycleReady)
}, testutil.WaitShort, testutil.IntervalMedium)

// Start close asynchronously so that we an inspect the state.
Expand All @@ -1191,8 +1191,8 @@ func TestAgent_Lifecycle(t *testing.T) {

var got []codersdk.WorkspaceAgentLifecycle
assert.Eventually(t, func() bool {
got = client.GetLifecycleStates()[len(ready):]
return len(got) > 0 && got[len(got)-1] == want[len(want)-1]
got = client.GetLifecycleStates()
return slices.Contains(got, want[len(want)-1])
}, testutil.WaitShort, testutil.IntervalMedium)

require.Equal(t, want, got[:len(want)])
Expand All @@ -1207,8 +1207,7 @@ func TestAgent_Lifecycle(t *testing.T) {
}, 0)

assert.Eventually(t, func() bool {
ready = client.GetLifecycleStates()
return len(ready) > 0 && ready[len(ready)-1] == codersdk.WorkspaceAgentLifecycleReady
return slices.Contains(client.GetLifecycleStates(), codersdk.WorkspaceAgentLifecycleReady)
}, testutil.WaitShort, testutil.IntervalMedium)

// Start close asynchronously so that we an inspect the state.
Expand All @@ -1231,8 +1230,8 @@ func TestAgent_Lifecycle(t *testing.T) {

var got []codersdk.WorkspaceAgentLifecycle
assert.Eventually(t, func() bool {
got = client.GetLifecycleStates()[len(ready):]
return len(got) > 0 && got[len(got)-1] == want[len(want)-1]
got = client.GetLifecycleStates()
return slices.Contains(got, want[len(want)-1])
}, testutil.WaitShort, testutil.IntervalMedium)

require.Equal(t, want, got[:len(want)])
Expand All @@ -1247,8 +1246,7 @@ func TestAgent_Lifecycle(t *testing.T) {
}, 0)

assert.Eventually(t, func() bool {
ready = client.GetLifecycleStates()
return len(ready) > 0 && ready[len(ready)-1] == codersdk.WorkspaceAgentLifecycleReady
return slices.Contains(client.GetLifecycleStates(), codersdk.WorkspaceAgentLifecycleReady)
}, testutil.WaitShort, testutil.IntervalMedium)

// Start close asynchronously so that we an inspect the state.
Expand All @@ -1271,8 +1269,8 @@ func TestAgent_Lifecycle(t *testing.T) {

var got []codersdk.WorkspaceAgentLifecycle
assert.Eventually(t, func() bool {
got = client.GetLifecycleStates()[len(ready):]
return len(got) > 0 && got[len(got)-1] == want[len(want)-1]
got = client.GetLifecycleStates()
return slices.Contains(got, want[len(want)-1])
}, testutil.WaitShort, testutil.IntervalMedium)

require.Equal(t, want, got[:len(want)])
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.