Skip to content

[Fleet] Otel UI add health and implement it in OTelComponentDetail#267292

Merged
nchaulet merged 6 commits intomainfrom
feature-otel-component-details
May 5, 2026
Merged

[Fleet] Otel UI add health and implement it in OTelComponentDetail#267292
nchaulet merged 6 commits intomainfrom
feature-otel-component-details

Conversation

@nchaulet
Copy link
Copy Markdown
Member

@nchaulet nchaulet commented May 1, 2026

Summary

Resolve https://github.com/elastic/ingest-dev/issues/7070

That PR implement health reporting on the graph view in otel UI and in OTelComponentDetail, while working on this I found a few bugs that I fixed as part of that PR:

  • I am now building the graph with the node role, to avoid issue with component with same id
  • I fixed the layout for the OTelComponentDetail
  • I fixed the selection so the pipeline is selectable when viewing multiple pipeline.

UI Changes

Screenshot 2026-05-01 at 2 54 38 PM Screenshot 2026-05-01 at 3 27 25 PM Screenshot 2026-05-01 at 3 27 32 PM

Tests

What to test? you can try to add multiple Opamp agents with different config (claude is good for generating them) to fleet-server and verify it's correctly rendered

@nchaulet nchaulet changed the title [Fleet] Implement OTelComponentDetail [Fleet] Otel UI add health and implement it in OTelComponentDetail May 1, 2026
@nchaulet nchaulet force-pushed the feature-otel-component-details branch from 00298d1 to 23560d8 Compare May 1, 2026 19:05
@@ -0,0 +1,44 @@
/*
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved this to his own module

@nchaulet nchaulet added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Fleet Team label for Observability Data Collection Fleet team labels May 1, 2026
@nchaulet nchaulet marked this pull request as ready for review May 4, 2026 13:22
@nchaulet nchaulet requested a review from a team as a code owner May 4, 2026 13:22
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/fleet (Team:Fleet)

@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented May 4, 2026

Approvability

Verdict: Needs human review

1 blocking correctness issue found. This PR introduces new user-facing features (health status display, Health tab, clickable pipeline groups) and all changed files are owned by @elastic/fleet rather than the author. Additionally, there's an unresolved high-severity finding about a useEffect dependency bug that could cause health indicators to disappear.

You can customize Macroscope's approvability policy. Learn more.

@nchaulet
Copy link
Copy Markdown
Member Author

nchaulet commented May 4, 2026

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
fleet 2116 2120 +4

Async chunks

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

id before after diff
fleet 2.7MB 2.7MB +4.7KB

@jen-huang jen-huang self-requested a review May 4, 2026 21:25
Copy link
Copy Markdown
Contributor

@jen-huang jen-huang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, LGTM 🚀

@nchaulet nchaulet merged commit 2ade153 into main May 5, 2026
41 checks passed
@nchaulet nchaulet deleted the feature-otel-component-details branch May 5, 2026 01:22
mbondyra added a commit to mbondyra/kibana that referenced this pull request May 5, 2026
…ilder_new_vis_attachment

* commit '6fd683609eb6dee81f242f8ff6951edbe3bfd66c': (226 commits)
  Remove Model Author group-by option from external inference endpoints (elastic#264761)
  [Streams][Streamlang] Align ES|QL condition transpiler with Painless on null propagation (elastic#264751)
  chore(axios,workflows-eng): remove axios from workflows connector utils (elastic#267512)
  [failed-test-reporter] avoid opening issues for scout env failures (elastic#267649)
  [kbn-api-contracts] Detect request-body additionalProperties:false tightening (elastic#267546)
  [main] Sync bundled packages with Package Storage (elastic#267644)
  Centralize phase colors and descriptions (elastic#266680)
  [Unified Waterfall] Add "Scroll to origin" button  (elastic#266594)
  [APM] Add alert and SLO badges to the service map embeddable (elastic#266360)
  [CI] Speed up telemetry_check by pre-filtering to collector files (elastic#265978)
  [Discover] Address flaky large CSV test (elastic#266642)
  avoid passing unrelated props within integration card icon component conditional render (elastic#266569)
  [Cases][Templates] Extend cases search by template field label (elastic#266414)
  [Background search] Migrate custom SplitButton to EuiSplitButton (elastic#267447)
  [i18n] Report translation coverage during integrate (elastic#264124)
  [api-docs] 2026-05-05 Daily api_docs build (elastic#267639)
  [Scout] Update test config manifests (elastic#267636)
  [content list] Add saved object provider services (elastic#266428)
  [Fleet] Otel UI add health and implement it in OTelComponentDetail (elastic#267292)
  Update dependency msw to v2.13.4 (main) (elastic#266770)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants