Re: [openstack-dev] [cinder]Driver broken

2015-03-25 Thread Eduard Matei
Thanks,

Indeed, our CI doesn't test this code path, we focused mainly on our
internal functionality tests (which don't test for this), and the tempest
tests are not running.
This will be fixed asap.

Thanks everyone,
Eduard
__
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]Driver broken

2015-03-25 Thread Walter A. Boring IV
This is a real defect related to the multiattach patch that I worked 
on.   I have posted a fix for your driver.


https://review.openstack.org/#/c/167683/

Walt

Hi,

Just reported an issue: https://bugs.launchpad.net/cinder/+bug/1436367
Seems to be related to https://review.openstack.org/#/c/85847/ which 
introduced another parameter to be passed to the driver, but our 
driver didn't get updated so detach_volume fails for us.


How can we get this fixed asap?

Thanks,
Eduard



__
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]Driver broken

2015-03-25 Thread Eduard Matei
Hi,

Indeed our CI runs, and it reported SUCCESS, but it didn't run the tempest
tests, only the openvstorage tests (my bad, looking into it as we speak).

Meanwhile i'll create a patch to fix this.

Thanks,
Eduard
__
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]Driver broken

2015-03-25 Thread Anita Kuno
On 03/25/2015 11:44 AM, Jordan Pittier wrote:
> Hey,
> 
> From
> http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9/console.html
> :
> "Running ` python setup.py testr --testr-args='--subunit --concurrency 1
>  test_openvstorage'`"
> 
> ==
> Totals
> ==
> Ran: 10 tests in 2. sec.
>  - Passed: 10
>  - Skipped: 0
>  - Expected Fail: 0
>  - Unexpected Success: 0
>  - Failed: 0
> Sum of execute time for each test: 0.4477 sec.
> 
> 
> So his CI did run :)
> 
> Jordan
Part of the process of running a CI is to be able to answer questions
about your own system.

As a operator of a system maintaining a driver, Eduard, the solution for
fixing your driver is to offer a patch to fix it.

Thanks,
Anita.
> 
> 
> On Wed, Mar 25, 2015 at 4:31 PM, Asselin, Ramy  wrote:
> 
>>  Hi Eduard,
>>
>>
>>
>> Your third party ci reported success on that patch. The tempest volume
>> tests include attached & detaches. Seems your CI is not running them?
>>
>>
>> http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9/console.html
>>
>>
>>
>> *CloudFounders OpenvStorage CI check*
>>
>> *Mar 10, 2015 9:14 AM*
>>
>> openvstorage-cinder-functionality
>> <http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9>
>>
>> SUCCESS in 37m 16s
>>
>>
>>
>> Ramy
>>
>>
>>
>> *From:* Eduard Matei [mailto:eduard.ma...@cloudfounders.com]
>> *Sent:* Wednesday, March 25, 2015 8:05 AM
>> *To:* OpenStack Development Mailing List (not for usage questions)
>> *Subject:* [openstack-dev] [cinder]Driver broken
>>
>>
>>
>> Hi,
>>
>>
>>
>> Just reported an issue: https://bugs.launchpad.net/cinder/+bug/1436367
>>
>> Seems to be related to https://review.openstack.org/#/c/85847/ which
>> introduced another parameter to be passed to the driver, but our driver
>> didn't get updated so detach_volume fails for us.
>>
>>
>>
>> How can we get this fixed asap?
>>
>>
>>
>> Thanks,
>>
>> Eduard
>>
>>
>>
>> __
>> 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 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]Driver broken

2015-03-25 Thread Jordan Pittier
Hey,

From
http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9/console.html
:
"Running ` python setup.py testr --testr-args='--subunit --concurrency 1
 test_openvstorage'`"

==
Totals
==
Ran: 10 tests in 2. sec.
 - Passed: 10
 - Skipped: 0
 - Expected Fail: 0
 - Unexpected Success: 0
 - Failed: 0
Sum of execute time for each test: 0.4477 sec.


So his CI did run :)

Jordan


On Wed, Mar 25, 2015 at 4:31 PM, Asselin, Ramy  wrote:

>  Hi Eduard,
>
>
>
> Your third party ci reported success on that patch. The tempest volume
> tests include attached & detaches. Seems your CI is not running them?
>
>
> http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9/console.html
>
>
>
> *CloudFounders OpenvStorage CI check*
>
> *Mar 10, 2015 9:14 AM*
>
> openvstorage-cinder-functionality
> <http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9>
>
> SUCCESS in 37m 16s
>
>
>
> Ramy
>
>
>
> *From:* Eduard Matei [mailto:eduard.ma...@cloudfounders.com]
> *Sent:* Wednesday, March 25, 2015 8:05 AM
> *To:* OpenStack Development Mailing List (not for usage questions)
> *Subject:* [openstack-dev] [cinder]Driver broken
>
>
>
> Hi,
>
>
>
> Just reported an issue: https://bugs.launchpad.net/cinder/+bug/1436367
>
> Seems to be related to https://review.openstack.org/#/c/85847/ which
> introduced another parameter to be passed to the driver, but our driver
> didn't get updated so detach_volume fails for us.
>
>
>
> How can we get this fixed asap?
>
>
>
> Thanks,
>
> Eduard
>
>
>
> __
> 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]Driver broken

2015-03-25 Thread Asselin, Ramy
Hi Eduard,

Your third party ci reported success on that patch. The tempest volume tests 
include attached & detaches. Seems your CI is not running them?
http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9/console.html

CloudFounders OpenvStorage CI check

Mar 10, 2015 9:14 AM

openvstorage-cinder-functionality<http://packages.cloudfounders.com/ci_logs/47/85847/47/check/openvstorage-cinder-functionality/b073fb9>

SUCCESS in 37m 16s


Ramy

From: Eduard Matei [mailto:eduard.ma...@cloudfounders.com]
Sent: Wednesday, March 25, 2015 8:05 AM
To: OpenStack Development Mailing List (not for usage questions)
Subject: [openstack-dev] [cinder]Driver broken

Hi,

Just reported an issue: https://bugs.launchpad.net/cinder/+bug/1436367
Seems to be related to https://review.openstack.org/#/c/85847/ which introduced 
another parameter to be passed to the driver, but our driver didn't get updated 
so detach_volume fails for us.

How can we get this fixed asap?

Thanks,
Eduard

__
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]Driver broken

2015-03-25 Thread Eduard Matei
Hi,

Just reported an issue: https://bugs.launchpad.net/cinder/+bug/1436367
Seems to be related to https://review.openstack.org/#/c/85847/ which
introduced another parameter to be passed to the driver, but our driver
didn't get updated so detach_volume fails for us.

How can we get this fixed asap?

Thanks,
Eduard
__
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