Skip to content

feat: add customizable upgrade message on client/server version mismatch #11587

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 19 commits into from
Jan 30, 2024
Merged

feat: add customizable upgrade message on client/server version mismatch #11587

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
442125d
feat: add customizable upgrade message on client/server version mismatch
sreya Jan 11, 2024
b2a0b8b
make gen
sreya Jan 12, 2024
d3fb0b8
typo
sreya Jan 12, 2024
d5cf4e1
golden
sreya Jan 12, 2024
8e6385a
delete cruft
sreya Jan 12, 2024
d161287
make gen
sreya Jan 16, 2024
42cae32
Merge branch 'main' into jon/installmsg
sreya Jan 29, 2024
61bef0a
pr comments
sreya Jan 29, 2024
460f8cf
make gen
sreya Jan 29, 2024
790cfc3
update golden files
sreya Jan 29, 2024
1749686
pr comments
sreya Jan 30, 2024
0531bd3
fix default value
sreya Jan 30, 2024
c9921fc
lint
sreya Jan 30, 2024
d880b9e
fmt
sreya Jan 30, 2024
fc2c926
golden
sreya Jan 30, 2024
180212c
Merge branch 'main' into jon/installmsg
sreya Jan 30, 2024
7b1be43
fix version formatting
sreya Jan 30, 2024
423ab8b
move trimming into helper
sreya Jan 30, 2024
bf2dce1
fix merge
sreya Jan 30, 2024
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
move trimming into helper
  • Loading branch information
@sreya
sreya committed Jan 30, 2024
commit 423ab8be7ce8a5f6ff581c06f13a74a890568528
9 changes: 5 additions & 4 deletions cli/root.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -835,7 +835,7 @@ func (r *RootCmd) checkVersions(i *clibase.Invocation, client *codersdk.Client,
}

if !buildinfo.VersionsMatch(clientVersion, serverInfo.Version) {
upgradeMessage := defaultUpgradeMessage(strings.TrimPrefix(serverInfo.CanonicalVersion(), "v"))
upgradeMessage := defaultUpgradeMessage(serverInfo.CanonicalVersion())
if serverInfo.UpgradeMessage != "" {
upgradeMessage = serverInfo.UpgradeMessage
}
Expand Down Expand Up @@ -1218,11 +1218,12 @@ func SlimUnsupported(w io.Writer, cmd string) {
os.Exit(1)
}

func defaultUpgradeMessage(serverVersion string) string {
func defaultUpgradeMessage(version string) string {
// Our installation script doesn't work on Windows, so instead we direct the user
// to the GitHub release page to download the latest installer.
version = strings.TrimPrefix(version, "v")
if runtime.GOOS == "windows" {
return fmt.Sprintf("download the server version from: https://github.com/coder/coder/releases/v%s", serverVersion)
return fmt.Sprintf("download the server version from: https://github.com/coder/coder/releases/v%s", version)
}
return fmt.Sprintf("download the server version with: 'curl -L https://coder.com/install.sh | sh -s -- --version %s'", serverVersion)
return fmt.Sprintf("download the server version with: 'curl -L https://coder.com/install.sh | sh -s -- --version %s'", version)
}
2 changes: 1 addition & 1 deletion cli/root_internal_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ func Test_checkVersions(t *testing.T) {
err = r.checkVersions(inv, client, "v2.0.0")
require.NoError(t, err)

fmtOutput := fmt.Sprintf("version mismatch: client v2.0.0, server v1.0.0\n%s", defaultUpgradeMessage("1.0.0"))
fmtOutput := fmt.Sprintf("version mismatch: client v2.0.0, server v1.0.0\n%s", defaultUpgradeMessage("v1.0.0"))
expectedOutput := fmt.Sprintln(pretty.Sprint(cliui.DefaultStyles.Warn, fmtOutput))
require.Equal(t, expectedOutput, buf.String())
})
Expand Down