Merge lp:~joetalbott/qa-dashboard/add_variant into lp:qa-dashboard

Proposed by Joe Talbott
Status: Merged
Approved by: Chris Johnston
Approved revision: 193
Merged at revision: 195
Proposed branch: lp:~joetalbott/qa-dashboard/add_variant
Merge into: lp:qa-dashboard
Diff against target: 144 lines (+38/-12)
3 files modified
bootspeed/models.py (+27/-4)
bootspeed/utils.py (+4/-2)
scripts/add_bootspeed_result.py (+7/-6)
To merge this branch: bzr merge lp:~joetalbott/qa-dashboard/add_variant
Reviewer Review Type Date Requested Status
Chris Johnston Approve
Review via email: mp+140308@code.launchpad.net

Commit message

Add 'variant' as a column for CSV data imports.

Description of the change

Add 'variant' as a column for CSV data imports.

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve
Revision history for this message
Chris Johnston (cjohnston) wrote :

The attempt to merge lp:~joetalbott/qa-dashboard/add_variant into lp:qa-dashboard failed. Below is the output from the failed tests.

Creating test database for alias 'default'...
Type 'yes' if you would like to try deleting the test database 'test_qa_dashboard', or 'no' to cancel:
Got an error creating the test database: database "test_qa_dashboard" already exists

