Skip to content

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

Merged
kibanamachine merged 2 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-214356
Mar 13, 2025
Merged

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

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.18:

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 deec2c7 into elastic:8.18 Mar 13, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #8 / Policy Advanced Settings section should expand and collapse section when button is clicked

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

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