76 Commits

Author SHA1 Message Date
OpenDev Sysadmins
2c8ce5ec98 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:51:54 +00:00
Raissa Sarmento
fec9fd78a0 Change manila post_test_hook to use plugin repo
Fix in order for the tempest jobs run manila's
tempest plugin from the new plugin repository.

Change-Id: I4ba8711a155dc90790657245d34e58382b99737e
2017-10-23 17:24:00 +01:00
Raissa Sarmento
658c7a6847 Use generic user for zuul v3
The manila post_test_hook.sh script still
uses jenkins as user, but with zuulv3 that
should be zuul. $USER solves that issue.

Change-Id: I23299a15249be292e3a60450db2adea0abab64bb
2017-10-17 10:18:02 +01:00
Tom Barron
c0ecd9f069 remove tempest pin
Change [1] removed the tempest pin from the
pre test hook in manila proper.

Remove it here as well.

Closes-Bug: #1711812
[1] Ia402fb79e1de5c9925f575d635b9928475095f86

Change-Id: I7f06610820b81788b3d536a8114c69623ccf0896
2017-08-19 09:24:37 +00:00
Tom Barron
99a3d82609 manila: use local.conf in pre_test_hook script
As per, http://lists.openstack.org/pipermail/openstack-dev/2017-February/112872.html
a devstack plugin's pre_test_hook script can set options
in local.conf and not in localrc. Do so to unblock gate,
gate-manila-tempest-dsvm-glusterfs-*

Change-Id: Ia7c5ffeeff7e8cb86153f565b7b74f7e67b11a03
2017-02-27 12:57:23 -05:00
Tom Barron
a48b35f0d0 Fix lvcreate failure in glusterfs with manila
When running with the manila plugin on ubuntu Xenial,
devstack fails with a message when attempting to create
a logical volume within the glusterfs VG with a message
about insufficient allocatable extents [1].

Change the hard-coded number of extents supplied to
the lvcreate command to a percentage of the VG.

Closes-bug: #165238

Change-Id: Ia53f0e1b9b80ae0012d720a3f7b3696cadbd843c
2016-12-23 22:08:55 +00:00
Csaba Henk
1d46d65cf3 clean up dispatches over GLUSTERFS_MANILA_DRIVER_TYPE and os release
Change-Id: Ib8664c5aa71ff4efb64fd95262295fcf322e5a4e
2016-09-08 15:54:28 +02:00
vponomaryov
2a7be431ba Do not upload manila service image
Because it is not used by GlusterFS driver.

Change-Id: I56e825f0f14c21c52b3089f2529ae012f688032c
2016-07-08 09:58:06 +00:00
vponomaryov
d35a217dfb Fix devstack hooks for manila CI jobs
Replace not available anymore "JOB_NAME" env var with
"GLUSTERFS_MANILA_DRIVER_TYPE" one in devstack hooks to unblock
CI jobs.

Change-Id: Ice8395325d20060df511b9de82c944200ceea2aa
2016-07-05 18:42:55 +03:00
Csaba Henk
5d698eda71 Work around log file upload EPERM issue
The log files created during the Tempest run
get scp-d to their final location via Ansible.
Some root-owned log files are not readable
for the Ansible process and that results in
getting an EPERM and thus the gateway job
is qualified as UNSTABLE.

We don't have a hook in between the gluster
invocations creating those log files and the
call to the Ansible upload job, so we are not
able to chmod the logs as needed.

Instead we apply the following hack: we create
and mount a VFAT file system onto the log directory
where the trouble strikes; and VFAT does not implement
POSIX file permissions therefore all files within it
qualify as world readable.

Change-Id: I9a85888166b5b9c28a049e7c17335e2082a61f92
2016-05-02 11:22:30 +02:00
Ramana Raja
658d3cc5eb manila: disable ro access level tests for the two drivers
The two manila gluster drivers, glusterfs and glusterfs native do
not yet support 'ro' access level. So disable 'ro' access level
tempest tests for the two drivers by making the CI post hook
script modify the tempest config file appropriately.

