Merge lp://staging/~julian-edwards/maas/rndc-crash-bug-1386488 into lp://staging/~maas-committers/maas/trunk
Proposed by
Julian Edwards
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Julian Edwards | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 3313 | ||||
Proposed branch: | lp://staging/~julian-edwards/maas/rndc-crash-bug-1386488 | ||||
Merge into: | lp://staging/~maas-committers/maas/trunk | ||||
Diff against target: |
207 lines (+56/-14) 4 files modified
src/maasserver/middleware.py (+16/-0) src/maasserver/node_action.py (+7/-6) src/maasserver/tests/test_middleware.py (+23/-5) src/maasserver/tests/test_node_action.py (+10/-3) |
||||
To merge this branch: | bzr merge lp://staging/~julian-edwards/maas/rndc-crash-bug-1386488 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Graham Binns (community) | Approve | ||
Review via email: mp+239941@code.staging.launchpad.net |
Commit message
Catch ExternalProcess
Description of the change
I've done this with a middleware change for the API and followed the example of the RPC exceptions in the node actions and extended its concept to be a little more general, in particular the test.
To post a comment you must log in.
Marking WIP, something is wrong with the diff.