envoyproxy/envoy

Refactor outlier detection configuration fields

Open

#8.219 geöffnet am 11. Sept. 2019

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
api/v3help wanted

Repository-Metriken

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

Beschreibung

Title: Refactor outlier detection config message fields

Description: As discussed in https://github.com/envoyproxy/envoy/pull/8130#discussion_r320832679, the outlier detection configuration fields are a bit messy and could use a cleanup. The main work here is to de-dup all of the options into a common message around enabled, local origin split, etc. and then hopefully just have a single field per actual ejection type. This would avoid an explosion of almost identical fields for each different ejection type.

Contributor Guide

Research-Richtung
Studieren Sie die vorhandenen Konfigurationen zur Ausreißererkennung im Envoy Quellcode, wobei der Schwerpunkt auf der Diskussion in PR #8130 liegt. Verstehen Sie die Duplikation zwischen den Feldern für verschiedene Auswurfarten und schlagen Sie eine einheitliche Struktur mit gemeinsamen Feldern wie `enabled`, `local origin split` usw. vor. Planen Sie, eine gemeinsame Nachricht zu erstellen und dann ein einzelnes Feld pro Auswurfart zu haben.
Tech Stack
cpp
Domain
backendinfrastructure
Issue Type
Refactoring
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++Envoy architecture
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50