Skip to content

pytest: improve stragglers #19809

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.

Sign up for GitHub

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

Jump to bottom
Closed
icing wants to merge 1 commit into from
Closed

pytest: improve stragglers #19809

icing wants to merge 1 commit into from

Conversation

@icing
Copy link
Contributor

@icing icing commented Dec 2, 2025 •
edited
Loading

A fix for the tests that took the longest:

  • test_05: make the server close the HTTP/1.1 connection when simulating an error during a download. This eliminates waiting for a keepalive timeout
  • test_02: pause tests with slightly smaller documents, eliminate special setup for HTTP/2. We test stream window handling now elsewhere already
  • cli_hx_download: run loop in 500ms steps instead of 1sec, resuming paused tranfers earlier.

@icing
pytest: improve stragglers
55601f8 
A fix for the tests that took the longest:
- test_05: make the server close the HTTP/1.1 connection when
  simulating an error during a download. This eliminates waiting
  for a keepalive timeout
- test_02: pause tests with slightly smaller documents, eliminate
  special setup for HTTP/2. We test stream window handling now
  elsewhere already
- cli_hx_download: run look in 500ms steps instead of 1sec, resuming
  paused tranfers earlier.
@icing icing requested a review from vszakats December 2, 2025 14:56
@bagder bagder closed this in 9cf4a40 Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vszakats vszakats vszakats approved these changes

Assignees

No one assigned

Labels

tests tidy-up

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@icing @vszakats