Skip to content

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

Merged
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-214356
Mar 13, 2025
Merged

[8.x] [APM][OTel] Fix: Pass telemetry.sdk* data when loading a dashboard (#214356)#214420
kibanamachine merged 2 commits intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-214356

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…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:27
@botelastic botelastic bot added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Mar 13, 2025
@elasticmachine
Copy link
Contributor

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

@kibanamachine kibanamachine merged commit 5aeab40 into elastic:8.x Mar 13, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] x-pack/test/alerting_api_integration/spaces_only/tests/alerting/group1/config.ts / Alerting eventLog in space default should generate expected events for normal operation

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.6MB 2.6MB +42.0B

cc @jennypavlova

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 Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants