Segnalazione #743
Log azioni utente
0%
Description
Logging delle azioni fatte dai vari utenti, a scopo di analytics per vedere le feature piú o meno usate
Associated revisions
Write operation log. refs: #743
Use a logger class, to make it easier to implement log rotation in the future. refs: #743
Use a tempfile for the operation log during tests. refs: #743
Started a test suite for ops. refs: #743
Added slots to all Event classes, to prevent setting mistyped attributes. refs: #743
Implemented disabling a data source from config. refs: #743
Turn all stores ip/name/mac resolution functions into coroutines. refs: #743
Only run the chaos data source if debug is True. refs: #743
Cleaned up / expanded TestManagerMixin. refs: #743
Test AddGroup and DelGroup. refs: #743
Test that OperationLog does not fail if the log directory does not exist. refs: #743
History
Updated by Elena Grandi almost 6 years ago
Updated by Tenere traccia:
- delle azioni richiamate dall'interfaccia;
- delle azioni svolte dal backend
Updated by
Updated by Enrico Zini over 5 years ago
- Status changed from In elaborazione to Commenti
- Assignee changed from Enrico Zini to Elena Grandi
Ora le Operation vengono eseguite tramite il metodo Manager.execute, che le logga in un file un-json-per-riga.
Non ho implementato supporto per ruotare il file di log, che lascierei a un futuro in cui si ha un'idea piú chiara di come gestire reload dell'applicazione e rotazione dei suoi log in generale. Ho però predisposto il codice per implementare rotazione, incapsulando il logging in una OperationLog.
Ho aggiunto Operation.source per tracciare il punto dell'interfaccia in cui è stata richiesta un'operazione. In futuro, potremo aggiungerci anche il nome dell'utente e quant'altro possa servire.
Updated by Elena Grandi over 5 years ago
Su una macchina dove non esiste /var/log/fuss-manager/ i test fallivano per l'assenza della directory.
Ho cambiato i test per usare un tempfile perché possano girare in ambienti isolati (CI o simili).
Può essere utile (prima del merge?) aggiungere dei test che verifichino che l'errore specifico (configurazione in cui il file dei log non sia apribile per vari motivi) sia gestito?
Updated by Enrico Zini over 5 years ago
- Assignee changed from Enrico Zini to Elena Grandi
Fatto, e testato anche un po' meglio le ops già che c'ero
Updated by Elena Grandi over 5 years ago
- Status changed from Commenti to Chiuso
- Assignee changed from Elena Grandi to Enrico Zini
Mergiato in master
Track operation sources. refs: #743