Tuesday, 2022-10-25

opendevreviewRafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_project  https://review.opendev.org/c/openstack/openstacksdk/+/85793606:43
opendevreviewRafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_project  https://review.opendev.org/c/openstack/openstacksdk/+/85793606:44
opendevreviewRafael Castillo proposed openstack/openstacksdk master: Add all_projects parameter to find_server  https://review.opendev.org/c/openstack/openstacksdk/+/85793606:45
opendevreviewDenys Mishchenko proposed openstack/ansible-collections-openstack master: search_images replaced with image.images  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86251806:56
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85411308:01
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85435708:01
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85456908:02
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85411308:11
opendevreviewDenys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86256508:56
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85435709:30
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85456909:31
opendevreviewDenys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86256509:52
opendevreviewHaeIYoon proposed openstack/openstacksdk master: Add image metadef object operations  https://review.opendev.org/c/openstack/openstacksdk/+/85835012:38
dannosliwcdAs 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
gtemasdk need to be updated12:52
dannosliwcdOkay. I’ll look into making that change too13:10
servagemHi, Clould someone please review this patch?  Thanks https://review.opendev.org/c/openstack/python-openstackclient/+/86251316:31
noonedeadpunkgtema: 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-builder17:48
noonedeadpunkbut eventually faced this topic and how to search for images with new sdk/collections17:48
noonedeadpunkBasically 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 direction17:49
noonedeadpunkI'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
noonedeadpunkBut I think it's worth syncing efforts so that we won't dig into the wrong direction17:58
noonedeadpunk(or the one you're trying to get rid of for quite a while now)17:59
opendevreviewViolet Kurtz proposed openstack/python-openstackclient master: Moved hypervisor to the SDK  https://review.opendev.org/c/openstack/python-openstackclient/+/85836618:01
opendevreviewMerged openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85411318:05
opendevreviewDenys Mishchenko proposed openstack/ansible-collections-openstack master: Existing images update name, visibility etc  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/86256518:10
gtemanoonedeadpunk: so you stick to Ansible usecase or you are searching for also regular SDK interface for that?18:15
noonedeadpunkgtema: ansible usecase now18:32
noonedeadpunkso for rotation we need to get images with specific tags/properties and verify them. So classic search basically18:35
noonedeadpunkverify in terms to check if they're used by any server or not and if not delete them, otherwise hide18:39
noonedeadpunkwriting it makes me think how optimal end result would be though... as sounds as quite few loops which ansible not good at18:45
noonedeadpunkbut 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 believe18:47
noonedeadpunk*face search18:47
gtemaI would rather create special module packing all functionality 18:50
noonedeadpunkand move search logic to it?18:50
gtemaNot that, i mean all iterations 18:51
noonedeadpunkwell... 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 that18:53
noonedeadpunknext is permissions - most of oprations won't require admin, but rotation is. So we can provide different credentials to some tasks/roles18:54
noonedeadpunkand actually help out and get collection better :)18:55
noonedeadpunkbut yeah, I do understand why such module would be faster/easier. 18:57
noonedeadpunkjust how re-usable it would be (likely not)18:58
opendevreviewMerged openstack/ansible-collections-openstack master: Refactored baremetal_node_action module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85435719:55
opendevreviewMerged openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85456920:41

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!