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.

Sign up for GitHub

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

Jump to bottom

[8.16] [APM] Fix: Add tracing sample missing fields in the overview (#206932) #206983

Open
wants to merge 1 commit into
base: 8.16
Choose a base branch
from
Open

[8.16] [APM] Fix: Add tracing sample missing fields in the overview (#206932) #206983

wants to merge 1 commit into from
+27 −1

Conversation

jennypavlova
Copy link
Member

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

@jennypavlova
[APM] Fix: Add tracing sample missing fields in the overview (elastic…
d2164d7 
…#206932)

Closes elastic#200474

## Summary

This PR fixes the issue with tracing sample missing URL/Status Code/User
Agent fields in the overview

## Testing
- Open the APM UI and find APM traces that contains `url.full` /
`transaction.page.url`, `http.request.method` and
`http.response.status_code`
- One should be ingested using an otel collector the other should use an
apm-server
- if using oblt cluster you can check transactions from `loadgenerator`
and `opbeans-python` for example
    - check the trace summary:
        - Otel:

![image](https://github.com/user-attachments/assets/871172b6-8307-4aa2-844e-73a8405da746)

        - APM server:

![image](https://github.com/user-attachments/assets/ef233cf4-0fbb-49c2-8f09-d4299a34ec8c)

(cherry picked from commit 1d493c0)

# Conflicts:
#	x-pack/plugins/spaces/public/config.ts
@jennypavlova jennypavlova enabled auto-merge (squash) January 16, 2025 16:59
@jennypavlova jennypavlova mentioned this pull request Jan 16, 2025
Merged
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jan 16, 2025
@elasticmachine
Copy link
Contributor

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

@github-actions GitHub Actions
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 16, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bluefinger Bluefinger Bluefinger approved these changes

@kibanamachine kibanamachine Awaiting requested review from kibanamachine kibanamachine is a code owner

Assignees
No one assigned
Labels
backport ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jennypavlova @elasticmachine @Bluefinger