cmcfarlen opened a new pull request, #13520: URL: https://github.com/apache/trafficserver/pull/13520
Third round of cherry-picks for the 10.2.0 release candidate, covering PRs at "For v10.2.0" in the [ATS v10.2.x project](https://github.com/orgs/apache/projects/573). All picked with `git cherry-pick -x` in master merge order; each commit's diffstat matches its master commit exactly. | PR | Title | |---|---| | #13487 | Preserve the cache action while dispatching cache write events | | #13509 | Initialize logging queues before workers | | #13368 | Bound the ring walk of `ParentConsistentHash::selectParent` | | #13475 | slice: purge every block of an object, not just those before a gap | | #13441 | Return an empty view for a non-participating capture group | | #13491 | Destroy replaced configs on ET_TASK | | #13508 | Harden timing-sensitive AuTests | | #13504 | Avoid stale H2 writes after 100 Continue | | #13515 | Fix cache read VC replacement after a lost write lock | | #13406 | Deliver VCONN_CLOSE for parked TLS hooks; fix SNI queue accounting | This brings in four crash fixes (#13487, #13509, #13504, #13515) plus the regression #13515 addresses, which #12852 introduced and which is already present on this branch. Merge order was preserved because `src/proxy/http/HttpTransact.cc` is touched by both #13475 and #13515, and #13515 builds on #13487 in `HttpCacheSM`/`HttpSM`. **One adaptation.** #13406's new autests configured certificates through `ts.Disk.ssl_multicert_yaml`, which only exists on master; this branch has the flat `ssl_multicert_config`. Converted in all four affected files, matching the form the `else` branch of the existing `hasattr(ts.Disk, "ssl_multicert_yaml")` helpers in `tests/gold_tests/h3/` already uses: ```python ts.Disk.ssl_multicert_config.AddLine('dest_ip=* ssl_cert_name=server.pem ssl_key_name=server.key') ``` Files: `rate_limit_sni_expiry.test.py`, `rate_limit_sni_reject.test.py`, `rate_limit_sni_queue.test.py`, `tls_hooks_close_while_parked.test.py`. Amended into the #13406 pick rather than added as a follow-up commit. **#13328 is deliberately not in this batch.** "cache: shared-memory-backed Dir for fast restart" cannot be picked as-is: its `traffic_ctl` integration attaches `cache shm` to a `cache_command` and routes through `CacheCommand`, neither of which exists on 10.2.x. Both arrived with #13418 ("Add traffic_ctl cache clear command"), which is labeled `New Feature`, sits at milestone 11.0.0, and is not tracked for this release. Auto-merge also wanted to pull the master-only `ConvertConfigCommand.cc` and `SSLMultiCertCommand.cc` into `src/traffic_ctl/CMakeLists.txt`. Rather than freehand an adaptation of a 43-file change days before the RC, the pick was aborted and its project item left at "For v10.2.0" for a decision: either pick #13418 as a prerequisite, or ask the author for a PR against 10.2.x directly. Verified before pushing: every `Disk.*` attribute and `Test.*` helper used by the picked tests exists on this branch (`Test.AddConfigReload()` is present via #13502), and all 21 `proxy.config.*` records the new tests reference exist in `RecordsConfig.cc`. Local build is clean and `ctest` passes 164/165; the failure is the known macOS-local `test_jsonrpcserver` unix-socket/restart timing flake. `yapf` 0.43.0 reports the four edited test files clean, and the `format` target makes no changes. Draft so the full CI matrix runs before the release branch moves; it will be landed by fast-forward. -- 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]
