Skip to content

Commit

Permalink
remove client.address as default attribute on RPC metrics (grafana#1022)
Browse files Browse the repository at this point in the history
  • Loading branch information
mariomac authored Jul 16, 2024
1 parent b0fba9c commit 33af474
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions pkg/internal/export/attributes/attr_defs.go
Original file line number Diff line number Diff line change
Expand Up @@ -236,9 +236,6 @@ func getDefinitions(groups AttrGroups) map[Section]AttrReportGroup {
attr.RPCMethod: true,
attr.RPCSystem: true,
attr.RPCGRPCStatusCode: true,
// Overriding default serverInfo configuration because we want
// to report it by default
attr.ClientAddr: true,
},
},
DBClientDuration.Section: {
Expand Down
2 changes: 1 addition & 1 deletion pkg/internal/pipe/instrumenter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ func gctx(groups attributes.AttrGroups) *global.ContextInfo {
}

var allMetrics = attributes.Selection{
attributes.HTTPServerDuration.Section: attributes.InclusionLists{Include: []string{"*"}},
"*": attributes.InclusionLists{Include: []string{"*"}},
}

func allMetricsBut(patterns ...string) attributes.Selection {
Expand Down

0 comments on commit 33af474

Please sign in to comment.