store: do not log events internally on Publish

The external code that calls Publish has to deal with event logging
anyway for logging failures and whatnot. It's trivial for it to log the
error and success case from Publish too, so there's no reason to have
that logic duplicated in two different places. In fact, it's harder to
deal with it because the call site has to take care not to log errors
twice in that particular case.

https://code.launchpad.net/~niemeyer/juju/no-events-on-publish/+merge/92811

(do not edit description out of merge proposal)

[niemeyer, 2012-02-13 17:57:08]

Please take a look.

[rog, 2012-02-13 17:59:45]

On 2012/02/13 17:57:08, niemeyer wrote:
> Please take a look.

LGTM

[niemeyer, 2012-02-13 18:00:41]

Please take a look.

[fwereade, 2012-02-13 18:03:31]

LGTM

[niemeyer, 2012-02-13 18:07:37]

*** Submitted:

store: do not log events internally on Publish

The external code that calls Publish has to deal with event logging
anyway for logging failures and whatnot. It's trivial for it to log the
error and success case from Publish too, so there's no reason to have
that logic duplicated in two different places. In fact, it's harder to
deal with it because the call site has to take care not to log errors
twice in that particular case.

R=rog, fwereade
CC=
https://codereview.appspot.com/5654081