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

Add connection telemetry API #27

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Add connection telemetry API #27

wants to merge 8 commits into from

Conversation

mattludwigs
Copy link
Contributor

@mattludwigs mattludwigs commented Jun 1, 2021

What are your thoughts on what we want to measure?

We can change the telemetry to just be [:vintage_net_qmi, :connection] and have the state be in the measurement, maybe a timestamp? We can do duration of the last connection state or something?

@mattludwigs mattludwigs requested a review from fhunleth June 1, 2021 21:57
@mattludwigs mattludwigs force-pushed the telemetry branch 2 times, most recently from b3be8c4 to e3b3402 Compare June 2, 2021 18:53
@mattludwigs mattludwigs force-pushed the telemetry branch 4 times, most recently from d38adfb to 2f4d947 Compare July 19, 2021 18:17
@fhunleth fhunleth force-pushed the telemetry branch 2 times, most recently from 90b647d to 55bbf64 Compare August 26, 2021 22:45
@mattludwigs mattludwigs force-pushed the telemetry branch 3 times, most recently from 374c008 to 5bc4fb3 Compare September 8, 2021 16:51
@mattludwigs mattludwigs force-pushed the telemetry branch 3 times, most recently from afa4cb2 to 50ba9ea Compare September 22, 2021 21:52
@fhunleth fhunleth force-pushed the telemetry branch 2 times, most recently from 596b012 to 8414217 Compare May 2, 2022 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants