| 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 | 04:37 |
|---|---|---|
| *** diablo_rojo_phone is now known as Guest360 | 05:08 | |
| *** tkajinam_ is now known as tkajinam | 05:11 | |
| 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 |
| 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 | 08:45 |
| 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 |
| 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:02 |
| 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 |
| stephenfin | Yeah, I assume that is an oversight though and those will change | 10:15 |
| 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:16 |
| 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:17 |
| jm1 | stephenfin: gtema has public holiday today and will be back tomorrow ;) | 10:18 |
| stephenfin | jm1: Thanks for the info (on both fronts) | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove unused normalization helpers https://review.opendev.org/c/openstack/openstacksdk/+/843434 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for network functions https://review.opendev.org/c/openstack/openstacksdk/+/843435 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for accelerator functions https://review.opendev.org/c/openstack/openstacksdk/+/843436 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for image functions https://review.opendev.org/c/openstack/openstacksdk/+/843437 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for block storage functions https://review.opendev.org/c/openstack/openstacksdk/+/843438 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for identity functions https://review.opendev.org/c/openstack/openstacksdk/+/843439 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for object store functions https://review.opendev.org/c/openstack/openstacksdk/+/843440 | 12:00 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Update docstrings for compute functions https://review.opendev.org/c/openstack/openstacksdk/+/843441 | 12:00 |
| gtema | My garden shed is disassembled, pause before preparing foundation for the new one. Repeating for myself about filters: | 12:06 |
| 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 |
| gtema | We should try implementing filters deep in base proxy list function | 12:06 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Correct some docstrings https://review.opendev.org/c/openstack/openstacksdk/+/843448 | 12:24 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add support for instance actions https://review.opendev.org/c/openstack/openstacksdk/+/843449 | 12:24 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: compute: Add functional tests for server events https://review.opendev.org/c/openstack/openstacksdk/+/843450 | 12:24 |
| *** mnaser_ is now known as mnaser | 12:41 | |
| 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 | 12:50 |
| *** artom_ is now known as artom | 14:04 | |
| *** Guest360 is now known as diablo_rojo_phone | 15:00 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!