solana-labs/solana

Decouple metrics submission from RUST_LOG

Open

#15.215 geöffnet am 9. Feb. 2021

Auf GitHub ansehen
 (14 Kommentare) (6 Reaktionen) (1 zugewiesene Person)Rust (3.950 Forks)batch import
good first issue

Repository-Metriken

Stars
 (12.651 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Problem

It is occasionally desired to reduce text log size, but continue submitting full metrics. With metrics level taken from RUST_LOG, this is not possible.

Relevant Code

Datapoints are pushed to the MetricsAgent based on log level here: https://github.com/solana-labs/solana/blob/d4ce59eee7e2537f02c827b6918de9af559b06bd/metrics/src/datapoint.rs#L153-L165

If the datapoint has sufficient log-level based on RUST_LOG to get submitted, then it will eventually reach here where it is both logged to file and pushed into a vector to be sumitted. https://github.com/solana-labs/solana/blob/d4ce59eee7e2537f02c827b6918de9af559b06bd/metrics/src/metrics.rs#L261-L264

Proposed Solution

Look into the feasibility of decoupling metrics from RUST_LOG

Contributor Guide

Research-Richtung
Untersuchen Sie, wie Metriken übermittelt werden, und finden Sie einen Weg, die Protokollebene von der Metrikübermittlung zu trennen. Sehen Sie sich den relevanten Code in datapoint.rs und metrics.rs an, um die aktuelle Integration zu verstehen und einen Entkopplungsmechanismus vorzuschlagen.
Tech Stack
rust
Domain
backendobservability
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustLogging
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50