Commit f483992
fix(telemetry): include dependencies and integrations in app-extended-heartbeat
Per the instrumentation-telemetry-api-docs schema, app-extended-heartbeat
must carry the full state — configuration, dependencies, and integrations —
so the agent can reconstruct application records on data loss. dd-trace-go
and dd-trace-dotnet both ship the full triple. The Rust worker, however,
defines AppStarted as a configuration-only struct and reuses it for
app-extended-heartbeat, so dependencies and integrations are not included
in the heartbeat payload.
The ExtendedHeartbeat handler does call unflush_stored() on all three
collectors, evidently with the intent of re-emitting the full state.
Because the heartbeat payload omits the re-queued dependencies and
integrations and app_started_sent_success only pops configurations from
unflushed, the re-queued items remain in unflushed and are sent on the
next FlushData as a duplicate app-integrations-change /
app-dependencies-loaded.
Updating the shared AppStarted struct, build_app_started, and
app_started_sent_success addresses both Lifecycle(Start) and
Lifecycle(ExtendedHeartbeat) call sites.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 4ae8ebe commit f483992
8 files changed
Lines changed: 22 additions & 6 deletions
File tree
- libdd-crashtracker
- libdd-data-pipeline
- libdd-telemetry
- examples
- src
- data
- worker
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
53 | | - | |
| 53 | + | |
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
37 | | - | |
| 37 | + | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| 20 | + | |
| 21 | + | |
20 | 22 | | |
21 | 23 | | |
22 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
| 69 | + | |
| 70 | + | |
69 | 71 | | |
70 | 72 | | |
71 | 73 | | |
| |||
88 | 90 | | |
89 | 91 | | |
90 | 92 | | |
91 | | - | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
92 | 96 | | |
93 | 97 | | |
94 | 98 | | |
| |||
490 | 494 | | |
491 | 495 | | |
492 | 496 | | |
| 497 | + | |
| 498 | + | |
493 | 499 | | |
494 | 500 | | |
495 | 501 | | |
| |||
505 | 511 | | |
506 | 512 | | |
507 | 513 | | |
508 | | - | |
| 514 | + | |
| 515 | + | |
| 516 | + | |
509 | 517 | | |
510 | 518 | | |
511 | 519 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
| 51 | + | |
| 52 | + | |
51 | 53 | | |
52 | 54 | | |
53 | 55 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
673 | 673 | | |
674 | 674 | | |
675 | 675 | | |
| 676 | + | |
| 677 | + | |
676 | 678 | | |
677 | 679 | | |
678 | 680 | | |
679 | 681 | | |
680 | 682 | | |
681 | 683 | | |
682 | 684 | | |
| 685 | + | |
| 686 | + | |
683 | 687 | | |
684 | 688 | | |
685 | 689 | | |
| |||
0 commit comments