Skip to content

[Fleet] Add OpAMP mappings for UI search#261957

Merged
juliaElastic merged 2 commits intoelastic:mainfrom
juliaElastic:opamp-metadata-mappings
Apr 9, 2026
Merged

[Fleet] Add OpAMP mappings for UI search#261957
juliaElastic merged 2 commits intoelastic:mainfrom
juliaElastic:opamp-metadata-mappings

Conversation

@juliaElastic
Copy link
Copy Markdown
Contributor

@juliaElastic juliaElastic commented Apr 8, 2026

Summary

Relates https://github.com/elastic/ingest-dev/issues/7062

Added new mappings in elastic/elasticsearch#145824 to kibana so the UI search works on them.

To verify:

Start ES from source with so the new mappings apply (switch to the branch of the ES PR):

yarn es source --license trial -E xpack.security.authc.api_key.enabled=true -E xpack.security.authc.token.enabled=true  --source-path=/Users/juliabardi/elasticsearch  -E path.data=/tmp/es-data -E xpack.ml.enabled=false -E http.host=0.0.0.0

Add a collector (follow steps in elastic/fleet-server#6769) and test that the UI search works on the new fields:

image image image image

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, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • 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.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

@juliaElastic juliaElastic added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Apr 8, 2026
@juliaElastic juliaElastic marked this pull request as ready for review April 8, 2026 11:40
@juliaElastic juliaElastic requested a review from a team as a code owner April 8, 2026 11:40
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 8, 2026

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #96 / serverless security UI Security ML Data frame analytics jobs list page navigation renders job list and finds created job
  • [job] [logs] FTR Configs #100 / task_manager migrations 8.5.0 migrates active tasks to set enabled to true

Metrics [docs]

Page load bundle

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

id before after diff
fleet 202.1KB 202.6KB +540.0B

History

@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Apr 9, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp bot commented Apr 9, 2026

Approvability

Verdict: Needs human review

This PR adds new Elasticsearch field mappings for OpAMP search functionality in Fleet. While the changes are purely additive schema definitions, the modified file is owned by @elastic/fleet and the author is not a designated owner. Schema/mapping changes to Fleet's agent index should be reviewed by the owning team.

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

Copy link
Copy Markdown
Member

@jillguyonnet jillguyonnet left a comment

Choose a reason for hiding this comment

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

I'm struggling to run the collector, but I'm sure it's an issue with my config. Code looks good 👍

@juliaElastic juliaElastic merged commit 00e470d into elastic:main Apr 9, 2026
23 checks passed
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.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants