envoyproxy/envoy

Avoid lossy LRS during cluster updates

Open

#12.823 geöffnet am 26. Aug. 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/load reportingbughelp wanted

Repository-Metriken

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

Beschreibung

From https://github.com/grpc/proposal/pull/200, when a cluster update for X is received and the new cluster is active after warming, the old X cluster may still be draining for an indefinite period if it has active streams. Only the stats for new requests/streams on X will participate in LRS reporting at this point. This seems to be potentially misleading when reporting load only for the new X.

There are two scenarios to consider:

  1. When the EDS service name changes. I think in this case we should generate two LRS reports, one for old cluster X (including the older EDS service name) and another for new cluster X (including the new EDS service name).
  2. When the EDS service name is the same. I think a unified LRS report should be created aggregating old/new cluster X.

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung des LRS (Load Reporting Service) in Envoy, insbesondere wie Cluster Updates die Lastmeldung beeinflussen. Konzentrieren Sie sich auf die Logik, die das Drainting von Clustern und die Aktivierung neuer Cluster behandelt. Betrachten Sie den Code, der für die Generierung von LRS Berichten während Cluster Updates verantwortlich ist, und verstehen Sie, wie Änderungen des EDS Dienstnamens behandelt werden. Verfolgen Sie den Codepfad vom CDS Update bis zur LRS Berichtsgenerierung.
Tech Stack
cpp
Domain
networking
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.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25