| *** dviroel|ruck|afk is now known as dviroel|ruck | 00:48 | |
| *** dviroel|ruck is now known as dviroel|ruck|out | 00:57 | |
| *** dviroel|ruck|out is now known as dviroel|out | 00:57 | |
| *** sshnaidm|afk is now known as sshnaidm | 08:54 | |
| *** ysandeep|out is now known as ysandeep | 09:01 | |
| *** mnasiadka_ is now known as mnasiadka | 09:18 | |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Drop deprecated field in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | 09:37 |
|---|---|---|
| opendevreview | Merged openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_node_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827586 | 09:48 |
| opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | 10:57 |
| *** dviroel|out is now known as dviroel|ruck | 11:10 | |
| *** ysandeep is now known as ysandeep|break | 12:41 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test flavor changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828290 | 12:54 |
| *** ysandeep|break is now known as ysandeep | 13:13 | |
| opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | 13:20 |
| opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | 13:22 |
| opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827937 | 13:23 |
| opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | 13:58 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 14:06 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 14:07 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped deprecated tenant_id in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | 14:09 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped deprecated tenant_id in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | 14:13 |
| jm1 | sshnaidm gtema: how do i ask OpenStack SDK to reload server info? | 14:23 |
| jm1 | e.g. i am using openstack.compute.get_server/find_server, then a server gets updated externally and i want to reload the server object with SDK | 14:24 |
| gtema | depending on use case. Generally get_server always fetches current status | 14:25 |
| gtema | some operations (provisioning) are using wait_for_server. This is used when you trigger some action on server and you wait until it reaches "active" status | 14:25 |
| jm1 | ah cool, thanks :) get_server did the job for me, got the id with find_server before so it works :) | 14:27 |
| jm1 | is it safe to call server.remove_floating_ip ? | 14:32 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 14:32 |
| jm1 | Or better compute.remove_floating_ip_from_server? | 14:33 |
| gtema | where did you found remove_floating_ip_from_server? | 14:34 |
| gtema | ah, found | 14:34 |
| gtema | yeah - better call self.conn.compute.remove_floating_ip_from_server rather then directly | 14:34 |
| gtema | both does the same, but better only use proxy | 14:35 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 14:36 |
| jm1 | ok :) | 14:37 |
| jm1 | gtema: there is a add_ips_to_server function in the cloud layer (IIUC). it is used to add floating ip's to a server and is used e.g. in Ansible's openstack.cloud.floating_ip module. I am unsure whether this should be replaced or if this use in the Ansible module is okay | 15:05 |
| jm1 | https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/floating_ip.py#L266 | 15:06 |
| gtema | pls check how much logic was/is in the cloud layer. if it only invokes single API - no sense to use cloud layer then | 15:06 |
| jm1 | it might do multiple API calls depending on the input: first create and then assign a fip to a server. so i would continue to use it | 15:11 |
| jm1 | i was wondering because i cannot find it in the docs | 15:12 |
| jm1 | ..whether it is to be avoided | 15:12 |
| *** ysandeep is now known as ysandeep|out | 15:43 | |
| opendevreview | Merged openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827937 | 16:09 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 16:18 |
| opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | 16:32 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test flavor changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828290 | 16:33 |
| jm1 | gtema: can stuff from meta such as meta.add_server_interfaces be used or is it to be avoided? | 20:21 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM testing image info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828294 | 21:03 |
| opendevreview | James Kirsch proposed openstack/ansible-collections-openstack master: Add support for system role in role assignment https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826395 | 21:43 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | 21:55 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored and documented floating_ip module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | 22:04 |
| sshnaidm | when you look at module to change, please take a look if there is a bug for it in https://storyboard.openstack.org/#!/project/openstack/ansible-collections-openstack | 22:09 |
| sshnaidm | probably worth to fix it along.. | 22:09 |
| jm1 | sure | 22:21 |
| *** dviroel|ruck is now known as dviroel|out | 22:33 | |
| opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | 22:41 |
| opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!