| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-06 | |||
| 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 | |
| 15:32:02 | bauzas | in another guest | |
| 15:32:06 | zigo | bauzas: No that's not the case. | |
| 15:32:15 | zigo | It's the only instance of this cloud with a GPU. | |
| 15:32:33 | zigo | (even though we have many available: one per compute) | |
| 15:33:11 | zigo | bauzas: Will you be able to help me later on? | |
| 15:33:23 | zigo | bauzas: It's an instance for Debconf, so it's a good cause ! :) | |
| 15:35:12 | gibi | zigo: bauzas is about to drop of for 3 weeks of PTO in -34 minutes. so your question came just in time :) | |
| 15:35:30 | zigo | Ah, shit ... :) | |
| 15:35:36 | zigo | I'll figure it out, no worries. | |
| 15:35:42 | zigo | bauzas: Have nice holidays ! | |
| 15:35:51 | bauzas | and I need to work on stephenfin's concern | |
| 15:36:01 | gibi | zigo: so no other instance is using pci device 0000:41:00.0 on that host? | |
| 15:36:09 | zigo | gibi: Nop. | |
| 15:36:17 | gibi | still libvirt says it is used | |
| 15:36:22 | stephenfin | gibi: I've all but the nova-manage patch of the BW series reviewed. I assume that one can wait a while (it's in merge conflict). Also, melwitt is probably well placed to review that specifically | |
| 15:36:46 | gibi | stephenfin: thanks a lot. I'm still chewing on on your db series. | |
| 15:37:07 | gibi | stephenfin: sure the nova-manage part is the least important and it still misses some test coverage | |
| 15:37:34 | gibi | stephenfin: so you are out next week, will you be back still before the FF? | |
| 15:37:47 | stephenfin | nope, I'm here next week but I'm starting my move the new team | |
| 15:37:50 | gibi | zigo: have you tried to restart libvirtd ? | |
| 15:38:00 | gibi | stephenfin: ohh, then I mixed it | |
| 15:38:11 | gibi | stephenfin: I will be out next week :/ | |
| 15:38:14 | stephenfin | I was supposed to be away but things came up | |
| 15:38:25 | stephenfin | I'll still be around for the rest of the cycle but in a reduced capacity | |
| 15:38:52 | stephenfin | so keep reviewing what I've already reviewed and rebasing my own patches, but no new work | |
| 15:38:56 | gibi | stephenfin: OK, I might corner you with some re-review for the packet rate series if needed and you wil see if you have time or not | |
| 15:39:05 | stephenfin | yup, sounds good :) | |
| 15:39:17 | stephenfin | there are two patches with TODOs left that I can see still | |
| 15:39:34 | gibi | zigo: it feels like it is some internal incosistency inside libvirtd | |