Skip to content

Overhaul stats: add a configuration option to enable/disable stats per server #1401

Description

@josecelano

This feature is not implemented yet. In fact this is the first step. I want to add a new config option to enable/disable stats for each service (associated to a socket address).

[core]
tracker_usage_statistics = true

[[udp_trackers]]
bind_address = "0.0.0.0:6969"
tracker_usage_statistics = true  <- the new option

  [udp_trackers.cookie_lifetime]
  secs = 120
  nanos = 0

[[http_trackers]]
bind_address = "0.0.0.0:7070"
tracker_usage_statistics = true

[http_api]
bind_address = "0.0.0.0:1212"

  [http_api.access_tokens]
  admin = "***"

[health_check_api]
bind_address = "127.0.0.1:1313"
  • I will use the same name we are using for the global metrics: tracker_usage_statistics.
  • The default value will be false.
  • For the time being, events are only used for stats even if they are already decoupled from stats. But for this issue and the parent issue we will continue enabling events when stats are enabled. We will introduce a new config option in the future to enable events even if you want to disable stats (why? because you might want to build your own stats).

This new config option will have no effect until the feature is implemented.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions