Re: [openstack-dev] [Devstack] add support for ceph

2014-04-22 Thread Soren Hansen
2014-04-18 12:32 GMT+02:00 Sean Dague : > On 04/18/2014 12:03 AM, Scott Devoid wrote: >> So I have had a chance to look over the whole review history again. I >> agree with Sean Dague and Dean Troyer's concerns that the current >> patch affects code outside of lib/storage and extras.d. We should >>

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-18 Thread Scott Devoid
On Fri, Apr 18, 2014 at 11:41 AM, Dean Troyer wrote: > On Fri, Apr 18, 2014 at 10:51 AM, Scott Devoid wrote: > >> The issue is that it is very easy to suggest new features and refactoring >> when you are very familiar with the codebase. To a newcomer, though, you >> are basically asking me to do

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-18 Thread Dean Troyer
On Fri, Apr 18, 2014 at 10:51 AM, Scott Devoid wrote: > The issue is that it is very easy to suggest new features and refactoring > when you are very familiar with the codebase. To a newcomer, though, you > are basically asking me to do something that is impossible, so the logical > interpretatio

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-18 Thread Scott Devoid
On Fri, Apr 18, 2014 at 5:32 AM, Sean Dague wrote: > On 04/18/2014 12:03 AM, Scott Devoid wrote: > > So I have had a chance to look over the whole review history again. I > > agree with Sean Dague and Dean Troyer's concerns that the current patch > > affects code outside of lib/storage and extras

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-18 Thread Chmouel Boudjnah
On Fri, Apr 18, 2014 at 6:32 AM, Sean Dague wrote: > That being said, there are 2 devstack sessions available at design > summit. So proposing something around addressing the ceph situation > might be a good one. It's a big and interesting problem. > I have add a session that just do that here

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-18 Thread Sean Dague
On 04/18/2014 12:03 AM, Scott Devoid wrote: > So I have had a chance to look over the whole review history again. I > agree with Sean Dague and Dean Troyer's concerns that the current patch > affects code outside of lib/storage and extras.d. We should make the > Devstack extension system more flexi

Re: [openstack-dev] [Devstack] add support for ceph

2014-04-17 Thread Scott Devoid
So I have had a chance to look over the whole review history again. I agree with Sean Dague and Dean Troyer's concerns that the current patch affects code outside of lib/storage and extras.d. We should make the Devstack extension system more flexible to allow for more extensions. Although I am not