Project

General

Profile

Wiki » History » Version 22

okurz, 2016-05-09 12:40
scenario includes "@machine" now consistently, see https://github.com/os-autoinst/openQA/pull/638

1 3 okurz
# Introduction
2 1 alarrosa
3 3 okurz
This is the organisation wiki for the **openQA Project**.
4 1 alarrosa
5 14 okurz
{{toc}}
6
7 3 okurz
# Organisational
8 1 alarrosa
9 3 okurz
## openQA calls
10
11
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.
12
13
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.
14
15
Standard rules of good "standup meetings" apply here, too, e.g.
16
17
* Be on time (be there at meeting start)
18
* Be concise (help keep the time limit)
19
* Be polite
20
* focus on
21
 * what you achieved
22
 * what you plan
23
 * where did you face problems where you could use help
24
25
26
### "openQA backend coordination" call
27
28
**objectives**:
29
30
* Coordination on openQA backend development
31
32
**execution**: A regular daily call from Mon-Fri at 0900 UTC
33
34
35
### "SUSE QA test coordination" call
36
37
**objectives**:
38
39
* Coordination on openQA based test development, especially SLE products
40
* Information about important development in openQA backend by backend responsibles
41
42
**execution**: Mon + Wed, at 0930 UTC
43
44
If somebody from SUSE QA team will do back-end development he can attend the first call as well, of course.
45
46
47 13 okurz
## ticket templates
48
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.
49
50
51
### defects
52
53
Subject: `<Short description, example: "openQA dies when triggering any Windows ME tests">`
54
55
56
```
57
## observation
58
<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>
59
60
## steps to reproduce
61
* <do this>
62
* <do that>
63
* <observe result>
64
65
## problem
66
<problem investigation, can also include different hypotheses, should be labeled as "H1" for first hypothesis, etc.>
67
68
## suggestion
69
<what to do as a first step>
70
71
## workaround
72
<example: retrigger job>
73
```
74
75
example ticket: #10526
76
77
### feature requests
78
79
Subject: `<Short description, example: "grub3 btrfs support" (feature)>`
80
81
82
```
83
## User story
84
<As a <role>, I want to <do an action>, to <achieve which goal> >
85
86
## acceptance criteria
87
* <**AC1:** the first acceptance criterion that needs to be fulfilled to do this, example: Clicking "restart button" causes restart of the job>
88
* <**AC2:** also think about the "not-actions", example: other jobs are not affected>
89
90
## tasks
91
* <first task to do as an easy starting point>
92
* <what do do next>
93
* <optional: mark "optional" tasks>
94
95
## further details
96 17 okurz
<everything that does not fit into above sections>
97 13 okurz
```
98
99
example ticket: #10212
100
101 4 okurz
# User stories
102
103 7 okurz
## User story 1
104 6 okurz
**User:** QA-Project Managment
105 4 okurz
**primary actor:** QA Project Manager, QA Team Leads
106
**stakeholder:** Directors, VP
107 1 alarrosa
**trigger:** product milestones, providing a daily status
108 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.“
109 1 alarrosa
	
110 7 okurz
## User story 2
111 4 okurz
**User:** openQA-Admin
112
**primary actor:** Backend-Team
113 1 alarrosa
**stakeholder:** Qa-Prjmgr, QA-TL, openQA Tech-Lead
114 4 okurz
**trigger:** Bugs, features, new testcases
115 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.“
116 5 okurz
117 7 okurz
## User story 3
118 1 alarrosa
**User:** QA-Reviewer
119
**primary actor:** QA-Team
120
**stakeholder:** QA-Prjmgr, Release-Mgmt, openQA-Admin
121 4 okurz
**trigger:** every new build
122 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.“
123
124
## User story 4
125 1 alarrosa
**User:** Testcase-Contributor
126
**primary actor:** All development teams, Maintenance QA
127 4 okurz
**stakeholder:** QA-Reviewer, openQA-Admin, openQA Tech-Lead
128 5 okurz
**trigger:** features, new functionality, bugs, new product/package
129 1 alarrosa
**user story:** 4. „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.“
130 7 okurz
131
## User story 5
132 4 okurz
**User:** Release-Mgmt
133
**primary actor:** Release Manager
134
**stakeholder:** Directors, VP, PM, TAMs, Partners
135 1 alarrosa
**trigger:** Milestones
136 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.“
137
138
## User story 6
139 4 okurz
**User:** Staging-Admin
140
**primary actor:** Staging-Manager for the products
141
**stakeholder:** Release-Mgmt, Build-Team
142
**trigger:** every single submission to projects
143 8 okurz
**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.“
144
145 15 okurz
# Glossary
146
147
The following terms are used within the context of openQA:
148
149
 * ***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)
