envoyproxy/envoy

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

Open

#5 054 ouverte le 15 nov. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/buildhelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez la structure actuelle de envoy filter example et son intégration via WORKSPACE. Consultez la discussion dans le commentaire de la PR #5031 à l'adresse https://github.com/envoyproxy/envoy/pull/5031#issuecomment 438757841 pour le contexte. Examinez le fichier WORKSPACE dans envoyproxy/envoy pour comprendre les modèles de dépôt local vs distant. Envisagez d'utiliser git subtree ou filter branch pour exporter l'exemple en tant que miroir en lecture seule, similaire à l'approche de data plane api.
Stack technique
cpp
Domaine
backendbuild systemdeveloper experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitBazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30