[ARVADOS] updated: 2.1.0-792-g44aa68895

[Git user]

Summary of changes:
 doc/user/cwl/costanalyzer.html.textile.liquid | 7 +++++++
 1 file changed, 7 insertions(+)

  discards  6b5a2f0f2d75907be970083e6c29eddf2b773b4b (commit)
       via  44aa688953d658a23c32581a2bad01342007a191 (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (6b5a2f0f2d75907be970083e6c29eddf2b773b4b)
            \
             N -- N -- N (44aa688953d658a23c32581a2bad01342007a191)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

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 44aa688953d658a23c32581a2bad01342007a191
Author: Ward Vandewege <ward at curii.com>
Date:   Mon May 17 15:20:28 2021 -0400

    17678: implement review feedback.
    
    Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward at curii.com>

diff --git a/doc/user/cwl/costanalyzer.html.textile.liquid b/doc/user/cwl/costanalyzer.html.textile.liquid
index 70a1be5da..fc39ada52 100644
--- a/doc/user/cwl/costanalyzer.html.textile.liquid
+++ b/doc/user/cwl/costanalyzer.html.textile.liquid
@@ -24,7 +24,7 @@ The @arvados-client costanalyzer@ tool has a number of command line arguments:
 <notextile>
 <pre><code>~$ <span class="userinput">arvados-client costanalyzer -h</span>
 Usage:
-  arvados-client costanalyzer [options ...] <uuid> ...
+  arvados-client costanalyzer [options ...] uuid [uuid ...]
 
   This program analyzes the cost of Arvados container requests. For each uuid
   supplied, it creates a CSV report that lists all the containers used to
@@ -59,6 +59,13 @@ Usage:
   a column that indicates the preemptible state of the instance that ran the
   container.
 
+  - This program does not take into account overhead costs like the time spent
+  starting and stopping compute nodes that run containers, the cost of the
+  permanent cloud nodes that provide the Arvados services, the cost of data
+  stored in Arvados, etc.
+
+  - When provided with a project uuid, subprojects will not be considered.
+
   In order to get the data for the uuids supplied, the ARVADOS_API_HOST and
   ARVADOS_API_TOKEN environment variables must be set.
 
diff --git a/lib/costanalyzer/costanalyzer.go b/lib/costanalyzer/costanalyzer.go
index df8fd4960..402e0ec81 100644
--- a/lib/costanalyzer/costanalyzer.go
+++ b/lib/costanalyzer/costanalyzer.go
@@ -57,7 +57,7 @@ func parseFlags(prog string, args []string, loader *config.Loader, logger *logru
 	flags.Usage = func() {
 		fmt.Fprintf(flags.Output(), `
 Usage:
-  %s [options ...] <uuid> ...
+  %s [options ...] uuid [uuid ...]
 
 	This program analyzes the cost of Arvados container requests. For each uuid
 	supplied, it creates a CSV report that lists all the containers used to
@@ -92,6 +92,13 @@ Usage:
 	a column that indicates the preemptible state of the instance that ran the
 	container.
 
+	- This program does not take into account overhead costs like the time spent
+	starting and stopping compute nodes that run containers, the cost of the
+	permanent cloud nodes that provide the Arvados services, the cost of data
+	stored in Arvados, etc.
+
+	- When provided with a project uuid, subprojects will not be considered.
+
 	In order to get the data for the uuids supplied, the ARVADOS_API_HOST and
 	ARVADOS_API_TOKEN environment variables must be set.
 

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


hooks/post-receive
--