envoyproxy/envoy

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

Open

#42.217 aberto em 24 de nov. de 2025

Ver no GitHub
 (24 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/ciarea/dynamic_forward_proxyarea/test flakesbughelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 7d 21h) (260 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue como simular a resolução de DNS nos testes para evitar consultas DNS reais. Considere usar uma biblioteca de simulação de DNS ou modificar a configuração do teste para usar um resolvedor DNS fictício.
Pilha de tecnologia
cpp
Domain
testing
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++GitTesting
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65