Merge branch '17175-boot-fail-rake-seed'
authorTom Clegg <tom@curii.com>
Wed, 6 Jan 2021 14:45:14 +0000 (09:45 -0500)
committerTom Clegg <tom@curii.com>
Wed, 6 Jan 2021 14:45:14 +0000 (09:45 -0500)
fixes #17175

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

1  2 
lib/boot/supervisor.go

diff --combined lib/boot/supervisor.go
index 0e4472b8a03c4972898ec24ad742ca11ed16eca6,0e96beabd5ed6a29aeea2e74dec25e9defd5629c..f2e715a76669ed7ea1db21a7720b3fbd382cb8a3
@@@ -216,16 -216,16 +216,16 @@@ func (super *Supervisor) run(cfg *arvad
                createCertificates{},
                runPostgreSQL{},
                runNginx{},
-               runServiceCommand{name: "controller", svc: super.cluster.Services.Controller, depends: []supervisedTask{runPostgreSQL{}}},
+               runServiceCommand{name: "controller", svc: super.cluster.Services.Controller, depends: []supervisedTask{seedDatabase{}}},
                runGoProgram{src: "services/arv-git-httpd", svc: super.cluster.Services.GitHTTP},
                runGoProgram{src: "services/health", svc: super.cluster.Services.Health},
                runGoProgram{src: "services/keepproxy", svc: super.cluster.Services.Keepproxy, depends: []supervisedTask{runPassenger{src: "services/api"}}},
                runGoProgram{src: "services/keepstore", svc: super.cluster.Services.Keepstore},
                runGoProgram{src: "services/keep-web", svc: super.cluster.Services.WebDAV},
-               runServiceCommand{name: "ws", svc: super.cluster.Services.Websocket, depends: []supervisedTask{runPostgreSQL{}}},
+               runServiceCommand{name: "ws", svc: super.cluster.Services.Websocket, depends: []supervisedTask{seedDatabase{}}},
                installPassenger{src: "services/api"},
-               runPassenger{src: "services/api", svc: super.cluster.Services.RailsAPI, depends: []supervisedTask{createCertificates{}, runPostgreSQL{}, installPassenger{src: "services/api"}}},
-               installPassenger{src: "apps/workbench", depends: []supervisedTask{installPassenger{src: "services/api"}}}, // dependency ensures workbench doesn't delay api startup
+               runPassenger{src: "services/api", svc: super.cluster.Services.RailsAPI, depends: []supervisedTask{createCertificates{}, seedDatabase{}, installPassenger{src: "services/api"}}},
+               installPassenger{src: "apps/workbench", depends: []supervisedTask{seedDatabase{}}}, // dependency ensures workbench doesn't delay api install/startup
                runPassenger{src: "apps/workbench", svc: super.cluster.Services.Workbench1, depends: []supervisedTask{installPassenger{src: "apps/workbench"}}},
                seedDatabase{},
        }
@@@ -451,8 -451,8 +451,8 @@@ func (super *Supervisor) RunProgram(ct
        super.logger.WithField("command", cmdline).WithField("dir", dir).Info("executing")
  
        logprefix := prog
-       if logprefix == "setuidgid" && len(args) >= 3 {
-               logprefix = args[2]
+       if logprefix == "setuidgid" && len(args) >= 2 {
+               logprefix = args[1]
        }
        logprefix = strings.TrimPrefix(logprefix, super.tempdir+"/bin/")
        if logprefix == "bundle" && len(args) > 2 && args[0] == "exec" {
@@@ -611,6 -611,9 +611,6 @@@ func (super *Supervisor) autofillConfig
        if cluster.ManagementToken == "" {
                cluster.ManagementToken = randomHexString(64)
        }
 -      if cluster.API.RailsSessionSecretToken == "" {
 -              cluster.API.RailsSessionSecretToken = randomHexString(64)
 -      }
        if cluster.Collections.BlobSigningKey == "" {
                cluster.Collections.BlobSigningKey = randomHexString(64)
        }