| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-08-22 | |||
| 14:59:37 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 15:32:11 | jm1 | gtema: for the second issue with add_ips_to_server we have some debug output now https://dbb517edecb69c839e64-a1eedb38eefc88bab97a9b5aab34511e.ssl.cf1.rackcdn.com/828613/13/check/ansible-collections-openstack-functional-devstack-ansible-2.12/2bd2d7b/job-output.txt | |
| 15:32:42 | jm1 | gtema: this is the patch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 15:33:33 | gtema | cool, thks | |
| 15:34:40 | gtema | okay, this means clearly we are in the "reuse existing unbound fip" | |
| 15:34:57 | gtema | this is what I was suspecting. | |
| 15:42:21 | gtema | @jm1: do you see that port_id in the query is of a fixed ip and not floating? | |
| 15:44:37 | jm1 | gtema:? | |
| 15:46:51 | gtema | "Get floating ip attached to server 2" task in the log uses "port": "1e08abff-72cd-465c-b92d-18806df814d6" as filter | |
| 15:47:09 | gtema | and this port is of a fixed address and not floating | |
| 15:48:16 | gtema | I am wondered that in the test you simply use server2_ports.ports[1].id | |
| 16:04:37 | opendevreview | mbu proposed openstack/osc-lib stable/train: Do not output log on bash autocomplete generation https://review.opendev.org/c/openstack/osc-lib/+/854016 | |
| 16:07:06 | matbu | Hey folks, is it possible to merge this patch, which is a cherry pick from master: https://review.opendev.org/c/openstack/osc-lib/+/854016 | |
| 16:07:10 | matbu | thank you by advance | |
| 16:07:25 | matbu | (and all the previous stack for wallaby & cie) | |
| 16:09:29 | opendevreview | Merged openstack/ansible-collections-openstack master: Deprecate special value 'auto' for attribute 'id' of compute_flavor https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852936 | |
| 16:28:25 | opendevreview | Sharon Koech proposed openstack/python-openstackclient master: Add ``--cascade`` CLI option to network delete command https://review.opendev.org/c/openstack/python-openstackclient/+/854032 | |
| 16:44:02 | frickler | matbu: do you have info how to reproduce the issue to be fixed by that patch? to me the fix looks at least incomplete and shouldn't have been merged in master either | |
| 16:57:04 | matbu | frickler: I have a bug here: https://bugzilla.redhat.com/show_bug.cgi?id=2083742 | |
| 16:57:26 | matbu | frickler: i dont have much more information than that | |
| 16:57:48 | matbu | frickler: I was just tracking it, but I take a closer look to get a proper fix if needed | |
| 17:03:27 | frickler | matbu: thx, I just commented on the reviews in parallel. "Steps to reproduce" isn't really helpful, with that input I'd reject a bug as invalid. are you with the RDO team? getting a useable reproducer would be really helpful | |
| 17:04:01 | matbu | frickler: yep I will ask for a reproducer to the bug reporter | |
| #openstack-sdks - 2022-08-23 | |||
| 03:30:29 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 03:37:24 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 06:26:25 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 07:27:15 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 07:53:09 | opendevreview | Sharon Koech proposed openstack/python-openstackclient master: Add ``--cascade`` CLI option to network delete command https://review.opendev.org/c/openstack/python-openstackclient/+/854032 | |
| 08:15:12 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 08:15:48 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 09:04:31 | opendevreview | ribaudr proposed openstack/openstacksdk master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/openstacksdk/+/853949 | |
| 09:16:43 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 09:26:46 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 09:32:03 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 11:00:40 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 11:22:37 | jm1 | gtema: can we rewrite register_machine to use baremetal/node resources? https://opendev.org/openstack/openstacksdk/src/commit/6bc56b0eb6eeb21569aae72ee8033ac64db54fb9/openstack/cloud/_baremetal.py#L160 | |
| 11:22:38 | jm1 | for example, we could fill a Node object with all kwargs and then use baremetal.create_node() instead of self._baremetal_client.post() | |
| 11:23:10 | jm1 | gtema: or will _baremetal_client.post() do exactly that? | |
| 11:28:36 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 11:41:48 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 11:44:09 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 11:46:04 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 11:53:05 | frickler | gtema: stephenfin: could you please add https://review.opendev.org/c/openstack/python-openstackclient/+/749038 to your review list? I added some questions for other reviewers in my comments. I know it is late in the cycle but it would be really great if this patch would make it for Zed. otherwise the feature is in Neutron but noone can use it. | |
| 11:59:59 | opendevreview | Areg Grigoryan proposed openstack/openstacksdk master: Bugfix for paginating for complex paths https://review.opendev.org/c/openstack/openstacksdk/+/853656 | |
| 12:37:34 | gtema | <jm1> "gtema: or will _baremetal_client..." <- it does exactly that | |
| 12:38:05 | gtema | frickler, but with this amount of comments ... Do you think they would be addressed quickly? | |
| 12:48:41 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored baremetal_node and baremetal_node_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113 | |
| 12:52:08 | jm1 | gtema: oh cool! so this should work? https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854113/7/plugins/modules/baremetal_node.py#655 | |
| 12:53:15 | jm1 | gtema: kwargs uses attribute names of the Node object such as 'chassis_id' and no longer ironic names such as 'chassis_uuid' | |
| 12:54:29 | 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 | |
| 12:59:50 | gtema | jm1: for BM I would really suggest to try not to use cloud layer. Because I have no clue whether and how ironic folks use it I have fear to touch it really. whatever is on the proxy side is always verified by tests and therefore can be fixed when necessary | |
| 13:07:36 | jm1 | gtema: but that would imply replicating code from the sdk in aoc | |
| 13:08:19 | gtema | which particular cases? | |
| 13:08:36 | jm1 | gtema: maybe we should ask dtantsur how or if ironic is using this register_machine function.. | |
| 13:08:47 | gtema | I was meaning similarly to the create_server case: for the complex cases of course we use cloud. | |
| 13:09:08 | dtantsur | I highly suspect we do use it | |
| 13:09:28 | jm1 | gtema: register_machine is not only creating a node but also creating baremetal nodes | |
| 13:09:46 | frickler | gtema: yes, I've been in contact with the author already | |
| 13:11:24 | frickler | I just would like to avoid needing another set of iterations when other reviewers have different views than me | |
| 13:11:35 | jm1 | dtantsur: codesearch does not return any match. how could ironic possibly use it? maybe though aoc? https://codesearch.opendev.org/?q=register_machine&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 13:11:55 | dtantsur | jm1: bifrost uses aoc, yes | |
| 13:12:43 | dtantsur | I started doing something about this call, but never had time to finish: https://review.opendev.org/c/openstack/openstacksdk/+/849402/7 | |
| 13:14:37 | jm1 | dtantsur, gtema: to me it looks as if aoc is the only user of register_machine(). so we could touch it without breaking ironic. | |
| 13:14:59 | jm1 | dtantsur: oh great, so someone could pick this up :) | |
| 13:26:13 | jm1 | wow, first X server crash for... a decade? omg.. | |
| 13:28:09 | gtema | lol | |
| 14:04:17 | opendevreview | Areg Grigoryan proposed openstack/openstacksdk master: Bugfix for paginating for complex paths https://review.opendev.org/c/openstack/openstacksdk/+/853656 | |
| 14:19:55 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: Allow passing explicit microversions to Resource methods https://review.opendev.org/c/openstack/openstacksdk/+/849394 | |
| 14:19:58 | dtantsur | rebase party begins ^^ | |
| 14:21:57 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: rework node creation to be closer to the backend https://review.opendev.org/c/openstack/openstacksdk/+/849505 | |
| 14:22:19 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: Actually use openstacksdk from source in the Bifrost job https://review.opendev.org/c/openstack/openstacksdk/+/849564 | |
| 14:22:40 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Migrate register_machine to use the proxy layer https://review.opendev.org/c/openstack/openstacksdk/+/849402 | |
| 14:53:26 | 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 | |
| 15:33:22 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 17:04:45 | 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 | |
| 17:45:30 | matbu | frickler: hey, according to our discussion yesterday I'm about to revert this patch in master: https://review.opendev.org/c/openstack/osc-lib/+/841307 | |
| 17:45:59 | matbu | frickler: and see if we need a proper fix in train only or not .. i dont think it's something really critical anyway so.. | |
| 19:46:49 | 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 | |
| 22:54:27 | 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 | |
| #openstack-sdks - 2022-08-24 | |||
| 01:53:49 | opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Use /volumes/detail endpoint in find_volume proxy method https://review.opendev.org/c/openstack/openstacksdk/+/854293 | |
| 01:54:14 | 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 | |
| 05:28:32 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 05:37:58 | opendevreview | Merged openstack/openstacksdk master: Use /volumes/detail endpoint in find_volume proxy method https://review.opendev.org/c/openstack/openstacksdk/+/854293 | |
| 06:22:18 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 06:37:53 | opendevreview | yangjianfeng proposed openstack/python-openstackclient master: Add router ndp proxy commands https://review.opendev.org/c/openstack/python-openstackclient/+/749038 | |
| 09:25:17 | 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 | |
| 09:30:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored baremetal_node_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854357 | |
| 09:37:00 | stephenfin | gtema: easy OSC review for a new nova microversion here https://review.opendev.org/c/openstack/python-openstackclient/+/831902 | |
| 09:37:09 | gtema | oki, looking | |
| 09:37:42 | stephenfin | as I paste that, I wonder if we need to bump the novaclient version in requirements.txt? | |
| 09:37:45 | stephenfin | we probably do | |
| 09:38:02 | stephenfin | (pity lower-constraints is no longer a thing for stuff like this) | |
| 10:04:13 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Bump python-novaclient version https://review.opendev.org/c/openstack/python-openstackclient/+/854363 | |
| 10:04:13 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove invalid note from requirements files https://review.opendev.org/c/openstack/python-openstackclient/+/854364 | |
| 11:41:26 | stephenfin | frickler: I left a lot of comments on https://review.opendev.org/c/openstack/python-openstackclient/+/749038/ Maybe you can help me understand some of the naming decisions? | |
| 11:52:14 | frickler | stephenfin: already at it ;) | |
| 11:52:23 | stephenfin | thanks! | |
| 12:04:44 | frickler | slaweq: ^^ maybe you have some opinion on this, too. I actually like stephenfin's idea of using "router ndp proxy" as command prefix instead of "network ndp proxy" | |
| 12:37:16 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 12:40:01 | jm1 | gtema: are you sure that openstacksdk's self._baremetal_client.post() will accept sdk attribute names instead of openstack api attribute names? e.g. schema validation fails because i am passing 'id' https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_908/854113/12/check/bifrost-collections-src/90849ec/job-output.txt | |