| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-27 | |||
| 14:50:38 | opendevreview | Gihong Lee proposed openstack/cyborg master: Migrate accelerator drivers to shared lspci helpers https://review.opendev.org/c/openstack/cyborg/+/998817 | |
| 16:49:20 | melwitt | sean-k-mooney: fyi this is the KNOWN_NICS patch https://review.opendev.org/c/openstack/cyborg/+/997782 that came out of the smartnic spike https://gist.github.com/melwitt/bb7979d9b1e06cba4a0702e90b4211ab | |
| 16:49:57 | sean-k-mooney | yes i saw it brifely last week | |
| 16:50:14 | melwitt | ah ok cool | |
| 16:50:25 | sean-k-mooney | long term i think we shoudl look to deprecat that driver and the nic driver type in general | |
| 16:50:31 | sean-k-mooney | but for now that is cool | |
| 16:50:36 | sean-k-mooney | were you able to test this end to end? | |
| 16:51:12 | melwitt | yeah I was thinking that too and briefly discussed during the last cyborg meeting | |
| 16:51:20 | melwitt | yes the end to end test worked | |
| 16:51:31 | melwitt | (it's in that github gist) | |
| 16:51:38 | sean-k-mooney | wiht neutron or just with reqing it via a device profile | |
| 16:52:21 | sean-k-mooney | so there are 2 wasy to consume this right now, you can create a neutron port of type acclerator-direct | |
| 16:52:43 | melwitt | with neutron | |
| 16:52:44 | sean-k-mooney | and the sriov nic agent shoudl be able to manage it and you shoudl be able to use the vf via a neutron network | |
| 16:52:48 | sean-k-mooney | ok cool | |
| 16:52:56 | sean-k-mooney | it shoudl also work via the flavor | |
| 16:53:16 | melwitt | there is a minimalistic patch I could do to duplicate the logic from the pci driver (I think?) to use a whitelist conf instead of the KNOWN_NICS list in the shorter term since removing the nic driver would be a larger change | |
| 16:53:21 | sean-k-mooney | melwitt: so looking at https://review.opendev.org/c/openstack/cyborg/+/997782 i think its ok | |
| 16:53:34 | sean-k-mooney | it may be nice to ether add docs to this patch or to have a seperate followup | |
| 16:54:12 | sean-k-mooney | melwitt: well so if you jsut used teh whitelist tha woudl work with the pci driver today without code changes | |
| 16:54:35 | sean-k-mooney | the biter question is will that still work with neutron | |
| 16:55:04 | sean-k-mooney | the intel driver in theory allwos use to encode info about the pshenet as traits on the RP | |
| 16:55:24 | sean-k-mooney | i think you can do that with the generic driver manually | |
| 16:55:39 | sean-k-mooney | the larger question i had was does neutron or nova care | |
| 16:55:58 | sean-k-mooney | i.e. is anythign today enfocing the VF and phsnet match end to end | |
| 16:56:50 | melwitt | sean-k-mooney: you mean like "how to" kind of docs? | |
| 16:57:09 | melwitt | I can add that, just wasn't sure if I should or if the support matrix is all that was wanted | |
| 16:57:40 | sean-k-mooney | yes a how to use the intel driver to provide cyborg manged neutron ports | |
| 16:57:54 | sean-k-mooney | and if it worked with the generic drive we coudl add that later | |
| 16:58:19 | sean-k-mooney | i assume the generic pci driver will fail in nova | |
| 16:58:37 | sean-k-mooney | becuase in teh nic driver we encode the physnet info in the attachmetn handel https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/nic/intel/sysinfo.py#L236-L243 | |
| 16:58:44 | melwitt | hm ok, I can give that a try | |
| 16:59:14 | sean-k-mooney | so assume that is checked somewhere | |
| 16:59:24 | sean-k-mooney | but maybe it is not | |
| 16:59:53 | melwitt | gotcha | |
| 17:00:04 | sean-k-mooney | melwitt: basiclly what i want to confirm is if we have 2 nic ports on phsynet A and B | |
| 17:00:15 | sean-k-mooney | do we actully passh the phsyent info end to end and enforce it | |
| 17:00:26 | melwitt | yeah, last meeting we were chatting about the idea of "moving" the nic driver stuff into the pci driver and just have a little bit of "if device type" to handle them both | |
| 17:00:28 | sean-k-mooney | or was that missed when this was orgianlly devleoped | |
| 17:00:47 | sean-k-mooney | melwitt: ya that could work | |
| 17:01:19 | sean-k-mooney | or like in nova we coudl add teh physical_network tag to the whitelist config option | |
| 17:01:23 | melwitt | ok I can look into that with the physnet | |
| 17:01:48 | sean-k-mooney | cool for now i think we can proceed with your small patch | |
| 17:01:57 | sean-k-mooney | adn then we can followup with docs and more testing | |
| 17:02:04 | sean-k-mooney | melwitt: does that work for you? | |
| 17:02:06 | melwitt | cool sounds good | |
| 17:09:15 | sean-k-mooney | melwitt: jgilaber incase this passed ye by this is an interisting patch https://review.opendev.org/c/openstack/cyborg/+/998726 | |
| 17:09:38 | sean-k-mooney | i was going to look at this as part fo large rework next cycle | |
| 17:09:56 | sean-k-mooney | but even without that the cleanup form conslitdating woudl be nice to have | |
| 17:10:40 | sean-k-mooney | my incliation is to allow that to prpoceed with a fairly narrow scope this cycle and build on it next cycle to intoduce a PCIBusManager with real objects instead of just strings | |
| 17:10:43 | melwitt | oh yeah, that's cool | |
| 17:11:03 | melwitt | makes sense | |
| 17:27:02 | sean-k-mooney | amoralej: so while the spec was proposed in yoga the keyston bootstrap command only created the manager role form bobcat | |
| 17:27:04 | sean-k-mooney | https://github.com/openstack/keystone/commit/bbbbd10e482a67b53187a93daa58f8bee770852f | |
| 17:27:39 | sean-k-mooney | amoralej: so ya htat why it works upstream and fails with keystone form antelope | |
| 18:10:19 | opendevreview | Merged openstack/cyborg master: Add Intel XXV710 25GbE backplane NIC to known NIC list https://review.opendev.org/c/openstack/cyborg/+/997782 | |
| 18:19:44 | jgilaber | sean-k-mooney, thanks I saw the patch and added it to my review list, but have not really looked much | |
| 18:42:18 | sean-k-mooney | jgilaber: no worries i am more higlighing it in case it impact your other work and jsut incase you found it interesting | |
| 18:42:32 | sean-k-mooney | its proably more impactfult ot chandan actully | |
| 18:43:12 | jgilaber | yes I don't think it'll conflict with my patches | |
| #openstack-cyborg - 2026-07-28 | |||
| 07:16:41 | opendevreview | chandan kumar proposed openstack/cyborg master: Allow non-CUSTOM traits in device profile groups https://review.opendev.org/c/openstack/cyborg/+/997945 | |
| 10:31:42 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | |
| 10:31:42 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Add Cyborg MDEV driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997352 | |
| 12:23:09 | opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | |
| 12:23:10 | opendevreview | Gihong Lee proposed openstack/cyborg master: Migrate accelerator drivers to shared lspci helpers https://review.opendev.org/c/openstack/cyborg/+/998817 | |
| 12:59:21 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | |
| 12:59:21 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Add Cyborg MDEV driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997352 | |
| 13:09:50 | chandankumar | cyborg irc meeting will start in another 50 mins, Add your agenda here: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L47 | |
| 14:00:54 | chandankumar | time for cyborg meeting! | |
| 14:01:01 | chandankumar | #startmeeting cyborg | |
| 14:01:01 | opendevmeet | Meeting started Tue Jul 28 14:01:01 2026 UTC and is due to finish in 60 minutes. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 14:01:01 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:01:01 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:01:06 | jgilaber | o/ | |
| 14:01:10 | morenod | o/ | |
| 14:01:11 | chandankumar | who all are around? | |
| 14:01:11 | rlandy | o/ | |
| 14:01:46 | opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | |
| 14:01:54 | chandankumar | #courtesy ping sean-k-mooney amoralej | |
| 14:02:34 | chandankumar | Here is the link to agenda | |
| 14:02:42 | chandankumar | #link https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | |
| 14:02:56 | chandankumar | feel free to add there | |
| 14:03:05 | sean-k-mooney | o/ | |
| 14:03:13 | melwitt | o/ | |
| 14:03:40 | chandankumar | Let me start with the first topic | |
| 14:03:56 | chandankumar | #topic Proposal for pci-sim to support one device type per pf | |
| 14:04:06 | chandankumar | jgilaber: added it Add per-PF device type via pf_mode parameter | https://review.opendev.org/c/openstack/cyborg/+/998603 | |
| 14:04:17 | chandankumar | #link https://review.opendev.org/c/openstack/cyborg/+/998603 | |
| 14:04:19 | jgilaber | yes, we've discussed about this in the past | |
| 14:04:29 | jgilaber | since we plan to add mdevs and nvme emulation to pci-sim | |
| 14:04:40 | jgilaber | we need some interface for configuring that | |
| 14:04:53 | jgilaber | I've pushed that patch to propose one possibility | |
| 14:05:05 | jgilaber | and I'm looking for feedback before rebasing my mdev work on top of it | |
| 14:05:37 | jgilaber | it adds a 'pf_mode' parameter to the module | |
| 14:05:51 | jgilaber | which is a comma-separated list of types, one per pf | |
| 14:06:21 | jgilaber | e.g 'insmod fake_pci_sriov.ko num_pfs=2 pf_mode=sriov,mdev' would create two pfs, the first one sriov, the second with mdevs | |
| 14:06:50 | sean-k-mooney | that shoudl not be via kernel args | |
| 14:07:00 | sean-k-mooney | it shoudl be via per PF files in /sys | |
| 14:07:41 | sean-k-mooney | that the same feedback i gave to chandankumar for the nvm devices as well | |
| 14:08:05 | sean-k-mooney | each PF shoudl have a personality file that the pf reads to determin the type of VF to create | |