Code review comment for lp:~fgimenez/snappy/integration-tests-verbosity-flag

Revision history for this message
Federico Gimenez (fgimenez) wrote :

@Michael I totally agree, it would be nice if we could use a single logger.

I had a look at the logger package before beginning with this and it seemed to me that it was oriented to a cli tool, with two "channels" to output messages but without giving the user the option to decide which kind of messages to show. In the case of the test logger there should be just one way of output (it shouldn't use syslog), but with different levels, and the level to be used should be selectable at invocation time. If we can make both things work at once it would be great :)

When we'll execute the tests on CI, for instance, we would like always the more verbose output on stdout, while perhaps a developer executing the suite locally would prefer to have a less verbose output, and enable the verbose one if there's an error.

Leo, what do you think?

« Back to merge proposal