[ARVADOS] updated: 1.3.0-3263-gdda20df04

Git user git at public.arvados.org
Wed Nov 18 15:10:15 UTC 2020


Summary of changes:
 lib/costanalyzer/costanalyzer.go | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

       via  dda20df04e5329c6f8f2f2f867554dc3d7b7d6cd (commit)
      from  e1937e57fe2c0e99b6b636049142cc7598f80231 (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 dda20df04e5329c6f8f2f2f867554dc3d7b7d6cd
Author: Ward Vandewege <ward at curii.com>
Date:   Wed Nov 18 10:09:51 2020 -0500

    16950: make specifying of the output directory mandatory (remove
           default).
    
    Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward at curii.com>

diff --git a/lib/costanalyzer/costanalyzer.go b/lib/costanalyzer/costanalyzer.go
index 8f5395f58..9c8291220 100644
--- a/lib/costanalyzer/costanalyzer.go
+++ b/lib/costanalyzer/costanalyzer.go
@@ -92,8 +92,8 @@ Options:
 		flags.PrintDefaults()
 	}
 	loglevel := flags.String("log-level", "info", "logging `level` (debug, info, ...)")
-	resultsDir = *flags.String("output", "results", "output `directory` for the CSV reports")
-	flags.Var(&uuids, "uuid", "Toplevel `project or container request` uuid. May be specified more than once.")
+	resultsDir = *flags.String("output", "", "output `directory` for the CSV reports (required)")
+	flags.Var(&uuids, "uuid", "Toplevel `project or container request` uuid. May be specified more than once. (required)")
 	flags.BoolVar(&cache, "cache", true, "create and use a local disk cache of Arvados objects")
 	err = flags.Parse(args)
 	if err == flag.ErrHelp {
@@ -112,6 +112,13 @@ Options:
 		return
 	}
 
+	if resultsDir == "" {
+		flags.Usage()
+		err = fmt.Errorf("Error: output directory must be specified")
+		exitCode = 2
+		return
+	}
+
 	lvl, err := logrus.ParseLevel(*loglevel)
 	if err != nil {
 		exitCode = 2

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list