Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(helm): yaml indentation for ServiceMonitor #1757

Merged
merged 2 commits into from
Nov 18, 2024

Conversation

nlamirault
Copy link
Contributor

@nlamirault nlamirault commented Nov 14, 2024

---
# Source: grafana-operator/templates/servicemonitor.yaml
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: release-name-grafana-operator
  namespace: default
  labels:
    helm.sh/chart: grafana-operator-v5.15.1
    app.kubernetes.io/name: grafana-operator
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "v5.15.1"
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/part-of: grafana-operator
    app.kubernetes.io/component: operator
spec:
  jobLabel: release-name
  namespaceSelector:
    matchNames:
      - default
  selector:
    matchLabels:
      app.kubernetes.io/name: grafana-operator
      app.kubernetes.io/instance: release-name
  endpoints:
    - port: metrics
      path: /metrics
      interval: 1m
      scrapeTimeout: 10s
      metricRelabelings:
        - action: replace
          replacement: my-cluster
          targetLabel: cluster
      relabelings:
        - action: replace
          replacement: my-cluster
          targetLabel: cluster

Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Copy link
Collaborator

@weisdd weisdd 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 PR! Please, take a look at my comments.

Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
@theSuess theSuess added this pull request to the merge queue Nov 18, 2024
Merged via the queue into grafana:master with commit 1ef7aab Nov 18, 2024
13 of 14 checks passed
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