| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-06 | |||
| 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 | |
| 15:16:05 | gibi | an added warning works for me | |
| 15:16:11 | stephenfin | actually | |
| 15:16:21 | stephenfin | I have a patch to register options ourselves somewhere | |
| 15:16:31 | stephenfin | wonder can I simply del that option... | |
| 15:16:55 | stephenfin | actually, that won't help the docs generation problem | |
| 15:17:02 | stephenfin | warning it is | |
| 15:17:29 | gibi | ack | |
| 15:22:12 | opendevreview | Sylvain Bauza proposed openstack/nova master: Provide the mdev class for every PCI device https://review.opendev.org/c/openstack/nova/+/802918 | |
| 15:22:12 | opendevreview | Sylvain Bauza proposed openstack/nova master: Provide and use other RCs for mdevs if needed https://review.opendev.org/c/openstack/nova/+/803233 | |
| 15:22:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: Expose the mdev class https://review.opendev.org/c/openstack/nova/+/801743 | |
| 15:22:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Cleanup GPU vs. mdev wording https://review.opendev.org/c/openstack/nova/+/803379 | |
| 15:22:17 | bauzas | gibi: stephenfin: uploaded latest rev of mdev series due to stephenfin's nits ^ | |
| 15:22:36 | bauzas | stephenfin: to make it clear, we accept FOO as a mdev_class option value | |
| 15:22:45 | bauzas | then, we persist FOO in the object set | |
| 15:22:52 | stephenfin | bauzas: okay, what is FOO used for in that case? | |
| 15:23:09 | stephenfin | does it correspond to something on the device or in libvirt? | |
| 15:23:14 | bauzas | but when looking at the mdev_classes, we use the normalize_rc command to provide a CUSTOM_FOO RC name | |
| 15:23:17 | stephenfin | or is it purely used as an identifier? | |
| 15:23:36 | bauzas | that's why we have the helper and why we don't directly return the list of classes | |
| 15:23:55 | bauzas | stephenfin: no, purely an identifier given by the operator | |
| 15:24:08 | stephenfin | Okay, then why not insist on the CUSTOM_ prefix? | |
| 15:24:08 | bauzas | it could be STEPHENFIN | |
| 15:24:40 | bauzas | stephenfin: we could... but we haven't said this in the spec ;) | |
| 15:24:48 | stephenfin | it seems sensible, given this will be used placement | |
| 15:25:20 | stephenfin | I personally would expect my device with mdev_class FOO to use the same value in placement | |
| 15:25:24 | stephenfin | ...if I didn't know better | |