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

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

Reviewers: mp+100489_code.launchpad.net,

Message:
Please take a look.

Description:
juju/control should be aware of subordinates

Includes support for add-unit, deploy, status, remove-unit

https://code.launchpad.net/~bcsaller/juju/subordinate-control-status/+merge/100489

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/5970068/

Affected files:
   A [revision details]
   M juju/control/add_unit.py
   M juju/control/deploy.py
   M juju/control/remove_unit.py
   M juju/control/status.py
   M juju/control/tests/test_add_unit.py
   M juju/control/tests/test_deploy.py
   M juju/control/tests/test_remove_unit.py
   M juju/control/tests/test_status.py
   M juju/state/tests/test_charm.py
   M juju/unit/tests/test_lifecycle.py

« Back to merge proposal