Skip to content
New issue

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

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

Already on GitHub? # to your account

Prometheus Helm release name doesn't match #192

Open
jordifebrer opened this issue Jun 10, 2022 · 0 comments · May be fixed by #193
Open

Prometheus Helm release name doesn't match #192

jordifebrer opened this issue Jun 10, 2022 · 0 comments · May be fixed by #193

Comments

@jordifebrer
Copy link

BUG REPORT

What happened:
The release name prom on Prometheus helm installation details on this page: https://docs.litmuschaos.io/docs/integrations/prometheus/#prometheus-community-version-helm---kube-prometheus-stack-with-pod-monitor doesn't match with the provided PodMonitor release name prometheus-stack.

The service discovery never discover the podmonitor because is looking for a podmonitor with a label release: prom instead of release: prometheus-stack

What you expected to happen:

helm install prom prometheus-community/kube-prometheus-stack --namespace monitoring

should be

helm install prometheus-stack prometheus-community/kube-prometheus-stack --namespace monitoring

How to reproduce it (as minimally and precisely as possible):

  1. Run:
helm install prom prometheus-community/kube-prometheus-stack --namespace monitoring
  1. Apply the PodMonitor:
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
metadata:
  name: chaos-exporter-monitor
  namespace: monitoring
  labels:
    release: prometheus-stack
spec:
  selector:
    matchLabels:
      app: chaos-exporter
  namespaceSelector:
    matchNames:
      - litmus
  podMetricsEndpoints:
    - port: tcp
    - interval: 1s
      metricRelabelings:
        - targetLabel: instance
          replacement: 'chaos-exporter-service'

Anything else we need to know?:
I suggest using a release name prometheus-stack with helm install because is following the naming convention used in the rest of the repository instead of changing the release name of the PodMonitor to prom.

jordifebrer added a commit to jordifebrer/litmus-docs that referenced this issue Jun 10, 2022
Signed-off-by: Jordi Febrer <jordi.febrer@gmail.com>
@jordifebrer jordifebrer linked a pull request Jun 10, 2022 that will close this issue
3 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant