benoitc/gunicorn

`reload_extra_files` should accept wildcards

Open

#1643 aperta il 14 nov 2017

Vedi su GitHub
 (5 commenti) (14 reazioni) (0 assegnatari)Python (1707 fork)batch import
Feature/Reloadgood first issue

Metriche repository

Star
 (9334 star)
Metriche merge PR
 (Merge medio 8h 8m) (1 PR mergiata in 30 g)

Descrizione

Hi,

My app structure is such that each view in my app has it's own config.json located at ui/<view_name>/config.json.

I would love to reload on modification for each of those JSON files.

Currently, I'm doing, for each view:

reload_extra_file = [
  'ui/<view_1>/config.json',
  'ui/<view_2>/config.json',
  'ui/<view_3>/config.json,
  ...
]

but this is a little cumbersome, since I have to add a new line everytime I add a view to my app (and I tend to forget about it).

We should support wildcards for this kind of use case, something like:

reload_extra_file = ['*.json', '*.js', '*.py', '*.yaml']

@benoitc Is that something you would consider implementing ? I can maybe help with this and create a PR if you let me know where this needs to be changed in the current code.

Guida contributor

Direzione di ricerca
Esamina la funzionalità reload extra files nel codice di Gunicorn, in particolare come vengono attualmente confrontati i percorsi dei file durante il ricaricamento automatico. Guarda il modulo 'reloader' (gunicorn/reloader.py) per comprendere la logica esistente. Quindi implementa il supporto per i pattern wildcard usando il modulo glob di Python o fnmatch. Assicurati di gestire sia i percorsi esatti che i pattern (ad esempio '*.json') e aggiorna i test pertinenti.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
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
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70