Juju Charms Collection
rabbitmq-server package

Merge lp://staging/~mbruzek/charms/trusty/rabbitmq-server/add-tests into lp://staging/charms/trusty/rabbitmq-server

  1. Trusty Tahr (14.04)
  2. add-tests
  3. Merge into trunk
Proposed by Matt Bruzek
Status: Merged
Merged at revision: 76
Proposed branch: lp://staging/~mbruzek/charms/trusty/rabbitmq-server/add-tests
Merge into: lp://staging/charms/trusty/rabbitmq-server
Diff against target: 15 lines (+5/-1)
1 file modified
Makefile (+5/-1)
To merge this branch: bzr merge lp://staging/~mbruzek/charms/trusty/rabbitmq-server/add-tests
Reviewer Review Type Date Requested Status
José Antonio Rey (community) charmers Approve
Review via email: mp+246173@code.staging.launchpad.net

Description of the change

Please consider adding these tests to the trusty version of the rabbitmq-server charm

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

charm_unit_test #712 trusty-rabbitmq-server for mbruzek mp246173
    UNIT OK: passed

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

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

charm_lint_check #683 trusty-rabbitmq-server for mbruzek mp246173
    LINT OK: passed

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

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

charm_amulet_test #869 trusty-rabbitmq-server for mbruzek mp246173
    AMULET FAIL: amulet-test failed

AMULET Results (max last 2 lines):
  ERROR subprocess encountered error code 1
  make: *** [functional_test] Error 1

Full amulet test output: http://paste.ubuntu.com/9719168/
Build: http://10.245.162.77:8080/job/charm_amulet_test/869/

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

Oops. It appears that the Makefile was updated, but the amulet tests weren't added.

Revision history for this message
Matt Bruzek (mbruzek) wrote :

Good catch thanks for checking, sorry about that problem.

Revision history for this message
José Antonio Rey (jose) wrote :

Hey Matt,

I have ran the tests with bundletester and found no errors at all. Thank you for your work on these!

+1

review: Approve (charmers)

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