| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-05-16 | |||
| 16:35:24 | bauzas | #endmeeting | |
| 16:35:24 | opendevmeet | Meeting ended Tue May 16 16:35:24 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:35:24 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.html | |
| 16:35:24 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.txt | |
| 16:35:24 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.log.html | |
| 16:35:27 | gibi | thanks bauzas | |
| 16:35:31 | bauzas | np | |
| 16:35:48 | bauzas | as I said, I'll be off starting tonight until end of the week | |
| 16:36:17 | elodilles | thanks bauzas, have a nice time off o/ | |
| 16:36:18 | dansmith | bauzas: I hope you get some well-needed rest.. it's been like at least a week since you had one | |
| 16:36:22 | bauzas | (taking the opportunity of the bank holiday to visit some friends on the West coast of France, 7 hours by drive) | |
| 16:36:38 | dvo-plv | sean-k-monney, gibi, bauzas: Hello, i have updated bp. Could you please review it again | |
| 16:36:44 | dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075/6..7 | |
| 16:37:12 | bauzas | dansmith: that's why I refused our gorgeous all-company PTO :) | |
| 16:37:22 | dansmith | smh :) | |
| 16:37:34 | bauzas | but yeah, May is our French thanskgiving-like month :) | |
| 16:37:39 | dvo-plv | bauzas, have a nice holiday) | |
| 16:38:25 | bauzas | thanks | |
| 18:07:41 | opendevreview | Merged openstack/nova stable/2023.1: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/882293 | |
| 19:34:00 | opendevreview | Merged openstack/nova stable/xena: Use force=True for os-brick disconnect during delete https://review.opendev.org/c/openstack/nova/+/882867 | |
| 19:44:30 | melwitt | elodilles: I added a link on the etherpad too but fyi gmann is working on fixing ceph on stable/wallaby here https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/871920 | |
| 19:58:16 | elodilles | melwitt: oh, cool, thanks for the information \o/ | |
| 22:08:08 | opendevreview | Carl Morris proposed openstack/nova master: Improving logging at '_numa_cells_support_network_metadata'. https://review.opendev.org/c/openstack/nova/+/860930 | |
| 22:35:23 | opendevreview | Merged openstack/nova stable/xena: Enable use of service user token with admin context https://review.opendev.org/c/openstack/nova/+/882868 | |
| #openstack-nova - 2023-05-17 | |||
| 00:17:01 | opendevreview | melanie witt proposed openstack/nova master: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/883317 | |
| 00:22:23 | melwitt | gibi, sean-k-mooney: ^ this is an attempt to improve stability wrt to the leaked greenlets in unit/func test jobs issue, if you are interested to take a look | |
| 00:24:12 | melwitt | *further improve stability | |
| 00:58:05 | opendevreview | melanie witt proposed openstack/nova master: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/883317 | |
| 01:49:53 | opendevreview | David Wlazlo proposed openstack/python-novaclient master: Typo - nova CLI deprecation warning https://review.opendev.org/c/openstack/python-novaclient/+/883320 | |
| 02:21:48 | opendevreview | David Wlazlo proposed openstack/python-novaclient master: Typo - nova CLI deprecation warning https://review.opendev.org/c/openstack/python-novaclient/+/883320 | |
| 03:21:24 | opendevreview | melanie witt proposed openstack/nova master: Add debug logging when Instance raises OrphanedObjectError https://review.opendev.org/c/openstack/nova/+/883325 | |
| 05:28:50 | opendevreview | Merged openstack/python-novaclient master: Typo - nova CLI deprecation warning https://review.opendev.org/c/openstack/python-novaclient/+/883320 | |
| 06:55:27 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: [wip]network: convert usage of neutronclient to openstacksdk https://review.opendev.org/c/openstack/nova/+/882714 | |
| 07:24:02 | zigo | There's something that I still don't understand. When Moving from "nova server-migraiton-list" to "openstack server migration list --server <UUID>", how do I see the migration ID, so I can give it as argument to "server migration show" ? | |
| 07:24:21 | zigo | It this fixed in a later openstackclient version ? | |
| 07:28:22 | gibi | melwitt: Thanks for that func test patch. I'm totally +2 to merge it | |
| 08:38:56 | sean-k-mooney | melwitt: that is an interesting approch | |
| 08:39:43 | sean-k-mooney | gibi: i still kind of want to follow up wiht moveing all our greenthread usage into a global pool | |
| 08:40:01 | sean-k-mooney | but i think im ok with mergeing melwitt's patch for now | |
| 08:40:13 | sean-k-mooney | i dont knwo when ill get time to work on that again | |
| 08:48:14 | sean-k-mooney | melwitt: once your patch merges i might rebase https://review.opendev.org/c/openstack/nova/+/873061 on top of it | |
| 08:56:07 | sean-k-mooney | kashyap: lol slack is dead | |
| 08:56:22 | kashyap | sean-k-mooney: Oh, is it? | |
| 08:56:33 | kashyap | sean-k-mooney: Yeah, I was trying to say something back :) | |
| 08:56:34 | kashyap | LOL | |
| 08:56:46 | sean-k-mooney | my web client and weechat both cant connect | |
| 08:57:00 | sean-k-mooney | 503 | |
| 08:58:53 | kashyap | Yeah, same here | |
| 09:02:02 | gibi | my web client seems to be OK | |
| 09:02:36 | gibi | and I see your messages so I guess slack is back | |
| 09:02:44 | sean-k-mooney | ya its back | |
| 09:02:58 | gibi | sean-k-mooney: sure I think the melwitt fix is a good approach for now, and we can get back to your approach in the future as well | |
| 09:21:00 | opendevreview | Merged openstack/nova master: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/883317 | |
| 10:16:40 | opendevreview | Rodolfo Alonso proposed openstack/nova master: [alembic] Alembic operations require keywords only arguments https://review.opendev.org/c/openstack/nova/+/883344 | |
| 10:33:34 | opendevreview | John Garbutt proposed openstack/nova master: WIP: Deprecate ironic.peer_list https://review.opendev.org/c/openstack/nova/+/883346 | |
| 10:36:19 | sean-k-mooney | stephenfin: by the way i found an nova api that is not supproted by osc that we missed (its also nto supproted in nova client) | |
| 10:36:35 | stephenfin | oh? | |
| 10:36:40 | sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/show-server-numa-topology.html | |
| 10:37:58 | sean-k-mooney | https://docs.openstack.org/api-ref/compute/?expanded=show-server-topology-detail#show-server-topology | |
| 10:38:15 | sean-k-mooney | i was considerign trying to add that to the sdk | |
| 10:38:21 | sean-k-mooney | just to get used to that process | |
| 10:38:49 | stephenfin | sean-k-mooney: https://github.com/openstack/openstacksdk/blob/master/openstack/compute/v2/server.py#L695-L707 ? | |
| 10:38:50 | sean-k-mooney | maybe on firday as a hack and hussel thing | |
| 10:39:12 | sean-k-mooney | hum ok so its in the sdk | |
| 10:39:18 | sean-k-mooney | so its just missing form osc | |
| 10:40:08 | sean-k-mooney | so adding a new "openstack server toplogy show" command shoudl not be hard | |
| 10:40:24 | sean-k-mooney | the show i guess is a littel reduntant | |
| 10:40:57 | sean-k-mooney | since there is only one action on that endpoint but i think its consitent with the other commands | |
| 10:41:35 | stephenfin | Yup, should be minimally complicated. I'd swear we had that also though... | |
| 10:41:37 | sean-k-mooney | if i try and write a patch for this on friday can you take a look if and give me pointers if i gest stuck | |
| 10:41:57 | sean-k-mooney | i think we missed it because its not in nova client as far as i can see | |
| 10:42:01 | sean-k-mooney | i tought we did too | |
| 10:42:14 | stephenfin | https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/server.html#server-show | |
| 10:42:27 | stephenfin | openstack server show --topology <server> | |
| 10:42:28 | sean-k-mooney | its not in server show intentionally | |
| 10:42:57 | sean-k-mooney | so that returns only the toplogy and not the rest of the server detail? | |
| 10:43:28 | sean-k-mooney | we made the point in the spec that this should not be part fo the server show api respocne for performance reasons | |
| 10:43:40 | sean-k-mooney | i guess as a client opt in it not terible | |
| 10:44:02 | stephenfin | Yeah. It's not how I'd personally do that normally since it's really a separate command but it predates my involvement in the project. Ditto for the --diagnostics argument | |
| 10:44:09 | sean-k-mooney | but that means the client is merging in a second api request into the output | |
| 10:44:20 | stephenfin | I moved 'server migrate --confirm' to 'server migration confirm' | |
| 10:45:12 | sean-k-mooney | ok if this is in the osc we have downstream for 17 i could suggest using it for verifying an instnac ehas been booted with a numa toplogy | |
| 10:45:19 | stephenfin | Yeah, it is. Client-side though | |
| 10:45:35 | sean-k-mooney | ok so if we have the supprot should i just leave it as it is | |
| 10:45:54 | stephenfin | For 17 I'd pull the Antelope version of OSC. It's so much better than the Wallaby version or whatever | |
| 10:45:58 | sean-k-mooney | i didnt find it as i was expecting it to be a seperate command sicne its a differnt api but its not without precended | |
| 10:46:00 | stephenfin | Ditto for SDK | |
| 10:46:07 | sean-k-mooney | we use server migrate for cold and live | |
| 10:46:11 | stephenfin | openstacksdk 1.0 is a big move forward | |
| 10:46:30 | sean-k-mooney | stephenfin: that has issues depeningon where you isntall it | |
| 10:46:50 | sean-k-mooney | you cant intall the pypi verisonon the undercloud becasue it use the rpm version to run tripleo | |
| 10:47:09 | sean-k-mooney | but any of our custoemr using it on there laptops shoudl defintly use it form pypi | |
| 10:47:18 | stephenfin | True | |
| 10:48:07 | sean-k-mooney | ok well thanks for confirming this is techinaly supproted | |
| 10:48:19 | sean-k-mooney | i guess ill park doing any refactorign for now | |
| 10:48:35 | sean-k-mooney | we could rename it but its proably not worth the upgrade impact | |
| 11:40:04 | dvo-plv_ | sen-k-mooney, Hello. Will you have some free time to verify new test for packed ring, which were created ? https://review.opendev.org/c/openstack/nova/+/876075 | |
| 11:48:25 | dvo-plv_ | sean-k-mooney: sorry, I've made a mistake in your nick | |
| 11:49:32 | sean-k-mooney | hehe i have done that in comments i left in the code so dont worry about it | |
| 11:50:23 | sean-k-mooney | dvo-plv_: and no i have not yet but ill try and take a look. i have a meeting at the top of the hour so im oging to go grab coffee but ill take a look at them when i get back | |