Merge lp://staging/~sinzui/juju-ci-tools/app-status into lp://staging/juju-ci-tools/repository
Proposed by
Curtis Hovey
| Status: | Merged |
|---|---|
| Merged at revision: | 48 |
| Proposed branch: | lp://staging/~sinzui/juju-ci-tools/app-status |
| Merge into: | lp://staging/juju-ci-tools/repository |
| Diff against target: |
161 lines (+68/-0) 14 files modified
charms-centos/dummy-sink/hooks/install (+4/-0) charms-centos/dummy-sink/hooks/source-relation-changed (+10/-0) charms-centos/dummy-sink/hooks/start (+4/-0) charms-centos/dummy-source/hooks/config-changed (+10/-0) charms-centos/dummy-source/hooks/install (+4/-0) charms-centos/dummy-source/hooks/start (+4/-0) charms/dummy-sink/hooks/install (+2/-0) charms/dummy-sink/hooks/source-relation-changed (+10/-0) charms/dummy-sink/hooks/start (+2/-0) charms/dummy-sink/hooks/stop (+2/-0) charms/dummy-source/hooks/config-changed (+10/-0) charms/dummy-source/hooks/install (+2/-0) charms/dummy-source/hooks/start (+2/-0) charms/dummy-source/hooks/stop (+2/-0) |
| To merge this branch: | bzr merge lp://staging/~sinzui/juju-ci-tools/app-status |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Martin Packman (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Add application status to the juju 2 centos and ubuntu dummy charms
The charms now use application status to indicate when they are ready. The status message contains the token for us to see. The windows charms were updated on Monday to enable testing in Azure.
This branch is one part of an effort reliably test in GCE. I have another branch that waits for workload status. The GCE scenario passes. I can see we wait about 12 dots. The centos charms continue to pass in aws.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good, a couple of nitpicky comments inline.