envoyproxy/envoy

Update observability dependency libcircllhist

Open

#12.804 aperta il 25 ago 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
dependencieshelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Title: Update observability dependency libcircllhist

Description: When trying to update the observability dependencies in https://github.com/envoyproxy/envoy/pull/12799 the latest libcircllhist the following tests fail with not returning expected results.

  • envoy/bazel-out/k8-fastbuild/testlogs/test/common/stats/thread_local_store_test/test.log
  • envoy/bazel-out/k8-fastbuild/testlogs/test/extensions/stats_sinks/hystrix/hystrix_test/test.log
  • envoy/bazel-out/k8-fastbuild/testlogs/test/server/admin/prometheus_stats_test/test.log
  • envoy/bazel-out/k8-fastbuild/testlogs/test/server/admin/stats_handler_test/test.log

I lost the detailed logs with subsequent commits but will try to reproduce

Relevant Links: The commits / changes between the lastest June 25 2020 commit and the current Envoy commit on February 11 2019 is https://github.com/circonus-labs/libcircllhist/compare/63a16dd6f2fc7bc841bb17ff92be8318df60e2e1...18f1b97603757d5e65baefa65b90d6ec182e8a30

Guida contributor

Direzione di ricerca
Esamina i log dei test falliti per identificare gli specifici errori di asserzione e traccia le modifiche in libcircllhist tra i commit di riferimento per capire quale comportamento è cambiato.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
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.
Abbastanza chiara
Prerequisiti
GitBazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70