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

t: Reduce timeout of t/44-scripts.t again after stabilization #3454

Merged
merged 1 commit into from Oct 9, 2020

Conversation

okurz
Copy link
Member

@okurz okurz commented Oct 8, 2020

We just increased the timeout recently in 4e00e6b but it seems we
needed to workaround a temporary problem as I can confirm with
reproducible runs that the normal expected runtime is well below the
specified timeout value so we can reduce it again a bit.

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

@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

Merging #3454 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3454      +/-   ##
==========================================
- Coverage   95.57%   95.56%   -0.02%     
==========================================
  Files         364      364              
  Lines       31516    31516              
==========================================
- Hits        30122    30117       -5     
- Misses       1394     1399       +5     
Impacted Files Coverage Δ
t/44-scripts.t 100.00% <100.00%> (ø)
t/ui/27-plugin_obs_rsync_status_details.t 80.70% <0.00%> (-3.51%) ⬇️
t/34-developer_mode-unit.t 98.72% <0.00%> (-0.77%) ⬇️
lib/OpenQA/WebSockets.pm 88.88% <0.00%> (+3.17%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c6fb7b...e7baf5d. Read the comment docs.

@mergify
Copy link
Contributor

mergify bot commented Oct 9, 2020

This pull request is now in conflicts. Could you fix it? 🙏

We just increased the timeout recently in 4e00e6b but it seems we
needed to workaround a temporary problem as I can confirm with
reproducible runs that the normal expected runtime is well below the
specified timeout value so we can reduce it again a bit.

Related progress issue: https://progress.opensuse.org/issues/72199
@okurz okurz merged commit 755d8f3 into os-autoinst:master Oct 9, 2020
@okurz okurz deleted the fix/t_scripts branch October 9, 2020 16:17
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