cloudmarker / cloudmarker / 723

84%

push

travis-ci-com

Update logger format, filename, and backupCount

The log format now includes the process ID (PID). This is useful in
troubleshooting when a worker process crashes. Usually, when a worker
process crashes and the main process is still alive, the worker process
appears as `<defunct>` in the `ps` output. So comparing the PID of the
defunct process with the PIDs in the logs help us to understand which
process crashed.

The default `filename` as `logs/cloudmarker.log` has always required a
`mkdir -p logs` hack in `Makefile`. In this change, we are updating this
path to `/tmp/cloudmarker.log`, so that we no longer need this hack.
This will also ensure that anyone installing the project with `pip` for
the first time can run the mock plugins without any issues without
having to follow any additional steps.

Further, `backupCount` is set to `5`. This is useful during live usage
because the log files can be large when large clouds are audited. This
setting ensures that the log files do not fill up all the disk space. A
maximum of only 5 most recent backup log files are kept by default and
the older ones are deleted.

341 of 388 branches covered (87.89%)

Branch coverage included in aggregate %.

1661 of 1824 relevant lines covered (91.06%)

2.73 hits per line