jaegertracing/jaeger

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

Offen

#4.743 geöffnet am 10.09.2023

Auf GitHub ansehenContributor Guide
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (2.326 Forks)batch import
buggood first issuehelp wanted

Repository-Metriken

Stars
 (18.974 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4T 11h) (85 gemergte PRs in 30 T)

Beschreibung

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).

Contributor Guide

Research-Richtung
Überprüfen Sie den flaky Test in plugin/storage/es/factory test.go und verfolgen Sie die Passwortbehandlung. Prüfen Sie, ob der es.Client Pings sendet, die Passwortänderungen stören. Erwägen Sie, den Client ordnungsgemäß zu schließen.
Tech Stack
goelasticsearch
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Recherche nötig
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65