summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge "Set context username if session client is used"15.0.0.0rc115.0.0Zuul2020-09-201-0/+2
|\
| * Set context username if session client is usedAndrii Ostapenko2020-07-061-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Now context username is set from X-Auth-User header, that is not set with session client. This results in stack_owner attribute None for newly created stacks. This commit sets context username from auth token in case of absent X-Auth-User header. Affects all releases at least since Ocata. Story: 2007856 Change-Id: I9cba3495fd472769498486d52d3909bff03010d4
* | Stop testing ceilometer and aodhRabi Mishra2020-09-142-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ceilometer uses gnocchi as the default backend. Also we use gnocchi based aodh alarms in tests. gnocchi seems unmaintained with last commit a year or so ago and does not look like the openstack telemetry team is involved in that project. It's better to disable the services and tests in heat to avoid broken gate like last time, where we fixed it by banning latest pecan release[1] that does not work with python-gnocchiclient. [1] https://review.opendev.org/#/c/746261/ Change-Id: Id2ffdf6b9d342e800bab4a94ec46742228361ee8
* | Fix lower-constraints errorsEmilien Macchi2020-09-104-22/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch changes the lower-constraint requirements to make them py3.8 compatible. See https://bugs.launchpad.net/nova/+bug/1886298 Markupsafe==1.1.1 PyMySQL==0.8.0 PyYAML==3.13 cffi==1.14.0 greenlet==0.4.15 kombu==5.0.1 lxml==4.5.0 oslo.db==6.0.0 paramiko==2.7.1 psycopg2==2.8 sqlalchemy-migrate==0.13.0 tenacity==6.1.0 Also update the test-setup.sh to work with recent version of mysql. Change-Id: Ibe60d1e536f0e887eceea32b7cb4eec6318e96af Partial-Bug: #1886298
* | Merge "Lazily cache parsed value of list/json parameters"Zuul2020-09-012-22/+21
|\ \
| * | Lazily cache parsed value of list/json parametersZane Bitter2020-08-282-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. Depends-On: https://review.opendev.org/#/c/748359/ Change-Id: I70e341c344d6254173ad4519276626230087263a Story: 2007957 Task: 40443
* | | Merge "Bump MANILACLIENT_VERSION and use sharev2 endpoint"Zuul2020-08-254-6/+35
|\ \ \
| * | | Bump MANILACLIENT_VERSION and use sharev2 endpointJan Horstmann2020-08-254-6/+35
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit bumps MANILACLIENT_VERSION to 2.13 and changes the used service type to sharev2. In order to support the value of "cephx" in property '{"access_rules": [{"access_type": ""}]}' in ressource OS::Manila::Share, manilaclient needs to use at least version 2.13 of the manila API ([1]). The default minimum version in manilaclient, which is used when only a mayor version is specified is 2.0. Additionally the sharev2 service type endpoint has to be used to access the manila v2 API. The export_locations response was removed in version 2.9 of manila API from the "Show share details" request ([2]) and moved to its own endpoint in [3]. Thus it is requested from there now. Additionally the new endpoint is more verbose, so only its path attribute is returned, in order to match the previous behaviour. [1] https://docs.openstack.org/manila/latest/contributor/api_microversion_history.html [2] https://docs.openstack.org/api-ref/shared-file-system/?expanded=show-single-export-location-detail,show-share-details-detail#show-share-details [3] https://docs.openstack.org/api-ref/shared-file-system/?expanded=show-single-export-location-detail,list-export-locations-detail#list-export-locations Change-Id: I4c37be8fad1edb05d812fed260e97e9188fd23ce Story: 2007986 Task: 40612
* | | Merge "Assigning resource_id to resource OS::Cinder::QoSAssociation"Zuul2020-08-241-0/+1
|\ \ \
| * | | Assigning resource_id to resource OS::Cinder::QoSAssociationArun Vinod2020-07-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, the QoSAssociation resource is being created without a resource_id. This raises an Exception while peforming stack_adopt on such resources, since adopt requires resource_id for all resources. The proposed approach is to set uuid to QoSAssociation as well. Change-Id: Ic4c50b36d40592e1159827376ef78eaa86c31b53 Task: #39994 Story: #39994
* | | | Merge "Correct release version for OS::Neutron::ProviderNet dns_domain"Zuul2020-08-191-1/+1
|\ \ \ \
| * | | | Correct release version for OS::Neutron::ProviderNet dns_domainZane Bitter2020-08-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The patch adding this new property was approved during the Ussuri release cycle, but only merged in the Victoria release. Therefore the version in the SupportStatus should be 15.0.0, not 14.0.0. Change-Id: I1e825834049f47a470280cce6c8bd89ef7e9ae61
* | | | | Merge "Add 'smart-nic' vnic type support for OS::Neutron::Port"Zuul2020-08-172-1/+6
|\ \ \ \ \
| * | | | | Add 'smart-nic' vnic type support for OS::Neutron::PortRabi Mishra2020-08-132-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I91f63810626ce4e054e358f5de5e46434c4da131 added it in neutron. Task: 40436 Depends-On: https://review.opendev.org/746034 Change-Id: If04e080d2e399795d501d0317196589dcbd934e5
* | | | | | Merge "Fix OS::Heat::Delay resource type"Zuul2020-08-172-3/+7
|\ \ \ \ \ \
| * | | | | | Fix OS::Heat::Delay resource typeZane Bitter2020-08-132-3/+7
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't override ResourceStatus.ACTIONS, as it's used internally by the Resource class. Change-Id: Icf306e981c19e3d5410905befa7907dcdba33337 Story: 2008013 Task: 40663
* | | | | | Merge "Supports 'virtio-forwarder' vnic type for port"Zuul2020-08-152-1/+7
|\ \ \ \ \ \ | |/ / / / / |/| / / / / | |/ / / /
| * | | | Supports 'virtio-forwarder' vnic type for portEmilien Macchi2020-07-242-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
* | | | | Add U + V release codenames for docs generationZane Bitter2020-08-121-14/+18
| |_|/ / |/| | | | | | | | | | | | | | | | | | | This makes for nicer output in the resource types docs. Change-Id: I5c7b0e7454df0c5dd33efb03c08d15ea231bf530
* | | | Merge "Eliminate Master/Slave terminology from Designate Zone resource"Zuul2020-07-243-6/+125
|\ \ \ \ | |/ / / |/| | |
| * | | Eliminate Master/Slave terminology from Designate Zone resourceZane Bitter2020-07-143-6/+125
| | |/ | |/| | | | | | | Change-Id: If136f5e270db5df1f871dc9473afb178e242bc18
* | | Merge "Rename variables in sort key validation"Zuul2020-07-202-10/+10
|\ \ \
| * | | Rename variables in sort key validationZane Bitter2020-07-142-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | Avoid unnecessarily divisive terms. Change-Id: Ida9384c849cd664c5fc5d31449350d7cb6a68b78
* | | | Merge "Improve naming of get_allowed_params() argument"Zuul2020-07-2010-62/+62
|\ \ \ \ | |/ / /
| * | | Improve naming of get_allowed_params() argumentZane Bitter2020-07-1410-62/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This used to be just a list of allowed names, but now it's a dict mapping names to types. Use variable names that reflect the current meaning, fix the docs, and use named constants where available. Change-Id: I3aadca4e1e9db5da5d07d521c2313162062639b7
* | | | Merge "Use 'skiplist' to describe skipped indices in ResourceGroup code"Zuul2020-07-203-86/+86
|\ \ \ \ | |/ / /
| * | | Use 'skiplist' to describe skipped indices in ResourceGroup codeZane Bitter2020-07-143-86/+86
| |/ / | | | | | | | | | | | | | | | Don't use needlessly divisive terminology. Change-Id: I86a06cd2c5cebcc3d2cd07b675f626a917bb9db4
* | | Merge "Use correct attribute from entrypoint"Zuul2020-07-163-3/+3
|\ \ \ | |/ / |/| |
| * | Use correct attribute from entrypointRabi Mishra2020-07-153-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The type of the entry point objects returned in stevedore has been changed from`pkg_resources.EntryPoint` to `importlib.metadata.EntryPoint`[1] and it does not have module_name attribute. It also bumps requirements/lower-constraints as stevedore==3.1.0 is not backward compatible. [1] https://review.opendev.org/#/c/740515/ Task: 40355 Change-Id: Icd68f6eab73e44406f2ff5073636dcdd111e00ea
* | | Merge "Deprecate wrappertask decorator"Zuul2020-07-066-91/+64
|\ \ \
| * | | Deprecate wrappertask decoratorZane Bitter2020-04-306-91/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we only support Python 3, use the 'yield from' keyword instead of the @wrappertask decorator for calling co-routines from a co-routine. This should greatly simplify backtraces when calling nested co-routines. Change-Id: If9beaff74cf4facbc4aa4b30f31a3a087bdcad8a
* | | | Don't check stack staus for already migrated stacksRabi Mishra2020-07-021-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
* | | Merge "Use unittest.mock instead of mock"Zuul2020-07-012-2/+2
|\ \ \
| * | | Use unittest.mock instead of mockHervé Beraud2020-06-102-2/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | 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: Ie0e43904d375e542ca00cd43b3c84c7afb31ee85
* | | Check for nova exception.Conflict rather than task_stateRabi Mishra2020-06-263-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
* | | Imported Translations from ZanataOpenStack Proposal Bot2020-06-171-4/+4
|/ / | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: Ia0a23d75ec36fd5d729b64e4bcc63edafdb52cba
* | Use never expiring alarm_url for resource attributesRabi Mishra2020-06-066-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
* | Don't store signal_url for ec2 signaling of deploymentsRabi Mishra2020-06-046-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. [1] https://review.opendev.org/#/c/724124/ Change-Id: I6b74faed820caccd39210bd48a212b2dedca46b9 Related-Bug: #1872737
* | Imported Translations from ZanataOpenStack Proposal Bot2020-05-291-7/+30
| | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I4a51049476956eb9d823ab1cf73cfc0175c37f5a
* | Allow scale-down of ASG as part of updateRabi Mishra2020-05-284-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
* | Switch to newer openstackdocstheme and reno versionsAndreas Jaeger2020-05-214-16/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Switch to openstackdocstheme 2.2.1 and reno 3.1.0 versions. Using these versions will allow especially: * Linking from HTML to PDF document * Allow parallel building of documents * Fix some rendering problems Update Sphinx version as well. Set openstackdocs_pdf_link to link to PDF file. Note that the link to the published document only works on docs.openstack.org where the PDF file is placed in the top-level html directory. The site-preview places the PDF in a pdf directory. Change pygments_style to 'native' since old theme version always used 'native' and the theme now respects the setting and using 'sphinx' can lead to some strange rendering. openstackdocstheme renames some variables, so follow the renames before the next release removes them. A couple of variables are also not needed anymore, remove them. See also http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014971.html Change-Id: I4ebb6e663471a7096bd9d0e1a90d8114d1875546
* | Merge "Add propagate_uplink_status support to OS::Neutron::Port"Zuul2020-05-203-3/+66
|\ \
| * | Add propagate_uplink_status support to OS::Neutron::PortJason Li2020-05-063-3/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This resource depends on Neutron API extension ``uplink-status-propagation`` and the default is ``False``. If this property is set to ``True``, the VF link state can follow that of PF. See release note at https://docs.openstack.org/releasenotes/neutron/stein.html. Change-Id: I46749f0458fd69e3f62d03e5b4648e1704750e15
* | | Add native grenade zuul v3 jobsLuigi Toscano2020-05-134-103/+17
| | | | | | | | | | | | | | | | | | Moved from https://review.opendev.org/#/c/639774/ Change-Id: Id7f9be610e99cac9cf1f097066cc8748752d92dd
* | | Fix hacking min version to 3.0.1Ghanshyam Mann2020-05-121-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
* | | Use Fedora 31 for testsRabi Mishra2020-05-125-7/+7
| | | | | | | | | | | | Change-Id: Ia45e957bb24f5596a5ef16271f71ba8b77df9e42
* | | Fix pep8 E741 errorsRabi Mishra2020-05-124-7/+7
| | | | | | | | | | | | Change-Id: Ibb0da11fd89876602a6da62d08834937b81c2ce6
* | | Merge "Return empty list instead of None for tags"Zuul2020-05-112-2/+2
|\ \ \
| * | | Return empty list instead of None for tagsZane Bitter2019-10-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a stack has no tags, return an empty list [] instead of None for its tags list. Change-Id: I3c3c083d7261172d08b933de7c63b6a489ed6ccd Depends-On: https://review.opendev.org/687769 Task: 37009
* | | | Merge "Monkey patch original current_thread _active"Zuul2020-05-104-0/+24
|\ \ \ \
View Lorry Controller Status