Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

Commit f44c5ca

Browse files
kylecarbskylecarbs
authored
chore: Test RTC listener reconnects (#342)
* chore: Test RTC listener reconnects * Remove version check for tunnel * Fix linting issues * Update Go version * Pin to 1.16.3
1 parent 08ef03e commit f44c5ca

File tree

19 files changed

+118
-41
lines changed

19 files changed

+118
-41
lines changed

‎.github/workflows/build.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ jobs:
77
steps:
88
- name: Checkout
99
uses: actions/checkout@v1
10+
- uses: actions/setup-go@v2
11+
with:
12+
go-version: '^1.16.3'
1013
- name: Build
1114
run: make -j build/linux build/windows
1215
- name: Upload
@@ -19,6 +22,9 @@ jobs:
1922
steps:
2023
- name: Checkout
2124
uses: actions/checkout@v1
25+
- uses: actions/setup-go@v2
26+
with:
27+
go-version: '^1.16.3'
2228
- name: Install Gon
2329
run: |
2430
brew tap mitchellh/gon

‎ci/image/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM golang:1
1+
FROM golang:1.16.3
22

33
ENV GOFLAGS="-mod=readonly"
44
ENV CI=true

‎internal/cmd/agent.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ coder agent start --coder-url https://my-coder.com --token xxxx-xxxx
5151
var ok bool
5252
coderURL, ok = os.LookupEnv("CODER_URL")
5353
if !ok {
54-
client, err := newClient(ctx)
54+
client, err := newClient(ctx, true)
5555
if err != nil {
5656
return xerrors.New("must login, pass --coder-url flag, or set the CODER_URL env variable")
5757
}

‎internal/cmd/auth.go

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ var errNeedLogin = clog.Fatal(
2323
const tokenEnv = "CODER_TOKEN"
2424
const urlEnv = "CODER_URL"
2525

26-
func newClient(ctx context.Context) (coder.Client, error) {
26+
func newClient(ctx context.Context, checkVersion bool) (coder.Client, error) {
2727
var (
2828
err error
2929
sessionToken = os.Getenv(tokenEnv)
@@ -55,10 +55,14 @@ func newClient(ctx context.Context) (coder.Client, error) {
5555
return nil, xerrors.Errorf("failed to create new coder.Client: %w", err)
5656
}
5757

58-
apiVersion, err := c.APIVersion(ctx)
59-
if apiVersion != "" && !version.VersionsMatch(apiVersion) {
60-
logVersionMismatchError(apiVersion)
58+
if checkVersion {
59+
var apiVersion string
60+
apiVersion, err = c.APIVersion(ctx)
61+
if apiVersion != "" && !version.VersionsMatch(apiVersion) {
62+
logVersionMismatchError(apiVersion)
63+
}
6164
}
65+
6266
if err != nil {
6367
var he *coder.HTTPError
6468
if xerrors.As(err, &he) {

‎internal/cmd/configssh.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ func configSSH(configpath *string, remove *bool, p2p *bool) func(cmd *cobra.Comm
8888
return nil
8989
}
9090

91-
client, err := newClient(ctx)
91+
client, err := newClient(ctx, true)
9292
if err != nil {
9393
return err
9494
}

‎internal/cmd/envs.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ func lsEnvsCommand() *cobra.Command {
5959
Long: "List all Coder environments owned by the active user.",
6060
RunE: func(cmd *cobra.Command, args []string) error {
6161
ctx := cmd.Context()
62-
client, err := newClient(ctx)
62+
client, err := newClient(ctx, true)
6363
if err != nil {
6464
return err
6565
}
@@ -128,7 +128,7 @@ coder envs --user charlie@coder.com ls -o json \
128128
Args: cobra.MinimumNArgs(1),
129129
RunE: func(cmd *cobra.Command, args []string) error {
130130
ctx := cmd.Context()
131-
client, err := newClient(ctx)
131+
client, err := newClient(ctx, true)
132132
if err != nil {
133133
return xerrors.Errorf("new client: %w", err)
134134
}
@@ -189,7 +189,7 @@ coder envs create my-new-powerful-env --cpu 12 --disk 100 --memory 16 --image ub
189189
return xerrors.New("image unset")
190190
}
191191

192-
client, err := newClient(ctx)
192+
client, err := newClient(ctx, true)
193193
if err != nil {
194194
return err
195195
}
@@ -315,7 +315,7 @@ coder envs create-from-config --name="dev-env" -f coder.yaml`,
315315
)
316316
}
317317

318-
client, err := newClient(ctx)
318+
client, err := newClient(ctx, true)
319319
if err != nil {
320320
return err
321321
}
@@ -438,7 +438,7 @@ func editEnvCmd() *cobra.Command {
438438
coder envs edit back-end-env --disk 20`,
439439
RunE: func(cmd *cobra.Command, args []string) error {
440440
ctx := cmd.Context()
441-
client, err := newClient(ctx)
441+
client, err := newClient(ctx, true)
442442
if err != nil {
443443
return err
444444
}
@@ -532,7 +532,7 @@ func rmEnvsCmd() *cobra.Command {
532532
Args: cobra.MinimumNArgs(1),
533533
RunE: func(cmd *cobra.Command, args []string) error {
534534
ctx := cmd.Context()
535-
client, err := newClient(ctx)
535+
client, err := newClient(ctx, true)
536536
if err != nil {
537537
return err
538538
}

‎internal/cmd/images.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ func lsImgsCommand(user *string) *cobra.Command {
3838
RunE: func(cmd *cobra.Command, args []string) error {
3939
ctx := cmd.Context()
4040

41-
client, err := newClient(ctx)
41+
client, err := newClient(ctx, true)
4242
if err != nil {
4343
return err
4444
}

‎internal/cmd/providers.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ coder providers create my-provider --hostname=https://provider.example.com --clu
5050
RunE: func(cmd *cobra.Command, args []string) error {
5151
ctx := cmd.Context()
5252

53-
client, err := newClient(ctx)
53+
client, err := newClient(ctx, true)
5454
if err != nil {
5555
return err
5656
}
@@ -143,7 +143,7 @@ coder providers ls`,
143143
RunE: func(cmd *cobra.Command, args []string) error {
144144
ctx := cmd.Context()
145145

146-
client, err := newClient(ctx)
146+
client, err := newClient(ctx, true)
147147
if err != nil {
148148
return err
149149
}
@@ -174,7 +174,7 @@ func deleteProviderCmd() *cobra.Command {
174174
coder providers rm my-workspace-provider`,
175175
RunE: func(cmd *cobra.Command, args []string) error {
176176
ctx := cmd.Context()
177-
client, err := newClient(ctx)
177+
client, err := newClient(ctx, true)
178178
if err != nil {
179179
return err
180180
}
@@ -232,7 +232,7 @@ func cordonProviderCmd() *cobra.Command {
232232
coder providers cordon my-workspace-provider --reason "limit cloud clost"`,
233233
RunE: func(cmd *cobra.Command, args []string) error {
234234
ctx := cmd.Context()
235-
client, err := newClient(ctx)
235+
client, err := newClient(ctx, true)
236236
if err != nil {
237237
return err
238238
}
@@ -265,7 +265,7 @@ func unCordonProviderCmd() *cobra.Command {
265265
coder providers uncordon my-workspace-provider`,
266266
RunE: func(cmd *cobra.Command, args []string) error {
267267
ctx := cmd.Context()
268-
client, err := newClient(ctx)
268+
client, err := newClient(ctx, true)
269269
if err != nil {
270270
return err
271271
}
@@ -296,7 +296,7 @@ func renameProviderCmd() *cobra.Command {
296296
coder providers rename build-in us-east-1`,
297297
RunE: func(cmd *cobra.Command, args []string) error {
298298
ctx := cmd.Context()
299-
client, err := newClient(ctx)
299+
client, err := newClient(ctx, true)
300300
if err != nil {
301301
return err
302302
}

‎internal/cmd/rebuild.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func rebuildEnvCommand() *cobra.Command {
2929
coder envs rebuild backend-env --force`,
3030
RunE: func(cmd *cobra.Command, args []string) error {
3131
ctx := cmd.Context()
32-
client, err := newClient(ctx)
32+
client, err := newClient(ctx, true)
3333
if err != nil {
3434
return err
3535
}
@@ -166,7 +166,7 @@ func watchBuildLogCommand() *cobra.Command {
166166
Args: xcobra.ExactArgs(1),
167167
RunE: func(cmd *cobra.Command, args []string) error {
168168
ctx := cmd.Context()
169-
client, err := newClient(ctx)
169+
client, err := newClient(ctx, true)
170170
if err != nil {
171171
return err
172172
}

‎internal/cmd/resourcemanager.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ coder resources top --sort-by memory --show-empty`,
6262
func runResourceTop(options *resourceTopOptions) func(cmd *cobra.Command, args []string) error {
6363
return func(cmd *cobra.Command, args []string) error {
6464
ctx := cmd.Context()
65-
client, err := newClient(ctx)
65+
client, err := newClient(ctx, true)
6666
if err != nil {
6767
return err
6868
}

0 commit comments

Comments
 (0)