[ARVADOS] updated: 62a5bae0cf096fee69add487cd28a8bdbd4fa17c

Git user git at public.curoverse.com
Sat Oct 8 18:40:09 EDT 2016


Summary of changes:
 services/crunch-dispatch-slurm/crunch-dispatch-slurm.go | 4 +---
 services/crunch-dispatch-slurm/usage.go                 | 2 +-
 2 files changed, 2 insertions(+), 4 deletions(-)

       via  62a5bae0cf096fee69add487cd28a8bdbd4fa17c (commit)
      from  25236212b8f726335ede5d4aa5da120b81ef19a4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 62a5bae0cf096fee69add487cd28a8bdbd4fa17c
Author: Tom Clegg <tom at curoverse.com>
Date:   Sat Oct 8 18:25:54 2016 -0400

    10041: Set ARVADOS_KEEP_SERVICES from Client.KeepServicesURIs field.

diff --git a/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go b/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
index 85a71d4..aaea51c 100644
--- a/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
+++ b/services/crunch-dispatch-slurm/crunch-dispatch-slurm.go
@@ -32,8 +32,6 @@ type Config struct {
 	//
 	// Example: []string{"crunch-run", "--cgroup-parent-subsystem=memory"}
 	CrunchRunCommand []string
-
-	ArvadosKeepServices []string
 }
 
 func main() {
@@ -85,7 +83,7 @@ func doMain() error {
 		if theConfig.Client.Insecure {
 			os.Setenv("ARVADOS_API_INSECURE", "1")
 		}
-		os.Setenv("ARVADOS_KEEP_SERVICES", strings.Join(theConfig.ArvadosKeepServices, " "))
+		os.Setenv("ARVADOS_KEEP_SERVICES", strings.Join(theConfig.Client.KeepServiceURIs, " "))
 		os.Setenv("ARVADOS_EXTERNAL_CLIENT", "")
 	} else {
 		log.Printf("warning: Client credentials missing from config, so falling back on environment variables (deprecated).")
diff --git a/services/crunch-dispatch-slurm/usage.go b/services/crunch-dispatch-slurm/usage.go
index cd2d8ad..ddfbdb0 100644
--- a/services/crunch-dispatch-slurm/usage.go
+++ b/services/crunch-dispatch-slurm/usage.go
@@ -12,9 +12,9 @@ var exampleConfigFile = []byte(`
 	    "APIHost": "zzzzz.arvadosapi.com",
 	    "AuthToken": "xyzzy",
 	    "Insecure": false
+	    "KeepServiceURIs": [],
 	},
 	"CrunchRunCommand": ["crunch-run"],
-	"ArvadosKeepServices": [""],
 	"PollPeriod": "10s",
 	"SbatchArguments": ["--partition=foo", "--exclude=node13"]
     }`)

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list