lp:~javier.collado/utah/bug1035227

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:utah at revision 633

Max Brustkern (community): Approve on 2012-08-14

Javier Collado (community): Needs Resubmitting on 2012-08-14

Diff: 47 lines (+19/-0) (has conflicts)

2 files modified

utah/provisioning/provisioning.py (+4/-0)
utah/run.py (+15/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1035227: ssh commands error return codes ignored

Medium

Fix Released

Link a bug report

Related blueprints

625. By Javier Collado on 2012-08-14

Replaced sys.stderr.write with machine.logger.error

624. By Javier Collado on 2012-08-13

Replaced logger.error with sys.stderr.write

logger is used in the code above to set logging level, but isn't supposed to be
used to write logging messages. sys.stderr.write seems to be the more
reasonable option (machine.logger could be used as well, but that would look
odd for something that is executed outside the machine object)

623. By Javier Collado on 2012-08-13

Added error message when client fails

622. By Javier Collado on 2012-08-13

Added return code to ssh commands logging

When return code is failure (!=0) it will be printed as a warning

621. By Javier Collado on 2012-08-10

Added check to make sure that utah clien ran test cases successfully

620. By Max Brustkern on 2012-08-10

Merged Javier's change that Fixes 1034348

619. By Max Brustkern on 2012-08-10

Merged Javier's readability improvements

618. By Max Brustkern on 2012-08-10

Merged Javier's client installation related exceptions

617. By Max Brustkern on 2012-08-10

Merged Javier's changes for propagating exit status so exceptions still show

616. By Joe Talbott on 2012-08-09

Remove extra 'testsuites' directory