Merge lp://staging/~canonical-platform-qa/ubuntu-ota-tests/adding-initial-autopkgtesting into lp://staging/ubuntu-ota-tests

Proposed by Christopher Lee
Status: Merged
Approved by: Christopher Lee
Approved revision: 6
Merged at revision: 2
Proposed branch: lp://staging/~canonical-platform-qa/ubuntu-ota-tests/adding-initial-autopkgtesting
Merge into: lp://staging/ubuntu-ota-tests
Diff against target: 102 lines (+65/-0)
7 files modified
README.rst (+6/-0)
debian/changelog (+5/-0)
debian/control (+1/-0)
debian/copyright (+21/-0)
debian/docs (+1/-0)
debian/tests/control (+2/-0)
debian/tests/ota_basic (+29/-0)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/ubuntu-ota-tests/adding-initial-autopkgtesting
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Federico Gimenez (community) Approve
Leo Arias (community) Approve
Review via email: mp+251546@code.staging.launchpad.net

Commit message

Initial packaging and autopkgtest setup.

Description of the change

Initial packaging and autopkgtest setup.

To post a comment you must log in.
Revision history for this message
Christopher Lee (veebers) wrote :

This isn't perfect as I'm still trying to get the example to run but I'm having issues with my device.

Revision history for this message
Martin Pitt (pitti) wrote :

Do you actually want to package this? It seems to me this kind of test is more practical to run straight out of bzr. Then you don't need debian/control etc. at all. For that you need "Depends:" in debian/tests/control, as "@" (i. e. "all binaries from that source") is the default.

4. By Christopher Lee

Minimal working example.

5. By Christopher Lee

Update readme with a little information

Revision history for this message
Christopher Lee (veebers) wrote :

Ah right, thanks for the clarification pitti :-)

Revision history for this message
Leo Arias (elopio) wrote :

69 \ No newline at end of file

Other than that, looks good to me. Thanks for getting it started.

review: Approve
Revision history for this message
Federico Gimenez (fgimenez) wrote :

LGTM +1

By the way, how did you get the colors in the test execution output? Or does this come from adt-run? They look nice :) it would be great if they could express info about the results and we could be always in green :)

review: Approve
6. By Christopher Lee

Addition of EOF newline

Revision history for this message
Christopher Lee (veebers) wrote :

Since there are 2 LGTMs(now that I've added the newline) I'm going to go ahead and approve it (will also test the automerge).

Federico, the colouring is all adt-run, just by not logging does it show the colour.

Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

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: