HYPERFLEET-581 - feat: Add ServiceMonitor to Helm chart for Prometheus Operator

[rafabene]

Summary

• Add ServiceMonitor template to enable automatic Prometheus Operator scraping of metrics
• Add serviceMonitor configuration section in values.yaml with configurable options:
  • enabled flag (default: false)
  • interval setting (default: 30s)
  • scrapeTimeout setting (default: 10s)
  • labels for Prometheus selector matching
  • namespace override option

Test plan

• Helm lint passes
• Template renders correctly with serviceMonitor.enabled=true
• Template renders correctly with custom namespace and labels
• Deploy to cluster with Prometheus Operator and verify metrics scraping

Resolves: https://issues.redhat.com/browse/HYPERFLEET-581

Summary by CodeRabbit

• New Features
  • Optional Prometheus Operator integration: enable ServiceMonitor-based metrics scraping with configurable enable flag, scrape interval, timeout, labels, and namespace.
• Documentation
  • Expanded Monitoring docs with Prometheus Operator Integration guide, examples for enabling ServiceMonitor, custom labels, and separate namespace deployment.

[coderabbitai]

Walkthrough

The pull request adds Prometheus Operator integration to the Helm chart by introducing a ServiceMonitor template (rendered when .Values.serviceMonitor.enabled is true), updates values.yaml with a serviceMonitor configuration block (enabled, interval, scrapeTimeout, labels, namespace), and extends documentation with a "Prometheus Operator Integration" section showing installation and configuration examples. The ServiceMonitor template sets apiVersion, kind, metadata, selector.matchLabels, optional namespace/namespaceSelector, and endpoints (port, path, interval, scrapeTimeout) based on chart values.

Sequence Diagram(s)

sequenceDiagram
    participant Dev as Developer (Helm)
    participant Helm as Helm CLI
    participant K8s as Kubernetes API
    participant PO as Prometheus Operator
    participant Svc as Target Service

    Dev->>Helm: helm install/upgrade chart (serviceMonitor.enabled=true)
    Helm->>Helm: Render templates (ServiceMonitor created)
    Helm->>K8s: Apply ServiceMonitor manifest
    K8s->>PO: Persist ServiceMonitor resource
    PO->>K8s: Discover ServiceMonitor.selector -> find matching Service
    PO->>Svc: Scrape /metrics on configured port/path/interval

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a ServiceMonitor resource to the Helm chart for Prometheus Operator integration, which is the primary objective of the pull request.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

• ✨ 136.113.208.247/32 (new)
• 34.170.211.100/32
• 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@charts/templates/servicemonitor.yaml`:
- Line 1: Exclude Helm chart templates from yamllint by updating your CI
yamllint configuration to ignore the charts/templates directory (so directives
like "{{- if .Values.serviceMonitor.enabled }}" won't be parsed by yamllint),
and while editing charts/templates/servicemonitor.yaml also fix the brace
spacing complaint by normalizing template tag whitespace (e.g. remove extra
spaces inside mustache tags so occurrences of "{{ .Values.xxx }}" become
"{{.Values.xxx}}" or follow your project's template whitespace convention), then
run yamllint to confirm no remaining errors.

[rh-amarin]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rh-amarin

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [rh-amarin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment