| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-06 | |||
| 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: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732 | |
| 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: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:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add support for 'hostname' parameter https://review.opendev.org/c/openstack/nova/+/778550 | |
| 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: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: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: Update docs for microversion 2.89 https://review.opendev.org/c/openstack/nova/+/803758 | |
| 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: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 and use other RCs for mdevs if needed https://review.opendev.org/c/openstack/nova/+/803233 | |
| 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:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Cleanup GPU vs. mdev wording https://review.opendev.org/c/openstack/nova/+/803379 | |
| 15:22:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: Expose the mdev class https://review.opendev.org/c/openstack/nova/+/801743 | |
| 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 | bauzas | it could be STEPHENFIN | |
| 15:24:08 | stephenfin | Okay, then why not insist on the CUSTOM_ prefix? | |
| 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 | |
| 15:25:27 | stephenfin | ;) | |
| 15:25:52 | stephenfin | bauzas: does it make sense to you to do that though? We can amend the spec if needed | |
| 15:26:16 | bauzas | well, I wanted to stop working at 5pm :p | |
| 15:26:30 | bauzas | but I could do this this evening | |
| 15:26:54 | bauzas | I need to change a few things but ok | |
| 15:27:50 | bauzas | stephenfin: do you know how to accept an option value to have a specific prefix ? | |
| 15:28:02 | stephenfin | bauzas: like oslo.confg? | |
| 15:28:11 | bauzas | in oslo.config yes | |
| 15:28:13 | stephenfin | I just assumed we'd regex it away | |
| 15:28:26 | bauzas | oh shit you're right | |
| 15:28:36 | stephenfin | i.e. something like (VGPU|CUSTOM_[A-Z_]) | |
| 15:28:45 | stephenfin | but more complete | |
| 15:28:46 | stephenfin | :) | |
| 15:31:07 | zigo | bauzas: Hi there ! After a failed migration (mistake in the host ssh keys), I cannot start a GPU instance, it's telling me: "libvirt.libvirtError: internal error: Device 0000:41:00.0 is already in use" | |
| 15:31:10 | zigo | How can I fix this ? | |
| 15:31:22 | zigo | bauzas: I'm asking you because it's an instance with a GPU, as you can see... | |
| 15:31:45 | bauzas | zigo: sorrry, I need to go off | |
| 15:31:55 | bauzas | but looks like you already use the mdev | |