Thursday, 2022-08-04

*** undefined_ is now known as Guest704900:09
*** Guest7049 is now known as rcastillo02:44
*** ysandeep|out is now known as ysandeep06:04
*** ysandeep is now known as ysandeep|afk07:27
*** undefined is now known as Guest707907:43
*** ysandeep|afk is now known as ysandeep08:22
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored port and port_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85062708:23
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/82861308:33
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:48
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive openstacksdk.get_server() in server module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85211908:52
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive openstacksdk.get_server() in server module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85211908:55
opendevreviewShnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move keypair module to proxy layer  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84796009:17
*** rlandy|out is now known as rlandy10:35
arxcruzsshnaidm jm1 hey, can you guys please review https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84928 ?10:42
arxcruzi need it merged in order to work on the other federation_* modules10:42
arxcruzall my other patches on this subject are blocked10:43
sshnaidm849282 ?10:43
arxcruzops, miss a number at the end10:48
arxcruzsshnaidm hold on, i'll update the patch in a few 10:48
jm1arxcruz: you can rebase your other modules on this patch and gerrit will show the dependencies10:50
arxcruzjm1 that would add too much complexity to manage the other patches10:52
arxcruzif you can't merge it for now, i can wait, no problem 10:52
sshnaidmarxcruz, do you have "find_identity_provider"? instead of get_identity_provider10:53
arxcruzsshnaidm need to check10:54
arxcruzgive me a few, i'm finishing add the return 10:54
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/+/84928211:11
arxcruzsshnaidm yes, there is, and i update the patch with the find_identity_provider ^11:11
arxcruzjm1: ^11:11
arxcruzbr11:11
arxcruzbrb11:11
*** dviroel|afk is now known as dviroel11:34
*** ysandeep is now known as ysandeep|afk12:03
*** Guest7079 is now known as rcastillo13:03
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/+/84928213:29
*** ysandeep|afk is now known as ysandeep13:45
*** frenzyfriday|rover is now known as frenzyfriday|rover|lunch14:16
*** ysandeep is now known as ysandeep|out14:52
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Dropped extra interfaces_info attribute from routers_info module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85217915:01
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Dropped extra interfaces_info attribute from routers_info module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85217915:03
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/+/84928215:17
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/+/84928215:24
arxcruzsshnaidm rcastillo jm1 ^ now covered all the comments15:24
opendevreviewRafael Castillo proposed openstack/ansible-collections-openstack master: Update quota for 2.0.0  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85148315:35
jm1arxcruz: great, thanks! its on my review list ^^15:42
*** dviroel is now known as dviroel|lunch15:47
*** ysandeep|out is now known as ysandeep|PTO15:51
*** dviroel|lunch is now known as dviroel16:52
*** frenzyfriday|rover|lunch is now known as frenzyfriday|rover16:55
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: Dropped extra interfaces_info attribute from routers_info module  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/85217919:01
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/+/84928219:18
*** dviroel is now known as dviroel|afk20:59
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/+/84928221:15
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/+/84928222:42
*** rlandy is now known as rlandy|out22:54
*** dviroel|afk is now known as dviroel23:30
*** dviroel is now known as dviroel|out23:36

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