]> review.fuel-infra Code Review - openstack-build/cinder-build.git/log
openstack-build/cinder-build.git
10 years agoImported Translations from Transifex
OpenStack Jenkins [Thu, 27 Mar 2014 06:12:40 +0000 (06:12 +0000)]
Imported Translations from Transifex

Change-Id: Ibb7c3b8a65f1356a06bcec5d23f03598a9966315

10 years agoMerge "NetApp cmode nfs: Fix QOS extra spec"
Jenkins [Thu, 27 Mar 2014 04:50:37 +0000 (04:50 +0000)]
Merge "NetApp cmode nfs: Fix QOS extra spec"

10 years agoMerge "Fixes a problem in attach volume in EMC driver."
Jenkins [Thu, 27 Mar 2014 04:38:47 +0000 (04:38 +0000)]
Merge "Fixes a problem in attach volume in EMC driver."

10 years agoMerge "NetApp cmode iscsi: Fix QOS extra spec"
Jenkins [Thu, 27 Mar 2014 04:38:40 +0000 (04:38 +0000)]
Merge "NetApp cmode iscsi: Fix QOS extra spec"

10 years agoMerge "Changes to correct name of missing NetApp license."
Jenkins [Thu, 27 Mar 2014 04:38:32 +0000 (04:38 +0000)]
Merge "Changes to correct name of missing NetApp license."

10 years agoMerge "Pass the mirrorlog option as two arguments."
Jenkins [Thu, 27 Mar 2014 04:38:24 +0000 (04:38 +0000)]
Merge "Pass the mirrorlog option as two arguments."

10 years agoChanges to correct name of missing NetApp license.
Glenn M. Gobeli [Fri, 21 Mar 2014 14:05:07 +0000 (10:05 -0400)]
Changes to correct name of missing NetApp license.

Change to have the NetApp ZAPI error of "SIS clone not licensed"
refer to the correct NetApp license of "FlexClone not licensed".
This code path is encountered when the NetApp 7mode driver uses
the clone interface for snapshotting or cloning.

Change-Id: I3e4d07d38870928a324ad044e03938826b0d0a98
Closes-Bug: #1295349

10 years agoNetApp cmode nfs: Fix QOS extra spec
Alex Meade [Thu, 20 Mar 2014 19:44:44 +0000 (15:44 -0400)]
NetApp cmode nfs: Fix QOS extra spec

This patch fixes the incorrect behavior where the NetApp cmode
nfs driver will choose to create a volume on a flexVol that has
a QOS policy group that matches the specified QOS policy group
intended for the cinder volume. The correct behavior is to ignore
the QOS policy group of the flexVol and instead assign the QOS policy
group to the newly created cinder volume.

Change-Id: I45d905da2a9a07b3ae8c00a225ab3b7f7ceb12d8
Closes-Bug: #1288283

10 years agoNetApp cmode iscsi: Fix QOS extra spec
Alex Meade [Wed, 19 Mar 2014 18:03:15 +0000 (14:03 -0400)]
NetApp cmode iscsi: Fix QOS extra spec

This patch fixes the incorrect behavior where the NetApp cmode
iscsi driver will choose to create a volume on a flexVol that has
a QOS policy group that matches the specified QOS policy group
intended for the cinder volume. The correct behavior is to ignore
the QOS policy group of the flexVol and instead assign the QOS policy
group to the newly created cinder volume.

Change-Id: Ibeb8e3b965af8e79d0294c5cf97410da2b44d40c
Partial-Bug: #1288283

10 years agoFixes a problem in attach volume in EMC driver.
Xing Yang [Sat, 22 Mar 2014 19:05:42 +0000 (15:05 -0400)]
Fixes a problem in attach volume in EMC driver.

This patch fixes a problem in attach volume in EMC SMI-S driver.
The existing logic checks if a volume is already attached to any host,
but it doesn't check whether a volume is already attached to the specific
host that nova wants cinder to attach.  As a result, initialize_connection
could return success (thinking it is already attached), but nova will
fail to discover the LUN later and fail the attach.

This patch adds a check to see if a volume is already attached to a
specific host.  If not, it will do the attach.  The reason that the
volume being attached already could be due to a nova live-migration
use case.  Cinder doesn't support multiple attaches currently, but
allows a volume to be attached multiple times from nova during
live-migration.

Change-Id: I05a2f57cd8708d7fcbe902ec13665a9cfb44db07
Closes-Bug: #1295906

10 years agoUpdate config generator from OSLO
john-griffith [Wed, 26 Mar 2014 22:47:21 +0000 (16:47 -0600)]
Update config generator from OSLO

Cinders sample config is out of date with recent
updates needed for keystone, however we had an
old version of the oslo generator so there was
no way to pick these changes up locally by running
the generate tool.

This pulls the latest generator.py from oslo, and
we run the update script to get us back in sync.

