Skip to content

Add opentelemetry category to OTEL integrations - #15475

Merged
5 commits merged into
mainfrom
unknown repository
Oct 8, 2025
Merged

Add opentelemetry category to OTEL integrations#15475
5 commits merged into
mainfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Sep 26, 2025 •
edited by ghost
Loading

Copy link
Copy Markdown

Overview

This PR adds opentelemetry category to all OTEL integrations.

  • apache_otel
  • docker_otel
  • iis_otel
  • kubernetes_otel
  • mysql_otel
  • nginx_ingress_controller_otel
  • nginx_otel
  • postgresql_otel
  • system_otel

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

Closes: https://github.com/elastic/obs-integration-team/issues/631

Screenshots
@ghost
ghost requested review from ishleenk17 and mykola-elastic September 26, 2025 09:37
@ghost ghost self-assigned this Sep 26, 2025
@ghost
ghost marked this pull request as ready for review September 26, 2025 09:59
@ghost
ghost requested review from a team as code owners September 26, 2025 09:59
@mykola-elastic

mykola-elastic commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

Shouldn't we update manifest.yml?

  kibana:
    version: "^9.2.0"
@elastic-sonarqube

elastic-sonarqube Bot commented Sep 26, 2025

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube
@andrewkroh andrewkroh added Integration:system_otel System OpenTelemetry Assets Integration:apache_otel Apache OpenTelemetry Assets Integration:iis_otel IIS OpenTelemetry assets Integration:postgresql_otel PostgreSQL OpenTelemetry Assets Integration:kubernetes_otel Kubernetes OpenTelemetry Assets Integration:nginx_otel NGINX OpenTelemetry Assets Integration:mysql_otel MySQL OpenTelemetry Assets Integration:docker_otel Docker OpenTelemetry Assets Integration:nginx_ingress_controller_otel Nginx Ingress Controller OpenTelemetry Logs Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Sep 26, 2025
@ishleenk17

ishleenk17 commented Oct 8, 2025

Copy link
Copy Markdown
Member

@gpop63 @mykola-elastic : Can one of you please add the screenshot of how this would look in 9.2 categorization
@ishleenk17

ishleenk17 commented Oct 8, 2025

Copy link
Copy Markdown
Member

Shouldn't we update manifest.yml?

  kibana:
    version: "^9.2.0"

I think without this change also, if customer is on 9.2 stack they will be able to see it under the OTEL category. We need not upgrade this ? Does this impact the categorisation ?
@ghost

ghost commented Oct 8, 2025

Copy link
Copy Markdown
Author

@gpop63 @mykola-elastic : Can one of you please add the screenshot of how this would look in 9.2 categorization

image

I think without this change also, if customer is on 9.2 stack they will be able to see it under the OTEL category. We need not upgrade this ? Does this impact the categorisation ?

https://github.com/elastic/obs-integration-team/issues/584#issuecomment-3312062787

It's mentioned in that comment that kibana version must be 9.2.0.
@ishleenk17

ishleenk17 commented Oct 8, 2025

Copy link
Copy Markdown
Member

It's mentioned in that comment that kibana version must be 9.2.0.

The change would work on 9.2, but we need not upgrade the version. As anyone on 9.2 would start seeing the category.
@andrewkroh andrewkroh removed Integration:kubernetes_otel Kubernetes OpenTelemetry Assets Integration:docker_otel Docker OpenTelemetry Assets Integration:system_otel System OpenTelemetry Assets labels Oct 8, 2025
@gizas

gizas commented Oct 8, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Should we also add the docker_otel and kubernetes_otel in this PR? Or is there going to be a follow up PR for the above? If yes please update also the description

Sorry missed your commit
@ishleenk17

ishleenk17 commented Oct 8, 2025

Copy link
Copy Markdown
Member

Should we also add the docker_otel and kubernetes_otel in this PR? Or is there going to be a follow up PR for the above? If yes please update also the description

Sorry missed your commit

Yes @gizas , will have another PR for the remaining packages. Wanted to do this to have quicker approval process.
@gpop63 : Could you please update the description and create another PR with the remaining ones.
@ghost
ghost enabled auto-merge (squash) October 8, 2025 14:56
@ghost
ghost merged commit b375d4c into elastic:main Oct 8, 2025
7 checks passed
@elasticmachine

elasticmachine commented Oct 8, 2025

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @gpop63
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package apache_otel - 0.1.1 containing this change is available at https://epr.elastic.co/package/apache_otel/0.1.1/
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package iis_otel - 0.1.2 containing this change is available at https://epr.elastic.co/package/iis_otel/0.1.2/
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package mysql_otel - 0.1.2 containing this change is available at https://epr.elastic.co/package/mysql_otel/0.1.2/
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package nginx_ingress_controller_otel - 0.2.2 containing this change is available at https://epr.elastic.co/package/nginx_ingress_controller_otel/0.2.2/
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package nginx_otel - 0.1.1 containing this change is available at https://epr.elastic.co/package/nginx_otel/0.1.1/
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Package postgresql_otel - 0.1.1 containing this change is available at https://epr.elastic.co/package/postgresql_otel/0.1.1/
@ghost ghost mentioned this pull request Oct 9, 2025
Merged
8 tasks
agithomas pushed a commit to agithomas/integrations that referenced this pull request Oct 30, 2025
Gabriel Pop @agithomas
Add opentelemetry category to OTEL integrations (elastic#15475)
4bfa9d6 
* add opentelemetry category to otel pkgs

* fix pr ids

* bump kibana version

* remove packages not owned by infraobs

* revert kibana version changes
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
Gabriel Pop
Add opentelemetry category to OTEL integrations (elastic#15475)
c3e030f 
* add opentelemetry category to otel pkgs

* fix pr ids

* bump kibana version

* remove packages not owned by infraobs

* revert kibana version changes
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:apache_otel Apache OpenTelemetry Assets Integration:iis_otel IIS OpenTelemetry assets Integration:mysql_otel MySQL OpenTelemetry Assets Integration:nginx_ingress_controller_otel Nginx Ingress Controller OpenTelemetry Logs Integration:nginx_otel NGINX OpenTelemetry Assets Integration:postgresql_otel PostgreSQL OpenTelemetry Assets Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

7 participants

@mykola-elastic @ishleenk17 @gizas @elasticmachine @Linu-Elias @andrewkroh @mrg-elastic