diff --git a/pkg/kopia/cli/internal/opts/opts.go b/pkg/kopia/cli/internal/opts/opts.go index fb53f5d679..f5d07fc2ff 100644 --- a/pkg/kopia/cli/internal/opts/opts.go +++ b/pkg/kopia/cli/internal/opts/opts.go @@ -36,11 +36,6 @@ func ReadOnly(enabled bool) command.Applier { return command.NewOption("--readonly", enabled) } -// GRPC creates a new gRPC option. -func GRPC(enabled bool) command.Applier { - return command.NewToggleOption("--grpc", "--no-grpc", enabled) -} - // CheckForUpdates creates a new check for updates option. func CheckForUpdates(enabled bool) command.Applier { return command.NewToggleOption("--check-for-updates", "--no-check-for-updates", enabled) diff --git a/pkg/kopia/cli/internal/opts/opts_test.go b/pkg/kopia/cli/internal/opts/opts_test.go index c0e955fd85..810aecf9bf 100644 --- a/pkg/kopia/cli/internal/opts/opts_test.go +++ b/pkg/kopia/cli/internal/opts/opts_test.go @@ -47,11 +47,6 @@ var _ = check.Suite(&test.ArgumentSuite{Cmd: "cmd", Arguments: []test.ArgumentTe Argument: command.NewArguments(opts.ReadOnly(true), opts.ReadOnly(false)), ExpectedCLI: []string{"cmd", "--readonly"}, }, - { - Name: "GRPC", - Argument: command.NewArguments(opts.GRPC(true), opts.GRPC(false)), - ExpectedCLI: []string{"cmd", "--grpc", "--no-grpc"}, - }, { Name: "CheckForUpdates", Argument: command.NewArguments(opts.CheckForUpdates(true), opts.CheckForUpdates(false)), diff --git a/pkg/kopia/cli/repository/repository_connect_server.go b/pkg/kopia/cli/repository/repository_connect_server.go index da4f2925b8..e9314cdb13 100644 --- a/pkg/kopia/cli/repository/repository_connect_server.go +++ b/pkg/kopia/cli/repository/repository_connect_server.go @@ -43,7 +43,6 @@ func ConnectServer(args ConnectServerArgs) (*safecli.Builder, error) { opts.Common(args.Common), cmdRepository, subcmdConnect, subcmdServer, opts.CheckForUpdates(false), - opts.GRPC(false), optReadOnly(args.ReadOnly), opts.Cache(args.Cache), optHostname(args.Hostname), diff --git a/pkg/kopia/cli/repository/repository_connect_server_test.go b/pkg/kopia/cli/repository/repository_connect_server_test.go index b50a03b0f6..25fdae08c8 100644 --- a/pkg/kopia/cli/repository/repository_connect_server_test.go +++ b/pkg/kopia/cli/repository/repository_connect_server_test.go @@ -50,7 +50,6 @@ var _ = check.Suite(test.NewCommandSuite([]test.CommandTest{ "connect", "server", "--no-check-for-updates", - "--no-grpc", "--readonly", "--cache-directory=/tmp/cache.dir", "--content-cache-size-limit-mb=0", diff --git a/pkg/kopia/command/const.go b/pkg/kopia/command/const.go index 125e871975..f958a57e29 100644 --- a/pkg/kopia/command/const.go +++ b/pkg/kopia/command/const.go @@ -39,12 +39,10 @@ const ( filterFlag = "--filter" globalFlag = "--global" jsonFlag = "--json" - legacyAPIFlag = "--legacy-api" logDirectoryFlag = "--log-dir" logLevelFlag = "--log-level" LogLevelError = "error" LogLevelInfo = "info" - noGrpcFlag = "--no-grpc" parallelFlag = "--parallel" passwordFlag = "--password" progressUpdateIntervalFlag = "--progress-update-interval" diff --git a/pkg/kopia/command/repository.go b/pkg/kopia/command/repository.go index 8ce69f7dee..c36b2611f8 100644 --- a/pkg/kopia/command/repository.go +++ b/pkg/kopia/command/repository.go @@ -132,7 +132,7 @@ func RepositoryConnectServerCommand(cmdArgs RepositoryServerCommandArgs) []strin ConfigFilePath: cmdArgs.ConfigFilePath, LogDirectory: cmdArgs.LogDirectory, }) - args = args.AppendLoggable(repositorySubCommand, connectSubCommand, serverSubCommand, noCheckForUpdatesFlag, noGrpcFlag) + args = args.AppendLoggable(repositorySubCommand, connectSubCommand, serverSubCommand, noCheckForUpdatesFlag) if cmdArgs.ReadOnly { args = args.AppendLoggable(readOnlyFlag) diff --git a/pkg/kopia/command/repository_test.go b/pkg/kopia/command/repository_test.go index 9166159db7..df8217fbe3 100644 --- a/pkg/kopia/command/repository_test.go +++ b/pkg/kopia/command/repository_test.go @@ -251,7 +251,6 @@ func (s *RepositoryUtilsSuite) TestRepositoryConnectServerUtil(c *check.C) { "connect", "server", "--no-check-for-updates", - "--no-grpc", "--readonly", "--cache-directory=/tmp/cache.dir", "--content-cache-size-limit-mb=0", diff --git a/pkg/kopia/command/server.go b/pkg/kopia/command/server.go index 6beaabaf5f..c328bc73e6 100644 --- a/pkg/kopia/command/server.go +++ b/pkg/kopia/command/server.go @@ -45,10 +45,6 @@ func ServerStart(cmdArgs ServerStartCommandArgs) []string { args = args.AppendLoggableKV(serverControlUsernameFlag, cmdArgs.ServerUsername) args = args.AppendRedactedKV(serverControlPasswordFlag, cmdArgs.ServerPassword) - // TODO: Remove when GRPC support is added - args = args.AppendLoggable(noGrpcFlag) - args = args.AppendLoggable(legacyAPIFlag) - if cmdArgs.EnablePprof { args = args.AppendLoggable(enablePprof) } diff --git a/pkg/kopia/command/server_test.go b/pkg/kopia/command/server_test.go index b6881cae91..03f42d056c 100644 --- a/pkg/kopia/command/server_test.go +++ b/pkg/kopia/command/server_test.go @@ -51,7 +51,7 @@ func (kServer *KopiaServerTestSuite) TestServerCommands(c *C) { } return ServerStart(args) }, - expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password --no-grpc --legacy-api --enable-pprof --metrics-listen-addr=a-server-address:51516 > /dev/null 2>&1 &", + expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password --enable-pprof --metrics-listen-addr=a-server-address:51516 > /dev/null 2>&1 &", }, { f: func() []string { @@ -67,7 +67,7 @@ func (kServer *KopiaServerTestSuite) TestServerCommands(c *C) { } return ServerStart(args) }, - expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password --no-grpc --legacy-api > /dev/null 2>&1 &", + expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password > /dev/null 2>&1 &", }, { f: func() []string { @@ -83,7 +83,7 @@ func (kServer *KopiaServerTestSuite) TestServerCommands(c *C) { } return ServerStart(args) }, - expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password --no-grpc --legacy-api", + expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --tls-generate-cert --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password", }, { f: func() []string { @@ -99,7 +99,7 @@ func (kServer *KopiaServerTestSuite) TestServerCommands(c *C) { } return ServerStart(args) }, - expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password --no-grpc --legacy-api > /dev/null 2>&1 &", + expectedLog: "bash -o errexit -c kopia --log-level=error --config-file=path/kopia.config --log-dir=cache/log server start --address=a-server-address --tls-cert-file=/path/to/cert/tls.crt --tls-key-file=/path/to/key/tls.key --server-username=a-username@a-hostname --server-password=a-user-password --server-control-username=a-username@a-hostname --server-control-password=a-user-password > /dev/null 2>&1 &", }, { f: func() []string {