5636 Commits

Author SHA1 Message Date
Zuul
a340217965 Merge "[DOC] Incorrect response code for share network creation" 2022-02-08 01:43:21 +00:00
Zuul
1537aa5e07 Merge "Add admin only keys to share metadata" 2022-02-05 00:18:07 +00:00
Zuul
354b3bccbb Merge "Remove usage of six lib for third party/vendors drivers." 2022-02-04 23:39:20 +00:00
Zuul
ad8e3b644e Merge "Cinder: Remove unused API" 2022-01-31 22:11:05 +00:00
archanaserver
96823254c1 [DOC] Incorrect response code for share network creation
The response code of share network creation operation is
mentioned 202 while it should be 200.

Closes-Bug: #1958072

Change-Id: I27c24c71a7c388033fbdf206c2d3e8004839d98a
2022-01-31 08:52:31 +00:00
haixin
a73b299374 Remove usage of six lib for third party/vendors drivers.
Python2 is no longer supported, so in this patch
set we remove the usage of the six (py2 and py3
compatibility library) in favor of py3 syntax.

Change-Id: I3ddfad568a1b578bee23a6d1a96de9551e336bb4
2022-01-29 03:01:17 +00:00
Zuul
82b9dfe0a6 Merge "Fix note in the share manager" 2022-01-27 21:10:18 +00:00
Zuul
40c9635a23 Merge "Force disk wipe when running lvcreate" 2022-01-27 16:54:47 +00:00
Takashi Kajinami
1bbd229080 Cinder: Remove unused API
The get_volume_encryption_metadata method is implemented in the API
class but it is not used.

Change-Id: I24ebca15bc65b6894bca478c29023770e23b5dcc
2022-01-26 23:22:39 +09:00
ashrod98
0f161ce17a Add admin only keys to share metadata
In this change, we add admin policy check for metadata keys
"__affinity_same_host" and "__affinity_different_host".
These keys are added an constants, allowing for the addition of other
admin only keys if necessary in future changes.

This change allows for the affinity filter spec[1] implementation without
the additional changes necessary in the metadata spec[2].

Bumps microversion to 2.68.

[1] https://specs.openstack.org/openstack/manila-specs/specs/xena/affinity-antiaffinity-filter.html
[2] https://specs.openstack.org/openstack/manila-specs/specs/wallaby/metadata-for-share-resources.html

Partially-Implements: metadata-for-share-resources
Change-Id: I6ee02cb66727a3c2b425fad1d9d1245d4e9c1b24
2022-01-24 21:38:18 +00:00
silvacarloss
89028725d1 Force disk wipe when running lvcreate
The LVM driver may fail while creating new volumes using the
lvcreate command. Sometimes it can say an existing ext4 filesystem
was found in the destination, but we shouldn't be worried about it.
This change modifies the behavior and forces the lvcreate command
to wipe the space if a signature was found.

Change-Id: I4c10bcdfc499452d4b05dfbee8059be35dfe5586
2022-01-20 17:42:45 +00:00
Zuul
d1509256da Merge "OnlyHostFilter allows user to specify host during share create." 2022-01-14 04:38:59 +00:00
silvacarloss
8c25a0b297 Fix note in the share manager
Addresses a comment that was added regarding a NOTE introduced by
change [1]

[1] https://review.opendev.org/c/openstack/manila/+/820071

Change-Id: Ib4a03db65552bcfce0b21acd7499d0e1ed88db3e
2022-01-13 19:22:01 -03:00
Zuul
5b6197821b Merge "Force share server selection on nondisruptive migration" 2022-01-13 18:47:28 +00:00
Fabio Oliveira
603e484fb6 api-ref: Add share instances by share endpoint
The endpoint for /shares/{share_id}/instances was missing for the
share section.
This patch adds the section and update the existing one on
v2/shate_instances.

Closes-Bug: 1952673
Change-Id: Ie8412379c20528575d2e67485ce955bcf8c13eee
2022-01-11 14:57:05 -03:00
Kiran Pawar
746fb7e2df OnlyHostFilter allows user to specify host during share create.
e.g. manila create NFS 1 --name Share1 --share-network net1 \
--scheduler_hint="only_host=host1@generic1#GENERIC1"

