You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafficserver.apache.org by James Peach <jp...@apache.org> on 2015/04/24 23:36:54 UTC

Re: [1/2] trafficserver git commit: TS-3545: Make traffic_line and traffic_ctl more verbose Add message when setting config options that don't require a restart

> On Apr 24, 2015, at 1:45 PM, bcall@apache.org wrote:
> 
> 
[snip]
> http://git-wip-us.apache.org/repos/asf/trafficserver/blob/27cd3df7/cmd/traffic_ctl/config.cc
> ----------------------------------------------------------------------
> diff --git a/cmd/traffic_ctl/config.cc b/cmd/traffic_ctl/config.cc
> index ce19f3e..ec9019a 100644
> --- a/cmd/traffic_ctl/config.cc
> +++ b/cmd/traffic_ctl/config.cc
> @@ -232,7 +232,7 @@ config_set(unsigned argc, const char **argv)
>     printf("set %s, restart required\n", file_arguments[0]);
>     break;
>   case TS_ACTION_RECONFIGURE:
> -    // printf("Set %s, reconfiguration required\n", file_arguments[0]);
> +    printf("set %s, please wait 10 seconds for traffic server to sync configuration, restart is not required\n", file_arguments[0]);

10s is not true is it? AFAIK this depends on proxy.config.config_update_interval_ms (and possibly other stuff).

J