bryancall opened a new pull request, #12899:
URL: https://github.com/apache/trafficserver/pull/12899
* **New autest for `msdms` milestone timing fields** -- sends a cacheable
request twice (miss then hit) and validates all Phase 1 timing fields in the
log output. The validation script (`verify_milestone_fields.py`) checks:
- All 16 key=value fields are present on every line
- All timing values are integers with no epoch-length garbage (> 1 billion)
- Cache miss lines: origin-phase chain (`c_hdr + c_proc + cache + dns +
o_tcp + o_wait + o_hdr + o_proc`) sums to `c_ttfb` within 2ms tolerance
- Cache hit lines: `hit_proc` and `hit_xfer` are non-negative
This test exercises the full msdms unmarshal path end-to-end and will catch
regressions like the epoch leak fixed in #12897.
**Note:** This PR also includes the fixes from #12897 (`difference_msec`
epoch leak) and #12898 (`UA_BEGIN_WRITE` unconditional) which are required for
the test to pass. Once those PRs merge, this branch can be rebased to drop the
duplicate commits.
**Testing:** The test itself is the test.
--
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]
