On 06/03/14 11:50 +0100, Thierry Carrez wrote:
Flavio Percoco wrote:I'd like to request a FFE for the oslo.messaging migration in Cinder.Some projects have already switched over oslo.messaging and others are still doing so. I think we should switch remaining projects to oslo.messaging as soon as possible and keep the RPC library in use consistent throughout OpenStack. Cinder's patch has been up for review for a couple of weeks already and it's been kept updated with master. Besides some of the gate failures we've had in the last couple of weeks, it seems to work as expected. As a final note, most of the work on this patch followed the style and changes done in Nova, for better or for worse. The review link is: https://review.openstack.org/#/c/71873/So on one hand this is a significant change that looks like it could wait (little direct feature gain). On the other we have oslo.messaging being adopted in a lot of projects, and we reduce the maintenance envelope if we switch most projects to it BEFORE release. This one really boils down to how early it can be merged. If it's done before the meeting next Tuesday, it's a net gain. If not, it becomes too much of a distraction from bugfixes for reviewers and any regression it creates might get overlooked.
FWIW, I just rebased it earlier today and the patch could be merged today if it gets enough reviews. Cheers, Fla. -- @flaper87 Flavio Percoco
pgpC_dt24wcQj.pgp
Description: PGP signature
_______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev