Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Telemetry could be aggregated at a configurable interval, or at configurable times #5977

Closed
kennsippell opened this issue Sep 23, 2019 · 3 comments
Assignees
Labels
Monitoring For the visibility of system status Priority: 2 - Medium Normal priority Type: Feature Add something new Won't fix: Duplicate Covered by a different issue

Comments

@kennsippell
Copy link
Member

kennsippell commented Sep 23, 2019

Is your feature request related to a problem? Please describe.
The Muso-Mali project team is anecdotally reporting that the upgrade from v3.5 to v3.6.1 caused a performance regression. v3.5 was deployed on July 5th and the upgrade to v3.6.1 occurred within the same month. @senseibara attempted to analyse the telemetry data to support or refute the claim that performance regressed, but can only look at aggregate date for the full month.

A build rollout is a significant event and tracking telemetry metrics before/after would be extremely useful for understanding the performance impacts of our code changes.

Describe the solution you'd like
Currently we aggregate all telemetry data into a single monthly document. Perhaps we could aggregate by month-build so if a deployment rolls out mid-month, we can compare data before/after the rollout.

@kennsippell kennsippell added the Type: Feature Add something new label Sep 23, 2019
@kennsippell kennsippell reopened this Sep 23, 2019
@SCdF
Copy link
Contributor

SCdF commented Sep 24, 2019

I think it's worth building up these learnings about things we'd like and channeling them into an investigation of "real" (ie third party) tools to replace the existing telemetry work we've done.

This gives us the opportunity to either perform that replacement (and not spend time improving something we've ultimately decided to throw away), or decide that our situation is unique enough to warrant custom work and then treat the existing telemetry as first class (and so then start caring about UIs and proper support and the like).

@MaxDiz MaxDiz added the Analytics Affects the Analytics (aka Targets) page label Oct 9, 2019
@garethbowen garethbowen added the Help wanted Good for first time contributions label Oct 16, 2019
@kennsippell
Copy link
Member Author

kennsippell commented Sep 23, 2020

Although a core-upgrade specific workflow is an improvement. Reflecting on this need more generally, it would be great if telemetry could be aggregated at a configurable interval, or at configurable times.

Essentially what we want is to be able to look at telemetry before an event (core upgrade, config rollout, training, etc), and look at telemetry after an event to make comparisons.

Some projects have lots of air time, strong connectivity, newer phones and they can easily accomodate a higher rate of telemetry reporting - daily probably makes sense here. Other projects maybe do need to aggregate less frequently and the configuration should be more conservative - like monthly (or disabled).

@kennsippell kennsippell changed the title Telemetry data should be aggregated by build and month Telemetry could be aggregated at a configurable interval, or at configurable times Sep 24, 2020
@MaxDiz MaxDiz added Monitoring For the visibility of system status and removed Analytics Affects the Analytics (aka Targets) page labels Oct 12, 2020
@MaxDiz MaxDiz added the Priority: 2 - Medium Normal priority label Oct 18, 2020
@mrsarm mrsarm self-assigned this Mar 26, 2021
@mrsarm
Copy link
Contributor

mrsarm commented Apr 6, 2021

@kennsippell ,

I'm going to close this ticket taking into account that #6915 supersedes it, but feel free to re-open it if you think #6915 doesn't cover all the needs explained in this ticket, or open a new one that covers the missed bits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitoring For the visibility of system status Priority: 2 - Medium Normal priority Type: Feature Add something new Won't fix: Duplicate Covered by a different issue
Projects
None yet
Development

No branches or pull requests

5 participants