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

Proposed by Ryan Beisner
Status: Merged
Merged at revision: 133
Proposed branch: lp:~1chb1n/charms/trusty/ceph/next-amulet-1602
Merge into: lp:~openstack-charmers-archive/charms/trusty/ceph/next
Diff against target: 30 lines (+7/-3)
1 file modified
tests/basic_deployment.py (+7/-3)
To merge this branch: bzr merge lp:~1chb1n/charms/trusty/ceph/next-amulet-1602
Reviewer Review Type Date Requested Status
OpenStack Charmers Pending
Review via email: mp+285940@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 #387 ceph-next for 1chb1n mp285940
    LINT OK: passed

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

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

charm_unit_test #308 ceph-next for 1chb1n mp285940
    UNIT OK: passed

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

134. By Ryan Beisner

No need to not wait for nonexistent nrpe

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

charm_unit_test #309 ceph-next for 1chb1n mp285940
    UNIT OK: passed

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

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

charm_lint_check #389 ceph-next for 1chb1n mp285940
    LINT OK: passed

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

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

charm_amulet_test #158 ceph-next for 1chb1n mp285940
    AMULET OK: passed

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

Preview Diff

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

Subscribers

People subscribed via source and target branches