[ARVADOS] updated: 1.3.0-609-gbac750df0

Git user git at public.curoverse.com
Thu Mar 28 17:44:25 UTC 2019


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

       via  bac750df0495c10d7181edfda48ba6e28e807e1e (commit)
       via  3091e5692d3c558f861355aad4448a8fd7b9bdee (commit)
      from  b3be9ab4ce2c15a4340a5304afa1b3ddccb0e05d (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 bac750df0495c10d7181edfda48ba6e28e807e1e
Merge: b3be9ab4c 3091e5692
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Mar 28 13:44:05 2019 -0400

    Merge branch '14998-hoststat-total'
    
    fixes #14998
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>


commit 3091e5692d3c558f861355aad4448a8fd7b9bdee
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Tue Mar 19 15:49:49 2019 -0400

    14998: Report total_* (hierarchy) stats if available.
    
    Otherwise, host stats only include the root cgroup.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/lib/crunchstat/crunchstat.go b/lib/crunchstat/crunchstat.go
index 8afe82819..028083fa0 100644
--- a/lib/crunchstat/crunchstat.go
+++ b/lib/crunchstat/crunchstat.go
@@ -256,8 +256,13 @@ func (r *Reporter) doMemoryStats() {
 	}
 	var outstat bytes.Buffer
 	for _, key := range wantStats {
-		if val, ok := thisSample.memStat[key]; ok {
-			outstat.WriteString(fmt.Sprintf(" %d %s", val, key))
+		// Use "total_X" stats (entire hierarchy) if enabled,
+		// otherwise just the single cgroup -- see
+		// https://www.kernel.org/doc/Documentation/cgroup-v1/memory.txt
+		if val, ok := thisSample.memStat["total_"+key]; ok {
+			fmt.Fprintf(&outstat, " %d %s", val, key)
+		} else if val, ok := thisSample.memStat[key]; ok {
+			fmt.Fprintf(&outstat, " %d %s", val, key)
 		}
 	}
 	r.Logger.Printf("mem%s\n", outstat.String())

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list