Comment 21 for bug 2030976

Revision history for this message
Dan Smith (danms) wrote :

Jeremy, that's fine as it is. However, it might be good to use a different word than "payload" for clarity. The whole thing is a payload, but there's also a payload field, which is what nova (et al) asked to be sent as the notification. The distinction may be important because nova itself has no control over anything *but* the actual payload field, and the context is included in the top-level message by oslo.messaging.

I would just s/payloads/messages/ perhaps if you agree the disambiguation is worthwhile. Otherwise, looks fine to me.