| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-08-04 | |||
| 14:04:49 | jgilaber | courtesy ping: sean-k-mooney amoralej bogdando kovili | |
| 14:05:41 | sean-k-mooney | o/ | |
| 14:06:21 | jgilaber | ok, let's start with first topic | |
| 14:06:32 | jgilaber | #topic pci-sim personalities | |
| 14:06:50 | jgilaber | I wanted to ask a couple of questions about the pci-sim work | |
| 14:07:09 | jgilaber | since I think I assumed some things that are not necessarily true | |
| 14:07:16 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/998603 | |
| 14:07:36 | jgilaber | first thing is, Do we expect that mdevs will be created from a VF? | |
| 14:07:47 | sean-k-mooney | yes | |
| 14:08:17 | sean-k-mooney | technically vfio-mdev are entirly unrelated to pci | |
| 14:08:24 | sean-k-mooney | so you can creat them form block device or usb | |
| 14:08:26 | jgilaber | ack, I assumed that the mdev will be exposed from the PF | |
| 14:08:37 | sean-k-mooney | but nvida moved them form the pf to the vf 4 or 5 years ago | |
| 14:09:09 | sean-k-mooney | so ampreare+ creats them on the vfs only | |
| 14:09:32 | sean-k-mooney | older card like t4 created them on the pf only | |
| 14:09:45 | jgilaber | that makes the feedback from the review clear from me | |
| 14:10:12 | jgilaber | and a second question, just to clarify, we still expect one type of personality per PF, right? | |
| 14:10:19 | sean-k-mooney | yes | |
| 14:10:31 | sean-k-mooney | so vf_personality is the type of VF to create form this pf | |
| 14:11:02 | jgilaber | cool, I'll try to update the patch with that in mind, but I don't think I'll be able to rework the mdev suppor in pci-sim before I go on PTO | |
| 14:11:15 | sean-k-mooney | you could have vf_personality be on the indivugal VF but the problem with that is you want to choose the personality before you create the pci config space represetaion | |
| 14:11:30 | sean-k-mooney | since that shoudl not change after the VFs are enumerated on the pci bus | |
| 14:12:00 | jgilaber | yes I don't think we have any reason to take on the additional complexity for that | |
| 14:12:09 | jgilaber | just wanted to ensure I understood it correctly | |
| 14:12:13 | sean-k-mooney | ack i think we can rework this while your away | |
| 14:12:34 | sean-k-mooney | perhaps chandan can take over that patch? | |
| 14:12:43 | sean-k-mooney | or i can | |
| 14:12:45 | chandankumar | sure | |
| 14:13:23 | chandankumar | Once I pushed the nvme series, I will update the patch | |
| 14:13:24 | sean-k-mooney | jgilaber: did you have any other questions or discussion points on the pci sim? | |
| 14:13:32 | jgilaber | nope, that is all from me, thanks | |
| 14:13:43 | jgilaber | if there are no more questions/comments we can move on | |
| 14:14:17 | sean-k-mooney | before you go on pto it woudl be good i think to rebase your sers off of the personaltity patch | |
| 14:14:42 | sean-k-mooney | i.e. this https://review.opendev.org/c/openstack/cyborg/+/997979/8 can go directly on master right | |
| 14:15:04 | jgilaber | yes, sure | |
| 14:15:05 | sean-k-mooney | we can proceed with the vf personalites as part fo the nvme work | |
| 14:15:26 | sean-k-mooney | and the testing change at the end of your seres can merge later | |
| 14:15:45 | jgilaber | technically everything before https://review.opendev.org/c/openstack/cyborg/+/997054/15 can go directly on master | |
| 14:15:54 | sean-k-mooney | im mostly ok with merging the ci/pci-sim changes to test this after FF up to RC1 | |
| 14:16:01 | jgilaber | I'll reorder the patches after the meeting | |
| 14:16:05 | sean-k-mooney | +1 | |
| 14:16:49 | sean-k-mooney | on a related note i think the nova patches are getting closer too but i left some comment on them this morning | |
| 14:17:02 | sean-k-mooney | not sure if you will have time to adress them or not before you go just an fyi | |
| 14:17:11 | jgilaber | yes, I saw them and am working on a new patchset, thanks for looking | |
| 14:17:26 | jgilaber | I think I'll have time to address them | |
| 14:19:23 | jgilaber | anything else before moving on? | |
| 14:20:11 | jgilaber | moving to reviews | |
| 14:20:23 | jgilaber | #topic Reviews | |
| 14:20:29 | jgilaber | #link https://etherpad.opendev.org/p/cyborg-2026.2-status | |
| 14:21:03 | jgilaber | I should update the mdev patches sections and add the pci-sim ones | |
| 14:21:13 | jgilaber | I'll do that after the meeting as well | |
| 14:21:26 | jgilaber | does anyone want to highligth anything from the status etherpad? | |
| 14:21:54 | chandankumar | I have added the fpga program fixes and tempest tests there. Thank you jgilaber for already reviewing it | |
| 14:22:09 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/995079/6/ | |
| 14:22:21 | chandankumar | sorry https://review.opendev.org/q/topic:%22fpga_program%22 | |
| 14:23:30 | jgilaber | thanks chandankumar it'll be good to land those fixes | |
| 14:24:32 | jgilaber | anything else? | |
| 14:24:48 | chandankumar | that's it from my side | |
| 14:25:26 | jgilaber | thanks, moving on | |
| 14:25:53 | jgilaber | documentation pages reorganization | |
| 14:26:00 | jgilaber | #link https://review.opendev.org/q/topic:%22cyborg-doc-reorg%22 | |
| 14:26:05 | rlandy | yep - that's mine | |
| 14:26:12 | rlandy | thank you for all the reviews | |
| 14:26:27 | sean-k-mooney | chandankumar: sorry ill try an take a look at those later | |
| 14:26:44 | rlandy | two patches had multiple comments and I'd like to confirm what the final suggestion is ... | |
| 14:26:50 | chandankumar | np, take your time | |
| 14:27:00 | rlandy | for the drivers table, | |
| 14:27:16 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/999116 | |
| 14:27:23 | rlandy | ack - the generic PCI driver is missing | |
| 14:27:44 | rlandy | jgilaber: not sure if you wanted to add that or I should? | |
| 14:28:03 | jgilaber | you can add the pci driver in you patch | |
| 14:28:11 | rlandy | also should this move before completing the review or was that a comment about moving all this work in general? | |
| 14:28:41 | rlandy | lastly, is the team ok with the status choices of CI and Missing? | |
| 14:28:48 | rlandy | is there another status | |
| 14:29:06 | chandankumar | good to add CI status there | |
| 14:29:16 | rlandy | I removed all the references to individual test results - they were out of date | |
| 14:29:23 | jgilaber | I think that CI and missing is accurate for now | |
| 14:29:25 | sean-k-mooney | so we currently have 2 idffent places where we list the driver in a table | |
| 14:29:33 | sean-k-mooney | the suppoort matic sand in the amdin docs | |
| 14:29:55 | sean-k-mooney | everuatly i think we shoudl have only one table in the admin docs not underfernce and hte supprot statement should move | |
| 14:30:01 | sean-k-mooney | but we can do it as 2 commtis | |
| 14:30:08 | sean-k-mooney | so if you want to defer the move that is ok | |
| 14:30:17 | sean-k-mooney | i woudl add the pci drvier now however | |
| 14:30:26 | rlandy | I can do the move in this patch, no problem | |
| 14:30:53 | rlandy | the final place would be the admin docs? | |
| 14:31:09 | sean-k-mooney | i think so | |
| 14:31:11 | rlandy | ok | |
| 14:31:22 | rlandy | next ... | |
| 14:31:22 | sean-k-mooney | refence is not realy something we expect peole to read on a first reading of the doc | |
| 14:31:31 | sean-k-mooney | its more a place we keep technial notes | |
| 14:31:44 | sean-k-mooney | so puting it in the main admin guide will make this much more visable | |
| 14:31:46 | rlandy | https://review.opendev.org/c/openstack/cyborg/+/999278 | |
| 14:31:48 | sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_805/openstack/8055e6a20e464edcbb23ba49701ca037/docs/reference/support-matrix.html | |
| 14:31:54 | sean-k-mooney | is what it looks liek for now | |
| 14:32:03 | sean-k-mooney | i woudl add a docs colume as well | |
| 14:32:24 | sean-k-mooney | and maybe make status the last of the colums | |
| 14:32:46 | sean-k-mooney | so name, description, testing, docs, status | |
| 14:33:03 | rlandy | ok | |
| 14:33:18 | rlandy | doc will be blank until we have them all? | |
| 14:33:42 | sean-k-mooney | well you can set it to missing for most of the driver and partial for the intel driver | |