When the cDOT driver receives a create_share_from_snapshot command,
it clones the share/flexvol from the specified snapshot in the
parent share/flexvol. But if the resulting clone is not split
from its parent, the snapshot remains busy and may not be deleted.
This is resolved by starting a clone split operation anytime a
clone is created.
During the split operation, which usually completes within
seconds, the snapshot remains busy. This patch detects that
condition in the snapshot delete path and waits up to a minute
for the busy condition to clear.
Change-Id: I44bac106f7f7a1acc968c1d9a526dcef85f657b7
Closes-Bug: #1259988
Replace explicit usage of SQL code with appropriate analog of ORM.
It allows to keep abstraction between DB and python clear without
potential incompatibilities due to differencies in SQL dialects.
Change-Id: Ied97264cafcec0bcebd4c3ddc7b29185825c56a6
Closes-Bug: #1437278
We have three tempest tests that are skipped due to bugs that won't be fixed.
Delete them as useless.
Change-Id: I5faa841887de24f483fdb3a0b5a413ecbb3717f7
Closes-Bug: #1431939
A common user error with the single-SVM cDOT driver is failing to
assign aggregates to the storage virtual machine (aka vserver) used
by the driver. The driver should fail fast if there are no available
aggregates for share provisioning.
We want the multi-SVM driver to fail fast as well, if only for
consistency and to alert an admin of a problem with the
netapp_aggregate_name_search_pattern regex in manila.conf.
Change-Id: I2a2015bd07b68177b0a5e7ac322ffa50b3b41c7d
Closes-Bug: #1433871
Minor update, as per the review comments on previous doc change.
Moving Ganesha protocol version support statement from
'Known Restrictions' to 'Requirements' section.
Change-Id: I870146a68691143af6b5ca51fb41eb390af12de6
Implements: blueprint gpfs-driver
Currently v1_1 version of novaclient is deprecated, so it's required
to switch for v2 version of novaclient.
Change-Id: I824e8659febfeb86721b610a03a8c3027042cb3d
In order to be consistent with other Manila drivers, change the
generic driver to return export paths with backslashes instead of
forward slashes.
Change-Id: Ibba236e4fcc5aaf88bee8898b51eedd4852f5aae
Closes-bug: #1434816
This cDOT multi-SVM driver is coded to always create VLANs on
physical ports prior to creating logical interfaces (LIFs). Given
the new network plug-ins in Kilo, StandaloneNetworkPlugin and
NovaNetworkPlugin, the cDOT plug-in must be fixed to allow
creating LIFs on non-VLAN ports.
Closes-Bug: #1434919
Change-Id: Ie9f30774b5191876bd42a1d502493e4e8e3ceeb3
The multi-SVM cDOT driver that ships with Kilo does not support the
8.3 cDOT release. The driver should fail fast if 8.3 is detected.
Change-Id: Iac7bafd101f0b0ad190a49cd2bba79bd25dee028
Related-Bug: #1425754
Closes-Bug: #1433872
The EMC VNX driver returns CIFS paths in the form: //server/share.
It should be \\server\share. This patch fixed it.
Change-Id: I3d53a2e01cf673a50f9a1e951a29ce29b4cd4f66
Closes-Bug: #1434817
Shares with snapshots should not be able to be deleted or unmanaged.
But Manila does not make verification for presence of snapshots for share
that is requested to be unmanaged.
Change-Id: I8a58cb3e801ac123b153177e316b712ee999142e
Closes-Bug: #1437269
Export locations stored in separate table are not deleted. Never.
Fix it for two cases: when share deletion is performed and export locations
update operation that 'share driver's do.
Change-Id: I7b66c8843647862e398e71898ac258f07f264e3a
Closes-Bug: #1435313
Adapt manager and add first initial test for Manila. A ping check will
be performed to check the connectivity from the VM to the share
server/service. The test can be triggered using the experimental test
job introduced by [1].
Open todo's that can be addressed later:
- Support of flat network
- More flexible configuration options
[1]: https://review.openstack.org/#/c/164102/
Change-Id: Ie72e877a88c337866e9cb591f8099218d84ee5dd
Partially-implements: blueprint scenario-tests
API 'manage' allows us to set name and description for newly managed share.
But it always sets empty strings for both. Root cause for it lays in different
naming of mentioned attrs in API and DB layers.
So, make API 'manage' understand both names (API-like and DB-like) for each of
mentioned things.
Change-Id: I5a752c7960492070c3f5ad5974156c4dea324ac1
Closes-Bug: #1434056
Share types have one required extra spec - "driver_handles_share_servers".
And API disallows us to set some additional extra specs if we do not provide
required one too. It should not be so, because if we do not want to update
already existing spec then we should not be obligated to provide it.
Change-Id: Id2b2c9dfae3da551ae79e66199a84773f333b883
Closes-Bug: #1435819
Wikipedia's list of common misspellings [1] has a machine-readable
version. This patch fixes those misspellings mentioned in the list
which don't have multiple right variants (as e.g. "accension", which can
be both "accession" and "ascension"), such misspellings are left
untouched. The list of changes was manually re-checked for false
positives.
[1] https://en.wikipedia.org/wiki/Wikipedia:Lists_of_common_misspellings/For_machines
Change-Id: I2446995aa8b7c89c7f78c056f9ddb6d8e7899e4e
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Shares with transitional states like 'creating' or 'deleting' should not be
allowed to be unmanaged.
For the moment "manage/unmanage" operation is allowed only for driver mode
without handling of share servers. So, API operations against shares created
on top of share servers should be restricted on API level.
Change-Id: I73b554448eadcb96ef00f44535014e14dea91472
Closes-Bug: #1434642
Closes-Bug: #1434511
We are forced to sync our tempest plugin to latest tempest due to changes
done to CI system, that are based on recent tempest project changes.
Such as reusage oslo libs instead of built-in incubator code, that leads to
changes of config opts that are set/read within tempest config used in CI.
Change-Id: I3e64dc373f5d81034afc4c5d8dfd5d7f4abaa603
Closes-Bug: #1434958
Recently was added support of manage/unmanage operations to deneric driver.
But it does not update export locations according to server info.
So, make driver update it and add appropriate change to share manager to
expect export locations.
Change-Id: I50726ce0b82b50363f5651ffced8b6cb9f0d853f
Closes-Bug: #1434009
PEP-0476 introduced more thorough certificate chain verfication
for HTTPS connectivity; this was introduced in Python 2.7.9, and
breaks a number of unit tests in the manila codebase.
Disable certificate chain verification for SSL tests
using the backwards compatible SSLContext provided for this
purpose.
Change-Id: I512e4bd5a10fcfdf6241329dbd1eae6e9350854b
Closes-Bug: #1403068
Recently was added support of manage/unmanage functionality to generic driver.
See change: I8767a7af75d575c57e8708563fa4a6bf2f4aa8ad
Changes here should had been included to mentioned commit, but were not by
mistake. Without this change manage/unmanage functionality does not work
using generic driver.
Change-Id: Ib034a1715351cbcd71a61a013571427af0b14492
Closes-Bug: #1433973
