Skip to content

[RUM-10044] ✨ add new configuration parameters to telemetry - #290

Merged
dd-mergequeue[bot] merged 1 commit into
masterfrom
adlrb/overridable-source
Jul 28, 2025
Merged

[RUM-10044] ✨ add new configuration parameters to telemetry#290
dd-mergequeue[bot] merged 1 commit into
masterfrom
adlrb/overridable-source

Conversation

@mormubis

@mormubis mormubis commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

We implemented overridable fields for cross platform extensions in the Browser SDK. We want to report those fields as part of the telemetry events.

@mormubis
mormubis force-pushed the adlrb/overridable-source branch from 305142e to c891e32 Compare July 24, 2025 13:38
@mormubis mormubis changed the title ✨ add new configuration parameters to telemetry [RUM-10044] ✨ add new configuration parameters to telemetry Jul 24, 2025
@mormubis
mormubis marked this pull request as ready for review July 24, 2025 13:56
@mormubis
mormubis requested review from a team as code owners July 24, 2025 13:57
@mormubis

mormubis commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jul 28, 2025 •
edited
Loading

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2025-07-28 08:59:52 UTC ℹ️ Start processing command /merge

2025-07-28 08:59:58 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 10m (p90).

2025-07-28 09:03:16 UTC ❌ MergeQueue: The checks failed on this merge request

Tests failed on this commit ea38f07:

What to do next?

  • Investigate the failures and when ready, re-add your pull request to the queue!
  • If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
  • Any question, go check the FAQ.

@mormubis

mormubis commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jul 28, 2025 •
edited
Loading

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2025-07-28 09:32:42 UTC ℹ️ Start processing command /merge

2025-07-28 09:32:48 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 10m (p90).

2025-07-28 09:36:02 UTC ❌ MergeQueue: The checks failed on this merge request

Tests failed on this commit 62ea8fb:

What to do next?

  • Investigate the failures and when ready, re-add your pull request to the queue!
  • If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
  • Any question, go check the FAQ.

@mormubis

mormubis commented Jul 28, 2025

Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jul 28, 2025 •
edited
Loading

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2025-07-28 11:44:10 UTC ℹ️ Start processing command /merge

2025-07-28 11:44:16 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 10m (p90).

2025-07-28 11:51:34 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue
dd-mergequeue Bot merged commit 0aaecb0 into master Jul 28, 2025
13 checks passed
@dd-mergequeue
dd-mergequeue Bot deleted the adlrb/overridable-source branch July 28, 2025 11:51
eloytoro pushed a commit that referenced this pull request Sep 11, 2025
@mormubis @eloytoro
✨ add new configuration parameters to telemetry (#290)
bb71881
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BenoitZugmeyer BenoitZugmeyer BenoitZugmeyer approved these changes
@RomanWuattier RomanWuattier RomanWuattier approved these changes
@barboraplasovska barboraplasovska barboraplasovska approved these changes
@rgaignault rgaignault rgaignault approved these changes
@sethfowler-datadog sethfowler-datadog sethfowler-datadog approved these changes

Assignees

No one assigned

Labels

mergequeue-status: done

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@mormubis @BenoitZugmeyer @RomanWuattier @barboraplasovska @rgaignault @sethfowler-datadog