[GitHub] [trafficserver] ezelkow1 commented on a change in pull request #8279: stats_over_http: demote error to note

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

ezelkow1 commented on a change in pull request #8279:
URL: https://github.com/apache/trafficserver/pull/8279#discussion_r696040283



##########
File path: plugins/stats_over_http/stats_over_http.c
##########
@@ -1006,8 +1006,8 @@ load_config_file(config_holder_t *config_holder)
   }
 
   if (!fh) {
-    TSError("[%s] Unable to open config: %s. Will use the param as the path, or %s if null\n", PLUGIN_NAME,
-            config_holder->config_path, DEFAULT_URL_PATH);
+    TSNote("[%s] Unable to open config: %s. Will use the param as the path, or %s if null\n", PLUGIN_NAME,
+           config_holder->config_path, DEFAULT_URL_PATH);

Review comment:
       yea thats why I originally made it an error, just so users would be completely aware if the file did not exist or it couldnt be opened. But yes it is also emitted if the user specified no 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: github-unsubscribe@trafficserver.apache.org

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