[Profiling] Add colors for new frame types (process name, thread name, root) #204977
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rockdaboot
added
release_note:enhancement
release_note:skip
|
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
botelastic
bot
added
the
ci:project-deploy-observability
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
inge4pres
approved these changes
Dec 23, 2024
MiriamAparicio
approved these changes
Dec 24, 2024
rockdaboot
force-pushed
the
flamegraph-executable-name
branch
2 times, most recently
from
d0a4cf4 to
1758e7a
Compare
rockdaboot
force-pushed
the
flamegraph-executable-name
branch
from
d9acea6 to
14391a0
Compare
rockdaboot
changed the title
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
cc @rockdaboot |
crespocarlos
pushed a commit
to crespocarlos/kibana
that referenced
this pull request
Jan 8, 2025
…, root) (elastic#204977) ## Summary This PR is a pre-requisite for adding aggregation by process name and by thread name to the Universal Profiling flamegraph view. It adds three artificial node types to the flamegraph including color codes. As a side-effect, the root node now has its own color code. Previously, it (accidentally) used the color code of "unknown" type frames. The PR is backwards compatible, so it doesn't change anything in the UI when connecting with current Elasticsearch. As soon as [the PR for ES](elastic/elasticsearch#119115) is merged, the new aggregations show up.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR is a pre-requisite for adding aggregation by process name and by thread name to the Universal Profiling flamegraph view.
It adds three artificial node types to the flamegraph including color codes.
As a side-effect, the root node now has its own color code. Previously, it (accidentally) used the color code of "unknown" type frames.
The PR is backwards compatible, so it doesn't change anything in the UI when connecting with current Elasticsearch.
As soon as the PR for ES is merged, the new aggregations show up.