Skip to content

Conversation

akshaymankar
Copy link
Member

@akshaymankar akshaymankar commented Nov 9, 2021

The old version of prometheus operator chart is no longer compatible with latest
K8s versions. The chart has also since moved to kube-prometheus-stack on the
prometheus-community helm chart reposistory from the helm-stable repository.

The kube-prometheus-stack helm chart also allows creating dashboard using labels
on configmaps, so now we don't have to refer to non-pinned versions of the
dashboard definitions. Instead each dashboard now becomes a configmap with a
specific label and grafana discovers it.

This commit also updates the dashboards to use new ways of addressing the pods
which came with the move from old version of prometheus-operator chart to the
kube-prometheus-stack chart. Along with these changes, the dashboards now also
use the qualified endpoints and federation endpoints to display information
about prekey claims and message sending stats.

https://wearezeta.atlassian.net/browse/FS-64

Checklist

  • The PR Title explains the impact of the change.
  • The PR description provides context as to why the change should occur and what the code contributes to that effect. This could also be a link to a JIRA ticket or a Github issue, if there is one.
  • changelog.d contains the following bits of information:
    • A file with the changelog entry in one or more suitable sub-sections. The sub-sections are marked by directories inside changelog.d.

@akshaymankar akshaymankar force-pushed the fs64/fix-wire-server-metrics branch 3 times, most recently from b72ed7f to fdeda03 Compare November 16, 2021 12:51
@akshaymankar akshaymankar changed the title Upgrade wire-server-metrics and fix dashboards charts/wire-server-metrics: Use kube-prometheus-stack Nov 16, 2021
@akshaymankar akshaymankar marked this pull request as ready for review November 16, 2021 12:54
The old version of prometheus operator chart is no longer compatible with latest
K8s versions. The chart has also since moved to kube-prometheus-stack on the
prometheus-community helm chart reposistory from the helm-stable repository.

The kube-prometheus-stack helm chart also allows creating dashboard using labels
on configmaps, so now we don't have to refer to non-pinned versions of the
dashboard definitions. Instead each dashboard now becomes a configmap with a
specific label and grafana discovers it.

This commit also updates the dashboards to use new ways of addressing the pods
which came with the move from old version of prometheus-operator chart to the
kube-prometheus-stack chart. Along with these changes, the dashboards now also
use the qualified endpoints and federation endpoints to display information
about prekey claims and message sending stats.
@akshaymankar akshaymankar force-pushed the fs64/fix-wire-server-metrics branch from fdeda03 to 5e6d38e Compare November 16, 2021 13:04
@akshaymankar akshaymankar merged commit 68278f3 into develop Nov 17, 2021
@akshaymankar akshaymankar deleted the fs64/fix-wire-server-metrics branch November 17, 2021 08:22
@smatting smatting mentioned this pull request Dec 2, 2021
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