BIND 10 #3095: Generic traceback handling

BIND 10 Development do-not-reply at isc.org
Thu Sep 12 13:37:22 UTC 2013


#3095: Generic traceback handling
-------------------------------------+-------------------------------------
            Reporter:  vorner        |                        Owner:
                Type:  enhancement   |  vorner
            Priority:  medium        |                       Status:
           Component:  Unclassified  |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130917
         Sub-Project:  Core          |                   Resolution:
Estimated Difficulty:  4             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by muks):

 * owner:  muks => vorner


Comment:

 Hi Michal

 Here is my review of the branch:

 * It seems to pass `make distcheck`.

 * In `util_messages.mes`, the following comment must be fixed:
 {{{
 +# No namespace declaration - these constants go in the global namespace
 +# of the libddns_messages python module.
 }}}

 * I think it's better to rename `UNHANDLED_EXCEPTION` message id to
   `PYTHON_UNHANDLED_EXCEPTION`, as this message id is used specifically
   for Python components.

 * Do we want to use this traceback handler for `dbutil` and `loadzone`
   too? I'm just raising this point; I don't have a strong preference.

 * I have pushed some minor commits which I didn't feel needed more
   discussing. Please check if they are okay.

 Apart from these, the branch is perfect, so please go ahead and merge
 after addressing the above.

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


More information about the bind10-tickets mailing list