BIND 10 #1190: complete logging conversion for msgq

BIND 10 Development do-not-reply at isc.org
Wed Dec 19 13:06:54 UTC 2012


#1190: complete logging conversion for msgq
-------------------------------------+-------------------------------------
            Reporter:  jreed         |                        Owner:
                Type:  defect        |  UnAssigned
            Priority:  medium        |                       Status:
           Component:  msgq          |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130108
         Sub-Project:  Core          |                   Resolution:
Estimated Difficulty:  5 (for the    |                 CVSS Scoring:
  workaround version)                |              Defect Severity:  Low
         Total Hours:  0             |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => UnAssigned
 * status:  accepted => reviewing


Comment:

 Hello

 It is ready for review. Currently, the msgq uses the logging system and
 outputs
 either INFO or DEBUG/99, depending on if -v is provided.

 However, I didn't make it output to file, it outputs to standard output.
 It is
 quite hard to make it output to file, because:
  * The `isc.log.init` doesn't take any parameter that would change it.
  * To call `isc.log.log_config_update`, one needs to have the spec file
 data.
    But to have that, we should be connected to the CC and then, we would
 use
    the usual logging auto-configuration.
  * Updating the wrappers to add a method to modify it sounds like too much
 for
    this workaround ticket, considering the method wouldn't be needed when
 we
    adopt a real approach.
  * Tweaking `B10_LOGGER_` environment variables and then calling
    `isc.log.resetUnitTestRootLogger` sounds like too ugly hack even for
 the
    workaround.

 So, what I propose is merging this ticket as it is (after the review, of
 course), as it provides the conversion of messages. Then we create a new
 ticket
 to connect msgq to itself. I think it should be quite easy:
  * Wait for the ConfigManager to connect and subscribe (otherwise it won't
 work).
  * Create a connection to the socket using common cc library, with
 allowing the
    logging autoconfiguration.
  * Provide an empty handler (or handler with only 'shutdown' command, or
 something).
  * Push the cc file descriptor to the poller/kqueue to watch over.

 Also, this probably calls for a changelog entry:
 {{{
 The b10-msgq now uses the same logging format as the rest of the system.
 However,
 it still doesn't obey the common configuration, as due to technical issues
 it is
 not able to read it yet.
 }}}

-- 
Ticket URL: <http://bind10.isc.org/ticket/1190#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list