Project

General

Profile

Wiki » History » Version 16

okurz, 2016-02-24 15:14
Add help for reviewing tests based on recent PRs by me

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
<verything that does not fit into above sections>
97
```
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
146
# Glossary
147
148
The following terms are used within the context of openQA:
149
150
 * ***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)
151
 * ***test suite***: a collection of *test modules*, e.g. "textmode". All *test modules* within one *test suite* are run serially
152
 * ***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
153
 * ***test run***: equivalent to *job*
154
 * ***test result***: the result of one job, e.g. "passed" with the details of each individual *test module*
155
 * ***test step***: the execution of one *test module* within a *job*
156
 * ***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")
157
 * ***product***: the main "system under test" (SUT), e.g. "openSUSE"
158
 * ***job group***: equivalent to *product*, used in context of the webUI
159
 * ***version***: one version of a *product*, don't confuse with *builds*, e.g. "Tumbleweed"
160
 * ***flavor***: a specific variant of a *product* to distinguish differing variants, e.g. "DVD"
161
 * ***arch***: an architecture variant of a *product*, e.g. "x86_64"
162
 * ***scenario***: A composition of `<distri>-<version>-<flavor>-<arch>-<scenario>`, e.g. "openSUSE-Tumbleweed-DVD-x86_64-gnome", nicknamed *koala*
163
 * ***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
166 8 okurz
# Thoughts about categorizing test results, issues, states within openQA
167
by okurz
168
169
When reviewing test results it is important to distinguish between different causes of "failed tests"
170
171
## Nomenclature
172
173
### Test status categories
174 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".
175 1 alarrosa
176 10 okurz
Another common but potentially ambiguous categorization:
177 1 alarrosa
178 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
179
* *failing*: the test is behaving as expected, but the test output is a fail --> "true positive"
180
* *working*: the test is behaving as expected (with no comment regarding the result, though some might ambiguously imply 'result is negative')
181
* *passing*: the test is behaving as expected, but the result is a success --> "true negative"
182
183 8 okurz
If in doubt declare a test as "broken". We should review the test and examine if it is behaving as expected.
184 9 okurz
185 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/
186 8 okurz
187
### Priorization of work regarding categories
188 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".
189 3 okurz
190 11 okurz
## Further categorization of statuses, issues and such in testing, especially automatic tests
191
By okurz
192
193
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.
194
While used for naming it should also be used as a decision tree and can be followed from the top following each branch.
195
196
### Categorization scheme
197
198
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.
199
Every leaf of the tree must have an action assigned to it.
200
201 12 okurz
1 **failed** (ZULU)
202 11 okurz
11 new (passed->failed) (YANKEE)
203
111 product issue ("true positive") (WHISKEY)
204
1111 unfiled issue (SIERRA)
205
11111 hard issue (KILO)
206
111121 critical / potential ship stopper (INDIA) --> immediately file bug report with "ship_stopper?" flag; opt. inform RM directly
207
111122 non-critical hard issue (HOTEL) --> file bug report
208
11112 soft issue (JULIETT) --> file bug report
209
1112 bugzilla bug exists (ROMEO)
210
11121 bug was known to openqa / openqa developer --> cross-reference (bug->test, test->bug) AND raise review process issue, improve openqa process
211
11122 bug was filed by other sources (e.g. beta-tester) --> cross-reference (bug->test, test->bug)
212
112 test issue ("false positive") (VICTOR)
213
1121 progress issue exists (QUEBEC) --> cross-reference (issue->test, test->issue)
214
1122 unfiled test issue (PAPA)
215
11221 easy to do w/o progress issue
216
112211 need needles update --> re-needle if sure, TODO how to notify?
217
112212 pot. flaky, timeout
218
1122121 retrigger yields PASS --> comment in progress about flaky issue fixed
219
1122122 reproducible on retrigger --> file progress issue
220
11222 needs progress issue filed --> file progress issue
221
12 existing / still failing (failed->failed) (XRAY)
222
121 product issue (UNIFORM)
223
1211 unfiled issue (OSCAR) --> file bug report AND raise review process issue (why has it not been found and filed?)
224
1212 bugzilla bug exists (NOVEMBER) --> ensure cross-reference, also see rules for 1112 ROMEO
225
122 test issue (TANGO)
226
1221 progress issue exists (MIKE) --> monitor, if persisting reprioritize test development work
227
1222 needs progress issue filed (LIMA) --> file progress issue AND raise review process issue, see 1211 OSCAR
228 12 okurz
2 **passed** (ALFA)
229 11 okurz
21 stable (passed->passed) (BRAVO)
230
211 existing "true negative" (DELTA) --> monitor, maybe can be made stricter
231
212 existing "false negative" (ECHO) --> needs test improvement
232
22 fixed (failed->passed) (CHARLIE)
233
222 fixed "true negative" (FOXTROTT) --> TODO split monitor, see 211 DELTA
234
2221 was test issue --> close progress issue
235
2222 was product issue
236
22221 no bug report exists --> raise review process issue (why was it not filed?)
237
22222 bug report exists
238
222221 was marked as RESOLVED FIXED
239
221 fixed but "false negative" (GOLF) --> potentially revert test fix, also see 212 ECHO
240
241
242
Priority from high to low: INDIA->OSCAR->HOTEL->JULIETT->…
243
244
### Further decision steps working on test issues
245
246
Test issues could be one of the following sources
247
248
* "accepted product changes"
249
 * product changed slightly but in an acceptable way without the need for communication with DEV+RM --> adapt test
250
 * product changed slightly but in an acceptable way found after feedback from RM --> adapt test
251
 * product changed significantly --> after approval by RM adapt test
252
253
* changes in test setup, e.g. our test hardware equipment behaves different or the network
254
* changes in test infrastructure software, e.g. os-autoinst, openQA
255
* changes in test management configuration, e.g. openQA database settings
256
* changes in the test software itself
257
258 16 okurz
259
# Test reviewing
260
261
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.
262
263
## Show previous results in test results page [gh#538](https://github.com/os-autoinst/openQA/pull/538)
264
265
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".
266
267
Querying the database for former test runs of the same scenario is a rather
268
costly operation which we do not want to do for multiple test results at once
269
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.
270
271
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
272
`http://openqa.opensuse.org/tests/1234?limit_previous=30#previous`
273
274
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.
275
276
Related issue: #10212
277
278
Screenshot of feature:
279
![screenshot_20160210_142024](https://cloud.githubusercontent.com/assets/1693432/12948308/7e915a3c-d001-11e5-840b-2f070c3cb8a5.png)
280
281
282
## Show bug or label icon on overview if labeled [gh#550](https://github.com/os-autoinst/openQA/pull/550)
283
284
* Show bug icon with URL if mentioned in test comments
285
* Show bug or label icon on overview if labeled
286
287
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.
288
289
Example for a generic label:
290
![openqa_generic_label](https://cloud.githubusercontent.com/assets/1693432/13027322/7bce7992-d24a-11e5-99ee-839fb5e82169.png)
291
292
Example for bug label:
293
![openqa_bug_label](https://cloud.githubusercontent.com/assets/1693432/13027323/8555238a-d24a-11e5-83d5-5bb2d2140860.png)
294
295
Related issue: #10212
296
297
298
## Show certificate next to builds on overview if all failures are labeled [gh#560](https://github.com/os-autoinst/openQA/pull/560)
299
300
Based on comments in the individual job results for each build a certificate
301
icon is shown on the group overview page as well as the index page to indicate
302
that every failure has been reviewed, e.g. a bug reference or a test issue
303
reason is stated. Only the failed and incomplete jobs are regarded for the
304
evaluation if a build is considered "reviewed".
305
306
If the badge appears you know you are done for one complete build :-)
307
308
Example screenshot:
309
![openqa_reviewed_label](https://cloud.githubusercontent.com/assets/1693432/13145996/eb1bb78a-d653-11e5-9f0f-40898915578e.png)
310
311
312
## Allow group overview query by result [gh#531](https://github.com/os-autoinst/openQA/pull/531)
313
314
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".
315
316
Example: Add query parameters like `…&result=failed&arch=x86_64` to show only failed for the single architecture selected.
317
318
319 3 okurz
# Old content
320
## Sprints
321 2 okurz
322 1 alarrosa
323
[[Sprint 01]]
324
[[Sprint 02]]
325
[[Sprint 03]]