cmcfarlen opened a new pull request, #13539:
URL: https://github.com/apache/trafficserver/pull/13539

   Cherry-picks for the 10.2.0 RC1, covering the PRs at "For v10.2.0" in the 
[ATS v10.2.x project](https://github.com/orgs/apache/projects/573) — the fixes 
and feedback that came out of rc0 testing.
   
   All picked with `git cherry-pick -x` in master merge order. **Every pick 
applied cleanly with no conflicts**, and each commit's diffstat matches its 
master commit exactly.
   
   | PR | Title |
   |---|---|
   | #13532 | Fix signature on `flz_maxcopy` to avoid cast error |
   | #13531 | Change libswoc test for undefined errno to a much higher number |
   | #13535 | Fix an intermittent cache unit test deadlock at exit |
   | #13536 | cache: fix shm sizing on large-page Linux |
   | #13537 | Share regex_remap rule sets |
   
   Two of these are follow-ups to work already on this branch, so ordering 
relative to the earlier rounds matters more than ordering among themselves (no 
files overlap within this batch):
   
   - **#13536** fixes shm sizing for the shared-memory Dir added by #13328, 
which landed here in round 4. It touches only `cache_shm` headers, 
`CacheShm.cc` and its unit test — no on-disk cache format — so there is no 
cache-compatibility consideration for upgrades.
   - **#13537** addresses a performance regression that shipped in rc0: since 
the PCRE2 migration (#12685, present on this branch), every regex_remap plugin 
instance JIT-compiles its own copy of a rule file, so `remap.config` reload 
time scales with mapping count rather than unique rule sets.
   
   **One thing worth noting about the #13536 pick.** This branch carries a 
local adaptation to `include/shared/cache_shm/Layout.h`: the 
`storage_signature` comment says `storage.config` rather than master's 
`storage.yaml`, because that is the file this release actually reads. #13536 
also edits `Layout.h`, but in a different region (the control-segment size 
helper around line 87 versus the comment at line 70), so the pick applied 
without touching the adaptation. Verified afterward that no `storage.yaml` 
reference has crept back into `include/`, `src/`, `doc/` or `tests/`.
   
   Verified before pushing: every `Disk.*` attribute and `Test.*` helper used 
by the new and modified tests exists on this branch — `Test.AddConfigReload` 
(via #13502) and `Test.AddAwaitFileContainsTestRun` in particular — and no 
picked test reaches for a master-only config idiom.
   
   Local build is clean and `ctest` is **167/167**. `cmake --build ... --target 
format` makes no changes.
   
   Draft so the full CI matrix runs against the picked set 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]

Reply via email to