Commit Graph

1141 Commits

Author SHA1 Message Date
Jenkins
072cd9c004 Merge "Add error reporting to generate_sample.sh on import failure." 2013-06-02 16:00:28 +00:00
Jenkins
5efce1a99e Merge "Update 3PAR driver session management" 2013-06-02 10:43:58 +00:00
Jenkins
5ff9a8ae43 Merge "HDS Cinder Driver. Rev #1" 2013-06-02 08:56:28 +00:00
Dirk Mueller
50667f178e Fix path for pylint Gate
Change-Id: Ibd36a2a0a7ebe572a4dd660a3e35378ae02b8187
2013-06-01 16:20:45 +02:00
Lakhinder Walia
da00b6bcca HDS Cinder Driver. Rev #1
blueprint hds-hus-iscsi-cinder-driver
This is the first rev of Hitachi Data Systems Cinder iSCSI driver.
This driver works with HUS (df850) array.
This driver contains all the base-line features specified for Havana release.
Amended into this submission are changes from code-reviews.

Docimpact: Bug #1180648

Change-Id: Ia27d076443b10da2c653456f9292dd192362b853
2013-05-31 20:16:45 -07:00
OpenStack Jenkins
c62d980caa Merge "Imported Translations from Transifex" 2013-05-31 22:16:42 +00:00
Jenkins
310597cf66 Merge "Migrate to Oslo DB code" 2013-05-31 21:22:29 +00:00
OpenStack Jenkins
4d0befc8d0 Imported Translations from Transifex
Change-Id: Ic2b5d57ad16da67e9b152853338a2ef488eaeee7
2013-05-31 18:11:06 +00:00
Jenkins
17ed858455 Merge "Prevent force delete if the volume is attached" 2013-05-31 13:48:27 +00:00
Jenkins
8ed6644cb8 Merge "Updating HACKING to disallow the use of locals()" 2013-05-31 13:45:54 +00:00
Jenkins
984e889431 Merge "Rename requires files to standard names." 2013-05-31 11:01:45 +00:00
Anton Frolov
27e261376f Add error reporting to generate_sample.sh on import failure.
Currently tools/conf/generate_sample.sh collect options from project
modules by importing each module and looking at it's global variables
for instances of classes derived from cfg.Opt (or lists of such
instances). Sometimes this import can fail due to various reasons,
cfg.DuplicateOptError for example. And if import fails no options
from imported module written to sample config file. Error reporting
will at least warn user that some of options didn't get into sample
config.

Change-Id: I831464b1c1575c106d866a936a50ec74b2d44e59
Fixes: bug #1182611
2013-05-31 14:54:11 +04:00
Jenkins
04e175aba4 Merge "Fix incorrect authorization rule in quota contrib api." 2013-05-31 08:32:42 +00:00
Mike Perez
6251df9068 Updating HACKING to disallow the use of locals()
Change-Id: I7bf2720bdb0456274dc81a73d91296dff0e3fced
2013-05-31 01:14:37 -07:00
Mike Perez
8829bb1a1d Prevent force delete if the volume is attached
Force deletes were eventually failing on the volume manager layer due to
being in an attached state. This will check that up front to inform the
user that they need to detach first.

Fixes: bug #1164929
Change-Id: I24ade24fd750dc647331ef25b835f45f29c10fd7
2013-05-31 00:33:24 -07:00
Jenkins
f30afa6e70 Merge "InvalidUUID can not be raised" 2013-05-31 05:02:02 +00:00
Sergey Skripnick
d0c089fbe8 InvalidUUID can not be raised
There is incomplete format in exception's message.

Fixes: bug 1184870

Change-Id: I27f231c8c5cb41b1ec5fdb34fba6a8dff6a556c7
2013-05-31 17:04:32 +08:00
Jenkins
b113336e12 Merge "Add testrepository to git ignores." 2013-05-31 04:59:47 +00:00
Nikolay Sobolevskiy
f7d4210d7c Fix incorrect authorization rule in quota contrib api.
Incorrect policy and authorization extension name in
 contrib quota api. Legacy from nova.

Change-Id: Ida8262ad63d6f881dd50a385a9913c2ed57b441c
Fixes: bug #1185974
2013-05-31 04:48:24 +00:00
Zhenguo Niu
15a907c491 Rename requires files to standard names.
Rename tools/pip-requires to requirements.txt and tools/test-requires
to test-requirements.txt. These are standard files, and tools in the
general world are growing intelligence about them.

