X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/f79d1cdf7696a3bbcc374df5ce4d1761a28a5ea5..e0dfdad390b88924f15e4f6e7d8e7eadd585e06f:/services/keepstore/command.go

diff --git a/services/keepstore/command.go b/services/keepstore/command.go
index d01b30c907..9f14c13384 100644
--- a/services/keepstore/command.go
+++ b/services/keepstore/command.go
@@ -7,100 +7,16 @@ package keepstore
 import (
 	"context"
 	"errors"
-	"flag"
-	"io"
 
-	"git.arvados.org/arvados.git/lib/cmd"
-	"git.arvados.org/arvados.git/lib/config"
 	"git.arvados.org/arvados.git/lib/service"
 	"git.arvados.org/arvados.git/sdk/go/arvados"
-	"git.arvados.org/arvados.git/sdk/go/ctxlog"
 	"github.com/prometheus/client_golang/prometheus"
-	"github.com/sirupsen/logrus"
 )
 
 var (
 	Command = service.Command(arvados.ServiceNameKeepstore, newHandlerOrErrorHandler)
 )
 
-func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.Writer) int {
-	args, ok, code := convertKeepstoreFlagsToServiceFlags(prog, args, ctxlog.FromContext(context.Background()), stderr)
-	if !ok {
-		return code
-	}
-	return Command.RunCommand(prog, args, stdin, stdout, stderr)
-}
-
-// Parse keepstore command line flags, and return equivalent
-// service.Command flags. If the second return value ("ok") is false,
-// the program should exit, and the third return value is a suitable
-// exit code.
-func convertKeepstoreFlagsToServiceFlags(prog string, args []string, lgr logrus.FieldLogger, stderr io.Writer) ([]string, bool, int) {
-	flags := flag.NewFlagSet("", flag.ContinueOnError)
-	flags.String("listen", "", "Services.Keepstore.InternalURLs")
-	flags.Int("max-buffers", 0, "API.MaxKeepBlobBuffers")
-	flags.Int("max-requests", 0, "API.MaxConcurrentRequests")
-	flags.Bool("never-delete", false, "Collections.BlobTrash")
-	flags.Bool("enforce-permissions", false, "Collections.BlobSigning")
-	flags.String("permission-key-file", "", "Collections.BlobSigningKey")
-	flags.String("blob-signing-key-file", "", "Collections.BlobSigningKey")
-	flags.String("data-manager-token-file", "", "SystemRootToken")
-	flags.Int("permission-ttl", 0, "Collections.BlobSigningTTL")
-	flags.Int("blob-signature-ttl", 0, "Collections.BlobSigningTTL")
-	flags.String("trash-lifetime", "", "Collections.BlobTrashLifetime")
-	flags.Bool("serialize", false, "Volumes.*.DriverParameters.Serialize")
-	flags.Bool("readonly", false, "Volumes.*.ReadOnly")
-	flags.String("pid", "", "-")
-	flags.String("trash-check-interval", "", "Collections.BlobTrashCheckInterval")
-
-	flags.String("azure-storage-container-volume", "", "Volumes.*.Driver")
-	flags.String("azure-storage-account-name", "", "Volumes.*.DriverParameters.StorageAccountName")
-	flags.String("azure-storage-account-key-file", "", "Volumes.*.DriverParameters.StorageAccountKey")
-	flags.String("azure-storage-replication", "", "Volumes.*.Replication")
-	flags.String("azure-max-get-bytes", "", "Volumes.*.DriverParameters.MaxDataReadSize")
-
-	flags.String("s3-bucket-volume", "", "Volumes.*.DriverParameters.Bucket")
-	flags.String("s3-region", "", "Volumes.*.DriverParameters.Region")
-	flags.String("s3-endpoint", "", "Volumes.*.DriverParameters.Endpoint")
-	flags.String("s3-access-key-file", "", "Volumes.*.DriverParameters.AccessKeyID")
-	flags.String("s3-secret-key-file", "", "Volumes.*.DriverParameters.SecretAccessKey")
-	flags.String("s3-race-window", "", "Volumes.*.DriverParameters.RaceWindow")
-	flags.String("s3-replication", "", "Volumes.*.Replication")
-	flags.String("s3-unsafe-delete", "", "Volumes.*.DriverParameters.UnsafeDelete")
-
-	flags.String("volume", "", "Volumes")
-
-	flags.Bool("version", false, "")
-	flags.String("config", "", "")
-	flags.String("legacy-keepstore-config", "", "")
-
-	if ok, code := cmd.ParseFlags(flags, prog, args, "", stderr); !ok {
-		return nil, false, code
-	}
-
-	args = nil
-	ok := true
-	flags.Visit(func(f *flag.Flag) {
-		if f.Name == "config" || f.Name == "legacy-keepstore-config" || f.Name == "version" {
-			args = append(args, "-"+f.Name, f.Value.String())
-		} else if f.Usage == "-" {
-			ok = false
-			lgr.Errorf("command line flag -%s is no longer supported", f.Name)
-		} else {
-			ok = false
-			lgr.Errorf("command line flag -%s is no longer supported -- use Clusters.*.%s in cluster config file instead", f.Name, f.Usage)
-		}
-	})
-	if !ok {
-		return nil, false, 2
-	}
-
-	flags = flag.NewFlagSet("", flag.ContinueOnError)
-	loader := config.NewLoader(nil, lgr)
-	loader.SetupFlags(flags)
-	return loader.MungeLegacyConfigArgs(lgr, args, "-legacy-keepstore-config"), true, 0
-}
-
 func newHandlerOrErrorHandler(ctx context.Context, cluster *arvados.Cluster, token string, reg *prometheus.Registry) service.Handler {
 	serviceURL, ok := service.URLFromContext(ctx)
 	if !ok {