From: john-griffith <john.griffith@solidfire.com>
Date: Wed, 19 Mar 2014 22:19:28 +0000 (-0600)
Subject: Add RequestContextSerializer for rpc notifications
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=e145ef798d2c80ea48be2826f4c648e72958f5d1;p=openstack-build%2Fcinder-build.git

Add RequestContextSerializer for rpc notifications

RequestContext should be serialized when sent via oslo.messaging. The
serializer is correctly used for the general RPC mechanism, but has been
forgotten in the notifier. This patch fixes that.

Thanks goes to eharney for noticing this and pointing it out!!!

Same bug existed in Nova (1275771)

Change-Id: I99a571ef74bbcd13bf801313eea145ea69f821e1
Closes-Bug: #1294724
---

diff --git a/cinder/rpc.py b/cinder/rpc.py
index d539cfd30..bb34f950d 100644
--- a/cinder/rpc.py
+++ b/cinder/rpc.py
@@ -60,7 +60,8 @@ def init(conf):
     TRANSPORT = messaging.get_transport(conf,
                                         allowed_remote_exmods=exmods,
                                         aliases=TRANSPORT_ALIASES)
-    NOTIFIER = messaging.Notifier(TRANSPORT)
+    NOTIFIER = messaging.Notifier(TRANSPORT,
+                                  serializer=RequestContextSerializer(None))
 
 
 def initialized():