Code review comment for lp:~bcsaller/pyjuju/subordinate-removes

Revision history for this message
Benjamin Saller (bcsaller) wrote :


Please take a look.

Destroying a subordinate shouldn't result in tracebacks

Destroying a subordinate service after destroying its primary services
results in a traceback
This patch corrects this. The associated test also previously failed to
exercise all the needed
behavior to demonstrate this, which is corrected as well.

(do not edit description out of merge proposal)

Please review this at

Affected files:
   A [revision details]
   M juju/control/
   M juju/control/
   M juju/control/tests/
   M juju/control/tests/
   M juju/hooks/tests/
   M juju/lib/
   M juju/state/tests/
   M juju/state/tests/
   M juju/unit/
   M juju/unit/tests/
   M juju/unit/

« Back to merge proposal