Skip to content

Commit

Permalink
Fix filtering issue for uncontained nodes in DNS name view (#2170).
Browse files Browse the repository at this point in the history
  • Loading branch information
fbarl committed Feb 20, 2017
1 parent 76fec79 commit f1904a6
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions render/container.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ const (
// NB We only want processes in container _or_ processes with network connections
// but we need to be careful to ensure we only include each edge once, by only
// including the ProcessRenderer once.
var ContainerRenderer = MakeFilter(
var ContainerRenderer = ApplyDecorators(MakeFilter(
func(n report.Node) bool {
// Drop deleted containers
state, ok := n.Latest.Lookup(docker.ContainerState)
Expand All @@ -43,7 +43,7 @@ var ContainerRenderer = MakeFilter(

SelectContainer,
),
)
))

const originalNodeID = "original_node_id"
const originalNodeTopology = "original_node_topology"
Expand Down Expand Up @@ -204,7 +204,7 @@ func (r containerWithImageNameRenderer) Render(rpt report.Report, dct Decorator)

// ContainerWithImageNameRenderer is a Renderer which produces a container
// graph where the ranks are the image names, not their IDs
var ContainerWithImageNameRenderer = ApplyDecorators(containerWithImageNameRenderer{ContainerRenderer})
var ContainerWithImageNameRenderer = containerWithImageNameRenderer{ContainerRenderer}

// ContainerImageRenderer is a Renderer which produces a renderable container
// image graph by merging the container graph and the container image topology.
Expand Down

0 comments on commit f1904a6

Please sign in to comment.