Change state.User to take a names.UserTag
removed Action trigger from worker ModeHookError

This patches a bug in the uniter which can result in ModeHookError being
cleared by a successful Action run on the impacted unit.  Actions can no
longer be run in ModeHookError.

This will be resolved after the Context refactor on worker.
Merge pull request #730 from binary132/actions-modehookerror-fix

removed Action trigger from worker ModeHookError

This patches a bug in the uniter which can result in ModeHookError being
cleared by a successful Action run on the impacted unit.  Actions can no
longer be run in ModeHookError.

This will be resolved after the Context refactor on worker.
Merge pull request #723 from howbazaar/name-tag-update

Change state.User to take a names.UserTag

There are a few other drive by changes:
 - bootstrap agent test also checks created environment
 - the logging from the apiserver permission test has some breaks in it (to visually separate the test output when it fails)
 - the user manager made me sad, we are looking at consistency of tags/names
 - adding a service no longer enforces that the user exists, however an envuser must exist (remote users will exist soon)
 - the UserTag returned from state.User instances is now explicitly a local user tag (rather than implicitly)