| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-10-25 | |||
| 13:03:45 | sean-k-mooney | be there in a sec | |
| 13:03:48 | sean-k-mooney | feel free to start | |
| 13:05:38 | gmann | joining | |
| 13:27:33 | kashyap | bauzas: I'd appreciate if someone is taking notes of the PCI device scheduling thing. I'm on another call, but will join late to listen in | |
| 13:27:50 | sean-k-mooney | kashyap: we are not talking about that | |
| 13:28:01 | kashyap | Oh, okay; when will that topic be discussed? | |
| 13:28:03 | sean-k-mooney | it was not the first topic and im not sure if there is a topic on that | |
| 13:28:18 | sean-k-mooney | oh you looked at thrusday | |
| 13:28:25 | sean-k-mooney | not wednesday | |
| 13:28:31 | kashyap | sean-k-mooney: Ohh, duh, I just realized, silly me! | |
| 13:28:36 | kashyap | It's tech-debt now | |
| 13:28:38 | sean-k-mooney | we are talking about tech debth and speiccally upgrades | |
| 13:29:25 | kashyap | Thanks! Just noticed it. | |
| 13:40:17 | gokhanisi | hello folks, can we enable thick or thin provisioning per vm ? | |
| 14:57:55 | dvo-plv | Hello, All maybe someone will have a chance to review reproposed spec file ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 15:33:09 | sean-k-mooney | gokhanisi: no | |
| 15:33:38 | sean-k-mooney | gokhanisi: if its in realtion to a cinder volume then you coudl have diffent volume types for each | |
| 15:33:45 | sean-k-mooney | and hten you can do it on a per volume basis | |
| 15:34:14 | bauzas | stephenfin: are you able to join us at least for 'exposing original AZ' topic in 8 mins or are you still blocked in another room ? | |
| 15:34:53 | sean-k-mooney | gokhanisi: nova has a deprecated config option for sparce voluem when using images_type=lvm | |
| 15:34:55 | sean-k-mooney | https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.sparse_logical_volumes | |
| 15:35:33 | sean-k-mooney | that is not quite the same as an lvm thing pool and it shodl be avoidied | |
| 15:35:36 | whoami-rajat | bauzas, hey, the cinder team is free right now if nova would like to do the cross project, or we can discuss another topic in the meantime? | |
| 15:35:58 | bauzas | whoami-rajat: we're just on a break now | |
| 15:36:04 | bauzas | until 3.40 | |
| 15:36:10 | sean-k-mooney | if the lvm group specificved in images_volume_group is a thin pool i think nova can boot vms with thin volume but we never offically supproted that in nova | |
| 15:36:21 | bauzas | and then I'd like to discuss some small topic before we meet | |
| 15:36:48 | whoami-rajat | bauzas, okay, sounds good, let's meet at the decided time then (1600 UTC) | |
| 15:57:44 | stephenfin | bauzas: apologies, I was afk and have another meeting in 3 mins. I could join after that otherwise I'm available all day tomorrow | |
| 15:58:18 | bauzas | stephenfin: unfortunately, we have cinder meeting in 3 mins and then we're done for the day. Available tomorrow 1pm UTC? | |
| 15:59:06 | stephenfin | yup | |
| 16:00:03 | bauzas | cool, sold | |
| 16:05:52 | gibi | sean-k-mooney: on the healthcheck catching RPC issues: do you consider differentiating RPC call exceptions i) due to getting a negative result from the remote RPC endpoint, and ii) RPC call is failed as we cannot sent the message at all? | |
| 17:40:28 | bauzas | johnthetubaguy: noonedeadpunk: lemme know later or tomorrow morning if you can join our nova sessions tomorrow by 1pm | |
| 17:40:37 | bauzas | both of you have topics that are next | |
| 17:41:17 | bauzas | if either of you can't attend at that time, fine, we'll starting discussing other points | |
| 17:41:24 | bauzas | start* | |
| 19:16:17 | opendevreview | Ghanshyam proposed openstack/nova stable/2023.1: hyperv: Mark driver as experimental https://review.opendev.org/c/openstack/nova/+/899159 | |
| 19:17:11 | opendevreview | Ghanshyam proposed openstack/nova stable/2023.1: vmwareapi: Mark driver as experimental https://review.opendev.org/c/openstack/nova/+/899160 | |
| 20:04:38 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Deprecate use_cow_images and 'default' images_type https://review.opendev.org/c/openstack/nova/+/898229 | |
| 20:04:38 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix image conversion check in finish_migration https://review.opendev.org/c/openstack/nova/+/897842 | |
| #openstack-nova - 2023-10-26 | |||
| 02:46:27 | gmann | elodilles: are you aware of the sdk job failure on stable/2023.1 https://zuul.opendev.org/t/openstack/build/321e4f28999c4c3dbfc4128c70be6bdf | |
| 02:46:35 | gmann | it seems it is broken there due to requirement conflict | |
| 05:22:43 | opendevreview | Rico Lin proposed openstack/nova master: Simple fix for sRBAC release note https://review.opendev.org/c/openstack/nova/+/899355 | |
| 06:40:41 | frickler | gmann: yes, this was noted yesterday already, fix would be either https://review.opendev.org/c/openstack/openstacksdk/+/899154 or to relax the requirements of the sdk again | |
| 09:05:18 | opendevreview | Sylvain Bauza proposed openstack/nova master: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/899234 | |
| 09:05:19 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM(yet): Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | |
| 09:05:19 | opendevreview | Sylvain Bauza proposed openstack/nova master: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/899235 | |
| 09:09:56 | kashyap | bauzas: Minor comment: line wrapping is broken in your commit message here: https://review.opendev.org/c/openstack/nova/+/899234 | |
| 09:10:10 | kashyap | Where single words (e.g. "to") are on a whole line | |
| 09:11:00 | bauzas | ah yeah | |
| 09:12:08 | bauzas | that's because the line number is less than 79 | |
| 09:12:48 | bauzas | I'll then cut on L72 | |
| 09:13:03 | kashyap | Thanks; and sorry for the nit-pick. It just makes it reading hard :) | |
| 09:15:08 | opendevreview | Sylvain Bauza proposed openstack/nova master: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/899234 | |
| 09:15:08 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM(yet): Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | |
| 09:15:09 | opendevreview | Sylvain Bauza proposed openstack/nova master: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/899235 | |
| 09:15:12 | bauzas | np, | |
| 09:15:16 | kashyap | Looks good now! | |
| 09:15:19 | bauzas | just reuploaded them | |
| 09:15:57 | bauzas | I'm paginating with VSCodium for a 80-line | |
| 09:16:10 | kashyap | I see; I'm still "stuck in vim" | |
| 09:16:18 | bauzas | but for the commit, then I need to make sure about the length of 72 chars | |
| 09:17:04 | bauzas | tbh I said yesterday loudly that I appreciate correctly written commit messages, so I need to do this for me :) | |
| 09:17:40 | bauzas | tbc, that's not really because I prefer commit messages over code, but we can simply fix a code while a commit message can't be simply be modified | |
| 09:17:57 | bauzas | (once it's merged) | |
| 09:18:13 | bauzas | anyway, let's see what grenade is telling to us :) | |
| 09:19:47 | kashyap | bauzas: Yeah, I highly good commit messages that explain the _why_ in good detail. Not just the "what" (the code tells that anyway!) | |
| 09:20:08 | kashyap | bauzas: DanPB's epic guide on commit messages is still the best I've ever seen: https://wiki.openstack.org/wiki/GitCommitMessages | |
| 09:20:28 | kashyap | See the section on "Examples of bad practise" and "Examples of good practise" | |
| 09:27:01 | elodilles | gmann: no, i did not realize that yet, though now that you have mentioned i remember seeing a fix for this. yes, here it is: https://review.opendev.org/c/openstack/openstacksdk/+/899154 | |
| 09:36:48 | opendevreview | Pierre Libeau proposed openstack/nova master: Fix request_specs not updated after failed rebuild https://review.opendev.org/c/openstack/nova/+/899379 | |
| 09:51:10 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect maximum number of SEV(-ES) guests automatically https://review.opendev.org/c/openstack/nova/+/899381 | |
| 09:53:41 | dvo-plv | Hello, All maybe someone will have a chance to review reproposed spec file ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 09:59:47 | gokhani | thanks for your reply sean-k-mooney, I also saw preallocate_images config in nova. Does it provide thick provisioning ? | |
| 10:00:21 | kashyap | dvo-plv: From a quick look, the core change reads good to me, to gain better performance from using the more efficient virtq structure | |
| 10:00:46 | kashyap | dvo-plv: Also this week is the PTG, so folks might be busy w/ many other things | |
| 10:03:18 | dvo-plv | oh, I see, sorry for dіstraction | |
| 10:03:18 | dvo-plv | oh, I see, sorry for dÑ–straction | |
| 10:04:48 | kashyap | No-no; it's fine. Just keeping you posted in case if you don't hear much this week | |
| 10:05:22 | kashyap | dvo-plv: I gave my ACK there, FWIW | |
| 11:26:15 | dvo-plv | thanks | |
| 12:05:02 | opendevreview | Amit Uniyal proposed openstack/nova-specs master: WIP: Enforce console session timeout https://review.opendev.org/c/openstack/nova-specs/+/898553 | |
| 12:14:45 | opendevreview | Sylvain Bauza proposed openstack/nova master: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/899234 | |
| 12:14:46 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM(yet): Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | |
| 12:14:46 | opendevreview | Sylvain Bauza proposed openstack/nova master: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/899235 | |
| 12:15:37 | opendevreview | Sylvain Bauza proposed openstack/nova master: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | |
| 12:29:42 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM : Test a new compute RPC version with an error https://review.opendev.org/c/openstack/nova/+/899392 | |
| 12:51:04 | opendevreview | Amit Uniyal proposed openstack/nova-specs master: WIP: Enforce console session timeout https://review.opendev.org/c/openstack/nova-specs/+/898553 | |
| 12:57:46 | auniyal | ack frickler | |
| 12:57:58 | bauzas | stephenfin: johnthetubaguy: folks, you haven't replied yet. Are you available in 3 mins for discussing about your topics ? | |
| 13:02:43 | bauzas | like I said in the PTG website, avengers assemble ! | |
| 13:06:09 | gmann | frickler: thanks | |
| 13:12:56 | gmann | elodilles: ack. thanks did the recheck now | |
| 13:15:03 | elodilles | ++ | |
| 13:44:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect maximum number of SEV(-ES) guests automatically https://review.opendev.org/c/openstack/nova/+/899381 | |
| 14:41:54 | gmann | sean-k-mooney: on service role, i captured some note in nova etherpad about what we discussed on Monday, please check/correct if I missed anything in your idea there https://etherpad.opendev.org/p/nova-caracal-ptg#L594 | |
| 15:08:33 | opendevreview | Merged openstack/nova stable/2023.2: Revert "Add upgrade check for compute-object-ids linkage" https://review.opendev.org/c/openstack/nova/+/898953 | |
| 15:12:54 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Use configuration for single enabled mdev type https://review.opendev.org/c/openstack/nova/+/899406 | |
| 15:17:23 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Optionally delete mdevs on instance cleanup https://review.opendev.org/c/openstack/nova/+/899407 | |