Skip to content

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

Merged
herrBez merged 3 commits intoelastic:mainfrom
herrBez:fix-nginx-host-field
Nov 4, 2025
Merged

[NGINX][OTEL]Use OpenTelemetry host.name instead of host.hostname in Dashboard control#15851
herrBez merged 3 commits intoelastic:mainfrom
herrBez:fix-nginx-host-field

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

Reviewers

@ishleenk17 ishleenk17 ishleenk17 approved these changes

@stefans-elastic stefans-elastic stefans-elastic approved these changes

Assignees

No one assigned

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]

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

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