envoyproxy/envoy

Make envoyproxy/envoy the source-of-truth for envoy-filter-example

Open

#5.054 geöffnet am 15. Nov. 2018

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/buildhelp wantedtech debt

Repository-Metriken

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

Beschreibung

As discussed in https://github.com/envoyproxy/envoy/pull/5031#issuecomment-438757841, we have a multiple commit dance needed for changes that break the filter API today, where both envoyproxy/envoy and envoyproxy/envoy-filter-example need multiple commits. One way to avoid this would be:

  1. We move envoy-filter-example into the main Envoy repo, using a local repository in the WORKSPACE to point the example at Envoy source.
  2. We export to envoy-filter-example as a read-only mirror (like data-plane-api), and swizzle the WORKSPACE to one that uses a remote repository.

This preserves envoy-filter-example as a standalone example of integrating Envoy, while avoiding the multi-phase commit dance (and breakages) we hit regularly these days.

Contributor Guide

Research-Richtung
Untersuche die aktuelle Struktur von envoy filter example und seine Integration über WORKSPACE. Überprüfe die Diskussion im PR #5031 Kommentar unter https://github.com/envoyproxy/envoy/pull/5031#issuecomment 438757841 für den Kontext. Untersuche die WORKSPACE Datei in envoyproxy/envoy, um die Muster für lokale vs. Remote Repositorys zu verstehen. Erwäge die Verwendung von git subtree oder filter branch, um das Beispiel als schreibgeschütztes Mirror zu exportieren, ähnlich dem data plane api Ansatz.
Tech Stack
cpp
Domain
backendbuild systemdeveloper experience
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.
Halber Tag
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
GitBazel
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30