From f5b7b5bec2337b1e751d8135a4afdb16d3e59317 Mon Sep 17 00:00:00 2001 From: Mike Lang Date: Wed, 23 Nov 2016 20:46:23 -0800 Subject: [PATCH] ecs reporter: Fix a bug where parents weren't actually set --- probe/awsecs/reporter.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/probe/awsecs/reporter.go b/probe/awsecs/reporter.go index 3932151ff..bfbbc0c15 100644 --- a/probe/awsecs/reporter.go +++ b/probe/awsecs/reporter.go @@ -93,10 +93,10 @@ func (r Reporter) Tag(rpt report.Report) (report.Report, error) { // parents sets to merge into all matching container nodes parentsSets := report.MakeSets() - parentsSets.Add(report.ECSTask, report.MakeStringSet(taskID)) + parentsSets = parentsSets.Add(report.ECSTask, report.MakeStringSet(taskID)) if serviceName, ok := taskServices[taskArn]; ok { serviceID := report.MakeECSServiceNodeID(serviceName) - parentsSets.Add(report.ECSService, report.MakeStringSet(serviceID)) + parentsSets = parentsSets.Add(report.ECSService, report.MakeStringSet(serviceID)) } for _, containerID := range info.containerIDs {