Current head of oslo:
  commit 2eab986ef3c43f8d1e25065e3cbc1307860c25c7
  Merge: 2b08343 0dbcae1
  Author: Jenkins <jenkins@review.openstack.org>
  Date:   Wed Mar 26 14:47:07 2014 +0000

      Merge "Adds test condition in test_periodic"

Change-Id: Id43becfaa39d04153167e592a1233a5aef6016c9

10 years agoPass the mirrorlog option as two arguments.
Vishvananda Ishaya [Wed, 26 Mar 2014 19:09:28 +0000 (12:09 -0700)]
Pass the mirrorlog option as two arguments.

We don't use shell=True when we Popen, so we have to keep all
arguments as separate values.

Change-Id: Ic35fad36be72386d3496920e259c32fe16625046
Closes-bug: 1298017

10 years agoMerge "vmware: Remove pbm_default_policy config option"
Jenkins [Wed, 26 Mar 2014 11:23:30 +0000 (11:23 +0000)]
Merge "vmware: Remove pbm_default_policy config option"

10 years agoMerge "Netapp iscsi: allow snapshots with unspecified block range."
Jenkins [Wed, 26 Mar 2014 08:29:53 +0000 (08:29 +0000)]
Merge "Netapp iscsi: allow snapshots with unspecified block range."

10 years agoNetapp iscsi: allow snapshots with unspecified block range.
Jeff Applewhite [Mon, 24 Mar 2014 21:37:22 +0000 (17:37 -0400)]
Netapp iscsi: allow snapshots with unspecified block range.

Snapshots were failing to create in all cases in the NetApp iscsi
driver. The clone lun function was never issuing the zapi call
because the block count was set to 0.

Closes-Bug: 1296881

Change-Id: Ic2bf67b57a566632103a3fa2ff032555f42a3961

10 years agoSerialize the notification payload
john-griffith [Thu, 20 Mar 2014 03:53:41 +0000 (21:53 -0600)]
Serialize the notification payload

Notification payload in oslomessaging needs to be
serialized correctly, currently it's not.

Use json to encode the payload.
Closes-Bug: 1294949

Change-Id: Idb2c13981cc1c60ed02445a28220d1dab1529d2b

10 years agoMerge "Adds xiv_chap to xiv/ds8k driver configuration"
Jenkins [Tue, 25 Mar 2014 06:25:17 +0000 (06:25 +0000)]
Merge "Adds xiv_chap to xiv/ds8k driver configuration"

10 years agoMerge "Can't force-create snapshot by an non-exist error volume"
Jenkins [Tue, 25 Mar 2014 05:59:23 +0000 (05:59 +0000)]
Merge "Can't force-create snapshot by an non-exist error volume"

10 years agoUpdated from global requirements
OpenStack Jenkins [Mon, 24 Mar 2014 11:47:40 +0000 (11:47 +0000)]
Updated from global requirements

Change-Id: I02c65db0ea9ef2c86d4bf280d811e8cd28deadb9

10 years agoMerge "Allow NetApp iSCSI driver to sub-clone large volumes"
Jenkins [Fri, 21 Mar 2014 08:22:41 +0000 (08:22 +0000)]
Merge "Allow NetApp iSCSI driver to sub-clone large volumes"

10 years agoMerge "init_host should be called before RPC consumer is created"
Jenkins [Fri, 21 Mar 2014 08:22:37 +0000 (08:22 +0000)]
Merge "init_host should be called before RPC consumer is created"

10 years agoAdds xiv_chap to xiv/ds8k driver configuration
alonma [Thu, 20 Mar 2014 17:41:56 +0000 (19:41 +0200)]
Adds xiv_chap to xiv/ds8k driver configuration

In order to support CHAP, xiv_chap was added to
the driver configuration. Valid values are disabled
and enabled (default: disabled)

Change-Id: I0c68f581ad78a87a506b00ab308806f4a8fa78a8
DocImpact
Closes-Bug: #1288816

10 years agoinit_host should be called before RPC consumer is created
wanghong [Thu, 20 Mar 2014 02:39:37 +0000 (10:39 +0800)]
init_host should be called before RPC consumer is created

Currently, the init_host method is called after RPC consumer is
created. This behavior will lead to a bug that when a rpc request
is received the manager can not handle it because the driver has not
been initialized!

Change-Id: Ieedba1adeb3dd98eb4ff78427fe347423c016469
Closes-Bug: #1271568

10 years agoAdd RequestContextSerializer for rpc notifications
john-griffith [Wed, 19 Mar 2014 22:19:28 +0000 (16:19 -0600)]
Add RequestContextSerializer for rpc notifications

RequestContext should be serialized when sent via oslo.messaging. The
serializer is correctly used for the general RPC mechanism, but has been
forgotten in the notifier. This patch fixes that.

Thanks goes to eharney for noticing this and pointing it out!!!

Same bug existed in Nova (1275771)

Change-Id: I99a571ef74bbcd13bf801313eea145ea69f821e1
Closes-Bug: #1294724

