Segnalazione #758
Configurazione data source solo con class method
100%
Description
Al momento vengono passati pezzi di configurazione delle data source nei costruttori (path di arpwatch, path di syslog), ma vengono testati nell'is_viable, che accede solo ai membri della classe.
Togliere questi parametri dai costruttori per non dare l'idea che sia quello che c'è da fare per farne override, e nei test fare override dei class member
Associated revisions
Pass Config to is_viable, use paths from Config, and remove them from class methods. refs: #758
Merge branch 't758' Fixes: #758
History
Updated by Enrico Zini almost 6 years ago
Per poter lanciare monitor e runserver da utente, ho aggiunto una classe manager.config.Config
che raccoglie le configurazioni per i vari componenti di Fuss Manager.
L'idea è passare questa classe ai vari costruttori, in modo che si possano configurare, ed eventualmente modificarne i contenuti da fuss-manager
con opzioni da linea di comando.
Risolverei la questione di questo ticket passando l'istanza di Config
anche alle is_viable
.
Updated by Enrico Zini almost 6 years ago
- Status changed from Nuovo to Commenti
- Assignee changed from Enrico Zini to Elena Grandi
Dovrebbe essere fatto (branch t758), te lo passo per review
Updated by Elena Grandi almost 6 years ago
- Status changed from Commenti to Risolto
- % Done changed from 0 to 100
Applicata nel changeset fuss-manager|e3f6943ff8f2d766ca5c188734afedf7dd180717.
Deal with the stats store not existing the first time it is run. refs: #758