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 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 always ask whether the driver is merged into Cinder,
 please see my comments in nova spec
 https://review.openstack.org/#/c/130919/, thank you very much!







 Best regards



 ZhangNi



 __
 OpenStack Development Mailing List (not for usage questions)
 Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


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


$0.02
Walt


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 always ask whether the driver is merged into 
Cinder, please see my comments in nova spec 
https://review.openstack.org/#/c/130919/, thank you very much!


Best regards

ZhangNi



__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[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 always ask whether the driver is merged into Cinder, please 
see my comments in nova spec https://review.openstack.org/#/c/130919/, thank 
you very much!



Best regards

ZhangNi

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev