Checkbox

Merge lp://staging/~schwuk/checkbox/stress into lp://staging/checkbox

  1. stress
  2. Merge into trunk
Proposed by David Murphy
Status: Rejected
Rejected by: David Murphy
Proposed branch: lp://staging/~schwuk/checkbox/stress
Merge into: lp://staging/checkbox
Diff against target: 23 lines (+7/-1)
2 files modified
debian/control (+1/-1)
tests/cpu.txt (+6/-0)
To merge this branch: bzr merge lp://staging/~schwuk/checkbox/stress
Reviewer Review Type Date Requested Status
Marc Tardif (community) Needs Information
Review via email: mp+20449@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Marc Tardif (cr3) wrote :

What value does the stress test provide in terms of validating the compatibility of Ubuntu with the hardware? In other words, what kind of bug can this uncover in the distro?

review: Needs Information
Revision history for this message
David Murphy (schwuk) wrote :

On Tue, Mar 2, 2010 at 7:50 PM, Marc Tardif <email address hidden> wrote:
> Review: Needs Information
> What value does the stress test provide in terms of validating the compatibility of Ubuntu with the hardware? In other words, what kind of bug can this uncover in the distro?

In the distro? Unknown.

This test has been developed as part of our like-for-like coverage effort.

--
David Murphy <email address hidden>
IRC: schwuk, Skype: schwuk
1024D/09B7D2A8 61BC EAA4 2C68 8FF2 A1FC B803 6D78 8B8C 09B7 D2A8

Revision history for this message
Marc Tardif (cr3) wrote :

The stress test takes 10 minutes to run, which is a lot of time just for the sake of "like-for-like" coverage. Since the test already exists as a package, people can readily run them if necessary but I would not recommend doing this as part of Checkbox.

Revision history for this message
David Murphy (schwuk) wrote :

It's been agreed with Marjo that this test can be excluded.

Unmerged revisions

764. By David Murphy

First pass at stress test.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2010-02-04 17:59:21 +0000
3+++ debian/control 2010-03-02 12:56:16 +0000
4@@ -14,7 +14,7 @@
5 Replaces: hwtest (<< 0.1-0ubuntu12)
6 Provides: hwtest
7 Depends: ${misc:Depends}, ${python:Depends}, debconf, udev
8-Recommends: dpkg, lsb-release, python-apport, python-apt, python-dbus, python-gst0.10
9+Recommends: dpkg, lsb-release, python-apport, python-apt, python-dbus, python-gst0.10, stress
10 Conflicts: hwtest (<< 0.1-0ubuntu12)
11 XB-Python-Version: ${python:Versions}
12 Description: Checkbox System Testing
13
14=== added file 'tests/cpu.txt'
15--- tests/cpu.txt 1970-01-01 00:00:00 +0000
16+++ tests/cpu.txt 2010-03-02 12:56:16 +0000
17@@ -0,0 +1,6 @@
18+plugin: shell
19+name: cpu-stress
20+requires: package.name == 'stress'
21+command: stress --cpu 12 --io 12 --vm 12 --vm-bytes 128M --timeout 10m
22+description: 10 minute CPU stress test.
23+

Subscribers

People subscribed via source and target branches