Code review comment for lp:~stevanr/linaro-ci-dashboard/fix_jenkins_dashboard_lib

Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

I would really suggest to use log.exception() in exception handlers - let's see what actual underlying error. Later, when we go into production mode, we can replace those which appear to frequently and don't represent real error with .info()/.warning() . But for time being ability to see real error which happens is a must imho.

« Back to merge proposal