brbzull0 commented on issue #12711: URL: https://github.com/apache/trafficserver/issues/12711#issuecomment-3602180839
The convert script changes the records that were [renamed](https://github.com/apache/trafficserver/blob/b38147f37095f0bf6f7ebdfb449ea98b1ace0690/tools/records/convert2yaml.py#L42-L52) because of the migration to YAML. It seems I missed a some that were renamed [afterwards](https://github.com/apache/trafficserver/commit/65f07be4de2644f1bf8decdc41f15e6fe5151484). [This](https://github.com/apache/trafficserver/pull/12713) should fix that issue. On the warning issue: ``` Warn: Warn: Ignoring field 'connect_attempts_max_retries_dead_server' [proxy.config.http.connect_attempts_max_retries_dead_server] at line=43, col=5. Not registered and Unknown tag type '?' Warn: Ignoring field 'connect_attempts_timeout' [proxy.config.http.parent_proxy.connect_attempts_timeout] at line=60, col=7. Not registered and Unknown tag type '?' Warn: Ignoring field 'post_connect_attempts_timeout' [proxy.config.http.post_connect_attempts_timeout] at line=62, col=5. Not registered and Unknown tag type '?' ``` This were [renamed](https://docs.trafficserver.apache.org/en/latest/release-notes/upgrading.en.html#configuration-changes) for other reasons, so they aren't in the convert2yaml script The warning message is because they seems to still exist in the records.config with the old name, so the script will convert them to yaml anyways then but then ATS will ignore them and use the default value for the renamed ones. -- 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]
