Skip to content

LuaJIT language unwinder#419

Closed
gnurizen wants to merge 4 commits into
open-telemetry:mainfrom
parca-dev:luajit-up
Closed

LuaJIT language unwinder#419
gnurizen wants to merge 4 commits into
open-telemetry:mainfrom
parca-dev:luajit-up

Conversation

@gnurizen
Copy link
Copy Markdown
Contributor

  • Go custom labels from pprof labels
  • Use gotoolchain instead of github matrix strategy
  • Native custom labels
  • LuaJIT unwinding support for openresty 1.15+

gnurizen and others added 3 commits March 18, 2025 18:52
Add support for a collection of 10 labels that can be attached to
every sample.  Process the go pprof labels into these labels.

Test that this works for Go 1.23 and Go 1.24 with new kernel tests.

Co-authored-by: umanwizard <brennan@umanwizard.com>
If a process links against the custom labels library:

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

Or is statically linked and has the custom_labels_abi_version exported
symbol then we will try to read custom labels out of the TLS name
'custom_labels_current_set'.

Co-authored-by: umanwizard brennan@umanwizard.com
@gnurizen gnurizen changed the title luajit up LuaJIT language unwinder Mar 24, 2025
@fabled
Copy link
Copy Markdown
Contributor

fabled commented Jul 9, 2025

Seems this needs a rebase. And contains multiple features in single PR. Perhaps you can rebase and make this LuaJIT -only PR? Please ping me if when its ready for a pre-review. Thanks!

@florianl
Copy link
Copy Markdown
Member

Closing in favor of #1236. If this is still relevant, please rebase, update and reopen this PR @gnurizen.

@florianl florianl closed this Mar 18, 2026
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.

3 participants