Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dist: Bump timeout scaling factor for slow OBS tests #1754

Merged
merged 1 commit into from Aug 26, 2021

Conversation

okurz
Copy link
Member

@okurz okurz commented Aug 26, 2021

Executed locally and the relevant group of tests locally only takes 1
second without coverage:

$ time prove -I. t/18-backend-qemu.t t/18-qemu.t
t/18-backend-qemu.t .. ok
t/18-qemu.t .......... ok
All tests successful.
Files=2, Tests=57,  4 wallclock secs ( 0.05 usr  0.00 sys +  1.70 cusr  0.30 csys =  2.05 CPU)
Result: PASS

Originally on OBS the output was not clear which test timed out. The
related ticket mentions that likely t/18-backend-qemu.t would be the
culprit which is actually ignored from testing in OBS so I suspect it is
t/18-qemu.t which has the same base 5 second timeout.

Related progress issue: https://progress.opensuse.org/issues/97241

Executed locally and the relevant group of tests locally only takes 1
second without coverage:

```
$ time prove -I. t/18-backend-qemu.t t/18-qemu.t
t/18-backend-qemu.t .. ok
t/18-qemu.t .......... ok
All tests successful.
Files=2, Tests=57,  4 wallclock secs ( 0.05 usr  0.00 sys +  1.70 cusr  0.30 csys =  2.05 CPU)
Result: PASS
```

Originally on OBS the output was not clear which test timed out. The
related ticket mentions that likely t/18-backend-qemu.t would be the
culprit which is actually ignored from testing in OBS so I suspect it is
t/18-qemu.t which has the same base 5 second timeout.

Related progress issue: https://progress.opensuse.org/issues/97241
@mergify mergify bot merged commit 40d2561 into os-autoinst:master Aug 26, 2021
@okurz okurz deleted the fix/obs branch August 26, 2021 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants