envoyproxy/envoy

adaptive concurrency: Allow specification of runtime names

Open

#8.638 geöffnet am 17. Okt. 2019

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
enhancementhelp wanted

Repository-Metriken

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

Beschreibung

As a prerequisite to supporting both egress and ingress adaptive concurrency, it will be necessary to specify the runtime configuration names to avoid conflicts. Before declaring the filter production ready, the adaptive concurrency configuration parameters should be replaced by their runtime counterparts.

To be more specific for anyone who gets to this before I do, we want to change any parameters in the adaptive_concurrency.proto that correspond to any of the runtime options that override them. For example, changing parameters in the proto from Uint32 to RuntimeUInt32.

Contributor Guide

Research-Richtung
Studieren Sie die Datei adaptive concurrency.proto und die entsprechenden Laufzeitoptionen. Identifizieren Sie Parameter mit Laufzeitüberschreibungen (z. B. minRtt, sampleAggregatePercentile) und ändern Sie deren Typen von UInt32 in RuntimeUInt32. Aktualisieren Sie die Filterimplementierung, um bei Verfügbarkeit Laufzeitwerte zu lesen.
Tech Stack
cpp
Domain
backendnetworking
Issue Type
Funktion
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++ProtobufEnvoy internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30