Merge lp://qastaging/~sylvain-pineau/cdts/html-exporter-unit into lp://qastaging/cdts

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 62
Merged at revision: 62
Proposed branch: lp://qastaging/~sylvain-pineau/cdts/html-exporter-unit
Merge into: lp://qastaging/cdts
Diff against target: 654 lines (+621/-0)
5 files modified
plainbox-provider-canonical-driver-test-suite/data/cdts.css (+359/-0)
plainbox-provider-canonical-driver-test-suite/data/cdts.html (+241/-0)
plainbox-provider-canonical-driver-test-suite/launcher/canonical-driver-test-suite (+1/-0)
plainbox-provider-canonical-driver-test-suite/launcher/canonical-driver-test-suite-cli (+6/-0)
plainbox-provider-canonical-driver-test-suite/units/exporters.pxu (+14/-0)
To merge this branch: bzr merge lp://qastaging/~sylvain-pineau/cdts/html-exporter-unit
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+262775@code.qastaging.launchpad.net

Description of the change

This MR adds the new html template and the corresponding exporter unit.

CLI and GUI launchers are updated to select the new unit

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved

review: Approve
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Sylvain: please add the right meta-data to debian/copyright so that we
know where all those bits come from. Nobody can stop you from shipping
this but it is our responsibility to track the origin of all third
party content.

On Thu, Jun 25, 2015 at 8:49 PM, Sylvain Pineau
<email address hidden> wrote:
> The proposal to merge lp:~sylvain-pineau/cdts/html-exporter-unit into lp:cdts has been updated.
>
> Status: Needs review => Approved
>
> For more details, see:
> https://code.launchpad.net/~sylvain-pineau/cdts/html-exporter-unit/+merge/262775
> --
> You are subscribed to branch lp:cdts.

Revision history for this message
Daniel Manrique (roadmr) wrote :

The attempt to merge lp:~sylvain-pineau/cdts/html-exporter-unit into lp:cdts failed. Below is the output from the failed tests.

[trusty] starting container
[trusty] (timing) 0.07user 0.01system 0:07.26elapsed 1%CPU (0avgtext+0avgdata 7856maxresident)k
[trusty] (timing) 0inputs+32outputs (0major+8465minor)pagefaults 0swaps
[trusty] provisioning container
[trusty] (timing) 46.58user 14.61system 1:17.36elapsed 79%CPU (0avgtext+0avgdata 53532maxresident)k
[trusty] (timing) 0inputs+240outputs (0major+2490973minor)pagefaults 0swaps
[trusty-testing] Starting tests...
Found a test script: ./plainbox-provider-canonical-driver-test-suite/requirements/container-tests-canonical-driver-test-suite
[trusty-testing] container-tests-canonical-driver-test-suite: FAIL
[trusty-testing] stdout: http://paste.ubuntu.com/11775127/
[trusty-testing] stderr:
[trusty-testing] (timing) Command exited with non-zero status 1
[trusty-testing] (timing) 3.21user 0.06system 0:03.30elapsed 99%CPU (0avgtext+0avgdata 25888maxresident)k
[trusty-testing] (timing) 0inputs+88outputs (0major+9069minor)pagefaults 0swaps
[trusty-testing] Destroying container

You are trying to send an empty document, exiting.
trusty-testing doesn't exist

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

You need to wait for plainbox packages to build.

On Thu, Jun 25, 2015 at 9:03 PM, Daniel Manrique
<email address hidden> wrote:
> The proposal to merge lp:~sylvain-pineau/cdts/html-exporter-unit into lp:cdts has been updated.
>
> Status: Approved => Needs review
>
> For more details, see:
> https://code.launchpad.net/~sylvain-pineau/cdts/html-exporter-unit/+merge/262775
> --
> You are subscribed to branch lp:cdts.

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