Skip to content

[8.19] [Profiling] Add FrameType and color for Go (#215697)#220511

Merged
cauemarcondes merged 3 commits intoelastic:8.19from
cauemarcondes:backport/8.19/pr-215697
May 8, 2025
Merged

[8.19] [Profiling] Add FrameType and color for Go (#215697)#220511
cauemarcondes merged 3 commits intoelastic:8.19from
cauemarcondes:backport/8.19/pr-215697

Conversation

@cauemarcondes
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

## Summary

OTel Semantic Conventions
[defines](open-telemetry/semantic-conventions#2003)
a type for Go and OTel eBPF profiler is about to start with pushing Go
frames (either with
open-telemetry/opentelemetry-ebpf-profiler#409
or
open-telemetry/opentelemetry-ebpf-profiler#408)

FYI: @elastic/ingest-otel-data

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [ ] ~~Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)~~
not relevant
- [ ]
~~[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials~~ not
relevant
- [ ] ~~[Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios~~ not relevant
- [ ] ~~If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)~~
not relevant
- [ ] ~~This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.~~
not relevant
- [ ] ~~[Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed~~ not relevant
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- `release_note:skip`

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
(cherry picked from commit 7c4af05)
@cauemarcondes cauemarcondes added the backport This PR is a backport of another PR label May 8, 2025
@cauemarcondes cauemarcondes enabled auto-merge (squash) May 8, 2025 13:14
auto-merge was automatically disabled May 8, 2025 15:14

Pull Request is not mergeable

@cauemarcondes cauemarcondes enabled auto-merge (squash) May 8, 2025 17:54
@cauemarcondes cauemarcondes merged commit 64d4bb3 into elastic:8.19 May 8, 2025
8 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
profiling 373.1KB 373.1KB +34.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
profiling 18.1KB 18.1KB +28.0B

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants