jrushford commented on a change in pull request #8689:
URL: https://github.com/apache/trafficserver/pull/8689#discussion_r818789840



##########
File path: src/traffic_server/traffic_server.cc
##########
@@ -2162,7 +2162,6 @@ main(int /* argc ATS_UNUSED */, const char **argv)
     RecProcessStart();
     initCacheControl();
     IpAllow::startup();
-    HostStatus::instance().loadHostStatusFromStats();

Review comment:
       @brbzull0 when the parent.config or strategies.yaml files are loaded, 
they call HostStatus::instance().  However, I'll  add HostStatus::instance() 
back in.

##########
File path: doc/appendices/command-line/traffic_ctl_jsonrpc.en.rst
##########
@@ -366,7 +365,8 @@ endpoint.
    :ref:`admin_lookup_records`
 
    Get the current status of the specified hosts with respect to their use as 
targets for parent
-   selection. This returns the same information as the per host stat.
+   selection. This returns the same information as the per host record. If the 
HOSTNAME arguments

Review comment:
       I'm not sure what your asking?  Do you want a change?




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


Reply via email to