summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update TOX_CONSTRAINTS_FILE for stable/2023.1stable/2023.1OpenStack Release Bot2023-02-241-1/+1
| | | | | | | | | | | | 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/2023.1 branch, tests will continue to use the upper-constraints list on master. Change-Id: I2b23745d88d6a84e18b60cb43e40d18c3abaada2
* Update .gitreview for stable/2023.1OpenStack Release Bot2023-02-241-0/+1
| | | | Change-Id: Ifada382311aa0b0c769fdabe8ff42e69d6be424c
* Fix test_while_is_not with python 3.115.1.0Gregory Thiemonge2022-12-051-11/+18
| | | | | | | | | | | | | | The test_while_is_not function relied on strings and literals to test the 'while_is_not' function. But the while_is_not function uses the 'is' operator to test the equivalency of 2 objects. This triggers a bug with python 3.11 where using 'is' with literals is not advised (it is not recommended since python 3.8 [0]). The test now uses objects from a specific class to evaluate the while_is_not function. [0] https://docs.python.org/3/whatsnew/3.8.html#changes-in-python-behavior Change-Id: I38a0135aaf73e25aa20a11c0685d5c2a7b587a07
* Merge "Adapt to new jsonschema versions"Zuul2022-11-211-7/+8
|\
| * Adapt to new jsonschema versionsCorey Bryant2022-08-031-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | This change provides fixes that were dectected by unit test failures with new jsonschema (and py310). The types argument has been removed in favor of providing a type_checker to jsonschema.validators.extend: https://github.com/python-jsonschema/jsonschema/issues/577 Closes-Bug: #1983412 Change-Id: I86f12b3d264320308e7f4841910fc21a6e8b3fa9
* | remove unicode prefix from codeniuke2022-08-242-10/+10
| | | | | | | | Change-Id: I59e8ae94eb83ae3a1ddcc99091e53686d2d55186
* | Merge "Fix formattiing of release list"Zuul2022-06-211-11/+11
|\ \
| * | Fix formattiing of release listHervé Beraud2022-05-271-11/+11
| | | | | | | | | | | | Change-Id: Ia68da313c393ee506911b73bc7ff9bc37677d607
* | | Merge "Quote string representations"5.0.0Zuul2022-06-137-11/+12
|\ \ \
| * | | Quote string representationsMichael Johnson2022-06-107-11/+12
| |/ / | | | | | | | | | | | | | | | | | | | | | networkx 2.8.3 will raise errors if certain characters, such as a colon, in the node names and the string is not quoted. This patch double quotes flow, node, and task string representations to make sure there are not issues with these characters occuring in the names. Story: 2010083 Change-Id: Ib0941cddf14dde5d6b9f97fe0224d6e6f3975226
* | | Merge "Remove six"Zuul2022-06-1079-455/+266
|\ \ \ | |/ / |/| |
| * | Remove sixTakashi Kajinami2022-05-1879-455/+266
| | | | | | | | | | | | | | | | | | | | | | | | This library no longer supports Python 2, thus usage of six can be removed. This also removes workaround about pickle library used in Python 2 only. Change-Id: I19d298cf0f402d65f0b142dea0bf35cf992332a9
* | | Drop python3.6/3.7 support in testing runtimeHervé Beraud2022-05-051-3/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | In Zed cycle testing runtime, we are targetting to drop the python 3.6/3.7 support, project started adding python 3.8 as minimum, example nova: - https://github.com/openstack/nova/blob/56b5aed08c6a3ed81b78dc216f0165ebfe3c3350/setup.cfg#L13 Also indicate that we support py3.9 Change-Id: If2369df30179bd5311c37b74f994fb802549ee81
* | Update CI to use unversioned jobs template4.7.0Stephen Finucane2022-03-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | As part of the migration of this project to the independent release model, we failed to notice that the job template was still tied to a specific release. We've now introduced a new unversioned job template, 'openstack-python3-jobs' [1], which was can and should use. Do this. [1] https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/833286/ Depends-On: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/833286/ Change-Id: Ic90c4231bfdc2a17b525ee3a382dfa34c8c39874 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
* | Fix atomdetails failure column sizePavlo Shchelokovskyy2022-01-271-0/+36
| | | | | | | | | | | | | | | | | | failure and revert_failure fields in atomdetails is defined as a JSON type, but its data type is 'text' in mysql, which is limited to 64kbytes. JSON data type should have the same size as a LONGTEXT. Closes-Bug: #1959243 Change-Id: I65b6a6d896d3e8aad871dc19b0f8d0eddf48bdd6
* | Fix unit testsGregory Thiemonge2022-01-252-2/+2
|/ | | | | | | | | | Use fasteners 0.17.3, it fixes a deadlock issue. Also fixed a new error when building the doc. Depends-On: https://review.opendev.org/c/openstack/requirements/+/826318 Change-Id: I667725886ea424cf62c5b7a07a2e58ecb3399e0c
* Merge "Handle invalid redis entries in RedisJobBoard"4.6.4Zuul2022-01-101-1/+4
|\
| * Handle invalid redis entries in RedisJobBoardGregory Thiemonge2022-01-101-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | When catching that a redis buffer is not formatted correctly, _fetch_jobs now deletes the data from the hashmap, so the same error will not occur during the next call to the function. The commit also adds JobFailure in the list of caught exceptions (might be raised by _loads) in _fetch_jobs Closes-Bug: #1949950 Change-Id: I9078086cd4c3dbfa954422dfa3d5b19fefd7264f
* | Merge "Use LOG.warning instead of deprecated LOG.warn"Zuul2021-12-212-3/+3
|\ \
| * | Use LOG.warning instead of deprecated LOG.warnTakashi Kajinami2021-11-292-3/+3
| |/ | | | | | | | | | | | | | | | | 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: I4321a489c56eb1aa650e776ee35d8f88d4d8910c
* | Fix minor typo in ActionEngine exception messageEric Harney2021-12-021-1/+1
|/ | | | Change-Id: I66b9937df5c58561792328e08f836dc05927e7f1
* Merge "Use custom JSONType columns"4.6.3Zuul2021-10-261-8/+27
|\
| * Use custom JSONType columnsPavlo Shchelokovskyy2021-07-131-8/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the JSONType from sqlalchemy_utils is quite brittle as it only does primitive json.dumps on values. This leads to various sorts of StorageFailure exceptions in taskflow when, for example, an unserializable exception bubbles up to the 'failure' field of AtomDetails. This patch sublclasses the JSONType from sqlalchemy_utils and overrides two of its methods that do (de)serialization to work via oslo.serialization functions. They deal with such occurencies much better, for example, by providing 'str' as a fallback default. Change-Id: I3b9e9498b155199a4e707006a0cf22cda0567c06 Related-Bug: #1935957
* | Replace deprecated import of ABCs from collections4.6.2Takashi Kajinami2021-07-171-2/+2
| | | | | | | | | | | | | | | | ABCs in collections should be imported from collections.abc and direct import from collections is deprecated since Python 3.3. Closes-Bug: #1936667 Change-Id: Ia6316894a67cc0ea7ee0d2110565b45546c813e7
* | Updating for OFTC IRC network4.6.1Michael Johnson2021-06-072-1/+7
| | | | | | | | Change-Id: I47faea9b3c0324d3860a5209d521bc76de03bc47
* | Merge "Fix flowdetails meta size"Zuul2021-06-011-0/+34
|\ \
| * | Fix flowdetails meta sizeGregory Thiemonge2021-05-031-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | meta field in flowdetails is defined as a JSON data type, but its data type is 'text' in mysql, which is limited to 64kbytes. JSON data type should have the same size as a LONGTEXT. Closes-Bug: #1926304 Change-Id: I9f89badfc697f0f26245ca7f4c22d62e220be5f9
* | | Merge "setup.cfg: Replace dashes with underscores"Zuul2021-05-101-4/+4
|\ \ \
| * | | setup.cfg: Replace dashes with underscoresmaaoyu2021-04-271-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: Ic58a1e2c0c3b456c3506f99d97e2db1c57423812
* | | Use unittest.mock instead of mockHervé Beraud2021-04-274-4/+3
|/ / | | | | | | | | | | | | | | The mock third party library was needed for mock support in py2 runtimes. Since we now only support py36 and later, we can use the standard lib unittest.mock module instead. Change-Id: Ib169e3deb7ddb2bc93a206ebec4043552281aa7f
* | Merge "Use TOX_CONSTRAINTS_FILE"Zuul2021-04-161-1/+1
|\ \
| * | Use TOX_CONSTRAINTS_FILEHervé Beraud2020-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | UPPER_CONSTRAINTS_FILE is old name and deprecated This allows to use upper-constraints file as more readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>. [1] https://review.opendev.org/#/c/722814/ [2] https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file Change-Id: If9ddf23af3a63ae89ccd8af95fd34a074ce91812
* | | Merge "Use py3 as the default runtime for tox"Zuul2021-04-161-1/+1
|\ \ \
| * | | Use py3 as the default runtime for toxHervé Beraud2020-11-041-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moving on py3 as the default runtime for tox to avoid to update this at each new cycle. Wallaby support officially the following runtimes [1]: - Python 3.6 - Python 3.8 During Victoria Python 3.7 was used as the default runtime [2] however this version isn't longer officially supported. [1] https://governance.openstack.org/tc/reference/runtimes/wallaby.html#python-runtimes-for-wallaby [2] https://governance.openstack.org/tc/reference/runtimes/victoria.html#python-runtimes-for-victoria Change-Id: I51a863bb5baeb51d5ec7488ddc9dd518ab56a002
* | | Merge "Remove lower-constraints remnants"Zuul2021-04-162-80/+0
|\ \ \ | |_|/ |/| |
| * | Remove lower-constraints remnantswu.shiming2021-02-202-80/+0
| | | | | | | | | | | | | | | | | | These were missed in change: Ide5deeb1313552e76468281971f72b7db11d3801 Change-Id: I46311cfaee098dabdb063d6e97145424f707f21f
* | | Move flake8 as a pre-commit local target.Daniel Bengtsson2021-03-231-3/+7
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The goal here is to avoid conflicts between flake8 and hacking version each 2 days. Inspired from nova's approach[1]. The flake8 version to install will be determined by hacking and requirements[2] will stay aligned instead of relying on different versions. [1] https://opendev.org/openstack/nova/src/branch/master/.pre-commit-config.yaml#L26-L35 [2] https://opendev.org/openstack/hacking/src/branch/master/requirements.txt#L1 Change-Id: Ie493e49188488eb2fdea244975feb8643e1ab18c
* | Fix deprecated Alembic function args4.6.0Adam Harwell2021-01-281-9/+9
| | | | | | | | | | | | | | | | create_index and create_foreign_key were using long-deprecated argument names which were finally removed as of 1.5.0: https://alembic.sqlalchemy.org/en/latest/changelog.html#change-da0bea3199b905783ddbd0cde968beec Change-Id: I03405e3d0daf085d0849a6e3effc65ec05ceb7a1
* | Dropping lower constraints testingHervé Beraud2020-12-181-1/+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: Ide5deeb1313552e76468281971f72b7db11d3801
* | Merge "ignore reno generated artifacts"Zuul2020-11-051-0/+2
|\ \ | |/ |/|
| * ignore reno generated artifactsHervé Beraud2020-09-081-0/+2
| | | | | | | | Change-Id: Iaf8370e62918bb5a83222f13796a923a6d157c0f
* | Merge "Adding pre-commit"Zuul2020-09-226-5/+55
|\ \
| * | Adding pre-commitHervé Beraud2020-09-086-5/+55
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduced changes: - pre-commit config and rules. - Add pre-commit to pep8 gate, Flake8 is covered in the pre-commit hooks. - Applying fixes for pre-commit compliance in all code. Also commit hash will be used instead of version tags in pre-commit to prevend arbitrary code from running in developer's machines. pre-commit will be used to: - trailing whitespace; - Replaces or checks mixed line ending (mixed-line-ending); - Forbid files which have a UTF-8 byte-order marker (check-byte-order-marker); - Checks that non-binary executables have a proper shebang (check-executables-have-shebangs); - Check for files that contain merge conflict strings (check-merge-conflict); - Check for debugger imports and py37+ breakpoint() calls in python source (debug-statements); - Attempts to load all yaml files to verify syntax (check-yaml); - Run flake8 checks (flake8) (local) For further details about tests please refer to: https://github.com/pre-commit/pre-commit-hooks Change-Id: I3640bb690c552a428ae0e8250adffbc3a51a310e Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
* | Add Python3 wallaby unit testsOpenStack Release Bot2020-09-111-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: Id03da2e0c09a286af97fee9a9768673d481ccbdf
* | Update master for stable/victoriaOpenStack Release Bot2020-09-112-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: I157171b91a085d2db985552a253053c163df4999 Sem-Ver: feature
* | [goal] Migrate testing to ubuntu focalvictoria-em4.5.0Ghanshyam Mann2020-09-064-11/+11
|/ | | | | | | | | | | | | | | | | | | As per victoria cycle testing runtime and community goal[1] we need to migrate upstream CI/CD to Ubuntu Focal(20.04). Fixing: - bug#1886298 Bump the lower constraints for required deps which added python3.8 support in their later version. Story: #2007865 Task: #40207 Closes-Bug: #1886298 [1] https://governance.openstack.org/tc/goals/selected/victoria/migrate-ci-cd-jobs-to-ubuntu-focal.h> Change-Id: I18419ce93363fbf2eed174e81ed212ddae216b8c
* Merge "Add sentinel redis support"4.4.0Zuul2020-08-243-1/+54
|\
| * Add sentinel redis supportAnn Taraday2020-08-053-1/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | Redis client has an ability to connect to Redis server using Sentinel[1] (especially important for Redis clusters), but this ability was missing here. Allow to pass 'sentinel' variable to Redis conf, extend _make_client to use sentinel for that case. [1] - https://github.com/andymccurdy/redis-py#sentinel-support Change-Id: Ia8cc98e701435fd0231da3724f5d7108fc4f96f4
* | Avoid endless loop on StorageFailureAnn Taraday2020-08-242-0/+18
|/ | | | | | | | | | | If an error occures with writing atom detail in database( persistence backend) flow execution enters an endless loop throwing errors and retrying to save details. Avoid this situation and log exception message. Change-Id: Ic6b0a78d20124cc027468ecc6aeff189c25d1a8a Closes-bug: 1889773
* Switch from unittest2 compat methods to Python 3.x methodsmelissaml2020-07-054-50/+50
| | | | | | | | | | | | | | | | | With the removal of Python 2.x we can remove the unittest2 compat wrappers and switch to assertCountEqual instead of assertItemsEqual We have been able to use them since then, because testtools required unittest2, which still included it. With testtools removing Python 2.7 support [3][4], we will lose support for assertItemsEqual, so we should switch to use assertCountEqual. [1] - https://bugs.python.org/issue17866 [2] - https://hg.python.org/cpython/rev/d9921cb6e3cd [3] - testing-cabal/testtools#286 [4] - testing-cabal/testtools#277 Change-Id: Iaa8251a1e9965a00fe99b7a740a104c011260340