Skip to content

[AGTMETRICS-347]Add cgroup origin detection#211

Merged
StephenWakely merged 19 commits intomasterfrom
stephen/cgroup_origindetection
Aug 12, 2025
Merged

[AGTMETRICS-347]Add cgroup origin detection#211
StephenWakely merged 19 commits intomasterfrom
stephen/cgroup_origindetection

Conversation

@StephenWakely
Copy link
Copy Markdown
Contributor

@StephenWakely StephenWakely commented Jul 16, 2025

Adds cgroup origin detection. If origin detection is enabled, will attempt to determine the container id via several different procfs entries typically used by container runtimes. The container id is added to the metric as a c: field.

@StephenWakely StephenWakely requested a review from a team as a code owner July 16, 2025 13:39
@StephenWakely StephenWakely marked this pull request as draft July 16, 2025 15:57
@StephenWakely StephenWakely marked this pull request as ready for review August 11, 2025 13:27
Comment thread src/StatsdClient/StatsdConfig.cs Outdated
/// <summary>
/// Gets or sets the container id to tag the telemetry with.
/// </summary>
public string ContainerID { get; set; }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We actually let callers set the container ID manually in other DSD client libraries? 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@StephenWakely StephenWakely requested a review from tobz August 11, 2025 14:46
@StephenWakely StephenWakely merged commit 8481adb into master Aug 12, 2025
50 of 54 checks passed
@tobz tobz deleted the stephen/cgroup_origindetection branch August 12, 2025 13:06
@StephenWakely StephenWakely changed the title Add cgroup origin detection [AGTMETRICS-347]Add cgroup origin detection Aug 12, 2025
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