This exposes more unit attributes to the watchers. The only true
additions in this branch are to the params.go file, and the associated
test. After this change, much more (but unfortunately still not all) of
the values we need will be exposed in the watcher.
The branch is very large because of mechanical changes that prevent
circular imports. UnitStatus, Port, and ResolvedMode needed to move
into the params package, and this required a very large number of
mechanical import changes. The rest of the changes in this branch,
other than some test fix-ups in megawatcher_internal_test.go, are just
these mechanical changes to handle the new location.
*** Submitted:
Expose more delta information
This exposes more unit attributes to the watchers. The only true
additions in this branch are to the params.go file, and the associated
test. After this change, much more (but unfortunately still not all) of
the values we need will be exposed in the watcher.
The branch is very large because of mechanical changes that prevent internal_ test.go, are just
circular imports. UnitStatus, Port, and ResolvedMode needed to move
into the params package, and this required a very large number of
mechanical import changes. The rest of the changes in this branch,
other than some test fix-ups in megawatcher_
these mechanical changes to handle the new location.
R=thumper, fwereade /codereview. appspot. com/8086045
CC=
https:/
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go params/ params_ test.go (right):
File state/api/
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go# newcode55 params/ params_ test.go: 55: Series: "series", // E.g. precise.
state/api/
On 2013/04/01 20:46:58, thumper wrote:
> Can we change the test so instead of using comments for "precise" and
> "wordpress" we instead have the test output show this?
Done.
https:/ /codereview. appspot. com/8086045/