Change-Id: Ifb2735151dc8c7f89ec01d237226669f7051ffa8
Fixes: bug #1179008
2013-05-31 06:09:05 +08:00
Jenkins
d1e7c9a220 Merge "Fix typos" 2013-05-30 20:26:58 +00:00
Walter A. Boring IV
7ad2f80d05 Update 3PAR driver session management
The 3PAR REST API server has a limited number of active sessions.
This change to the driver removes the ability of keeping the
REST API session alive for the life of the driver.  Now on every
driver entry point, which is synchronized, we login to the 3PAR
and when the work is complete, we log out of the 3PAR.  This has
the side affect of more overhead to the 3PAR but helps eliminate
failed commands due to maximum sessions reached to the 3PAR.

This patch also changes the locking to be the same lock for all
driver entry points and adds locking around the volume stats.
This patch also migrates all of the client code access to the common
class, which simplifies the drivers.

Change-Id: Ibcec4cf1781262521ccbdf99c4ba4167634a09c4
2013-05-30 12:05:26 -07:00
Eric Harney
20a728a99a Fix typos
Fix servcie -> service typo in test code

Fix typo: "metedata" -> "metadata" typo in
Glance metadata database migration code error message.

Change-Id: I21f4539b7dedeaed6b2e6c3b34803d1b996ce79c
2013-05-30 14:39:43 -04:00
John Griffith
e6f7751afa Add testrepository to git ignores.
Now that we're updating our test runner we need
to add the git ignore rule for testrepository.

Change-Id: I0cf33598ef947750953624eccb5d34d78df462ed
2013-05-30 11:14:36 -06:00
John Griffith
b61669475e Fix incorrect copyright
Copy paste when drivers were moved out of san.py carried
over the copyright that was included in that file.

Remove that and update.

Change-Id: I38315009a41b9a868c674f63ebf449a78642b2a4
2013-05-30 11:11:13 -06:00
Jenkins
c45d5ddaae Merge "Add missing tests for cinder.db.api.quota_" 2013-05-30 16:59:35 +00:00
OpenStack Jenkins
34d12e65e6 Merge "Imported Translations from Transifex" 2013-05-30 16:59:31 +00:00
Jenkins
76062be494 Merge "Fix incorrect status for volume clone from image" 2013-05-30 16:53:21 +00:00
Jenkins
fe440369d8 Merge "Fix 'Inheritance-based rule deprecated' log warning" 2013-05-30 16:52:38 +00:00
Jenkins
3ecb24aa9c Merge "Return 404 from delete of extra spec if not found" 2013-05-30 15:57:21 +00:00
Sergey Skripnick
b7044c891b Add missing tests for cinder.db.api.quota_
blueprint db-api-tests

Change-Id: Ib47c1620ce124919d9f3365a0abaa53b4b3e5cd4
2013-05-30 15:23:43 +00:00
Jenkins
9a87bb6381 Merge "Add missing tests for cinder.db.api" 2013-05-30 15:11:42 +00:00
Cian O'Driscoll
de84fd7b15 Return 404 from delete of extra spec if not found
API service now returns a 404 if the volume type that the
user is trying to delete doesn't exist.

Fixed Bug 1090306.

Change-Id: I3c1d4fdac1c2492f532ab4dbfd5c7aaf0ffa6fa4
2013-05-30 13:14:59 +00:00
Edward Hope-Morley
4271464375 Fix incorrect status for volume clone from image
If, when using a Ceph as a backend for Cinder and Glance, you
create a volume from a Glance image, the request is actioned
in one of two ways; (1) if Glance does not supply sufficient
information to Cinder, a full copy is made of the Glance
image or (2) Ceph uses it's copy-on-write capability to do a
clone. If a full copy is done, the volume is marked as
Bootable=True but if a clone is done it is not.

This does not appear to break nova boot from volume at present
but the status displayed by the cinder client is incorrect.

Change-Id: I2959fc3dfaf0e073781583019797bcd44f7789d6
Fixes: bug 1185533
2013-05-30 13:47:05 +01:00
Jenkins
301d72cfae Merge "Support for NFS shares with spaces in path." 2013-05-30 11:58:53 +00:00
Jenkins
12c8dbef9c Merge "Fixes 3PAR Host already exists error." 2013-05-30 11:52:30 +00:00
Jenkins
544f1b7ca9 Merge "rbd: send ceph monitor addresses with connection info" 2013-05-29 18:17:08 +00:00
Jenkins
35fc0f2404 Merge "Ensure that pbr>=0.5.10 is installed" 2013-05-29 18:16:03 +00:00
OpenStack Jenkins
1338f52435 Imported Translations from Transifex
Change-Id: Ia0a6aa414a71f25baa1555dc004340e7b29c0f20
2013-05-29 18:11:27 +00:00
Jenkins
0d74075898 Merge "test_glance.py: Stub out _get_member_model as well." 2013-05-29 17:53:45 +00:00
Anton Frolov
924640705c Support for NFS shares with spaces in path.
Unescape share address string listed in nfs_shares_config so that
'x.y.z.w:/foo\040bar' would become 'x.y.z.w:/foo bar' to support
mounting shares with spaces in name.
Use stat command to get available and total space on device instead
of df since it's output is easier to parse.

