brbzull0 opened a new pull request, #13075: URL: https://github.com/apache/trafficserver/pull/13075
Replace the fragile pattern of fire-and-forget `traffic_ctl config reload` followed by sleep/log-grepping with a deterministic helper that uses monitor mode (`-m`) to block until completion and validates reload tasks via the get_reload_config_status JSONRPC endpoint. The extension supports expected exit codes (success/fail/timeout/any), task presence and absence assertions, per-task status validation, custom tokens, and a delay_start parameter for filesystem timestamp sensitivity. Migrate 21 existing tests to use the new helper, removing manual sleep synchronization and When.FileContains polling. Add an optional filename parameter to ConfigReloadTask so sub-tasks carry their associated config file for precise identification in test assertions. #### Tiny change to the reload core: Reload sub-tasks previously only carried a description (e.g."SNIConfig"), making it impossible to assert which config file triggereda task. The C++ change adds an optional `filename` parameter to`ConfigReloadTask::add_child()` and `ConfigContext::add_dependent_ctx()`so the JSONRPC status endpoint returns the actual file path (e.g.`sni.yaml`, `remap.yaml`) for each sub-task. This allows tests to validate that the right config handlers ran -- or didn't run. **Includes RST documentation for the new extension API.** Fixes https://github.com/apache/trafficserver/issues/12965 -- 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]
