New Charm Submission for autotest

Bug #1516654 reported by Michael Reed
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Fix Released
Undecided
Unassigned

Bug Description

The propose of this Charm is to provide the autotest framework to execute client tests.

Revision history for this message
Review Queue (review-queue) wrote : AWS Test Results: New Charm Submission for autotest

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

Revision history for this message
Review Queue (review-queue) wrote : LXC Test Results: New Charm Submission for autotest

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

Revision history for this message
Konstantinos Tsakalozos (kos.tsakalozos) wrote :

There are a couple of problems found in the tests of this charm:
1. bundletester is not able to find and deploy the the charm because the deployer-autotest_amd64.yaml (http://bazaar.launchpad.net/~mreed8855/charms/trusty/autotest/trunk/view/head:/tests/deployer_files/deployer-autotest_amd64.yaml ) references a local charm. Instead it should reference the autotest charm available from a public repository something like:
           autotest:
               charm: lp:~<user>/<branch>/autotest

2. The units should be accessed like this: d.sentry.unit['autotest'][0] instead of d.sentry.unit['autotest/0']

You can have a look at the fixes in this branch: https://code.launchpad.net/~kos.tsakalozos/+junk/autotest
Unfortunately, I still couldn't make it to pass the tests.

In terms of functionality the charm is deployable and a couple of actions I tried looked fine. So, to the best of my understanding the charm works but the charm tests need some more work.

Thank you for your effort, almost there.

Changed in charms:
status: New → In Progress
Revision history for this message
Michael Reed (mreed8855) wrote :

I pushed the changes call the charm from cs:~mreed8855/trusty/autotest and I updated the amulet tests to reference units correctly.

Changed in charms:
status: In Progress → New
Revision history for this message
Konstantinos Tsakalozos (kos.tsakalozos) wrote :

Hi Michael.

Thank you for your effort. The charm looks good and the tests run fine (although the 3.5 hours time-out looked a bit scary). Make sure you have the latest version of your charm pushed to cs:~mreed8855/trusty/autotest and we will take it form here :)

Good work and apologies for the delay.

Revision history for this message
Cory Johns (johnsca) wrote :

This has been promulgated and is now available in the store (https://jujucharms.com/autotest/) and deployable as cs:trusty/autotest

Changed in charms:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.