← Back to team overview

openstack team mailing list archive

Re: ZeroMQ driver (review needed)

 

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).  

The changes are:
https://review.openstack.org/#/c/7633/  # zeromq
https://review.openstack.org/#/c/7921/2  # matchmaker  
https://review.openstack.org/#/c/7770/ # new common rpc tests and fake_impl.py bugfix

--  
Eric Windisch


On Wednesday, May 23, 2012 at 11:34 AM, Eric Windisch wrote:

> Looking for code reviews of the ZeroMQ driver:  
> https://review.openstack.org/#/c/7633/
>  
> I believe I have addressed all the concerns of the previous reviewers and have ironed out all the pep8/hacking and unit testing issues.  
>  
> This patch also introduces the matchmaker and two new common RPC unit tests (and a bug-fix for nova/rpc/impl_fake.py)
>  
> --  
> Eric Windisch
>  


References