<div>
<span style="font-size: 12px;">Still needing reviews. I've addressed all the concerns to date… and to make it easier for reviewers, I've split out the generic testing changes and the matchmaker addition (as dependencies).</span>
</div><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px;">The changes are:</span></div><div><span style="font-size: 12px;"><a href="https://review.openstack.org/#/c/7633/">https://review.openstack.org/#/c/7633/</a> # zeromq</span></div><div><a href="https://review.openstack.org/#/c/7921/2">https://review.openstack.org/#/c/7921/2</a> # matchmaker </div>
<div><div><a href="https://review.openstack.org/#/c/7770/">https://review.openstack.org/#/c/7770/</a> # new common rpc tests and fake_impl.py bugfix</div><div><span style="font-size: 12px;"><br></span></div>-- <br>Eric Windisch<div><br></div></div>
<p style="color: #A0A0A8;">On Wednesday, May 23, 2012 at 11:34 AM, Eric Windisch wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div>
<div>
<span style="font-size: 12px;">Looking for code reviews of the ZeroMQ driver:</span>
</div><div><span style="font-size: 12px;"><a href="https://review.openstack.org/#/c/7633/">https://review.openstack.org/#/c/7633/</a></span></div><div><br></div><div><span style="font-size: 12px;">I believe I have addressed all the concerns of the previous reviewers and have ironed out all the pep8/hacking and unit testing issues.</span></div>
<div><div><br></div><div><span style="font-size: 12px;">This patch also introduces the matchmaker and two new common RPC unit tests (and a bug-fix for nova/rpc/impl_fake.py)</span></div><div><br></div>-- <br>Eric Windisch<div><br></div></div>
</div></div></span>
</blockquote>
<div>
<br>
</div>