*** ysandeep|out is now known as ysandeep | 05:38 | |
*** ysandeep is now known as ysandeep|holiday | 06:18 | |
opendevreview | Jakob 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/+/839640 | 07:25 |
---|---|---|
opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update address scope to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848635 | 07:52 |
opendevreview | Arx 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/+/849282 | 08:07 |
opendevreview | Arx 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/+/849283 | 08:47 |
*** rlandy__ is now known as rlandy | 10:28 | |
*** dviroel|out is now known as dviroel|rover | 11:43 | |
dtantsur | anyone working on switching baremetal_node to the proxy layer? | 12:05 |
dtantsur | I need it for a change of mine | 12:05 |
dtantsur | or maybe I'll actually update openstacksdk with more logic | 12:07 |
jm1 | dtantsur: 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 collection | 12:37 |
jm1 | dtantsur: 'waiting' := we-are-working-on-other-modules ^^ | 12:38 |
dtantsur | I don't think this one will be trivial though: it combines port creation with node enrolling.. | 12:39 |
jm1 | dtantsur: ack, atm nobody is actively working on baremetal_node according to our modules list https://hackmd.io/7NtovjRkRn-tKraBXfz9jw?view | 12:52 |
jm1 | dtantsur: so you could go ahead and implement your change, then we will merge it and refactor the modules later | 12:53 |
jm1 | dtantsur: but it would be cool if your patch gets merged in bifrost because this would make baremetal modules in the aoc easier | 12:54 |
jm1 | *simpler | 12:54 |
rcastillo | jm1 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#L319 | 12:57 |
dtantsur | jm1: yeah, it's not blocking my bifrost changes | 12:58 |
dtantsur | munch-munch | 12:58 |
rcastillo | ^ gtema intended, or do register_machine and unregister_machine still need to get updated? | 12:58 |
jm1 | rcastillo: the latter ^^ | 12:59 |
jm1 | rcastillo: i guess.. | 12:59 |
gtema | uff, I think I missed them | 12:59 |
dtantsur | I can take this action item as well | 13:09 |
*** dviroel|rover is now known as dviroel|rover|lunch | 15:07 | |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849423 | 15:35 |
*** dviroel|rover|lunch is now known as dviroel|rover | 16:11 | |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849423 | 17:41 |
opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: DNM: Test openstacksdk dependency https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844859 | 18:47 |
opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use openstacksdk dependency in tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844859 | 18:51 |
opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849423 | 19:55 |
*** dviroel|rover is now known as dviroel|rover|biab | 20:36 | |
*** dviroel|rover|biab is now known as dviroel|rover | 21:15 | |
*** dviroel|rover is now known as dviroel|rover|biab | 21:41 | |
*** dviroel|rover|biab is now known as dviroel|rover| | 21:59 | |
*** dviroel|rover| is now known as dviroel|out | 22:01 | |
*** rlandy is now known as rlandy|bbl | 22:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!