Add leader and status new hooks #129

Merged
merged 3 commits into from May 20, 2015

Conversation

Projects
None yet
4 participants
Contributor

perrito666 commented May 11, 2015

These hooks are required for:
Leader election, these are alredy defined but in the wrong place
inside juju core.
Health status, we add an update-status hook.

perrito666 added some commits Apr 27, 2015

Add leader and status new hooks
These hooks are required for:
Leader election, these are alredy defined but in the wrong place
inside juju core.
Health status, we add an update-status hook.
Member

mattyw commented May 20, 2015

LGTM

Member

frankban commented May 20, 2015

👍 thank you!

hooks/hooks.go
+ ConfigChanged Kind = "config-changed"
+ UpgradeCharm Kind = "upgrade-charm"
+ Stop Kind = "stop"
+ ActionRequested Kind = "action-requested" // TODO: remove in charm v5, DEPRECATED
@rogpeppe

rogpeppe May 20, 2015

Owner

We could probably remove this now, as we've long since moved to charm v5 and beyond.

Owner

rogpeppe commented May 20, 2015

LGTM with one thought.

perrito666 added a commit that referenced this pull request May 20, 2015

@perrito666 perrito666 merged commit 2b813ba into juju:v6-unstable May 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment