/juju

LeadershipSettingsWatcher fixes. #1813

Merged
merged 1 commit into from Mar 12, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@kat-co @jujubot @wallyworld
@kat-co
Contributor

kat-co commented Mar 12, 2015

  • LeadershipSettingsWatcher now conforms to NotifyWatcher.
  • The test for watching leadership settings now works as intended.

(Review request: http://reviews.vapour.ws/r/1141/)
@kat-co
LeadershipSettingsWatcher fixes. 
- LeadershipSettingsWatcher now conforms to NotifyWatcher.
- The test for watching leadership settings now works as intended.
e698029
@kat-co
Contributor

kat-co commented Mar 12, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Mar 12, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented Mar 12, 2015

Build failed: Does not match ['fixes-1430898']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/2461
@kat-co
Contributor

kat-co commented Mar 12, 2015

jfdi
@wallyworld
Owner

wallyworld commented Mar 12, 2015

$$JFDI$$
@jujubot
Contributor

jujubot commented Mar 12, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Mar 12, 2015

@jujubot
Merge pull request #1813 from katco-/leader-settings-fixes 
LeadershipSettingsWatcher fixes.

- LeadershipSettingsWatcher now conforms to NotifyWatcher.
- The test for watching leadership settings now works as intended.

(Review request: http://reviews.vapour.ws/r/1141/)
0d9ef25

@jujubot jujubot merged commit 0d9ef25 into juju:master Mar 12, 2015

@kat-co kat-co deleted the kat-co:leader-settings-fixes branch Oct 4, 2016

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