10 years agoAllow NetApp iSCSI driver to sub-clone large volumes
Andrew Kerr [Tue, 11 Mar 2014 14:28:55 +0000 (10:28 -0400)]
Allow NetApp iSCSI driver to sub-clone large volumes

The NetApp zapi used during certain extend operations has several limits
imposed on it.  Each block-range provided can only be 2^24 in size, and
there can only be 32 block-ranges per zapi call.  This fix allows the
NetApp iSCSI driver to send multiple zapi calls if necessary, to allow
for extend operations on volumes of an arbitrary size.

Closes-Bug: #1288962
Change-Id: I981d22f32cb2182112fbea3ea9880d1e8c8c91ab

10 years agoMerge "Simplify test force delete snapshot unit test"
Jenkins [Wed, 19 Mar 2014 17:38:05 +0000 (17:38 +0000)]
Merge "Simplify test force delete snapshot unit test"

10 years agoCan't force-create snapshot by an non-exist error volume
lirenke [Tue, 4 Mar 2014 12:59:07 +0000 (04:59 -0800)]
Can't force-create snapshot by an non-exist error volume

If we create an LVM volume whose size larger than
rest space in VG, the volume would not exist with
error status.
But then we still can force create a snapshot
by this volume, and the snapshot's status is available.
In the code,I found in create_lv_snapshot,
it return False when fail to get the volume.
However, raising an exception is only way to be
handle outside.
So, we should raise exception instead of return False.

Closes-Bug: #1283338

Change-Id: I80256f19d66da460c95ff23834abb79a557763bf

10 years agoMerge "resolve KeyError for IBM Storwize/SVC driver"
Jenkins [Tue, 18 Mar 2014 22:58:31 +0000 (22:58 +0000)]
Merge "resolve KeyError for IBM Storwize/SVC driver"

10 years agoSimplify test force delete snapshot unit test
Mike Perez [Mon, 17 Mar 2014 05:24:38 +0000 (22:24 -0700)]
Simplify test force delete snapshot unit test

Make this test simple in just verifying if we have a snapshot and we do
a force delete that we get back an ok response from the controller.
Removing out RPC call and verifying snapshot was deleted since that's
out of the scope here.

Closes-Bug: #1291108
Change-Id: If10b1057cf170e77527a0933a1ddf1a6a59baa1d

10 years agoMerge "Remove unused method from NetApp iscsi driver"
Jenkins [Tue, 18 Mar 2014 21:04:41 +0000 (21:04 +0000)]
Merge "Remove unused method from NetApp iscsi driver"

10 years agoresolve KeyError for IBM Storwize/SVC driver
Xiao Chen [Thu, 6 Mar 2014 10:13:46 +0000 (18:13 +0800)]
resolve KeyError for IBM Storwize/SVC driver

When using iSCSI protocol to conenct IBM v7000, and compression
is not enabled for the system, KeyError may occur because of the response
item's lack of 'license_compression_enclosures'. So add a check to
resolve it.

Change-Id: Ie53631ea5b047650897313ad5614f6e1df5377f2
Closes-Bug: #1288645

10 years agoMerge "VMware: Implement vmdk extend_volume"
Jenkins [Mon, 17 Mar 2014 22:45:54 +0000 (22:45 +0000)]
Merge "VMware: Implement vmdk extend_volume"

10 years agoRemove unused method from NetApp iscsi driver
Alex Meade [Mon, 17 Mar 2014 19:16:09 +0000 (15:16 -0400)]
Remove unused method from NetApp iscsi driver

This removes _get_qos_type() from the NetApp iscsi code
as it is never used.

Change-Id: I903245b5c018bce80bdc0b8edf9ab7241e475fa4

10 years agovmware: Remove pbm_default_policy config option
Vipin Balachandran [Mon, 17 Mar 2014 18:16:22 +0000 (23:46 +0530)]
vmware: Remove pbm_default_policy config option

The pbm_default_policy config option is used to associate a storage
policy with the volume backing if there is no associated volume_type
or the volume_type doesn't have the storage_profile option. A better
approach is to set the storage_profile option in the default_volume_type.
This change removes the pbm_default_policy and queries the policy
using the storage_profile option in the given volume_type.

Closes-Bug: #1291181
Closes-Bug: #1291804
DocImpact

Change-Id: I2519ec1145712ce6927dacde91db468d066af021

10 years agoMerge "Use the error_out_volume from flow common instead"
Jenkins [Mon, 17 Mar 2014 12:07:13 +0000 (12:07 +0000)]
Merge "Use the error_out_volume from flow common instead"

10 years agoVMware: Implement vmdk extend_volume
Vincent Hou [Fri, 7 Mar 2014 16:55:34 +0000 (11:55 -0500)]
VMware: Implement vmdk extend_volume

Add the API implementation of extend_volume for the VMware
vmdk driver.

