summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Added a unit test for get_environment_hidden_param" into stable/wallabystable/wallabyZuul2023-04-131-0/+50
|\
| * Added a unit test for get_environment_hidden_paramChengen Du2023-03-291-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | A unit test for verifying that hidden parameters should be masked in the get stack environment API Story: 2010555 Task: 47229 Change-Id: I44ef26849ebb450a5643907a0ef47166cfd9b1cb (cherry picked from commit bdcb2c3bd115f1893baeab1d946eefa4aab49552) (cherry picked from commit 89b57e5e2eee091f18afbc03e83536109388dcb6) (cherry picked from commit 41a6b91362c3cf90bd3692c88b9840d6b1a8d56b) (cherry picked from commit 31778416befbf435b33e4bd2d8639f537de4c727)
* | Merge "Honor 'hidden' parameter in 'stack environment show' command" into ↵Zuul2023-04-133-2/+19
|\ \ | |/ | | | | stable/wallaby
| * Honor 'hidden' parameter in 'stack environment show' commandChengen Du2023-03-293-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backport note: This includes change I0abbd535aacc03446ada0fa806dfdfdaa4522afe which fixed the wrong explanation in the release note file. Related-Bug: #1999665 Story: 2010484 Task: 47052 Change-Id: Ifc51ff6a4deab05002ccded59383416f9a586aa0 (cherry picked from commit c1b5fcfbe37d3f94d5b73ae36e071d1aa8c4e0f5) (cherry picked from commit 436ecad6511b30683f6b9af4e484ed64c3bcaab9) (cherry picked from commit c50d720aa36fd7553b133a5a2f4fefdb3040211f) (cherry picked from commit 1a49525c547a607743160a33b6a23041378ff177)
* | Fix setting the tempest virtual env constraints env varGhanshyam Mann2023-04-114-4/+39
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. Note: This backport includes change Ie552e6398c10797e4df4afe757760c48ca7f80af to unblock gate by a single commit. Closes-Bug: #2003993 Change-Id: I024cd134577338fc1075e7742df7f006dc914646 (cherry picked from commit dbbba907d396158ce3bf3a207a8875b9fbb5a5b1) (cherry picked from commit b29266398fbb4e623faaa6a894de2a33bb2846a7) (cherry picked from commit 391684581b60bf1ca7a88f2c52898cd22cfd48f1) (cherry picked from commit 1360cfdfa3026ce09e86c9f67e38c390229477fd)
* Ignore resource_data decryption errorsramishra2023-01-191-3/+8
| | | | | | | | | | | | If the auth_encryption_key changes We can possibly ignore these errors when deleting stacks. Task: 42056 Change-Id: I326e415db194a5b9c67acd038d7d2d993293ecb3 (cherry picked from commit 9407b4897e77db24bf1948153afbb1479d5dad42) (cherry picked from commit af617facc5d6a63f416c1a047c91a66008ee4364) (cherry picked from commit 83a983678cdfde938e42db15c17fbc775cb88cac) (cherry picked from commit b135a99575ed42882d17b4bef2006a10cfe8afaa)
* Allow deleting user_creds when can't be decryptedramishra2022-11-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 (cherry picked from commit 520e2389d3123efc9269bbf82c6c9998b6c62564) (cherry picked from commit c3a8e3d64ebb37ab83b3cebb868cd68d4135f59d) (cherry picked from commit c37ec5180cefbc992640bdb2e80ed67b92dc5664)
* zuul: Declare queue at top levelwallaby-em16.1.0Takashi Kajinami2022-09-271-1/+1
| | | | | | | | | | | | | | | | Zuul deprecated declaring shared queues at a pipeline level with release 4.1.0[1]. This updates the job definition to use the top level declaration instead. See [2] for details. [1] https://zuul-ci.org/docs/zuul/latest/releasenotes.html#relnotes-4-1-0-deprecation-notes [2] http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028603.html Change-Id: Ib54f5944cfd803baa4044858c097fc71f1ffb789 (cherry picked from commit 4a898171f0b512518aa94dbcdd58cfc707630af9) (cherry picked from commit f4904ff7af40c8683d0dd3550eaf979fa90a2e47) (cherry picked from commit 782da94b34d17b2bf06ba8bf06f43c091c52266f)
* Fix AttributeError in Python 3.8Takashi Kajinami2022-08-172-6/+51
| | | | | | | | | | | | | | | | | | | | | | | | The importlib-metadata package was imported to the core Python since Python 3.8, and stevedore uses the standard library implementation instead of the 3rd party package in Python 3.8 and later. However the implementation available in Python 3.8 is quite old and especially it does not include the change which added the module attribute[1]. This change makes sure we avoid referring to that attribute if it does not exist, to avoid AttributeError in Python 3.8. [1] https://github.com/python/importlib_metadata/commit/dfc96137ac3583477c02aa97291b0896fb61f282 Backport note: This change includes an additional fix for Python < 3.8, which uses the 3rd party importlib-metadata. Story: 2010095 Task: 45650 Change-Id: Ieba5f353b0fe8a9b187bac1f6ff1b3a8f3b896b3 (cherry picked from commit eec1f92bc02e8beb4ce1db9646f8b3b678fb8329) (cherry picked from commit ecca0fcbeb96608e56005ec40fe586f498b62f66) (cherry picked from commit b8e27f7c47f0f4def7bbd7dc28e6c5c6aba91f9e)
* Merge "Remove TripleO job" into stable/wallabyZuul2022-08-171-3/+0
|\
| * Remove TripleO jobTakashi Kajinami2022-05-251-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the experimental TripleO job. The job has been kept in experimental queue and it has never attracted actual attention. Also the job does not use the proposed change in heat but the current master instead, so it's not useful to be in the Heat jobs. (yoga to xena) Conflicts: .zuul.yaml Change-Id: I1b99f8d039271bff8e64d64ad5482231bf2a2118 (cherry picked from commit b3430d9fc2d82820ea14d21381efabe5e22f8925) (cherry picked from commit c2b8f3c710b05b24c075a98204de9e121a0dc743) (cherry picked from commit e6a8e66c78f36b9b7f609a8ec7f80e48b155b957)
* | Merge "Log parameters merged" into stable/wallabyZuul2022-07-181-0/+6
|\ \
| * | Log parameters mergedrabi2022-06-141-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can help in troubleshooting. Task: 45579 Change-Id: I6aee0719b0cdef2b17666a43074c5c18645fc180 (cherry picked from commit c0be1f01fd9c208130857efd7c01ece94d4fb0bb) (cherry picked from commit d997d70ca13065d1caf96c06af05d084353ca827) (cherry picked from commit f56ca96325c200749144b2922d20d84ac47c63ef)
* | | Don't always replace FAILED ServerGroup resourcesRabi Mishra2022-07-152-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ServerGroup resources work as scheduler hints for server resources and replacement of it replaces the servers which is undesirable in many circumstances. - We don't allow update of ServerGroup resource properties - If not created (CREATE_FAILED) without resource_id they will be replaced - If does not exist in nova they would be created - If exist in nova, FAILED resource status would be just changed to COMPLETE Conflicts: heat/tests/openstack/nova/test_server_group.py Fixed conflicts caused by 74164424016702bac6d52c6618360287ce7ea461 , which is present in stable/xena and later. Task: 45748 Change-Id: I096dff2b541a5aa8afbbbcea5161e7ca1c244039 (cherry picked from commit a0e072b3204d736b9a4a5b2ceb2157f57405ac6f) (cherry picked from commit f9f5d2cc368e5f850b962c8d89669a50544058d1) (cherry picked from commit ead7cd61bd685ea46dd21c4ef3ea6b3fde273c10)
* | | Use openstack commands in upgrade pluginRabi Mishra2022-07-141-3/+3
| | | | | | | | | | | | | | | | | | | | | Change-Id: I94075f261ce095c509635aab395df42878aceb1c (cherry picked from commit e446ecbb21c2c6e8909ec20121536073d66bbcb7) (cherry picked from commit 08a0a218ef406d3027fd28be8e77f9876fd10f4d) (cherry picked from commit 65fb6c7dadee7412213eeea1b36bf9befcfd5873)
* | | Merge "Make grenade job voting again" into stable/wallabyZuul2022-07-052-3/+6
|\ \ \
| * | | Make grenade job voting againTakashi Kajinami2022-07-052-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because change I7d1017bcd7147329c4e187336daee1a1ae280fa9 was backported to the previous stable branch. Backport note: This includes the following follow-up commit. (commit fee8c40c23fb160a4fe64e95a71f827528287690) Use constraints when creating venv for grenade As a followup for change I6a8cffdc86c895eebe4269c5cd37841325566c54 let's use branch specific upper constraints when running tests in grenade. Change-Id: I2de594258231e3f1c217904afce88a83a8f94df3 (cherry picked from commit fee8c40c23fb160a4fe64e95a71f827528287690) (cherry picked from commit 03e4f9af62ed66a4df72f5da793f8ce4caf8ac3e) (cherry picked from commit e3ec8946b3de50c76b2c3b08c2f7356842d3f45a)
* | | | Fix parameter merging with merge strategiesRabi Mishra2022-07-042-15/+16
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Earlier users had to specify merge strategy for a parameter/parameter_default in every environment file where they are used(or the defaults for that environment was used). However, the strategy for a parameter in the first environment was ignored as there is nothing to merge. With Iec3367667248d0b46196a6d569f3cbb3b200d842, we changed to use existing merge strategies, but the strategies in the first environment were still ignored. This patch fixes it. Task: 45578 Change-Id: Ic6125c6af60c1007243523cc8510a17f49c7b5af (cherry picked from commit d9fe7301b3af2db9fa56d59eb405add94730ba50) (cherry picked from commit 154691d21d24351edbe066fd23c731afad456b85) (cherry picked from commit b8d09f347c2d478e7d69c940152a4d85fbc58f7c)
* | | Drop lower-constraints.txt and its testingGhanshyam Mann2022-06-164-171/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed in TC PTG[1] and TC resolution[2], we are dropping the lower-constraints.txt file and its testing. We will keep lower bounds in the requirements.txt file but with a note that these are not tested lower bounds and we try our best to keep them updated. [1] https://etherpad.opendev.org/p/tc-zed-ptg#L326 [2] https://governance.openstack.org/tc/resolutions/20220414-drop-lower-constraints.html#proposal Conflicts: .zuul.yaml lower-constraints.txt Change-Id: I703229320320295b4a90e72c543f8959f85daaa9 (cherry picked from commit 49cbb92ed611899728425877e86396a70f5d7c5f) (cherry picked from commit 80ce36214bf2d8b2e28740916b2e87ee9e10ae44) (cherry picked from commit d1c0f628945caca01b0c986adf97e93e7795cf49)
* | | Use constraints when creating tempest venvramishra2022-06-152-2/+24
|/ / | | | | | | | | | | | | | | | | | | | | | | When running tempest we run in-tree tests as well. Lots of these tests including tempest tests may not work with new client libraries. Let's use the relevant branch upper constraints when running functional tests. Change-Id: I6a8cffdc86c895eebe4269c5cd37841325566c54 (cherry picked from commit 22c48f57bb29ed62a8c3249ab953043cfd2617d3) (cherry picked from commit b95e6025f61417e7abc24a26d991bd26c57ee444) (cherry picked from commit 92db02a56e85dd5f890160217a16dc99394b927d)
* | Use previous parameter merge strategyrabi2022-06-142-3/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't allow different parameter merge strategies in different environment files for a parameter. If a merge strategy has already been provided in one environment file, let's use that for subsequent ones unless a different merge strategy has been provided. Task: 45578 Change-Id: Iec3367667248d0b46196a6d569f3cbb3b200d842 (cherry picked from commit 94f32ee39f4d705614079bd0ac113cde67dbb7e4) (cherry picked from commit f0507980586711014e28411d53faba30138c3044) (cherry picked from commit 66dd43c6555451e4a6f60d82ef8da33cc83f6397)
* | Use f36 instead of f33 for testingMatthias Runge2022-06-015-8/+9
|/ | | | | | | | | F33 is unsupported for a while. We should not use it for any testing. Change-Id: I7d1017bcd7147329c4e187336daee1a1ae280fa9 (cherry picked from commit 78244c5f0f26064cd53b68e13fa8c4f0e235e698) (cherry picked from commit eefb921612b6d560e9f28767ba3a40e25bdd4f7d) (cherry picked from commit b32163d9e35e522577b99110ebd1614de55f5653)
* Fix None comparision when sorting by `updated_at`Erik Panter2022-04-132-6/+26
| | | | | | | | | | | | When sorting resource candidates in `_get_best_existing_rsrc_db`, resources with the same score are sorted by `updated_at`, which can be `None`. If that is the case, use `created_at` instead. Task: 43815 Story: 2009653 Change-Id: Ic0265fcf7ceb811803cdebaa8932fe80dc59a627 (cherry picked from commit 403fa55fe94ae1063d2cb4b8db3b63b76b1ee5cf) (cherry picked from commit 5ea5276a3e76829fd72345e3aae7482cbd260b51)
* Merge "Fix timeout calc of nested resource groups" into stable/wallabyZuul2021-11-302-15/+13
|\
| * Fix timeout calc of nested resource groupsBalazs Gibizer2021-11-192-15/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The timeout calculation of nested resource groups are using seconds but then it passing the value in seconds to the stack resource that is expected it in minutes. This leads to a rapidly increasing timeout value in each nesting levels. This patch converts the seconds to minutes before passing it forward. Story: 2009237 Task: 43372 Change-Id: I3f132d4889723f7b4a3a416779ac5ee7663249b8 (cherry picked from commit ef731bca05ea683c30448a2db397ae631c121d95) (cherry picked from commit f3c74a6632c5ced8c25f5811315e3d943be13c17)
* | Merge "Reproduce bug story/2009237" into stable/wallabyZuul2021-11-291-0/+62
|\ \ | |/
| * Reproduce bug story/2009237Balazs Gibizer2021-11-191-0/+62
| | | | | | | | | | | | | | | | | | | | | | Add a functional test that reproduces the bug that nested resource group timeout value is increasing rapidly with every nesting level. Story: 2009237 Task: 43373 Change-Id: I8e85434a5fb9cc6d0ef3240c8ae6085f5124174b (cherry picked from commit 863d645d7ef223a2534423e623fbc1c420c8557f) (cherry picked from commit 74280022e8dbe306c59a49f73b78ab40688fe6a6)
* | Provide install_command to pip install with upper-constraintsBrendan Shephard2021-11-201-0/+1
|/ | | | | | | SQLAlchemy latest (1.4.27) installed with docs job breaks it. Change-Id: Ida63e1d865dc0b2ac6a13594455962f53068c931 (cherry picked from commit 7f15b14d800654474ccab26af40384ab4515a657)
* Fallback to upstream for Fedora imageIan Wienand2021-11-041-1/+10
| | | | | | | | | | | OpenDev infra only keep around the latest two Fedora releases in their mirrors. Probe for the image from the local test mirror, but if not found, fallback to upstream. This will be much less reliable, but can avoid gate breakage until new images can be used. Depends-On: https://review.opendev.org/c/openstack/heat/+/816604 Cherry-Picked-From: f7574531d3dc3bbbf55fd7c17d55a10b5320c5ff Change-Id: I96ab14871ee8c5d5b83cc0cd4abc840ef0218ca8
* Make grenade job voting againBrendan Shephard2021-08-261-1/+0
| | | | | Change-Id: Ia65dabd7e357ad4ed8076fef3a3134c87100d4ed (cherry picked from commit 42b5f68bf047252a7ec6e0db98f0546c042286f3)
* Detect EL8 platform-pythonLuke Short2021-08-181-0/+9
| | | | | | | | Otherwise, this script fails to run on Enterprise Linux 8 distros. Change-Id: I0168b380d0e201d9c8f8121b384d25a2412c2129 Signed-off-by: Luke Short <ekultails@gmail.com> (cherry picked from commit 2d692e22f42bc64476ab01ff7e8c4d4717ef3786)
* Use Block Storage API v3 instead of API v2Takashi Kajinami2021-08-037-47/+7
| | | | | | | | | | | | | | | Block Storage API v2 was deprecated during Pike cycle and is being removed during Xena cycle, and current v3 API should be used instead. Unused volume_client and network_client in integration test code are also removed by this change. Note: granade tests is made non-voting temporally until the same issue is fixed in stable/victoria. Depends-on: https://review.opendev.org/802150/ Change-Id: I6a2b5afa13480791971bbd8bba1f43b9f2db8294 (cherry picked from commit 1828df32fa946d49d8449ff1d8a26c05b3990a65)
* Preserve order in list_concat_uniqueramishra2021-04-242-7/+9
| | | | | | | | | | Also, we should not modify a list when iterating over it. Task: 42359 Closes-Bug: #1925373 Change-Id: Iaa2c05b4155d93f44de60b6f98a69450c1512817 (cherry picked from commit a8f8528d1ea1952cf76bd09fd27cbd65a978af65)
* Merge "Update TOX_CONSTRAINTS_FILE for stable/wallaby" into stable/wallabyZuul2021-04-141-3/+3
|\
| * Update TOX_CONSTRAINTS_FILE for stable/wallabyOpenStack Release Bot2021-03-261-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | 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: Ia8acf5aa80b5635069a28d8caec40f428003807d
* | Merge "Update .gitreview for stable/wallaby" into stable/wallabyZuul2021-04-141-0/+1
|\ \ | |/
| * Update .gitreview for stable/wallabyOpenStack Release Bot2021-03-261-0/+1
| | | | | | | | Change-Id: I44c8dc9c1086188f34bb18c213f9b1c4ee2d2083
* | Don't update status for replaced resourceramishra2021-04-024-12/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With I04e7ad90944c2d03ce0e59ba16af9d60d6e01222 we allowed update of previously-existing resource if replacement creation has failed during last update. However, we force update the status of the existing resource to 'COMPLETE'. Therefore, in the next update if properties/type has not changed for the resource it won't try to update/replace the previously existing resource resulting in false positive of stack updated successfully. Task: 42194 Change-Id: Icc90a921ec67e49aec2c4acfad72235c57c78421 (cherry picked from commit bfbaab107d709c37548a22239b952c6221b9ac85)
* | Assume interface detach called for port not foundramishra2021-04-022-2/+2
|/ | | | | | | | | | | | If the interface has been detached before we call detach_interface, then we keep on calling it as we get a 404 from nova. Let's assume it's being called and check if the interface has been detached. Task: 42195 Change-Id: I768eeb10c2c24a0c93749b11636ef3e003ecfcd9 (cherry picked from commit 6ff0dc2e19ddf3a3447810f8a55b29d9bec52136)
* Merge "Allow creating a Neutron port without fixed ips"16.0.0.0rc116.0.0Zuul2021-03-153-20/+84
|\
| * Allow creating a Neutron port without fixed ipsTakashi Kajinami2021-01-253-20/+84
| | | | | | | | | | | | | | | | | | | | | | | | Since Newton release, Neutron supports ports without fixed ips[1]. This change introduces a new no_fixed_ips property so that we can create an unaddressed port from Heat. [1] https://specs.openstack.org/openstack/neutron-specs/specs/newton/unaddressed-port.html Story: 2008554 Task: 41685 Change-Id: I6f16905155038db44c7e01dd5d34d65032dca061
* | Merge "Add separate policy for updates with no changes"Zuul2021-03-154-1/+60
|\ \
| * | Add separate policy for updates with no changesZane Bitter2021-03-154-1/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow operators to set a different (presumably looser) policy on PATCH updates that don't make any changes to the stack, but just retrigger a new update traversal (that will result in e.g. replacing any unhealthy resources). Change-Id: Id29e7ec7f6cf127177ea7ab29127b0568afaa18b Task: 37305
* | | Merge "Adds backups and backups_gigabytes parameters to cinder quota resource"Zuul2021-03-152-3/+50
|\ \ \
| * | | Adds backups and backups_gigabytes parameters to cinder quota resourceSampat P2020-11-252-3/+50
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3b2a088597ec46cd43cb5fbcb2be0c79a6b0fa94 Story:2008120 Task:40841
* | | | Merge "Remove references to basestring"Zuul2021-03-151-1/+1
|\ \ \ \
| * | | | Remove references to basestringStephen Finucane2021-02-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This doesn't exist in Python 3. Change-Id: I5aff8a205d524cc15c161a961b0603efeb66a2e8 Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
* | | | | Merge "Allow turning off max_stacks limit"Zuul2021-03-152-4/+5
|\ \ \ \ \
| * | | | | Allow turning off max_stacks limitDirk Mueller2019-08-052-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The limit of 100 is fairly low for a normal production environment. Allow setting the limit to -1 to turn it off. Also raise limit to 512 which should be a bit more reasonable as a default. Change-Id: I9e54b20437875ed88b79414aa4fc17b17cbd305b Related-Bug: #1221849
* | | | | | Merge "Add segments attribute to OS::Neutron::ProviderNet"Zuul2021-03-152-2/+11
|\ \ \ \ \ \
View Lorry Controller Status