summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Fix setting the tempest virtual env constraints env varGhanshyam Mann2023-02-213-2/+37
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Heat upgrade script set the env var TOX_CONSTRAINTS_FILE/UPPER_CONSTRAINTS_FILE which are used to use the constraints during Tempest virtual env installation. Those env var are set to non-master constraint when we need to use non-master constraints but when we need to use the master constraints we do not set/reset them point to master constraints. This create the issue when this grenade script install the tempest with stable constraints but with master Tempest and as there is mismatch of constraints now with fasteners of stable branches. Below is failure of heat greande job on stable/yoga - https://zuul.opendev.org/t/openstack/build/3aaec4d59bb84068bb4d4428ea747cbd/log/controller/logs/grenade.sh_log.txt#3245 Similar way, role should set stable constraints only for the stable EM branch not for all otherwise it fail when constraints/requirements are bumped. - https://zuul.opendev.org/t/openstack/build/74f86b8097f44c35acaffdcfe41d9693 We should set/reset those constraint env var to master constraints if configuration tell to use the master constraints. Closes-Bug: #2003993 Change-Id: I024cd134577338fc1075e7742df7f006dc914646
* | Fix unit tests for python-barbicanclient 5.5.0Michael Johnson2023-02-161-0/+2
|/ | | | | | | | | | | | | | | Version 5.5.0 of python-barbicanclient adds microversion support, which causes it to call keystoneauth1.discover.get_version_data to get the microversion information from the server. Adding additional mock.patch for keystoneauth1.discover fixes the breaking unit tests. This is a gate fix for the python-barbicanclient upper-constraints bump[1]. [1] https://review.opendev.org/c/openstack/requirements/+/873906 Co-Authored-By: Grzegorz Grasza <xek@redhat.com> Change-Id: Ic1c26d255ff0690e327b3cb6c622d3617e02df5f
* Fix an assert statementTakashi Natsume2023-02-121-2/+1
| | | | | | | | | | Fix an assert statement in heat/tests/aws/test_eip.py. Change-Id: I3ae3e8d3132f8e100067fb86460b7c3f3ae35d07 Story: 2010265 Task: 46164 Signed-off-by: Takashi Natsume <takanattie@gmail.com>
* Merge "Deploy healthcheck middleware as app instead of filter"Zuul2023-02-012-22/+79
|\
| * Deploy healthcheck middleware as app instead of filterTakashi Kajinami2022-06-062-22/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the healthcheck middleware as a filter is deprecated and the middleware should be used as an application[1]. [1] 6feaa13610c450c8486f969703768db5319b4846 This change updates definition and usage of the healthcheck middleware accordingly to avoid the following deprecation warning. DeprecationWarning: Using function/method 'Healthcheck.factory()' is deprecated: The healthcheck middleware must now be configured as an application, not as a filter. This also refactors composite definitions based on flavor by the new pipeline factory. Story: 2009071 Task: 42881 Change-Id: I75386dc4a7dc14b3c753dfff01f147ef8233bf94
* | Merge "Honor 'hidden' parameter in 'stack environment show' command"Zuul2023-01-273-2/+17
|\ \
| * | Honor 'hidden' parameter in 'stack environment show' commandChengen Du2023-01-113-2/+17
| | | | | | | | | | | | | | | | | | | | | Related-Bug: #1999665 Story: 2010484 Task: 47052 Change-Id: Ifc51ff6a4deab05002ccded59383416f9a586aa0
* | | Merge "Use new get_rpc_client API from oslo.messaging"Zuul2023-01-272-3/+3
|\ \ \
| * | | Use new get_rpc_client API from oslo.messagingTobias Urdin2023-01-192-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the new API that is consistent with the existing API instead of instantiating the client class directly. This was introduced in release 14.1.0 here [1] and added into oslo.messaging here [2] [1] https://review.opendev.org/c/openstack/requirements/+/869340 [2] https://review.opendev.org/c/openstack/oslo.messaging/+/862419 Change-Id: Ia48de36e3cb97378785bb4946b1a387c0da1ff81
* | | | Merge "Explicitly pass error kwarg in nested StackValidationFailed"Zuul2023-01-251-1/+1
|\ \ \ \
| * | | | Explicitly pass error kwarg in nested StackValidationFailedTobias Urdin2022-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the issue when a nested stack is serialized in oslo.messaging and deserialization fails because we pass a kwarg as an arg when we generate the exception remotely. Story: #2010115 Task: #45695 Change-Id: Id75398d2ed2d4fab467df51057c4167cd77bb32f
* | | | | Merge "Replace deprecated Query.as_scalar()"Zuul2023-01-242-2/+2
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Replace deprecated Query.as_scalar()ramishra2022-09-062-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has been deprecated since SQLAlchemy 1.4. Change-Id: Id3edcc188bb44d9752fe1bd7625308c5abed4d9c
* | | | | Merge "Decode UTF-8 body data in SwiftSignal"Zuul2023-01-192-54/+59
|\ \ \ \ \
| * | | | | Decode UTF-8 body data in SwiftSignalZane Bitter2023-01-192-54/+59
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie12c2df93f846629a7c8b7c6415cebc31eaba1bc Story: #2010485 Task: 47053
* | | | | Merge "Switch to 2023.1 Python3 unit tests and generic template name"Zuul2023-01-111-2/+2
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Switch to 2023.1 Python3 unit tests and generic template nameOpenStack Release Bot2022-09-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an automatically generated patch to ensure unit testing is in place for all the of the tested runtimes for antelope. Also, updating the template name to generic one. See also the PTI in governance [1]. [1]: https://governance.openstack.org/tc/reference/project-testing-interface.html Change-Id: I1df5bb05ea5ba40bea88ae0fe4cd8e8221429579
* | | | | Skip test_stack_update_with_replacing_userdataTakashi Kajinami2022-12-233-7/+10
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test case is frequently failing because of a known libvirt issue in Ubuntu Jammy. We already disabled one functional test case, but will disable this test case as well, to reduce failure rate of CI runs and unblock gate. The existing skip for a different test case[1] is re-implemented using the proper configuration knob. This also fixes tox.ini to adapt to new tox 4.0. - Update how passenv is defined because space-separated list is no longer allowed. Also the values are not case sensitive. - skipdist=True breaks installation so is removed. [1] https://review.opendev.org/c/openstack/heat/+/866545 Co-Authored-By: Rabi Mishra <ramishra@redhat.com> Related-Bug: #1998274 Story: 2010487 Task: 47056 Change-Id: I915dc83ccde6b6b8497642857292f9974fd84e98
* | | | Merge "Skip test_cancel_update_server_with_port"Zuul2022-12-061-0/+3
|\ \ \ \
| * | | | Skip test_cancel_update_server_with_portTakashi Kajinami2022-12-051-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently this test consistently fails because detaching an interface does not progress, which is likely to be a libvirt issue. This change disables the test case to temporally unblock CI. Related-Bug: #1998274 Story: 2010458 Task: 46978 Change-Id: I7d9a1b2239a8406d12f3482062f4efd1d3e55d78
* | | | | Ignore resource_data decryption errorsramishra2022-11-151-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the auth_encryption_key changes We can possibly ignore these errors when deleting stacks. Task: 42056 Change-Id: I326e415db194a5b9c67acd038d7d2d993293ecb3
* | | | | Don't load all resources for policy checkramishra2022-11-152-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We only need the resource types for the check. Let's avoid loading all resources. Change-Id: Iaf5650b2ae972064292cfe0d17f98b5635c51bad
* | | | | Imported Translations from ZanataOpenStack Proposal Bot2022-11-1311-375/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I5db9cc7f9a13db05a7d0ab0320e53bfdf4b419d9
* | | | | Merge "Retry to detach volume if nova returned error 409"Zuul2022-11-112-5/+31
|\ \ \ \ \
| * | | | | Retry to detach volume if nova returned error 409Mitya_Eremeev2022-10-122-5/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During volume detachment an instance can be in error state or locked. Nova returns error 409. Instance can be unlocked or fixed a bit later, but heat doesn't retry to detach volume. The patch makes heat to retry detachments untill stack update timeout. story: 2010355 Change-Id: I9a08b938401cd71eddd0eb80782d10392f92bf45
* | | | | | Merge "Improve map_merge logging"Zuul2022-11-091-1/+3
|\ \ \ \ \ \
| * | | | | | Improve map_merge loggingBrendan Shephard2022-11-031-1/+3
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Templates can be complex and contain multiple instances of map_merges. Debugging this can be complex, so this change aims to improve the mapping and help users determine where the problem has occured. With this change, instead of just saying an error occured, we will now return the object that the error occurred with and a note about it's type. This will allow for quick identification of what the problem was. Change-Id: I1c4da88029bbf1a3cfb58f5e21da28ec2912f924
* | | | | | Update stestr args for inclusive languageBrendan Shephard2022-11-091-1/+1
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | stestr has removed whitelist / blacklist. This change updates the devstack upgrade tests to ensure we use the new include-list instead. https://github.com/mtreinish/stestr/commit/9ffeb470fb4eaed484152d35411668db19a64ace Change-Id: Ia0df9b3468fee9382c42c8bd6a35b76ed7f2b4e5
* | | | | Set cache_ok to avoid further SAWarningTakashi Kajinami2022-11-011-0/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change fixes the one remaining usage without the flag which was missed by the previous attempt[1]. [1] 14d2d859ef4eb3d21bf6e419a77c2f3185d5cf77 Change-Id: Ic0b34b11d669851e47cae670f5299f934f396257
* | | | Merge "Allow deleting user_creds when can't be decrypted"Zuul2022-10-041-2/+4
|\ \ \ \
| * | | | Allow deleting user_creds when can't be decryptedramishra2021-03-161-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are situations when the auth_encryption_key changes and customer wants to delete old stacks. We should allow deleting those stacks. Task: #42055 Change-Id: Ifc8c19e181902566d4f295fa979ab6869a4e0852
* | | | | Merge "Fix get_attr syntax in template guide"Zuul2022-09-281-1/+1
|\ \ \ \ \
| * | | | | Fix get_attr syntax in template guideZane Bitter2022-09-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I328fbe089731d1d2a2e8aa1a6f05f6621a9201b8 Story: 2010312 Task: 46373
* | | | | | Drop implementation for OS::Magnum::Bay/BayModelTakashi Kajinami2022-09-215-322/+2
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Magnum has deprecated it's Bay API and BayModel API a long ago and these resource types were deprecated and are hidden in Heat now. Magnum has removed the old API during Zed cycle and the resources are no longer supported, so we should remove the implementation now. Change-Id: If4c6a83a83f9eb29407eb632f47c7229b5492ffd
* | | | | Update master for stable/zedOpenStack Release Bot2022-09-132-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add file to the reno documentation build to show release notes for stable/zed. Use pbr instruction to increment the minor version number automatically so that master versions are higher than the versions on stable/zed. Sem-Ver: feature Change-Id: I58401373b25d20594edd1dd443c29fa7453731b3
* | | | | Imported Translations from Zanata19.0.0.0rc1OpenStack Proposal Bot2022-09-1111-251/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I791e0ca3ccd221411f0ec9222a4c3f8ed9126f99
* | | | | Fix compatibility with oslo.db 12.1.0Stephen Finucane2022-09-051-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | oslo.db 12.1.0 has changed the default value for the 'autocommit' parameter of 'LegacyEngineFacade' from 'True' to 'False'. This is a necessary step to ensure compatibility with SQLAlchemy 2.0. However, we are currently relying on the autocommit behavior and need changes to explicitly manage sessions. Until that happens, we need to override the default. Signed-off-by: Stephen Finucane <stephenfin@redhat.com> Change-Id: I57b8cce28c2ad7899b831067c94b9d913f3f1334
* | | | | Merge "Floating IP port forwarding resource"Zuul2022-08-174-0/+430
|\ \ \ \ \
| * | | | | Floating IP port forwarding resourceBrendan Shephard2022-08-164-0/+430
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new heat resource for Floating IP port forwarding extension. Story: 2009321 Task: 43742 Change-Id: I729f11873940a83e77038c5ba8e8eb50965623f6
* | | | | | Merge "Remove remaining implementation for Heat CloudWatch API"Zuul2022-08-174-116/+4
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Remove remaining implementation for Heat CloudWatch APITakashi Kajinami2022-05-064-116/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The CloudWatch API was removed during Queens cycle[1]. This change removes the remaining parameters because these have been kept for enough cycles to let users aware of the removal. Change-Id: I4f57f02332cb1b61a0681794e519dd5ecfbec6c3
* | | | | | Merge "Remove CloudWatch cleanup tasks"Zuul2022-08-022-12/+0
|\ \ \ \ \ \
| * | | | | | Remove CloudWatch cleanup tasksBrendan Shephard2022-07-212-12/+0
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change removes the cloud watch cleanup tasks as part of general housecleaning. Change-Id: Id8f678dfd504ff27d6fdb86011129d353a31f3cf
* | | | | | OS::Nova::KeyPair: Require public_key when api version >= 2.92Takashi Kajinami2022-08-012-4/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... because Nova requires the public_key property when creating a keypair since api microversion 2.92. Story: 2010201 Task: 45920 Change-Id: Iec6f1c570ec3bb19ca58b336e6e3401cc11c4c33
* | | | | | Replace KeyPair resource by TestResourceTakashi Kajinami2022-08-011-13/+8
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... because we don't really intend to test the actual resource logic but behavior of the engine logic to manage resources. Depends-on: https://review.opendev.org/851669 Story: 2010201 Task: 45918 Change-Id: If651dbe34792c8ec72320a6d389a1bc1f2d23d13
* | | | | Merge "Set cache_ok to avoid SAWarning"Zuul2022-07-181-0/+2
|\ \ \ \ \
| * | | | | Set cache_ok to avoid SAWarningTakashi Kajinami2022-06-151-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since sqlalchemy 1.4.14[1], the following warning is raised unless the cache_ok flag is explicitly set in classes inheriting TypeDecorator. SAWarning: TypeDecorator Json() will not produce a cache key because the ``cache_ok`` attribute is not set to True. This can have significant performance implications including some performance degradations in comparison to prior SQLAlchemy versions. Set this attribute to True if this type object's state is safe to use in a cache key, or False to disable this warning. (Background on this error at: https://sqlalche.me/e/14/cprf) Because the flag was set to True by default in older releases, this adds explicit cache_ok = True, to restore the previous behavior in Wallaby and older. (wallaby u-c has SQLAlchemy===1.3.23 while xena u-c has SQLAlchemy===1.4.23) [1] https://github.com/zzzeek/sqlalchemy/commit/6967b4502079e199b12f5eb307d10d27ec92d537 Change-Id: I6b84a626994543468f49373a554de44025576a9a
* | | | | | Accept sha256 hash for swift tempurlTakashi Kajinami2022-07-182-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since swiftclient 4.0.1, sha256 is used when generating tempurl signature[1]. This change extends the regex pattern to assert signatues so that it can accept both sha1(old) and sha256(current). [1] https://review.opendev.org/c/openstack/python-swiftclient/+/845157 Change-Id: I063fb5faf3546e306f41821f14fc14a7b28c9743
* | | | | | Merge "Followup to I7736373d03c23884158e4a1d41defafb4e2b8a4c"Zuul2022-07-141-1/+1
|\ \ \ \ \ \
| * | | | | | Followup to I7736373d03c23884158e4a1d41defafb4e2b8a4cPavlo Shchelokovskyy2022-07-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | that patch erroneously replaced dict.values() with dict.items(), and now ExtraRouteSet can not be created due to AttributeError ('tuple' has no attribute 'has_interface'). Story: 2010142 Task: 45778 Change-Id: I2643b319f3a91f98097b96639f7298b25724aa62