/juju

state: allwatcher now reflects instance status changes #7679

Merged
merged 1 commit into from Jul 28, 2017

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@mjs @jujubot @wallyworld
@mjs
Contributor

mjs commented Jul 28, 2017

Description of change

Due to a silly bug (and missing test coverage), instance status
changes were never reflected in the output from the allwatcher.

QA steps

  • Watch the allwatcher API output using a client (for example)
  • Add a machine to the model
  • Observe that updates for the machine reflect instance status changes

Documentation changes

N.A.

Bug reference

Fixes https://bugs.launchpad.net/juju/+bug/1695335
@mjs
state: allwatcher now reflects instance status changes 
Due to a silly bug (and missing test coverage), instance status
changes were never reflected in the output from the allwatcher.

Fixes https://bugs.launchpad.net/juju/+bug/1695335
23f8b9e
@mjs
Contributor

mjs commented Jul 28, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Jul 28, 2017

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

@jujubot jujubot merged commit 171ce34 into juju:2.2 Jul 28, 2017
1 check failed

1 check failed

github-check-merge-juju Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@mjs mjs deleted the mjs:1695335-allwatcher-instance-status-2.2 branch Jul 28, 2017

@mjs mjs referenced this pull request Jul 28, 2017

Merged

state: allwatcher now reflects instance status changes #7681

jujubot added a commit that referenced this pull request Jul 28, 2017

@jujubot
Merge pull request #7681 from mjs/1695335-allwatcher-instance-status 
state: allwatcher now reflects instance status changes

Forward port of #7679
a0b1cb3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment