Remove legacy-api and no-grpc flags from kopia commands (#2836)

* Remove legacy-api and no-grpc flags * Remove grpc opt for repo connect server cmd --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
kanisterio · Apr 24, 2024 · b5a9d2e · b5a9d2e
1 parent 300d81a
commit b5a9d2e
Show file tree

Hide file tree

Showing 9 changed files with 5 additions and 24 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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 {