> On March 25, 2014, 12:23 a.m., Ben Mahler wrote: > > src/logging/flags.hpp, line 42 > > <https://reviews.apache.org/r/19357/diff/3/?file=533901#file533901line42> > > > > This flag name is inconsistent with our other flag names which use > > snake case. It would be nice if we could call this "logging_level" :) > > Alexandra Sava wrote: > The only one that has sneak case is 'log_dir'. For example, there is > 'logbufsecs' as well that does not have it. Anyway, for me is not a problem > changing this but the code has already been commited (commit > 7c366bad61e8c241797ef5c92562e0539c0dd02a). I'm not sure how we should proceed > in this case: reopen the jira ticket(which has been marked as resolved) or > create another one? > > Vinod Kone wrote: > FWIW, I actually prefer minloglevel because it is more accurate. And I > think it is nice that our logging flag names map directly to GLOG flags. > (flags.log_dir => FLAGS_log_dir, flags.logbufsecs => FLAGS_logbufsecs and > flags.minloglevel => FLAGS_FLAGS_logbufsecs")
That's not quite true since we have 'quiet', right? Do you recall when we watched an entire room of operators get confused when using the (replicated) log tooling and thinking that 'log_dir' was the directory for the replicated log? Because logging flags are global to all of our binaries, it would be nice to make these very explicit. 'logging_level', 'logging_directory', 'logging_quiet' read more clearly to me when faced with flags coming from various components, including the replicated log. Alexandra, in the rest of our flags across the project, we use snake case. - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19357/#review38382 ----------------------------------------------------------- On March 24, 2014, 12:51 p.m., Alexandra Sava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19357/ > ----------------------------------------------------------- > > (Updated March 24, 2014, 12:51 p.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-1067 > https://issues.apache.org/jira/browse/MESOS-1067 > > > Repository: mesos-git > > > Description > ------- > > minloglevel flag is passed as an argument to the command line and it > configures the level of logging to stderr and file. It can take INFO, > WARNING, ERROR or FATAL values. All log messages > at or above the configured log level will be printed. > > If 'quiet' flag is also passed to the command line, minloglevel will > affect just the logs from file (if configured by log_dir flag) > > If a log directory is present, the log file that will be available for > viewing in the browser > will be the one for the minimum log level configured (with or without the > presence of minloglevel flag) > If FATAL is the minimum log level, there is an exception: no log file will > be available in the browser. This is because FATAL logs will cause the > program to crash. > So if at any point the user can still access mesos page in the browser, it > means no > FATAL logs have been issued. > > > Diffs > ----- > > src/logging/flags.hpp e2e4afccc4d8b8cc269960f90d59aab9c1e53807 > src/logging/logging.hpp 3c24211cb711a43d6f23950607f5a30e1351dd7d > src/logging/logging.cpp a46a3f133830297cbc466e761e1682fb49df09df > src/master/master.cpp 6da776699beb6f449e8160dcb6a125d94c1ab437 > src/slave/slave.cpp d8d3e0fa54972201d72b2650ec0ba922a4912d54 > src/webui/master/static/js/controllers.js > afb24fb9c2184772f7314162f5637dbabaa2ab94 > > Diff: https://reviews.apache.org/r/19357/diff/ > > > Testing > ------- > > 1.Test1 - run master and slave with log_dir flag defined and minloglevel flag > set to WARNING > - expected results: - logs at and above WARNING level are logged into > log_dir and stderr > - the log file for WARNING logs is accessible in > the browser > > 2.Test2 - run master and slave with log_dir flag defined and minloglevel flag > set to FATAL > - expected results: - logs at FATAL level are logged into log_dir and > stderr > - no log file is accessible in the browser > because as soon as a FATAL log will be issued, the program will crash > > 3.Test3 - run master and slave with log_dir flag defined, minloglevel flag > set to INFO and quiet flag defined > - expected results: - logs at and above INFO level are logged into > log_dir and no logs (just FATAL) are displayed at stderr > - the log file for INFO logs is accessible in the > browser > > 4.Test4 - run master and slave with minloglevel flag set to INFO and quiet > flag defined > - expected results: - no logs (just FATAL) are displayed at stderr > (quiet flag takes precedence) > - no log file is accessible in the browser > > 5.Test5 - run master and slave with log_dir and quiet flags defined > - expected results: - no logs (just FATAL) are displayed at stderr > - the log file for INFO logs is accessible in the > browse > > > Thanks, > > Alexandra Sava > >
