summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "[stable-only] Add missing classes to cinderclient.v3" into stable/wallabywallaby-em7.4.1stable/wallabyZuul2021-08-313-0/+54
|\
| * [stable-only] Add missing classes to cinderclient.v3Brian Rosmaita2021-08-063-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | The QuotaSet class was never added to cinderclient.v3.quotas, and the VolumeTransfer class was never added to cinderclient.v3. volume_transfers. Note: this fix is proposed directly to the stable branches, as the issue was fixed differently in master (Xena development) when the cinderclient.v2 classes were removed by change I335db5c1799e. Change-Id: Ie00f068d7818e20683a0c242b51027d657382dd4 Closes-bug: #1938451
* | Merge "Unset tempest.lib timeout in functional tests" into stable/wallabyZuul2021-08-311-0/+5
|\ \ | |/
| * Unset tempest.lib timeout in functional testsBrian Rosmaita2021-08-061-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test_cli.CinderBackupTests.test_backup_create_and_delete test is hitting timeout errors in the python-cinderclient-functional-py36 zuul job. This is happening because it's inheriting the OS_TEST_TIMEOUT value of 60 from the base testenv, and that value is being used by the tempest.lib class we inherit from as a timeout for each test. This is a problem for test_backup_create_and_delete because it creates a volume, waits for available, creates a backup, waits for available, deletes the volume, waits for deletion, deletes the backup, waits for deletion. Our functional tests have their own timeout handling, so turn off the tempest.lib timeout and use ours. An alternative to turning it off is to set it at a value that respects our timeout for our longest test, which would be: - time-to-available: 120 sec (x2) - time-to-deleted: 60 sec (x2) that is, 360 sec. Change-Id: I33399b4c094af2cc059da6e332f4c0a91e6ab57e (cherry picked from commit d04ded6a6ff4cc9ba8377a1b025493e939047b2c)
* | Make instance_uuid optional in attachment createRajat Dhasmana2021-07-296-3/+79
|/ | | | | | | | | | | | Cinder and cinderclient assumes an attachment create request will always contain instance_uuid. This is not true when glance calls cinder for attachment in glance cinder configuration. This patch (along with the cinder patch) make the instance_uuid optional and allow glance to do attachments without passing instance_uuid. Change-Id: Ifbaca4aa87d890bc5130069638d42665b914b378 (cherry picked from commit d714249ca9e95297f5e53c9dcc0af9ba58d0ab6d)
* Remove skip_missing_interpretersBrian Rosmaita2021-07-073-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | This prevents a job reporting 'success' when the appropriate python interpreter cannot be found, when actually it didn't run at all. Also change the default envlist to use generic 'py3' instead of a specific version which might not be present. Also change zuul config so the python-cinderclient-functional-py36 job runs on centos-8-stream nodes, where py36 should be available. And change bindep.txt to specify the correct package name for centos-8. Jeremy Stanley has given a more thorough explanation of why this is a good change: http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014810.html This isn't a theoretical issue. If you look at recent python-cinderclient-functional-py36 job results (for example, [0]), you'll see that Zuul reported 'success', but on a closer look, you'll see that no tests were run. [0] https://zuul.opendev.org/t/openstack/build/1bfc80638086405f8b29905cdd6f71be/log/job-output.txt#25470 Change-Id: I2e2aa24e1592b66b287c84eda97b5079c40a36ec (cherry picked from commit b891c9980f316bd603a9f1429eebad41adf43825)
* Use default nodeset for functional jobsGhanshyam Mann2021-05-071-2/+0
| | | | | | | | | | | | | | | | | | | | | Specifying specific nodes for these jobs was a holdover from ussuri. For wallaby and victoria, py36 and py38 should be available on all nodes, so we can just use the default. original commit message: Run functional job on Ubuntu Focal Devstack is planning to remove the Ubuntu Bionic support. - https://review.opendev.org/c/openstack/devstack/+/788754 Before that we need to switch Bionic job to focal. devstack-tox-functional define the latest nodeset which is single node focal currently so removing nodeset setting from python-cinderclient functional jobs. Change-Id: Ibbbfb20c86bb2ea0d3d74f6a6a1bc913874f67ad (cherry picked from commit f54b873ca3f9900e17b42f3600a20a36abe2b1a7)
* Drop lower-constraints jobBrian Rosmaita2021-04-091-1/+0
| | | | | | | | | | Running a lower-constraints job is optional in master, and seems pointless in the stable branches, so drop it. Additional information here: http://lists.openstack.org/pipermail/openstack-discuss/2021-March/021204.html Change-Id: I2872eee0bd43dcef75a1508d7f707823fb6eccfa
* Update TOX_CONSTRAINTS_FILE for stable/wallabyOpenStack Release Bot2021-03-151-4/+4
| | | | | | | | | | | | Update the URL to the upper-constraints file to point to the redirect rule on releases.openstack.org so that anyone working on this branch will switch to the correct upper-constraints list automatically when the requirements repository branches. Until the requirements repository has as stable/wallaby branch, tests will continue to use the upper-constraints list on master. Change-Id: I589decfa42329da00789cb2021e0a62c911a80ee
* Update .gitreview for stable/wallabyOpenStack Release Bot2021-03-151-0/+1
| | | | Change-Id: Iaffb8d63a7ea933faf92fed87247bd9cf96fdad1
* Merge "Add note for Wallaby release"7.4.0Zuul2021-03-121-0/+16
|\
| * Add note for Wallaby releaseBrian Rosmaita2021-03-121-0/+16
| | | | | | | | Change-Id: Ia958840739cab51236391e4d8d6d1569604d9a3e
* | Update requirements for wallaby releaseBrian Rosmaita2021-03-114-70/+18
| | | | | | | | | | | | | | Updates requirements to what we're using now and revises the lower-constraints file to contain only direct dependencies. Change-Id: I1a3cc999b94dc6b6482b1ca44218b42ee2638a8c
* | Merge "Add flake8-import-order extension"Zuul2021-03-1122-52/+44
|\ \
| * | Add flake8-import-order extensionSean McGinnis2021-03-1022-52/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the import order extension to match what we have in the cinder repo. This is a linting extension that will check that imports are in the correct order and the correct grouping so they automatically get flagged, and it won't be whether reviewers notice and decide to do anything or not. Cinder change was Ic13ba238a4a45c6219f4de131cfe0366219d722f for a little more wordy reasoning. Also includes updates for noqa tags. Newer version of the linters appear to want these on the function definition line, not on the decorator line. Change-Id: Ibf3f3afbf3bb6ec6613b35f91d4a353c6a391f41 Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
* | | Dropping explicit unicode literaltushargite962021-03-113-19/+19
| |/ |/| | | | | | | | | | | | | | | In python 3, all strings are considered as unicode string. This patch drops the explicit unicode literal (u'...') or (u"..") appearances from the unicode strings. Change-Id: I9902966892a1dc4f85d449dfe580fb128647487b
* | Merge "Use TOX_CONSTRAINTS_FILE"Zuul2021-03-111-4/+4
|\ \
| * | Use TOX_CONSTRAINTS_FILElikui2021-03-101-4/+4
| |/ | | | | | | | | | | | | | | | | | | | | UPPER_CONSTRAINTS_FILE is old name and deprecated -https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file This allows to use lower-constraints file as more readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>. [1] https://review.opendev.org/#/c/722814/ Change-Id: I26f4c8fcaaaf0eca5e5d6fcb6fd443b56ea41f35
* | Merge "Bump API max version to 3.64"Zuul2021-03-101-1/+1
|\ \
| * | Bump API max version to 3.64Alan Bishop2021-03-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Bump MAX_VERSION to 3.64 to support including the encryption_key_id attribute in volume and backup details. Implements: blueprint include-encryption-key-id-in-details Depends-On: I16f54e6722cdbcbad4af1eb0d30264b0039412fd Change-Id: I6e1f3ff62d4b7b9b8299f7bd73071c3c7856f6df
* | | Merge "Doc: Functional Tests in python-cinderclient"Zuul2021-03-101-2/+2
|\ \ \ | |_|/ |/| |
| * | Doc: Functional Tests in python-cinderclienttushargite962021-02-231-2/+2
| | | | | | | | | | | | | | | | | | | | | ostestr command is deprecated. Closes-bug: #1866376 Change-Id: I24398d481002f30754368ac601ff3ac304f3fb82
* | | Merge "Changed minversion in tox to 3.18.0"Zuul2021-03-101-4/+4
|\ \ \
| * | | Changed minversion in tox to 3.18.0tushargite962021-02-251-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The patch 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: Ia637e459b9401ab3f07fdf6e88868080ce59e245
* | | | Merge "Remove install unnecessary packages"Zuul2021-03-101-2/+0
|\ \ \ \
| * | | | Remove install unnecessary packageswu.shiming2021-02-191-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The docs requirements migrated to doc/requirements.txt we need not install things from requirements.txt. Change-Id: I94c2ba1ddfc3a54edc2f69b8d7017e09b02e54a5
* | | | | Merge "Remove more python2 compat code"Zuul2021-03-102-15/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Remove more python2 compat codeEric Harney2021-03-082-15/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove code still adjusting for Python 2 behavior based on version checks. Change-Id: I29576a824278611d80991dce2501f14f1b262c76
* | | | | Merge "Add MV 3.63 to the max supported version"Zuul2021-03-101-1/+1
|\ \ \ \ \
| * | | | | Add MV 3.63 to the max supported versionRafael Weingärtner2021-01-271-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During the implementation on [1], it was requested to extend the Cinder client to support the newly created microversion (MV). Therefore, this patch is doing exactly that. We do not need to do any other work forward bcause the list volumes will display whatever we have in the response. I mean, the default is ``['ID', 'Status', 'Name', 'Size', 'Volume Type', 'Bootable', 'Attached to']``, but one can customize these fields using ``--fields`` [1] https://review.opendev.org/c/openstack/cinder/+/666886 Closes-Bug: https://bugs.launchpad.net/python-cinderclient/+bug/1913474 Depends-on: https://review.opendev.org/c/openstack/cinder/+/666886 Change-Id: I0815570a50e9b38fe18733c727acd52a406bfc1e
* | | | | Merge "Remove all usage of six library"Zuul2021-03-0832-164/+83
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Remove all usage of six libraryhaixin2021-03-0432-164/+83
| |/ / / | | | | | | | | | | | | | | | | | | | | Replace six with Python 3 style code. Change-Id: I4b97e040f3e790ac114dcd43c68e6b67b1079adf
* | | | Merge "Move cinderclient to new hacking 4.0.0"Zuul2021-03-041-1/+1
|\ \ \ \
| * | | | Move cinderclient to new hacking 4.0.0tushargite962021-02-231-1/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: Id3e61949550e70e2c4235406d258bbd9c66e67cf
* | | | | Merge "Bump pylint to 2.6.0"Zuul2021-03-032-2/+2
|\ \ \ \ \
| * | | | | Bump pylint to 2.6.0Luigi Toscano2021-02-192-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current version does not work anymore with the current testing stack. Due to the bump, the wrapper needed a few changes (maybe it could be dropped at some point? Version 2.3.0 (used by cinder right now) was considered too, but it requires a specific version of isort: https://github.com/PyCQA/isort/issues/1273 Change-Id: I07fa32e7f49bde041d101a2d09860d0bc27acda0
* | | | | Merge "Stop configuring install_command in tox."Zuul2021-02-271-1/+0
|\ \ \ \ \
| * | | | | Stop configuring install_command in tox.tushargite962021-02-031-1/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It turns out that this is the the default value provided by tox: https://tox.readthedocs.io/en/latest/config.html#conf-install_command So we can remove the line and simply use the default value. Change-Id: I7a703e0ee91b73b7ab736712756caaba6981f8e4
* | | | | Uncap PrettyTableWander Way2021-02-181-1/+1
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | This is now maintained as a Jazzband project [1]. [1] https://github.com/jazzband/prettytable Change-Id: I71ace4c2857b9e12e9023aadaf72bbe97c2fda1c
* | | | Support backup-restore to a specific volume type or AZAlan Bishop2021-01-113-0/+232
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enhance the 'backup-restore' shell command to support restoring a backup to a newly created volume of a specific volume type and/or in a different AZ. New '--volume-type' and '--availability-zone' arguments leverage the existing cinder API's ability to create a volume from a backup, which was added in microversion v3.47. The shell code is a new v3 implementation, and it drops support for the v2 command's deprecated '--volume-id' argument. Change-Id: Ic6645d3b973f8487903c5f57e936ba3b4b3bf005
* | | Update requirements and lower-constraints7.3.0Alan Bishop2020-12-233-61/+74
| |/ |/| | | | | | | | | | | | | | | | | | | Sync the versions with cinder's, which were recently updated by I42af21b1c4247d04d479f1fc1ecd6f9baac0cfc9. Also increased the minimum version of tempest to the most recent release. Also added indirect dependencies to test-requirements.txt in order to limit the number of versions considered by the resolver. Change-Id: I7b4bc7b392b2192e0c832c4f0148546a5920b9e2
* | Fix undesirable raw Python errorEduardo Santos2020-10-293-0/+22
|/ | | | | | | | | | | | | | Using the cinderclient without a subcommand while passing an optional argument triggers the raw Python error `ERROR: 'Namespace' object has no attribute 'func'`. This bug can be reproduced by issuing the command `cinder --os-volume-api-version 3.40`. Added a default value to `func` and an empty value to `command` as placeholders so that a help message is shown instead of the Python error. Change-Id: Idb51e8635b97f0da2976f3268d5e19100ec77203 Closes-Bug: #1867061
* Merge "Add Python3 wallaby unit tests"Zuul2020-10-221-1/+1
|\
| * Add Python3 wallaby unit testsOpenStack Release Bot2020-09-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | This is an automatically generated patch to ensure unit testing is in place for all the of the tested runtimes for wallaby. See also the PTI in governance [1]. [1]: https://governance.openstack.org/tc/reference/project-testing-interface.html Change-Id: I4aed8f56609dea21a8170bd5f5d242cfac045dde
* | Merge "doc: Update Py37 instead of py27"Zuul2020-10-151-3/+3
|\ \ | |/ |/|
| * doc: Update Py37 instead of py27tushargite962020-09-171-3/+3
| | | | | | | | | | | | | | Examples in this section should use a Python 3 environment and not py27. Change-Id: If082b92e089af980e411b4b4c1319e462862a55f Closes-Bug: #1866375
* | Update master for stable/victoriaOpenStack Release Bot2020-09-162-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | Add file to the reno documentation build to show release notes for stable/victoria. Use pbr instruction to increment the minor version number automatically so that master versions are higher than the versions on stable/victoria. Change-Id: Ia36030fb2858ddf4c6672173cde510dd8f8b7a53 Sem-Ver: feature
* | Merge "Add functional-py38 job"7.2.0Zuul2020-09-162-19/+10
|\ \
| * | Add functional-py38 jobBrian Rosmaita2020-09-162-19/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | The openstack-python3-victoria-jobs template is running unit tests for py36 and py38. We should do the same for our functional test jobs. So replace the functional-py37 job with a functional-py38 job. Change-Id: Icb338611169975be04bb27b86b5b3de0a37a6f5f
* | | Merge "Add note for Victoria release"Zuul2020-09-161-0/+11
|\ \ \
View Lorry Controller Status