Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ObsUX][APM-Otel] Fix index pattern to accept otel indices, bump up version #196164

Conversation

MiriamAparicio
Copy link
Contributor

@MiriamAparicio MiriamAparicio commented Oct 14, 2024

Closes #195653

The problem

The current service definition used by the transforms uses thetraces-apm* indices and service.name as identifier fields, otel has traces-*.otel-*

Solution

Modify the definition of the index patterns to look for traces*

We can see now otel service in the entities inventory

image

fix index pattern to accept otel indices, bump up version
@MiriamAparicio MiriamAparicio added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.16.0 labels Oct 14, 2024
@MiriamAparicio MiriamAparicio requested a review from a team as a code owner October 14, 2024 15:08
@elasticmachine
Copy link
Contributor

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

Copy link
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! ❤️ I added a question but it should be fine as is, I think (I haven't tested it). Have you tested this against the e2e PoC?

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #60 / discover/esql discover esql view switch modal should not show switch modal when switching to a data view while a saved search is open

Metrics [docs]

✅ unchanged

@@ -10,14 +10,14 @@ import { BUILT_IN_ID_PREFIX } from './constants';

export const builtInServicesFromEcsEntityDefinition: EntityDefinition =
entityDefinitionSchema.parse({
version: '0.4.0',
version: '0.5.0',
id: `${BUILT_IN_ID_PREFIX}services_from_ecs_data`,
name: 'Services from ECS data',
description:
'This definition extracts service entities from common data streams by looking for the ECS field service.name',
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we mention that otel services will also be extracted from traces?

@MiriamAparicio MiriamAparicio merged commit 27548be into elastic:main Oct 14, 2024
31 checks passed
@MiriamAparicio MiriamAparicio deleted the 195653-fix-index-pattern-to-work-with-otel branch October 14, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[APM] The new service view should work with Otel native data
7 participants