Skip to content

[NGINX][OTEL]Use OpenTelemetry host.name instead of host.hostname in Dashboard control #15851

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
Merged
herrBez merged 3 commits into from
Nov 4, 2025
Merged

[NGINX][OTEL]Use OpenTelemetry host.name instead of host.hostname in Dashboard control #15851

herrBez merged 3 commits into from
Nov 4, 2025

Conversation

@herrBez
Copy link
Contributor

@herrBez herrBez commented Nov 3, 2025

The current version of the dashboard uses host.hostname, which is ECS instead of OTEL host.name as listed here: https://www.elastic.co/docs/reference/ecs/ecs-otel-alignment-details.

Proposed commit message

[NGINX][OTEL] Use OTEL host.name instead of ECS host.hostname in Dashboard Control

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

How to test this PR locally

Related issues

Screenshots
@herrBez herrBez requested a review from a team as a code owner November 3, 2025 16:17
@andrewkroh andrewkroh added Integration:nginx_otel NGINX OpenTelemetry Assets dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Nov 3, 2025
@herrBez herrBez changed the title Use OpenTelemetry host.name instead of host.hostname in Dashboard control [NGINX][OTEL]Use OpenTelemetry host.name instead of host.hostname in Dashboard control Nov 4, 2025
stefans-elastic
stefans-elastic approved these changes Nov 4, 2025
View reviewed changes
Copy link
Contributor

@stefans-elastic stefans-elastic left a comment

Choose a reason for hiding this comment

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

LGTM
@elasticmachine
Copy link

elasticmachine commented Nov 4, 2025

💚 Build Succeeded

History
@ishleenk17
Copy link
Member

ishleenk17 commented Nov 4, 2025

I noticed this issue just yesterday as well.
Thanks for fixing it.
@herrBez herrBez merged commit a8fcb8e into elastic:main Nov 4, 2025
7 checks passed
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Nov 4, 2025

Package nginx_otel - 0.2.1 containing this change is available at https://epr.elastic.co/package/nginx_otel/0.2.1/
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
@herrBez
[NGINX][OTEL]Use OpenTelemetry host.name instead of host.hostname in …
f0895d5 
…Dashboard control (elastic#15851) * Use OpenTelemetry host.name instead of host.hostname in Dashboard control * Update changelog to use right PR
@andrewkroh andrewkroh added the bugfix Pull request that fixes a bug issue label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes a bug issue dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Integration:nginx_otel NGINX OpenTelemetry Assets Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

5 participants

@herrBez @elasticmachine @ishleenk17 @stefans-elastic @andrewkroh