envoyproxy/envoy

More flexibility in choosing static vs dynamic config in integration tests

Open

#6.327 geöffnet am 20. März 2019

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
enhancementhelp wantedtech debt

Repository-Metriken

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

Beschreibung

In discussion on #6214, @alyssawilk outlined a nice redesign of the integration testing framework's interaction with configs. Here are the relevant comments:

I'm hoping by end of year the integration test framework will just let you live load any component as we can in prod

I'd like to have config booleans for static vs dynamic for all the DSes. So if they were all false, we'd get what we have today, and if they were all true we'd have what is described above, and whatever section is true has the hard-coded configuration swapped out for the discovery service config.

This issue will track that work.

Contributor Guide

Research-Richtung
Untersuchen Sie die vorgeschlagenen Booleans für statische vs. dynamische Konfiguration in Integrationstests und implementieren Sie die Änderung für einen Discovery Dienst als Proof of Concept.
Tech Stack
cpp
Domain
backend
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++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20