summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Use Block Storage API v3 instead of API v2stable/ussuriTakashi Kajinami2021-08-2511-54/+17
| | | | | | | | | | | | | | | | | | | | | | | | | 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. Also, changes to use fedora 33 for tests. Depends-on: https://review.opendev.org/802150/ Change-Id: I6a2b5afa13480791971bbd8bba1f43b9f2db8294 (cherry picked from commit 1828df32fa946d49d8449ff1d8a26c05b3990a65) Move testing to Fedora 33 Now that the gate is unblocked, we can also test F33 for gating purposes here. Change-Id: Ie1c9075623d85b27aaf1ac67a3063e219726bc6f (cherry picked from commit 6a9c35d89804057b56e1b0b995265ce3eb68129d)
* Preserve order in list_concat_uniqueussuri-em14.2.0ramishra2021-04-272-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 "Net attr: Sort segments without name attribute first" into stable/ussuriZuul2021-04-142-1/+22
|\
| * Net attr: Sort segments without name attribute firstHarald Jensås2021-03-052-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Useing get_attr, pulling index 0 in the segments list of a network to associate the a subnet with the "first" segment is useful since the "first" segment is created by neutron behind the scenes on network create. A resource reference cannot be used since the "first" segment is'nt a heat resource. The issue is the order of the segments list is'nt reliable. On stack update index 0 may be a different segment, and we end up trying to update the segment_id for a subnet. Changeing the segment association is not allowed, so the stack update fails. While not perfect, sorting the list so that segments where name is None comes first will ensure that index 0 can be used. The template author should ensure segments defined in the heat template all have a names set, so that only the segment creted implicitly by neutron have 'None' name. Closes-Bug: #1894920 Change-Id: I097aba2a97144327bec188e6c71629d0f6c95901 (cherry picked from commit 539b2a4c4993787db0cd71750d3d159e584deccb)
* | Don't update status for replaced resourceramishra2021-04-044-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)
* Add SOURCE_IP_PORT to LB_ALGORITHM allowed valuesKevin Carter2021-01-202-2/+6
| | | | | | | | | | | | | | | | | This change adds SOURCE_IP_PORT to lb_algorithm property of OS::Octavia::Pool resource which is missing from the available options and required when using the OVN provider driver. [0] https://docs.openstack.org/ovn-octavia-provider/latest/admin/driver.html#limitations-of-the-ovn-provider-driver [1] https://docs.openstack.org/releasenotes/octavia/train.html Story: 2008361 Task: 41268 Change-Id: Ibed37eea817a5125c5751b2bc9f44acf3c016fe1 Signed-off-by: Kevin Carter <kecarter@redhat.com> (cherry picked from commit 20b599cdf893af5165b07b3d1f70864693ec35ff) (cherry picked from commit 3cf3a68a40d6a12e6abc5f2dd582b7e66a66dabc)
* Replace Fedora test image with F32Matthias Runge2021-01-148-13/+14
| | | | | | | | | | | | | | | | | | | Fedora 31 was retired and the image is gone from mirrors. heat-cfntools have been dropped from fedora images, disable the test till that's resolved. NOTE(elod.illes): the following changes were made compared to the original patch: grenade jobs don't need to be non-voting anymore. bandit removed B322 test, so it needs to be removed from tox.ini, too. Fix contradicting lower constraints to make lower-constraints job pass. Change-Id: Id869f83a46454897c2fe7a532eebfa2863befe5e (cherry picked from commit 61e53c4eca3bc4f0c60754269cb2811afc569705) (cherry picked from commit 779b39c975c32e8fd0a3982197787c294cf92589)
* As of nova microversion 2.57, personality is deprecatedDavid Hill2020-11-172-1/+50
| | | | | | | | | | | This patch returns a stack validation error if the nova microversion is 2.57 or greater and the personality parameter is also specified in the templates. Change-Id: I0919150baa36112ccbd8bfd7ccebc7aab5735cf3 Task: 41229 (cherry picked from commit b068f45adda72fa12f50008a603c8f37fcabcdff) (cherry picked from commit 3f98fd9b1dfadf0390cc8bef9f42d08840b34840)
* Merge "Lazily cache parsed value of list/json parameters" into stable/ussuriZuul2020-10-132-22/+21
|\
| * Lazily cache parsed value of list/json parametersZane Bitter2020-09-082-22/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Parsing the value of JSON parameters at the time we create them (or update the default value) results in exceptions occurring while creating the Stack object, that are then not caught and transformed. The result is an HTTP 500 Internal Error reported by the API, with an unhelpful error message. By not parsing the JSON until it is needed, we ensure that any errors will occur during validation, where errors are appropriately transformed (to StackValidationFailed) and annotated with the necessary information to help the user determine the source. Change-Id: I70e341c344d6254173ad4519276626230087263a Story: 2007957 Task: 40443 (cherry picked from commit b603470a1207fcbc9ee30f3a3b4b457a9f83a03a)
* | Merge "Fix OS::Heat::Delay resource type" into stable/ussuriZuul2020-10-122-3/+7
|\ \
| * | Fix OS::Heat::Delay resource typeZane Bitter2020-10-062-3/+7
| |/ | | | | | | | | | | | | | | | | | | Don't override ResourceStatus.ACTIONS, as it's used internally by the Resource class. Change-Id: Icf306e981c19e3d5410905befa7907dcdba33337 Story: 2008013 Task: 40663 (cherry picked from commit c362c57854f7d841a72dd9de2f6ec3a9b9187f82)
* | Merge "Configure swap as before" into stable/ussuriZuul2020-10-121-0/+2
|\ \
| * | Configure swap as beforeRabi Mishra2020-10-091-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://review.opendev.org/750941 changed the default swap to 1024MB for jobs and this seems to resulting in high failure rate of the functional and grenade jobs. Let's configure it back to what used to before i.e 8192MB. Change-Id: I627f2df7f6135b3d194e7146ae45c3b7f8b6394d (cherry picked from commit f419923a0d3aa62485296ab7666485e74af57c8e)
* | | Supports 'virtio-forwarder' vnic type for portEmilien Macchi2020-10-072-1/+7
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supports the 'virtio-forwarder' vnic type for port sync with Neutron. The 'virtio-forwarder' is an indirect mode for SR-IOV interfaces used to connect the instances into the Neutron networks. It has been supported since the Queens release: I28315fb12f62dc7d52d099c211086b2b81eca39a Note that 'virtio-forwarder' is already supported by Horizon: openstack_dashboard/api/neutron.py: ('virtio-forwarder', _('Virtio Forwarder')) Nova: nova/network/model.py: VNIC_TYPE_VIRTIO_FORWARDER = 'virtio-forwarder' openstackclient: openstackclient/network/v2/port.py: choices=(...), virtio-forwarder neutron-lib: neutron_lib/api/definitions/portbindings.py VNIC_VIRTIO_FORWARDER = 'virtio-forwarder' Task: 40435 Change-Id: Ie65f8858d6d1a1854702eb33c40ea827505bfe51 (cherry picked from commit 03ab18d4599183d13dceb6d1130358f717197136)
* | Use Fedora 31 for testsRabi Mishra2020-10-075-7/+7
|/ | | | | | | | | Fedora 30 does not seem to be available anymore. http://mirror.bhs1.ovh.opendev.org/fedora/releases/ Change-Id: Ia45e957bb24f5596a5ef16271f71ba8b77df9e42 (cherry picked from commit f18a6767927965b930579995eb0b3d13c647bc26)
* Add U release codename for docs generation14.1.0Zane Bitter2020-08-121-1/+2
| | | | | | This makes for nicer output in the resource types docs. Change-Id: I5c7b0e7454df0c5dd33efb03c08d15ea231bf530
* Merge "Check external resources after creation" into stable/ussuriZuul2020-07-273-1/+16
|\
| * Check external resources after creationZane Bitter2020-07-233-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The method Resource.check() is not a generator function, but it does return a generator in most cases. Simply calling the function does not cause the resource to actually be checked. Only in the case where the resource did not provide a handle_check() method would it do anything at all - and in that case it simply changes the state to CHECK_COMPLETE. Yield the generator function so that the resource will actually be checked if the resource type supports it. Change-Id: I2a78b6f6235529a95838307e3d38731d31ff5c4b Task: 37828 (cherry picked from commit 5c6038f7a272238eef8978a4b26ab4ae4efd745b)
* | Merge "Monkey patch original current_thread _active" into stable/ussuriZuul2020-07-234-0/+24
|\ \ | |/ |/|
| * Monkey patch original current_thread _activeCorey Bryant2020-05-124-0/+24
| | | | | | | | | | | | | | | | | | | | Monkey patch the original current_thread to use the up-to-date _active global variable. This solution is based on that documented at: https://github.com/eventlet/eventlet/issues/592 Change-Id: I0540923755ac3969b584eeba2e19c037a7f2c261 Story: 2007614 (cherry picked from commit 8fd17214a97322a8a0136e50ced1d65a7d9dd75e)
* | Merge "Allow scale-down of ASG as part of update" into stable/ussuriZuul2020-07-154-39/+55
|\ \
| * | Allow scale-down of ASG as part of updateRabi Mishra2020-05-294-39/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When user wants to do manual scale-up of ASG by increasing the desired_capacity and also update some member properties as part of a stack update, if it fails due to insufficient resources when resizing, trying to manually scale-down by decreasing the desired_capacity won't work, as it would always expect to update the group with the earlier size before scaling down. This patch uses the target_size when building the batches. Task: 39867 Change-Id: Id851530b424f68b5e0e967fe34431483bfffd852 (cherry picked from commit 26d8f64fc90ed9b3ad0413ed0c932fb91b51d666)
* | | Merge "Don't check stack staus for already migrated stacks" into stable/ussuriZuul2020-07-141-4/+4
|\ \ \
| * | | Don't check stack staus for already migrated stacksRabi Mishra2020-07-031-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If stacks are already migrated to convergence, there is no point in checking stack status before returning. This will allow re-run of the command inspite of migrated stacks in FAILED state. Change-Id: Ia0e34423377843adee8efc7f23d2c2df5dac8e20 Task: 40266 (cherry picked from commit 8d5d2a8fb6145caa7aac1eb90a4787706c298e47)
* | | | Merge "Check for nova exception.Conflict rather than task_state" into ↵Zuul2020-07-143-49/+25
|\ \ \ \ | |/ / / |/| | | | | | | stable/ussuri
| * | | Check for nova exception.Conflict rather than task_stateRabi Mishra2020-07-013-49/+25
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | When detaching and attaching volumes nova would raise an exception.Conflict, if attach and detach can't happen. Let's use that to retry rather than using task_state. Change-Id: I50904e4254568cd807b9ff18eef482cadb626ce5 Task: 40169 (cherry picked from commit 01c230e793043310614d5b88096476ce0eac2c78)
* | | Use never expiring alarm_url for resource attributesRabi Mishra2020-06-256-16/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't use timestamp when generating ec2 signed url for alarm url attributes of resource. This will resolve the issue for new resources even if they are cached in attributes or resource_data. However, for existing stacks containing these resources the current best option is to not cache these attributes and allow alarm resources to be able to get a new never expiring urls in the next forced update. Change-Id: If2ebc3deacb770294004ae023500367af603b59e Task: 39985 Related-Bug: #1872737 (cherry picked from commit d0e44ded0a07cfe6b3413693d79c7f1f67b27701)
* | | Don't store signal_url for ec2 signaling of deploymentsRabi Mishra2020-06-256-12/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As part of a CVE keystone has started checking[1] the timestamp of signed ec2 token with default TTL of 15 mins. We can't store the ec2 url anymore for future use for those. This moves the caching logic to BaseWaitConditionHandle class. Conflicts: heat/engine/resources/signal_responder.py heat/tests/test_signal.py [1] https://review.opendev.org/#/c/724124/ Change-Id: I6b74faed820caccd39210bd48a212b2dedca46b9 Task: 39985 Related-Bug: #1872737 (cherry picked from commit 3047ca7d36baaa59ab2960602da956d2087a2a62)
* | | Add native grenade zuul v3 jobsLuigi Toscano2020-06-254-103/+17
|/ / | | | | | | | | | | | | | | Moved from https://review.opendev.org/#/c/639774/ Task: 40166 Change-Id: Id7f9be610e99cac9cf1f097066cc8748752d92dd (cherry picked from commit 86fc1740c405e73c094012e8d341cf9d1e71e347)
* | Fix hacking min version to 3.0.1Ghanshyam Mann2020-05-201-1/+1
|/ | | | | | | | | | | | | | | | | | flake8 new release 3.8.0 added new checks and gate pep8 job start failing. hacking 3.0.1 fix the pinning of flake8 to avoid bringing in a new version with new checks. Though it is fixed in latest hacking but 2.0 and 3.0 has cap for flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also break the pep8 job if new check are added. To avoid similar gate break in future, we need to bump the hacking min version. - http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html Change-Id: I4586bde1bd9f33dcdd803900bc648720073bdb62 (cherry picked from commit a61c4bbe29b4df6d00b74316d4b650727b15381d)
* Merge "Update TOX_CONSTRAINTS_FILE for stable/ussuri" into stable/ussuriZuul2020-05-061-2/+2
|\
| * Update TOX_CONSTRAINTS_FILE for stable/ussuriOpenStack Release Bot2020-04-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | 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/ussuri branch, tests will continue to use the upper-constraints list on master. Change-Id: I6b4f94d7345e85f250882eb46d72e5a50df89665
* | Merge "Update .gitreview for stable/ussuri" into stable/ussuriZuul2020-05-061-0/+1
|\ \ | |/
| * Update .gitreview for stable/ussuriOpenStack Release Bot2020-04-271-0/+1
| | | | | | | | Change-Id: Id728aa5abfa9f0f2f1bfd22bcb1706785301fd6d
* | Imported Translations from ZanataOpenStack Proposal Bot2020-04-285-3283/+8
|/ | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Idb25c2542e9e30bb7dcf0c304d7803ce37698f62
* Merge "New resource OS::Neutron::QoSMinimumBandwidthRule"14.0.0.0rc114.0.0Zuul2020-04-274-1/+221
|\
| * New resource OS::Neutron::QoSMinimumBandwidthRuleBence Romsics2020-04-274-1/+221
| | | | | | | | | | | | | | | | The 'minimum_bandwidth_rule' QoS rule type of Neutron (supported in both egress and ingress directions since the Stein release) did not have a Heat resource yet. This change adds it. Change-Id: I693fe2f7801f78f827ef1c74e5874018cd9cf51b
* | Merge "Add Ironic Port resource type support"Zuul2020-04-274-0/+519
|\ \
| * | Add Ironic Port resource type supportricolin2020-04-264-0/+519
| | | | | | | | | | | | | | | Change-Id: If6b1d77d055b9a3803e50563a4f41c2dcdcb1f31 Task: 36286
* | | Merge "Remove six and python 2.7 full support"Zuul2020-04-2710-29/+16
|\ \ \ | |_|/ |/| |
| * | Remove six and python 2.7 full supportHervé Beraud2020-04-2310-29/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Six is in use to help us to keep support for python 2.7. Since the ussuri cycle we decide to remove the python 2.7 support so we can go ahead and also remove six usage from the python code. Review process and help ----------------------- Removing six introduce a lot of changes and an huge amount of modified files To simplify reviews we decided to split changes into several patches to avoid painful reviews and avoid mistakes. To review this patch you can use the six documentation [1] to obtain help and understand choices. Additional informations ----------------------- Changes related to 'six.b(data)' [2] ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ six.b [2] encode the given datas in latin-1 in python3 so I did the same things in this patch. Latin-1 is equal to iso-8859-1 [3]. This encoding is the default encoding [4] of certain descriptive HTTP headers. I suggest to keep latin-1 for the moment and to move to another encoding in a follow-up patch if needed to move to most powerful encoding (utf8). HTML4 support utf8 charset and utf8 is the default charset for HTML5 [5]. Note that this commit message is autogenerated and not necesserly contains changes related to 'six.b' [1] https://six.readthedocs.io/ [2] https://six.readthedocs.io/#six.b [3] https://docs.python.org/3/library/codecs.html#standard-encodings [4] https://www.w3schools.com/charsets/ref_html_8859.asp [5] https://www.w3schools.com/html/html_charset.asp Patch 10 of a serie of 28 patches Change-Id: Ic745ceced9989497c489f3df3ee9ff84fe507b88
* | | Merge "Mark OS::Neutron::QoSDscpMarkingRule as admin only"Zuul2020-04-271-0/+3
|\ \ \
| * | | Mark OS::Neutron::QoSDscpMarkingRule as admin onlyricolin2020-04-261-0/+3
| | |/ | |/| | | | | | | | | | | | | | | | Update the policy for admin only resource type and added in ``OS::Neutron::QoSDscpMarkingRule`` as other OS::Neutron::QoS* are. Change-Id: I0f0fa5f92816886b10f07e6d093e7bdeb58bc3f6
* | | Merge "Remove six and python 2.7 full support"Zuul2020-04-2610-38/+26
|\ \ \ | | |/ | |/|
| * | Remove six and python 2.7 full supportHervé Beraud2020-04-2310-38/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Six is in use to help us to keep support for python 2.7. Since the ussuri cycle we decide to remove the python 2.7 support so we can go ahead and also remove six usage from the python code. Review process and help ----------------------- Removing six introduce a lot of changes and an huge amount of modified files To simplify reviews we decided to split changes into several patches to avoid painful reviews and avoid mistakes. To review this patch you can use the six documentation [1] to obtain help and understand choices. Additional informations ----------------------- Changes related to 'six.b(data)' [2] ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ six.b [2] encode the given datas in latin-1 in python3 so I did the same things in this patch. Latin-1 is equal to iso-8859-1 [3]. This encoding is the default encoding [4] of certain descriptive HTTP headers. I suggest to keep latin-1 for the moment and to move to another encoding in a follow-up patch if needed to move to most powerful encoding (utf8). HTML4 support utf8 charset and utf8 is the default charset for HTML5 [5]. Note that this commit message is autogenerated and not necesserly contains changes related to 'six.b' [1] https://six.readthedocs.io/ [2] https://six.readthedocs.io/#six.b [3] https://docs.python.org/3/library/codecs.html#standard-encodings [4] https://www.w3schools.com/charsets/ref_html_8859.asp [5] https://www.w3schools.com/html/html_charset.asp Patch 9 of a serie of 28 patches Change-Id: I89ec972a566ce211b5de4b02ca4d9b1b6e178525
* | | Merge "Remove six and python 2.7 full support"Zuul2020-04-2610-37/+21
|\ \ \ | |/ /
| * | Remove six and python 2.7 full supportHervé Beraud2020-04-2310-37/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Six is in use to help us to keep support for python 2.7. Since the ussuri cycle we decide to remove the python 2.7 support so we can go ahead and also remove six usage from the python code. Review process and help ----------------------- Removing six introduce a lot of changes and an huge amount of modified files To simplify reviews we decided to split changes into several patches to avoid painful reviews and avoid mistakes. To review this patch you can use the six documentation [1] to obtain help and understand choices. Additional informations ----------------------- Changes related to 'six.b(data)' [2] ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ six.b [2] encode the given datas in latin-1 in python3 so I did the same things in this patch. Latin-1 is equal to iso-8859-1 [3]. This encoding is the default encoding [4] of certain descriptive HTTP headers. I suggest to keep latin-1 for the moment and to move to another encoding in a follow-up patch if needed to move to most powerful encoding (utf8). HTML4 support utf8 charset and utf8 is the default charset for HTML5 [5]. Note that this commit message is autogenerated and not necesserly contains changes related to 'six.b' [1] https://six.readthedocs.io/ [2] https://six.readthedocs.io/#six.b [3] https://docs.python.org/3/library/codecs.html#standard-encodings [4] https://www.w3schools.com/charsets/ref_html_8859.asp [5] https://www.w3schools.com/html/html_charset.asp Patch 8 of a serie of 28 patches Change-Id: If54d634abfd345c1e672d31cb11de14260b34cc0
* | | Merge "Reduce Heat engine workers"Zuul2020-04-261-0/+4
|\ \ \
| * | | Reduce Heat engine workersCédric Ollivier2020-04-151-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It conforms with API_WORKERS default calculation to avoid too much processes consuming the memory. Change-Id: If2b483711668715047662a286cb0f0e3b52bbdac Signed-off-by: Cédric Ollivier <ollivier.cedric@gmail.com>
View Lorry Controller Status