Descriptionstate: add machine status; refractor unit status
Added Machine.Status() and SetStatus(), as with a unit.
Since we're using the same code, a refactoring was in
order: common code for managing statuses moved to a
separate state collection (see status.go); status consts
by the API and state added to state/api/params, so they
can be accessed from both.
Some changes to allWatcher/megawatcher code: because
now Status and StatusInfo are no longer part of the
unit document, you cannot watch for these changes, but
arguably only the juju status command actually needs
to read them, so the API should be handle this eventually.
Drive-by fix: constranits and annotations (and now status)
of the last unit of a service were not removed properly.
Now they are.
https://code.launchpad.net/~dimitern/juju-core/021-state-add-machine-status/+merge/156534
(do not edit description out of merge proposal)
Patch Set 1 #
Total comments: 22
Patch Set 2 : state: add machine status; refractor unit status #
Total comments: 23
Patch Set 3 : state: add machine status; refractor unit status #
Total comments: 2
Patch Set 4 : state: add machine status; refractor unit status #
MessagesTotal messages: 13
|