Code review comment for lp:~javier.collado/utah/bug1035227
Revision history for this message
| Javier Collado (javier.collado) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
I'm completely fine with using machine.logger, so that those error messages are also available in the log file. However, I think the code isn't consistent and it uses "print" statements when it could use "machine.logger" for example in utah/run.
What do you think about updating them in a separate merge?