X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/e8d1a643cdbc3a5f4c0e5c745da58d9f7e1248d8..37d9f94b06ff367a3514b58ec6f0e4d4d0116030:/services/keep-balance/main.go diff --git a/services/keep-balance/main.go b/services/keep-balance/main.go index 8b4ee84c71..8a95d389c8 100644 --- a/services/keep-balance/main.go +++ b/services/keep-balance/main.go @@ -9,13 +9,18 @@ import ( "flag" "fmt" "io" + "net/http" + _ "net/http/pprof" "os" + "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" "git.arvados.org/arvados.git/sdk/go/health" + "github.com/jmoiron/sqlx" + _ "github.com/lib/pq" "github.com/prometheus/client_golang/prometheus" "github.com/sirupsen/logrus" ) @@ -28,21 +33,35 @@ func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.W logger := ctxlog.FromContext(context.Background()) var options RunOptions - flags := flag.NewFlagSet(prog, flag.ExitOnError) + flags := flag.NewFlagSet(prog, flag.ContinueOnError) flags.BoolVar(&options.Once, "once", false, "balance once and then exit") flags.BoolVar(&options.CommitPulls, "commit-pulls", false, "send pull requests (make more replicas of blocks that are underreplicated or are not in optimal rendezvous probe order)") flags.BoolVar(&options.CommitTrash, "commit-trash", false, "send trash requests (delete unreferenced old blocks, and excess replicas of overreplicated blocks)") - flags.Bool("version", false, "Write version information to stdout and exit 0") + flags.BoolVar(&options.CommitConfirmedFields, "commit-confirmed-fields", true, + "update collection fields (replicas_confirmed, storage_classes_confirmed, etc.)") dumpFlag := flags.Bool("dump", false, "dump details for each block to stdout") + pprofAddr := flags.String("pprof", "", "serve Go profile data at `[addr]:port`") + // "show version" is implemented by service.Command, so we + // don't need the var here -- we just need the -version flag + // to pass flags.Parse(). + flags.Bool("version", false, "Write version information to stdout and exit 0") + + if *pprofAddr != "" { + go func() { + logrus.Println(http.ListenAndServe(*pprofAddr, nil)) + }() + } loader := config.NewLoader(os.Stdin, logger) loader.SetupFlags(flags) munged := loader.MungeLegacyConfigArgs(logger, args, "-legacy-keepbalance-config") - flags.Parse(munged) + if ok, code := cmd.ParseFlags(flags, prog, munged, "", stderr); !ok { + return code + } if *dumpFlag { dumper := logrus.New() @@ -55,14 +74,15 @@ func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.W // service.Command args = nil dropFlag := map[string]bool{ - "once": true, - "commit-pulls": true, - "commit-trash": true, - "dump": true, + "once": true, + "commit-pulls": true, + "commit-trash": true, + "commit-confirmed-fields": true, + "dump": true, } flags.Visit(func(f *flag.Flag) { if !dropFlag[f.Name] { - args = append(args, "-"+f.Name, f.Value.String()) + args = append(args, "-"+f.Name+"="+f.Value.String()) } }) @@ -78,6 +98,18 @@ func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.W return service.ErrorHandler(ctx, cluster, fmt.Errorf("error initializing client from cluster config: %s", err)) } + db, err := sqlx.Open("postgres", cluster.PostgreSQL.Connection.String()) + if err != nil { + return service.ErrorHandler(ctx, cluster, fmt.Errorf("postgresql connection failed: %s", err)) + } + if p := cluster.PostgreSQL.ConnectionPool; p > 0 { + db.SetMaxOpenConns(p) + } + err = db.Ping() + if err != nil { + return service.ErrorHandler(ctx, cluster, fmt.Errorf("postgresql connection succeeded but ping failed: %s", err)) + } + if options.Logger == nil { options.Logger = ctxlog.FromContext(ctx) } @@ -89,6 +121,7 @@ func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.W Metrics: newMetrics(registry), Logger: options.Logger, Dumper: options.Dumper, + DB: db, } srv.Handler = &health.Handler{ Token: cluster.ManagementToken,