Skip to content

Commit

Permalink
Merge pull request #2060 from weaveworks/mike/awsecs/fix-log-formatting
Browse files Browse the repository at this point in the history
ecs reporter: Fix some log lines that were passing *string instead of string
  • Loading branch information
ekimekim authored Dec 14, 2016
2 parents 3560bbc + fb12df6 commit 5ed63de
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions probe/awsecs/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ func (c ecsClient) getServices() map[string]*ecs.Service {
}

for _, failure := range resp.Failures {
log.Warnf("Failed to describe ECS service %s, ECS service report may be incomplete: %s", failure.Arn, failure.Reason)
log.Warnf("Failed to describe ECS service %s, ECS service report may be incomplete: %s", *failure.Arn, *failure.Reason)
}

lock.Lock()
Expand Down Expand Up @@ -110,7 +110,7 @@ func (c ecsClient) getTasks(taskArns []string) (map[string]*ecs.Task, error) {
}

for _, failure := range resp.Failures {
log.Warnf("Failed to describe ECS task %s, ECS service report may be incomplete: %s", failure.Arn, failure.Reason)
log.Warnf("Failed to describe ECS task %s, ECS service report may be incomplete: %s", *failure.Arn, *failure.Reason)
}

results := make(map[string]*ecs.Task, len(resp.Tasks))
Expand Down

0 comments on commit 5ed63de

Please sign in to comment.