postfinance / kubenurse

Kubernetes network monitoring
MIT License
407 stars 39 forks source link

Grafana Dashboard #35

Closed liyongxian closed 2 years ago

liyongxian commented 2 years ago

Doc has no Json of Grafana Dashboard For kubenurse

eli-halych commented 2 years ago

I was also looking for it and couldn't find it in the repo.

eli-halych commented 2 years ago

@djboris9 can you help us with that, please?

djboris9 commented 2 years ago

Sorry @liyongxian and @eli-halych I'm currently busy and had missed the issues.

Can you please try out this dashboard: https://github.com/djboris9/kubenurse/blob/d6b2d260ce150d69c5e39240dd3d4ea3e0e61232/doc/grafana-kubenurse.json

If it works, I will merge it into this repository and probably add it to helm too.

ywllongge commented 2 years ago

Thanks@[djboris9]

eli-halych commented 2 years ago

@djboris9 It worked! I had to change a few things in the helm chart though. I added ServiceMonitor:

apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: {{ include "name" . | quote }}
  namespace: {{ .Values.namespace }}
  labels:
    release: {{ .Values.release }}
    k8s-app: {{ include "name" . | quote }}
{{ include "common-labels" . | indent 4 }}
spec:
  jobLabel: k8s-app
  endpoints:
  - port: {{ .Values.service.name }}
    interval: 30s
    scheme: http
  selector:
    matchLabels:
     app: {{ include "name" . | quote }}
  namespaceSelector:
    matchNames:
    - {{ .Values.namespace }}

and add release: prometheus label to ServiceMonitor and DaemonSet resounces. (release: {{ .Values.release }} in this case and the value itself is placed in values.yaml)

Source: https://programmer.ink/think/cluster-network-monitoring-using-kubenurse.html

djboris9 commented 2 years ago

Hi @eli-halych

Thanks for the feedback! I just merged the dashboard into the master branch with #40 .

I will open a new issue to incorporate and test the ServiceMonitor into the helm chart as you've written and close this issue.

Is this ok for you?

/cc @liyongxian

djboris9 commented 2 years ago

PR #42 implements the ServiceMonitor. Therefore I will close this issue. Feel free to provide feedback or reopen it.