Viewing all changes in revision 2164.
- Committer: Tarmac
- Date: 2013-12-19 20:44:26 UTC
- mfrom: (2156.2.10 227-cli-deploy-via-api)
- Revision ID: tarmac-20131219204426-cfsh47m3kcdr1mfk
[r=dimitern] cmd/juju: deploy using the API + 1.16 compat.
juju deploy starts to use the API only, if possible.
Older API servers, not supporting charm uploads are
detected and the old 1.16-compatible code with a
direct state connection is executed.
In addition to that, bug #1216830 is fixed (no juju.Conn
usage in state/apiserver antmore), which required some
refactoring, namely moving conn.DeployService and also
conn.AddUnits out of juju.Conn and into a juju/deploy.go,
taking a *state.State as argument.
https://codereview.appspot.com/44240045/
R=rogpeppe
juju deploy starts to use the API only, if possible.
Older API servers, not supporting charm uploads are
detected and the old 1.16-compatible code with a
direct state connection is executed.
In addition to that, bug #1216830 is fixed (no juju.Conn
usage in state/apiserver antmore), which required some
refactoring, namely moving conn.DeployService and also
conn.AddUnits out of juju.Conn and into a juju/deploy.go,
taking a *state.State as argument.
https://codereview.appspot.com/44240045/
R=rogpeppe
- files added:
- juju/deploy.go
- files modified:
- cmd/juju/deploy.go
expand all
collapse all
added
removed