150
 * ***test suite***: a collection of *test modules*, e.g. "textmode". All *test modules* within one *test suite* are run serially
151
 * ***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
152
 * ***test run***: equivalent to *job*
153
 * ***test result***: the result of one job, e.g. "passed" with the details of each individual *test module*
154
 * ***test step***: the execution of one *test module* within a *job*
155
 * ***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")
156
 * ***product***: the main "system under test" (SUT), e.g. "openSUSE"
157
 * ***job group***: equivalent to *product*, used in context of the webUI
158
 * ***version***: one version of a *product*, don't confuse with *builds*, e.g. "Tumbleweed"
159
 * ***flavor***: a specific variant of a *product* to distinguish differing variants, e.g. "DVD"
160 1 alarrosa
 * ***arch***: an architecture variant of a *product*, e.g. "x86_64"
161 22 okurz
 * ***machine***: additional variant of machine, e.g. used for "64bit", "uefi", etc.
162
 * ***scenario***: A composition of `<distri>-<version>-<flavor>-<arch>-<scenario>@<scenario>`, e.g. "openSUSE-Tumbleweed-DVD-x86_64-gnome@64bit", nicknamed *koala*
163 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)
164
165 8 okurz
# Thoughts about categorizing test results, issues, states within openQA
166
by okurz
167
168
When reviewing test results it is important to distinguish between different causes of "failed tests"
169
170
## Nomenclature
171
172
### Test status categories
173 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".
174 1 alarrosa
175 10 okurz
Another common but potentially ambiguous categorization:
176 1 alarrosa
177 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
178
* *failing*: the test is behaving as expected, but the test output is a fail --> "true positive"
179
* *working*: the test is behaving as expected (with no comment regarding the result, though some might ambiguously imply 'result is negative')
180
* *passing*: the test is behaving as expected, but the result is a success --> "true negative"
181
182 8 okurz
If in doubt declare a test as "broken". We should review the test and examine if it is behaving as expected.
183 9 okurz
184 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/
185 8 okurz
186
### Priorization of work regarding categories
187 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".
188 3 okurz
189 11 okurz
## Further categorization of statuses, issues and such in testing, especially automatic tests
190
By okurz
191
192
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.
193
While used for naming it should also be used as a decision tree and can be followed from the top following each branch.
194
195
### Categorization scheme
196
197
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.
198
Every leaf of the tree must have an action assigned to it.
199
200 12 okurz
1 **failed** (ZULU)
201 11 okurz
11 new (passed->failed) (YANKEE)
202
111 product issue ("true positive") (WHISKEY)
203
1111 unfiled issue (SIERRA)
204
11111 hard issue (KILO)
205
111121 critical / potential ship stopper (INDIA) --> immediately file bug report with "ship_stopper?" flag; opt. inform RM directly
206
111122 non-critical hard issue (HOTEL) --> file bug report
207
11112 soft issue (JULIETT) --> file bug report
208
1112 bugzilla bug exists (ROMEO)
209
11121 bug was known to openqa / openqa developer --> cross-reference (bug->test, test->bug) AND raise review process issue, improve openqa process
210
11122 bug was filed by other sources (e.g. beta-tester) --> cross-reference (bug->test, test->bug)
211
112 test issue ("false positive") (VICTOR)
212
1121 progress issue exists (QUEBEC) --> cross-reference (issue->test, test->issue)
213
1122 unfiled test issue (PAPA)
214
11221 easy to do w/o progress issue
215
112211 need needles update --> re-needle if sure, TODO how to notify?
216
112212 pot. flaky, timeout
217
1122121 retrigger yields PASS --> comment in progress about flaky issue fixed
218
1122122 reproducible on retrigger --> file progress issue
219
11222 needs progress issue filed --> file progress issue
220
12 existing / still failing (failed->failed) (XRAY)
221
121 product issue (UNIFORM)
222
1211 unfiled issue (OSCAR) --> file bug report AND raise review process issue (why has it not been found and filed?)
223
1212 bugzilla bug exists (NOVEMBER) --> ensure cross-reference, also see rules for 1112 ROMEO
224
122 test issue (TANGO)
225
1221 progress issue exists (MIKE) --> monitor, if persisting reprioritize test development work
226
1222 needs progress issue filed (LIMA) --> file progress issue AND raise review process issue, see 1211 OSCAR
227 12 okurz
2 **passed** (ALFA)
228 11 okurz
21 stable (passed->passed) (BRAVO)
229
211 existing "true negative" (DELTA) --> monitor, maybe can be made stricter
230
212 existing "false negative" (ECHO) --> needs test improvement
231
22 fixed (failed->passed) (CHARLIE)
232
222 fixed "true negative" (FOXTROTT) --> TODO split monitor, see 211 DELTA
233
2221 was test issue --> close progress issue
234
2222 was product issue
235
22221 no bug report exists --> raise review process issue (why was it not filed?)
236
22222 bug report exists
237
222221 was marked as RESOLVED FIXED
238
221 fixed but "false negative" (GOLF) --> potentially revert test fix, also see 212 ECHO
239
240
241
Priority from high to low: INDIA->OSCAR->HOTEL->JULIETT->…
242
243
### Further decision steps working on test issues
244
245
Test issues could be one of the following sources
246
247
* "accepted product changes"
248
 * product changed slightly but in an acceptable way without the need for communication with DEV+RM --> adapt test
