Skip to content

Conversation

@nsavoire
Copy link
Collaborator

What does this PR do?

Update to latest upstream changes.

Motivation

  • Adapt reporter to changes because of string / mapping interning
    • Strings in traceMetadata has been replaced with libpf.String.
    • CustomLabels has been replaced with []map[libpf.String]libpf.String.
    • frame.MappingFile has been replaced with frame.Mapping.
  • Adapt runner:
    • TraceHandler has been removed.
  • Adapt symbol uploader:
    • ExtractAsFile has been removed from Process interface while GetExe and
    • GetProcessMeta have been added
  • Adapt gopclntab parsing to latest upstream changes
    • SymbolMap has been removed, replace it with VisitSymbols.
  • Adapt runner:
    • TraceHandler has been removed.

@nsavoire nsavoire requested a review from a team as a code owner November 19, 2025 15:02
@nsavoire nsavoire force-pushed the nsavoire/upstream_changes branch from f845e38 to 29f6453 Compare November 20, 2025 09:10
return fmt.Errorf("failed to start map monitors: %w", err)
}

_, err := tracehandler.Start(ctx, rep, trc.TraceProcessor(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Need to check if it's worth disabling the frame cache that replaces the trace cache.

@nsavoire nsavoire force-pushed the nsavoire/upstream_changes branch 4 times, most recently from 6beff06 to a29d857 Compare November 26, 2025 17:37
@nsavoire nsavoire changed the title Update to latest upstream changes [PROF-13166] Update to latest upstream changes Nov 27, 2025
SymbolMap has been removed, replace it with VisitSymbols.
ExtractAsFile has been removed from Process interface while GetExe and
GetProcessMeta have been added
TraceHandler has been removed.
Changes because of string / mapping interning
* Strings in traceMetadata has been replaced with libpf.String.
* CustomLabels has been replaced with []map[libpf.String]libpf.String.
* frame.MappingFile has been replaced with frame.Mapping.
@nsavoire nsavoire force-pushed the nsavoire/upstream_changes branch from a29d857 to d99e9f9 Compare November 27, 2025 23:45
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