Change-Id: I32f255eafe4938196c54108a45d7ddb48eceb8bf
2016-03-15 14:48:04 +05:30
Csaba Henk
701ce2bb77 add a glusterfs driver + heketi layout backend variant
Change-Id: I4bd7abc83605687fc5d990b54e728a113b4f37fe
2016-03-03 12:46:26 +01:00
Ramana Raja
2cfbc1db61 Fix Manila CI Tempest authentication issue
With the change [1] to DevStack project, preconfigured creds for Tempest,
required by Manila, cannot be used anymore. And for now, latest Tempest
interfaces do not allow use of preconfigured creds due to bug #1524717.
So, until this bug is fixed, workaround Devstack changes by setting legacy
options explicitly.

This workaround fix was carried over from the change [2] in the Manila
project.

[1] I65b56ff681d6c27094380693c953fbc3664eceb0
[2] I516a6c9ab4396b14e3984d5f810210e4fcf7ec85

Change-Id: I09df56aa611ba755c282c575f712261c48385fe9
Related-Bug: #1531049
2016-01-07 15:22:29 +05:30
Ramana Raja
7a0ff013db Fix for tox 2.3.1 failure
Remove inline comments after line escape that isn't understood by the
tox's parser.

This is due to the backward incompatible fix,
https://bitbucket.org/hpk42/tox/issues/181

Related Bug: #1527562

Change-Id: Ib92b65b510fc3a6261e0e890c91af0718f24fc33
2015-12-23 20:43:49 +05:30
Jenkins
f30a7381dc Merge "_create_thin_lv_gluster_vol: retry mounting the lv" 2015-12-14 05:41:07 +00:00
Csaba Henk
21d2009e02 _create_thin_lv_gluster_vol: retry mounting the lv
Also took the opportunity to move to test_with_retry
instead of using a hand-rolled retry loop.

Closes-Bug: #1524656
Change-Id: I0fff95b434a6572efd2af3ac2ec89b5772459cd5
2015-12-11 10:33:44 +01:00
Ramana Raja
92c2072121 manila: branch aware Tempest checkout
Another take on the problem described
in 5b4dd3c, ie. Manila Tempest update
in

6b8319e32a (diff-e0705abec56bf0f83781d686e109d63eR63)

