Fix minor `go tool vet` warning

Was getting:

    $ go tool vet -methods -printf -rangeloops .
    worker/deployer/simple_test.go:250: possible formatting directive in Log call
Merge pull request #842 from johnweldon/go-vet-warning-fix

Fix minor `go tool vet` warning

Was getting:

    $ go tool vet -methods -printf -rangeloops .
    worker/deployer/simple_test.go:250: possible formatting directive in Log call
ActionComplete / ActionFailed called after Action completion

The worker now calls ActionComplete or ActionFailed on state, depending
on the success state of the Action and any errors.

The hook is set to an error state if the Action hook is missing, or if
other errors take place during hook execution.  If the Action hook sets
a fail state, the Hook will not error, rather the Action will fail.
ActionComplete / ActionFailed called after Action completion

ActionTags are now properly initialized on worker uniter.HookContext at the
time it is created.
ActionComplete / ActionFailed called after Action completion

Final bits and pieces to complete PR 617 put in place; tests fleshed out
and passing.
ActionComplete / ActionFailed called after Action completion

Addressed davecheney's points in PR 617.
ActionComplete / ActionFailed called after Action completion

Addressed further points in PR 617:
 - Action results no longer a pointer, thus no nil checks needed
 - ActionFail call-home now sends a sane message to the user, without
   discarding error context
 - Minor tweaks to relation write-out logic
 - Removed pointless actionStatus type
 - Checks for nil state handle for ActionComplete / ActionFail
worker/uniter finalizeContext refactor for ActionFinish
ActionComplete / ActionFailed called after Action completion

Addressed remarks in PR #617:
 - Action-related HookContext types bundled into a single type.
 - Comment added to State handle in HookContext.
 - actionData constructor called from outside the HookContext, in the
   case of Actions only.
ActionFinish called after Action completion

Removed actionFailed() helper function, made naming tweak.
ActionFinish on the worker after Action hook run

This commit splits the logic of Uniter.runHook into Uniter.runAction and
Uniter.runHook.  It also uses the new `Action` Hook Kind, improves error
handling, improves tests, and addresses other comments from PR #617.

Depends on github.com/juju/charm/ PR #43.  This commit will be
amended to include the dependency once the changes to juju/charm have
been finalized.
ActionFinish on the worker after Action hook run

Bugfixes to several worker uniter tests; addressed @fwereade's comments
on PR 617.
ActionFinish on the worker after Action hook run

This commit addresses requested changes to PR 617, including smarter
handling of error states in Actions, and cleaner runAction handling
of action param validation errors.
ActionFinish on the worker after Action hook run

This commit addresses @TheMue's comments on PR 617, renaming errors to
something more sensible in worker/context finalizeContext.