Since there is no way to create share server in manila, we can use a
workaround of creating first share on specific host
(e.g. host@backend#pool). This will then create the share server
automatically on that host and admin can use idle host when other
hosts are overloaded.

New microversion 2.67 introduced.

DocImpact

Closes-Bug: #1946462
Change-Id: I603434cac246e2c0946672d3f0fe469ed5423fa4
2022-01-10 10:06:27 +00:00
Zuul
142990edc0 Merge "ratelimit should not be enabled in nolimit pipeline" 2022-01-06 00:09:26 +00:00
Takashi Kajinami
d3c4e9c31a ratelimit should not be enabled in nolimit pipeline
This change fixes the wrong instruction in admin guide which enables
ratelimit in the keystone_nolimit pipeline. To use a pipeline with
ratelimit, [DEFAULT] api_rate_limit should be set to True.

Change-Id: I0cfde1c5a18eeb6ace5bc96e728e58954d0ac046
2021-12-27 12:45:13 +00:00
lkuchlan
466794964f Add enforce_scope setting support for Manila
Allow Manila for secure RBAC and set enforce_scope
for manila-tempest-plugin.

Change-Id: I3ae6efaa7d69690dca60e90b6d31dcdce9d9280d
2021-12-26 11:56:28 +02:00
Zuul
1ebae738c2 Merge "Set tempest api version config in devstack plugin" 2021-12-23 19:45:38 +00:00
Goutham Pacha Ravi
1caecc598e Set tempest api version config in devstack plugin
manila's tempest plugin is branchless and can at
any time add tests for API microversions incompatible
with stable branches.

We used to configure test API versions via the post test
hook when relying on devstack-gate. However, API version
config makes sense whenever we use devstack and have
tempest enabled. This will help us simplify default
job configuration in the manila-tempest-plugin repository.

Since the API version won't change on a stable branch,
hard-coding the versions (as other micro-versioned services
do in their devstack) doesn't pose any risk of failing
reality.

Change-Id: Ia671fa74c0ee338199bd92a1613882328d24c9e2
Related-Bug: #1928879
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
2021-12-22 12:12:08 -08:00
haixin
03fea855d9 remove usage of six library in api layer
Replace six with Python 3 style code.

Change-Id: I091a45c04686ac8d036107618945fea65fb4a2cf
2021-12-22 10:08:11 +08:00
Zuul
37c41e57cf Merge "Support group specs search for share group type API" 2021-12-21 23:26:46 +00:00
Zuul
09cf65a36a Merge "Move db release note to correct folder" 2021-12-20 17:41:02 +00:00
silvacarloss
af3513d2b0 Force share server selection on nondisruptive migration
While performing nondisruptive migrations, share servers can not
change, because such thing will cause the migration to be
disruptive, since share servers have different network allocations
and it will cause the destination share to change its export
locations.
This change fixes an issue that allowed manila/the share driver
to chose where a share instance is going to land in case of a
nondisruptive migration. Now, when such migration is required,
manila will automatically place the destination share instance in
the same share server that the source instance is located at.

Closes-Bug: #1920942
Change-Id: I16110ec46c25be65760b15d7fbed67cd005f3873
2021-12-17 11:12:03 -03:00
Zuul
c29fa635a1 Merge "NetApp OnTap: Fix compatibility check for share migrate" 2021-12-15 23:54:48 +00:00
Zuul
23ac0b6d5d Merge "Fix ignored [neutron] options" 2021-12-15 02:38:35 +00:00
Zuul
5712fe596a Merge "Move release notes to correct folders" 2021-12-14 19:27:55 +00:00
Victoria Martinez de la Cruz
3a1e23801f Support group specs search for share group type API
Add support for group_specs filter search in share group type list API.

Implements: blueprint support-group-spec-search-share-group-type-api
Change-Id: I10caed1d524614a74a75e8f856b0579ab11dc4b5
2021-12-09 17:26:11 +00:00
Zuul
5ff97f51a2 Merge "early return for _share_replica_update() if there is no active replica" 2021-12-09 00:09:13 +00:00
Zuul
00332b3d19 Merge "[NetApp] Fix default thin_provisioned volumes on AFF" 2021-12-08 22:11:17 +00:00
Zuul
1856bf632c Merge "Follow up for share server migration enhancements" 2021-12-08 21:18:28 +00:00
Zuul
dad3cc805c Merge "Update admin, user and contributor guide" 2021-12-08 20:48:38 +00:00
Fabio Oliveira
9417ece456 NetApp OnTap: Fix compatibility check for share migrate
The issue happens when the parameter 'encrypt-destination' is sent to
OnTap devices with version lower than 9.0 or OnTapi lower then 1.110.

This patch adds a condition to check these versions and ignore the flag
addition if needed on '_send_volume_move_request'.

Closes-Bug: 1915237
Change-Id: Id9faa4f73996cb7467b7361d0406c980f06905c2
2021-12-08 16:17:57 -03:00
Zuul
c09a5f08c4 Merge "Add api-ref for security service update" 2021-12-08 15:21:06 +00:00
Goutham Pacha Ravi
578e3c78c2 Move release notes to correct folders
Idb4b6de5108dd1862df7de97d0ffdc15282b1d74 recently merged and
is backport worthy to all maintained older stable releases.

I34020da6a929f5c7ae1c63bc0f2ef338c49dbe10 was committed
to stable/xena when it was still under development; the bug-fix
in question is not back portable to other stable branches, so
backporting this release note fix to stable/xena should suffice.

Change-Id: I751b6e4a3288cde7a98c0ddc55deac7a0682010d
Related-bug: #1942124
Related-Bug: #1475351
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
2021-12-07 14:21:27 -08:00
Goutham Pacha Ravi
021e7ab02f Move db release note to correct folder
This change must be backported to stable/wallaby
as well.

Change-Id: I6d4f2cc263fc7dd099ce27e674709baf3916ea9b
Related-Bug: #1908352
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
2021-12-07 14:14:32 -08:00
silvacarloss
40da25f286 Add api-ref for security service update
Updates the API reference to fit into the changes merged in the
add/update security service in share networks feature.

Change-Id: Id7dd45dc23c41c1ade28b042f87b216142b8a062
2021-12-06 19:07:13 -03:00
Zuul
21d93dc5d1 Merge "Handle successful deletion of snapshot if quota commit fails" 2021-12-03 23:28:13 +00:00
Megharth
a6177bcc0d Handle successful deletion of snapshot if quota commit fails
Replaced a non-existing snapshot delete method
with an appropriate one and added tests for
the same

Closes-Bug: #1475351
Change-Id: Idb4b6de5108dd1862df7de97d0ffdc15282b1d74
2021-12-03 19:24:10 +00:00
Maurice Escher
7b7d18e142 early return for _share_replica_update() if there is no active replica
If the parent share of the replica has no active replica,
this is an error situation where _share_replica_update() simply
returns and logs a warning if the replica being acted on has not
been already in error state itself before.

Change-Id: I9bdbbaafba413fa4b2a783130fd58a4b6c2cea4b
Partial-Bug: #1924230
2021-12-03 15:49:55 +00:00
Goutham Pacha Ravi
6943b31697 [devstack] Use OSC to set up share types
We used a workaround to solve the lack of legacy credentials
in our devstack plugin script by sourcing credentials early [1].

This workaround breaks when using the devstack-system-admin
persona from the /etc/openstack/clouds.yaml that devstack
sets up, because we'd be sourcing project credentials that aren't
popped off in time [2]. Cloud profiles work with OSC, we could
switch to that in the devstack plugin rather than rely on legacy
credential helpers from devstack. The only remaining piece is the
group type setup [3] which is expected to be complete soon.

[1] https://review.opendev.org/c/openstack/manila/+/818617
[2] https://review.opendev.org/c/openstack/devstack/+/817074
[3] https://review.opendev.org/c/openstack/python-manilaclient/+/805064

Change-Id: Icc15f427cc02054021fa23626ea14e6cd37b18fb
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
2021-12-02 15:43:19 -08:00
Zuul
72ecaf0fab Merge "[api-ref] Fix datatype of cast_rules_to_readonly" 2021-12-02 11:59:13 +00:00
tutkuna
d4c0f4a907 [api-ref] Fix datatype of cast_rules_to_readonly
Fixing cast_rules_to_readonly type from string to boolean
for share instances and share replicas.

The API-REF had the wrong type for these fields.

Closes-Bug: #1948717

Change-Id: Ie8defe1fcb03bced9887ba1b41d5b062ba477924
2021-12-02 00:39:34 +00:00
Zuul
7b30e33ebd Merge "Deletes the six library and all its usages from manila/utils.py" 2021-12-01 06:57:36 +00:00
Zuul
d4e16a30b9 Merge "[TrivialFix] Fix up a bugfix release note" 2021-11-30 12:26:43 +00:00
Zuul
7e392f3acb Merge "Fix glusterfs test bug" 2021-11-23 19:33:21 +00:00
Zuul
7dd17f4ed0 Merge "replace deprecated pyparsing method" 2021-11-23 15:13:55 +00:00
Zuul
469e547b88 Merge "Modify docker instalation for fedora systems" 2021-11-23 12:33:08 +00:00
Goutham Pacha Ravi
0dbe028009 replace deprecated pyparsing method
pyparsing renamed a method used to define the operator
precedence hierarchy [1]. The method's behavior was preserved
although opt-in override enhancements to the paranthesis were
introduced.

Needed-by: https://review.opendev.org/818614/
[1] ab2f220dd2

Change-Id: I77ea03284c1db6b4eb171ef6e2e88ac506304502
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
2021-11-22 13:08:20 -08:00