Juju Charms Collection
ceilometer-agent package

Merge lp://staging/~1chb1n/charms/trusty/ceilometer-agent/next-amulet-15.10 into lp://staging/~openstack-charmers-archive/charms/trusty/ceilometer-agent/next

  1. Trusty Tahr (14.04)
  2. next-amulet-15.10
  3. Merge into next
Proposed by Ryan Beisner
Status: Merged
Merged at revision: 70
Proposed branch: lp://staging/~1chb1n/charms/trusty/ceilometer-agent/next-amulet-15.10
Merge into: lp://staging/~openstack-charmers-archive/charms/trusty/ceilometer-agent/next
Diff against target: 463 lines (+178/-35)
5 files modified
hooks/charmhelpers/contrib/openstack/amulet/deployment.py (+40/-0)
hooks/charmhelpers/contrib/openstack/utils.py (+1/-0)
hooks/charmhelpers/core/host.py (+12/-1)
tests/basic_deployment.py (+85/-34)
tests/charmhelpers/contrib/openstack/amulet/deployment.py (+40/-0)
To merge this branch: bzr merge lp://staging/~1chb1n/charms/trusty/ceilometer-agent/next-amulet-15.10
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+275029@code.staging.launchpad.net

Description of the change

Update amulet tests for Trusty-Liberty, Wily-Liberty.

Sync charmhelpers.

Add service and relations to satisfy workload status ready state.

Add new logic to wait for extended status message to confirm deploy is ready, before testing.

To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #12207 ceilometer-agent-next for 1chb1n mp275029
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/12207/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #11328 ceilometer-agent-next for 1chb1n mp275029
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/11328/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #7449 ceilometer-agent-next for 1chb1n mp275029
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/7449/

Revision history for this message
Liam Young (gnuoy) wrote :

Approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches