envoyproxy/envoy

API request: xds async NACK

Open

#9 447 ouverte le 21 déc. 2019

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

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

When envoy receives a listener from management server it has 3 choices

  1. accept listener and put it in ready state
  2. Reject listener with a NACK
  3. Accept config and put listener in a warming state awaiting an async operation.

If the async operation fails for any reason and it was performed against separate management server, the original management server has no way to know what happened.

There should be a way to indicate a NACK after async operations complete.

This happens with SDS, Wasm code fetch via url or fcds.

Related issues: https://github.com/envoyproxy/envoy/issues/5739 https://github.com/envoyproxy/envoy/issues/7867

Guide contributeur

Direction de recherche
Examinez le protocole xDS et l'implémentation actuelle du NACK dans Envoy. Comprenez comment fonctionnent les opérations asynchrones comme SDS, Wasm fetch ou FCDS. Concevez ensuite un moyen d'envoyer un NACK après la fin de l'opération asynchrone, éventuellement en étendant la réponse xDS ou en introduisant un nouveau message. Consultez les issues associées #5739 et #7867 pour le contexte.
Stack technique
cpp
Domaine
backend
Type d'issue
Fonctionnalité
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.
3-5 jours
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
C++envoy internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25