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

Proposed by Ryan Beisner on 2016-02-13
Status: Merged
Merged at revision: 105
Proposed branch: lp:~1chb1n/charms/trusty/openstack-dashboard/next-amulet-1602
Merge into: lp:~openstack-charmers-archive/charms/trusty/openstack-dashboard/next
Diff against target: 35 lines (+7/-3)
1 file modified
tests/basic_deployment.py (+7/-3)
To merge this branch: bzr merge lp:~1chb1n/charms/trusty/openstack-dashboard/next-amulet-1602
Reviewer Review Type Date Requested Status
David Ames 2016-02-13 Approve on 2016-02-17
Review via email: mp+285955@code.launchpad.net

Description of the Change

Update amulet test targets; Wait for workload status before testing.

...

FYI - The remaining test targets are not yet enabled here because:

# haproxy not listening on *-Mitaka
https://bugs.launchpad.net/charms/+source/openstack-dashboard/+bug/1545217

# apache2 not listening on Wily-Liberty
https://bugs.launchpad.net/charms/+source/openstack-dashboard/+bug/1546209

To post a comment you must log in.

charm_lint_check #456 openstack-dashboard-next for 1chb1n mp285955
    LINT FAIL: lint-test failed

LINT Results (max last 2 lines):
make: *** [lint] Error 1
ERROR:root:Make target returned non-zero.

Full lint test output: http://paste.ubuntu.com/15032322/
Build: http://10.245.162.36:8080/job/charm_lint_check/456/

charm_unit_test #376 openstack-dashboard-next for 1chb1n mp285955
    UNIT OK: passed

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

charm_amulet_test #173 openstack-dashboard-next for 1chb1n mp285955
    AMULET FAIL: amulet-test failed

AMULET Results (max last 2 lines):
make: *** [functional_test] Error 1
ERROR:root:Make target returned non-zero.

Full amulet test output: http://paste.ubuntu.com/15033326/
Build: http://10.245.162.36:8080/job/charm_amulet_test/173/

105. By Ryan Beisner on 2016-02-13

Disable Mitaka re: lp1545217

106. By Ryan Beisner on 2016-02-13

Fix lint

charm_lint_check #457 openstack-dashboard-next for 1chb1n mp285955
    LINT OK: passed

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

charm_unit_test #377 openstack-dashboard-next for 1chb1n mp285955
    UNIT OK: passed

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

charm_amulet_test #174 openstack-dashboard-next for 1chb1n mp285955
    AMULET FAIL: amulet-test failed

AMULET Results (max last 2 lines):
make: *** [functional_test] Error 1
ERROR:root:Make target returned non-zero.

Full amulet test output: http://paste.ubuntu.com/15033887/
Build: http://10.245.162.36:8080/job/charm_amulet_test/174/

107. By Ryan Beisner on 2016-02-16

Disable Wily test re: bug 1545217

charm_lint_check #658 openstack-dashboard-next for 1chb1n mp285955
    LINT OK: passed

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

charm_unit_test #568 openstack-dashboard-next for 1chb1n mp285955
    UNIT OK: passed

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

charm_amulet_test #265 openstack-dashboard-next for 1chb1n mp285955
    AMULET OK: passed

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

David Ames (thedac) wrote :

Looks good. Merging.

review: Approve

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/basic_deployment.py'
3--- tests/basic_deployment.py 2015-09-12 10:58:20 +0000
4+++ tests/basic_deployment.py 2016-02-16 17:12:43 +0000
5@@ -2,7 +2,6 @@
6
7 import amulet
8 import os
9-import time
10 import urllib2
11 import yaml
12
13@@ -35,6 +34,13 @@
14 self._add_relations()
15 self._configure_services()
16 self._deploy()
17+
18+ u.log.info('Waiting on extended status checks...')
19+ exclude_services = ['mysql']
20+
21+ # Wait for deployment ready msgs, except exclusions
22+ self._auto_wait_for_status(exclude_services=exclude_services)
23+
24 self._initialize_tests()
25
26 def _add_services(self):
27@@ -139,8 +145,6 @@
28 self._get_openstack_release()))
29 u.log.debug('openstack release str: {}'.format(
30 self._get_openstack_release_string()))
31- # Let things settle a bit before moving forward
32- time.sleep(30)
33
34 # NOTE(beisner): Switch to helper once the rabbitmq test refactor lands.
35 def crude_py_parse(self, file_contents, expected):

Subscribers

People subscribed via source and target branches