Change-Id: Idf09c9e9cf015c78c1c5e91c05b897e3e9b7c006
Closes-Bug: #1232172

10 years agoMerge "Don't send untextified exc to webob"
Jenkins [Sun, 16 Mar 2014 15:37:22 +0000 (15:37 +0000)]
Merge "Don't send untextified exc to webob"

10 years agoMerge "GPFS unit tests: increased coverage, uses mock"
Jenkins [Sun, 16 Mar 2014 11:41:00 +0000 (11:41 +0000)]
Merge "GPFS unit tests: increased coverage, uses mock"

10 years agoMerge "Use six.moves.urllib.parse instead of urlparse"
Jenkins [Sun, 16 Mar 2014 10:13:33 +0000 (10:13 +0000)]
Merge "Use six.moves.urllib.parse instead of urlparse"

10 years agoMerge "Add snapshot related data to limits api"
Jenkins [Sun, 16 Mar 2014 06:39:48 +0000 (06:39 +0000)]
Merge "Add snapshot related data to limits api"

10 years agoMerge "Fix create_export/remove_export in driver.py"
Jenkins [Sat, 15 Mar 2014 20:35:45 +0000 (20:35 +0000)]
Merge "Fix create_export/remove_export in driver.py"

10 years agoMerge "NetApp api fix structure conversion methods"
Jenkins [Sat, 15 Mar 2014 18:50:26 +0000 (18:50 +0000)]
Merge "NetApp api fix structure conversion methods"

10 years agoMerge "Switch over to oslosphinx"
Jenkins [Sat, 15 Mar 2014 17:29:03 +0000 (17:29 +0000)]
Merge "Switch over to oslosphinx"

10 years agoFix create_export/remove_export in driver.py
Avishay Traeger [Wed, 26 Feb 2014 09:27:06 +0000 (11:27 +0200)]
Fix create_export/remove_export in driver.py

1. There was a call to rpcapi.create_export which does not have a
matching volume manager function, causing volume migration to crash.
This call was not necessary because there is already an
initialize_connection RPC call, which calls the driver's create_export
function. Removed the create_export RPC call and function. Added better
error handling to that code in _attach_volume in driver.py as well.

2. The manager called remove_export from detach_volume, which was not
being called by these functions. I believe it makes more sense to call
it from terminate_connection. Moved it there, and fixed up the
corresponding code in _detach_volume in driver.py.

3. Remove leftovers of export code from create_volume manager flow.

Change-Id: I2b192630ebed54368f151db47b49cbc72601a8d7
Closes-Bug: #1285060

10 years agoImported Translations from Transifex
OpenStack Jenkins [Sat, 15 Mar 2014 06:10:12 +0000 (06:10 +0000)]
Imported Translations from Transifex

Change-Id: I639b951c583c008b9784e1ea89cc3cda205086a2

10 years agoMerge "Updated from global requirements"
OpenStack Jenkins [Sat, 15 Mar 2014 02:06:49 +0000 (02:06 +0000)]
Merge "Updated from global requirements"

10 years agoMerge "Add missing config values for vmwware_vmdk test"
Jenkins [Sat, 15 Mar 2014 00:47:25 +0000 (00:47 +0000)]
Merge "Add missing config values for vmwware_vmdk test"

10 years agoMerge "Ensure name is utf-8 when deleting rbd vol or snap"
Jenkins [Fri, 14 Mar 2014 22:45:12 +0000 (22:45 +0000)]
Merge "Ensure name is utf-8 when deleting rbd vol or snap"

10 years agoMerge "get volumes with limit and filters does not work"
Jenkins [Fri, 14 Mar 2014 15:57:09 +0000 (15:57 +0000)]
Merge "get volumes with limit and filters does not work"

10 years agoEnsure name is utf-8 when deleting rbd vol or snap
Edward Hope-Morley [Fri, 14 Mar 2014 10:07:45 +0000 (10:07 +0000)]
Ensure name is utf-8 when deleting rbd vol or snap

If the name supplied to librbd is not utf-8 e.g. unicode,
an exception is raised. This was previously fixed and
subsequently broken by commit cbe1d5f.

Change-Id: I29f5e9db87ceb5f9cfe3be43e76d3e860edaf6d7
Closes-Bug: bug 1292433

10 years agoUse six.moves.urllib.parse instead of urlparse
Fengqian Gao [Mon, 10 Mar 2014 02:03:08 +0000 (10:03 +0800)]
Use six.moves.urllib.parse instead of urlparse

To keep Python 3.x compatibility, use six.moves.urllib.parse to
replace urlparse. tools/lintstack is changed to pass pylint test.

Partial-Bug: #1279611

Change-Id: I0ceaaccbf6c67b45609b4f2f5639c362b32d10ad

10 years agoUse the error_out_volume from flow common instead
Joshua Harlow [Thu, 13 Mar 2014 00:31:29 +0000 (17:31 -0700)]
Use the error_out_volume from flow common instead

