InfluxDB Metrics Backend #3601
InfluxDB Metrics Backend #3601
Conversation
remove round_time, not workable fix config add missing stats exemption srsly
|
Sidelined pending implementation of Otherwise the timestamps are rounded to 1-2 minutes resolution which defeats half the point of this Edit: Done as a temporary measure, now tick is timed to microsecond resolution (wow, very useful) |
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
About the pull request
Wider scope version of #3480
This ships the following game metrics to InfluxDB:
It uses two separate subsystems, one for collecting metrics, the other for shipping them - and provides procs for easily sending more data to wire up other code.
Explain why it's good for the game
Gives fine grained control over game data that we don't have. Currently SOME of these are logged to SQL, and we'd need to create our own vis UI. Backing them by a TSDB allows to view evolutions over time, and means we can plug in standard dashboarding tooling like Grafana.
Testing Photographs and Procedure
(okay it's technically from a TM of the older PR)
Changelog
🆑
add: Added InfluxDB backed metrics logging for some of the most used game statistics. This will allow to graph them over time and give better insight as to what happens in rounds.
/:cl: