envoyproxy/envoy

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

Open

#5054 aperta il 15 nov 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/buildhelp wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga la struttura attuale di envoy filter example e la sua integrazione tramite WORKSPACE. Rivedi la discussione nel commento della PR #5031 all'URL https://github.com/envoyproxy/envoy/pull/5031#issuecomment 438757841 per contesto. Esamina il file WORKSPACE in envoyproxy/envoy per comprendere i pattern di repository locali vs remoti. Considera l'uso di git subtree o filter branch per esportare l'esempio come mirror di sola lettura, simile all'approccio di data plane api.
Tech stack
cpp
Dominio
backendbuild systemdeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitBazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30