Fixes: bug #1180984
Change-Id: I552aba91ca8db14130f854c739594a818186bbe2
2013-05-29 20:42:12 +04:00
Jim Branen
9b6dc3dc58 Fixes 3PAR Host already exists error.
Fixed “Host already exists” error by,
* Screen scraping error message and extracting the host name
used by the 3PAR backend.
* Cache the host name used by the 3PAR backend for subsequent
methods using host name.
* After a restart, and 3PAR host name it no longer available,
retrieve 3PAR host name using wwn/iqn, if a terminate_connection
fails.

Fixes bug 1182134

Change-Id: Ia08a311af168ae19dbe7c1405db9c606fd878e53
2013-05-29 09:23:11 -07:00
Jay S. Bryant
94220b145c Ensure that pbr>=0.5.10 is installed
A bug in PBR levels less than 0.5.10 causes the
/usr/lib/python2.X/site-packages/cinder/db/sqlalchemy/migrate_repo/migrate.cfg
file to not be properly installed.  This causes 'cinder-manage db sync'
to fail due to the missing file.

This change sets the required PBR level to >=0.5.10,<0.6 to avoid the problem.
In the case that user already has a 0.5.X level of PBR installed that is less
than 0.5.10 it should be updated rather than encountering issues as our
test system did.

Change-Id: I532158277a4acbb2f8cb75d03c91097e6d7e0312
fixes: bug 1182203
2013-05-29 10:12:23 -05:00
Jenkins
33657e576b Merge "Remove execute permissions from test files." 2013-05-29 14:44:36 +00:00
Sergey Skripnick
36b7039843 Add missing tests for cinder.db.api
Methods covered:

 reservation_*
 service_*
 volume_*

blueprint db-api-tests

Change-Id: Ieb396b3de40eaa2fca9d285f75c0a3d631745e74
2013-05-29 17:38:49 +03:00
Avishay Traeger
fa6404eb02 Remove execute permissions from test files.
Remove execute permissions from test_volumes.py and
test_storwize_svc.py, which caused the tests defined in them
to be skipped.

Fixing the permissions caused test_create_volume_with_volume_type to
execute, which fails. The cause of the failure is that
(1) LoggingVolumeDriver doesn't implement get_volume_stats(), and
(2) The fake RPC doesn't pass the statistics to the scheduler for
some reason.

Because this test doesn't test the scheduler, just change to use the
simple scheduler and work around the issue.

Change-Id: Ie01331d484c5c48c78b626eff1ad793dbbe7182c
Fixes: bug 1183416
2013-05-29 15:58:04 +03:00
Roman Podolyaka
7055b78265 Migrate to Oslo DB code
- engine and session handling code is reused
 - integrity errors caused by duplicating entries
   are handled properly
 - the base class for models is mostly reused
   (the only exception is SoftDeleteMixin, which
    requires changes to existing DB schema; this
    will be implemented in BP db-unique-keys)
 - use common exceptions handling code

Blueprint: db-session-cleanup

Change-Id: I7f5eb53daddc57066af9c73caa633b37a2afd9c2
2013-05-29 10:59:42 +03:00
John Griffith
b2371aeff9 Catch and report errors from copy image to volume.
The copy image-to-volume errors weren't being handled
properly and the result was the lvcreate being retried
even though the lvcreate itself succeeded.

The result of this was misleading errors stating that
the volume couldn't be created because it already existed
(which it did, becuase the create itself was succesful).

Fixes bug: 1183283

Change-Id: I23f05fe64475c3efe285e05a258c4625b801375c
2013-05-28 22:57:48 -07:00
Jenkins
77a77456af Merge "Update cloned volumes QoS settings." 2013-05-29 02:46:47 +00:00
Adam Gandelman
1365ada3bc test_glance.py: Stub out _get_member_model as well.
glanceclient's v2 client needs to also have its _get_member_model()
stubbed out as well, to avoid attempts to reach 'fake_host' during
client instantiation.

Update: Only stub when _get_member_model() when required.

Change-Id: I943cbb017ccdbe4f6c994bc83cbefe8693f4672b
Fixes: bug #1185178.
2013-05-28 16:11:04 -07:00