Skip to content

pytest: re-enable test test_05_01 and test_05_02 for quiche 0.29.0+#21784

Closed
vszakats wants to merge 5 commits into
curl:masterfrom
vszakats:quichemaybe
Closed

pytest: re-enable test test_05_01 and test_05_02 for quiche 0.29.0+#21784
vszakats wants to merge 5 commits into
curl:masterfrom
vszakats:quichemaybe

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented May 27, 2026

The upstream issue seems to have been fixed or mitigated in quiche
v0.29.0. Though the original upstream report and patch remain open at
the time of writing this.

Ref: cloudflare/quiche#2277
Ref: cloudflare/quiche#2278

Follow-up to 252b82f #21730
Follow-up to 91b422d #20952
Follow-up to 1447842 #19916


This was a flaky issue, but previously all bumps failed, and with 0.29.1
neither the automatic bump, nor this PR failed, which makes me think
this has indeed seen an upstream fix or mitigation.

test 05_02 still failed at 0.28.0:
https://github.com/curl/curl/actions/runs/23865652425/job/69583976618

@vszakats vszakats marked this pull request as draft May 27, 2026 20:44
@vszakats vszakats marked this pull request as ready for review May 27, 2026 20:53
@vszakats vszakats changed the title pytest: try dropping quiche workaround pytest: re-enable test test_05_01 and test_05_02 for quiche May 27, 2026
@github-actions github-actions Bot added the CI Continuous Integration label May 27, 2026
@vszakats vszakats changed the title pytest: re-enable test test_05_01 and test_05_02 for quiche pytest: re-enable test test_05_01 and test_05_02 for quiche 0.29.0+ May 27, 2026
This reverts commit f686f02.
@vszakats vszakats closed this in cdb2667 May 27, 2026
@vszakats vszakats deleted the quichemaybe branch May 27, 2026 21:28
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
The upstream issue seems to have been fixed or mitigated in quiche
v0.29.0. Though the original upstream report and patch remain open at
the time of writing this.

Ref: cloudflare/quiche#2277
Ref: cloudflare/quiche#2278

Follow-up to 252b82f curl#21730
Follow-up to 91b422d curl#20952
Follow-up to 1447842 curl#19916

Closes curl#21784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration HTTP tests

Development

Successfully merging this pull request may close these issues.

1 participant