[ARVADOS] updated: 2.1.0-231-g2c8874c06

Git user git at public.arvados.org
Mon Dec 21 02:55:46 UTC 2020


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

       via  2c8874c0625a2126813d4df7170076f95cdc7327 (commit)
      from  92086abf6fd3425b5c98e32a5d009734e7210fc9 (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 2c8874c0625a2126813d4df7170076f95cdc7327
Author: Ward Vandewege <ward at curii.com>
Date:   Sun Dec 20 21:55:19 2020 -0500

    Fix a few staticcheck warnings.
    
    No issue #
    
    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 46ff655dd..e8dd18605 100644
--- a/lib/costanalyzer/costanalyzer.go
+++ b/lib/costanalyzer/costanalyzer.go
@@ -120,7 +120,7 @@ Options:
 
 	if len(uuids) < 1 {
 		flags.Usage()
-		err = fmt.Errorf("Error: no uuid(s) provided")
+		err = fmt.Errorf("error: no uuid(s) provided")
 		exitCode = 2
 		return
 	}
@@ -285,7 +285,7 @@ func loadObject(logger *logrus.Logger, ac *arvados.Client, path string, uuid str
 
 func getNode(arv *arvadosclient.ArvadosClient, ac *arvados.Client, kc *keepclient.KeepClient, cr arvados.ContainerRequest) (node nodeInfo, err error) {
 	if cr.LogUUID == "" {
-		err = errors.New("No log collection")
+		err = errors.New("no log collection")
 		return
 	}
 
@@ -509,7 +509,7 @@ func costanalyzer(prog string, args []string, loader *config.Loader, logger *log
 			var crCsv map[string]float64
 			crCsv, err = generateCrCsv(logger, uuid, arv, ac, kc, resultsDir, cache)
 			if err != nil {
-				err = fmt.Errorf("Error generating CSV for uuid %s: %s", uuid, err.Error())
+				err = fmt.Errorf("error generating CSV for uuid %s: %s", uuid, err.Error())
 				exitcode = 2
 				return
 			}
@@ -521,9 +521,9 @@ func costanalyzer(prog string, args []string, loader *config.Loader, logger *log
 			// It is identified by the user uuid. As such, cost analysis for the
 			// "Home" project is not supported by this program. Skip this uuid, but
 			// keep going.
-			logger.Errorf("Cost analysis is not supported for the 'Home' project: %s", uuid)
+			logger.Errorf("cost analysis is not supported for the 'Home' project: %s", uuid)
 		} else {
-			logger.Errorf("This argument does not look like a uuid: %s\n", uuid)
+			logger.Errorf("this argument does not look like a uuid: %s\n", uuid)
 			exitcode = 3
 			return
 		}
@@ -554,7 +554,7 @@ func costanalyzer(prog string, args []string, loader *config.Loader, logger *log
 		aFile := resultsDir + "/" + time.Now().Format("2006-01-02-15-04-05") + "-aggregate-costaccounting.csv"
 		err = ioutil.WriteFile(aFile, []byte(csv), 0644)
 		if err != nil {
-			err = fmt.Errorf("Error writing file with path %s: %s", aFile, err.Error())
+			err = fmt.Errorf("error writing file with path %s: %s", aFile, err.Error())
 			exitcode = 1
 			return
 		}

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list