Re: [openstack-dev] [cinder]Request to to revisit this patch

2015-03-15 Thread Jay Bryant
I agree with Walt that reverting this is the right answer. Please make sure to have your ci running as well for liberty. Jay On Mar 4, 2015 2:30 AM, Zhangni zhan...@huawei.com wrote: Hi Mike, Jay, and Walter, Please revisit this patch https://review.openstack.org/#/c/151959/ and don’t

Re: [openstack-dev] [cinder]Request to to revisit this patch

2015-03-04 Thread Walter A. Boring IV
Since the Nova side isn't in, and won't land for Kilo, then there is no reason for Cinder to have it for Kilo, as it will simply not work. We can revisit this for the L release if you like. Also, make sure you have 3rd Party CI setup for this driver, or it won't be accepted in the L release

[openstack-dev] [cinder]Request to to revisit this patch

2015-03-04 Thread Zhangni
Hi Mike, Jay, and Walter, Please revisit this patch https://review.openstack.org/#/c/151959/ and don't revert this, thank you very much! I think it's apposite to merge the SDSHypervisor driver in cinder first, and next to request nova to add a new libvirt volume driver. Meanwhile nova side