| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-05-24 | |||
| 10:46:41 | gtema | dtantsur: is there a chance you can try whether https://review.opendev.org/c/openstack/openstacksdk/+/816790 helps? | |
| 10:47:01 | dtantsur | gtema: will try after the painful meeting I'm in.. | |
| 10:47:08 | gtema | :) | |
| 10:47:14 | gtema | a typical Tuesday | |
| 10:47:19 | dtantsur | I'll probably need some booze after it, dunno | |
| 10:48:14 | dtantsur | gtema: I wonder if line 180 should be changed too | |
| 10:48:38 | gtema | yeah, may be | |
| 10:57:40 | priteau | Hello. We've started to see failures uploading images to Glance through Ansible following the release of openstacksdk 0.99.0. It seems to be fixed with ansible-collections-openstack from master, but not with the latest release 1.8.0. | |
| 10:58:02 | priteau | 400 Bad Request\n\nProvided object does not match schema 'image': 'False' is not of type 'boolean' Failed validating 'type' in schema['properties']['protected']: {'description': 'If true, image will not be deletable.', 'type': 'boolean'} On instance['protected']: 'False'\n\n | |
| 10:58:11 | priteau | Is a new collection release planned? | |
| 10:58:40 | frickler | priteau: iiuc you need to cap openstacksdk<0.99 currently. new collection is being worked on, but not ready yet | |
| 10:58:47 | gtema | you should be asking this in another room #openstack-ansible-sig | |
| 10:59:02 | gtema | yes, there is a new collection version, but the work there is not complete | |
| 10:59:03 | priteau | Sorry, just noticed this is the wrong channel for htis | |
| 11:00:18 | frickler | oh, I didn't know about that channel either | |
| 12:08:23 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update catalog service for the new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839352 | |
| 12:18:04 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update endpoint to new proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840640 | |
| 12:22:12 | frickler | os-client-config also needs a py38 fix https://zuul.opendev.org/t/openstack/build/caa6cdd3f37449178759671730329503 guess that's what we get for slacking so long | |
| 12:23:03 | gtema | the whole thing depresses. So many things broke at once | |
| 12:27:13 | frickler | yeah, maybe not the best timing with the sdk release. otoh fixing it all in one go isn't that bad, either | |
| 12:27:57 | gtema | I wouldnt say sdk release was bad timing. I mean so much things broke due to oslo.log + py36/37 drop | |
| 12:28:26 | gtema | and neither SDK nor OSC have real problems with py36 | |
| 12:28:45 | gtema | and we "should" stay backward compatible | |
| 12:30:43 | opendevreview | Dr. Jens Harbott proposed openstack/os-client-config master: Fix zuul jobs https://review.opendev.org/c/openstack/os-client-config/+/843136 | |
| 12:31:42 | frickler | ^^ not sure if it is a good idea to do all these changes at once, will see what zuul says | |
| 12:37:41 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Raise minimum OpenStack SDK version to 0.99.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839691 | |
| 12:48:22 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Lowered maximum OpenStack SDK version to 0.98.999 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839693 | |
| 13:04:08 | frickler | more regressions :( https://zuul.opendev.org/t/openstack/build/31e084f8cb20412a89c62681da712b04 https://zuul.opendev.org/t/openstack/build/4f7468d7526b4b53a08c4d0aaf00eea8 | |
| 13:34:57 | opendevreview | Dr. Jens Harbott proposed openstack/os-client-config master: Fix zuul jobs https://review.opendev.org/c/openstack/os-client-config/+/843136 | |
| 13:50:42 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Updated pip constraints for release candidates of OpenStackSDK's first major release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842946 | |
| 15:22:53 | opendevreview | Jan Horstmann proposed openstack/ansible-collections-openstack master: Return details in baremetal_node_info when iterating over all machines https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839776 | |
| 17:55:09 | opendevreview | Merged openstack/ansible-collections-openstack master: Raise minimum OpenStack SDK version to 0.99.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839691 | |
| 18:36:06 | frickler | gtema: stephenfin: https://review.opendev.org/c/openstack/os-client-config/+/843136 to hopefully unbreak sdk gate. I hope you agree with skipping shade jobs and deferring fixing those until the major issues are resolved | |
| 18:42:20 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Lowered maximum OpenStack SDK version to 0.98.999 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839693 | |
| 18:55:07 | eandersson | is the openstacksdk >0.99 meant to be part of Zed? | |
| 19:01:18 | frickler | eandersson: I think so, at least the release has been made under the zed tree in the releases repo | |
| 19:04:35 | jm1 | frickler: whuat? 😱 i thought its going to be part of zed+1? then we have to speed up porting ansible openstack collection.. somehow.. | |
| 19:10:25 | frickler | jm1: well the authoritative answer will have to come from gtema, but I don't see how we could still release an older version again for zed | |
| 19:20:02 | eandersson | I bumped senlin requirements to only use >=0.99 for zed | |
| 19:20:20 | eandersson | but realized that it might cause issues if not all projects support 0.99 | |
| 19:20:51 | eandersson | https://github.com/openstack/senlin/blob/master/requirements.txt#L13 | |
| 19:23:49 | jm1 | frickler: ack, maybe a release with zed would bring some new attention to aoc 🙈 | |
| 19:29:13 | stephenfin | frickler: done | |
| 21:16:24 | opendevreview | Merged openstack/os-client-config master: Fix zuul jobs https://review.opendev.org/c/openstack/os-client-config/+/843136 | |
| #openstack-sdks - 2022-05-25 | |||
| 00:45:51 | opendevreview | Ronelle Landy proposed openstack/ansible-collections-openstack master: Revert "Raise minimum OpenStack SDK version to 0.99.0" https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843192 | |
| 06:12:22 | opendevreview | Merged openstack/python-openstackclient master: Migrate osc-tox-py3N-tips to Python 3.8 https://review.opendev.org/c/openstack/python-openstackclient/+/843115 | |
| 06:33:55 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Revert "Raise minimum OpenStack SDK version to 0.99.0" https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843238 | |
| 06:34:42 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Revert "Raise minimum OpenStack SDK version to 0.99.0" https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843192 | |
| 09:08:30 | opendevreview | Merged openstack/ansible-collections-openstack master: Revert "Raise minimum OpenStack SDK version to 0.99.0" https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843192 | |
| 09:12:21 | opendevreview | Merged openstack/openstacksdk master: Fix python-dev reference in bindep https://review.opendev.org/c/openstack/openstacksdk/+/843124 | |
| 09:51:50 | opendevreview | Merged openstack/cliff master: Migrate Python 3.6/7 jobs to Python 3.8 https://review.opendev.org/c/openstack/cliff/+/843123 | |
| 11:55:52 | opendevreview | Takashi Kajinami proposed openstack/os-service-types master: Add Python 3.8 and 3.9 to supported runtimes https://review.opendev.org/c/openstack/os-service-types/+/843266 | |
| 12:25:30 | opendevreview | Merged openstack/openstacksdk master: Rework caching https://review.opendev.org/c/openstack/openstacksdk/+/837802 | |
| 13:52:34 | frickler | gtema: ^^ as I mentioned in the review, do you think it is worth doing a 0.99.1 release with the new caching? or give it a few days first? or do you have something else in the queue you'd want to wait for? | |
| 13:53:36 | gtema | we can give it a try. I wanted overall to gather some more feedback on where it is painful (as I see now in Ansible/TripleO/Bifrost/etc) | |
| 13:53:47 | gtema | but no problem making 0.99.1 | |
| 14:07:49 | opendevreview | Jan Horstmann proposed openstack/ansible-collections-openstack master: Return details in baremetal_node_info when iterating over all machines https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839776 | |
| 16:18:29 | opendevreview | Stephen Finucane proposed openstack/cliff master: requirements: Remove explicit pbr dependency https://review.opendev.org/c/openstack/cliff/+/842346 | |
| 16:18:29 | opendevreview | Stephen Finucane proposed openstack/cliff master: requirements: Remove explicit pyparsing dependency https://review.opendev.org/c/openstack/cliff/+/842345 | |
| 16:18:30 | opendevreview | Stephen Finucane proposed openstack/cliff master: Defer loading cmd2 https://review.opendev.org/c/openstack/cliff/+/842348 | |
| 16:18:30 | opendevreview | Stephen Finucane proposed openstack/cliff master: requirements: Remove explicit python-subunit dependency https://review.opendev.org/c/openstack/cliff/+/842347 | |
| 16:18:31 | opendevreview | Stephen Finucane proposed openstack/cliff master: Remove final use of pkg_resources https://review.opendev.org/c/openstack/cliff/+/842350 | |
| 16:18:31 | opendevreview | Stephen Finucane proposed openstack/cliff master: Defer loading PyYAML https://review.opendev.org/c/openstack/cliff/+/842349 | |
| 16:18:32 | opendevreview | Stephen Finucane proposed openstack/cliff master: Update Python testing per Zed cycle testing runtime https://review.opendev.org/c/openstack/cliff/+/843312 | |
| 17:38:49 | eandersson | gtema, frickler: yea please make a 0.99.1 so we can test it with Senlin | |
| 18:01:25 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds mechanisms to extend OpenstackModule behaviors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843324 | |
| 18:01:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_node_info to use new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843325 | |
| 18:03:56 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_node_info to use new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843325 | |
| 18:07:23 | opendevreview | Merged openstack/cliff stable/train: [Train-Only] Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826876 | |
| 18:07:25 | opendevreview | Merged openstack/cliff master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/cliff/+/787931 | |
| 18:48:08 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_inspect to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843334 | |
| 18:55:05 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds mechanisms to extend OpenstackModule behaviors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843324 | |
| 19:01:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_inspect to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843334 | |
| 19:17:57 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_inspect to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843334 | |
| #openstack-sdks - 2022-05-26 | |||
| 04:37:13 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_inspect to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843334 | |
| 05:33:41 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update baremetal_inspect to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843334 | |
| 08:45:43 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Warn users about us breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843419 | |
| 10:02:04 | stephenfin | gtema: A lot of the cloud layer functions take a 'filters' argument, and the docstrings for this suggest you can use jmespath expressions to generate the filters. However, it seems this functionality has been lost in the move to using the proxy. What do we want to do? | |
| 10:02:38 | stephenfin | I think we can either update the docstring to reflect the change in behavior, or modify all those functions to re-add this capability. The former is easier, the latter is better if people actually use this functionality | |
| 10:15:21 | jm1 | stephenfin: some cloud layer functions still use the filters argument to postprocess results, i.e. identity search_* functions such as search_users() still use _utils._filter_list() | |
| 10:15:48 | stephenfin | Yeah, I assume that is an oversight though and those will change | |
| 10:16:29 | jm1 | stephenfin: we had some discussions about that some time ago and i think gtema did not yet decide on that yet. in the end someone has to (re)implement them | |
| 10:17:23 | jm1 | stephenfin: we use filters in a lot of *_info modules in ansible openstack collection and thus would like to continue using them to break less use cases | |
| 10:18:05 | jm1 | stephenfin: gtema has public holiday today and will be back tomorrow ;) | |
| 10:20:47 | stephenfin | jm1: Thanks for the info (on both fronts) | |
| 12:00:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for network functions https://review.opendev.org/c/openstack/openstacksdk/+/843435 | |
| 12:00:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove unused normalization helpers https://review.opendev.org/c/openstack/openstacksdk/+/843434 | |
| 12:00:08 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for image functions https://review.opendev.org/c/openstack/openstacksdk/+/843437 | |
| 12:00:08 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for accelerator functions https://review.opendev.org/c/openstack/openstacksdk/+/843436 | |
| 12:00:09 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for identity functions https://review.opendev.org/c/openstack/openstacksdk/+/843439 | |
| 12:00:09 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for block storage functions https://review.opendev.org/c/openstack/openstacksdk/+/843438 | |
| 12:00:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for object store functions https://review.opendev.org/c/openstack/openstacksdk/+/843440 | |
| 12:00:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for compute functions https://review.opendev.org/c/openstack/openstacksdk/+/843441 | |
| 12:06:13 | gtema | My garden shed is disassembled, pause before preparing foundation for the new one. Repeating for myself about filters: | |
| 12:06:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add support for os-simple-tenant-usages API https://review.opendev.org/c/openstack/openstacksdk/+/843442 | |
| 12:06:37 | gtema | We should try implementing filters deep in base proxy list function | |
| 12:24:00 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add support for instance actions https://review.opendev.org/c/openstack/openstacksdk/+/843449 | |
| 12:24:00 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Correct some docstrings https://review.opendev.org/c/openstack/openstacksdk/+/843448 | |
| 12:24:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: compute: Add functional tests for server events https://review.opendev.org/c/openstack/openstacksdk/+/843450 | |
| 12:50:26 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | |