Skip to content

custom labels upstream#52

Closed
gnurizen wants to merge 16 commits intomainfrom
custom-labels-upstream
Closed

custom labels upstream#52
gnurizen wants to merge 16 commits intomainfrom
custom-labels-upstream

Conversation

@gnurizen
Copy link
Copy Markdown
Collaborator

christos68k and others added 16 commits March 6, 2025 07:35
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Co-authored-by: Joel Höner <joel@zystem.io>
Co-authored-by: Florian Lehner <florianl@users.noreply.github.com>
Co-authored-by: Tim Rühsen <tim.ruehsen@gmx.de>
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Add custom labels for native processes utilizing the PolarSignals
TLS based native custom labels library:

https://github.com/polarsignals/custom-labels

Go custom labels are auto extracted from the go env pprof labels set.

Go 1.23 and 1.24 style map and slice approaches are both supported.

Label keys are 16 bytes and values are 48 bytes.  Each sample can have
up to 10 labels which are stored in the Trace object adding 640 bytes.

Unit test added for go custom labels and an Rust based example for
the native custom labels is added.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 17, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 7 committers have signed the CLA.

✅ korniltsev
✅ gnurizen
❌ christos68k
❌ florianl
❌ RononDex
❌ athre0z
❌ rockdaboot
You have signed the CLA already but the status is still pending? Let us recheck it.

@gnurizen gnurizen closed this Mar 17, 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.

8 participants