Traceback (most recent call last):
  File "manage.py", line 26, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/lib/python2.7/dist-packages/django/core/management/__init__.py", line 429, in execute_from_command_line
    utility.execute()
  File "/usr/lib/python2.7/dist-packages/django/core/management/__init__.py", line 379, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/lib/python2.7/dist-packages/django/core/management/base.py", line 191, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/usr/lib/python2.7/dist-packages/django/core/management/base.py", line 220, in execute
    output = self.handle(*args, **options)
  File "/usr/lib/python2.7/dist-packages/south/management/commands/test.py", line 8, in handle
    super(Command, self).handle(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/django/core/management/commands/test.py", line 37, in handle
    failures = test_runner.run_tests(test_labels)
  File "/usr/lib/python2.7/dist-packages/django/test/simple.py", line 359, in run_tests
    old_config = self.setup_databases()
  File "/usr/lib/python2.7/dist-packages/django/test/simple.py", line 296, in setup_databases
    test_db_name = connection.creation.create_test_db(self.verbosity, autoclobber=not self.interactive)
  File "/usr/lib/python2.7/dist-packages/django/db/backends/creation.py", line 351, in create_test_db
    self._create_test_db(verbosity, autoclobber)
  File "/usr/lib/python2.7/dist-packages/django/db/backends/creation.py", line 431, in _create_test_db
    confirm = raw_input("Type 'yes' if you would like to try deleting the test database '%s', or 'no' to cancel: " % test_database_name)
EOFError: EOF when reading a line

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bootspeed/models.py'
2--- bootspeed/models.py 2012-12-13 02:32:26 +0000
3+++ bootspeed/models.py 2012-12-17 22:24:21 +0000
4@@ -170,6 +170,7 @@
5 result.plumbing,
6 result.xorg,
7 result.desktop,
8+ result.desktop + result.xorg,
9 result.boot,
10 ]
11 result_nums.append(result_data)
12@@ -187,28 +188,32 @@
13 plumbing_std = std_ar[2]
14 xorg_std = std_ar[3]
15 desktop_std = std_ar[4]
16- boot_std = std_ar[5]
17+ xorg_desktop_std = std_ar[5]
18+ boot_std = std_ar[6]
19
20 kernel_avg = avg_ar[0]
21 kernel_init_avg = avg_ar[1]
22 plumbing_avg = avg_ar[2]
23 xorg_avg = avg_ar[3]
24 desktop_avg = avg_ar[4]
25- boot_avg = avg_ar[5]
26+ xorg_desktop_avg = avg_ar[5]
27+ boot_avg = avg_ar[6]
28
29 kernel_min = min_ar[0]
30 kernel_init_min = min_ar[1]
31 plumbing_min = min_ar[2]
32 xorg_min = min_ar[3]
33 desktop_min = min_ar[4]
34- boot_min = min_ar[5]
35+ xorg_desktop_min = min_ar[5]
36+ boot_min = min_ar[6]
37
38 kernel_max = max_ar[0]
39 kernel_init_max = max_ar[1]
40 plumbing_max = max_ar[2]
41 xorg_max = max_ar[3]
42 desktop_max = max_ar[4]
43- boot_max = max_ar[5]
44+ xorg_desktop_max = max_ar[5]
45+ boot_max = max_ar[6]
46
47
48 agg_data, created = ImageMachineAgg.objects.get_or_create(
49@@ -244,6 +249,11 @@
50 desktop_max=desktop_max,
51 desktop_min=desktop_min,
52
53+ xorg_desktop_std=xorg_desktop_std,
54+ xorg_desktop_avg=xorg_desktop_avg,
55+ xorg_desktop_max=xorg_desktop_max,
56+ xorg_desktop_min=xorg_desktop_min,
57+
58 boot_std=boot_std,
59 boot_avg=boot_avg,
60 boot_max=boot_max,
61@@ -288,6 +298,11 @@
62 desktop_avg = models.FloatField(default=0)
63 desktop_std = models.FloatField(default=0)
64
65+ xorg_desktop_min = models.FloatField(default=0)
66+ xorg_desktop_max = models.FloatField(default=0)
67+ xorg_desktop_avg = models.FloatField(default=0)
68+ xorg_desktop_std = models.FloatField(default=0)
69+
70 boot_min = models.FloatField(default=0)
71 boot_max = models.FloatField(default=0)
72 boot_avg = models.FloatField(default=0)
73@@ -315,6 +330,10 @@
74 desktop_min,
75 desktop_max,
76 desktop_std,
77+ xorg_desktop_avg,
78+ xorg_desktop_min,
79+ xorg_desktop_max,
80+ xorg_desktop_std,
81 boot_avg,
82 boot_min,
83 boot_max,
84@@ -343,6 +362,10 @@
85 self.desktop_min = desktop_min
86 self.desktop_max = desktop_max
87 self.desktop_std = desktop_std
88+ self.xorg_desktop_avg = xorg_desktop_avg
89+ self.xorg_desktop_min = xorg_desktop_min
90+ self.xorg_desktop_max = xorg_desktop_max
91+ self.xorg_desktop_std = xorg_desktop_std
92 self.boot_avg = boot_avg
93 self.boot_min = boot_min
94 self.boot_max = boot_max
95
96=== modified file 'bootspeed/utils.py'
97--- bootspeed/utils.py 2012-12-11 02:36:52 +0000
98+++ bootspeed/utils.py 2012-12-17 22:24:21 +0000
99@@ -36,11 +36,11 @@
100 'image_build_number',
101 'image_arch',
102 'image_md5',
103+ 'image_release',
104+ 'image_variant',
105 'ran_at',
106 ]
107
108- logging.debug("fields: {}".format(fields))
109-
110 for field in fields:
111 if field not in data:
112 raise MissingField(
113@@ -62,6 +62,8 @@
114 image, new_image = Image.objects.get_or_create(
115 build_number=data['image_build_number'],
116 arch=data['image_arch'],
117+ release=data['image_release'],
118+ variant=data['image_variant'],
119 md5=data['image_md5'],
120 )
121
122
123=== modified file 'scripts/add_bootspeed_result.py'
124--- scripts/add_bootspeed_result.py 2012-12-11 21:03:37 +0000
125+++ scripts/add_bootspeed_result.py 2012-12-17 22:24:21 +0000
126@@ -25,11 +25,12 @@
127 'desktop': row[4],
128 'boot': row[5],
129 'image_release': row[6],
130- 'image_arch': row[7],
131- 'ran_at': row[8][:19],
132- 'image_build_number': row[9],
133- 'image_md5': row[10],
134- 'machine_name': row[11],
135- 'machine_mac': row[12],
136+ 'image_variant': row[7],
137+ 'image_arch': row[8],
138+ 'ran_at': row[9],
139+ 'image_build_number': row[10],
140+ 'image_md5': row[11],
141+ 'machine_name': row[12],
142+ 'machine_mac': row[13],
143 }
144 add_bootspeed_result(data)

Subscribers

People subscribed via source and target branches