Skip to content

Conversation

ronaldosaheki
Copy link

Currently DCGM exports metrics to port 9400, but prometheus default is 9090 and is missing annotation:

apiVersion: v1
kind: Service
metadata:
  annotations:
    prometheus.io/scrape: "true"
  creationTimestamp: "2025-07-25T07:04:08Z"
  labels:
    app: nvidia-dcgm-exporter
  name: nvidia-dcgm-exporter

Changes:

  • Introduced an Annotations field in DCGMExporterServiceConfig to allow custom annotations for the DCGM Exporter service.
  • Modified the service transformation logic to apply specified annotations to the DCGM Exporter service.
  • Enhanced the CRD definition to include the new annotations property.
  • Added unit tests to verify the correct application of annotations in various scenarios.

Copy link

copy-pr-bot bot commented Aug 27, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

- Introduced an `Annotations` field in `DCGMExporterServiceConfig` to allow custom annotations for the DCGM Exporter service.
- Updated the deepcopy function to handle the new `Annotations` field.
- Modified the service transformation logic to apply specified annotations to the DCGM Exporter service.
- Enhanced the CRD definition to include the new annotations property.
- Added unit tests to verify the correct application of annotations in various scenarios.

Signed-off-by: Ronaldo Saheki <[email protected]>
@ronaldosaheki ronaldosaheki force-pushed the add-dcgemexporter-service-annotations branch from 0a50b0f to 67c592f Compare August 27, 2025 09:59
- Enhanced the CRD definitions for both `nvidia.com_clusterpolicies.yaml` files to include an `annotations` field.
- This new field allows users to specify custom annotations for the DCGM Exporter service.

Signed-off-by: Ronaldo Saheki <[email protected]>
@ronaldosaheki ronaldosaheki changed the title DRAFT: Add annotations support to DCGM Exporter service configuration for add prometheus scrape port. Add annotations support to DCGM Exporter service configuration for add prometheus scrape port. Aug 27, 2025
@ronaldosaheki ronaldosaheki changed the title Add annotations support to DCGM Exporter service configuration for add prometheus scrape port. Add annotations support to DCGM Exporter service configuration Aug 27, 2025
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.

2 participants