Merge lp://qastaging/~james-page/charms/trusty/mongodb/ch-resync-newton into lp://qastaging/charms/trusty/mongodb

Proposed by James Page
Status: Merged
Merged at revision: 88
Proposed branch: lp://qastaging/~james-page/charms/trusty/mongodb/ch-resync-newton
Merge into: lp://qastaging/charms/trusty/mongodb
Diff against target: 536 lines (+223/-61)
5 files modified
charmhelpers/contrib/python/packages.py (+22/-7)
charmhelpers/core/hookenv.py (+31/-0)
charmhelpers/core/host.py (+159/-53)
charmhelpers/fetch/__init__.py (+8/-0)
charmhelpers/fetch/giturl.py (+3/-1)
To merge this branch: bzr merge lp://qastaging/~james-page/charms/trusty/mongodb/ch-resync-newton
Reviewer Review Type Date Requested Status
Review Queue (community) automated testing Needs Fixing
Ryan Beisner (community) Approve
Review via email: mp+297046@code.qastaging.launchpad.net

Description of the change

Resync charm-helpers to ensure that charm understands the Newton UCA.

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

charm_lint_check #2850 mongodb for james-page mp297046
    LINT OK: passed

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

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

charm_unit_test #2174 mongodb for james-page mp297046
    UNIT OK: passed

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

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

charm_amulet_test #779 mongodb for james-page mp297046
    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/17168378/
Build: http://10.245.162.36:8080/job/charm_amulet_test/779/

Revision history for this message
Ryan Beisner (1chb1n) wrote :

I believe the amulet test failure is an existing problem with the 03_deploy_replicaset.py test logic, https://bugs.launchpad.net/charms/+source/mongodb/+bug/1518468.

00:30:46.446 juju-test.conductor DEBUG : State for 1.25.5: started
00:30:46.447 juju-test.conductor.03_deploy_replicaset.py DEBUG : Running 03_deploy_replicaset.py (tests/03_deploy_replicaset.py)
00:30:47.717 2016-06-10 11:34:08 Starting deployment of osci-sv08
00:30:48.010 2016-06-10 11:34:08 Deploying services...
00:30:48.081 2016-06-10 11:34:08 Deploying service mongodb using /tmp/charmn47kajm4/trusty/mongodb
00:34:24.871 2016-06-10 11:37:45 Adding relations...
00:34:24.950 2016-06-10 11:37:45 Exposing service 'mongodb'
00:34:25.167 2016-06-10 11:37:45 Deployment complete in 217.69 seconds
00:35:05.382 juju-test.conductor.03_deploy_replicaset.py DEBUG : Expected 2 secondary units! (Found 0) [5, 5, 1]
00:35:05.382
00:35:05.382 juju-test.conductor.03_deploy_replicaset.py DEBUG : Got exit code: 1
00:35:05.382 juju-test.conductor.03_deploy_replicaset.py RESULT : FAIL
00:35:05.382 juju-test.conductor INFO : Breaking here as requested by --set-e
00:35:05.382 juju-test INFO : Results: 3 passed, 1 failed, 0 errored
00:35:05.397 make: *** [functional_test] Error 1

review: Approve
Revision history for this message
Review Queue (review-queue) wrote :

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/4675/

review: Needs Fixing (automated testing)

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

to all changes: