| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-06 | |||
| 09:59:05 | kashyap | I'll correct my mistake on a bugzilla | |
| 10:01:51 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803717 | |
| 10:01:52 | opendevreview | Lee Yarwood proposed openstack/nova stable/wallaby: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803718 | |
| 10:02:04 | pjakuszew | kashyap: I wrote a script which modifes extra_specs on existing instances, though I consider it dangerous | |
| 10:02:38 | kashyap | pjakuszew: Heh, understood. The intersting question is - did it work reliably for you, without any nasty side-effects? | |
| 10:03:28 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803720 | |
| 10:03:29 | opendevreview | Lee Yarwood proposed openstack/nova stable/victoria: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803721 | |
| 10:04:59 | pjakuszew | kashyap: In my case it was a case of adding a cpu_type key to a flavor, then syncing the flavor metadata to instance and reqspec extra_specs - in your case (hw_video_model=virtio) I guess you will have to do a hard reboot so the libvirt xmls will be regenerated | |
| 10:05:32 | kashyap | pjakuszew: Yep; in this case hard-reboot is unavoidable | |
| 10:06:44 | pjakuszew | let me check it, one sec | |
| 10:13:47 | kashyap | pjakuszew: Note, though: I don't have any environment to try this out. I'm just documenting some stuff due to a fallout downstream | |
| 10:13:53 | kashyap | But thank you! :) | |
| 10:15:22 | pjakuszew | kashyap: sure, I'm checking the differences between the xmls with and without properties right now | |
| 10:15:43 | pjakuszew | so I can check if the metadata change was properly propagated after hard reboot later | |
| 10:16:39 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803722 | |
| 10:16:40 | opendevreview | Lee Yarwood proposed openstack/nova stable/ussuri: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803723 | |
| 10:17:28 | bauzas | wow, I can see light at the end of the tunnel : https://events.linuxfoundation.org/kubecon-cloudnativecon-north-america/register/ | |
| 10:21:47 | kashyap | pjakuszew: Thank you; do keep me posted. And post a link to your script - perhaps that can be added in some dangerous_utils/ directory | |
| 10:22:21 | kashyap | bauzas: That's still living in la-la-land for LF -- I doubt many will register until the dust settles with the Delta variant | |
| 10:22:59 | bauzas | kashyap: yup and the travel ban is still effective, but that's a fresh change at least for dometic flights | |
| 10:24:06 | kashyap | bauzas: Domestic flights in the US, you mean? | |
| 10:24:13 | bauzas | yup | |
| 10:24:33 | bauzas | this gives us some direction | |
| 10:24:48 | pjakuszew | kashyap: this is something I wanted to add into nova_manage in a more accessible form, actually | |
| 10:25:05 | opendevreview | Merged openstack/nova master: func: Increase rpc_response_timeout in TestMultiCellMigrate tests https://review.opendev.org/c/openstack/nova/+/803714 | |
| 10:25:22 | bauzas | conferences could run for a mixed virtual+physical system until restrictions slowly improve | |
| 10:25:43 | kashyap | Mixed confs suck like hell :-( It frustrates both in-person and the remotees | |
| 10:26:10 | kashyap | "Can you hear me? Oh, I was talking to myself. Anybody there? Oh, you all went for a coffee break? Never mind" | |
| 10:34:02 | pjakuszew | kashyap: are these server properties visible in the libvirt xml in your case? looks like adding that property both under server create --property and manually on another one doesn't seem to propagate | |
| 10:34:26 | kashyap | pjakuszew: They are not; see stephenfin's comment earlier | |
| 10:34:51 | pjakuszew | oh I overlooked that | |
| 10:35:10 | kashyap | pjakuszew: Only way for existing instances are what lyarwood mentioned earlier (snapshot the image; adjust the property; and then boot an instance off the updated snapshot image) | |
| 10:43:57 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803727 | |
| 10:43:58 | opendevreview | Lee Yarwood proposed openstack/nova stable/train: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803728 | |
| 10:50:08 | opendevreview | Merged openstack/nova master: fup: Move _wait_for_volume_attach into InstanceHelperMixin https://review.opendev.org/c/openstack/nova/+/802623 | |
| 11:50:55 | lyarwood | stephenfin / gibi ; https://review.opendev.org/c/openstack/nova/+/802801/ could I get a +W on this now it's passing again with the above func test landed | |
| 12:58:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| 13:15:32 | bauzas | stephenfin: hola, before I leave, I'd appreciate if you could just look at the last rev for https://review.opendev.org/q/topic:%22bp%252Fgeneric-mdevs%22+(status:open%20OR%20status:merged) we need a second pair of eyes | |
| 13:15:44 | bauzas | since you already looked at it | |
| 13:27:21 | sean-k-mooney | bauzas: im off today but i will sync with the rest of the team and see if i can push the mdev work forward while your on pto | |
| 13:27:38 | kashyap | sean-k-mooney: Don't taint your free day with work... | |
| 13:28:17 | sean-k-mooney | :) that is why im not going to review today | |
| 13:28:38 | sean-k-mooney | anyway i have almost finished what i need to do on my laptop soo ill be gone soonish | |
| 13:29:29 | bauzas | kashyap: by the way, I apologize for you, my co-worker for the mdev series ;) | |
| 13:29:43 | bauzas | kashyap: we hadn't time to discuss about this blueprint :p | |
| 13:30:06 | bauzas | sean-k-mooney: as you want, anyway I'll be back on Tuesday, 2 days before FF | |
| 13:30:09 | kashyap | bauzas: I was supposed to pair on it, right? Yeah, downstream-- stuff completely engulfed me :-( | |
| 13:30:15 | bauzas | ;) | |
| 13:30:21 | bauzas | kashyap: heh no worries | |
| 13:31:25 | stephenfin | bauzas: will do | |
| 13:31:31 | bauzas | ta | |
| 13:33:30 | kashyap | bauzas: I was reading the mdev class code yesterday and then had to reboot the laptop and my memory flushed it. I know, I sound lame. | |
| 13:36:34 | kashyap | bauzas: Did you get around to testing it on real hardware? Or "don't ask?" | |
| 13:37:00 | bauzas | kashyap: I did | |
| 13:37:19 | bauzas | I even bubbled the hostname in some pastebin, whoops | |
| 13:37:26 | bauzas | :) | |
| 13:37:56 | bauzas | I was smart enough to keep the hardware that was given for me for testing the Ampere GPU :p | |
| 13:38:13 | bauzas | so, technically, I tested this series against MIG-backed vGPUs ;) | |
| 13:38:14 | kashyap | I see; yeah, I know the hardware acquis on this PITA | |
| 13:38:46 | kashyap | s/on this/here is a/ | |
| 13:39:04 | bauzas | hopefully I should still have this server when I'm back :p | |
| 13:39:53 | kashyap | bauzas: The only thing I know about MIG is that it can divide the GPU into several "mini GPUs". | |
| 13:40:36 | kashyap | s/mini/flexible/ | |
| 13:52:36 | gibi | I associate to a fighter jet when I see MIG. Can we virtualize fighter jets these days? :) | |
| 13:55:08 | kashyap | gibi: Heh, I too had to overwrite my memory to refer to a GPU instead of a fighter-jet | |
| 13:58:40 | bauzas | gibi: lolelly | |
| 13:59:39 | gibi | we had MIGs in the past, nowadays we have Grippens | |
| 14:00:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Block servers with vGPU and device profile in heal_allocations https://review.opendev.org/c/openstack/nova/+/803603 | |
| 14:01:31 | gibi | bauzas: fixed the failing unit tests there ^^ | |
| 14:08:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Rename 'parameter_types.hostname' -> 'fqdn' https://review.opendev.org/c/openstack/nova/+/778549 | |
| 14:08:17 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732 | |
| 14:08:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add support for 'hostname' parameter https://review.opendev.org/c/openstack/nova/+/778550 | |
| 14:08:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Pass request to 'addresses' view builder https://review.opendev.org/c/openstack/nova/+/803753 | |
| 14:08:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary 'base_' prefix from schemas https://review.opendev.org/c/openstack/nova/+/803754 | |
| 14:08:19 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Sanity check some tests https://review.opendev.org/c/openstack/nova/+/803755 | |
| 14:08:20 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove various legacy extension prefixes https://review.opendev.org/c/openstack/nova/+/803756 | |
| 14:08:21 | opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Rename prefixed parameter definitions https://review.opendev.org/c/openstack/nova/+/803757 | |
| 14:08:21 | opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Update docs for microversion 2.89 https://review.opendev.org/c/openstack/nova/+/803758 | |
| 14:11:06 | bauzas | gibi: maybe you got my last +2 before I leave | |
| 14:11:38 | gibi | bauzas: thanks | |
| 14:27:58 | opendevreview | Felix Huettner proposed openstack/nova stable/stein: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803762 | |
| 14:27:59 | opendevreview | Felix Huettner proposed openstack/nova stable/stein: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803763 | |
| 14:29:57 | opendevreview | Felix Huettner proposed openstack/nova stable/rocky: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803764 | |
| 14:29:58 | opendevreview | Felix Huettner proposed openstack/nova stable/rocky: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803765 | |
| 14:30:50 | opendevreview | Felix Huettner proposed openstack/nova stable/queens: Add functional test for bug 1937375 https://review.opendev.org/c/openstack/nova/+/803766 | |
| 14:30:51 | opendevreview | Felix Huettner proposed openstack/nova stable/queens: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/803767 | |
| 14:34:34 | opendevreview | Merged openstack/nova master: Add regression test for bug 1938326 https://review.opendev.org/c/openstack/nova/+/802801 | |
| 14:38:02 | stephenfin | bauzas: done. Left comments on the last three patches in the series | |
| 14:38:23 | bauzas | stephenfin: OK, I have 20 mins to look at them ;) | |
| 15:11:22 | gibi | stephenfin: I have a question about the documentation of the ignoring of use_db_reconnect in https://review.opendev.org/c/openstack/nova/+/799524/3//COMMIT_MSG#24 | |
| 15:12:06 | stephenfin | gibi: I can look into removing it from nova's config opts, but I have deprecated the option in oslo.db so we will be dropping it soon enough | |
| 15:12:45 | stephenfin | gibi: https://review.opendev.org/c/openstack/oslo.db/+/798133 | |
| 15:13:31 | stephenfin | oh, wait, wrong config option | |
| 15:13:34 | stephenfin | :) | |
| 15:13:51 | gibi | you almost tricked me :) | |
| 15:14:09 | stephenfin | gibi: how about a simple check on nova-compute startup to warn if the option is set to a non-default value? | |
| 15:14:23 | stephenfin | i.e. "this option don't do nothing, foooool" | |
| 15:14:50 | stephenfin | I don't think the config generator supports excluding individual opts | |
| 15:15:37 | gibi | stephenfin: you mean other services than nova-compute as nova-compute cannot use direct db access | |
| 15:15:50 | stephenfin | sorry, yes | |