yugabyte/yugabyte-db

[platform] Enhance metrics that use sum without to also exclude namespace_name

Open

#9 759 ouverte le 17 août 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (2 assignés)C (1 003 forks)batch import
area/platformgood first issue

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

Description

We have a problem with Platform metrics, related to all the aggregations where we normally exclude tables (ie: sum without (table_id, table_name)), due to now also adding namespace_name into the Prometheus KV labels see https://yugabyte.slack.com/archives/C01HD5HPK8A/p1629237388110200?thread_ts=1628874648.049900&cid=C01HD5HPK8A for context

TL;DR: we probably need to augment all metrics.yml entries which use sum without, to also exclude namespace_name I think the source of this is this diff from a long time back: https://phabricator.dev.yugabyte.com/D9182

Guide contributeur

Direction de recherche
Identifiez tous les fichiers metrics.yml dans le dépôt qui contiennent des clauses 'sum without'. Pour chaque entrée de ce type, ajoutez 'namespace name' à la liste des labels exclus. Consultez le diff historique D9182 (référencé dans l'issue) pour le contexte sur l'ajout précédent de namespace name. Consultez également le fil de discussion Slack lié pour des détails supplémentaires sur le problème.
Stack technique
Aucun
Domaine
backenddatabaseobservability
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPrometheus
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70