Monday, 2022-07-11

*** ysandeep|out is now known as ysandeep05:38
*** ysandeep is now known as ysandeep|holiday06:18
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Update project module to be compatible with new sdk  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/83964007:25
opendevreviewArx Cruz proposed openstack/ansible-collections-openstack master: Update address scope to use proxy  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84863507:52
opendevreviewArx Cruz proposed openstack/ansible-collections-openstack master: Update federation_idp_info module to use proxy layer  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84928208:07
opendevreviewArx Cruz proposed openstack/ansible-collections-openstack master: DNM - Update federation_mapping_info module to use proxy  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84928308:47
*** rlandy__ is now known as rlandy10:28
*** dviroel|out is now known as dviroel|rover11:43
dtantsuranyone working on switching baremetal_node to the proxy layer?12:05
dtantsurI need it for a change of mine12:05
dtantsuror maybe I'll actually update openstacksdk with more logic12:07
jm1dtantsur: it is on our list but we are waiting for https://review.opendev.org/c/openstack/bifrost/+/848741 to merge. then we can get rid of the ironic-specific module logic in the collection12:37
jm1dtantsur: 'waiting' := we-are-working-on-other-modules ^^12:38
dtantsurI don't think this one will be trivial though: it combines port creation with node enrolling..12:39
jm1dtantsur: ack, atm nobody is actively working on baremetal_node according to our modules list https://hackmd.io/7NtovjRkRn-tKraBXfz9jw?view12:52
jm1dtantsur: so you could go ahead and implement your change, then we will merge it and refactor the modules later12:53
jm1dtantsur: but it would be cool if your patch gets merged in bifrost because this would make baremetal modules in the aoc easier12:54
jm1*simpler12:54
rcastillojm1 it seems the cloud layer methods baremetal_node use are actually still returning munches https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_baremetal.py#L31912:57
dtantsurjm1: yeah, it's not blocking my bifrost changes12:58
dtantsurmunch-munch12:58
rcastillo^ gtema intended, or do register_machine and unregister_machine still need to get updated?12:58
jm1rcastillo: the latter ^^12:59
jm1rcastillo: i guess..12:59
gtemauff, I think I missed them12:59
dtantsurI can take this action item as well13:09
*** dviroel|rover is now known as dviroel|rover|lunch15:07
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84942315:35
*** dviroel|rover|lunch is now known as dviroel|rover16:11
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84942317:41
opendevreviewRafael Castillo proposed openstack/ansible-collections-openstack master: DNM: Test openstacksdk dependency  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84485918:47
opendevreviewRafael Castillo proposed openstack/ansible-collections-openstack master: Use openstacksdk dependency in tripleo job  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84485918:51
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84942319:55
*** dviroel|rover is now known as dviroel|rover|biab20:36
*** dviroel|rover|biab is now known as dviroel|rover21:15
*** dviroel|rover is now known as dviroel|rover|biab21:41
*** dviroel|rover|biab is now known as dviroel|rover|21:59
*** dviroel|rover| is now known as dviroel|out22:01
*** rlandy is now known as rlandy|bbl22:06

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