Code review comment for lp:~thumper/juju-core/lxc-broker

Revision history for this message
Tim Penhey (thumper) wrote :

Reviewers: mp+170713_code.launchpad.net,

Message:
Please take a look.

Description:
Implement the lxc broker for the provisioner

Had to make some test functions and helpers exported.

https://code.launchpad.net/~thumper/juju-core/lxc-broker/+merge/170713

Requires:
https://code.launchpad.net/~thumper/juju-core/lxc-container/+merge/169980

(do not edit description out of merge proposal)

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

Affected files:
   A [revision details]
   D container/lxc/export_test.go
   M container/lxc/lxc_test.go
   M container/lxc/mock/mock-lxc.go
   A container/lxc/test.go
   A worker/provisioner/lxc-broker.go
   A worker/provisioner/lxc-broker_test.go
   M worker/provisioner/provisioner.go
   M worker/provisioner/provisioner_task.go

« Back to merge proposal