From a46b9c9c24e4e4c72f71dce0a475ba60d5365bbb Mon Sep 17 00:00:00 2001 From: Matthias Radestock Date: Thu, 28 Dec 2017 01:53:56 +0000 Subject: [PATCH] don't propagate ContainerHostname in MapContainer2Hostname The container hostname is already the id of the node, and that's what summarisation renders. Nothing looks at the docker.ContainerHostname metadata of nodes in the ContainerHostname group topology. --- render/container.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/render/container.go b/render/container.go index b97d08852..adb71f928 100644 --- a/render/container.go +++ b/render/container.go @@ -352,13 +352,12 @@ func MapContainer2Hostname(n report.Node) report.Nodes { // Otherwise, if some some reason the container doesn't have a hostname // (maybe slightly out of sync reports), just drop it - id, timestamp, ok := n.Latest.LookupEntry(docker.ContainerHostname) + id, ok := n.Latest.Lookup(docker.ContainerHostname) if !ok { return report.Nodes{} } node := NewDerivedNode(id, n).WithTopology(MakeGroupNodeTopology(n.Topology, docker.ContainerHostname)) - node.Latest = node.Latest.Set(docker.ContainerHostname, timestamp, id) node.Counters = node.Counters.Add(n.Topology, 1) return report.Nodes{id: node} }