Instead of a custom piece of logic that errors out
a volume due to the driver not initialized issue
just use the same one that is used in other flows
to accomplish the same update.

Change-Id: I2c8f2fbb4d9bb25a19f26cba3dee5e87913d609b

10 years agoMerge "Imported Translations from Transifex"
OpenStack Jenkins [Fri, 14 Mar 2014 01:19:20 +0000 (01:19 +0000)]
Merge "Imported Translations from Transifex"

10 years agoMerge "Fix exception message of CoraidESMConfigureError"
Jenkins [Fri, 14 Mar 2014 00:10:48 +0000 (00:10 +0000)]
Merge "Fix exception message of CoraidESMConfigureError"

10 years agoMerge "Sync latest Oslo code for imageutils"
Jenkins [Fri, 14 Mar 2014 00:10:40 +0000 (00:10 +0000)]
Merge "Sync latest Oslo code for imageutils"

10 years agoMerge "Revert "Re-enable lazy translation""
Jenkins [Fri, 14 Mar 2014 00:10:37 +0000 (00:10 +0000)]
Merge "Revert "Re-enable lazy translation""

10 years agoMerge "Fixed some FCZM unit tests hacking issues"
Jenkins [Fri, 14 Mar 2014 00:08:34 +0000 (00:08 +0000)]
Merge "Fixed some FCZM unit tests hacking issues"

10 years agoMerge "Fixes cinder-volume service startup on Windows"
Jenkins [Thu, 13 Mar 2014 23:02:09 +0000 (23:02 +0000)]
Merge "Fixes cinder-volume service startup on Windows"

10 years agoMerge "Use debug level logging during unit tests"
Jenkins [Thu, 13 Mar 2014 22:50:26 +0000 (22:50 +0000)]
Merge "Use debug level logging during unit tests"

10 years agoRevert "Re-enable lazy translation"
John Griffith [Mon, 10 Mar 2014 22:00:50 +0000 (22:00 +0000)]
Revert "Re-enable lazy translation"

This reverts commit 6338d4656cb71aeeee2e1a9facb22278e208aa00

Change-Id: I078217b96b2d5fcca78b57c8827883a4b15ef9f0

10 years agoSync latest Oslo code for imageutils
Andrew Kerr [Thu, 13 Mar 2014 19:01:27 +0000 (15:01 -0400)]
Sync latest Oslo code for imageutils

This sync pulls the latest Oslo imageutils code over to cinder
to fix QemuImgInfo parsing needed by NFS drivers.

-> b455fac Enable decimal value input in imageutils.QemuImgInfo
--> cinder/openstack/common/imageutils.py

Oslo version:
-> 0e98afd Re-raise exception of unloadable library
-> Date:   Tue Mar 4 17:26:40 2014 +0100

Change-Id: I770b0f534c6949c2ff6995a03923a21ed26e04da
Closes-Bug: 1285764

10 years agoMerge "cinder-rtstool imports a not existing module"
Jenkins [Thu, 13 Mar 2014 17:43:44 +0000 (17:43 +0000)]
Merge "cinder-rtstool imports a not existing module"

10 years agoDon't send untextified exc to webob
Jay S. Bryant [Wed, 12 Mar 2014 22:39:18 +0000 (17:39 -0500)]
Don't send untextified exc to webob

As part of commit cbe1d5f5e22e5f792128643e4cdd6afb2ff2b5bf I
accidentally removed the str() from exceptions that were being
passed out of OpenStack to webob.  This broke the contract with
webob which was expecting strings (text) to be sent.  This
resulted in webob doing bad things if we encountered an exception,
like trying to create a duplicate volume type.

This commit replaces the str()'s that shouldn't have been removed
with six.text_type() which should be used in place of str() to
resolve this issue.  There was one instance where a message was
instead being unicoded, this also was changed to use six.text_type.

This commit also fixes one message that should have been translated
but was not.

Change-Id: Iad6a81108f58c1bfd13397479022c4c4ad1ccc56
Closes-bug: 1290715

10 years agoMerge "vmware: Mark VMware ESX vmdk driver as deprecated"
Jenkins [Thu, 13 Mar 2014 16:37:05 +0000 (16:37 +0000)]
Merge "vmware: Mark VMware ESX vmdk driver as deprecated"

10 years agoImported Translations from Transifex
OpenStack Jenkins [Thu, 13 Mar 2014 06:12:48 +0000 (06:12 +0000)]
Imported Translations from Transifex

Change-Id: Id72f32cac941e6b65b03d91bfccdb4d5b508c976

10 years agoMerge "Add conversion types in some strings"
Jenkins [Thu, 13 Mar 2014 05:07:23 +0000 (05:07 +0000)]
Merge "Add conversion types in some strings"

10 years agoMerge "Sync log.py from oslo-incubator"
Jenkins [Thu, 13 Mar 2014 05:07:06 +0000 (05:07 +0000)]
Merge "Sync log.py from oslo-incubator"

