action #72127
check and/or reduce runtime of t/44-scripts.t
Start date:
2020-09-30
Due date:
% Done:
0%
Estimated time:
Difficulty:
Description
Observation¶
[07:25:28] t/44-scripts.t ............................................ 16/? Bailout called. Further testing stopped: test exceeds runtime limit of '120' seconds FAILED--Further testing stopped: test exceeds runtime limit of '120' seconds
Related issues
History
#1
Updated by okurz 4 months ago
- Copied to action #72130: check and/or reduce runtime of t/api/04-jobs.t added
#2
Updated by okurz 4 months ago
- Status changed from In Progress to Feedback
I could reproduce the problem locally, e.g. from lord.arch:
t/44-scripts.t .. ok 1 - Calling 'client --help' returns exit code 0 ok 2 - Calling 'create_admin --help' returns exit code 0 ok 3 - Calling 'dump_templates --help' returns exit code 0 ok 4 - Calling 'initdb --help' returns exit code 0 ok 5 - Calling 'load_templates --help' returns exit code 0 ok 6 - Calling 'modify_needle --help' returns exit code 0 ok 7 - Calling 'openqa --help' returns exit code 0 ok 8 - Calling 'openqa-cli --help' returns exit code 0 ok 9 - Calling 'openqa-clone-job --help' returns exit code 0 ok 10 - Calling 'openqa-label-all --help' returns exit code 0 ok 11 - Calling 'openqa-livehandler --help' returns exit code 0 ok 12 - Calling 'openqa-scheduler --help' returns exit code 0 ok 13 - Calling 'openqa-validate-yaml --help' returns exit code 0 ok 14 - Calling 'openqa-websockets --help' returns exit code 0 ok 15 - Calling 'openqa-workercache --help' returns exit code 0 Bailout called. Further testing stopped: test exceeds runtime limit of '20' seconds Bail out! test exceeds runtime limit of '20' seconds FAILED--Further testing stopped: test exceeds runtime limit of '20' seconds ## count_fail_ratio: Run: 5. Fails: 5. Fail ratio 100%
Prepared https://github.com/os-autoinst/openQA/pull/3435 to cover this for now with higher timeout. In general it is unfortunate that our scripts take that long already to just show a "--help" screen. This is something that could be improved.
#3
Updated by okurz 4 months ago
- Copied to action #72199: check and reduce runtime of t/44-scripts.t added
#4
Updated by okurz 4 months ago
- Status changed from Feedback to Resolved
https://github.com/os-autoinst/openQA/pull/3435 merged so that the timeout is not hit. Created #72199 to bring down the runtime.