summaryrefslogtreecommitdiff
path: root/tempest/cmd
Commit message (Collapse)AuthorAgeFilesLines
* tempest cleanup - don't initialize admin id'sBas de Bruijne2023-01-251-26/+0
| | | | | | | | | | Tempest cleanup gathers the project, user, and role id. Since commit a857880423, this info is collected but not used. In specific cases this method raises an error so it's better to remove it. Closes-bug: #2003125 Change-Id: Icd71a2962808628edb611257ac544f0277cde1f3
* Remove compute api_extensions config optionGhanshyam Mann2022-05-051-3/+1
| | | | | | | | | | | | | Nova has remvoed the API extensions concept since stein release - https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-stein Nova cannot be configured to enable/disable extension now and all extensions are enabled by default. Stein is not supported by the Tempest master so it is safe to remove the config optionn as well as the checks from tests. Change-Id: I05c0a3b23d0824e1eb9a79353dcf2976c6bcf602
* Merge "Use LOG.warning instead of deprecated LOG.warn"Zuul2022-03-021-1/+1
|\
| * Use LOG.warning instead of deprecated LOG.warnTakashi Kajinami2022-01-191-1/+1
| | | | | | | | | | | | | | | | | | The LOG.warn method is deprecated[1] and the LOG.warning method should be used instead. [1] https://docs.python.org/3/library/logging.html#logging.warning Change-Id: I43e2269d4b1c425119c86919eb869941d621c6de
* | Set default value of --concurrency to 2lpiwowar2022-01-141-6/+6
|/ | | | | | | | | | | | | There is a non-negligible number of tests in tempest that need accounts generated by account-generator with a concurrency of at least 2. Running these tests with accounts generated with the current default value for concurrency makes them therefore fail. This change sets the default value for concurrency to 2 and updates the documentation for --concurrency parameter. Closes-Bug: #1948935 Change-Id: Iada980beff9db3c8cd02c84d0d7602264b66ddf8
* Merge "Update subunit_describe_calls about new ports"Zuul2021-03-041-10/+14
|\
| * Update subunit_describe_calls about new portsMasayuki Igawa2021-03-031-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit updates the default version explanation in subunit_describe_calls. It seems the 'Firewalls and default ports' is moved to a new location. In Newton it was here [0] but from Ocata it gives 404 [1] because it got moved to [2]. And this commit also fixes some markups as inline literals. [0] https://docs.openstack.org/newton/config-reference/firewalls-default-ports.html [1] https://docs.openstack.org/ocata/config-reference/ [2] https://docs.openstack.org/install-guide/firewalls-default-ports.html Change-Id: I82bb3c5bbeaa4de04f607266311e8452e25ce6b5
* | Merge "Remove deprecated config options for volume API selection"Zuul2021-02-231-14/+0
|\ \
| * | Remove deprecated config options for volume API selectionGhanshyam Mann2020-12-311-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Volume APIs tests have been moved to v3 as default and can be run under v2 API via catalog_type. - https://review.opendev.org/#/q/topic:volume-default-testing-v3+(status:open+OR+status:merged) Below config options were deprecated since Rocky and now time to remove them: * ``CONF.volume-feature-enabled.api_v2`` * ``CONF.volume-feature-enabled.api_v3`` Change-Id: I31f487694e5f5e586f99323c81e45f43306606fb
* | | Merge "Remove usage of six"Zuul2021-02-192-3/+4
|\ \ \
| * | | Remove usage of sixsongwenping2021-02-182-3/+4
| |/ / | | | | | | | | | | | | | | | | | | Remove six-library Replace the following items with Python 3 style code. - six.moves Change-Id: Ied212eb0947c7441cc51193399185b4fb0f87271
* | | Inclusive jargonMartin Kopec2021-01-201-21/+95
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following stestr's example where arguments such as --blacklist-file, --black-regex and --whitelist-file are deprecated since its 3.1.0 release, let's do the change here as well in order to get tempest consumers some time for the transition. This change deprecates the following arguments and replaces them by new ones which are functionally equivavelnt: * --black-regex is replaced by --exclude-regex * --blacklist-file is replaced by --exclude-list * --whitelist-file is replaced by --include-list For now, Tempest will accept both (new and old) arguments to make the transition smoother for all consumers. The patch also bumps min version of tox to 3.18.0 in order to replace tox's whitelist_externals by allowlist_externals option: https://github.com/tox-dev/tox/blob/master/docs/changelog.rst#v3180-2020-07-23 Change-Id: I3e09b31f63d2cd7ea41c48e62432bd3bc54fcf44
* | Merge "Remove six.PY3/six.PY2"Zuul2020-11-271-6/+1
|\ \
| * | Remove six.PY3/six.PY2likui2020-11-101-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | The Python 2.7 Support has been removed by: https://review.opendev.org/#/c/704096/ So remove hacking rules for compatibility between python 2 and 3 Change-Id: I0e78b982b79674520c7695c0cca7352677d7b7a0
* | | [Trivial]Remove unused variableszhufl2020-11-201-1/+1
|/ / | | | | | | | | | | This is to remove unused variables and to use merging-isinstance. Change-Id: I9d7121678a8b18a03b7b872526bfc74351560776
* | Use os.path.join as possibleMasayuki Igawa2020-09-061-3/+6
| | | | | | | | | | | | | | | | | | | | This commit makes to use os.path.join instead of using the Unix path character '/'. This change is not mandatory since we don't support Windows environments as Tempest execution and, it seems Windows supports '/' as a path separator. However, using both os.path.join and '/' is a bit awkward. Change-Id: I117acb281c352179d526808009e761335bb314fc
* | Improve unit tests for subunit_describe_callsDoug Schveninger2020-08-181-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in order to enhance the cmd to support kubernetes url based routing over port based routing. Once this is merged I will submit another patch set to subunit_describe_calls to support -u --urls (Optional) The path to a JSON file describing the urls being used by different services. Note Can not be used with -p. Added test cases for cliff subprocess and found a bug with the -v. The standard -v --verbose of cliff conflict with this command so I changed the -v --verbose to -a --all for print header with printing to stdout I am also getting ready to add test cases for cliff subprocess support for different options. Bug #1890060 Correct os join in test cases to avoid a conflict with https://review.opendev.org/#/c/683026 Closes-bug: #1890060 Change-Id: I9459db0dbeda721187ea5f4802c7453c2092dac3
* | Merge "Fix the argument check for account-generator"Zuul2020-06-231-1/+10
|\ \
| * | Fix the argument check for account-generatorLukas Piwowarski2020-06-151-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Concurrency parameter for account-generator command was accepting negative values and zero. This patch makes sure that when the account-generator is passed a negative value or zero, it ends with an error. Closes-Bug: #1811349 Partially-Implements: blueprint tempest-cli-unit-test-coverage Change-Id: I4d7de89b2ad3ee91d113da3746fe393d8cce2aa2
* | | Merge "Don't use deprecated configparser.readfp"Zuul2020-06-091-4/+4
|\ \ \ | |/ / |/| |
| * | Don't use deprecated configparser.readfpAndreas Jaeger2020-05-051-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Running verify_tempest_config, it complains with: /opt/stack/tempest/tempest/cmd/verify_tempest_config.py:444: DeprecationWarning: This method will be removed in future versions. Use 'parser.read_file()' instead. Use read_file. Also, remove one extra use of six in this file. Change-Id: Ia32df8e2be29bb0951efd08db9eed9e67d749e1f
* | | Improve logging in tempest cleanupMartin Kopec2020-05-072-7/+52
|/ / | | | | | | | | | | | | | | | | | | | | | | | | * add additional handler so that INFO messages are printed to stdout besided tempest.log * replace print commands by LOG.info calls * when --debug argument used print DEBUG messages to stdout * add additional debug messages to improve transparency of the cleanup execution * use --debug argument within tempest-cleanup ansible role to see more details directly in the output Change-Id: Iee72dc411fdf5cda8a8eed8dd7f349e0866648ab
* | Merge "Remove the deprecated tempest-account-generator CLI"Zuul2020-04-301-40/+14
|\ \
| * | Remove the deprecated tempest-account-generator CLISoniya Vyas2020-03-021-40/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tempest CLI tempest-account-generator was deprecated 4 years ago. devstack still uses the deprecated CLI for which patch has been proposed. As deprecation is done long time back, 'tempest-account-generator' should be now used as 'tempest account-generator' The following commit gives the idea of deprecation message:- https://opendev.org/openstack/tempest/commit/68b8b9d26fae5c07fa7c4c443e4b0ca51ad98258 Depends-on: I624e1dc57a3d3533322fb298c01f70241d0400ed Signed-off by: Soniya Vyas<svyas@redhat.com> Change-Id: I7c18998dd877d7fd065e4d1d0950b6f6413587c3
* | | Merge "Add --worker-file option in tempest"Zuul2020-04-171-0/+41
|\ \ \
| * | | Add --worker-file option in tempestArx Cruz2020-03-301-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch add the option --worker-file on tempest run command. This will enable users to specify a worker file that will be used by stestr to manually schedule how the tests will run. Change-Id: I747f3abe179492da063fcaaf1123ffcf6362f966
* | | | Merge "Fix deletion of network quotas"Zuul2020-04-131-1/+2
|\ \ \ \ | |/ / / |/| | |
| * | | Fix deletion of network quotasLukas Piwowarski2020-03-261-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Network quotas could be deleted after deletion of project. This caused the cleanup to fail as network quotas have to be deleted before the deletion of project. This fix makes sure that network quotas are deleted before the deletion of project. Change-Id: I8576e2c6f1ae6dc90850dd3d8eade7c10942e413
* | | | Merge "Cleanup images properly"Zuul2020-03-191-1/+10
|\ \ \ \ | |/ / / |/| | |
| * | | Cleanup images properlyLukas Piwowarski2020-03-121-1/+10
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Images weren't deleted properly due to bad request sent to glance API. Also ImageService ignored 'next' parameter returned from glance API when listing images which is required when more images are present. The fix lists all images properly. If 'next' parameter is present in the glance API response then the API is asked to send next part of image list. Closes-Bug: 1866988 Change-Id: I9ebeccb545576e43ef0fc96c29169ae16f261f34
* | | Delete network quotasLukas Piwowarski2020-02-181-0/+20
|/ / | | | | | | | | | | | | NetworkQuotaService class enables deletion of network quotas which were neglected by tempest cleanup before. Change-Id: I83ac30960c0be39fc5147d445cb39cb3ff4c56b2
* | Merge "Remove the deprecated verify-tempest-config CLI"Zuul2020-02-171-9/+0
|\ \
| * | Remove the deprecated verify-tempest-config CLISoniya Vyas2019-12-261-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tempest CLI verify-tempest-config was deprecated 4 years ago. With assumption of all the users of this CLI have moved to new CLI 'tempest verify_config', it is time to remove the old deprecated CLI now. 'verify_tempest_config' should be now used as 'tempest verify_config' The following commit gives the idea of deprecation message:- https://opendev.org/openstack/tempest/commit/e45aa842bc823b56560af247eb5e26b1fd686151 Signed-off by: Soniya Vyas<svyas@redhat.com> Change-Id: Ic3dbd6f5e854e017a5bec4b1a625730ab68e767d
* | | Merge "Removal of deprecated argument[os-tenant-name]"Zuul2020-02-171-13/+1
|\ \ \
| * | | Removal of deprecated argument[os-tenant-name]Soniya Vyas2020-01-281-13/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use of os-tenant-name or OS_TENANT_NAME is deprecated. Instead of it, os-project-name is expected to be used. This patch depends on below mentioned patch, since devstack still uses the deprecated argument. Depends-on: I624e1dc57a3d3533322fb298c01f70241d0400ed Signed-off by: Soniya Vyas<svyas@redhat.com> Change-Id: I3b926f8abb6fce3d2eab93310ce3959c68aa9a6d
* | | Tempest cleanup: improve iterating over projectsMartin Kopec2019-12-202-22/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently cleanup iterates over projects and deletes all resources tied to a particular project. However, most of the resources can be deleted all at once without iteration over the projects. This will fix the mentioned bug and also make the tool much more efficient as we'll avoid many iterations and queries. Closes-bug: #1830943 Change-Id: I82c7899a32bee0e714e71d703195085e10ea224f
* | | Extend cleanup CLI to delete regionsLukas Piwowarski2019-12-201-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | RegionService class enables deletion of regions which were neglected by tempest cleanup before. Change-Id: I2c978a7b2151b37bb0a224bafd1be490f04950ee Closes-Bug: #1848672
* | | Merge "Added tempest subcommand for subunit_describe_calls"Zuul2019-12-121-26/+63
|\ \ \
| * | | Added tempest subcommand for subunit_describe_callsSoniya Vyas2019-11-221-26/+63
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are adding a subcommand to tempest that is a parser for subunit streams to determine what REST API calls are made inside of a test and in what order they are called. Right now the implementation of utility is as a console script and which is in argparse. We are porting it to cliff to give a similar experience across the codebase. Change-Id: If95042898d8b2a148246c991d73d1a577fbcf7a0 Signed-off-by: Soniya Vyas <svyas@redhat.com>
* | | tempest cleanup - use admin_mgr onlyMartin Kopec2019-10-041-58/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid using temporary managers and use only the admin one. The tool has been using the admin_mgr for listing projects, roles, for listing all resources during initializing a saved state so why don't use it also for deleting all of the resources? This wil solve the Unauthorized issue happening when the tool was requesting credentials for the project from a different domain. Closes-bug: #1766582 Change-Id: Ibb5599ce48712b94dbf591e4a30cf3906cb5cdde
* | Merge "Remove python shebangs from python modules"Zuul2019-08-133-6/+0
|\ \
| * | Remove python shebangs from python modulesMasayuki Igawa2019-08-083-6/+0
| | | | | | | | | | | | | | | | | | | | | This commit removes python shebangs from the python modules. They're just python modules, so we don't need to have python shebangs for them. Change-Id: I6f4428cf20be6f5ef2142030b4ede235444a4336
* | | tempest cleanup fix return codeMartin Kopec2019-08-082-2/+18
|/ / | | | | | | | | | | | | | | | | | | | | | | It's a follow up fix for tempest cleanup NotImplemented error issue. Tempest cleanup shouldn't return a non-zero return code for NotImplemented errors as they are actually not errors, they are just an outcome of tempest not checking availability of extensions in order to keep it simple without any complicated logic. Closes-bug: #1832566 Change-Id: I09e28c245c11dd3dd061df786bd137eceb97be07
* | tempest cleanup should use keystone v3 clientMartin Kopec2019-07-131-1/+1
| | | | | | | | | | | | | | | | | | | | tempest cleanup returns an error when it's initializing a saved state, because it uses already deprecated v2 client. The patch makes tempest cleanup use keystone role v3 client. Closes-Bug: #1788561 Change-Id: Id607dec012ce48af6d4c7ee263a90c3ea2e1799c
* | Fix tempest run doc indentationMasayuki Igawa2019-07-051-5/+5
| | | | | | | | | | | | | | | | | | | | This commit fixes the tempest run documentation indentations for more appropriate rendering[0]. Currently, it's rendered as a quote which is weird. [0] https://docs.openstack.org/tempest/latest/run.html#test-selection Change-Id: I764cfff7dd07214fa44dacfa2a86476471e8fb20
* | Fix cleanup NotImplemented errorMartin Kopec2019-06-182-9/+28
| | | | | | | | | | | | | | | | | | | | | | tempest cleanup doesn't check if the APIs it uses are implemented or not. Therefore the patch adds a try except block preventing tempest cleanup to fail when some of the APIs are not implemented. Closes-bug: #1832566 Change-Id: I9ced4af40eb0c2a22e3557caded56045a397d539
* | Improve router deletion logging in tempest cleanupMartin Kopec2019-04-021-5/+9
| | | | | | | | | | | | | | | | | | | | | | Improve logging in tempest cleanup when a router and its interfaces are being deleted. The review adds a try except block for every attempt to remove a port. By that addition any port error will not be hidden behind the router error the port is attached to. It will provide more precise error logging. Change-Id: I475deec7b29600627f68ff07c5546e55cdab100f
* | Improve logging in tempest cleanupMartin Kopec2019-04-011-24/+30
| | | | | | | | | | | | | | Add id or name of the resource which failed to delete to the logging. Change-Id: I04b04406e3917a63332199adf5c15b8e4ed6ca5a
* | Checking config file actually exist or not in tempest runManik Bindlish2019-03-181-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | This PS will check config file passed from command line is actually existing or not in tempest run command. If file exists, only then it will go for set config path otherwise will fail with value error. Also adding and modifying unit test cases for this change. Partially-Implements: blueprint tempest-cli-unit-test-coverage Change-Id: I09d756be69cb3a9be8d0638c41d45a089d62b238 Closes-Bug: #1808473
* | Merge "Add autopep8 to tox.ini"Zuul2019-03-162-4/+4
|\ \
View Lorry Controller Status