envoyproxy/envoy

RTDS snapshot should be pinned to each stream

Open

#10.731 geöffnet am 9. Apr. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/runtimehelp wanted

Repository-Metriken

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

Beschreibung

Title: we should keep runtime's consistency during a request's lifecycle.

Description: runtime layers should be consistent during processing of a request.

Repro steps:

a bug scenario: filter A reads version 1 of runtime snapshot, filter A stopIteration, waits for a some callback result, say on-demand-rds a new Runtime snapshot (version 2) is pushed by RTDS, filter A resume the filter iteration, and filter B now sees version 2 when it reads from loader.runtime().snapshot()

IMO, We should latch the runtime snapshot to a stream during its creation time.

Contributor Guide

Research-Richtung
Untersuchen Sie, wie Laufzeit Snapshots in Envoy erstellt und verwendet werden, und ändern Sie den Code, um den Snapshot bei der Stream Erstellung an jeden Stream zu binden.
Tech Stack
Keine
Domain
backend
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.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55