[GitHub] [trafficserver] ywkaras commented on a change in pull request #6930: AuTest: Log and When condition for ATS initialized

[GitBox <gi...@apache.org>]

ywkaras commented on a change in pull request #6930:
URL: https://github.com/apache/trafficserver/pull/6930#discussion_r443573300



##########
File path: src/traffic_server/traffic_server.cc
##########
@@ -2125,6 +2136,9 @@ main(int /* argc ATS_UNUSED */, const char **argv)
 
     ink_set_thread_name("[TS_MAIN]");
 
+    if (ports_are_open && cache_is_initialized) {
+      Note("traffic server initialized");

Review comment:
       What about Au tests that disable caching?  https://github.com/apache/trafficserver/blob/c7e80542aa1a5323399226f636ef196955f60791/tests/gold_tests/continuations/openclose.test.py#L46 to give one example.  Will this message still be output when cache is disabled?
   
   It looks like this Au tests (openclose) explicitly uses port probing for the ready test for TS.  Is the plan to use a non-default test for test cases that disable caching?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org