Code review comment for lp:~tim-simpson/nova/multiple-notifiers

Revision history for this message
Sandy Walsh (sandy-walsh) wrote :

Looks good ... minor nits

+34 single _ for private functions

+38 needs some error handling. If the import fails it'll bring down the service. It should simply not load that driver and log an error.

review: Needs Fixing

« Back to merge proposal