summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge "update some scripts"Zuul2021-09-012-335/+1
|\ \
| * | update some scriptsdengzhaosen2021-09-012-335/+1
| |/ | | | | | | | | | | | | remove the the colorizer.py which hasn't been used since https://review.opendev.org/c/openstack/python-cinderclient/+/502120 Change-Id: I9bed2ec947705cd53daa04678093a80581dc3282
* | Merge "Add functional jobs to the gate"Zuul2021-08-301-0/+9
|\ \
| * | Add functional jobs to the gateBrian Rosmaita2021-08-241-0/+9
| | | | | | | | | | | | | | | | | | Voting check jobs are supposed to also be gate jobs. Change-Id: Ibdf4c30606552fa0c2660a24c97354f814e5be24
* | | Merge "Fix list resources when use limit and with-count"Zuul2021-08-301-0/+6
|\ \ \
| * | | Fix list resources when use limit and with-countzhu.boxiang2020-12-171-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we use with-count query param, we will get the count of resources. While the _list function return two different result, one is just list, another is tuple with list and int. So we must to handle the items which return from _list function. Closes-bug: #1908474 Change-Id: If8e31a637cf098cca60d8a10e9835ef66a3e66bc
* | | | Merge "Increase default quotas for zuul jobs"Zuul2021-08-301-0/+5
|\ \ \ \ | | |/ / | |/| |
| * | | Increase default quotas for zuul jobsBrian Rosmaita2021-08-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're seeing occasional failures in the functional test jobs due to exceeding volumes quota, so increase these from the default value of 10 to something more reasonable like 25. Change-Id: Id1ab94849bfb35256dd0705b888708f3fa1106c4
* | | | Merge "client: Stop logging request-id twice in DEBUG"Zuul2021-08-301-14/+0
|\ \ \ \
| * | | | client: Stop logging request-id twice in DEBUGLee Yarwood2021-03-101-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is already logged by http_log_resp in DEBUG along with the other response headers so drop the duplicate logging in _log_request_id. Change-Id: I3c5ad2b480d80611ecb10449068d836c4bbe7bdb
* | | | | Add consumes quota field supportGorka Eguileor2021-08-253-16/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder microversion v3.65 adds consumes_quota key to volume and snapshots as a way to differentiate between use generated resources and temporary ones. This patch adds support for this microversion and presents the consumes_quota field when the server sends it (which only happens when we request this microversion). Change-Id: I524490aa988fa4d654bfa8050d89cf99ce50bb4b Depends-On: I655a47fc75ddc11caf1defe984d9a66a9ad5a2e7 Implements: blueprint temp-resources
* | | | | Merge "Unset tempest.lib timeout in functional tests"Zuul2021-08-051-0/+5
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Unset tempest.lib timeout in functional testsBrian Rosmaita2021-07-291-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
* | | | | Merge "Dropping lower constraints testing"Zuul2021-08-043-25/+0
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Dropping lower constraints testingzhangboye2021-07-073-25/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We facing errors related to the new pip resolver, this topic was discussed on the ML and QA team proposed to to test lower-constraints [1]. I propose to drop this test because the complexity and recurring pain needed to maintain that now exceeds the benefits provided by this mechanismes. [1] http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019390.html Change-Id: I324cd145da8469f505ae2c135ee5035ee7008ca1
* | | | | Merge "Make instance_uuid optional in attachment create"8.0.0Zuul2021-07-206-3/+79
|\ \ \ \ \
| * | | | | Make instance_uuid optional in attachment createRajat Dhasmana2021-07-146-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
* | | | | | Merge "Remove v2 classes"Zuul2021-07-2072-2039/+1122
|\ \ \ \ \ \
| * | | | | | Remove v2 classesBrian Rosmaita2021-07-1472-2039/+1122
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove all cinderclient.v2 classes, mostly incorporating them into their v3 counterparts and updating the tests and test fixtures. Depends-on: https://review.opendev.org/c/openstack/horizon/+/800814 Change-Id: I335db5c1799edb2273bf8bfc9e1bc9de404a4ba5
* | | | | | | Merge "Remove v2 support from the shell"Zuul2021-07-2013-1454/+188
|\ \ \ \ \ \ \ | |/ / / / / / | | | / / / / | |_|/ / / / |/| | | | |
| * | | | | Remove v2 support from the shellBrian Rosmaita2021-07-1213-1454/+188
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also removes the v2 support from the generic client and restores a skipped test. Additionally, the cinderclient.tests.v2.test_availablity_zone module depends on the v2.shell class, so move that module to v3, update the v3 AvailablityZone class, and make appropriate adjustments to the tests and test fixtures. Change-Id: I7a3cca15f5944141d510a75af6684221c297963b
* | | | | | Remove skip_missing_interpretersBrian Rosmaita2021-07-013-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
* | | | | | Merge "Add Python3 xena unit tests"Zuul2021-06-181-1/+1
|\ \ \ \ \ \
| * | | | | | Add Python3 xena unit testsOpenStack Release Bot2021-03-151-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an automatically generated patch to ensure unit testing is in place for all the of the tested runtimes for xena. See also the PTI in governance [1]. [1]: https://governance.openstack.org/tc/reference/project-testing-interface.html Change-Id: I966cfcc1f1aa528385a57f2012bf2540e3d3edc4
* | | | | | Merge "setup.cfg: Replace dashes with underscores"Zuul2021-05-211-4/+4
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | setup.cfg: Replace dashes with underscoresYuehuiLei2021-04-281-4/+4
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Setuptools v54.1.0 introduces a warning that the use of dash-separated options in 'setup.cfg' will not be supported in a future version [1]. Get ahead of the issue by replacing the dashes with underscores. Without this, we see 'UserWarning' messages like the following on new enough versions of setuptools: UserWarning: Usage of dash-separated 'description-file' will not be supported in future versions. Please use the underscore name 'description_file' instead [1] https://github.com/pypa/setuptools/commit/a2e9ae4cb Change-Id: I77d52471fea255982ad677e9d4bc318f24a9ace7
* | | | | Run functional job on Ubuntu FocalGhanshyam Mann2021-04-291-2/+0
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "Support passing client certificates for server version requests"Zuul2021-04-206-15/+56
|\ \ \ \ | |/ / / |/| | |
| * | | Support passing client certificates for server version requestssri harsha mekala2021-03-096-15/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the cinderclient to fetch server versions will fail with error `OpenSSL.SSL.Error: [sslv3 alert handshake failure]` when the server requires client certificates to be passed with these requests. Added the optional parameter `cert` to both get_server_version get_highest_client_server_version and methods so that users can have the option to pass client certificates while fetching server versions. Also support passing mTLS certificate/key to HTTPClient Closes-Bug: #1915996 Change-Id: I57c665dd9d4b8c32e5f10994d891d1e0f5315548 Signed-off-by: sri harsha mekala <smekala@oath.com>
* | | | Update master for stable/wallabyOpenStack Release Bot2021-03-152-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add file to the reno documentation build to show release notes for stable/wallaby. Use pbr instruction to increment the minor version number automatically so that master versions are higher than the versions on stable/wallaby. Sem-Ver: feature Change-Id: I7a9c0cd51c45f05c75d3fa5be137aad7e62a016c
* | | | 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