Lower log level of "Scheduler type KIST has been enabled"

I like seeing it, but that doesn't mean everyone should have to see it. Tor doesn't log about EWMA vs RR.

changed milestone to %Tor: 0.3.5.x-final in legacy/trac

added 035-triaged-in-20180711 in Legacy / Trac component::core tor/tor in Legacy / Trac easy in Legacy / Trac kist in Legacy / Trac milestone::Tor: 0.3.5.x-final in Legacy / Trac priority::very low in Legacy / Trac resolution::fixed in Legacy / Trac reviewer::dgoulet in Legacy / Trac severity::trivial in Legacy / Trac status::closed in Legacy / Trac tor-scheduler in Legacy / Trac type::defect in Legacy / Trac version::tor 0.3.2.9 in Legacy / Trac labels

In the early days of KIST, having that line was SO useful. I propose we put it in the summary line (if we can or in a new type of summary line) like this one:

[notice] Tor 0.3.4.0-alpha-dev (git-facde43b0e0ad3ea) running on Linux with Libevent 2.1.8-stable, OpenSSL 1.0.2n, Zlib 1.2.11, Liblzma 5.2.2, and Libzstd N/A.

Trac:
Keywords: N/A deleted, easy added
Type: enhancement to defect

See https://gitweb.torproject.org/user/pastly/tor.git/log/?h=issue26703 for changing it to INFO.

I would support moving it to the summary.

FWIW, I gave 5 minutes of effort to adding the scheduler type to the summary. It not as easy as log_info because the required stuff (get_scheduler_type_string() and the_scheduler) is static. Furthermore (and actually importantly), I don't think we've initialized the scheduler at the point the summary is logged.

Working off the assumption that we don't want to overcome these challenges, I'm moving this to needs_review.

Trac:
Status: new to needs_review

Trac:
Keywords: easy deleted, easy kist tor-scheduler added

Trac:
Keywords: easy kist tor-scheduler deleted, 035-triaged-in-20180711, kist, tor-scheduler, easy added

Trac:
Reviewer: N/A to dgoulet

Trivial fix. I'm good with log_info for now and we can work out later to have it in a status line.

Merged!

Trac:
Status: needs_review to closed
Resolution: N/A to fixed

closed

moved from legacy/trac#26703 (moved)

added Bug First Contribution labels and removed 1 deleted label

removed 1 deleted label