Merge lp:~cypressyew/checkbox/tb-hdd-fix into lp:checkbox

Proposed by Po-Hsu Lin
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 3711
Merged at revision: 3711
Proposed branch: lp:~cypressyew/checkbox/tb-hdd-fix
Merge into: lp:checkbox
Diff against target: 11 lines (+1/-0)
1 file modified
providers/plainbox-provider-checkbox/jobs/thunderbolt.pxu (+1/-0)
To merge this branch: bzr merge lp:~cypressyew/checkbox/tb-hdd-fix
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+256765@code.launchpad.net

Description of the change

This is a workaround for the bug 1445419, when I was creating the Thunderbolt storage test case, I removed the requirement for testing this as root, after re-consideration I would like to bring it back.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/jobs/thunderbolt.pxu'
2--- providers/plainbox-provider-checkbox/jobs/thunderbolt.pxu 2015-04-16 07:29:03 +0000
3+++ providers/plainbox-provider-checkbox/jobs/thunderbolt.pxu 2015-04-20 03:34:31 +0000
4@@ -23,6 +23,7 @@
5
6 plugin: shell
7 id: thunderbolt/storage-test
8+user: root
9 imports: from 2013.com.canonical.plainbox import manifest
10 requires: manifest.has_thunderbolt == 'True'
11 depends: thunderbolt/insert

Subscribers

People subscribed via source and target branches