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:
[email protected]


Reply via email to