bryancall commented on issue #12941: URL: https://github.com/apache/trafficserver/issues/12941#issuecomment-4828331098
Fixed by [PR #13325](https://github.com/apache/trafficserver/pull/13325), merged on 2026-06-25 (first release: 10.1.3). The fix is exactly what this issue proposed: in `HttpSM::state_cache_open_read()`, the `TS_MILESTONE_CACHE_OPEN_READ_END` milestone is now stamped before `call_transact_and_set_next_state()` in the terminal cache-open-read paths, so it can no longer be set after milestones that logically depend on it (such as `UA_BEGIN_WRITE` and `SERVER_FIRST_CONNECT`). That removes the negative values in the derived `dns` and `hit_proc` timing fields. The compatibility-key retry path is intentionally left unstamped so the retried lookup owns a coherent cache-read begin/end pair. Verified on master in `src/proxy/http/HttpSM.cc`: the milestone is set on the `CACHE_EVENT_OPEN_READ` path before the transact call, and on the `CACHE_EVENT_OPEN_READ_FAILED` path after the retry is ruled out. Closing as resolved. -- 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]
