Project

General

Profile

Wiki » History » Version 50

okurz, 2016-12-01 18:58
Add description for the two new variants of the certificate icon (gh#1052)

1 3 okurz
# Introduction
2 1 alarrosa
3 3 okurz
This is the organisation wiki for the **openQA Project**.
4 49 okurz
The source code is hosted in the [os-autoinst github project](http://github.com/os-autoinst/), especially [openQA itself](http://github.com/os-autoinst/openQA) and the main backend [os-autoinst](http://github.com/os-autoinst/os-autoinst)
5 1 alarrosa
6 48 okurz
If you are interested in the tests for SUSE/openSUSE products take a look into the [openqatests](https://progress.opensuse.org/projects/openqatests) project.
7
8 14 okurz
{{toc}}
9
10 3 okurz
# Organisational
11 1 alarrosa
12 3 okurz
## openQA calls
13
14
Currently there are two recurring openQA calls conducted at SUSE on http://jangouts.suse.de/. If there would be more interest from the outside the call could be made on a public platform.
15
16
Both meetings should target to finish in 15 minutes each. If more time is needed, propose to stay in the call with the required subset of attendees.
17
18
Standard rules of good "standup meetings" apply here, too, e.g.
19
20
* Be on time (be there at meeting start)
21
* Be concise (help keep the time limit)
22
* Be polite
23
* focus on
24
 * what you achieved
25
 * what you plan
26
 * where did you face problems where you could use help
27
28
29
### "openQA backend coordination" call
30
31
**objectives**:
32
33
* Coordination on openQA backend development
34
35 39 okurz
**execution**: A regular daily call from Mon-Fri at 1000 CET/CEST
36 3 okurz
37
38
### "SUSE QA test coordination" call
39
40
**objectives**:
41
42
* Coordination on openQA based test development, especially SLE products
43
* Information about important development in openQA backend by backend responsibles
44
45 39 okurz
**execution**: Mon + Wed, at 1030 CET/CEST
46 3 okurz
47
If somebody from SUSE QA team will do back-end development he can attend the first call as well, of course.
48
49 13 okurz
## ticket templates
50
You can use these templates to fill in tickets and further improve them with more detail over time. Copy the code block, paste it into a new issue, replace every block marked with "<…>" with your content or delete if not appropriate.
51
52
53
### defects
54
55
Subject: `<Short description, example: "openQA dies when triggering any Windows ME tests">`
56
57
58
```
59
## observation
60
<description of what can be observed and what the symptoms are, provide links to failing test results and/or put short blocks from the log output here to visualize what is happening>
61
62
## steps to reproduce
63
* <do this>
64
* <do that>
65
* <observe result>
66
67
## problem
68
<problem investigation, can also include different hypotheses, should be labeled as "H1" for first hypothesis, etc.>
69
70
## suggestion
71
<what to do as a first step>
72
73
## workaround
74
<example: retrigger job>
75
```
76
77
example ticket: #10526
78
79
### feature requests
80
81
Subject: `<Short description, example: "grub3 btrfs support" (feature)>`
82
83
84
```
85
## User story
86
<As a <role>, I want to <do an action>, to <achieve which goal> >
87
88
## acceptance criteria
89
* <**AC1:** the first acceptance criterion that needs to be fulfilled to do this, example: Clicking "restart button" causes restart of the job>
90
* <**AC2:** also think about the "not-actions", example: other jobs are not affected>
91
92
## tasks
93
* <first task to do as an easy starting point>
94
* <what do do next>
95
* <optional: mark "optional" tasks>
96
97
## further details
98 17 okurz
<everything that does not fit into above sections>
99 13 okurz
```
100
101
example ticket: #10212
102
103 25 okurz
## pull request handling on github
104
105
As a reviewer of pull requests on github for all related repositories, e.g. https://github.com/os-autoinst/os-autoinst-distri-opensuse/pulls, apply labels in case PRs are open for a longer time and can not be merged so that we keep our backlog clean and know why PRs are blocked.
106
107
* **notready**: Triaged as not ready yet for merging, no (immediate) reaction by the reviewee, e.g. when tests are missing, other scenarios break, only tested for one of SLE/TW
108
* **wip**: Marked by the reviewee itself as "[WIP]" or "[DO-NOT-MERGE]" or similar
109
* **question**: Questions to the reviewee, not answered yet
110
111 28 okurz
# Use cases
112 1 alarrosa
113 40 okurz
The following use cases 1-6 have been defined within a SUSE workshop (others have been defined later) to clarify how different actors work with openQA. Some of them are covered already within openQA quite well, some others are stated as motivation for further feature development.
114 28 okurz
115
## Use case 1
116 6 okurz
**User:** QA-Project Managment
117 4 okurz
**primary actor:** QA Project Manager, QA Team Leads
118 1 alarrosa
**stakeholder:** Directors, VP
119
**trigger:** product milestones, providing a daily status
120 7 okurz
**user story:** „As a QA project manager I want to check on a daily basis the „openQA Dashboard“ to get a summary/an overall status of the „reviewers results“ in order to take the right actions and prioritize tasks in QA accordingly.“
121 1 alarrosa
	
122 28 okurz
## Use case 2
123 4 okurz
**User:** openQA-Admin
124 1 alarrosa
**primary actor:** Backend-Team
125
**stakeholder:** Qa-Prjmgr, QA-TL, openQA Tech-Lead
126 4 okurz
**trigger:** Bugs, features, new testcases
127 7 okurz
**user story:** „As an openQA admin I constantly check in the web-UI the system health and I manage its configuration to ensure smooth operation of the tool.“
128 5 okurz
129 28 okurz
## Use case 3
130 1 alarrosa
**User:** QA-Reviewer
131
**primary actor:** QA-Team
132
**stakeholder:** QA-Prjmgr, Release-Mgmt, openQA-Admin
133 4 okurz
**trigger:** every new build
134 7 okurz
**user story:** „As an openQA-Reviewer at any point in time I review on the webpage of openQA the overall status of a build in order to track and find bugs, because I want to find bugs as early as possible and report them.“
135
136 28 okurz
## Use case 4
137 1 alarrosa
**User:** Testcase-Contributor
138
**primary actor:** All development teams, Maintenance QA
139 4 okurz
**stakeholder:** QA-Reviewer, openQA-Admin, openQA Tech-Lead
140 5 okurz
**trigger:** features, new functionality, bugs, new product/package
141 40 okurz
**user story:** „As developer when there are new features, new functionality, bugs, new product/package in git I contribute my testcases because I want to ensure good quality submissions and smooth product integration.“
142 7 okurz
143 28 okurz
## Use case 5
144 4 okurz
**User:** Release-Mgmt
145
**primary actor:** Release Manager
146
**stakeholder:** Directors, VP, PM, TAMs, Partners
147 1 alarrosa
**trigger:** Milestones
148 7 okurz
**user story:** „As a Release-Manager on a daily basis I check on a dashboard for the product health/build status in order to act early in case of failures and have concrete and current reports.“
149
150 28 okurz
## Use case 6
151 4 okurz
**User:** Staging-Admin
152
**primary actor:** Staging-Manager for the products
153
**stakeholder:** Release-Mgmt, Build-Team
154 1 alarrosa
**trigger:** every single submission to projects
155
**user story:** „As a Staging-Manager I review the build status of packages with every staged submission to the „staging projects“ in the „staging dashboard“ and the test-status of the pre-integrated fixes, because I want to identify major breakage before integration to the products and provide fast feedback back to the development.“
156 40 okurz
157
## Use case 7
158
**User:** Bug investigator
159
**primary actor:** Any bug assignee for openQA observed bugs
160
**stakeholder:** Developer
161
**trigger:** bugs
162
**user story:** „As a developer that has been assigned a bug which has been observed in openQA I can review referenced tests, find a newer and the most recent job in the same scenario, understand what changed since the last successful job, what other jobs show same symptoms to investigate the root cause fast and use openQA for verification of a bug fix.“
163 8 okurz
164 15 okurz
# Glossary
165
166
The following terms are used within the context of openQA:
167
168
 * ***test modules***: an individual test case in a single perl module file, e.g. "sshxterm". If not further specified a test module is denoted with its "short name" equivalent to the filename including the test definition. The "full name" is composed of the *test group* (TBC), which itself is formed by the top-folder of the test module file, and the short name, e.g. "x11-sshxterm" (for x11/sshxterm.pm)
169
 * ***test suite***: a collection of *test modules*, e.g. "textmode". All *test modules* within one *test suite* are run serially
170
 * ***job***: one run of individual test cases in a row denoted by a unique number for one instance of openQA, e.g. one installation with subsequent testing of applications within gnome
171
 * ***test run***: equivalent to *job*
172
 * ***test result***: the result of one job, e.g. "passed" with the details of each individual *test module*
173
 * ***test step***: the execution of one *test module* within a *job*
174
 * ***distri***: a test distribution but also sometimes referring to a *product* (CAUTION: ambiguous, historically a "GNU/Linux distribution"), composed of multiple ***test modules*** in a folder structure that compose ***test suites***, e.g. "opensuse" (test distribution, short for "os-autoinst-distri-opensuse")
175
 * ***product***: the main "system under test" (SUT), e.g. "openSUSE"
176
 * ***job group***: equivalent to *product*, used in context of the webUI
177
 * ***version***: one version of a *product*, don't confuse with *builds*, e.g. "Tumbleweed"
178
 * ***flavor***: a specific variant of a *product* to distinguish differing variants, e.g. "DVD"
179 1 alarrosa
 * ***arch***: an architecture variant of a *product*, e.g. "x86_64"
180 22 okurz
 * ***machine***: additional variant of machine, e.g. used for "64bit", "uefi", etc.
181 30 okurz
 * ***scenario***: A composition of `<distri>-<version>-<flavor>-<arch>-<test_suite>@<machine>`, e.g. "openSUSE-Tumbleweed-DVD-x86_64-gnome@64bit", nicknamed *koala*
182 15 okurz
 * ***build***: Different versions of a product as tested, can be considered a "sub-version" of *version*, e.g. "Build1234"; CAUTION: ambiguity: either with the prefix "Build" included or not)
183
184 8 okurz
# Thoughts about categorizing test results, issues, states within openQA
185
by okurz
186
187
When reviewing test results it is important to distinguish between different causes of "failed tests"
188
189
## Nomenclature
190
191
### Test status categories
192 10 okurz
A common definition about the status of a test regarding the product it tests: "false|true positive|negative" as described on https://en.wikipedia.org/wiki/False_positives_and_false_negatives. "positive|negative" describes the outcome of a test ("positive": PASSED; "negative": FAILED) whereas "false|true" describes the conclusion of the test regarding the presence of issues in the SUT or product in our case ("true": correct reporting; "false": incorrect reporting), e.g. "true negative", test successful, no issues detected and there are no issues, product is working as expected by customer. Another example: Think of testing as of a fire alarm. An alarm (event detector) should only go off (be "positive") *if* there is a fire (event to detect) --> "true positive" whereas *if* there is *no* fire there should be *no* alarm --> "true negative".
193 1 alarrosa
194 10 okurz
Another common but potentially ambiguous categorization:
195 1 alarrosa
196 10 okurz
* *broken*: the test is not behaving as expected (Ambiguity: "as expected" by whom?) --> commonly a "false positive", can also be "false negative" but hard to detect
197
* *failing*: the test is behaving as expected, but the test output is a fail --> "true positive"
198
* *working*: the test is behaving as expected (with no comment regarding the result, though some might ambiguously imply 'result is negative')
199
* *passing*: the test is behaving as expected, but the result is a success --> "true negative"
200
201 8 okurz
If in doubt declare a test as "broken". We should review the test and examine if it is behaving as expected.
202 9 okurz
203 10 okurz
Be careful about "positive/negative" as some might also use "positive" to incorrectly denote a passing test (and "negative" for failing test) as an indicator of "working product" not an indicator about "issue present". If you argue what is "used in common speech" think about how "false positive" is used as in "false alarm" --> "positive" == "alarm raised", also see https://narainko.wordpress.com/2012/08/26/understanding-false-positive-and-false-negative/
204 8 okurz
205
### Priorization of work regarding categories
206 10 okurz
In this sense development+QA want to accomplish a "true negative" state whenever possible (no issues present, therefore none detected). As QA and test developers we want to prevent "false positives" ("false alarms" declaring a product as broken when it is not but the test failed for other reasons), also known as "type I error" and "false negatives" (a product issue is not catched by tests and might "slip through" QA and at worst is only found by an external outside customer) also known as "type II error". Also see https://en.wikipedia.org/wiki/Type_I_and_type_II_errors. In the context of openQA and system testing paired with screen matching a "false positive" is much more likely as the tests are very susceptible to subtle variations and changes even if they should be accepted. So when in doubt, create an issue in progress, look at it again, and find that it was a false alarm, rather than wasting more peoples time with INVALID bug reports by believing the product to be broken when it isn't. To quote Richard Brown: "I […] believe this is the route to ongoing improvement - if we have tests which produce such false alarms, then that is a clear indicator that the test needs to be reworked to be less ambiguous, and that IS our job as openQA developers to deal with".
207 3 okurz
208 11 okurz
## Further categorization of statuses, issues and such in testing, especially automatic tests
209
By okurz
210
211
This categorization scheme is meant to help in communication in either written or spoken discussions being simple, concise, easy to remember while unambiguous in every case.
212
While used for naming it should also be used as a decision tree and can be followed from the top following each branch.
213
214
### Categorization scheme
215
216
To keep it simple I will try to go in steps of deciding if a potential issue is of one of two categories in every step (maybe three) and go further down from there. The degree of further detailing is not limited, i.e. it can be further extended. Naming scheme should follow arabic number (for two levels just 1 and 2) counting schemes added from the right for every additional level of decision step and detail without any separation between the digits, e.g. "1111" for the first type in every level of detail up to level four. Also, I am thinking of giving the fully written form phonetic name to unambiguously identify each on every level as long as not more individual levels are necessary. The alphabet should be reserved for higher levels and higher priority types.
217
Every leaf of the tree must have an action assigned to it.
218
219 12 okurz
1 **failed** (ZULU)
220 11 okurz
11 new (passed->failed) (YANKEE)
221
111 product issue ("true positive") (WHISKEY)
222
1111 unfiled issue (SIERRA)
223 44 okurz
11111 hard issue (openqa *fail*) (KILO)
224 11 okurz
111121 critical / potential ship stopper (INDIA) --> immediately file bug report with "ship_stopper?" flag; opt. inform RM directly
225
111122 non-critical hard issue (HOTEL) --> file bug report
226 44 okurz
11112 soft issue (openqa *softfail* on job level, not on module level) (JULIETT) --> file bug report on failing test module
227 11 okurz
1112 bugzilla bug exists (ROMEO)
228
11121 bug was known to openqa / openqa developer --> cross-reference (bug->test, test->bug) AND raise review process issue, improve openqa process
229
11122 bug was filed by other sources (e.g. beta-tester) --> cross-reference (bug->test, test->bug)
230
112 test issue ("false positive") (VICTOR)
231
1121 progress issue exists (QUEBEC) --> cross-reference (issue->test, test->issue)
232
1122 unfiled test issue (PAPA)
233
11221 easy to do w/o progress issue
234
112211 need needles update --> re-needle if sure, TODO how to notify?
235
112212 pot. flaky, timeout
236
1122121 retrigger yields PASS --> comment in progress about flaky issue fixed
237
1122122 reproducible on retrigger --> file progress issue
238
11222 needs progress issue filed --> file progress issue
239
12 existing / still failing (failed->failed) (XRAY)
240
121 product issue (UNIFORM)
241
1211 unfiled issue (OSCAR) --> file bug report AND raise review process issue (why has it not been found and filed?)
242
1212 bugzilla bug exists (NOVEMBER) --> ensure cross-reference, also see rules for 1112 ROMEO
243
122 test issue (TANGO)
244
1221 progress issue exists (MIKE) --> monitor, if persisting reprioritize test development work
245
1222 needs progress issue filed (LIMA) --> file progress issue AND raise review process issue, see 1211 OSCAR
246 12 okurz
2 **passed** (ALFA)
247 11 okurz
21 stable (passed->passed) (BRAVO)
248
211 existing "true negative" (DELTA) --> monitor, maybe can be made stricter
249
212 existing "false negative" (ECHO) --> needs test improvement
250
22 fixed (failed->passed) (CHARLIE)
251
222 fixed "true negative" (FOXTROTT) --> TODO split monitor, see 211 DELTA
252
2221 was test issue --> close progress issue
253
2222 was product issue
254
22221 no bug report exists --> raise review process issue (why was it not filed?)
255
22222 bug report exists
256
222221 was marked as RESOLVED FIXED
257
221 fixed but "false negative" (GOLF) --> potentially revert test fix, also see 212 ECHO
258
259
260
Priority from high to low: INDIA->OSCAR->HOTEL->JULIETT->…
261
262
### Further decision steps working on test issues
263
264
Test issues could be one of the following sources
265
266
* "accepted product changes"
267
 * product changed slightly but in an acceptable way without the need for communication with DEV+RM --> adapt test
268
 * product changed slightly but in an acceptable way found after feedback from RM --> adapt test
269
 * product changed significantly --> after approval by RM adapt test
270
271
* changes in test setup, e.g. our test hardware equipment behaves different or the network
272
* changes in test infrastructure software, e.g. os-autoinst, openQA
273
* changes in test management configuration, e.g. openQA database settings
274 41 okurz
* changes in the test software itself (the test plan in source code as well as needles)
275
* if none of the above it could be a hidden sporadic issue, i.e. the root problem is already hidden in the system for a long time but does not show symptoms every time
276 11 okurz
277 35 okurz
# Advanced features in openQA
278 16 okurz
279
There are some features in openQA for reviewing test results and common practices. Some of these features are presented here based on the pull requests from github.
280
281
## Show previous results in test results page [gh#538](https://github.com/os-autoinst/openQA/pull/538)
282
283
On a tests result page there is a tab for "previous results" showing the result of test runs in the same scenario. This shows previous builds as well as test runs in the same build. This way you can easily check and compare results from before including any comments, labels, bug references (see next section). This helps to answer questions like "Is this a new issue", "Is it reproducable", "has it been seen in before", "how does the history look like".
284
285
Querying the database for former test runs of the same scenario is a rather
286
costly operation which we do not want to do for multiple test results at once
287
but only for each individual test result (1:1 relation). This is why this is done in each individual test result and not for a complete build.
288
289
The evaluation of previous jobs is limited but can be adjusted with the query parameter `limit_previous=<nr>` in the test URL, e.g. to provide a link to the tab in the results page showing the previous 30 results of test 1234 on openqa.opensuse.org you would write
290
`http://openqa.opensuse.org/tests/1234?limit_previous=30#previous`
291
292
Remember that the higher the limit, the more complex the database queries will be increasing the lookup time as well as the load on openQA to generate the result.
293
294
Related issue: #10212
295
296
Screenshot of feature:
297
![screenshot_20160210_142024](https://cloud.githubusercontent.com/assets/1693432/12948308/7e915a3c-d001-11e5-840b-2f070c3cb8a5.png)
298
299 36 okurz
## Link to latest in scenario name [gh#836](https://github.com/os-autoinst/openQA/pull/836)
300
301
Find the always latest job in a scenario with the link after the scenario name in the tab "Previous results"
302
Screenshot:
303
![openqa_link_to_latest_in_previous](https://cloud.githubusercontent.com/assets/1693432/18145393/5b5fb544-6fcb-11e6-967b-f24ffc6a498c.png)
304
305
306
307 34 okurz
## Add 'latest' query route [gh#815](https://github.com/os-autoinst/openQA/pull/815)
308
309
Should always refer to most recent job for the specified scenario.
310
311
* have the same link for test development, i.e. if one retriggers tests, the
312
person has to always update the URL. If there would be a static URL even the
313
browser can be instructed to reload the page automatically
314
315
* for linking to the always current execution of the last job within one
316
scenario, e.g. to respond faster to the standard question in bug reports "does
317
this bug still happen?"
318
319
Examples:
320
321
* `tests/latest?distri=opensuse&version=13.1&flavor=DVD&arch=x86_64&test=kde&machine=64bit`
322
* `tests/latest?flavor=DVD&arch=x86_64&test=kde`
323
* `tests/latest?test=foobar` - this searches for the most recent job using test_suite 'foobar' covering all distri, version, flavor, arch, machines. To be more specific, add the other query entries.
324
325 33 okurz
## Add web UI controls to select 20/50/100/400 previous results [gh#744](https://github.com/os-autoinst/openQA/pull/744)
326
327
The query parameter 'limit_previous' allows to show more than the default 10
328
previous results on demand for some time. There are web UI
329
selections below the table of the previous build to reload the same page with
330
higher number of previous results on demand.
331
332
Example screenshot:
333
![openqa_limit_previous_results_gui_100percent_padded](https://cloud.githubusercontent.com/assets/1693432/16642470/7f3cf080-440b-11e6-84b2-0485b2fd1810.png)
334
335 16 okurz
## Show bug or label icon on overview if labeled [gh#550](https://github.com/os-autoinst/openQA/pull/550)
336
337
* Show bug icon with URL if mentioned in test comments
338
* Show bug or label icon on overview if labeled
339
340
For bugreferences write `<bugtracker_shortname>#<bug_nr>` in a comment, e.g. "bsc#1234", for generic labels use `label:<keyword>` where `<keyword>` can be any valid character up to the next whitespace, e.g. "false_positive". The keywords are not defined within openQA itself. A valid list of keywords should be decided upon within each project or environment of one openQA instance.
341
342
Example for a generic label:
343
![openqa_generic_label](https://cloud.githubusercontent.com/assets/1693432/13027322/7bce7992-d24a-11e5-99ee-839fb5e82169.png)
344
345
Example for bug label:
346
![openqa_bug_label](https://cloud.githubusercontent.com/assets/1693432/13027323/8555238a-d24a-11e5-83d5-5bb2d2140860.png)
347
348 1 alarrosa
Related issue: #10212
349 16 okurz
350 42 okurz
Hint: You can also write (or copy-paste) full links to bugs and issues. The links are automatically changed to the shortlinks (e.g. https://progress.opensuse.org/issues/11110 turns into [poo#11110](https://progress.opensuse.org/issues/11110)). Related issue: #11110
351 16 okurz
352 43 okurz
Also github pull requests and issues can be linked using the generic format
353
`<marker>[#<project/repo>]#<id>`, e.g. [gh#os-autoinst/openQA#1234](https://github.com/os-autoinst/openQA/issues/1234), see [gh#973](https://github.com/os-autoinst/openQA/pull/973)
354
355 50 okurz
## Show certificate next to builds on overview if all failures are labeled [gh#560](https://github.com/os-autoinst/openQA/pull/560), [gh#1052](https://github.com/os-autoinst/openQA/pull/1052)
356 16 okurz
357
Based on comments in the individual job results for each build a certificate
358
icon is shown on the group overview page as well as the index page to indicate
359
that every failure has been reviewed, e.g. a bug reference or a test issue
360
reason is stated. Only the failed and incomplete jobs are regarded for the
361
evaluation if a build is considered "reviewed".
362
363 1 alarrosa
If the badge appears you know you are done for one complete build :-)
364 50 okurz
365
There are two variants of the certificates, a grey one after all failed jobs have been labeled and a black one if all softfailed (if any) have been reviewed additionally.
366
367
So earn the grey one first and aim for the black one :-)
368 16 okurz
369
Example screenshot:
370
![openqa_reviewed_label](https://cloud.githubusercontent.com/assets/1693432/13145996/eb1bb78a-d653-11e5-9f0f-40898915578e.png)
371
372
## Allow group overview query by result [gh#531](https://github.com/os-autoinst/openQA/pull/531)
373
374
This allows e.g. to show only failed builds. Could be included like in http://lists.opensuse.org/opensuse-factory/2016-02/msg00018.html for "known defects".
375
376
Example: Add query parameters like `…&result=failed&arch=x86_64` to show only failed for the single architecture selected.
377 1 alarrosa
378 31 okurz
## Add web UI controls to select more builds in group_overview [gh#804](https://github.com/os-autoinst/openQA/pull/804)
379
380
The query parameter 'limit_builds' allows to show more than the default 10
381
builds on demand. Just like we have for configuring previous results, the
382
current commit adds web UI selections to reload the same page with
383
higher number of builds on demand. For this, the limit of days is increased
384
to show more builds but still limited by the selected number.
385
386
Example screenshot:
387
![openqa_limit_builds_current_bold](https://cloud.githubusercontent.com/assets/1693432/17462279/59e344e6-5ca8-11e6-8350-42a0fbb5267d.png)
388
389
390 18 okurz
## Add more query parameters for configuring last builds [gh#575](https://github.com/os-autoinst/openQA/pull/575)
391
392
By using advanced query parameters in the URLs you can configure the search for builds.
393
Higher numbers would yield more complex database queries but can be selected
394
for special investigation use cases with the advanced query parameters, e.g. if one wants to get an overview of a longer history.
395
This applies to both the index dashboard and group overview page.
396
397
Example to show up to three week old builds instead of the default two weeks
398
with up to 20 builds instead of up to 10 being the default for the group
399
overview page:
400
401
    http://openqa/group_overview/1?time_limit_days=21&limit_builds=20
402 16 okurz
403 20 okurz
## Build tagging and keeping important builds [gh#591](https://github.com/os-autoinst/openQA/pull/591)
404 19 okurz
405
### Tag builds with special comments on group overview
406
407
Based on comments on the group overview individual builds can be tagged. As
408
'build' by themselves do not own any data the job group is used to store this
409 45 okurz
information. A tag has a `build` to link it to a build. It also has a `type`
410
and an optional `description`. The type can later on be used to distinguish
411 1 alarrosa
tag types.
412 45 okurz
413
The generic format for tags is
414
`tag:<build_id>:<type>[:<description>]`, e.g. `tag:1234:important:Beta1`.
415 19 okurz
416
The more recent tag always wins.
417
418
A 'tag' icon is shown next to tagged builds together with the description on
419
the group_overview page. The index page is not changed to prevent a potential
420
performance regression.
421
422
Within the sub group_overview the comments are parsed for comments and then
423
passed to the template explicitly to prevent duplicate database queries.
424
425
### Keeping important builds
426
427
As builds can now be tagged we come up with the convention that the
428
'important' type - the only one for now - is used to tag every job that
429
corresponds to a build as 'important' and keep the logs for these jobs so that
430
we can always refer to the attached data, e.g. for milestone builds, final
431
releases, jobs for which long-lasting bug reports exist, etc.
432
433
As these jobs are not cleaned up automatically a manual or external cleanup
434
scheme has to be applied for important builds and jobs.
435
436
### Example screenshot of a tag coment and corresponding tagged build
437
![openqa_tag_important](https://cloud.githubusercontent.com/assets/1693432/13468316/4fd8f586-e0a2-11e5-99df-4aa3fb787205.png)
438
439 1 alarrosa
Related issue: #9544
440
441 32 okurz
## Add web UI controls to filter only tagged or all builds [gh#807](https://github.com/os-autoinst/openQA/pull/807)
442
443
Using a new query parameter 'only_tagged=[0|1]' the list can be filtered, e.g. show only tagged (important) builds.
444
445
Example screenshot:
446
![openqa_limit_builds_current_bold_and_only_tagged](https://cloud.githubusercontent.com/assets/1693432/17464792/49bb6b18-5ce7-11e6-8053-7b74faf193a7.png)
447
448
Related issue: #11052
449
450 27 okurz
## Carry over labels from previous jobs in same scenario if still failing [gh#564](https://github.com/os-autoinst/openQA/pull/564)
451 21 okurz
452
It is possible to label all failing tests but tedious to do by a human user
453
as many failures are just having the same issue until it gets fixed.
454
It helps if a label is preserved for a build that is still failing. This
455
idea is inspired by
456
https://wiki.jenkins-ci.org/display/JENKINS/Claim+plugin
457
458
Does not carry over labels over passes: After a job passed a new issue in a subsequent fail is assumed to be failed
459
for a different reason.
460 1 alarrosa
461
Related issue: #10212
462 23 okurz
463
464 27 okurz
## Distinguish product and test issues bugref [gh#708](https://github.com/os-autoinst/openQA/pull/708)
465 26 okurz
466
"progress" is used to track test issues, bugzilla for product issues, at least for SUSE/openSUSE. openQA bugrefs distinguish this and show corresponding icons
467
468
![different_bug_icons](https://cloud.githubusercontent.com/assets/1693432/15814910/e4e74bf6-2bc9-11e6-83de-20f18a7494de.png)
469
470 37 mkittler
## Pinning comments as group description
471
This is possible by adding the keyword `pinned-description` anywhere in a comment on the group overview page. Then the comment will be shown at the top of the group overview page. However, it only works as operator or admin.
472
473 38 mkittler
## Filtering test results in test result overview
474 46 asmorodskyi
At the top of the test results overview page is a form which allows filtering tests by result, architecture and TODO-status.
475 38 mkittler
![screenshot_20160909_130610](https://cloud.githubusercontent.com/assets/10248953/18385016/3f966b36-768e-11e6-8ee3-fa48dcd0d31d.png)
476
477 23 okurz
## Proposals for uses of labels
478
With [Show bug or label icon on overview if labeled (gh#550)](https://github.com/os-autoinst/openQA/pull/550) it is possible to add custom labels just by writing them. Nevertheless, a convention should be found for a common benefit. Beware that labels are also automatically carried over with (Carry over labels from previous jobs in same scenario if still failing [gh#564])(https://github.com/os-autoinst/openQA/pull/564) which might make consistent test failures less visible when reviewers only look for test results without labels or bugrefs.
479
480
List of proposed labels with their meaning and where they could be applied.
481
482
* ***`fixed_<build_ref>`***: If a test failure is already fixed in a more recent build and no bug reference is known, use this label together with a reference to a more recent passed test run in the same scenario. Useful for reviewing older builds. Example (https://openqa.suse.de/tests/382518#comments):
483
484
```
485
label:fixed_Build1501
486
487
t#382919
488
```
489
490 24 okurz
* ***`needles_added`***: In case needles were missing for test changes or expected product changes caused needle matching to fail, use this label with a reference to the test PR or a proper reasoning why the needles were missing and how you added them. Example (https://openqa.suse.de/tests/388521#comments):
491
492
```
493
label:needles_added
494
495
needles for https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/1353 were missing, added by jpupava in the meantime.
496
```