OpenStack Compute (nova)

Code review comment for lp:~cerberus/nova/nova_notifications

  1. nova_notifications
  2. Merge into trunk
Revision history for this message
Brian Waldon (bcwaldon) wrote :

147-154, 179, 217-224: Can we use "message" instead of "payload"? The naming is a little confusing.

Would you mind documenting the notifier drivers a bit better through a base class?

review: Needs Fixing

« Back to merge proposal