search

matrix-org / panopticon

panopticon records usage metrics from homeservers
Apache License 2.0
13 stars 6 forks source link

Report log level. #17

Closed clokep closed 3 years ago

clokep commented 3 years ago

This matches the changes from matrix-org/synapse#8477.

clokep commented 3 years ago

Wasn't sure who to request review from, hopefully one of you is correct. 😄

clokep commented 3 years ago

While you are in the area could you also check if other datapoints are transmitted and not currently handled?

For instance I think count_monthly_users is an example.

I cross-referenced and it seems monthly_active_users is not handled, I assume this is the same thing!

I'll add that and take a look at aggregating it.

clokep commented 3 years ago

@neilisfragile I think I made the proper changes, the docker-tests.sh pass, but I'm unsure how/if there's anyway to test the aggregation.

clokep commented 3 years ago

This was deployed in release v0.1.3 and I confirmed that both new columns are having data added.