Skip to content

Comments

Update to Proxy Verifier v2.8.0#9686

Closed
bneradt wants to merge 2 commits intoapache:masterfrom
bneradt:update_to_pv_280
Closed

Update to Proxy Verifier v2.8.0#9686
bneradt wants to merge 2 commits intoapache:masterfrom
bneradt:update_to_pv_280

Conversation

@bneradt
Copy link
Contributor

@bneradt bneradt commented May 6, 2023

This updates our test suite to use Proxy Verifier v2.8.0. This release comes with HTTP/2 frame replay updates.

@bneradt bneradt added the AuTest label May 6, 2023
@bneradt bneradt added this to the 10.0.0 milestone May 6, 2023
@bneradt bneradt self-assigned this May 6, 2023
masaori335
masaori335 previously approved these changes May 8, 2023
@bneradt
Copy link
Contributor Author

bneradt commented May 8, 2023

Looks like the http2_rst_stream test is failing with this updated Proxy Verifier version. I know @duke8253 was using this updated Proxy Verifier for precisely testing RST_STREAM functionality. We'll need to look into what patch will be needed to address the new failure. Fei is currently on PTO this week, so this may wait until he gets back, unless I get some time to look at it sooner.

@bneradt
Copy link
Contributor Author

bneradt commented May 18, 2023

[approve ci centos fedora]

@bneradt
Copy link
Contributor Author

bneradt commented May 18, 2023

[approve ci fedora centos]

@ezelkow1
Copy link
Member

[approve ci centos]

This updates our test suite to use Proxy Verifier v2.8.0. This release
comes with HTTP/2 frame replay updates.
@bneradt bneradt force-pushed the update_to_pv_280 branch 3 times, most recently from 0efd3c0 to c3bedfd Compare May 20, 2023 21:19
Conflicts:
      proxy/http2/Http2ConnectionState.cc
      proxy/http2/Http2Stream.cc
      tests/gold_tests/cache/background_fill.test.py
      tests/gold_tests/cache/gold/background_fill_3_stdout.gold
      tests/gold_tests/cache/replay/bg_fill.yaml
      tests/gold_tests/h2/http2_rst_stream.test.py
@bneradt bneradt force-pushed the update_to_pv_280 branch from c3bedfd to 563c47f Compare May 20, 2023 23:16
@bneradt
Copy link
Contributor Author

bneradt commented May 24, 2023

Closing this because @duke8253 is working on a more complete PR that will update and use Proxy Verifier v2.8.0.

@bneradt bneradt closed this May 24, 2023
@zwoop zwoop removed this from the 10.0.0 milestone Feb 29, 2024
@bneradt bneradt deleted the update_to_pv_280 branch June 17, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants