Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Remove addresses on symbolized profiles #541

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

cyriltovena
Copy link
Collaborator

We realised that this create a lot of different stacktraces while just the address changes but not the binary itself.

In the future we'll introduce buildid to recognized binary change, so when a profiles is symbolized we can just get rids on the adresses.

Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this. LGTM

@cyriltovena
Copy link
Collaborator Author

Waiting to run this for a day and I'll merge

@cyriltovena
Copy link
Collaborator Author

Going to push this to next week, I still want to investigate some block.

@cyriltovena cyriltovena merged commit a7b0c34 into main Mar 2, 2023
@cyriltovena cyriltovena deleted the remove-adreses-on-symbolized branch March 2, 2023 21:33
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants