Skip to content

[8.16] [APM][OTel] Fix: Pass telemetry.sdk* data when loading a dashboard (#214356)#214417

Merged
kibanamachine merged 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-214356
Mar 13, 2025
Merged

[8.16] [APM][OTel] Fix: Pass telemetry.sdk* data when loading a dashboard (#214356)#214417
kibanamachine merged 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-214356

Conversation

@kibanamachine
Copy link
Contributor

@kibanamachine kibanamachine commented Mar 13, 2025

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

@jennypavlova
[APM][OTel] Fix: Pass telemetry.sdk* data when loading a dashboard (e…
fea6d98 
…lastic#214356)

Closes elastic#214328

## Summary

This PR fixes the issue with loading an otel native dashboard

## Testing
- Run ` node scripts/synthtrace otel_edot_simple_trace.ts`
- ⚠️ As our test data has only a java service there is a small manual
step needed (it will be listed below **^**)
- Open the metrics tab for the service (currently we don't have a
dashboard):

![image](https://github.com/user-attachments/assets/e5f1461c-be6f-4d18-9185-7cb25698764b)

- **^** Go to the `dashboard_catalog.ts` and add mapping for otel native
java case:
  - first: `'otel_native-edot-java',`

![image](https://github.com/user-attachments/assets/a3463f5b-6db9-466e-95fc-3ccd44a4a016)
  - second:
      ```js
     case 'otel_native-edot-java': {
      return import(
        /* webpackChunkName: "lazyJavaOtelNativeDashboard" */
        './opentelemetry_java.json'
      );
    }
   ``

-
![image](https://github.com/user-attachments/assets/34c12147-db1e-4494-8ea6-95dffc5975d4)

- Check the same page after refresh

![image](https://github.com/user-attachments/assets/e12b7046-9868-421b-9f9e-df004f82ddfc)

(cherry picked from commit ebe7ef6)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 13, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 13, 2025 15:26
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. labels Mar 13, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 13, 2025

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@kibanamachine kibanamachine mentioned this pull request Mar 13, 2025
Merged
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 13, 2025 •
edited
Loading

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: fea6d98
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-214417-fea6d98dcbf3

Failed CI Steps

Test Failures

  • [job] [logs] Jest Integration Tests #4 / Http1BasePathProxyServer shouldRedirect it will do a redirect if it detects what looks like a stale or previously used base path

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
apm 2.5MB 2.5MB +42.0B

History

cc @jennypavlova

@kibanamachine kibanamachine merged commit a92a384 into elastic:8.16 Mar 13, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees

@jennypavlova jennypavlova

Labels

backport This PR is a backport of another PR ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kibanamachine @elasticmachine @jennypavlova