Instead of checking out $MANILA_TEMPEST_COMMIT
blindly (ie. even if it's unset), we add a
fallback to the Tempest commit used prior to
5b4dd3c.

Change-Id: Idec3924a1b5476683e5520f83df8561d0bd0ccfd
Closes-Bug: #1524642
2015-12-10 09:01:50 +01:00
Ramana Raja
28fc5d375f Restrict scope of loop control variables to local
Some of the loop control variables are global. This as expected has
lead to corruption of their values, and produced undesired results.
Fix this issue by setting the scope of these variables to local.

Closes Bug: #1523348

Change-Id: I0f5676c5a0b97d3e027a1e7677b4b5a21470913b
2015-12-07 12:15:09 +05:30
Ramana Raja
5b4dd3cc70 manila: Update tempest code used by CI
The tempest code base used by the CI is outdated. So update tempest
tests as done for the  generic driver CI in the Manila code base,
6b8319e32a (diff-e0705abec56bf0f83781d686e109d63eR63)

Change-Id: Ide7966b44482dd13aadd5ef34a97c980e5752bfb
Closes-Bug: #1519012
2015-11-23 21:43:06 +05:30
Bharat Kumar Kobagana
f53859f89e Refactoring devstack-plugin-glusterfs code
This patch does the following:

* Moved variable setting part from plugin.sh to settings file.

* By default all Gluster-[Glance|Nova|Cinder|Manila] integrations
disabled. Look at README file to know how to enable.

* Keeping CINDER_ENABLED_BACKENDS intact when
CONFIGURE_GLUSTERFS_CINDER=False (Bug #1500381)

* Renamed CONFIGURE_GLUSTERFS_BACKUP to
CONFIGURE_GLUSTERFS_CINDER_BACKUP

* Avoid setting unnecessary variable when speicific configuraiton
is disabled.

* Documented the steps to enable in README.md file of this project.

Closes-Bug: #1500381
Depends-On: Id6967499729dfaa12d8a338d8f72471ebc90fb0d
Change-Id: I36f35f8af8e3491c8b0cffb40ba3980b7beab5df
2015-10-21 15:04:48 +05:30
Jeremy Stanley
ac3b13c572 Update .gitreview for new namespace
Change-Id: Iac83b5d5b354f2d73223b132fc48e3c3292d9d38
2015-10-17 22:06:33 +00:00
Csaba Henk
9110aa631b retry mkfs.xfs until subject LV becomes available (or a time bound is reached)
Closes-Bug: #1499223
Change-Id: I7ede18cadad1c7a30d740ae17cab52b3b9271ce0
2015-09-24 12:42:32 +02:00
Ramana Raja
8f92d69980 manila-CI: Use 'capability_snapshot_support' opt
The newly introduced Tempest config option,
'capability_snapshot_support' allows glusterfs_native driver
that can support snapshot support to run with snapshot Tempest tests
disabled. Set this option to True to let the glusterfs_native driver
CI not run snapshot tests - the present status-quo of the
glusterfs_native driver CI, and yet succeed to run.

Change-Id: I7e605502a9d6d3a42a26313a7c3bbb53ec08d8bc
2015-09-23 20:33:27 +05:30
Bharat Kumar Kobagana
daeb29f125 Install glusterfs 3.7 for manila jobs
This patch modifies devstack-plugin-glusterfs code to install
glusterfs 3.7 for manila specific jobs.

Change-Id: Ica4a44cd8dfe65a2330f65a1dcbb69a24fb2c805
2015-09-23 17:20:34 +05:30
Jenkins
aedb8a1050 Merge "Manila-CI: increase size of backend GlusterFS vol" 2015-09-23 09:46:33 +00:00
Jenkins
db2f1ec130 Merge "Manila: Fix share_type in glusterfs (NFS) CI" 2015-09-23 09:05:06 +00:00
Ramana Raja
830a57308b Manila-CI: increase size of backend GlusterFS vol
The glusterfs (NFS) driver CI that currently uses directly mapped
layout sometimes fails since the backend GlusterFS volume within
which the driver creates shares does not have sufficient space to
allow Tempest to create shares for testing.

Closes-Bug: #1498792
Change-Id: If9ac508834637908bb91df55e856e08bca01e88f
2015-09-23 14:03:21 +05:30
Ramana Raja
7f3e7268a3 Manila-CI: Fix tempest config option
The tempest config option, "storage_protocol" for glusterfs_native
driver CI needs to be "glusterfs" instead of "GLUSTERFS", as the
glusterfs_native driver reports its storage_protocol to be
"glusterfs" using update_share_stats share driver method.

Closes-Bug: #1498624
Change-Id: I1031cfaeab629b5aab316e611e9aae1833cf857b
2015-09-23 00:33:14 +05:30
Ramana Raja
2bd73dd2b6 Manila: Fix share_type in glusterfs (NFS) CI
During glusterfs (NFS) CI run, the extra spec "snapshot_support" used
for the default share-type is set to "True" even though the CI tests
the glusterfs backend with directory based share layout scheme that
does not support snapshot. So set the extra_spec to False which would
allow the Manila scheduler to pick up the glusterfs backend as a valid
host.

Change-Id: I7357417774cd16a2c114abd48a7bdb65a9eefc51
Closes-Bug: #1498399
2015-09-22 19:56:22 +05:30
Ramana Raja
3ae156a6de Manila: fix access rule type for gluster_native CI
Disable Tempest configuration that creates "ip" access type
rules for Manila CI using "glusterfs_native" driver.

Change-Id: I14061af63c3fddec5ec727f24c3aebba76080e11
Closes-Bug: #1498313
2015-09-22 12:18:03 +05:30
Ramana Raja
77c6200a86 Manila: Disable consistency group tests
The two glusterfs drivers in Manila do not yet support consistency
groups feature. So disable the related Tempest tests during the
CI runs of the two drivers.

Change-Id: I07e860971451c4aa875272a34ebf845603f9f0be
Closes-Bug: #1498101
2015-09-21 22:51:44 +05:30
Bharat Kumar Kobagana
fd30ab5d2e Manila-GlusterFS: Use Tempest plugin interface
This patch modifies the pre_, post_ test_hook.sh scripts to use
new tempest plugin interface as per below patch:
https://review.openstack.org/#/c/201955/

Closes-Bug: #1493802
Change-Id: I267fd14eb6324e533b32ae0bea2f3ecafac6fbc4
2015-09-12 08:55:29 +05:30
Bharat Kumar Kobagana
84f8ac4718 Manila-GlusterFS: Removing devstack/pre_, post_ test_hook scripts
As we modified Manila-GlusterFS job definition to use manila/pre_,
post_ test_hook scripts, now devstack/pre_, post_ test_hook scripts
not required.

This patch removes devstack/pre_, post_ test_hook scripts in
devstack/ directory.

Change-Id: I52d962a231c1b2bbc89a160fdc65ea218170a241
2015-08-24 09:22:46 +05:30
Ramana Raja
4467692d96 manila: add missing tempest config option
Set missing tempest config option for glusterfs_native driver.

Change-Id: I137d2a90bca3c92a57ef0cdbf3f5f8d05c12d392
2015-08-20 02:30:02 +05:30
Bharat Kumar Kobagana
9e03d4f71c Manila-GlusterFS: Fix Neutron config setting in pre_test_hook
This patch modifies the pre_test_hook.sh script to set
ENABLE_ISOLATED_METADATA in localrc for Neutron configuration.

Closes-Bug: #1486550
Change-Id: I7708c29998e5c0f46770c54e0c7d184aed388821
2015-08-19 18:17:18 +05:30
Jenkins
d223cc154b Merge "Changing file permissions of glusterfs log files" 2015-08-19 12:22:12 +00:00
Bharat Kumar Kobagana
a80ea210db Changing file permissions of glusterfs log files
This patch avoids creation of zero sized glusterfs log files by
changing the file permissions from 600 to 755.

Change-Id: I2c88781052f5cb334de21ffe1f98f13240820fa5
2015-08-19 16:29:10 +05:30
Bharat Kumar Kobagana
a8f8ba0a6e devstack-plugin-glusterfs: GlusterFS as a backup driver for Cinder
This patch allows devstack-plugin-glusterfs to configure GlusterFS
as Cinder backup target. Below are the new configuration options
added:
* CONFIGURE_GLUSTERFS_BACKUP (Set it to True to enable glusterfs backup)
* CINDER_GLUSTERFS_BACKUP_SHARE (ex: 127.0.0.1:/backup_vol)

Depends-On: I4f0c36cdaa7cc417cb26bc1e908c6d107a71da35
Change-Id: I03f1daa0c32fda20dc1900ed73e6ad586aee37a7
2015-08-19 15:29:33 +05:30
Jenkins
b52022d3e1 Merge "Configure Manila with glusterfs_native driver" 2015-08-18 05:35:17 +00:00
Ramana Raja
9868ce3fa0 Configure Manila with glusterfs_native driver
Allow the plugin to configure Manila with glusterfs_native driver as
the backend share driver.

Change-Id: I84e6978d681276a6de7b72ac29ccb7195f062bdb
2015-08-17 17:32:02 +05:30
Ramana Raja
858b2443f1 Manila: add pre_, post_ test_hook scripts for CI
Add pre_test_hook and post_test_hook scripts that enables Manila to
be setup with one of the GlusterFS based drivers as the backend
share_drive, and allows Tempest test cases to run with the Manila
setup.

Change-Id: I60c715350b8e53a7e19667574ce0e5ef02e51f22
2015-08-17 16:38:01 +05:30
Ramana Raja
e5404555a5 manila: skip snapshot related tempest cases
Skip snapshot related tempest cases for Manila.

Change-Id: Id36a42af595803357f2fc2a255b68dd7623e13bb
2015-08-15 19:28:44 +05:30
Bharat Kumar Kobagana
f0ce598c9b Configure Manila with glusterfs.py share driver
This patch configures Manila with glusterfs.py share driver.

Change-Id: I53c51dd213bd64c35fec54c45db92d22139d0e0e
2015-08-15 17:43:43 +05:30
Bharat Kumar Kobagana
b3c21638ac devstack-plugin-glusterfs: GlusterFS as a backend for Manila
This patch enables GlusterFS as a backend for Manila. This
patch also introduces two new configuration parameters:
 * CONFIGURE_GLUSTERFS_MANILA - set to 'True', to enable GlusterFS
as a backend for Manila.
 * GLUSTERFS_MANILA_DRIVER_TYPE - set Manila GlusterFS backend type
to either 'glusterfs' or 'glusterfs-native'.

Change-Id: I53e5b9e88b6ebc131c64dabc421875c06a3fafd4
2015-08-15 14:13:45 +05:30
Bharat Kumar Kobagana
7e7f058d84 devstack-plugin-glusterfs: GlusterFS as a backend for nova
This patch enables GlusterFS as a backend for Nova. This
patch also introduces two new configuration parameters
 * CONFIGURE_GLUSTERFS_NOVA - set to 'True', to enable GlusterFS
as a backend for Nova.
 * NOVA_GLUSTERFS_SHARE - which specifies the GlusterFS share
to store compute instances.

Change-Id: Iaebc0aff704d1203b91967af06b06bb08fde51bd
2015-08-10 15:19:23 +05:30
Bharat Kumar Kobagana
51d7091ed6 devstack-plugin-glusterfs: GlusterFS as a backend for glance
This patch enables GlusterFS as a backend for Glance. This
patch introduces two new configuration parameters:
 * CONFIGURE_GLUSTERFS_GLANCE - set to 'True', to enable GlusterFS
as a backend for Glance. By default set to True.
 * GLANCE_GLUSTERFS_SHARE - which specifies the GlusterFS share
to store glance images.

Change-Id: I613e4c7a3e359235f5d54abc820df95b521b8697
2015-08-10 15:13:18 +05:30
Jenkins
860ee45312 Merge "devstack-plugin-glusterfs: Adding checks for error conditions" 2015-08-10 09:39:45 +00:00
Deepak C Shetty
4a012088b2 Cinder: Make the default volume prov type as thick
Patch @ https://review.openstack.org/#/c/164527/14 changed the default
prov type to thin, which causes the volume backup tests in tempest
to fail, as we don't yet support backup for thin(qcow2) files.

This patch fixes the issue by making the default prov type as thick
for the CI job

Change-Id: I9edd0e0d0a82f08305b62951a87cf45b4df8d2f7
Closes-bug: #1483126
2015-08-10 12:13:15 +05:30
Bharat Kumar Kobagana
995c1ee62f devstack-plugin-glusterfs: Adding checks for error conditions
This patch introduces a new configuration parameter (knob)
CONFIGURE_GLUSTERFS_CINDER to enable/disable glusterfs as a
backend for Cinder. And added below error conditions:

* glusterfs plugin enabled but CONFIGURE_GLUSTERFS_CINDER set to
False
* CONFIGURE_GLUSTERFS_CINDER set to True, but CINDER_ENABLED_BACKENDS
doesn't have at least one backend of type glusterfs.

Change-Id: Ic802a2f2bbbdc1af04fd1220d2055be1e853fef8
2015-08-06 13:14:14 +05:30
Bharat Kumar Kobagana
f0e403da25 devstack-plugin-glusterfs: Changing gluster volume names
This patch changes the name(s) of gluster volume(s), required for
cinder to store cinder volumes.

Change-Id: I95ac68a1bba53d1831903596c43650a8b4b045bc
2015-07-21 17:02:08 +05:30