envoyproxy/envoy

stats: retain Scopes for some amount of time after the last reference to them is dropped, to avoid leaking data when xds removes a scope

Open

#23.619 geöffnet am 21. Okt. 2022

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/statsenhancementhelp wantedno stalebot

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

Title: When an xDS update removes a cluster, any stats that get updated after the most recent sink or scrape will get dropped when the ScopeSharedPtr held in the cluster gets dropped.

Description:

To avoid this we should delay the destruction of the Scope for some time interval, perhaps a few minutes. We must be careful to accelerate the Scope destruction when the stats store gets destructed (server exit, or unit-test completion).

Contributor Guide

Research-Richtung
Implementieren Sie die verzögerte Zerstörung von ScopeSharedPtr im Stats Modul. Fügen Sie einen Timer hinzu, um Scopes für ein konfigurierbares Intervall zu behalten, nachdem die letzte Referenz gelöscht wurde. Stellen Sie eine beschleunigte Zerstörung beim Aufruf des Destruktors des Statistikspeichers sicher.
Tech Stack
cpp
Domain
backendobservability
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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.
Meist klar
Voraussetzungen
C++Envoystats framework
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40