Merge lp://staging/~dimitern/juju-core/046-split-api-multiple-files-fixed into lp://staging/~juju/juju-core/trunk
Proposed by
Dimiter Naydenov
Status: | Merged |
---|---|
Merged at revision: | 1240 |
Proposed branch: | lp://staging/~dimitern/juju-core/046-split-api-multiple-files-fixed |
Merge into: | lp://staging/~juju/juju-core/trunk |
Diff against target: |
7640 lines (+3825/-3612) 26 files modified
state/api/client.go (+0/-578) state/api/machine.go (+145/-0) state/api/state.go (+77/-0) state/api/unit.go (+48/-0) state/api/watcher.go (+339/-0) state/apiserver/admin.go (+20/-0) state/apiserver/api_test.go (+30/-2069) state/apiserver/apiserver.go (+0/-965) state/apiserver/client.go (+212/-0) state/apiserver/client_test.go (+442/-0) state/apiserver/errors_test.go (+88/-0) state/apiserver/login_test.go (+61/-0) state/apiserver/machine.go (+132/-0) state/apiserver/machine_test.go (+498/-0) state/apiserver/perm_test.go (+584/-0) state/apiserver/resource.go (+82/-0) state/apiserver/root.go (+274/-0) state/apiserver/scenario_test.go (+154/-0) state/apiserver/server_test.go (+125/-0) state/apiserver/state.go (+76/-0) state/apiserver/state_test.go (+144/-0) state/apiserver/unit.go (+41/-0) state/apiserver/unit_test.go (+49/-0) state/apiserver/user.go (+61/-0) state/apiserver/utils.go (+53/-0) state/apiserver/watcher.go (+90/-0) |
To merge this branch: | bzr merge lp://staging/~dimitern/juju-core/046-split-api-multiple-files-fixed |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju Engineering | Pending | ||
Review via email: mp+165687@code.staging.launchpad.net |
Description of the change
state/api: Split to multiple files
To simplify maintenance and development, this
CL splits the API client and server packages
into multiple files. Tests are split as well.
Nothing new introduced, just mechanical.
I'm reproposing this, because the original CL
(https:/
had issue. No changes here, just merged trunk.
To post a comment you must log in.
Reviewers: mp+165687_ code.launchpad. net,
Message:
Please take a look.
Description:
state/api: Split to multiple files
To simplify maintenance and development, this
CL splits the API client and server packages
into multiple files. Tests are split as well.
Nothing new introduced, just mechanical.
I'm reproposing this, because the original CL /codereview. appspot. com/9746043)
(https:/
had issue. No changes here, just merged trunk.
https:/ /code.launchpad .net/~dimitern/ juju-core/ 046-split- api-multiple- files-fixed/ +merge/ 165687
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/9742044/
Affected files: apiclient. go apierror. go machine. go watcher. go /admin. go /api_test. go /apierror. go apiserver/ apiserver. go /apiserver. go /client. go /client_ test.go /errors_ test.go /login_ test.go /machine. go /machine_ test.go /perm_test. go /resource. go /root.go /scenario_ test.go /server_ test.go /state. go /state_ test.go /unit.go /unit_test. go /user.go /utils. go /watcher. go
A [revision details]
M state/api/
M state/api/
M state/api/client.go
A state/api/
A state/api/state.go
A state/api/unit.go
A state/api/
A state/apiserver
M state/apiserver
M state/apiserver
state/
D state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver
A state/apiserver