envoyproxy/envoy

Non-hermetic DNS testing in eg dynamic_forward_proxy is causing recurring failure/slowness

Open

#42 217 ouverte le 24 nov. 2025

Voir sur GitHub
 (24 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/ciarea/dynamic_forward_proxyarea/test flakesbughelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 7j 21h) (260 PRs mergées en 30 j)

Description

I stumbled upon this issue with the mobile test //test/common/integration:client_integration_test

in that case the test had a total timeout and was querying dns for doesnotexist - dns was retrying 3x, and taking 5+ seconds to respond on each try - blowing the tests total timeout budget

im struggling to find how i resolved the issue in mobile - but it seems i got past it

this problem has again occurred when trying to update the build image - asan and gcc are reliably timeouting on similar tests - and it looks like for similar reasons

i would have thought these tests should not be calling any real dns - it should be mocked and hermetic

Guide contributeur

Direction de recherche
Étudiez comment simuler la résolution DNS dans les tests pour éviter les requêtes DNS réelles. Envisagez d'utiliser une bibliothèque de simulation DNS ou de modifier la configuration du test pour utiliser un résolveur DNS factice.
Stack technique
cpp
Domaine
testing
Type d'issue
Bug
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
C++GitTesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65