opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_project https://review.opendev.org/c/openstack/openstacksdk/+/857936 | 06:43 |
---|---|---|
opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_project https://review.opendev.org/c/openstack/openstacksdk/+/857936 | 06:44 |
opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_server https://review.opendev.org/c/openstack/openstacksdk/+/857936 | 06:45 |
opendevreview | Denys Mishchenko proposed openstack/ansible-collections-openstack master: search_images replaced with image.images https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862518 | 06:56 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | 08:01 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | 08:01 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | 08:02 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | 08:11 |
opendevreview | Denys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565 | 08:56 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | 09:30 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | 09:31 |
opendevreview | Denys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565 | 09:52 |
opendevreview | HaeIYoon proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | 12:38 |
dannosliwcd | As part of the nova-gaps project topic, I'm making the implementation of `server migration list` use the SDK to get migrations. But the SDK compute proxy's `migrations()` function doesn't accept any input arguments, which prevents me from using any of the filters. Is there another way to handle that, or does the `migrations()` function need to be updated? | 12:51 |
gtema | sdk need to be updated | 12:52 |
dannosliwcd | Okay. I’ll look into making that change too | 13:10 |
servagem | Hi, Clould someone please review this patch? Thanks https://review.opendev.org/c/openstack/python-openstackclient/+/862513 | 16:31 |
noonedeadpunk | gtema: hey! I'm quite ashamed but on PTG I was side-pinged and missed beginning of the discussion regarding search_xxx - payed attention only starting from words - "it affects only ansible collections"... So basically right now we're working on implementation of roles for images lifecycle. And zuul folks were also interested in this approach as contrary to nodepool-builder | 17:48 |
noonedeadpunk | but eventually faced this topic and how to search for images with new sdk/collections | 17:48 |
noonedeadpunk | Basically we (under we I mean arddennis) have some time to help out on implementation, but would be great if you could somehow point to the right direction | 17:49 |
noonedeadpunk | I'm not sure there was a decision made on what direction to move (either implement search logic or a generic search_resource) but generic search sound like a good idea at the moment comparing to moving logic to collections (or at least more clean one) | 17:54 |
noonedeadpunk | But I think it's worth syncing efforts so that we won't dig into the wrong direction | 17:58 |
noonedeadpunk | (or the one you're trying to get rid of for quite a while now) | 17:59 |
opendevreview | Violet Kurtz proposed openstack/python-openstackclient master: Moved hypervisor to the SDK https://review.opendev.org/c/openstack/python-openstackclient/+/858366 | 18:01 |
opendevreview | Merged openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | 18:05 |
opendevreview | Denys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc https://review.opendev.org/c/openstack/ansible-collections-openstack/+/862565 | 18:10 |
gtema | noonedeadpunk: so you stick to Ansible usecase or you are searching for also regular SDK interface for that? | 18:15 |
noonedeadpunk | gtema: ansible usecase now | 18:32 |
noonedeadpunk | so for rotation we need to get images with specific tags/properties and verify them. So classic search basically | 18:35 |
noonedeadpunk | verify in terms to check if they're used by any server or not and if not delete them, otherwise hide | 18:39 |
noonedeadpunk | writing it makes me think how optimal end result would be though... as sounds as quite few loops which ansible not good at | 18:45 |
noonedeadpunk | but I want it to be native zuul job in periodic pipeline (for internal zuul), so it's either just ansible or make ansible module out of this which won't be much better as we still will face earch I believe | 18:47 |
noonedeadpunk | *face search | 18:47 |
gtema | I would rather create special module packing all functionality | 18:50 |
noonedeadpunk | and move search logic to it? | 18:50 |
gtema | Not that, i mean all iterations | 18:51 |
noonedeadpunk | well... several things here why I thought that plain ansible would work - this can be re-used in many places and help other operators to get fresh images when needed. In OSA we also mess up with images for octavia/magnum/trove/etc. So we can easily re-use that | 18:53 |
noonedeadpunk | next is permissions - most of oprations won't require admin, but rotation is. So we can provide different credentials to some tasks/roles | 18:54 |
noonedeadpunk | and actually help out and get collection better :) | 18:55 |
noonedeadpunk | but yeah, I do understand why such module would be faster/easier. | 18:57 |
noonedeadpunk | just how re-usable it would be (likely not) | 18:58 |
opendevreview | Merged openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | 19:55 |
opendevreview | Merged openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | 20:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!