diff --git a/agent/acl.go b/agent/acl.go
index 49f7b9555c5a..2aee3faadc1b 100644
--- a/agent/acl.go
+++ b/agent/acl.go
@@ -144,9 +144,11 @@ func (m *aclManager) lookupACL(a *Agent, id string) (acl.ACL, error) {
}
if cached != nil && time.Now().Before(cached.Expires) {
metrics.IncrCounter([]string{"consul", "acl", "cache_hit"}, 1)
+ metrics.IncrCounter([]string{"acl", "cache_hit"}, 1)
return cached.ACL, nil
}
metrics.IncrCounter([]string{"consul", "acl", "cache_miss"}, 1)
+ metrics.IncrCounter([]string{"acl", "cache_miss"}, 1)
// At this point we might have a stale cached ACL, or none at all, so
// try to contact the servers.
diff --git a/agent/config/builder.go b/agent/config/builder.go
index d8c33cef33fd..c85f9b2b8ad7 100644
--- a/agent/config/builder.go
+++ b/agent/config/builder.go
@@ -437,6 +437,14 @@ func (b *Builder) Build() (rt RuntimeConfig, err error) {
}
}
+ // Add a filter rule if needed for enabling the deprecated metric names
+ enableDeprecatedNames := b.boolVal(c.Telemetry.EnableDeprecatedNames)
+ if enableDeprecatedNames {
+ telemetryAllowedPrefixes = append(telemetryAllowedPrefixes, "consul.consul")
+ } else {
+ telemetryBlockedPrefixes = append(telemetryBlockedPrefixes, "consul.consul")
+ }
+
// raft performance scaling
performanceRaftMultiplier := b.intVal(c.Performance.RaftMultiplier)
if performanceRaftMultiplier < 1 || uint(performanceRaftMultiplier) > consul.MaxRaftMultiplier {
diff --git a/agent/config/config.go b/agent/config/config.go
index 20a291db9721..910984da9337 100644
--- a/agent/config/config.go
+++ b/agent/config/config.go
@@ -368,6 +368,7 @@ type Telemetry struct {
MetricsPrefix *string `json:"metrics_prefix,omitempty" hcl:"metrics_prefix" mapstructure:"metrics_prefix"`
StatsdAddr *string `json:"statsd_address,omitempty" hcl:"statsd_address" mapstructure:"statsd_address"`
StatsiteAddr *string `json:"statsite_address,omitempty" hcl:"statsite_address" mapstructure:"statsite_address"`
+ EnableDeprecatedNames *bool `json:"enable_deprecated_names" hcl:"enable_deprecated_names" mapstructure:"enable_deprecated_names"`
}
type Ports struct {
diff --git a/agent/config/runtime.go b/agent/config/runtime.go
index 28cb13ac85ce..ebbf88fe703c 100644
--- a/agent/config/runtime.go
+++ b/agent/config/runtime.go
@@ -48,17 +48,17 @@ type RuntimeConfig struct {
ConsulSerfWANSuspicionMult int
ConsulServerHealthInterval time.Duration
- ACLAgentMasterToken string
- ACLAgentToken string
- ACLDatacenter string
- ACLDefaultPolicy string
- ACLDownPolicy string
- ACLEnforceVersion8 bool
+ ACLAgentMasterToken string
+ ACLAgentToken string
+ ACLDatacenter string
+ ACLDefaultPolicy string
+ ACLDownPolicy string
+ ACLEnforceVersion8 bool
ACLEnableKeyListPolicy bool
- ACLMasterToken string
- ACLReplicationToken string
- ACLTTL time.Duration
- ACLToken string
+ ACLMasterToken string
+ ACLReplicationToken string
+ ACLTTL time.Duration
+ ACLToken string
AutopilotCleanupDeadServers bool
AutopilotDisableUpgradeMigration bool
diff --git a/agent/config/runtime_test.go b/agent/config/runtime_test.go
index 9caf9a216af8..f8b2e2d4068a 100644
--- a/agent/config/runtime_test.go
+++ b/agent/config/runtime_test.go
@@ -1659,10 +1659,28 @@ func TestConfigFlagsAndEdgecases(t *testing.T) {
patch: func(rt *RuntimeConfig) {
rt.DataDir = dataDir
rt.TelemetryAllowedPrefixes = []string{"foo"}
- rt.TelemetryBlockedPrefixes = []string{"bar"}
+ rt.TelemetryBlockedPrefixes = []string{"bar", "consul.consul"}
},
warns: []string{`Filter rule must begin with either '+' or '-': "nix"`},
},
+ {
+ desc: "telemetry.enable_deprecated_names adds allow rule for whitelist",
+ flags: []string{
+ `-data-dir=` + dataDir,
+ },
+ json: []string{`{
+ "telemetry": { "enable_deprecated_names": true, "filter_default": false }
+ }`},
+ hcl: []string{`
+ telemetry = { enable_deprecated_names = true filter_default = false }
+ `},
+ patch: func(rt *RuntimeConfig) {
+ rt.DataDir = dataDir
+ rt.TelemetryFilterDefault = false
+ rt.TelemetryAllowedPrefixes = []string{"consul.consul"}
+ rt.TelemetryBlockedPrefixes = []string{}
+ },
+ },
{
desc: "encrypt has invalid key",
flags: []string{
@@ -2312,6 +2330,7 @@ func TestFullConfig(t *testing.T) {
"dogstatsd_tags": [ "3N81zSUB","Xtj8AnXZ" ],
"filter_default": true,
"prefix_filter": [ "+oJotS8XJ","-cazlEhGn" ],
+ "enable_deprecated_names": true,
"metrics_prefix": "ftO6DySn",
"statsd_address": "drce87cy",
"statsite_address": "HpFwKB8R"
@@ -2728,6 +2747,7 @@ func TestFullConfig(t *testing.T) {
dogstatsd_tags = [ "3N81zSUB","Xtj8AnXZ" ]
filter_default = true
prefix_filter = [ "+oJotS8XJ","-cazlEhGn" ]
+ enable_deprecated_names = true
metrics_prefix = "ftO6DySn"
statsd_address = "drce87cy"
statsite_address = "HpFwKB8R"
@@ -3266,7 +3286,7 @@ func TestFullConfig(t *testing.T) {
TelemetryDogstatsdAddr: "0wSndumK",
TelemetryDogstatsdTags: []string{"3N81zSUB", "Xtj8AnXZ"},
TelemetryFilterDefault: true,
- TelemetryAllowedPrefixes: []string{"oJotS8XJ"},
+ TelemetryAllowedPrefixes: []string{"oJotS8XJ", "consul.consul"},
TelemetryBlockedPrefixes: []string{"cazlEhGn"},
TelemetryMetricsPrefix: "ftO6DySn",
TelemetryStatsdAddr: "drce87cy",
diff --git a/agent/consul/acl.go b/agent/consul/acl.go
index 92462d25c679..bea2a5617db3 100644
--- a/agent/consul/acl.go
+++ b/agent/consul/acl.go
@@ -42,6 +42,7 @@ type aclCacheEntry struct {
// using its replicated ACLs during an outage.
func (s *Server) aclLocalFault(id string) (string, string, error) {
defer metrics.MeasureSince([]string{"consul", "acl", "fault"}, time.Now())
+ defer metrics.MeasureSince([]string{"acl", "fault"}, time.Now())
// Query the state store.
state := s.fsm.State()
@@ -75,6 +76,7 @@ func (s *Server) resolveToken(id string) (acl.ACL, error) {
return nil, nil
}
defer metrics.MeasureSince([]string{"consul", "acl", "resolveToken"}, time.Now())
+ defer metrics.MeasureSince([]string{"acl", "resolveToken"}, time.Now())
// Handle the anonymous token
if len(id) == 0 {
@@ -158,9 +160,11 @@ func (c *aclCache) lookupACL(id, authDC string) (acl.ACL, error) {
// Check for live cache.
if cached != nil && time.Now().Before(cached.Expires) {
metrics.IncrCounter([]string{"consul", "acl", "cache_hit"}, 1)
+ metrics.IncrCounter([]string{"acl", "cache_hit"}, 1)
return cached.ACL, nil
}
metrics.IncrCounter([]string{"consul", "acl", "cache_miss"}, 1)
+ metrics.IncrCounter([]string{"acl", "cache_miss"}, 1)
// Attempt to refresh the policy from the ACL datacenter via an RPC.
args := structs.ACLPolicyRequest{
@@ -223,6 +227,7 @@ func (c *aclCache) lookupACL(id, authDC string) (acl.ACL, error) {
// Note we use the local TTL here, so this'll be used for that
// amount of time even once the ACL datacenter becomes available.
metrics.IncrCounter([]string{"consul", "acl", "replication_hit"}, 1)
+ metrics.IncrCounter([]string{"acl", "replication_hit"}, 1)
reply.ETag = makeACLETag(parent, policy)
reply.TTL = c.config.ACLTTL
reply.Parent = parent
diff --git a/agent/consul/acl_endpoint.go b/agent/consul/acl_endpoint.go
index 437202917b82..2a8b4b2340f7 100644
--- a/agent/consul/acl_endpoint.go
+++ b/agent/consul/acl_endpoint.go
@@ -146,6 +146,7 @@ func (a *ACL) Apply(args *structs.ACLRequest, reply *string) error {
return err
}
defer metrics.MeasureSince([]string{"consul", "acl", "apply"}, time.Now())
+ defer metrics.MeasureSince([]string{"acl", "apply"}, time.Now())
// Verify we are allowed to serve this request
if a.srv.config.ACLDatacenter != a.srv.config.Datacenter {
diff --git a/agent/consul/acl_replication.go b/agent/consul/acl_replication.go
index 41cbe5dca5f5..729b34da6342 100644
--- a/agent/consul/acl_replication.go
+++ b/agent/consul/acl_replication.go
@@ -150,6 +150,7 @@ func (s *Server) fetchLocalACLs() (structs.ACLs, error) {
// have replicated to, so this is expected to block until something changes.
func (s *Server) fetchRemoteACLs(lastRemoteIndex uint64) (*structs.IndexedACLs, error) {
defer metrics.MeasureSince([]string{"consul", "leader", "fetchRemoteACLs"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "fetchRemoteACLs"}, time.Now())
args := structs.DCSpecificRequest{
Datacenter: s.config.ACLDatacenter,
@@ -170,6 +171,7 @@ func (s *Server) fetchRemoteACLs(lastRemoteIndex uint64) (*structs.IndexedACLs,
// local ACLs in-line with the remote ACLs from the ACL datacenter.
func (s *Server) updateLocalACLs(changes structs.ACLRequests) error {
defer metrics.MeasureSince([]string{"consul", "leader", "updateLocalACLs"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "updateLocalACLs"}, time.Now())
minTimePerOp := time.Second / time.Duration(s.config.ACLReplicationApplyLimit)
for _, change := range changes {
@@ -217,6 +219,7 @@ func (s *Server) replicateACLs(lastRemoteIndex uint64) (uint64, error) {
// periods of time. This metric is a good measure of how expensive the
// replication process is.
defer metrics.MeasureSince([]string{"consul", "leader", "replicateACLs"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "replicateACLs"}, time.Now())
local, err := s.fetchLocalACLs()
if err != nil {
diff --git a/agent/consul/autopilot.go b/agent/consul/autopilot.go
index ba6d68a69cc0..c6ac21ec6cf8 100644
--- a/agent/consul/autopilot.go
+++ b/agent/consul/autopilot.go
@@ -360,10 +360,13 @@ func (s *Server) updateClusterHealth() error {
// Heartbeat a metric for monitoring if we're the leader
if s.IsLeader() {
metrics.SetGauge([]string{"consul", "autopilot", "failure_tolerance"}, float32(clusterHealth.FailureTolerance))
+ metrics.SetGauge([]string{"autopilot", "failure_tolerance"}, float32(clusterHealth.FailureTolerance))
if clusterHealth.Healthy {
metrics.SetGauge([]string{"consul", "autopilot", "healthy"}, 1)
+ metrics.SetGauge([]string{"autopilot", "healthy"}, 1)
} else {
metrics.SetGauge([]string{"consul", "autopilot", "healthy"}, 0)
+ metrics.SetGauge([]string{"autopilot", "healthy"}, 0)
}
}
diff --git a/agent/consul/catalog_endpoint.go b/agent/consul/catalog_endpoint.go
index 9c8e6dbd2e1b..c6997fa70228 100644
--- a/agent/consul/catalog_endpoint.go
+++ b/agent/consul/catalog_endpoint.go
@@ -25,6 +25,7 @@ func (c *Catalog) Register(args *structs.RegisterRequest, reply *struct{}) error
return err
}
defer metrics.MeasureSince([]string{"consul", "catalog", "register"}, time.Now())
+ defer metrics.MeasureSince([]string{"catalog", "register"}, time.Now())
// Verify the args.
if args.Node == "" || args.Address == "" {
@@ -114,6 +115,7 @@ func (c *Catalog) Deregister(args *structs.DeregisterRequest, reply *struct{}) e
return err
}
defer metrics.MeasureSince([]string{"consul", "catalog", "deregister"}, time.Now())
+ defer metrics.MeasureSince([]string{"catalog", "deregister"}, time.Now())
// Verify the args
if args.Node == "" {
@@ -272,13 +274,19 @@ func (c *Catalog) ServiceNodes(args *structs.ServiceSpecificRequest, reply *stru
if err == nil {
metrics.IncrCounterWithLabels([]string{"consul", "catalog", "service", "query"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}})
+ metrics.IncrCounterWithLabels([]string{"catalog", "service", "query"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}})
if args.ServiceTag != "" {
metrics.IncrCounterWithLabels([]string{"consul", "catalog", "service", "query-tag"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}, {Name: "tag", Value: args.ServiceTag}})
+ metrics.IncrCounterWithLabels([]string{"catalog", "service", "query-tag"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}, {Name: "tag", Value: args.ServiceTag}})
}
if len(reply.ServiceNodes) == 0 {
metrics.IncrCounterWithLabels([]string{"consul", "catalog", "service", "not-found"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}})
+ metrics.IncrCounterWithLabels([]string{"catalog", "service", "not-found"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}})
}
}
return err
diff --git a/agent/consul/client.go b/agent/consul/client.go
index 87fe32e11aae..b8258bc2f53c 100644
--- a/agent/consul/client.go
+++ b/agent/consul/client.go
@@ -240,8 +240,10 @@ func (c *Client) RPC(method string, args interface{}, reply interface{}) error {
// Enforce the RPC limit.
metrics.IncrCounter([]string{"consul", "client", "rpc"}, 1)
+ metrics.IncrCounter([]string{"client", "rpc"}, 1)
if !c.rpcLimiter.Allow() {
metrics.IncrCounter([]string{"consul", "client", "rpc", "exceeded"}, 1)
+ metrics.IncrCounter([]string{"client", "rpc", "exceeded"}, 1)
return structs.ErrRPCRateExceeded
}
@@ -267,8 +269,10 @@ func (c *Client) SnapshotRPC(args *structs.SnapshotRequest, in io.Reader, out io
// Enforce the RPC limit.
metrics.IncrCounter([]string{"consul", "client", "rpc"}, 1)
+ metrics.IncrCounter([]string{"client", "rpc"}, 1)
if !c.rpcLimiter.Allow() {
metrics.IncrCounter([]string{"consul", "client", "rpc", "exceeded"}, 1)
+ metrics.IncrCounter([]string{"client", "rpc", "exceeded"}, 1)
return structs.ErrRPCRateExceeded
}
diff --git a/agent/consul/fsm.go b/agent/consul/fsm.go
index 202fcdc2afe3..4ccaffebb2fa 100644
--- a/agent/consul/fsm.go
+++ b/agent/consul/fsm.go
@@ -125,6 +125,7 @@ func (c *consulFSM) Apply(log *raft.Log) interface{} {
func (c *consulFSM) applyRegister(buf []byte, index uint64) interface{} {
defer metrics.MeasureSince([]string{"consul", "fsm", "register"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "register"}, time.Now())
var req structs.RegisterRequest
if err := structs.Decode(buf, &req); err != nil {
panic(fmt.Errorf("failed to decode request: %v", err))
@@ -140,6 +141,7 @@ func (c *consulFSM) applyRegister(buf []byte, index uint64) interface{} {
func (c *consulFSM) applyDeregister(buf []byte, index uint64) interface{} {
defer metrics.MeasureSince([]string{"consul", "fsm", "deregister"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "deregister"}, time.Now())
var req structs.DeregisterRequest
if err := structs.Decode(buf, &req); err != nil {
panic(fmt.Errorf("failed to decode request: %v", err))
@@ -174,6 +176,8 @@ func (c *consulFSM) applyKVSOperation(buf []byte, index uint64) interface{} {
}
defer metrics.MeasureSinceWithLabels([]string{"consul", "fsm", "kvs"}, time.Now(),
[]metrics.Label{{Name: "op", Value: string(req.Op)}})
+ defer metrics.MeasureSinceWithLabels([]string{"fsm", "kvs"}, time.Now(),
+ []metrics.Label{{Name: "op", Value: string(req.Op)}})
switch req.Op {
case api.KVSet:
return c.state.KVSSet(index, &req.DirEnt)
@@ -219,6 +223,8 @@ func (c *consulFSM) applySessionOperation(buf []byte, index uint64) interface{}
}
defer metrics.MeasureSinceWithLabels([]string{"consul", "fsm", "session"}, time.Now(),
[]metrics.Label{{Name: "op", Value: string(req.Op)}})
+ defer metrics.MeasureSinceWithLabels([]string{"fsm", "session"}, time.Now(),
+ []metrics.Label{{Name: "op", Value: string(req.Op)}})
switch req.Op {
case structs.SessionCreate:
if err := c.state.SessionCreate(index, &req.Session); err != nil {
@@ -240,6 +246,8 @@ func (c *consulFSM) applyACLOperation(buf []byte, index uint64) interface{} {
}
defer metrics.MeasureSinceWithLabels([]string{"consul", "fsm", "acl"}, time.Now(),
[]metrics.Label{{Name: "op", Value: string(req.Op)}})
+ defer metrics.MeasureSinceWithLabels([]string{"fsm", "acl"}, time.Now(),
+ []metrics.Label{{Name: "op", Value: string(req.Op)}})
switch req.Op {
case structs.ACLBootstrapInit:
enabled, err := c.state.ACLBootstrapInit(index)
@@ -272,6 +280,8 @@ func (c *consulFSM) applyTombstoneOperation(buf []byte, index uint64) interface{
}
defer metrics.MeasureSinceWithLabels([]string{"consul", "fsm", "tombstone"}, time.Now(),
[]metrics.Label{{Name: "op", Value: string(req.Op)}})
+ defer metrics.MeasureSinceWithLabels([]string{"fsm", "tombstone"}, time.Now(),
+ []metrics.Label{{Name: "op", Value: string(req.Op)}})
switch req.Op {
case structs.TombstoneReap:
return c.state.ReapTombstones(req.ReapIndex)
@@ -291,6 +301,7 @@ func (c *consulFSM) applyCoordinateBatchUpdate(buf []byte, index uint64) interfa
panic(fmt.Errorf("failed to decode batch updates: %v", err))
}
defer metrics.MeasureSince([]string{"consul", "fsm", "coordinate", "batch-update"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "coordinate", "batch-update"}, time.Now())
if err := c.state.CoordinateBatchUpdate(index, updates); err != nil {
return err
}
@@ -307,6 +318,8 @@ func (c *consulFSM) applyPreparedQueryOperation(buf []byte, index uint64) interf
defer metrics.MeasureSinceWithLabels([]string{"consul", "fsm", "prepared-query"}, time.Now(),
[]metrics.Label{{Name: "op", Value: string(req.Op)}})
+ defer metrics.MeasureSinceWithLabels([]string{"fsm", "prepared-query"}, time.Now(),
+ []metrics.Label{{Name: "op", Value: string(req.Op)}})
switch req.Op {
case structs.PreparedQueryCreate, structs.PreparedQueryUpdate:
return c.state.PreparedQuerySet(index, req.Query)
@@ -324,6 +337,7 @@ func (c *consulFSM) applyTxn(buf []byte, index uint64) interface{} {
panic(fmt.Errorf("failed to decode request: %v", err))
}
defer metrics.MeasureSince([]string{"consul", "fsm", "txn"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "txn"}, time.Now())
results, errors := c.state.TxnRW(index, req.Ops)
return structs.TxnResponse{
Results: results,
@@ -337,6 +351,7 @@ func (c *consulFSM) applyAutopilotUpdate(buf []byte, index uint64) interface{} {
panic(fmt.Errorf("failed to decode request: %v", err))
}
defer metrics.MeasureSince([]string{"consul", "fsm", "autopilot"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "autopilot"}, time.Now())
if req.CAS {
act, err := c.state.AutopilotCASConfig(index, req.Config.ModifyIndex, &req.Config)
@@ -506,6 +521,7 @@ func (c *consulFSM) Restore(old io.ReadCloser) error {
func (s *consulSnapshot) Persist(sink raft.SnapshotSink) error {
defer metrics.MeasureSince([]string{"consul", "fsm", "persist"}, time.Now())
+ defer metrics.MeasureSince([]string{"fsm", "persist"}, time.Now())
// Register the nodes
encoder := codec.NewEncoder(sink, msgpackHandle)
diff --git a/agent/consul/health_endpoint.go b/agent/consul/health_endpoint.go
index 2967df094e81..1cf7460444eb 100644
--- a/agent/consul/health_endpoint.go
+++ b/agent/consul/health_endpoint.go
@@ -141,13 +141,19 @@ func (h *Health) ServiceNodes(args *structs.ServiceSpecificRequest, reply *struc
if err == nil {
metrics.IncrCounterWithLabels([]string{"consul", "health", "service", "query"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}})
+ metrics.IncrCounterWithLabels([]string{"health", "service", "query"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}})
if args.ServiceTag != "" {
metrics.IncrCounterWithLabels([]string{"consul", "health", "service", "query-tag"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}, {Name: "tag", Value: args.ServiceTag}})
+ metrics.IncrCounterWithLabels([]string{"health", "service", "query-tag"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}, {Name: "tag", Value: args.ServiceTag}})
}
if len(reply.Nodes) == 0 {
metrics.IncrCounterWithLabels([]string{"consul", "health", "service", "not-found"}, 1,
[]metrics.Label{{Name: "service", Value: args.ServiceName}})
+ metrics.IncrCounterWithLabels([]string{"health", "service", "not-found"}, 1,
+ []metrics.Label{{Name: "service", Value: args.ServiceName}})
}
}
return err
diff --git a/agent/consul/kvs_endpoint.go b/agent/consul/kvs_endpoint.go
index 19591e8c638b..ced5c4119b5c 100644
--- a/agent/consul/kvs_endpoint.go
+++ b/agent/consul/kvs_endpoint.go
@@ -82,6 +82,7 @@ func (k *KVS) Apply(args *structs.KVSRequest, reply *bool) error {
return err
}
defer metrics.MeasureSince([]string{"consul", "kvs", "apply"}, time.Now())
+ defer metrics.MeasureSince([]string{"kvs", "apply"}, time.Now())
// Perform the pre-apply checks.
acl, err := k.srv.resolveToken(args.Token)
diff --git a/agent/consul/leader.go b/agent/consul/leader.go
index c6047bb89dcb..e4e84b4a4a2e 100644
--- a/agent/consul/leader.go
+++ b/agent/consul/leader.go
@@ -100,6 +100,7 @@ RECONCILE:
return
}
metrics.MeasureSince([]string{"consul", "leader", "barrier"}, start)
+ metrics.MeasureSince([]string{"leader", "barrier"}, start)
// Check if we need to handle initial leadership actions
if !establishedLeader {
@@ -411,6 +412,7 @@ func (s *Server) reconcileMember(member serf.Member) error {
return nil
}
defer metrics.MeasureSince([]string{"consul", "leader", "reconcileMember"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "reconcileMember"}, time.Now())
var err error
switch member.Status {
case serf.StatusAlive:
@@ -774,6 +776,7 @@ func (s *Server) removeConsulServer(m serf.Member, port int) error {
// to avoid blocking.
func (s *Server) reapTombstones(index uint64) {
defer metrics.MeasureSince([]string{"consul", "leader", "reapTombstones"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "reapTombstones"}, time.Now())
req := structs.TombstoneRequest{
Datacenter: s.config.Datacenter,
Op: structs.TombstoneReap,
diff --git a/agent/consul/prepared_query_endpoint.go b/agent/consul/prepared_query_endpoint.go
index 35337075f330..ff7fa4d38673 100644
--- a/agent/consul/prepared_query_endpoint.go
+++ b/agent/consul/prepared_query_endpoint.go
@@ -33,6 +33,7 @@ func (p *PreparedQuery) Apply(args *structs.PreparedQueryRequest, reply *string)
return err
}
defer metrics.MeasureSince([]string{"consul", "prepared-query", "apply"}, time.Now())
+ defer metrics.MeasureSince([]string{"prepared-query", "apply"}, time.Now())
// Validate the ID. We must create new IDs before applying to the Raft
// log since it's not deterministic.
@@ -287,6 +288,7 @@ func (p *PreparedQuery) Explain(args *structs.PreparedQueryExecuteRequest,
return err
}
defer metrics.MeasureSince([]string{"consul", "prepared-query", "explain"}, time.Now())
+ defer metrics.MeasureSince([]string{"prepared-query", "explain"}, time.Now())
// We have to do this ourselves since we are not doing a blocking RPC.
p.srv.setQueryMeta(&reply.QueryMeta)
@@ -334,6 +336,7 @@ func (p *PreparedQuery) Execute(args *structs.PreparedQueryExecuteRequest,
return err
}
defer metrics.MeasureSince([]string{"consul", "prepared-query", "execute"}, time.Now())
+ defer metrics.MeasureSince([]string{"prepared-query", "execute"}, time.Now())
// We have to do this ourselves since we are not doing a blocking RPC.
p.srv.setQueryMeta(&reply.QueryMeta)
@@ -444,6 +447,7 @@ func (p *PreparedQuery) ExecuteRemote(args *structs.PreparedQueryExecuteRemoteRe
return err
}
defer metrics.MeasureSince([]string{"consul", "prepared-query", "execute_remote"}, time.Now())
+ defer metrics.MeasureSince([]string{"prepared-query", "execute_remote"}, time.Now())
// We have to do this ourselves since we are not doing a blocking RPC.
p.srv.setQueryMeta(&reply.QueryMeta)
diff --git a/agent/consul/rpc.go b/agent/consul/rpc.go
index b14fcb651d28..e083258912cd 100644
--- a/agent/consul/rpc.go
+++ b/agent/consul/rpc.go
@@ -60,6 +60,7 @@ func (s *Server) listen(listener net.Listener) {
go s.handleConn(conn, false)
metrics.IncrCounter([]string{"consul", "rpc", "accept_conn"}, 1)
+ metrics.IncrCounter([]string{"rpc", "accept_conn"}, 1)
}
}
@@ -97,6 +98,7 @@ func (s *Server) handleConn(conn net.Conn, isTLS bool) {
case pool.RPCRaft:
metrics.IncrCounter([]string{"consul", "rpc", "raft_handoff"}, 1)
+ metrics.IncrCounter([]string{"rpc", "raft_handoff"}, 1)
s.raftLayer.Handoff(conn)
case pool.RPCTLS:
@@ -155,10 +157,12 @@ func (s *Server) handleConsulConn(conn net.Conn) {
if err != io.EOF && !strings.Contains(err.Error(), "closed") {
s.logger.Printf("[ERR] consul.rpc: RPC error: %v %s", err, logConn(conn))
metrics.IncrCounter([]string{"consul", "rpc", "request_error"}, 1)
+ metrics.IncrCounter([]string{"rpc", "request_error"}, 1)
}
return
}
metrics.IncrCounter([]string{"consul", "rpc", "request"}, 1)
+ metrics.IncrCounter([]string{"rpc", "request"}, 1)
}
}
@@ -263,6 +267,8 @@ func (s *Server) forwardDC(method, dc string, args interface{}, reply interface{
metrics.IncrCounterWithLabels([]string{"consul", "rpc", "cross-dc"}, 1,
[]metrics.Label{{Name: "datacenter", Value: dc}})
+ metrics.IncrCounterWithLabels([]string{"rpc", "cross-dc"}, 1,
+ []metrics.Label{{Name: "datacenter", Value: dc}})
if err := s.connPool.RPC(dc, server.Addr, server.Version, method, server.UseTLS, args, reply); err != nil {
manager.NotifyFailedServer(server)
s.logger.Printf("[ERR] consul: RPC failed to server %s in DC %q: %v", server.Addr, dc, err)
@@ -372,6 +378,7 @@ RUN_QUERY:
// Run the query.
metrics.IncrCounter([]string{"consul", "rpc", "query"}, 1)
+ metrics.IncrCounter([]string{"rpc", "query"}, 1)
// Operate on a consistent set of state. This makes sure that the
// abandon channel goes with the state that the caller is using to
@@ -422,6 +429,7 @@ func (s *Server) setQueryMeta(m *structs.QueryMeta) {
// read. This is done by verifying leadership before the read.
func (s *Server) consistentRead() error {
defer metrics.MeasureSince([]string{"consul", "rpc", "consistentRead"}, time.Now())
+ defer metrics.MeasureSince([]string{"rpc", "consistentRead"}, time.Now())
future := s.raft.VerifyLeader()
if err := future.Error(); err != nil {
return err //fail fast if leader verification fails
diff --git a/agent/consul/segment_stub.go b/agent/consul/segment_stub.go
index c72d0c99fdd9..44f447124b96 100644
--- a/agent/consul/segment_stub.go
+++ b/agent/consul/segment_stub.go
@@ -60,6 +60,7 @@ func (s *Server) floodSegments(config *Config) {
// left nodes are de-registered.
func (s *Server) reconcile() (err error) {
defer metrics.MeasureSince([]string{"consul", "leader", "reconcile"}, time.Now())
+ defer metrics.MeasureSince([]string{"leader", "reconcile"}, time.Now())
members := s.serfLAN.Members()
knownMembers := make(map[string]struct{})
for _, member := range members {
diff --git a/agent/consul/session_endpoint.go b/agent/consul/session_endpoint.go
index 36d082a3bb26..54bb32f81abf 100644
--- a/agent/consul/session_endpoint.go
+++ b/agent/consul/session_endpoint.go
@@ -24,6 +24,7 @@ func (s *Session) Apply(args *structs.SessionRequest, reply *string) error {
return err
}
defer metrics.MeasureSince([]string{"consul", "session", "apply"}, time.Now())
+ defer metrics.MeasureSince([]string{"session", "apply"}, time.Now())
// Verify the args
if args.Session.ID == "" && args.Op == structs.SessionDestroy {
@@ -222,6 +223,7 @@ func (s *Session) Renew(args *structs.SessionSpecificRequest,
return err
}
defer metrics.MeasureSince([]string{"consul", "session", "renew"}, time.Now())
+ defer metrics.MeasureSince([]string{"session", "renew"}, time.Now())
// Get the session, from local state.
state := s.srv.fsm.State()
diff --git a/agent/consul/session_ttl.go b/agent/consul/session_ttl.go
index 53a1840edfe5..99247366f45b 100644
--- a/agent/consul/session_ttl.go
+++ b/agent/consul/session_ttl.go
@@ -85,6 +85,7 @@ func (s *Server) createSessionTimer(id string, ttl time.Duration) {
// need to invalidate the session.
func (s *Server) invalidateSession(id string) {
defer metrics.MeasureSince([]string{"consul", "session_ttl", "invalidate"}, time.Now())
+ defer metrics.MeasureSince([]string{"session_ttl", "invalidate"}, time.Now())
// Clear the session timer
s.sessionTimers.Del(id)
@@ -134,6 +135,7 @@ func (s *Server) sessionStats() {
select {
case <-time.After(5 * time.Second):
metrics.SetGauge([]string{"consul", "session_ttl", "active"}, float32(s.sessionTimers.Len()))
+ metrics.SetGauge([]string{"session_ttl", "active"}, float32(s.sessionTimers.Len()))
case <-s.shutdownCh:
return
diff --git a/agent/consul/txn_endpoint.go b/agent/consul/txn_endpoint.go
index fc2862e6ca90..7085548c8ad0 100644
--- a/agent/consul/txn_endpoint.go
+++ b/agent/consul/txn_endpoint.go
@@ -47,6 +47,7 @@ func (t *Txn) Apply(args *structs.TxnRequest, reply *structs.TxnResponse) error
return err
}
defer metrics.MeasureSince([]string{"consul", "txn", "apply"}, time.Now())
+ defer metrics.MeasureSince([]string{"txn", "apply"}, time.Now())
// Run the pre-checks before we send the transaction into Raft.
acl, err := t.srv.resolveToken(args.Token)
@@ -90,6 +91,7 @@ func (t *Txn) Read(args *structs.TxnReadRequest, reply *structs.TxnReadResponse)
return err
}
defer metrics.MeasureSince([]string{"consul", "txn", "read"}, time.Now())
+ defer metrics.MeasureSince([]string{"txn", "read"}, time.Now())
// We have to do this ourselves since we are not doing a blocking RPC.
t.srv.setQueryMeta(&reply.QueryMeta)
diff --git a/agent/dns.go b/agent/dns.go
index 6a2f748965a0..cd89640317bf 100644
--- a/agent/dns.go
+++ b/agent/dns.go
@@ -156,6 +156,8 @@ func (d *DNSServer) handlePtr(resp dns.ResponseWriter, req *dns.Msg) {
defer func(s time.Time) {
metrics.MeasureSinceWithLabels([]string{"consul", "dns", "ptr_query"}, s,
[]metrics.Label{{Name: "node", Value: d.agent.config.NodeName}})
+ metrics.MeasureSinceWithLabels([]string{"dns", "ptr_query"}, s,
+ []metrics.Label{{Name: "node", Value: d.agent.config.NodeName}})
d.logger.Printf("[DEBUG] dns: request for %v (%v) from client %s (%s)",
q, time.Now().Sub(s), resp.RemoteAddr().String(),
resp.RemoteAddr().Network())
@@ -226,6 +228,8 @@ func (d *DNSServer) handleQuery(resp dns.ResponseWriter, req *dns.Msg) {
defer func(s time.Time) {
metrics.MeasureSinceWithLabels([]string{"consul", "dns", "domain_query"}, s,
[]metrics.Label{{Name: "node", Value: d.agent.config.NodeName}})
+ metrics.MeasureSinceWithLabels([]string{"dns", "domain_query"}, s,
+ []metrics.Label{{Name: "node", Value: d.agent.config.NodeName}})
d.logger.Printf("[DEBUG] dns: request for %v (%v) from client %s (%s)",
q, time.Now().Sub(s), resp.RemoteAddr().String(),
resp.RemoteAddr().Network())
@@ -516,6 +520,7 @@ RPC:
goto RPC
} else if out.LastContact > staleCounterThreshold {
metrics.IncrCounter([]string{"consul", "dns", "stale_queries"}, 1)
+ metrics.IncrCounter([]string{"dns", "stale_queries"}, 1)
}
}
@@ -761,6 +766,7 @@ func (d *DNSServer) lookupServiceNodes(datacenter, service, tag string) (structs
if args.AllowStale && out.LastContact > staleCounterThreshold {
metrics.IncrCounter([]string{"consul", "dns", "stale_queries"}, 1)
+ metrics.IncrCounter([]string{"dns", "stale_queries"}, 1)
}
// redo the request the response was too stale
@@ -887,6 +893,7 @@ RPC:
goto RPC
} else if out.LastContact > staleCounterThreshold {
metrics.IncrCounter([]string{"consul", "dns", "stale_queries"}, 1)
+ metrics.IncrCounter([]string{"dns", "stale_queries"}, 1)
}
}
diff --git a/command/agent.go b/command/agent.go
index 580f32ef4159..78d6577e40cb 100644
--- a/command/agent.go
+++ b/command/agent.go
@@ -223,6 +223,8 @@ func startupTelemetry(conf *config.RuntimeConfig) (*metrics.InmemSink, error) {
metricsConf := metrics.DefaultConfig(conf.TelemetryMetricsPrefix)
metricsConf.EnableHostname = !conf.TelemetryDisableHostname
metricsConf.FilterDefault = conf.TelemetryFilterDefault
+ metricsConf.AllowedPrefixes = conf.TelemetryAllowedPrefixes
+ metricsConf.BlockedPrefixes = conf.TelemetryBlockedPrefixes
var sinks metrics.FanoutSink
addSink := func(name string, fn func(*config.RuntimeConfig, string) (metrics.MetricSink, error)) error {
diff --git a/website/source/docs/agent/options.html.md b/website/source/docs/agent/options.html.md
index 66a4320c9ae2..00c8f5a3f83c 100644
--- a/website/source/docs/agent/options.html.md
+++ b/website/source/docs/agent/options.html.md
@@ -1168,6 +1168,10 @@ Consul will not enable TLS for the HTTP API unless the `https` port has been ass
is overlap between two rules, the more specific rule will take precedence. Blocking will take priority if the same
prefix is listed multiple times.
+ * `enable_deprecated_names`
+ Added in Consul 1.0, this enables old metric names of the format `consul.consul...` to be sent alongside
+ other metrics. Defaults to false.
+
* `statsd_address` This provides the
address of a statsd instance in the format `host:port`. If provided, Consul will send various telemetry information to that instance for
aggregation. This can be used to capture runtime information. This sends UDP packets only and can be used with
diff --git a/website/source/docs/agent/telemetry.html.md b/website/source/docs/agent/telemetry.html.md
index 2164f889a24f..6f414634860a 100644
--- a/website/source/docs/agent/telemetry.html.md
+++ b/website/source/docs/agent/telemetry.html.md
@@ -92,6 +92,48 @@ These metrics are used to monitor the health of specific Consul agents.
| number of objects |
gauge |
+
+ | `consul.acl.cache_hit` |
+ The number of ACL cache hits. |
+ hits |
+ counter |
+
+
+ | `consul.acl.cache_miss` |
+ The number of ACL cache misses. |
+ misses |
+ counter |
+
+
+ | `consul.acl.replication_hit` |
+ The number of ACL replication cache hits (when not running in the ACL datacenter). |
+ hits |
+ counter |
+
+
+ | `consul.dns.stale_queries` |
+ This increments when an agent serves a query within the allowed stale threshold. |
+ queries |
+ counter |
+
+
+ | `consul.dns.ptr_query.` |
+ This measures the time spent handling a reverse DNS query for the given node. |
+ ms |
+ timer |
+
+
+ | `consul.dns.domain_query.` |
+ This measures the time spent handling a domain query for the given node. |
+ ms |
+ timer |
+
+
+ | `consul.http..` |
+ This tracks how long it takes to service the given HTTP request for the given verb and path. Paths do not include details like service or key names, for these an underscore will be present as a placeholder (eg. `consul.http.GET.v1.kv._`) |
+ ms |
+ timer |
+
## Server Health
@@ -147,6 +189,239 @@ These metrics are used to monitor the health of the Consul servers.
ms |
timer |
+
+ | `consul.acl.apply` |
+ This measures the time it takes to complete an update to the ACL store. |
+ ms |
+ timer |
+
+
+ | `consul.acl.fault` |
+ This measures the time it takes to fault in the rules for an ACL during a cache miss. |
+ ms |
+ timer |
+
+
+ | `consul.acl.fetchRemoteACLs` |
+ This measures the time it takes to fetch remote ACLs during replication. |
+ ms |
+ timer |
+
+
+ | `consul.acl.updateLocalACLs` |
+ This measures the time it takes to apply replication changes to the local ACL store. |
+ ms |
+ timer |
+
+
+ | `consul.acl.replicateACLs` |
+ This measures the time it takes to do one pass of the ACL replication algorithm. |
+ ms |
+ timer |
+
+
+ | `consul.acl.resolveToken` |
+ This measures the time it takes to resolve an ACL token. |
+ ms |
+ timer |
+
+
+ | `consul.rpc.accept_conn` |
+ This increments when a server accepts an RPC connection. |
+ connections |
+ counter |
+
+
+ | `consul.catalog.register` |
+ This measures the time it takes to complete a catalog register operation. |
+ ms |
+ timer |
+
+
+ | `consul.catalog.deregister` |
+ This measures the time it takes to complete a catalog deregister operation. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.register` |
+ This measures the time it takes to apply a catalog register operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.deregister` |
+ This measures the time it takes to apply a catalog deregister operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.acl.` |
+ This measures the time it takes to apply the given ACL operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.session.` |
+ This measures the time it takes to apply the given session operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.kvs.` |
+ This measures the time it takes to apply the given KV operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.tombstone.` |
+ This measures the time it takes to apply the given tombstone operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.coordinate.batch-update` |
+ This measures the time it takes to apply the given batch coordinate update to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.prepared-query.` |
+ This measures the time it takes to apply the given prepared query update operation to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.txn` |
+ This measures the time it takes to apply the given transaction update to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.autopilot` |
+ This measures the time it takes to apply the given autopilot update to the FSM. |
+ ms |
+ timer |
+
+
+ | `consul.fsm.persist` |
+ This measures the time it takes to persist the FSM to a raft snapshot. |
+ ms |
+ timer |
+
+
+ | `consul.kvs.apply` |
+ This measures the time it takes to complete an update to the KV store. |
+ ms |
+ timer |
+
+
+ | `consul.leader.barrier` |
+ This measures the time spent waiting for the raft barrier upon gaining leadership. |
+ ms |
+ timer |
+
+
+ | `consul.leader.reconcile` |
+ This measures the time spent updating the raft store from the serf member information. |
+ ms |
+ timer |
+
+
+ | `consul.leader.reconcileMember` |
+ This measures the time spent updating the raft store for a single serf member's information. |
+ ms |
+ timer |
+
+
+ | `consul.leader.reapTombstones` |
+ This measures the time spent clearing tombstones. |
+ ms |
+ timer |
+
+
+ | `consul.prepared-query.apply` |
+ This measures the time it takes to apply a prepared query update. |
+ ms |
+ timer |
+
+
+ | `consul.prepared-query.explain` |
+ This measures the time it takes to process a prepared query explain request. |
+ ms |
+ timer |
+
+
+ | `consul.prepared-query.execute` |
+ This measures the time it takes to process a prepared query execute request. |
+ ms |
+ timer |
+
+
+ | `consul.prepared-query.execute` |
+ This measures the time it takes to process a prepared query execute request that was forwarded to another datacenter. |
+ ms |
+ timer |
+
+
+ | `consul.rpc.raft_handoff` |
+ This increments when a server accepts a Raft-related RPC connection. |
+ connections |
+ counter |
+
+
+ | `consul.rpc.request_error` |
+ This increments when a server returns an error from an RPC request. |
+ errors |
+ counter |
+
+
+ | `consul.rpc.request` |
+ This increments when a server receives a Consul-related RPC request. |
+ requests |
+ counter |
+
+
+ | `consul.rpc.query` |
+ This increments when a server receives a (potentially blocking) RPC query. |
+ queries |
+ counter |
+
+
+ | `consul.rpc.consistentRead` |
+ This measures the time spent confirming that a consistent read can be performed. |
+ ms |
+ timer |
+
+
+ | `consul.session.apply` |
+ This measures the time spent applying a session update. |
+ ms |
+ timer |
+
+
+ | `consul.session.renew` |
+ This measures the time spent renewing a session. |
+ ms |
+ timer |
+
+
+ | `consul.session_ttl.invalidate` |
+ This measures the time spent invalidating an expired session. |
+ ms |
+ timer |
+
+
+ | `consul.txn.apply` |
+ This measures the time spent applying a transaction operation. |
+ ms |
+ timer |
+
+ `consul.txn.read` |
+ This measures the time spent returning a read transaction. |
+ ms |
+ timer |
+
## Cluster Health
@@ -179,39 +454,57 @@ These metrics give insight into the health of the cluster as a whole.
counter |
- | `consul.dns.domain_query.` |
- This tracks how long it takes to service forward DNS lookups on the given Consul agent. |
- ms |
- timer |
+ `consul.autopilot.failure_tolerance` |
+ This tracks the number of voting servers that the cluster can lose while continuing to function. |
+ servers |
+ gauge |
- | `consul.dns.ptr_query.` |
- This tracks how long it takes to service reverse DNS lookups on the given Consul agent. |
- ms |
- timer |
+ `consul.autopilot.healthy` |
+ This tracks the overall health of the local server cluster. If all servers are considered healthy by Autopilot, this will be set to 1. If any are unhealthy, this will be 0. |
+ boolean |
+ gauge |
- | `consul.dns.stale_queries` |
- Available in Consul 0.7.1 and later, this increments when an agent serves a DNS query based on information from a server that is more than 5 seconds out of date. |
+ `consul.session_ttl.active` |
+ This tracks the active number of sessions being tracked. |
+ sessions |
+ gauge |
+
+
+ | `consul.catalog.service.query.` |
+ This increments for each catalog query for the given service. |
queries |
counter |
- | `consul.http..` |
- This tracks how long it takes to service the given HTTP request for the given verb and path. Paths do not include details like service or key names, for these an underscore will be present as a placeholder (eg. `consul.http.GET.v1.kv._`) |
- ms |
- timer |
+ `consul.catalog.service.query-tag..` |
+ This increments for each catalog query for the given service with the given tag. |
+ queries |
+ counter |
- | `consul.autopilot.failure_tolerance` |
- This tracks the number of voting servers that the cluster can lose while continuing to function. |
- servers |
- gauge |
+ `consul.catalog.service.not-found.` |
+ This increments for each catalog query where the given service could not be found. |
+ queries |
+ counter |
- | `consul.autopilot.healthy` |
- This tracks the overall health of the local server cluster. If all servers are considered healthy by Autopilot, this will be set to 1. If any are unhealthy, this will be 0. |
- boolean |
- gauge |
+ `consul.health.service.query.` |
+ This increments for each health query for the given service. |
+ queries |
+ counter |
+
+
+ | `consul.health.service.query-tag..` |
+ This increments for each health query for the given service with the given tag. |
+ queries |
+ counter |
+
+
+ | `consul.health.service.not-found.` |
+ This increments for each health query where the given service could not be found. |
+ queries |
+ counter |