bneradt commented on PR #13456: URL: https://github.com/apache/trafficserver/pull/13456#issuecomment-5500086342
Addressed the review feedback and force-pushed the rebased commit `626eab03d0`: - Added an actionable startup warning when the configured YAML file is absent but a sibling `cache.config` exists, including the converter command and the legacy-record escape hatch. - Documented the new default, migration command, semantic difference, and legacy compatibility in the upgrading and cache configuration docs. - Made the converter warn when multiple legacy rules may change behavior under YAML first-match semantics. - Added paired runtime coverage demonstrating YAML first-match behavior and legacy additive behavior for equivalent rules. - Tightened converter behavior: reject embedded newlines, emit exactly one trailing newline, validate output/write completion, and cover `/dev/full` failure. - Rebased the PR onto current `master`. Validation in the `asfats5` container: - Full configure, format, build, and install: passed. - `test_tsconfig "[cache][config]"`: 63 assertions in 6 test cases passed. - Focused AuTests `cache-yaml` and `convert_cache`: 2/2 passed. -- 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]