10 years agoMerge "Add libffi-dev to list of packages to install in dev env"
Jenkins [Thu, 13 Mar 2014 01:58:42 +0000 (01:58 +0000)]
Merge "Add libffi-dev to list of packages to install in dev env"

10 years agoUpdated from global requirements
OpenStack Jenkins [Thu, 13 Mar 2014 01:47:18 +0000 (01:47 +0000)]
Updated from global requirements

Change-Id: I84bf7d09746829a35e71a208abe2e3bfb70208d0

10 years agoUse debug level logging during unit tests
Joe Gordon [Mon, 10 Mar 2014 18:02:13 +0000 (11:02 -0700)]
Use debug level logging during unit tests

When unit tests fail we need to debug, so we should have debug level logging on.
We use fixtures FakeLogger during testing, to enable debug logging pass
in log level as a parameter.

http://bazaar.launchpad.net/~python-fixtures/python-fixtures/trunk/vi
ew/head:/lib/fixtures/_fixtures/logger.py#L68

Taken from nova (Change-Id: I70790aa39c0774726de71e5fa5751c45e7e34ffd)

Change-Id: I24bcce3a9a50322f20f06670d1fe95d929aa2e72

10 years agoMerge "Fixed nova VM live migration issue with 3PAR"
Jenkins [Wed, 12 Mar 2014 19:23:21 +0000 (19:23 +0000)]
Merge "Fixed nova VM live migration issue with 3PAR"

10 years agoMerge "Adding domain to context"
Jenkins [Wed, 12 Mar 2014 19:17:42 +0000 (19:17 +0000)]
Merge "Adding domain to context"

10 years agoSync log.py from oslo-incubator
john-griffith [Wed, 12 Mar 2014 16:59:45 +0000 (10:59 -0600)]
Sync log.py from oslo-incubator

This pulls in the latest version of log.py from oslo-incubator
which elminates the constant user_identity key error.

Note that this sync does a full sync, in other words it uses
the oslo update script and pulls in the log.py file and all
of the associated dependendencies for that module.

Current HEAD in OSLO:
--------------------
Merge: d9ea4f0 fd33d1e
Date:   Wed Mar 12 17:00:13 2014 +0000
Merge "Fix gettextutil.Message handling of deep copy failures"
--------------------

Change-Id: I7c9f8acd22787f9649a5a1e796238c7788a0484a
Fixes-Bug: 1290503

10 years agoFixed some FCZM unit tests hacking issues
Walter A. Boring IV [Mon, 10 Mar 2014 17:27:43 +0000 (10:27 -0700)]
Fixed some FCZM unit tests hacking issues

This patch is a follow up to a review that merged
included and had some issues raised after it was merged.
This patch includes some hacking cleanup and some
unit test cleanup for the Fibre Channel Zone Manager.

Change-Id: Ibb6974b2279f08720c2469893245bc888d35899c
Closes-Bug: #1285029

10 years agoAdd missing config values for vmwware_vmdk test
Sascha Peilicke [Tue, 11 Mar 2014 14:50:38 +0000 (15:50 +0100)]
Add missing config values for vmwware_vmdk test

Import CONF.glance_api_version from cinder.common.config

Closes-Bug: 1290883
Change-Id: I9f64c2f60efe517ad8e4ee2a98ed8da4be257b47

10 years agocinder-rtstool imports a not existing module
Attila Fazekas [Tue, 11 Mar 2014 20:25:45 +0000 (21:25 +0100)]
cinder-rtstool imports a not existing module

cinder.openstack.common.gettextutils should be imported
in cinder-rtstool instead of gettextutils.

Change-Id: I6ce047100c967d72dc31ced0903bc4ad38fecaa4

10 years agoget volumes with limit and filters does not work
Steven Kaufer [Fri, 7 Mar 2014 23:01:51 +0000 (23:01 +0000)]
get volumes with limit and filters does not work

The /volumes and /volumes/detail REST APIs support filtering. Currently,
all filtering is done after the volumes are retrieved from the database
in the API.get_all function in /cinder/volume/api.py. Therefore, the usage
combination of filters and limit will only work if all volumes matching
the filters are in the page of data being retrieved from the database.

For example, assume that all of the volumes with a name of "foo" would be
retrieved from the database starting at index 100 and that you query for
all volumes with a name of "foo" while specifying a limit of 50.  In this
case, the query would yield 0 results since the filter did not match any of
the first 50 entries retrieved from the database.

This patch removes all filtering from the volume API layer and moves it
into the DB layer.

Test cases were added to verify filtering at the DB level.

Change-Id: Ia084e1f4cf59ea39bf8a0a36686146a315168cbb
Closes-bug: 1287813

10 years agoFixes cinder-volume service startup on Windows
Lucian Petrut [Fri, 7 Mar 2014 15:09:33 +0000 (17:09 +0200)]
Fixes cinder-volume service startup on Windows

