[GitHub] [trafficserver] jrushford commented on a change in pull request #8689: Hoststatus remove stats creation.

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

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: github-unsubscribe@trafficserver.apache.org

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