From 875fdba35254cc67f2f400eee7127fde43095df1 Mon Sep 17 00:00:00 2001 From: Bryan Boreham Date: Fri, 1 Nov 2019 19:14:35 +0000 Subject: [PATCH] rendering: simplify ContainerHostnameRenderer The second half of the render added all the same nodes, except for those with no Docker image information, so we could show a figure for those filtered out on that basis. This just isn't worth the effort. --- render/container.go | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/render/container.go b/render/container.go index e6d533aa8..791bc299d 100644 --- a/render/container.go +++ b/render/container.go @@ -171,20 +171,9 @@ var ContainerImageRenderer = Memoise(FilterEmpty(report.Container, // // not memoised var ContainerHostnameRenderer = FilterEmpty(report.Container, - MakeReduce( - MakeMap( - MapContainer2Hostname, - ContainerWithImageNameRenderer, - ), - // Grab *all* the hostnames, so we can count the number which were empty - // for accurate stats. - MakeMap( - MapToEmpty, - MakeMap( - MapContainer2Hostname, - ContainerRenderer, - ), - ), + MakeMap( + MapContainer2Hostname, + ContainerWithImageNameRenderer, ), )