The Windows service fails due to missing non-blocking IO features
in eventlet. This fix adds a conditional path on Windows to execute
the service accordingly.

This bug had already been fixed by this this commit:
https://review.openstack.org/#/c/44744/ but it had been reverted
here, causing the Windows service to fail again
https://review.openstack.org/67031

Fixes bug: #1219896

Change-Id: I46c6b332e614b6a84ec8372dbec3ee9fb09336de

10 years agoFixed nova VM live migration issue with 3PAR
Walter A. Boring IV [Thu, 6 Mar 2014 19:52:33 +0000 (11:52 -0800)]
Fixed nova VM live migration issue with 3PAR

Nova bypasses the cinder checks for a volume
being available, when it tries to attach a volume
to a new host during live migration.  The assumption
in cinder to this point has been that volumes can only
be attached to one host.  The 3PAR driver worked under
that assumption.  This assumption fell apart during detach
time as the driver was only looking for a VLUN on the
entire 3PAR, since it assumed it could only exist on one host.

This patch ensures that the driver looks for the VLUN on the
hostname it expects.

Change-Id: Ie894ad386990794d270ca1cb72f40095bd40c2e6
Closes-Bug: 1288927

10 years agoAdding domain to context
Luis A. Garcia [Tue, 11 Mar 2014 15:34:02 +0000 (09:34 -0600)]
Adding domain to context

Add support for upcoming "domain" concept in Keystone V3 API in
the context.

This fix ensures there is a user_identity in the context for the
corresponding new attribute in the log format string added in oslo. The
oslo change has the same change-id as this one.

Closes-Bug: #1290503
Change-Id: Ic2cf3e52cfcc0b8adccdf9c59afaa4014708a303

10 years agoSwitch over to oslosphinx
Dirk Mueller [Thu, 13 Feb 2014 13:54:44 +0000 (14:54 +0100)]
Switch over to oslosphinx

oslosphinx is the new name of oslo.sphinx

Closes-Bug: #1277168
Change-Id: Ib30a4a5eb3a86ad5c03324a59e1a8ee4d4897a70

10 years agoAdd libffi-dev to list of packages to install in dev env
Joe Gordon [Tue, 11 Mar 2014 03:34:48 +0000 (03:34 +0000)]
Add libffi-dev to list of packages to install in dev env

libffi-dev is needed by cffi

Change-Id: Ib91bd2e1a1005dc96f9d9054ae5cb150c5fae7db

10 years agoVMware: Take the volume size from the user input
Vincent Hou [Wed, 4 Dec 2013 08:51:20 +0000 (03:51 -0500)]
VMware: Take the volume size from the user input

When we create a volume from an image or a snapshot, we need to
take the size of the volume from the user input and validate whether
the size is appropriate instead of taking the image size as the
volume size directly.

Change-Id: If09933d8ffa989c4dacc0860c19ea332bc21092a
Closes-Bug: #1237557

10 years agoFix exception message of CoraidESMConfigureError
KIYOHIRO ADACHI [Tue, 11 Mar 2014 01:17:32 +0000 (10:17 +0900)]
Fix exception message of CoraidESMConfigureError

Since 'message' is the same name as the second argument to the
constructor of CinderException, 'message="Oorah"' is not a keyword
argument.

Therefore, "message = _('ESM configure request failed: %(message)s.')"
doesn't work correctly.

Then, remove the redundant period.

Change-Id: Iafc457197fc3993f9f942babcdda0cd79b7bd518
Closes-Bug: #1290627

10 years agovmware: Mark VMware ESX vmdk driver as deprecated
Vipin Balachandran [Tue, 11 Mar 2014 00:21:03 +0000 (05:51 +0530)]
vmware: Mark VMware ESX vmdk driver as deprecated

The VMware ESX driver in Nova is now marked as deprecated and will be
removed in Juno release. This change marks the VMware ESX vmdk driver
as deprecated since it is intended for use with the ESX driver in Nova.
The plan is to remove the ESX vmdk driver in Juno release.

Closes-Bug: #1291194
DocImpact
Change-Id: I285f3461c1ccef56329b70417fb023405dfceb5e

10 years agoMerge "Fixes ssh-injection error while using chap authentication"
Jenkins [Wed, 12 Mar 2014 05:37:44 +0000 (05:37 +0000)]
Merge "Fixes ssh-injection error while using chap authentication"

10 years agoMerge "Generate config samples for oslo.messaging"
Jenkins [Tue, 11 Mar 2014 21:04:21 +0000 (21:04 +0000)]
Merge "Generate config samples for oslo.messaging"

10 years agoFixes ssh-injection error while using chap authentication
Jacob M. Jacob [Mon, 10 Mar 2014 22:25:41 +0000 (17:25 -0500)]
Fixes ssh-injection error while using chap authentication

A space in the command construction was being caught by the
ssh-injection check. The fix is to separate the command strings.

