cmcfarlen commented on code in PR #13448:
URL: https://github.com/apache/trafficserver/pull/13448#discussion_r3675414800
##########
tests/gold_tests/tls/tls_secret_update_default.test.py:
##########
@@ -122,7 +117,7 @@ def _add_config_reload_run(self) -> 'TestRun':
def _add_config_touch_run(self) -> 'TestRun':
'''Mark the multicert configuration for reload.'''
tr = Test.AddTestRun('Mark the multicert configuration for reload')
- tr.Processes.Default.Command = f'touch
{self._ts.Disk.ssl_multicert_yaml.AbsPath}'
+ tr.Processes.Default.Command = f'touch
{self._ts.Disk.ssl_multicert_config.AbsPath}'
tr.Processes.Default.ReturnCode = 0
Review Comment:
Good catch, thanks. Fixed in the amended commit: the reload wait now looks
for `ssl_multicert.config finished loading` (SSLConfig.cc logs the actual
multicert filename, which is `ssl_multicert.config` on 10.2.x), and the touch
target was already updated to the same file.
--
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]
