14714: Updates KeepServiceTypes legacy mapping and error messages
[arvados.git] / services / keep-balance / main.go
index 84516a821060da1b795da1b40655a9a62157fd52..4379ae535e85f580f63d0e6f040084996cb5edf9 100644 (file)
@@ -5,97 +5,98 @@
 package main
 
 import (
-       "encoding/json"
+       "context"
        "flag"
        "fmt"
+       "io"
        "log"
-       "net/http"
        "os"
-       "time"
 
+       "git.curoverse.com/arvados.git/lib/config"
+       "git.curoverse.com/arvados.git/lib/service"
        "git.curoverse.com/arvados.git/sdk/go/arvados"
-       "git.curoverse.com/arvados.git/sdk/go/config"
+       "git.curoverse.com/arvados.git/sdk/go/ctxlog"
+       "github.com/prometheus/client_golang/prometheus"
        "github.com/sirupsen/logrus"
 )
 
-var debugf = func(string, ...interface{}) {}
+var (
+       version = "dev"
+       debugf  = func(string, ...interface{}) {}
+       command = service.Command(arvados.ServiceNameKeepbalance, newHandler)
+       options RunOptions
+)
 
-func main() {
-       var cfg Config
-       var runOptions RunOptions
-
-       configPath := flag.String("config", defaultConfigPath,
-               "`path` of JSON or YAML configuration file")
-       serviceListPath := flag.String("config.KeepServiceList", "",
-               "`path` of JSON or YAML file with list of keep services to balance, as given by \"arv keep_service list\" "+
-                       "(default: config[\"KeepServiceList\"], or if none given, get all available services and filter by config[\"KeepServiceTypes\"])")
-       flag.BoolVar(&runOptions.Once, "once", false,
-               "balance once and then exit")
-       flag.BoolVar(&runOptions.CommitPulls, "commit-pulls", false,
-               "send pull requests (make more replicas of blocks that are underreplicated or are not in optimal rendezvous probe order)")
-       flag.BoolVar(&runOptions.CommitTrash, "commit-trash", false,
-               "send trash requests (delete unreferenced old blocks, and excess replicas of overreplicated blocks)")
-       dumpConfig := flag.Bool("dump-config", false, "write current configuration to stdout and exit")
-       dumpFlag := flag.Bool("dump", false, "dump details for each block to stdout")
-       debugFlag := flag.Bool("debug", false, "enable debug messages")
-       getVersion := flag.Bool("version", false, "Print version information and exit.")
-       flag.Usage = usage
-       flag.Parse()
-
-       // Print version information if requested
-       if *getVersion {
-               fmt.Printf("keep-balance %s\n", version)
-               return
+func newHandler(ctx context.Context, cluster *arvados.Cluster, token string, registry *prometheus.Registry) service.Handler {
+       if !options.Once && cluster.Collections.BalancePeriod == arvados.Duration(0) {
+               return service.ErrorHandler(ctx, cluster, fmt.Errorf("cannot start service: Collections.BalancePeriod is zero (if you want to run once and then exit, use the -once flag)"))
+                       "If using the legacy keep-balance.yml config, RunPeriod is the equivalant of Collections.BalancePeriod."))
        }
 
-       mustReadConfig(&cfg, *configPath)
-       if *serviceListPath != "" {
-               mustReadConfig(&cfg.KeepServiceList, *serviceListPath)
+       ac, err := arvados.NewClientFromConfig(cluster)
+       ac.AuthToken = token
+       if err != nil {
+               return service.ErrorHandler(ctx, cluster, fmt.Errorf("error initializing client from cluster config: %s", err))
        }
 
-       if *dumpConfig {
-               log.Fatal(config.DumpAndExit(cfg))
+       if cluster.SystemLogs.LogLevel == "debug" {
+               debugf = log.Printf
        }
 
-       to := time.Duration(cfg.RequestTimeout)
-       if to == 0 {
-               to = 30 * time.Minute
+       if options.Logger == nil {
+               options.Logger = ctxlog.FromContext(ctx)
        }
-       arvados.DefaultSecureClient.Timeout = to
-       arvados.InsecureHTTPClient.Timeout = to
-       http.DefaultClient.Timeout = to
-
-       log.Printf("keep-balance %s started", version)
 
-       if *debugFlag {
-               debugf = log.Printf
-               if j, err := json.Marshal(cfg); err != nil {
-                       log.Fatal(err)
-               } else {
-                       log.Printf("config is %s", j)
-               }
+       srv := &Server{
+               Cluster:    cluster,
+               ArvClient:  ac,
+               RunOptions: options,
+               Metrics:    newMetrics(registry),
+               Logger:     options.Logger,
+               Dumper:     options.Dumper,
        }
+
+       srv.Start()
+       return srv
+}
+
+func main() {
+       os.Exit(runCommand(os.Args[0], os.Args[1:], os.Stdin, os.Stdout, os.Stderr))
+}
+
+func runCommand(prog string, args []string, stdin io.Reader, stdout, stderr io.Writer) int {
+       logger := ctxlog.FromContext(context.Background())
+
+       flags := flag.NewFlagSet(prog, flag.ExitOnError)
+       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")
+       dumpFlag := flags.Bool("dump", false, "dump details for each block to stdout")
+
+       loader := config.NewLoader(os.Stdin, logger)
+       loader.SetupFlags(flags)
+
+       munged := loader.MungeLegacyConfigArgs(logger, args, "-legacy-keepbalance-config")
+       flags.Parse(munged)
+
        if *dumpFlag {
                dumper := logrus.New()
                dumper.Out = os.Stdout
                dumper.Formatter = &logrus.TextFormatter{}
-               runOptions.Dumper = dumper
+               options.Dumper = dumper
        }
-       srv, err := NewServer(cfg, runOptions)
-       if err != nil {
-               // (don't run)
-       } else if runOptions.Once {
-               _, err = srv.Run()
-       } else {
-               err = srv.RunForever(nil)
-       }
-       if err != nil {
-               log.Fatal(err)
-       }
-}
 
-func mustReadConfig(dst interface{}, path string) {
-       if err := config.LoadFile(dst, path); err != nil {
-               log.Fatal(err)
-       }
+       // Only pass along the version flag, which gets handled in RunCommand
+       args = nil
+       flags.Visit(func(f *flag.Flag) {
+               if f.Name == "version" {
+                       args = append(args, "-"+f.Name, f.Value.String())
+               }
+       })
+
+       return command.RunCommand(prog, args, stdin, stdout, stderr)
 }