249
 * product changed slightly but in an acceptable way found after feedback from RM --> adapt test
250
 * product changed significantly --> after approval by RM adapt test
251
252
* changes in test setup, e.g. our test hardware equipment behaves different or the network
253
* changes in test infrastructure software, e.g. os-autoinst, openQA
254
* changes in test management configuration, e.g. openQA database settings
255
* changes in the test software itself
256
257 16 okurz
258
# Test reviewing
259
260
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.
261
262
## Show previous results in test results page [gh#538](https://github.com/os-autoinst/openQA/pull/538)
263
264
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".
265
266
Querying the database for former test runs of the same scenario is a rather
267
costly operation which we do not want to do for multiple test results at once
268
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.
269
270
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
271
`http://openqa.opensuse.org/tests/1234?limit_previous=30#previous`
272
273
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.
274
275
Related issue: #10212
276
277
Screenshot of feature:
278
![screenshot_20160210_142024](https://cloud.githubusercontent.com/assets/1693432/12948308/7e915a3c-d001-11e5-840b-2f070c3cb8a5.png)
279
280
281
## Show bug or label icon on overview if labeled [gh#550](https://github.com/os-autoinst/openQA/pull/550)
282
283
* Show bug icon with URL if mentioned in test comments
284
* Show bug or label icon on overview if labeled
285
286
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.
287
288
Example for a generic label:
289
![openqa_generic_label](https://cloud.githubusercontent.com/assets/1693432/13027322/7bce7992-d24a-11e5-99ee-839fb5e82169.png)
290
291
Example for bug label:
292
![openqa_bug_label](https://cloud.githubusercontent.com/assets/1693432/13027323/8555238a-d24a-11e5-83d5-5bb2d2140860.png)
293
294
Related issue: #10212
295
296
297
## Show certificate next to builds on overview if all failures are labeled [gh#560](https://github.com/os-autoinst/openQA/pull/560)
298
299
Based on comments in the individual job results for each build a certificate
300
icon is shown on the group overview page as well as the index page to indicate
301
that every failure has been reviewed, e.g. a bug reference or a test issue
302
reason is stated. Only the failed and incomplete jobs are regarded for the
303
evaluation if a build is considered "reviewed".
304
305
If the badge appears you know you are done for one complete build :-)
306
307
Example screenshot:
308
![openqa_reviewed_label](https://cloud.githubusercontent.com/assets/1693432/13145996/eb1bb78a-d653-11e5-9f0f-40898915578e.png)
309
310
## Allow group overview query by result [gh#531](https://github.com/os-autoinst/openQA/pull/531)
311
312
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".
313
314
Example: Add query parameters like `…&result=failed&arch=x86_64` to show only failed for the single architecture selected.
315 1 alarrosa
316 18 okurz
## Add more query parameters for configuring last builds [gh#575](https://github.com/os-autoinst/openQA/pull/575)
317
318
By using advanced query parameters in the URLs you can configure the search for builds.
319
Higher numbers would yield more complex database queries but can be selected
320
for special investigation use cases with the advanced query parameters, e.g. if one wants to get an overview of a longer history.
321
This applies to both the index dashboard and group overview page.
322
323
Example to show up to three week old builds instead of the default two weeks
324
with up to 20 builds instead of up to 10 being the default for the group
325
overview page:
326
327
    http://openqa/group_overview/1?time_limit_days=21&limit_builds=20
328 16 okurz
329 20 okurz
## Build tagging and keeping important builds [gh#591](https://github.com/os-autoinst/openQA/pull/591)
330 19 okurz
331
### Tag builds with special comments on group overview
332
333
Based on comments on the group overview individual builds can be tagged. As
334
'build' by themselves do not own any data the job group is used to store this
335
information. A tag has a 'build' to link it to a build. It also has a 'type'
336
and an optional 'description'. The type can later on be used to distinguish
337
tag types.
338
339
The more recent tag always wins.
340
341
A 'tag' icon is shown next to tagged builds together with the description on
342
the group_overview page. The index page is not changed to prevent a potential
343
performance regression.
344
345
Within the sub group_overview the comments are parsed for comments and then
346
passed to the template explicitly to prevent duplicate database queries.
347
348
### Keeping important builds
349
350
As builds can now be tagged we come up with the convention that the
351
'important' type - the only one for now - is used to tag every job that
352
corresponds to a build as 'important' and keep the logs for these jobs so that
353
we can always refer to the attached data, e.g. for milestone builds, final
354
releases, jobs for which long-lasting bug reports exist, etc.
355
356
As these jobs are not cleaned up automatically a manual or external cleanup
357
scheme has to be applied for important builds and jobs.
358
359
### Example screenshot of a tag coment and corresponding tagged build
360
![openqa_tag_important](https://cloud.githubusercontent.com/assets/1693432/13468316/4fd8f586-e0a2-11e5-99df-4aa3fb787205.png)
361
362 1 alarrosa
Related issue: #9544
363 21 okurz
364
### Carry over labels from previous jobs in same scenario if still failing  [gh#564](https://github.com/os-autoinst/openQA/pull/564)
365
366
It is possible to label all failing tests but tedious to do by a human user
367
as many failures are just having the same issue until it gets fixed.
368
It helps if a label is preserved for a build that is still failing. This
369
idea is inspired by
370
https://wiki.jenkins-ci.org/display/JENKINS/Claim+plugin
371
372
Does not carry over labels over passes: After a job passed a new issue in a subsequent fail is assumed to be failed
373
for a different reason.
374
375
Related issue: #10212
376 19 okurz
377 3 okurz
# Old content
378
## Sprints
379 2 okurz
380 1 alarrosa
381
[[Sprint 01]]
382
[[Sprint 02]]
383
[[Sprint 03]]