-
Notifications
You must be signed in to change notification settings - Fork 441
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
internal/telemetry: revamp #2996
base: main
Are you sure you want to change the base?
Conversation
BenchmarksBenchmark execution time: 2025-01-22 18:00:50 Comparing candidate commit bce2138 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 0 unstable metrics. |
bb195d0
to
58aaf77
Compare
Datadog ReportBranch report: ❌ 4 Failed (0 Known Flaky), 5225 Passed, 72 Skipped, 2m 27.22s Total Time ❌ Failed Tests (4)
|
3624406
to
54a315e
Compare
Signed-off-by: Eliott Bouhana <[email protected]> wip Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
centralize all http concerns in the writer struct such as: * Reusing the same top-level structures when possible * encoding properly in json the payload * separating the low level http writing from some higher level things we will concerns ourselves about in the telemetry client itself * Centralizing a lot of the static data the telemtry client is supposed to send on every request Signed-off-by: Eliott Bouhana <[email protected]>
…nge payloads Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
… the Size() method on data sources. Put back support for the payloadQueue in case Flush calls are failing Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
Signed-off-by: Eliott Bouhana <[email protected]>
71bf34b
to
6dad2a3
Compare
Signed-off-by: Eliott Bouhana <[email protected]>
What does this PR do?
This PR rebuilds fron scratch the current instrumentation telemetry client. More informations here:
https://docs.google.com/document/d/1C3xoTZHQZNQ-Jr1w5mMThIkpHNW_4qKRf-J5CNi7G-k/edit?pli=1&tab=t.0#heading=h.88xvn2cvs9dt
Reviewer's Checklist
Unsure? Have a question? Request a review!