Merge lp:~1chb1n/charms/trusty/ceph-osd/next-amulet-1602 into lp:~openstack-charmers-archive/charms/trusty/ceph-osd/next

Proposed by Ryan Beisner
Status: Merged
Merged at revision: 63
Proposed branch: lp:~1chb1n/charms/trusty/ceph-osd/next-amulet-1602
Merge into: lp:~openstack-charmers-archive/charms/trusty/ceph-osd/next
Diff against target: 28 lines (+7/-3)
1 file modified
tests/basic_deployment.py (+7/-3)
To merge this branch: bzr merge lp:~1chb1n/charms/trusty/ceph-osd/next-amulet-1602
Reviewer Review Type Date Requested Status
OpenStack Charmers Pending
Review via email: mp+285942@code.launchpad.net

Description of the change

Update amulet test definitions; Wait for workload status 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 #388 ceph-osd-next for 1chb1n mp285942
    LINT OK: passed

Build: http://10.245.162.36:8080/job/charm_lint_check/388/

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

charm_unit_test #310 ceph-osd-next for 1chb1n mp285942
    UNIT OK: passed

Build: http://10.245.162.36:8080/job/charm_unit_test/310/

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

charm_amulet_test #159 ceph-osd-next for 1chb1n mp285942
    AMULET OK: passed

Build: http://10.245.162.36:8080/job/charm_amulet_test/159/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/019-basic-trusty-mitaka' (properties changed: -x to +x)
2=== modified file 'tests/basic_deployment.py'
3--- tests/basic_deployment.py 2015-06-29 21:20:01 +0000
4+++ tests/basic_deployment.py 2016-02-12 21:25:45 +0000
5@@ -27,6 +27,13 @@
6 self._add_relations()
7 self._configure_services()
8 self._deploy()
9+
10+ u.log.info('Waiting on extended status checks...')
11+ exclude_services = ['mysql']
12+
13+ # Wait for deployment ready msgs, except exclusions
14+ self._auto_wait_for_status(exclude_services=exclude_services)
15+
16 self._initialize_tests()
17
18 def _add_services(self):
19@@ -114,9 +121,6 @@
20 u.log.debug('openstack release str: {}'.format(
21 self._get_openstack_release_string()))
22
23- # Let things settle a bit original moving forward
24- time.sleep(30)
25-
26 # Authenticate admin with keystone
27 self.keystone = u.authenticate_keystone_admin(self.keystone_sentry,
28 user='admin',

Subscribers

People subscribed via source and target branches