jaegertracing/jaeger

[test] Flaky test TestPasswordFromFile in plugin/storage/es/factory_test.go

Open

#4743 aperta il 10 set 2023

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)Go (2326 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (18.974 star)
Metriche merge PR
 (Merge medio 4g 11h) (85 PR mergiate in 30 g)

Descrizione

https://github.com/jaegertracing/jaeger/actions/runs/6134973238/job/16648278200?pr=4742#step:7:4523

One strange thing I noticed in the output that the very last logged request to the server was with Basic Auth string looking like the first password, not the second password, as if the request with the second password was made but then someone again made a request with the old one. I wonder if the es.Client object can continue sending pings to the server if it's not Close-ed (which we don't do).

Guida contributor

Direzione di ricerca
Ispeziona il test instabile in plugin/storage/es/factory test.go e traccia la gestione della password. Controlla se es.Client invia ping che interferiscono con i cambiamenti di password. Considera di chiudere correttamente il client.
Tech stack
goelasticsearch
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65