Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-30 Thread Jay S. Bryant
Ben, Thank you very much for the history on the driver here. This was all news to me. It helps me to understand why cloning was listed as supported but was no longer implemented. Also explains why the simpler 'cp' based solution hasn't been implemented. I was wondering why that hadn't

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-26 Thread Erlon Cruz
He is mentioning about the Cinder side: https://review.openstack.org/# /c/147186/ On Fri, Aug 26, 2016 at 7:01 AM, Jordan Pittier wrote: > > > On Thu, Aug 25, 2016 at 7:06 PM, Ben Swartzlander > wrote: > >> Originally the NFS driver did

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-26 Thread Jordan Pittier
On Thu, Aug 25, 2016 at 7:06 PM, Ben Swartzlander wrote: > Originally the NFS driver did support snapshots, but it was implemented by > just 'cp'ing the file containing the raw bits. This works fine (if > inefficiently) for unattached volumes, but if you do this on an

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-25 Thread Slade Baumann
quot; <openstack-dev@lists.openstack.org>, Erlon Cruz <sombra...@gmail.com>From: Ben Swartzlander <b...@swartzlander.org>Date: 08/25/2016 12:10PMSubject: Re: [openstack-dev] [tempest][cinder] Clone feature toggle not inclone tests Originally the NFS driver did support

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-25 Thread Ben Swartzlander
Originally the NFS driver did support snapshots, but it was implemented by just 'cp'ing the file containing the raw bits. This works fine (if inefficiently) for unattached volumes, but if you do this on an attached volume the snapshot won't be crash consistent at all. It was decided that we

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-25 Thread Erlon Cruz
Hi Jordan, Slade, Currently NFS driver does not support cloning neither snapshots (which are the base for implementing cloning). AFAIC, the NFS driver was in Cinder before the minimum requirements being discussed and set, so, it just stood there with the features it already supported. There is

Re: [openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-24 Thread Jordan Pittier
On Wed, Aug 24, 2016 at 6:06 PM, Slade Baumann wrote: > I am attempting to disable clone tests in tempest as they aren't > functioning in NFS. But the tests test_volumes_clone.py and > test_volumes_clone_negative.py don't have the "clone" feature > toggle in them. I thought

[openstack-dev] [tempest][cinder] Clone feature toggle not in clone tests

2016-08-24 Thread Slade Baumann
I am attempting to disable clone tests in tempest as they aren'tfunctioning in NFS. But the tests test_volumes_clone.py andtest_volumes_clone_negative.py don't have the "clone" featuretoggle in them. I thought it obvious that if clone is disabledin tempest, the tests that simply clone should be