Change-Id: If1f719f9c2ceff31ed5386c53cf60bc7f522f4d7
Closes-Bug: #1280409

10 years agoGenerate config samples for oslo.messaging
Flavio Percoco [Tue, 11 Mar 2014 09:33:45 +0000 (10:33 +0100)]
Generate config samples for oslo.messaging

This patch adds oslo.messaging to the
CINDER_CONFIG_GENERATOR_EXTRA_LIBRARIES env variable, which adds
oslo.messaging config options to cinder.conf.sample

Implements blueprint: oslo-messaging

Change-Id: I8aed8a8fdbd5b4ef41e3a3a7dcdebc6f216ec75d

10 years agoAdd conversion types in some strings
KIYOHIRO ADACHI [Fri, 7 Mar 2014 09:31:26 +0000 (18:31 +0900)]
Add conversion types in some strings

Some strings have no conversion types
for 'String Formatting Operations'.

Change-Id: I2c6176954643a93898ff2d06d686793fb0977458
Closes-Bug: #1289230

10 years agoMerge "3PAR: Support extend volume based on snapshot"
Jenkins [Tue, 11 Mar 2014 00:19:48 +0000 (00:19 +0000)]
Merge "3PAR: Support extend volume based on snapshot"

10 years agoMerge "Port to oslo.messaging"
Jenkins [Mon, 10 Mar 2014 21:12:07 +0000 (21:12 +0000)]
Merge "Port to oslo.messaging"

10 years agoMerge "Updated from global requirements"
OpenStack Jenkins [Mon, 10 Mar 2014 20:36:55 +0000 (20:36 +0000)]
Merge "Updated from global requirements"

10 years agoMerge "Convert cinder utils tests to use mock"
Jenkins [Mon, 10 Mar 2014 20:36:52 +0000 (20:36 +0000)]
Merge "Convert cinder utils tests to use mock"

10 years agoPort to oslo.messaging
Flavio Percoco [Fri, 7 Feb 2014 11:20:44 +0000 (12:20 +0100)]
Port to oslo.messaging

The oslo.messaging library takes the existing RPC code from oslo and
wraps it in a sane API with well defined semantics around which we can
make a commitment to retain compatibility in future.

The patch is large, but the changes can be summarized as:

    * oslo.messaging>=1.3.0a4 is required; a proper 1.3.0 release will be
    pushed before the icehouse release candidates.

    * The new rpc module has init() and cleanup() methods which manage the
    global oslo.messaging transport state. The TRANSPORT and NOTIFIER
    globals are conceptually similar to the current RPCIMPL global,
    except we're free to create and use alternate Transport objects
    in e.g. the cells code.

    * The rpc.get_{client,server,notifier}() methods are just helpers
    which wrap the global messaging state, specifiy serializers and
    specify the use of the eventlet executor.

    * In oslo.messaging, a request context is expected to be a dict so
    we add a RequestContextSerializer which can serialize to and from
    dicts using RequestContext.{to,from}_dict()

    * The allowed_rpc_exception_modules configuration option is replaced
    by an allowed_remote_exmods get_transport() parameter. This is not
    something that users ever need to configure, but it is something
    each project using oslo.messaging needs to be able to customize.

    * We maintain a global NOTIFIER object and create specializations of
    it with specific publisher IDs in order to avoid notification driver
    loading overhead.

    * rpc.py contains transport aliases for backwards compatibility
    purposes. setup.cfg also contains notification driver aliases for
    backwards compat.

    * The messaging options are moved about in cinder.conf.sample because
    the options are advertised via a oslo.config.opts entry point and
    picked up by the generator.

    * We use messaging.ConfFixture in tests to override oslo.messaging
    config options, rather than making assumptions about the options
    registered by the library.

Implements blueprint: oslo-messaging

Change-Id: Ib912809428d92e788558439e2d85b51272ebefdd

10 years agoUpdated from global requirements
OpenStack Jenkins [Mon, 10 Mar 2014 14:44:55 +0000 (14:44 +0000)]
Updated from global requirements

Change-Id: I864af28306bd6d3efc3c1101e3510456465b9a66

10 years agoMerge "vmware: PBM wsdl file configuration"
Jenkins [Sat, 8 Mar 2014 09:44:01 +0000 (09:44 +0000)]
Merge "vmware: PBM wsdl file configuration"

10 years agoMerge "vmware: default global pbm policy configuration"
Jenkins [Sat, 8 Mar 2014 09:43:52 +0000 (09:43 +0000)]
Merge "vmware: default global pbm policy configuration"

10 years agoMerge "vmware: check datastore availability during create"
Jenkins [Sat, 8 Mar 2014 09:42:04 +0000 (09:42 +0000)]
Merge "vmware: check datastore availability during create"

10 years agoMerge "vmware: Storage policy based volume placement."
Jenkins [Sat, 8 Mar 2014 09:42:00 +0000 (09:42 +0000)]
Merge "vmware: Storage policy based volume placement."