Skip to content

Conversation

@cyrille-leclerc
Copy link
Contributor

Promote more resource attributes:

  • host.name: host monitoring
  • k8s.node.name: K8s node oriented view when doing K8s monitoring
  • postgresql.* resource attributes used by otelcol/receiver/postgresql
  • database: resource attribute used by otelcol/receiver/mongodb
  • kafka.cluster.alias: resource attribute used by otelcol/receiver/kafkametrics

Promote more resource attributes:
* `host.name`: host monitoring
* `k8s.node.name`: K8s node oriented view when doing K8s monitoring
* `postgresql.*` resource attributes used by otelcol/receiver/postgresql
* `database`: resource attribute used by otelcol/receiver/mongodb
* `kafka.cluster.alias`: resource attribute used by otelcol/receiver/kafkametrics

Signed-off-by: Cyrille Le Clerc <[email protected]>
Copilot AI review requested due to automatic review settings November 27, 2025 16:46
@CLAassistant
Copy link

CLAassistant commented Nov 27, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant

This comment has been minimized.

Signed-off-by: Gregor Zeitlinger <[email protected]>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR promotes additional OpenTelemetry resource attributes to Prometheus labels in the LGTM observability stack. The changes expand monitoring capabilities for host-level metrics, Kubernetes node monitoring, and database-specific metrics from PostgreSQL, MongoDB, and Kafka receivers.

  • Adds host.name and k8s.node.name for infrastructure monitoring
  • Adds PostgreSQL-specific resource attributes (postgresql.database.name, postgresql.schema.name, postgresql.table.name, postgresql.index.name)
  • Adds database attribute for MongoDB receiver and kafka.cluster.alias for Kafka metrics
  • Adds deployment.environment for backward compatibility alongside deployment.environment.name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@zeitlinger zeitlinger 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 contribution!

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Gregor Zeitlinger <[email protected]>
Copilot AI review requested due to automatic review settings November 27, 2025 16:49
Signed-off-by: Gregor Zeitlinger <[email protected]>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zeitlinger zeitlinger enabled auto-merge (squash) November 27, 2025 16:51
@zeitlinger
Copy link
Member

@cyrille-leclerc please sign the CLA

@zeitlinger zeitlinger merged commit 2ceca57 into main Nov 28, 2025
44 checks passed
@zeitlinger zeitlinger deleted the cyrille-leclerc-patch-1 branch November 28, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants