X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/25d42fc3206cd6073d4a9d8358469628643fe1da..c8bfd534fc6e33b0b37f9fed1ee6232159edb631:/lib/config/deprecated.go diff --git a/lib/config/deprecated.go b/lib/config/deprecated.go index 0b0bb26689..593a9bdcbc 100644 --- a/lib/config/deprecated.go +++ b/lib/config/deprecated.go @@ -102,42 +102,448 @@ func applyDeprecatedNodeProfile(hostname string, ssi systemServiceInstance, svc svc.InternalURLs[arvados.URL{Scheme: scheme, Host: host}] = arvados.ServiceInstance{} } -const defaultKeepstoreConfigPath = "/etc/arvados/keepstore/keepstore.yml" +func (ldr *Loader) loadOldConfigHelper(component, path string, target interface{}) error { + if path == "" { + return nil + } + buf, err := ioutil.ReadFile(path) + if err != nil { + return err + } + + ldr.Logger.Warnf("you should remove the legacy %v config file (%s) after migrating all config keys to the cluster configuration file (%s)", component, path, ldr.Path) + + err = yaml.Unmarshal(buf, target) + if err != nil { + return fmt.Errorf("%s: %s", path, err) + } + return nil +} + +type oldCrunchDispatchSlurmConfig struct { + Client *arvados.Client + + SbatchArguments *[]string + PollPeriod *arvados.Duration + PrioritySpread *int64 + + // crunch-run command to invoke. The container UUID will be + // appended. If nil, []string{"crunch-run"} will be used. + // + // Example: []string{"crunch-run", "--cgroup-parent-subsystem=memory"} + CrunchRunCommand *[]string + + // Extra RAM to reserve (in Bytes) for SLURM job, in addition + // to the amount specified in the container's RuntimeConstraints + ReserveExtraRAM *int64 -type oldKeepstoreConfig struct { - Debug *bool + // Minimum time between two attempts to run the same container + MinRetryPeriod *arvados.Duration + + // Batch size for container queries + BatchSize *int64 } -// update config using values from an old-style keepstore config file. -func (ldr *Loader) loadOldKeepstoreConfig(cfg *arvados.Config) error { - path := ldr.KeepstorePath - if path == "" { +const defaultCrunchDispatchSlurmConfigPath = "/etc/arvados/crunch-dispatch-slurm/crunch-dispatch-slurm.yml" + +func loadOldClientConfig(cluster *arvados.Cluster, client *arvados.Client) { + if client == nil { + return + } + if client.APIHost != "" { + cluster.Services.Controller.ExternalURL.Host = client.APIHost + } + if client.Scheme != "" { + cluster.Services.Controller.ExternalURL.Scheme = client.Scheme + } else { + cluster.Services.Controller.ExternalURL.Scheme = "https" + } + if client.AuthToken != "" { + cluster.SystemRootToken = client.AuthToken + } + cluster.TLS.Insecure = client.Insecure + ks := "" + for i, u := range client.KeepServiceURIs { + if i > 0 { + ks += " " + } + ks += u + } + cluster.Containers.SLURM.SbatchEnvironmentVariables = map[string]string{"ARVADOS_KEEP_SERVICES": ks} +} + +// update config using values from an crunch-dispatch-slurm config file. +func (ldr *Loader) loadOldCrunchDispatchSlurmConfig(cfg *arvados.Config) error { + if ldr.CrunchDispatchSlurmPath == "" { return nil } - buf, err := ioutil.ReadFile(path) - if os.IsNotExist(err) && path == defaultKeepstoreConfigPath { + var oc oldCrunchDispatchSlurmConfig + err := ldr.loadOldConfigHelper("crunch-dispatch-slurm", ldr.CrunchDispatchSlurmPath, &oc) + if os.IsNotExist(err) && (ldr.CrunchDispatchSlurmPath == defaultCrunchDispatchSlurmConfigPath) { return nil } else if err != nil { return err - } else { - ldr.Logger.Warnf("you should remove the legacy keepstore config file (%s) after migrating all config keys to the cluster configuration file (%s)", path, ldr.Path) } + cluster, err := cfg.GetCluster("") if err != nil { return err } - var oc oldKeepstoreConfig - err = yaml.Unmarshal(buf, &oc) + loadOldClientConfig(cluster, oc.Client) + + if oc.SbatchArguments != nil { + cluster.Containers.SLURM.SbatchArgumentsList = *oc.SbatchArguments + } + if oc.PollPeriod != nil { + cluster.Containers.CloudVMs.PollInterval = *oc.PollPeriod + } + if oc.PrioritySpread != nil { + cluster.Containers.SLURM.PrioritySpread = *oc.PrioritySpread + } + if oc.CrunchRunCommand != nil { + if len(*oc.CrunchRunCommand) >= 1 { + cluster.Containers.CrunchRunCommand = (*oc.CrunchRunCommand)[0] + } + if len(*oc.CrunchRunCommand) >= 2 { + cluster.Containers.CrunchRunArgumentsList = (*oc.CrunchRunCommand)[1:] + } + } + if oc.ReserveExtraRAM != nil { + cluster.Containers.ReserveExtraRAM = arvados.ByteSize(*oc.ReserveExtraRAM) + } + if oc.MinRetryPeriod != nil { + cluster.Containers.MinRetryPeriod = *oc.MinRetryPeriod + } + if oc.BatchSize != nil { + cluster.API.MaxItemsPerResponse = int(*oc.BatchSize) + } + + cfg.Clusters[cluster.ClusterID] = *cluster + return nil +} + +type oldWsConfig struct { + Client *arvados.Client + Postgres *arvados.PostgreSQLConnection + PostgresPool *int + Listen *string + LogLevel *string + LogFormat *string + + PingTimeout *arvados.Duration + ClientEventQueue *int + ServerEventQueue *int + + ManagementToken *string +} + +const defaultWebsocketConfigPath = "/etc/arvados/ws/ws.yml" + +// update config using values from an crunch-dispatch-slurm config file. +func (ldr *Loader) loadOldWebsocketConfig(cfg *arvados.Config) error { + if ldr.WebsocketPath == "" { + return nil + } + var oc oldWsConfig + err := ldr.loadOldConfigHelper("arvados-ws", ldr.WebsocketPath, &oc) + if os.IsNotExist(err) && ldr.WebsocketPath == defaultWebsocketConfigPath { + return nil + } else if err != nil { + return err + } + + cluster, err := cfg.GetCluster("") if err != nil { - return fmt.Errorf("%s: %s", path, err) + return err + } + + loadOldClientConfig(cluster, oc.Client) + + if oc.Postgres != nil { + cluster.PostgreSQL.Connection = *oc.Postgres + } + if oc.PostgresPool != nil { + cluster.PostgreSQL.ConnectionPool = *oc.PostgresPool + } + if oc.Listen != nil { + cluster.Services.Websocket.InternalURLs[arvados.URL{Host: *oc.Listen}] = arvados.ServiceInstance{} + } + if oc.LogLevel != nil { + cluster.SystemLogs.LogLevel = *oc.LogLevel + } + if oc.LogFormat != nil { + cluster.SystemLogs.Format = *oc.LogFormat + } + if oc.PingTimeout != nil { + cluster.API.SendTimeout = *oc.PingTimeout + } + if oc.ClientEventQueue != nil { + cluster.API.WebsocketClientEventQueue = *oc.ClientEventQueue + } + if oc.ServerEventQueue != nil { + cluster.API.WebsocketServerEventQueue = *oc.ServerEventQueue + } + if oc.ManagementToken != nil { + cluster.ManagementToken = *oc.ManagementToken + } + + cfg.Clusters[cluster.ClusterID] = *cluster + return nil +} + +type oldKeepProxyConfig struct { + Client *arvados.Client + Listen *string + DisableGet *bool + DisablePut *bool + DefaultReplicas *int + Timeout *arvados.Duration + PIDFile *string + Debug *bool + ManagementToken *string +} + +const defaultKeepproxyConfigPath = "/etc/arvados/keepproxy/keepproxy.yml" + +func (ldr *Loader) loadOldKeepproxyConfig(cfg *arvados.Config) error { + if ldr.KeepproxyPath == "" { + return nil + } + var oc oldKeepProxyConfig + err := ldr.loadOldConfigHelper("keepproxy", ldr.KeepproxyPath, &oc) + if os.IsNotExist(err) && ldr.KeepproxyPath == defaultKeepproxyConfigPath { + return nil + } else if err != nil { + return err + } + + cluster, err := cfg.GetCluster("") + if err != nil { + return err + } + + loadOldClientConfig(cluster, oc.Client) + + if oc.Listen != nil { + cluster.Services.Keepproxy.InternalURLs[arvados.URL{Host: *oc.Listen}] = arvados.ServiceInstance{} + } + if oc.DefaultReplicas != nil { + cluster.Collections.DefaultReplication = *oc.DefaultReplicas + } + if oc.Timeout != nil { + cluster.API.KeepServiceRequestTimeout = *oc.Timeout + } + if oc.Debug != nil { + if *oc.Debug && cluster.SystemLogs.LogLevel != "debug" { + cluster.SystemLogs.LogLevel = "debug" + } else if !*oc.Debug && cluster.SystemLogs.LogLevel != "info" { + cluster.SystemLogs.LogLevel = "info" + } + } + if oc.ManagementToken != nil { + cluster.ManagementToken = *oc.ManagementToken + } + + // The following legacy options are no longer supported. If they are set to + // true or PIDFile has a value, error out and notify the user + unsupportedEntry := func(cfgEntry string) error { + return fmt.Errorf("the keepproxy %s configuration option is no longer supported, please remove it from your configuration file", cfgEntry) + } + if oc.DisableGet != nil && *oc.DisableGet { + return unsupportedEntry("DisableGet") + } + if oc.DisablePut != nil && *oc.DisablePut { + return unsupportedEntry("DisablePut") + } + if oc.PIDFile != nil && *oc.PIDFile != "" { + return unsupportedEntry("PIDFile") + } + + cfg.Clusters[cluster.ClusterID] = *cluster + return nil +} + +const defaultKeepWebConfigPath = "/etc/arvados/keep-web/keep-web.yml" + +type oldKeepWebConfig struct { + Client *arvados.Client + + Listen string + + AnonymousTokens []string + AttachmentOnlyHost string + TrustAllContent bool + + Cache struct { + TTL arvados.Duration + UUIDTTL arvados.Duration + MaxCollectionEntries int + MaxCollectionBytes int64 + MaxPermissionEntries int + MaxUUIDEntries int + } + + // Hack to support old command line flag, which is a bool + // meaning "get actual token from environment". + deprecatedAllowAnonymous bool + + // Authorization token to be included in all health check requests. + ManagementToken string +} + +func (ldr *Loader) loadOldKeepWebConfig(cfg *arvados.Config) error { + if ldr.KeepWebPath == "" { + return nil + } + var oc oldKeepWebConfig + err := ldr.loadOldConfigHelper("keep-web", ldr.KeepWebPath, &oc) + if os.IsNotExist(err) && ldr.KeepWebPath == defaultKeepWebConfigPath { + return nil + } else if err != nil { + return err + } + + cluster, err := cfg.GetCluster("") + if err != nil { + return err + } + + loadOldClientConfig(cluster, oc.Client) + + cluster.Services.WebDAV.InternalURLs[arvados.URL{Host: oc.Listen}] = arvados.ServiceInstance{} + cluster.Services.WebDAVDownload.InternalURLs[arvados.URL{Host: oc.Listen}] = arvados.ServiceInstance{} + cluster.Services.WebDAVDownload.ExternalURL = arvados.URL{Host: oc.AttachmentOnlyHost} + cluster.TLS.Insecure = oc.Client.Insecure + cluster.ManagementToken = oc.ManagementToken + cluster.Collections.TrustAllContent = oc.TrustAllContent + cluster.Collections.WebDAVCache.TTL = oc.Cache.TTL + cluster.Collections.WebDAVCache.UUIDTTL = oc.Cache.UUIDTTL + cluster.Collections.WebDAVCache.MaxCollectionEntries = oc.Cache.MaxCollectionEntries + cluster.Collections.WebDAVCache.MaxCollectionBytes = oc.Cache.MaxCollectionBytes + cluster.Collections.WebDAVCache.MaxPermissionEntries = oc.Cache.MaxPermissionEntries + cluster.Collections.WebDAVCache.MaxUUIDEntries = oc.Cache.MaxUUIDEntries + if len(oc.AnonymousTokens) > 0 { + cluster.Users.AnonymousUserToken = oc.AnonymousTokens[0] + if len(oc.AnonymousTokens) > 1 { + ldr.Logger.Warn("More than 1 anonymous tokens configured, using only the first and discarding the rest.") + } + } + + cfg.Clusters[cluster.ClusterID] = *cluster + return nil +} + +const defaultGitHttpdConfigPath = "/etc/arvados/git-httpd/git-httpd.yml" + +type oldGitHttpdConfig struct { + Client *arvados.Client + Listen string + GitCommand string + GitoliteHome string + RepoRoot string + ManagementToken string +} + +func (ldr *Loader) loadOldGitHttpdConfig(cfg *arvados.Config) error { + if ldr.GitHttpdPath == "" { + return nil + } + var oc oldGitHttpdConfig + err := ldr.loadOldConfigHelper("arv-git-httpd", ldr.GitHttpdPath, &oc) + if os.IsNotExist(err) && ldr.GitHttpdPath == defaultGitHttpdConfigPath { + return nil + } else if err != nil { + return err + } + + cluster, err := cfg.GetCluster("") + if err != nil { + return err + } + + loadOldClientConfig(cluster, oc.Client) + + cluster.Services.GitHTTP.InternalURLs[arvados.URL{Host: oc.Listen}] = arvados.ServiceInstance{} + cluster.TLS.Insecure = oc.Client.Insecure + cluster.ManagementToken = oc.ManagementToken + cluster.Git.GitCommand = oc.GitCommand + cluster.Git.GitoliteHome = oc.GitoliteHome + cluster.Git.Repositories = oc.RepoRoot + + cfg.Clusters[cluster.ClusterID] = *cluster + return nil +} + +const defaultKeepBalanceConfigPath = "/etc/arvados/keep-balance/keep-balance.yml" + +type oldKeepBalanceConfig struct { + Client *arvados.Client + Listen *string + KeepServiceTypes *[]string + KeepServiceList *arvados.KeepServiceList + RunPeriod *arvados.Duration + CollectionBatchSize *int + CollectionBuffers *int + RequestTimeout *arvados.Duration + LostBlocksFile *string + ManagementToken *string +} + +func (ldr *Loader) loadOldKeepBalanceConfig(cfg *arvados.Config) error { + if ldr.KeepBalancePath == "" { + return nil + } + var oc oldKeepBalanceConfig + err := ldr.loadOldConfigHelper("keep-balance", ldr.KeepBalancePath, &oc) + if os.IsNotExist(err) && ldr.KeepBalancePath == defaultKeepBalanceConfigPath { + return nil + } else if err != nil { + return err + } + + cluster, err := cfg.GetCluster("") + if err != nil { + return err + } + + loadOldClientConfig(cluster, oc.Client) + + if oc.Listen != nil { + cluster.Services.Keepbalance.InternalURLs[arvados.URL{Host: *oc.Listen}] = arvados.ServiceInstance{} + } + if oc.ManagementToken != nil { + cluster.ManagementToken = *oc.ManagementToken + } + if oc.RunPeriod != nil { + cluster.Collections.BalancePeriod = *oc.RunPeriod + } + if oc.LostBlocksFile != nil { + cluster.Collections.BlobMissingReport = *oc.LostBlocksFile + } + if oc.CollectionBatchSize != nil { + cluster.Collections.BalanceCollectionBatch = *oc.CollectionBatchSize + } + if oc.CollectionBuffers != nil { + cluster.Collections.BalanceCollectionBuffers = *oc.CollectionBuffers + } + if oc.RequestTimeout != nil { + cluster.API.KeepServiceRequestTimeout = *oc.RequestTimeout + } + + msg := "The %s configuration option is no longer supported. Please remove it from your configuration file. Keep-balance will operate on all configured volumes." + + // If the keep service type provided is "disk" silently ignore it, since + // this is what ends up being done anyway. + if oc.KeepServiceTypes != nil { + numTypes := len(*oc.KeepServiceTypes) + if numTypes != 0 && !(numTypes == 1 && (*oc.KeepServiceTypes)[0] == "disk") { + return fmt.Errorf(msg, "KeepServiceType") + } } - if v := oc.Debug; v == nil { - } else if *v && cluster.SystemLogs.LogLevel != "debug" { - cluster.SystemLogs.LogLevel = "debug" - } else if !*v && cluster.SystemLogs.LogLevel != "info" { - cluster.SystemLogs.LogLevel = "info" + if oc.KeepServiceList != nil { + return fmt.Errorf(msg, "KeepServiceList") } cfg.Clusters[cluster.ClusterID] = *cluster