Merge tag '2012.2.1' into debian/experimental

2012.2.1

Final versioning for 2012.2.1

Change-Id: I74e485f7fde8d34fa5e6e8852e06d522b2ccc1d1

Remove unused python-daemon dependency

python-daemon is listed as a dependency, but it is not needed.

Change-Id: I4d6eadfef5d60cd277ff7395a12f5ad255db7155
(cherry picked from commit 6217a3df0a34b1d0ac6fad8e9ea5044d2056c45d)

Improve provider_location cleanup code for RBD.

The RBD driver does not make use of the 'provider_location' field
but the current cleanup code assumes it does.  Ensure the field
is in use before testing whether or not it needs fixing.

Fixes bug 1083818.

Change-Id: Id6ff85101f85e70575ba244c2df7aca0196cf224
(cherry picked from commit d030c5b10e9e8d73d967562259e7db6146347108)

Overrides lintian false positive warning about unused debconf templates.

cinder-api depends on adduser

Ran debconf-updatepo

Remove now useless debian/rules targets (moved to pkgos.make)

Fixes cinder-api keystone registration.

Sets keystone auth token debconf variable as a password and not just a string.

Registers the correct service with keystone

Added missing (conditional) lsb-base depends.

Added service and endpoint creation in cinder-api.

Added upstream changelog

