124 Commits

Author SHA1 Message Date
Sean Dague
d5b52ca755 fix tgt to use 'service' instead of upstart calls
the comments in here were largely about oneric, which we don't
support any more. service is installed in a precise environment,
and will support debian and the upcoming transition to systemd
better, so use that instead.

Change-Id: If15493549a8c93a7387df9b3bba31443aed46995
2014-03-04 09:23:07 -05:00
Ian Wienand
aee18c749b Enforce function declaration format in bash8
Check that function calls look like ^function foo {$ in bash8, and fix
all existing failures of that check.  Add a note to HACKING.rst

Change-Id: Ic19eecb39e0b20273d1bcd551a42fe400d54e938
2014-02-28 07:59:03 +11:00
Ian Wienand
3b57829ece Don't use --skip-redirect for cinder restart on rhel6
RHEL6 doesn't support this flag so the restart fails.  Not exactly
sure why it is required, seems unchagned from the initial
commit 67787e6b4c6f31388cbee6d83b67371b31c443d4

(found running stack.sh with -e per [1])

[1] https://review.openstack.org/#/c/71996/

Change-Id: Ib34c3663409d7b96b932286cb5a6974e940075d3
2014-02-26 14:55:20 +11:00
Steve Martinelli
19685428e3 Change most keystoneclient commands to openstacklient in libs
migrated most keystoneclient commands from the following libs:
 ceilometer
 cinder
 ironic
 keystone
 marconi
 neutron
 nova
 savanna
 swift
 trove

Also need to set and unset openstackclient specific environment
variables from stack.sh

Change-Id: I725f30bc08e1df5a4c5770576c19ad1ddaeb843a
2014-02-14 21:30:57 -06:00
Jenkins
1d081ebc2b Merge "Cleanup cinder-rootwrap support" 2014-02-15 02:21:13 +00:00
Jenkins
f0029532f8 Merge "Replace custom cinder driver configs" 2014-02-12 22:38:53 +00:00
john-griffith
d0860cc26d Replace custom cinder driver configs
The devstack/lib/cinder file has a number of third party
driver config options hard-coded in it.  Rather than add
yet another if driver== statement here let's use plugin
files and do something similar to what's already
in place for nova_hypervisors and neutron plugins.

This works the same way folks were implementing their
drivers already, the key is to use a CINDER_DRIVER
variable in your localrc file that matches the name
of the lib/cinder_plugin file to use.

The existing third party driver entries that were
in lib/cinder have been migrated to cooresponding
plugin files.

Change-Id: I4ee51ea542d5aa63879afd5297311a9df727c57f
2014-02-11 11:30:04 -08:00
Dean Troyer
e4fa721322 Begin is_service_enabled() cleanup
This converts the special cases in the is_service_enabled() function to call
individual functions declared by the projects.  This allows projects that
are not in the DevStack repo and called via the extras.d plugin to handle
an equivalent service alias.

* Ceilometer
* Cinder
* Glance
* Neutron
* Nova
* Swift

TODO: remove the tests from is_service_enabled() after a transition period

Patch Set 2: Rebased

Change-Id: Ic78be433f93a9dd5f46be548bdbd4c984e0da6e7
2014-02-07 10:06:21 -06:00
Thierry Carrez
d5d4974cb7 Cleanup cinder-rootwrap support
cinder_rootwrap support in devstack handled a number of now-abandoned
use cases:

- no $CINDER_DIR/etc/cinder/rootwrap.d (old-style rootwrap)
- using oslo-rootwrap instead of cinder-rootwrap (abandoned experiment)

This change removes unused code paths and aligns
configure_cinder_rootwrap() with configure_nova_rootwrap().

Change-Id: I387808dae0e064cc9c894c74ab78e86124f08dd2
2014-02-06 16:00:08 +01:00
Dean Troyer
4237f590b7 Generate Tempest service list rather than hard-code it
The list of services that Tempest used to set its 'service_available'
config values was hard-coded. To be plugin-friendly have each
service (project) add its name to the TEMPEST_SERVICES variable
and use that for setting the 'service_avilable' values.

Change-Id: I208efd7fd0798b18ac2e6353ee70b773e84a2683
2014-01-29 17:25:45 -06:00
Jenkins
adfaf0cef0 Merge "clean.sh updates" 2014-01-26 19:21:43 +00:00
Dean Troyer
53ffc713b1 clean.sh updates
* Clean out data, log and state dirs
* Include lib/apache to clear is_apache_enabled_service not found error
* Clean errors removing tgt config files

* Clean errors removing VG backing file in lib/cinder

Change-Id: I33dfde17eb8daaaed7f7e76337fe6a8085a266bf
2014-01-25 13:26:58 -06:00
Jenkins
e5a5fe33e0 Merge "Remove duplicated name services" 2014-01-13 18:50:59 +00:00
Jenkins
3e98388d07 Merge "Robustify service shutdown" 2014-01-13 14:09:44 +00:00
Jenkins
1d8abb95b9 Merge "Put cinder rootwrap config in separate function" 2014-01-12 13:51:27 +00:00
Jenkins
ec528ecae2 Merge "Add a flexible API version choice for Cinder, Glance and Heat" 2014-01-12 00:16:11 +00:00
Dean Troyer
9fc8792b0a Robustify service shutdown
* Save PID when using screen in screen_it()
* Add screen_stop()
* Call out service stop_*() in unstack.sh functions so screen_stop()
  can do its thing

Closes-bug: 1183449
Change-Id: Iac84231cfda960c4197de5b6e8ba6eb19225169a
2014-01-11 11:46:19 -06:00
Thierry Carrez
63e1784354 Put cinder rootwrap config in separate function
Separate out Cinder's rootwrap configuration so that it can be called
from Grenade's upgrade scripts. This follows the same model as Nova uses
with configure_nova_rootwrap() which can be called from Grenade to
refresh rootwrap config.

Change-Id: Id808abc2b5754443362b3de4b3453e305d3720f3
2014-01-10 14:23:03 +01:00
Dirk Mueller
25049cd23d Use --tenant-id, not --tenant_id
Change-Id: I0e3d65d5b69ac82cbf7ee6ffc41ead369af8c126
2014-01-09 13:55:28 +01:00
Juan Manuel Olle
4ad37630a2 Remove duplicated name services
Due to the fact that keystone will not allow services with
duplicated names, cinder and nova services names were
changed

Closes-Bug: #1259425
Change-Id: I988aef477b418a289426e02e5e108aa57dd1076b
2014-01-06 15:25:31 -03:00
Vincent Hou
21fe4e76d5 Add a flexible API version choice for Cinder, Glance and Heat
The version of the authentication url is set to v1.0 for some
projects by default. We can make it configurable via the parameter
"$IDENTITY_API_VERSION".

Closes-Bug: #1253539
Change-Id: I6640e345d1317b1308403c95b13f8a998320241b
2014-01-06 01:22:57 -05:00
Thierry Carrez
0718568b12 Support oslo-rootwrap in lib/cinder
Make lib/cinder support both cinder-rootwrap (current case) and
oslo-rootwrap (future case) to handle the Cinder transition towards
oslo-rootwrap usage peacefully.

Related blueprint:
https://blueprints.launchpad.net/cinder/+spec/cinder-oslo-rootwrap

Change-Id: I663986304bd74cb6d72d51c553540fb5f9db1d1d
2013-12-13 15:20:26 +01:00
Matt Riedemann
7103a84e39 Workaround Cinder "wishlist" bug 1255593 for multi-backend volume_clear
Cinder currently only applies the volume_clear setting from the DEFAULT
section of cinder.conf if you're using a single backend.

The Cinder team has determined this is a 'wishlist' item to propagate
volume_clear to each backend, but it does impact usability and
performance.

To improve the performance of running Tempest with multi-backends in the
gate, workaround the bug in devstack.

Related-Bug: #1255593

Change-Id: Ia0ff5422f53eeda9a3ac4336eefec3b9bdea6da2
2013-12-07 13:53:33 -08:00
Jamie Lennox
bd24a8d0f8 Allow deploying keystone with SSL certificates
Allow providing certificates through environment variables to be used
for keystone, and provide the basis for doing this for other services.
It cannot be used in conjunction with tls-proxy as the service provides
it's own encrypted endpoint.

Impletmenting: blueprint devstack-https
Change-Id: I8cf4c9c8c8a6911ae56ebcd14600a9d24cca99a0
2013-11-25 22:27:51 +00:00
Stephan Renatus
e578effb33 Make use of STACK_USER instead of relying on USER
Quite easily one ends up calling ./stack.sh in an environment that,
albeit being user "stack" (for example), doesn't quite meet the
expectations of devstack.  The errors that follow can be rather hard to
track down, as the dependency on `USER` is not mentioned.

To remedy this situation, this commit

- uses STACK_USER instead of USER and
- mentions that dependency in the script headers of lib/*

Change-Id: If4cdc39b922ea64b4c0893a0e695ec06349fccc5
2013-11-19 13:39:58 +01:00
Jenkins
b7c11a0f98 Merge "Fix some Markdown formatting issues" 2013-10-31 14:19:15 +00:00
Adam Spiers
6a5aa7c6a2 Fix some Markdown formatting issues
Address miscellaneous issues with Markdown formatting in comments which
are consumed by shocco when generating the online documentation.

Change-Id: I953075cdbddbf1f119c6c7e35f039e2e54b79078
2013-10-24 17:38:19 +01:00
Mehdi Abaakouk
ae9c41727a Allow configure cinder with the nfs driver
This patch allow to use nfs as cinder driver.

To use it, in the localrc we can now set

CINDER_DRIVER=nfs
CINDER_NFS_SERVERPATH=172.16.0.50:/export_cinder

The nfs-server is not setup by devstack.

Change-Id: I8e240d00b58f272d04ab2c0922c551b1f7266260
2013-10-23 06:56:08 +00:00
Dan Prince
82dea7c64a Use cinder.conf for auth_token configs.
Updates lib/cinder so that we use the application config file (cinder.conf)
instead of the Cinder api-paste.ini config file.

Related-Bug #1240753

Change-Id: I6636d33ee522757145ac97fc354324a8b9379700
2013-10-17 12:33:50 -04:00
Sergey Kraynev
84783c72fe Adding value for lock_path in configuration file
Now oslo code include new lockutils. According this code
if lock_path is not set in configuration file, will be
raised Error message.

So for updating lockutils in cinder project is needed
lock_path definition in configuration file.

Change-Id: I413f0a2ccec0f9d9e06acaa8cc06c41206d9dcc2
2013-10-10 09:08:48 -04:00
Giulio Fidente
384454de57 ensure tgtd is running in debug mode
this change enables on-the-fly tgtd debug before starting cinder

Change-Id: I193bfd77c5a82e8347d75e2a7fe670a6e25f5558
2013-09-27 13:17:34 +02:00
Salvatore Orlando
05ae833be2 Colorize Neutron log output and refactor log setup code
Bug 1214616

This patch adds colors to on-screen Neutron log output in the same way nova,
cinder, and heat engine do.
To this aim, colorized logging configuration has been moved to ./functions.
The reason for this refactoring is that these instruction are the same or
very similar for each project, with the only exception of the target
configuration file.

Change-Id: Idf0d1b842bb9ab046c9ef826de1dfc55b3f1df9d
2013-09-13 10:34:40 -07:00
Gary Kotton
5c3b861c19 VMware: remove invalid configuration variable
The configuration variable vmware_cluster_name is not used by the cinder plugin.

Change-Id: I8c0ed58d1dcd66b6f8ea3325007bf5135216933c
2013-09-10 05:21:24 -07:00
Jenkins
d748b16ef2 Merge "Add my_ip option in cinder.conf" 2013-09-04 18:10:50 +00:00
Nikolay Sobolevskiy
08df29bff4 Add my_ip option in cinder.conf
Then cinder installed in multinode environment with more than one interface,
it's better to use CINDER_SERVICE_HOST option for cinder ip address.

Change-Id: I775b70407379d9c97696f536f5f89cecb33657be
2013-09-04 02:48:13 +04:00
Jorge Valderrama Romero
f39ee96f1d Fix deprecated params user_id, role_id, tenant_id
Update keystone cli by changing parameter options (user_id, role_id and tenant_id)
to use '-' rather than '_' in the method user_role_add and consequently the method
user_role_remove without maintain backward compatibility because these are deprecated.

python-keystoneclient - Bug #1150434
Change-Id: Ia5113718eb050cf7dba443b8d0caf3bdaa1730f0
2013-09-02 17:54:27 +02:00
Jenkins
0852604651 Merge "VMware: update cinder support for VMware configuration settings" 2013-08-27 16:07:45 +00:00
Gary Kotton
dc4f2342db VMware: update cinder support for VMware configuration settings
The review https://review.openstack.org/#/c/41600 was update to
have a 'vmware' prefix for all of the VMware cinder settings. These
were previously in a 'vmware' section and now they are in the
'DEFAULT' section.

Change-Id: I8eadfb0f64914d3b0667760aff651415df48f627
2013-08-26 06:39:38 -07:00
Attila Fazekas
fac533e38d Configure bash completion
Add bash completion rules to the /etc/bash_completion.d from the
 cinder, neutron, keystone, nova and nova-manage.

This is very fast operation and makes the cli usage easier.

Change-Id: Icdcdaf55d58efaaa1afe25fd55f088bf7dc8b3f1
2013-08-22 05:54:29 +02:00
Jenkins
70931c7e80 Merge "VMware: Add cinder support to devstack" 2013-08-15 14:53:03 +00:00
Jenkins
98d434af99 Merge "Add Cinder V2 API to keystone catalog" 2013-08-13 20:48:42 +00:00
Gary Kotton
c325227465 VMware: Add cinder support to devstack
The patch set adds cinder support to devstack. VMware cinder support
can be found at:
 - Nova - https://review.openstack.org/#/c/40245/
 - Cinder - https://review.openstack.org/#/c/41600/

Change-Id: I0a05643010ea6cfb6635505accc3dcf411fdd419
2013-08-13 00:32:20 -07:00
Ben Nemec
039979424b Allow disabling of debug logging
I find that enabling the debug log level often causes me to miss
important error messages due to the sheer volume of information
logged.  This change allows configuration of the debug option
in a number of the projects so it can be disabled globally
without having to make one-off changes after each re-stack.

Note that this does not apply to Keystone or Swift right now.
They use a different method to configure their logging level and
I'm not as familiar with them so I didn't want to mess with their
settings.

Change-Id: I185d496543d245a644854c8a37f3359377cb978c
2013-08-12 15:01:39 -05:00
Gordon Chung
2bfbc779c7 Cinder configuration is not set up for Ceilometer
enable cinder notifications when ceilometer is enabled

Change-Id: I55809f1cef35aca90f8513a73df1417dcf08098d
Fixes:Bug1210269
2013-08-09 10:55:12 -04:00
Mike Perez
b3862f9871 Add Cinder V2 API to keystone catalog
Support both SQL and templated keystone backend. This will add an
additional endpoint to go with v1.

Change-Id: I19168d55c2ffad2b1cd668b6c1341dc8e49e9c1f
2013-08-08 19:26:50 -07:00
stack
cf2d0d3db9 Add keystoneclient support for cinder.
Add an ability to ask keystone about users and groups
through keystoneclient in cinder.

blueprint volume-acl

Change-Id: Ice261e9709833d057722b4f13c404df54e10b204
2013-08-05 17:50:45 -04:00
John Griffith
e6d4fe5f0f Modify startup order of Cinder services.
There are cases where the timing between the start up
for cinder-volume and cinder-scheduler service can result
in a race where the scheduler doesn't know about the volume-service
until the next periodic update.

This change does attempts to do an easy fix by swapping the start
order of the cinder services to ensure that the scheduler will be
able to receive the volume service capabilities update.

Fixes bug: 1189595

Change-Id: I8f477ddc04c15c04493f7ce6863e08e1de8f0128
2013-07-15 18:08:43 -06:00
JordanP
022e991a54 clean logical volumes in init_cinder
This part of code was forgotten when _clean_volume_group()
has been renamed.

Change-Id: I96fe749013b5cf1fd3411b2ba661302873e92994
2013-07-10 17:02:59 +02:00
Attila Fazekas
a6ed3dcf3c lib/cinder comment
The default volume backing size is ~10G now.
Changing the comment to be more future-proof by including
the shell variable name.

Change-Id: I27879b240f3f8348fa069853de18e47871ce00d2
2013-07-03 09:15:09 +02:00
Ian Wienand
0db171378d Update LVM autoextend config for RHEL6
Cinder clones are slightly larger due to some extra metadata.  RHEL6
will not allow auto-extending of LV's without this, leading to clones
giving hard-to-track disk I/O errors.

See https://bugzilla.redhat.com/show_bug.cgi?id=975052

Change-Id: I09a5e061a9665c5310383f9f9eb281bfdc8e416d
2013-06-27 09:13:48 +10:00