Project

General

Profile

Wiki » History » Version 16

okurz, 2016-07-08 13:50
Add tips for test development and issue investigation

1 15 okurz
# Introduction
2 1 okurz
3 15 okurz
{{toc}}
4
5 1 okurz
Also see https://progress.opensuse.org/projects/openqav3/wiki
6
7 15 okurz
# test organization on https://openqa.suse.de/
8 1 okurz
9 15 okurz
## job group names
10 1 okurz
11 15 okurz
### Job group names should be consistent and structured for easy (daily) review of the current status
12 1 okurz
13
template:
14
```
15
<product_group_short_name> <order_nr>.<product_variant>
16
```
17
e.g. "SLE 12 SP1 1.Server". Keep the whitespace for separation consistent, also see https://progress.opensuse.org/issues/9916
18
19 15 okurz
### Released products should be named with a prefix 'x' to show up late in the overview page
20 1 okurz
21 2 okurz
This way we can keep track if tests fail even though the product does not produce new builds. This could help us crosscheck tests. E.g. "x-released SLE 12 SP1 1.Server".
22 1 okurz
23 2 okurz
lowercase "x" as all our product names start with capital letters. Sorting works regardless (or uppercase first?).
24 1 okurz
25
For now we do not retrigger tests on old builds automatically but any test developer may retrigger it manually, e.g. if he suspects the tests broke and he wants to confirm that local changes are not at fault.
26 4 okurz
27 15 okurz
# needling best practices
28 14 okurz
There are also other locations where "needling best practices" can be found but we should also have the possibility to keep something on the wiki. Feel free to contact me (okurz) and tell me where it should be instead if there is a better place. Also look into [openQA Pitfalls](https://github.com/os-autoinst/openQA/blob/master/docs/Pitfalls.asciidoc)
29 4 okurz
30 15 okurz
## applying "workaround" needles
31 4 okurz
If a test reveals a product issue of minor importance it can make sense create a needle with the property "workaround" set. This way, if the needle is matched, the test records this as a "soft-fail". To backtrack the product issue and follow on this and eventually delete the workaround needle if the product issue is fixed, the product issue should be recorded in the needle name itself and at best also in the git commit message adding the needle. If test changes are necessary the source code should have a corresponding comment referencing the issue as well as marking start and stop of the test procedure that is necessary for applying the workaround. Example for a needle name: "gdm-workaround-bsc962806-20160125" referencing bsc#962806
32 1 okurz
33
*keep in mind:*
34 13 okurz
Since [gh-os-autoinst#532](https://github.com/os-autoinst/os-autoinst/pull/532) workaround needles are always preferred, otherwise if two needles match, the first in alphabetical list wins.
35 5 okurz
36 15 okurz
## do not overwrite old needles because old date confuses people
37 1 okurz
With the needle editor a timestamp of the current day is automatically added to new neeedles. When updating a needle, don't overwrite a needle with the old date tag not to confuse people as it will look really weird in the needle editor.
38 8 okurz
39
40 15 okurz
## needle indidvidual column entries in tables
41 8 okurz
**Problem**: Tables might auto-adjust column size based on content. Therefore it is unsafe to create needles covering multiple columns in a row. Failing example: https://openqa.suse.de/tests/441169#step/yast2_snapper/23
42
**Solution**: Needles support multiple areas. Use them to needle individual cells in this example.
43 6 okurz
44 15 okurz
# Definition of DONE/READY
45 6 okurz
46
Each of the following points has to be fulfilled to regard individual contributions as *DONE*. Not every step has to be done by the same step. The overall completion is in responsibility of the complete team.
47
48 15 okurz
## Definition of DONE
49 6 okurz
50
Also see http://www.allaboutagile.com/definition-of-done-10-point-checklist/ and https://www.scrumalliance.org/community/articles/2008/september/what-is-definition-of-done-%28dod%29
51
52
The following definitions are used to ensure development on individual tests has been completed covering all existing different workflows, e.g. covering "hot-fixes" on the productive instance as well as contributions by new contributors with no previous experience and no control over needle generation on productive instances.
53
54 1 okurz
* Code changes are made available via a pull request on the github repository
55 6 okurz
* New tests as individual test modules (i.e. files under `tests/`): They are loaded in main.pm of sle and/or opensuse 
56
* "make test" works (e.g. automatic travis CI check triggered on each github PR)
57
* [Guidelines for git commits](http://chris.beams.io/posts/git-commit/) have been followed
58
* Code has been reviewed (e.g. in the github PR)
59
* Favored, but depending on criticality/complexity/size: A local verification test has been run, e.g. post link to a local openQA machine or screenshot or logfile
60
* Code has been merged (either by reviewer or reviewee after 'LGTM' from others)
61 1 okurz
* Code has been deployed to osd and o3 (automatic git sync every few minutes)
62
* If new variables are necessary (feature toggles): A test_suite is executing the test, e.g. test_suite is created or variable is added to existing test_suite over web interface configuration on osd and/or o3
63 7 okurz
* If a new test_suite has been created: The test_suite is added to at least one job_group
64 6 okurz
* Necessary needles are made available as PR for sle and/or opensuse (depending if executed, see above for 'main.pm') or are created on the productive instance
65
* At least one successful test run has been observed on osd or o3 and referenced in the corresponding progress item or bugzilla bug report if one exists
66
67 15 okurz
## Definition of READY for new tests
68 6 okurz
69
The following points should be considered before a new test is READY to be implemented:
70
71
* Either a product bug has been discovered for which there is no automated test in openQA or a FATE request for new features exists
72
* A test case description exists depicting the prerequisites of the test, the steps to conduct and the expected result
73
* The impact and applicability for both SLE and openSUSE products has been considered
74 10 okurz
75 15 okurz
# Test development instances (staging openQA instances)
76 10 okurz
77 1 okurz
Contributors cannot afford to verify a newly developed test in all scenarios run by o3 or osd, so tests will break sometime. It would be useful to use a machine to run a subset of the scenarios run in the official instance(s) to make sure the new tests can be deployed with some degree of confidence. But: Any "staging openQA instance" would not be able to run everything which is run in production. It just does not scale. So anyway only a subset can be run and there can be always something missing. Also, we don't have the hardware capacity to cover everything twice and also consider SLE plus openSUSE. Our [DOD](https://progress.opensuse.org/projects/openqatests/wiki/Wiki#Definition-of-DONEREADY) should cover some important steps so that external contributors are motivated to test something locally first. We have a good test review process and it has to be decided by the reviewer if he accepts the risk of a new test with or without a local verification and covering which scenarios. Depending on the contributors it might make sense to setup a staging server with a subset of tests which is used by multiple test developers to share the burden of openQA setup and administration. For example the YaST team has one available: https://wiki.microfocus.net/index.php/YAST/openQA
78 11 okurz
If you want to follow this model you can watch [this talk by Christopher Hofmann from the OSC16](https://events.opensuse.org/conference/oSC16/program/proposal/986) or ask the YaST team for their experiences.
79 16 okurz
80
81
# Tips for test development and issue investigation
82
83
Examples mentioned here write `clone_job` and `client`. Replace this by a call to the scripts within openQA installation with the corresponding name and proper arguments to provide your API key as well as the host selection, e.g. `/usr/share/openqa/client --host https://openqa.opensuse.org` with your API key configured in `~/.config/openqa/client.conf`
84
85
## Uploading image files to openqa server and run test on it
86
87
You can manually trigger a test job with explicit name as one-shot overriding the variables as necessary, for example:
88
89
as geekotest@openqa:
90
91
```
92
cd /var/lib/openqa/factory/hdd
93
wget http://<my_host>/<path>.qcow2 -O <new_image_name>.qcow2
94
cd /var/lib/openqa/factory/iso
95
/usr/share/openqa/script/client isos post --params SLE-12-SP2-Server-DVD-ppc64le-Build1651-Media1.iso.6.json HDD_1=SLE-12-Server-ppc64le-GM-gnome_with_snapper.qcow2 TEST=migration_offline_sle12_ppc BUILD=1651_<your_short_name>
96
```
97
98
why `SLE-12-SP2-Server-DVD-ppc64le-Build1651-Media1.iso.6.json`? I checked `SLE-12-SP2-Server-DVD-ppc64le-Build1651-Media1.iso.?.json`: There are `…5…` and `…6…`. `…5…` is for *HA* so I chose 6.
99
100
The job can be cleaned afterwards to tidy up the build history with:
101
102
```
103
client jobs/463859 delete
104
```
105
106
## Create new HDD image with openQA
107
```
108
client jobs post DISTRI=sle VERSION=12 FLAVOR=Server-DVD ARCH=ppc64le BACKEND=qemu \
109
NOVIDEO=1 OFW=1 QEMUCPU=host SERIALDEV=hvc0 BUILD=okurz_poo9714 \
110
ISO=SLE-12-Server-DVD-ppc64le-GM-DVD1.iso INSTALLONLY=1 QEMU_COMPRESS_QCOW2=1 \
111
PUBLISH_HDD_1=SLES-12-GM-gnome-ppc64le_snapper_20g.qcow2 TEST=create_gm_ppc_image \
112
MACHINE=ppc64le WORKER_CLASS=qemu_ppc64le HDDSIZEGB=20 MAX_JOB_TIME=86400 TIMEOUT_SCALE=10
113
```
114
115
The `MAX_JOB_TIME=86400 TIMEOUT_SCALE=10` allows for interactive login during the process in case you want to manually adjust or debug. Beware though that `TIMEOUT_SCALE=10` also scales the waiting time on `check_screen` so that the whole job might take longer to execute.
116
117
To run a test but based on the new HDD image search for a good example and clone it with adjusted parameter:
118
119
```
120
clone_job 462022 HDD_1=SLES-12-GM-gnome-ppc64le_snapper_20g.qcow2
121
```
122
123
124
## Interactive investigation
125
126
While a job is running one can connect to the worker (if network access is possible) using VNC. One challenge is that the test is still running and manual interaction with the system interferes with the test and vice versa.
127
128
129
### Making the test stop for long enough to be able to connect
130
131
If you can change the test code, i.e. if running on a development machine, you can for example add a `sleep 3600;` or `wait_serial 'CONTINUE';` at the point in test when you want to connect to the system and interact with it, e.g. to gather additional logs. In case of `wait_serial 'CONTINUE';` you can echo 'CONTINUE' to the serial point to let the test continue, e.g. call `echo 'CONTINUE' > /dev/ttyS0;`.
132
133
In case you can not or do not want to change the test code or your test run is stopping anyway at a certain point with long enough timeout you can also increase timeout with `TIMEOUT_SCALE`, e.g. trigger it with the job variable `TIMEOUT_SCALE=10`. For example a `script_run` with default timeout of 90 seconds will wait for 900 seconds (=15 minutes) which should give enough time in most cases already.
134
135
136
### Connecting over VNC
137
138
Then connected to the instance when it stalled over VNC. The VNC port is shown on the job live view as a hover text on the instance name. Make sure to use a "shared" connection in your vncviewer. `krdc`, the default KDE VNC viewer, as well as `vinagre`, default GNOME VNC viewer, do this already. For TigerVNC use for example:
139
140
```
141
vncviewer -Shared malbec.arch:91
142
```
143
144
145
### Forwarding of special shortcuts
146
147
The default `vncviewer` in openSUSE/SUSE systems is recommended as it can also be used to forward special keyboard shortcuts. E.g. to change to text console:
148
Press *F8* in vncviewer, select *ctrl* and *alt* in menue, exit menue, press *F2*.
149
150
151
### Requesting video when by default you do not have video in your environment
152
153
Example:
154
155
```
156
clone_job 464665 NOVIDEO=0
157
```