bneradt commented on PR #13889: URL: https://github.com/apache/trafficserver/pull/13889#issuecomment-6351501193
Thanks, Bryan. Addressed in e9d4db12d5: - Existing ENABLE_AUTEST and ENABLE_AUTEST_UDS values now map to the Uranium options with deprecation warnings, including OFF. Explicit Uranium values take precedence. All five old configure preset names and the autest build preset are retained, including the original build-autest directory and install prefix. - The volume replay now asserts directory entries, the 32 MiB RAM cache allocation, and zero RAM cache for volume 2. It also allocates a 512 MiB disk: the previous default was too small to create both volumes, so the replay could succeed with caching disabled. - The ConnectingEntry production fix is explicitly described in the PR description and commit message. - All four test shards and the other checks were green on the previous revision. I also hardened the HTTP/2 content-length client: it completes SETTINGS before sending malformed input and submits the request frames together, avoiding writes racing a connection-fatal response. Its GOAWAY/error-code and successful-response assertions are unchanged. Local validation passed: build/install, formatting, 125 framework tests, a real configure using the old autest preset and OFF options, the volume replay, and ten consecutive HTTP/2 content-length test runs. The updated revision is pushed for CI. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
