github/weave-scope
1
0
Fork 0
mirror of https://github.com/weaveworks/scope.git synced 2026-03-02 01:30:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

use BasicNodeSummary for rendering links in connection table

Browse Source 
instead of the full NodeSummary
This commit is contained in:
Matthias Radestock
2017-12-24 12:07:09 +00:00
parent 0237b13916
commit 8cf65ea8b6
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
render/detailed/connections.go
View File

@@ -125,10 +125,8 @@ func internetAddr(node report.Node, ep report.Node) (string, bool) {
func (c *connectionCounters) rows(r report.Report, ns report.Nodes, includeLocal bool) []Connection {
output := []Connection{}
for row, count := range c.counts {
// Use MakeNodeSummary to render the id and label of this node
// TODO(paulbellamy): Would be cleaner if we hade just a
// MakeNodeID(ns[row.remoteNodeID]). As we don't need the whole summary.
summary, _ := MakeNodeSummary(report.RenderContext{Report: r}, ns[row.remoteNodeID])
// Use MakeBasicNodeSummary to render the id and label of this node
summary, _ := MakeBasicNodeSummary(r, ns[row.remoteNodeID])
connection := Connection{
ID: fmt.Sprintf("%s-%s-%s-%s", row.remoteNodeID, row.remoteAddr, row.localAddr, row.port),
NodeID: summary.ID,