| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Replace six with Python 3 style code.
Change-Id: I4b97e040f3e790ac114dcd43c68e6b67b1079adf
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently ``id`` and ``volume_id`` filters are working
correctly for transfer-list command.
support for filtering by ``name`` is handled in patch
provided in Depends-On.
Since filtering by all parameters is supported by API,
we can happily add the filters option on the client for
volume transfers.
Also adds functional test for transfers.
Related-Bug: #1860100
Depends-On: https://review.opendev.org/#/c/703658/
Change-Id: I2fd3a6a7b9add65a9a21388df44efb6747065a74
|
| |
|
|
|
|
|
|
|
|
| |
The ability to enable multiattach on the command line was deprecated in
Queens with the full implementation of multiattach enabling it through
volume type extra specs. This removes the command line arg and handling
for specifying it with volume creation.
Change-Id: Ifc0c874657f959266050cd1a7a40e6ecccc8c114
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
We are past the deprecation period for these commands and they should
now be removed.
Change-Id: I37f0dc539da5d43f629ea726bb603fa995c1fe6f
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
|
| |/
|
|
|
|
|
|
|
| |
- rename it so that it follows the documented pattern
(<project>-<specifier>);
- remove the old job, including the hooks, hoping that no
3rd-party CI uses them.
Change-Id: Ib690c4fa96354dc422576d76c1296b1d5ba3494d
|
| |
|
|
|
|
|
|
| |
With the api-schema enforcement on the Cinder API side, the error
message returned when specifying a volume creation of 0 has changed.
This results in our functional tests failing.
Change-Id: I1a9a13f683134faa01ad50f7f073db8b1845a901
|
| |
|
|
|
|
|
|
|
| |
switch to use OS_VOLUME_API_VERSION = 3 to run
functional tests.
Closes-bug: #1775354
Co-Authored-By: liuyamin <liuyamin@fiberhome.com>
Change-Id: If6c4291aa7d8e85a4f8c1d5665a0e4a72aa1f1d1
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
After applying schema validation to volume actions
APIs[1], error messages are formatted as per the
standards. So the tests are failing because of error
messages.
This patch fixes failed functional tests by correcting
error messages.
[1]https://review.openstack.org/#/c/559042/
Change-Id: Id05bf637d77ab401fc0aec5be09eee423bf1a223
|
| |
|
|
|
|
|
| |
Zuul v2 uses 'jenkins' as user, but Zuul v3 uses 'zuul'.
Using $USER solves it for both cases.
Change-Id: I7bd4c3d5a0acc2c73557a65dcc274a97f16bfd07
|
| |
|
|
|
|
|
|
|
| |
Enforce ordering of imports with H306.
For tests, this is mostly done by grouping test
imports after other cinderclient imports.
Change-Id: Ie40fda014d1aedb057e5b4ea1f27f999c84e6373
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Gives more time to wait resource's status to change
in functional testcases and adds default interval
to reduce query amount.
Change-Id: I2a29a2d04836fd1261d45e404341bb1aa657417b
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The show command currently just dumps the entire attachments
dictionary into the output which is a real mess and completely
screws up the displayed output (shell command).
There's really no reason to do this, we can just give the
attachment ID's and then when you're on the newer versions
you can do an attachment-show for all the crazy details if
you want. Keep in mind that the list command already shows
the server-id we're attached too, but that might also be
nice from the show command rather than jumping through
multiple commands. To try and accomodate various use cases
we'll also add an "attached_servers" field to the show command,
but you'll have to coorelate manually from there.
Change-Id: I45ac49d8d9a185c52727c5bc24a6a1323be83689
Closes-Bug: #1494941
|
| |\ \ \
| |_|/
|/| | |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The behaviour of PY3's standard lib 'argparse'
differs from PY2's.
```
cinder extend 2
usage: cinder extend <volume> <new_size>
error: too few arguments
Try 'cinder help extend' for more information.
cinder extend 3
usage: cinder extend <volume> <new_size>
error: the following arguments are required: <new_size>
Try 'cinder help extend' for more information.
```
This could lead to the partly failure of functional testcase,
fix it.
Change-Id: I59e9ae149af0b4294b09a94a52a4bc86a1d90f2b
Closes-Bug: #1697428
|
| |/
|
|
|
|
|
| |
Use 'six.text_type' to wrap defined string value.
Change-Id: I229d58595494f59f03538be79de42f2e8007f442
Closes-Bug: #1697401
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: Icec43c572e43eccc0408667877329bedf0f2fc1a
|
| |/
|
|
|
|
|
|
|
| |
There are some PEP8 errors that are not caught by our gate because of
the pep8 package version installed by hacking.
This patch fixes E402, W503, and F999 erors from our codebase.
Change-Id: I0b86730b1493f161645a87c3daa91ec3d774d3b1
|
| |
|
|
|
|
|
| |
LVM driver doesn't support mulltiattach now. That's why this test fails
on gates. We have to disable it until multiattach will be supported.
Change-Id: I690982dfa81a566d1632ebe81c54f5596316b02f
|
| |\ |
|
| | |
| |
| |
| | |
Change-Id: Icae58a69d0accc0308d581951a26970ac3872a49
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| | |
assertTableStruct is the same like in ClientTestBase, so we don't need
to duplicate the code.
Change-Id: I96c3bc450223d5f3f6f84c6313a0205cfb247514
|
| |/
|
|
|
|
| |
For example: assertTrue(v1<v2) is equal to assertLess(v1,v2)
Change-Id: Ic4a0e80eab167079631ac7dc054ad4d2a8a02fab
|
| |
|
|
|
|
|
|
| |
Positive tests for the cinder CLI commands which check actions with
volume create command like create volume from snapshot,
create volume from volume.
Change-Id: I77912d413ac061eb8376233dfef772c55265d135
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
Negative tests for the cinder CLI commands which check actions with
volume extend command like extend volume without arguments or with
incorrect arguments and check that correct error message raised.
Change-Id: Ic7c846fa2d8e7c867ed575d922b4a7e935120705
|
| |\ \ |
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Cinder support both v2 and v3 auth. Use v3 if possible.
In consideration of backwards compatibility, when we have
OS_PROJECT_NAME, the v2 auth should be ok because
tenant_name can be set by env[OS_PROJECT_NAME].
Change-Id: I9eed9c41a9deb5ecafa8d9e12f6d1b50d34f986d
Closes-Bug: #1588261
|
| |/
|
|
|
|
|
|
|
| |
Negative tests for the cinder CLI commands which check actions with
volume create command like create volume without arguments or with
incorrect arguments and check that correct error message raised.
Partial-Bug: #1602592
Change-Id: Ic51842aeb50758171751ecb9bf162add187f963e
|
| |
|
|
|
|
|
|
|
|
| |
Since 10.0.0 release tempest-ib is deprecated now.
Cinderclient should use tempest.lib in test code
and add tempest>=11.0.0 in requirements following
global-requirements.
Closes-Bug: #1583583
Change-Id: I9d766f46eb6fff29ae9c0267a60ff2ef4bf2343e
|
| |
|
|
| |
Change-Id: I8cbe98ac2adfc00a482917a5bb8615662986441a
|
| |
|
|
|
|
|
| |
Pass OS_* env variables for functional tests to use environment
variables if they are available instead of config file
Change-Id: Ic1b4752bf558c57974b5f2a4d5a45aee4a940bbb
|
| |
|
|
|
|
|
| |
This patch adds functional tests for cinder volume
backup creation and deletion for python-cinderclient.
Change-Id: I5aaa84c014d1147e9415392d52241ca4819589f6
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| | |
This patch allows to use a standard approach to volumes, snapshots
and backups to avoid duplicating code.
Change-Id: I0059fcb3bb8823105a7f960a30e70036bcbded8b
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
| |
Per:
http://lists.openstack.org/pipermail/openstack-dev/2015-August/072982.html
The location of subunit2html changed on the images in the gate
so update the path used in the post_test_hook.
Long-term we should just use what's in devstack-gate.
Change-Id: I5e50e7d7ad845aba26403df1df412c0a139a6dc7
Closes-Bug: #1491646
|
| |
|
|
|
|
|
| |
Add functional tests for cinder volume snapshot creation and deletion.
Change-Id: I60f3a880102461f107a7f2d06c0b97f0bbe0c2e4
Partial-Implements: blueprint cinderclient-functional-tests
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add tests for cinder volume creation and deletion
by ID and by Name.
Add test for cinder show volume output.
Add method to wait for given volume status.
Add method to check that volume deleted.
Add method to check that volume is not deleted (for negative cases).
Add method for volume deletion.
Add method to get Volume properties from CLI output.
Change-Id: I1e1f9bea2afc63e408a88e64fc1b6b6ee9f4e79b
Implements: partial blueprint cinderclient-functional-tests
Co-Authored-by: Kyrylo Romanenko <kromanenko@mirantis.com>
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add test for cinder extra-specs-list command output.
Add test for cinder list-extension command output.
Fix pep8 issues "E128 continuation line under-indented for visual
indent" in other functions.
Change-Id: I28266af7a6ff23b83899c50cc230f926a44592dc
Partial-Bug: #1463404
|
| |/
|
|
|
|
|
| |
1) Set correct $CINDERCLIENT_DIR to make Jenkins happy
2) Set OS_VOLUME_API_VERSION=2 to run with Cinder API v2 on gates
Change-Id: Id6be541fa2862eb833ed645e42bedcac6cef19dd
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Parser.listing method return empty list if env doesn't have resources,
so assertTableStruct works wrong, doesn't check field names in the
output table.
This change add new assert(assertTableHeaders) to check resource
attributes.
Change-Id: Ieca67b5cc2202467527c5a2dfbd7b2f2b4d5ad8d
Closes-Bug: #1458932
|
| |/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With the recent update to tox 2.0.x, environment variables such as
OS_AUTH_URL are not passed by default, resulting in tests errors
mentionning Keystone authentication failures.
This patch reads credentials from the 'functional_creds.conf' config
file, like it is done in novaclient (and soon in glanceclient and
neutronclient).
Reading credentials the old way (the environment) is still possible.
Change-Id: I2ec1df481aba7a3866fc8dbc912311de02c22d11
Related-Bug: #1455102
|
| |
|
|
|
|
|
| |
This is an initial add of a simple post_test_hook for the
functional tests that are run in cinderclient.
Change-Id: If0b242c4caaa611da8fb55b8f04ab436c0ce9984
|
|
|
First pass at functional tests in cinderclient. This patch
just takes some of the basic CLI tests from tempest and moves
them into the cinderclient functional tests.
These are read-only tests, and just do simple field checks
on the tables generated by the basic list commands.
Note: to run use "tox -efunctional", this requires that you
have a full cinderclient env and credentials are set. You may
be either running this locally (say in a devstack env) or
you may have a remote cloud handy that you can just source the
credentials for and run it that way.
Change-Id: I2f09a63be265d6a74cb103d80579068b9ab66bf4
|