| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | 01:39 |
|---|---|---|
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | 01:42 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | 01:42 |
| *** ysandeep|out is now known as ysandeep | 05:52 | |
| *** frenzy_friday is now known as frenzyfriday|rover | 06:29 | |
| *** arxcruz|rover is now known as arxcruz | 07:18 | |
| *** ysandeep is now known as ysandeep|lunch | 08:26 | |
| jpodivin | jm1: Hi. I'm seeing some odd behavior from the openstack.cloud.baremetal_node_info module on wallaby. When I ask for a single node it fills all of the details in 'properties' but when I request in on entire cloud it leaves it as null | 08:57 |
| jpodivin | is that intended behavior or a bug? | 08:57 |
| jm1 | jpodivin: o/ baremetal_node_info on wallaby calls different sdk functions depending on whether you want to have a single node or all nodes. those sdk functions return different things in wallaby. it will be fixed in aoc2.0.0/sdk>=0.99.0 | 09:01 |
| jm1 | jpodivin: for wallaby we cannot do anything. its broken. in soo many ways.. | 09:02 |
| jpodivin | jm1: I see, that makes sense. I suppose the workaround would be to get the list first and feed it to the module | 09:02 |
| jm1 | jpodivin: yes | 09:02 |
| jpodivin | jm1: as for wallaby in general ... what do you mean by "so many things" . Like the collection or the release itself? | 09:03 |
| jm1 | jpodivin: aoc and the sdk. this issue with different values being returned based on the inputs is one of those things | 09:07 |
| *** ysandeep|lunch is now known as ysandeep | 09:08 | |
| jm1 | jpodivin: openstacksdk >=0.99.0 might still have some rough edges but its already much cleaner than the old releases. for example, you will get the same output for baremetal_node_info | 09:09 |
| jpodivin | jm1: I see. Well I'll have to deal with wallaby for the foreseeable future. But thank you very much. This will save some time. | 09:10 |
| jm1 | jpodivin: true and we cannot backport sdk >=0.99.0 to pre-zed releases. in your particular case you could in theory propose a patch to return the same stuff but this will have to be implemented in aoc's stable branch and could turn out to be difficult without breaking aoc's "api" | 09:12 |
| jm1 | jpodivin: so its probably not worth it. calling the module twice is easier | 09:13 |
| jpodivin | jm1: pretty much. It does waste a few cycles but it's not going to be running all the time so I think it's not that much of a deal. | 09:15 |
| jpodivin | certainly beats potentially breaking sdk or the collection while backporting. | 09:15 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.102.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/861329 | 10:09 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | 10:09 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored stack_info module and allow to filter by owner and project names https://review.opendev.org/c/openstack/ansible-collections-openstack/+/859540 | 10:22 |
| *** rlandy|out is now known as rlandy | 10:30 | |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | 11:34 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | 11:34 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | 11:35 |
| *** ysandeep is now known as ysandeep|afk | 11:51 | |
| *** ysandeep|afk is now known as ysandeep | 12:22 | |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_action module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | 12:53 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_port and baremetal_port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854569 | 12:53 |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| *** ysandeep is now known as ysandeep|out | 15:08 | |
| *** dviroel|lunch is now known as dviroel | 16:07 | |
| opendevreview | Merged openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.102.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/861329 | 17:08 |
| *** dviroel is now known as dviroel|out | 21:27 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!