We recently addressed a bug in Manila where
share networks couldn't be deleted when they were
still associated with any share groups [1]. After this
fix, we had a couple of tests failing sporadically
because they weren't waiting for the deletion of
the share groups prior to attempting deletion
of the share network subnets.
Change-Id: I532ff3f0b2613340e0cb46c3b7e57a22cfa50c61
Related-Bug: #2004212
Closes-Bug: #2064673
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
As 2024.1 is released, we should add its job on master
gate to keep branchless tempest plugins compatible to
stable branch.
At the same time, we're dropping the zed job
since the zed branch of openstack/manila has now
transitioned to "unmaintained". manila-tempest-plugin
may no longer support running against the zed branch.
Please use the "zed-last" tag instead [1]
Ref: Tempest plugins guide for stable branch testing:
- https://docs.openstack.org/tempest/latest/stable_branch_testing_policy.html
[1] https://review.opendev.org/c/openstack/releases/+/918346
Change-Id: I0625576d14d9e084aa65beed5d0c988dcebd85dc
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
These are detected as errors since the clean up was done[1] in
the requirements repository. Minimum versions are bumped to avoid
installing known bad versions.
[1] 314734e938f107cbd5ebcc7af4d9167c11347406
Also remove the note about old pip's behavior because resolver in
recent pip does not require specific order.
Change-Id: I2606a494c60342b9849f8c837602dc81d28a5f95
We added new field "backup_type" in share-backups API
in 2.85.
Change-Id: Iaaf3c3914089ad0d27b61a524f14353cae35c4d5
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
This is an effort to remove package based deployment
of Ceph from devstack-plugin-ceph in favor of ceph
orchestrator (cephadm) deployed ceph.
Change-Id: I0c35ef0eb4293aae37cd597d78d7aed0100e8ec7
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
Adds new test cases and updates current test cases for access allow
and deny APIs. Such APIs now allow users to restrict the visibility
of some fields during the access creation, or even restricting the
deletion of access rules.
Depends-On: https://review.opendev.org/c/openstack/manila/+/887725
Change-Id: If0f7655dc6c11c6c12eeb76bd1cc853466235cca
When backups fail to build, tempest is not displaying the correct
exception because the class name is not following the pattern we
expect it to follow.
This change fixes the class name for the share backups build error
exception.
Change-Id: I9553ba44544d84dbc452973252391967225c724e
There are few tests that generate random test names because
random parameters are given to the ddt.data decorator.
In the ddt version number 1.6.0 there is a possibility to
determine the name of the tests without relying on the
given values. I think this could be a good solution for
the randomly generated names.
This change is being re-introduced as ddt version 1.6.0 is
now usable in the downstream CI. The original change is: Iaf39f4a83e383fb9db7f519136b9169352ff84af
The following change is also included: I4d274869e5e9e7800be44bccf7a2641b4a5febe8
Which replaces periods to underscores, a needed fix.
Change-Id: Iaf39f4a83e383fb9db7f519136b9169352ff84ad
There are some metadata that only drivers or administrators can
manipulate, and if the tests haven't created those, we should not
be adding them to our asserts.
In this change, we are making sure that tests will only validate
the metadata that they are creating.
Change-Id: I168e1d9057d48b61eb325f2fd93dcc12a406f06e
The usage of sets introduces unneeded randomness, and the
utils.deduplicate function can be used instead.
This is a follow-up to similar past changes, see:
I9cbd26016238c25487ac8104c1188cd2cf4f467e
I15bfbf6e1ca604f2bf2fcf12dcd4e67bc728fe5e
I8fce553d927da6d94471ecdf4040bbbb4377d9cf
Change-Id: I5477b815f212c5468c5e52905c4e1e1d4ef082bb
Xena and Yoga are past the stable maintenance
cycles. We're soon going to tag these release
branches as "unmaintained" branches [1]. We don't
expect the trunk version of manila-tempest-plugin
to work against these older branches any more.
Instead, we have tagged a "xena-last" and will
soon [2] tag a "yoga-last" branch that will be more
appropriate.
Going forward we must be testing the "maintained"
stable branches, which are zed, antelope, bobcat
as of this commit.
[1] https://docs.openstack.org/project-team-guide/stable-branches.html#unmaintained
[2] https://review.opendev.org/c/openstack/releases/+/908381
Change-Id: Ib0e984a4acdc2f82f5e3a75e34c7ec922674b0a5
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
Modify "wait_for_resource_status" method so that we can pass the timeout
parameter from test cases and modify the test cases as per lib changes.
Change-Id: Id67589bf2d2094355003a16e3379867ba81bfb6b
The scenario test
`test_write_data_to_share_created_from_snapshot`
was being skipped because of a bug in ceph's MDS
service. It has now been addressed [1]
[1] https://tracker.ceph.com/issues/57210
Closes-Bug: #1989273
Change-Id: I967479458038b67c7892a71ac881491e1f8c87f7
Depends-On: Iea541d2edefc871bcac2d965997c88462fcbe521
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
In some share metadata operations performed on a share resource
in another project by member and reader users, the response
code is 403/Forbidden but actually it should be 404/Not Found.
Related-Bug: #2004230
Change-Id: I23f592aa0616dd000c91366056b19f73d4a92fa5
Use if=/dev/zero: Instead of reading from /dev/urandom,
which is slow for generating random data, use /dev/zero
to write zeros to the disk. Zeros are generated much faster.
Omit conv=fsync: The conv=fsync option forces dd to
synchronize the data after each write, which can be slow for
large writes.
Change-Id: Id845c60c39072a03171fa5906461eb254c2736a7
We had disabled scenario tests on the IPv6
test job manila-tempest-plugin-lvm [1] to prevent
gate failures at feature freeze. We can partially
re-enable these tests; while still working to
get IPv6 testing as well soon.
[1] https://review.opendev.org/c/openstack/manila-tempest-plugin/+/893651/
Related-bug: #1998489
Change-Id: Ie6599bc3533a002fa118ebe8b9243d677bdb410e
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
some places in scenario manager were still using very long deprecated
nova APIs, like nova images and nova networks,
which breaks if tempest is configured to use nova microverson > 2.35.
Change-Id: I5b83e0d120525f7e8959f68a340635cf954bc7f8
A multinode job creates a controller node
that runs manila with other OpenStack components;
and an additional compute node to spread the
compute VM workload around.
`manila-tempest-plugin-multinode-cephfs-nfs-cephadm`
replaces the existing `manila-tempest-plugin-cephfs-nfs`
in the check queue for this repository. These jobs
are different. `manila-tempest-plugin-cephfs-nfs`
uses devstack-plugin-ceph to install ceph and nfs-ganesha
as standalone components on the devstack host, whereas
`manila-tempest-plugin-multinode-cephfs-nfs-cephadm` uses
cephadm to deploy ceph, and it includes the deployment
of an nfs service. Manila's ceph driver communicates with
cephadm deployed nfs via ceph-mgr APIs, and with standalone
nfs-ganesha service via DBUS APIs. Ideally we'd run
both jobs all the time, however, since these jobs will
run individually against the manila repository, we'll just
pick the newer one here.
Depends-On: I84249ae268dfe00a112c67e5170b679acb318a25
Change-Id: I5162815b66d3f3e8cf8c1e246b61b0ea06c1a270
Co-Authored-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
Zuul has deprecated Perl-style syntax; and we're
getting warnings posted against the manifests:
The RE2 syntax error is: invalid perl operator: (?!
Change-Id: I498573532fd701b47684a54bda9b54b86836afdc
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
A couple of issues happening at the moment:
- Nova recently merged a change that bumps the libvirt version in
the Bobcat release [1]. The new libvirt version is not available
in Ubuntu Focal. In order to get the new libvirt version, our jobs
that were using Ubuntu Focal, are now forced to use Jammy, so we
bump it to the new version in this patch.
- The support for Ubuntu Focal has also been dropped in this
release [2]. Making it one more reason to get the new jammy
images.
While upgrading the image, some other issues surfaced:
- Some CI jobs are doing IPv6 testing, which use quagga. Quagga
does not have a candidate for installation in Jammy and must be
replaced. This issue is already documented [3].
- The inability to set up IPv6, combined with the scenario tests
being enabled in the LVM and CEPH drivers brought up another known
issue: when doing IPv6 setup, we don't set up only IPv6 networking
and we also end up setting crucial information for the scenario
tests to run in the function that does IPv6 setup. So in case the
job is only using IPv4 in scenario testing, we currently don't set
some key configuration to let the shares be created/mounted [4].
- Due to the reasons above, the scenario tests will never pass, as
we can't use Focal images, and when using Jammy, we can't do
scenario testing.
This is blocking the Manila gate and all other changes in the CI.
As we are forced to use the Jammy image and the scenario tests
won't pass, we disable them in this change, do some refactoring
in Manila [5], and re-enable them again for IPv4, until we have
figured out the quagga replacement.
[1] https://review.opendev.org/c/openstack/nova/+/887255
[2] https://lists.openstack.org/pipermail/openstack-discuss/2023-September/034968.html
[3] https://bugs.launchpad.net/manila/+bug/1998489
[4] https://github.com/openstack/manila/blob/master/devstack/plugin.sh#L1104-L1116
[5] https://review.opendev.org/c/openstack/manila/+/893804
Change-Id: Ia45a8e7f7c9cf9959bcc9b9a7243cccbbd1f4916
The tests validate policy of share groups actions for
admin, member and reader users in a project scope.
Change-Id: Iee0f739d9701b2efe9b275ea22859318b58a46be
We have a problem with glance's RBAC changes
in the generic driver. Disable the enforcement temporarily
while we explore a fix
Related-Bug: #2031191
Change-Id: Ia9c9eb4542c576e3e4d94334e62649abf3cb6faf
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
Add API tests for the resource locks APIs
Change-Id: Idf71e236b1b8a2558bb4ad3de1018fa33b41877f
Partially-implements: bp/allow-locking-shares-against-deletion
Depends-On: I146bc09e4e8a39797e22458ff6860346e11e592e
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>