Removed duplicate get-orig-source: (it's in pkgos.make)

Remove duplicate Build-Depends: openstack-pkg-tools

Move some stuff to pkgos.make

Merge branch 'debian/experimental' of ssh://git.debian.org/git/openstack/cinder into debian/experimental

Now auto-detects and prompts for a volume group using debconf.

Detect and fix issues caused by vol ID migration

The migration from volume ID to UUID neglected to update the provider_location
field on the volume.  As a result the iqn and volume name no long match and
existing volumes are no longer able to be attached after an upgrade
(essex -> folsom and then nova-vol->cinder).

This patch adds a method to the volume driver that will check for the
mismatch of volume name in the iqn during service start up.  If
detected it will update the provider_location field in the database
to include the new ID.  Also it will create a symlink to the device backing
file that also has the correct naming convention.

Note: We don't disturb an connections that are currently attached.
For this case we add a check in manager.detach and do any provider_location
cleanup that's needed at that time.  This ensures that connections
persist on restarts of tgtd and reboot.

Change-Id: I4683df4ef489972752dc58cb4e91d458a79a8ef2
Fixes: bug 1065702enter the commit message for your changes. Lines starting

pin sqlalchemy to the 0.7.x series

sqlalchemy 0.8beta is now out and has internal changes
which means it's not an in place seemless upgrade. This pins
sqlalchemy to the 0.7.x series to avoid those breaks.

Fixes bug #1073569

Change-Id: If204690569eb92b9077833a5da121fc072c6dcfa

Removed set -x

Removed set-x in debian/cinder-common.postinst.in

Start using factored pkgos_* stuff from openstack-pkg-tools

* Now asking for Keystone credentials using the new pkg_os function.
* Patches api-paste.ini for syntax.

Copy of pkgos_func from glance.

  * Now uses pkgos_func functions for debconf and dbconfig-common handling.
  * Renames /etc/sudoers.d/cinder_sudoers as /etc/sudoers.d/cinder-common.
  * Debconf now users cinder/<debconf-screen-name> and not cinder-common/.
  * Reviewed long and short descriptions in debian/control.
  * Bumped Build-Depends-Indep: python-all to 2.6.6-3~ since we are using dh_python2.
  * Added everyone in the team as Uploaders:.
  * Changed Homepage field to http://cinder.openstack.org/.
  * Dependencies are now by alpha order.
  * Removed useless empty cinder-volume.postrm, cinder-api.postrm,
  cinder-scheduler.postinst, cinder-scheduler.postrm.
  * Pre-Depends: dpkg (>= 1.15.6~) because we use xz compression.

Merge "Restore SIGPIPE default action for subprocesses" into stable/folsom

Set defaultbranch in .gitreview to stable/folsom

This allows people run git-review without any arguments.

Change-Id: I2b2488ffffd783b64e5c760324e7d4a00d5a39db

Fixes wrong handling (eg: policy violation) of /etc/cinder/cinder.conf

Added setuptools_git-*-py2.*.egg in .gitignore

Added a gbp.conf file.

Re-added ubuntu upstart files using the .in extension.

Added a get-vcs-source target and sets compression to use XZ level 9.

Restore SIGPIPE default action for subprocesses

Python ignores SIGPIPE on startup, because it prefers to check every
write and raise an IOError exception rather than taking the signal. Most
Unix subprocesses don't expect to work this way. This patch (adapted
from Colin Watson's post at http://tinyurl.com/2a7mzh5) sets SIGPIPE
back to the default action for cinder.utils.execute and cinder-rootwrap
created subprocesses.

Fixes bug 1053364

Change-Id: I4b3307bd2f0f5d0da529d8b7d80fabae28c57732

New upstream release 2012.2

Merge tag '2012.2' into debian/experimental

2012.2

Bump next version to 2012.2.1

Open stable/folsom by bumping next version to 2012.2.1.

Change-Id: If60a8e714ba78c8d9bfcc5f07d9abaea5c4cf785

Merge "Fix NetAppCmodeISCSIDriver._get_lun_handle() method" into milestone-proposed

Merge "Revert explicit usage of tgt-adm --conf option." into milestone-proposed

Fixes remove_export for IetAdm

* Corrects indentation.
* Uses specific iqn as keyword argument.

Fixes bug 1055401

Change-Id: Ie99dd8a34924f0d7f447cf41a305353659a51d11
(cherry picked from commit 5b61536fdf17332f48af4b422fe23c2e153934a1)

Fix NetAppCmodeISCSIDriver._get_lun_handle() method

bug 1055578

Change-Id: I06da6a9a59fa2320f1cae9698e5e63562e534916
(cherry picked from commit b2627b27dae521d719b90831dcc41695b389712f)

Revert explicit usage of tgt-adm --conf option.

Fixes bug 1056246.

This reverts commit 0903603971033536751ba40df98a3c39175d3189.

This ensures that we fail-fast if the tgtd configuration does
not include the directory for volume iscsi target configs,
instead of only blowing up when tgtd is restarted.

Change-Id: I30aa3f1d8944eb33cd4a42d65e227031023406b4

Add missing entries in setup, fix up pip-requires

setup.py still has references to nova/compute and was missing some
requires statements, and more importantly didn't hvae the install_requires.

Also, tools/pip-requires needed lxml versions set, as well as
added the missing setuptools_git entry.

This was exposed when trying to run Grenade as the requires.txt
file was never generated.

Fixes bug #1055693

Change-Id: Iaf9669d66e89507989ef3fde630dc2b488a97cfc
(cherry picked from commit 7f00d13640edac8136241e546076f2106f7bfa61)

Fix cinder-volume-usage-audit

It seems many point in the internal API and openstack.common changed, but
this binary has been forgotten.

Also, context ie needed when using the notifier, so import it.

This fixes bug #1053502

Change-Id: I306b8671e7b0ed8c2ce2548a51a4c5e6393e13cb
Signed-off-by: Julien Danjou <julien@danjou.info>
(cherry picked from commit 186143ecc81369c02aca51b81feb976259bcad41)

Return 400 if create volume snapshot force parameter is invalid

Fixes bug #1014689

* Add is_valid_boolstr function in utils.py
* Add force parameter check in SnapshotsController.create()
* Add unittest for invalid force parameter.

Change-Id: I0f64326f33eb4fad1cf384bd825f56f09e935f40
(cherry picked from commit c678b045c9839ed30e13fe5e7e655e3cb2e2c3f9)

Merge "Stop double logging to the console" into milestone-proposed

Stop double logging to the console

The code in nova for logging added an extra root handler to be
able to log messages to the console during log setup. This handler
was removed in the setup method. The common setup method no longer
removes this handler, so don't create it. Note that there may be
a small period before the logging setup is finished where messages
will not appear.

Fixes bug 1053512

Change-Id: I559cca8ffc2a33beff22025e42a9713487245980
(cherry picked from commit a1cbbe2e9e5d7a5ec55ec1197571cd68fcdbad15)

Remove cinder gating hack

The hack was necessary while we switched the name used for az's
in the gating tests. Now it can be removed

Change-Id: I970205910ac0950ab4a961aa0537eb964d7f0d3c
(cherry picked from commit bc0fb113af225878b4305da7d834673a6b7f5e97)

Merge "Set the default availability zone back to nova" into milestone-proposed

Set the default availability zone back to nova

Currently nova and cinder have to have the same set of az's for
the ec2 api to work properly. Therefore, set the default az for
cinder to nova so they will match.

Includes a hack to az to set cinder to nova to work around gate
issues. The hack can be removed once the gate sets the zone to
nova properly.

Fixes bug 1053508

Change-Id: I9c30ccc5c79b9c4e6c1ebeb3e80d0098e1ffbb11
(cherry picked from commit d051ca9cdf828b1e47ef5158c4b83beb9484109e)

Add lun number (0) to model_update in HpSanDriver

The HpSanDriver was not setting the lun number in model_update
and as a result default value of 1 was being used.  Trouble is the
Lun number used by LeftHand is 0, so the connect info would be
wrong and fail when trying to attach the volume.

Fixes bug #1053717

Change-Id: If82a66c6a78ab03da09223a7b79abe559a6f702f
(cherry picked from commit f55198570fd59b7a2a4ddd2f5e77b3d03d0a3d64)

Fixes to the SolarisISCSI Driver

Even when the san_is_local config option was set to false, the
SolarisISCSIDriver's _execute method was accidentally set to
util.execute by the VolumeDriver's __init__ method.

Fix bug 1053041

Change-Id: Ie3299f5065e2b084644ec2956f0c6ebae4264a49
(cherry picked from commit 3b8a6303e6c04f0c7b45043c2fcd53948910d37e)

Update MANIFEST.in patch

Merge snapshot 6b31057

Merge remote-tracking branch 'upstream/milestone-proposed' into debian/experimental

Don't suppress upstream file on dh_clean

dump version to folsom rc1

Merge tag 'folsom-rc1' into debian/experimental

folsom-rc1

Add rules to generate quickly the manifest.in

Set important path

Merge "Correct IetAdm remove_iscsi_target" into milestone-proposed

Add nova migrate_version check to cinder import

Cinder-manage migrate import only works from Nova/Folsom--->Cinder/Folsom
this change adds an explicit check of the nova migrate_version (133)
to make sure we have all of the volume id/uuid changes that are needed
to be compatable, and presents an error message if that's not the case.

Addresses bug #1052598

Change-Id: I7f32cf35e33320f4348b7bc44a8e92b5716b71d8
(cherry picked from commit 321d32379cd6044378f2f9c32c068de18f514df5)

Correct IetAdm remove_iscsi_target

Remove logical units should be done before delete the target while
unexport iSCSI volume. Or ietd will show "No such file or directory"
error.

Fix bug 1043107

Change-Id: Iab5b97044c5a30c788fcabce827923ff82a84b91
(cherry picked from commit f14e955735065a998cd78280661f6d33c850d9e2)

Change two remaining rename mistake

Mark Cinder 2012.2 Final as we near RC1

Mark Final=True as we prepare to publish Folsom RC1.

Change-Id: Ie95dd290aef49633586a4b483c3c80dca3e72aa2

Merge "Clean up db.volume_create()"

Merge "Fix volume deletion when device mapper is used"

Clean up db.volume_create()

 Adding the metadata return to db.volume_create() introduced some
 messy and unnecessary repitition in the code.  This patch
 cleans that up and makes use of existing volume_get functions rather
 than duplicating the code in volume_create.

 This syncs cinder up with the patch as it's been submitted for nova
 in https://review.openstack.org/#/c/10461/

 Addresses bug #1052176

Change-Id: I17aca069fbb0240770613f176430a1b2fcf2b25a

Merge "Update quota when deleting volume that failed to be scheduled"

Fix volume deletion when device mapper is used

Call dmsetup remove if there is a /dev/mapper/nova--volumes-
element present.

Resolves bug 979020

Change-Id: Iddaaed411a77dda4bd32f9a97687ff17744119eb

Merge "Fix FLAGS.volumes_dir help message"

Update quota when deleting volume that failed to be scheduled

If one volume was failed to get scheduled, removing such volume should
also clean up reservation.
Also when create_volume is ready to send to scheduler, reservation
should be committed no matter backend can successfully create that
volume or not since deleting volume call will do a minus reservation
even on volume with 'error' status.

Fix bug 1052052

Change-Id: Iaec9926024ad5adb38b08b07772095f49b8593c2

Add files that need to be installed

Add debconf po directory and dependency

Fix error in configuration file

Clean /var/lib/cinder on purge

Mispelling in init script arguments

Add dbconfig-common support

Fix some lintian warning

Add po rules

Remove update-rc.d, it's already done by DEBHELPER

Sync a change to rpc from openstack-common.

This commit pulls in the following change from openstack-common:

    ee44222 Change the qpid_heartbeat default to 60 seconds.

Fix bug 1050661.

Change-Id: I999e16c18227f9aab4dc8c2ff88cd6907c7c8191

Merge "Sync notifier changes from openstack-common"

Merge "Add a resume delete on volume manager startup"

Merge "Improve entity validation in volumes APIs"

Add a resume delete on volume manager startup

  Currently if for some reason the volume service was stopped
  during the zero out operation of a volume delete there was
  no way to get the volume removed from the system (it would
  be present in deleting status forever).

  This change adds a simple check of volumes in the DB with status
  of deleting, and if any are found it restarts the delete process
  on them.

  addresses bug #1011150

Change-Id: Id4c4a3bc61f95245ebc6658234b4b88029956562

Merge "Add entity body validation helper"

Improve entity validation in volumes APIs

Fixes bug #1048565

Use the new Controller.is_valid_body() helper to validate the entity
body in various volumes related POST/PUT handlers and return 422
as appropriate.

Change-Id: I04127972981522c1ed81903893396c4f9665bcd3

Add entity body validation helper

Add a _valid_body() helper for the to avoid repeating the same tests in
multiple methods. Include a check that the entity itself is actually a
dict.

(Cherry-picks part of commit d1ad73ee from Nova)

Change-Id: If8114cc76d68567005c85c803f29e30e034db89a

Should've added super().tearDown() in test_iscsi

  Made changes to use tempfiles and implemented teardown,
  but should have also called super().tearDown as well.

  Associated with chage: I9b9a2400

Change-Id: I2e9d1cb7bfbe78323630eb2f3270698a7ce7e4c4

Add debian dir

Fixes bug 1050135.

Regenerate cinder.conf.sample and fix a couple related bugs.

Change-Id: I3e472f9c69ad4888dc15f901f4bc7cd825fb5ba1

Fix FLAGS.volumes_dir help message

And other nova/volume/iscsi.py typos

Change-Id: I781a07b4f11b6cf336c278a6994bd5575ae1a70a

Merge "Use tmpdir and avoid leaving test files behind"

Merge "Sync log format changes from openstack-common."

Merge "Update rpc from openstack-common."

Use tmpdir and avoid leaving test files behind

 We were just creating a file for testing iscsi persist files and
 weren't cleaning up after the test.

 Change this to use a tmpdir and make sure we leave no tracks.

 Fixes bug #1050086

Change-Id: I9b9a2400a172d52987d76f3132793cb042fd201c

Merge "Add volume quota in volume/api.py and olume/manager.py"

Merge "Fixes bug 1049446."