envoyproxy/envoy

add memory filesystem for making tests faster, more hermetic, and more robust

Open

#11.121 aperta il 8 mag 2020

Vedi su GitHub
 (9 commenti) (1 reazione) (1 assegnatario)C++ (5373 fork)batch import
beginnerhelp wanted

Metriche repository

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

Descrizione

Currently real files are used for file i/o during tests. An alternative I've used in the past is to have an in-memory representation of a file-system: https://github.com/apache/incubator-pagespeed-mod/blob/master/pagespeed/kernel/base/mem_file_system.h

This has these advantages:

  • completely hermetic, with a clean slate between tests
  • very fast (no I/O), speeding up the running of massive numbers of tests
  • you can inject notifiers etc, so you can for example get a callback or condvar wakeup or similar when a file is written

Envoy already has a filesystem abstraction (yay!) and a way to squirrel that into code that needs to read/write files (Api::Api) just no in-memory impl for it.

I think this would be a good intern project. The example I gave above would not exactly meet Envoy's needs but it could be a reference point.

Guida contributor

Direzione di ricerca
Esamina l'astrazione del filesystem esistente in Envoy (probabilmente in source/common/filesystem/), concentrandoti sull'interfaccia Api::Api che fornisce l'implementazione corrente del filesystem. Rivedi il file Apache mem file system.h citato per i pattern di progettazione. Progetta un'implementazione minima in memoria che supporti le operazioni di I/O su file (lettura, scrittura, stat, ecc.) e si integri con il framework di test di Envoy. Controlla i commenti dell'issue per eventuali vincoli aggiuntivi o indicazioni dai maintainer.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy filesystem abstraction
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60