Merge lp://qastaging/~javier.collado/utah/bug1160857 into lp://qastaging/utah

Proposed by Javier Collado
Status: Merged
Approved by: Javier Collado
Approved revision: 846
Merged at revision: 844
Proposed branch: lp://qastaging/~javier.collado/utah/bug1160857
Merge into: lp://qastaging/utah
Diff against target: 86 lines (+12/-1)
7 files modified
debian/changelog (+6/-0)
examples/run_install_test.py (+1/-0)
examples/run_test_bamboo_feeder.py (+1/-1)
examples/run_test_cobbler.py (+1/-0)
examples/run_test_vm.py (+1/-0)
examples/run_utah_tests.py (+1/-0)
utah/run.py (+1/-0)
To merge this branch: bzr merge lp://qastaging/~javier.collado/utah/bug1160857
Reviewer Review Type Date Requested Status
Javier Collado (community) Approve
Max Brustkern (community) Approve
Review via email: mp+155713@code.qastaging.launchpad.net

Description of the change

This branch adds a new return code UNHANDLED_ERROR that is returned by the
server when an non UTAH exception is caught (before this change, a succes
return code was returned).

Note that I've started a new version number in debian/changelog since the
latest one was already released. For the future, I think that once a new
version is released a new commit has to be pushed to start the new version
number since that shouldn't be part of a merge request like this one.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

Looks good to me, but I haven't tested it. If you have, we should be fine.

review: Approve
Revision history for this message
Javier Collado (javier.collado) wrote :

Yes, I simulated the error condition adding a line to raise an exception in the
code and the return code was the expected one. Merging.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches