envoyproxy/envoy

server.hot_restart_generation does not include tags defined in stats_config.stats_tags

Open

#10,719 建立於 2020年4月9日

在 GitHub 查看
 (1 留言) (0 反應) (1 負責人)C++ (5,373 fork)batch import
area/statsbughelp wanted

倉庫指標

Star
 (27,997 star)
PR 合併指標
 (平均合併 8天) (30 天內合併 378 個 PR)

描述

Title: gauge server.hot_restart_generation is not tagged

Description:

I've been trying out the new 1.14.1 release, and happened to notice that there's this new server.hot_restart_generation metric, which is the only metric (in our setup) that doesn't get the tags defined in stats_config.tags.

stats_config:
  stats_tags:
    - tag_name: foo
      fixed_value: bar

stats_sinks:
  - name: "envoy.dog_statsd"
    typed_config:
      "@type": type.googleapis.com/envoy.config.metrics.v2.DogStatsdSink
      address:
        socket_address:
          protocol: "UDP"
          address: "x.x.x.x"
          port_value: 8125

Maybe this is working as designed, but I though it could be a bug. E.g. we tag our metrics by service id and environment (among other things), and in our infrastructure a metric without these tags provides no value since we can't tell where it comes from.

I thought of filtering it out via the stats_matcher config but I got second thoughts from the warning in the docs:

Excluding stats may affect Envoy’s behavior in undocumented ways. See issue #8771 for more information. If any unexpected behavior changes are observed, please open a new issue immediately.

貢獻者指南