Merge lp:~canonical-platform-qa/ubuntu-system-tests/sleep-to-settle-system into lp:ubuntu-system-tests

Proposed by Allan LeSage
Status: Work in progress
Proposed branch: lp:~canonical-platform-qa/ubuntu-system-tests/sleep-to-settle-system
Merge into: lp:ubuntu-system-tests
Diff against target: 56 lines (+12/-8)
1 file modified
ubuntu_system_tests/command_line.py (+12/-8)
To merge this branch: bzr merge lp:~canonical-platform-qa/ubuntu-system-tests/sleep-to-settle-system
Reviewer Review Type Date Requested Status
platform-qa-bot continuous-integration Needs Fixing
PS Jenkins bot continuous-integration Approve
Canonical Platform QA Team Pending
Review via email: mp+288025@code.launchpad.net

Commit message

Sleep to let the image settle, fixes some Jenkins errors for arale.

Description of the change

So I've discovered that a lot of the issues I was chasing are due to flaky adb on arale only. This "sleep to settle" does help there, but it's not a guarantee. What do we think about this, are we willing to wait thirty seconds more when running our tests locally?

To post a comment you must log in.
346. By Allan LeSage

Adjust sleep time.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

346. By Allan LeSage

Adjust sleep time.

345. By Allan LeSage

Merge trunk.

344. By Allan LeSage

Extend sleep further.

343. By Allan LeSage

Extend sleeps and tries?

342. By Allan LeSage

Add sleep for system-settling before tests begin.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu_system_tests/command_line.py'
2--- ubuntu_system_tests/command_line.py 2016-03-03 13:05:10 +0000
3+++ ubuntu_system_tests/command_line.py 2016-03-03 21:30:38 +0000
4@@ -426,6 +426,10 @@
5 else:
6 run_cmd.extend(_get_fs_ro_command())
7
8+ run_cmd.extend(
9+ ['--setup-command',
10+ 'echo "Sleeping 30s to let system settle"; sleep 30'])
11+
12 return run_cmd + ['---'] + ssh_cmd
13
14
15@@ -437,15 +441,15 @@
16 # emulate pitti's convenience 'ro-apt-update' script, see
17 # http://anonscm.debian.org/cgit/autopkgtest/autopkgtest.git/commit/?id=a51f26f61c376 # NOQA
18 'echo "Mounting rw to update"; RETRY=0; '
19- 'until mount -o remount,rw / || [ $RETRY -gt 10 ]; do sleep 1; '
20+ 'until mount -o remount,rw / || [ $RETRY -gt 20 ]; do sleep 1; '
21 'RETRY=$(($RETRY+1)); echo "mount attempt failed, trying '
22- 'again..."; done; if [ $RETRY -gt 10 ]; then exit 1; fi; '
23+ 'again..."; done; if [ $RETRY -gt 20 ]; then exit 1; fi; '
24 'echo "apt-get updating"; apt-get update || '
25 '(sleep 15; apt-get update); '
26 'echo "Mounting ro after updating"; RETRY=0; '
27- 'until mount -o remount,ro / || [ $RETRY -gt 10 ]; do '
28+ 'until mount -o remount,ro / || [ $RETRY -gt 20 ]; do '
29 'sleep 1; RETRY=$(($RETRY+1)); echo "mount attempt failed, '
30- 'trying again..."; done; if [ $RETRY -gt 10 ]; then exit 1; fi'
31+ 'trying again..."; done; if [ $RETRY -gt 20 ]; then exit 1; fi'
32 ]
33 return []
34
35@@ -466,9 +470,9 @@
36 return [
37 '--setup-commands',
38 'echo "Mounting rw to install packages"; '
39- 'sync; RETRY=0; until mount -o remount,rw / || [ $RETRY -gt 10 ]; '
40+ 'sync; RETRY=0; until mount -o remount,rw / || [ $RETRY -gt 20 ]; '
41 'do sleep 1; RETRY=$(($RETRY+1)); echo "mount attempt failed, trying '
42- 'again..."; done; if [ $RETRY -gt 10 ]; then exit 1; fi'
43+ 'again..."; done; if [ $RETRY -gt 20 ]; then exit 1; fi'
44 ]
45
46
47@@ -476,9 +480,9 @@
48 return [
49 '--setup-commands',
50 'echo "Mounting ro after installing packages"; '
51- 'sync; RETRY=0; until mount -o remount,ro / || [ $RETRY -gt 10 ]; '
52+ 'sync; RETRY=0; until mount -o remount,ro / || [ $RETRY -gt 20 ]; '
53 'do sleep 1; RETRY=$(($RETRY+1)); echo "mount attempt failed, trying '
54- 'again..."; done; if [ $RETRY -gt 10 ]; then exit 1; fi'
55+ 'again..."; done; if [ $RETRY -gt 20 ]; then exit 1; fi'
56 ]
57
58

Subscribers

People subscribed via source and target branches

to all changes: