| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-27 | |||
| 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 | |
| 14:08:17 | sean-k-mooney | which you set before you allocate teh VFs | |
| 14:08:55 | sean-k-mooney | that is similar ot how vGPUs work and other hader deivce that supprot configurable types for there VFs | |
| 14:10:09 | jgilaber | so the user/devstack plugin creates those files? | |
| 14:10:10 | sean-k-mooney | basiclly /sys/bus/pci/devices/<PF>/vf_personality | |
| 14:10:31 | sean-k-mooney | when you mode prope the module it will create teh file | |
| 14:10:35 | opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | |
| 14:10:35 | opendevreview | Gihong Lee proposed openstack/cyborg master: Migrate accelerator drivers to shared lspci helpers https://review.opendev.org/c/openstack/cyborg/+/998817 | |
| 14:10:40 | sean-k-mooney | and you need to write the desired value to it | |
| 14:11:05 | sean-k-mooney | we shoudl also have a /sys/bus/pci/devices/<PF>/vf_personalitys that list the avlaible type space seperated | |
| 14:11:39 | sean-k-mooney | that or follow the cpu governors nameing convetion of adding _avaialble | |
| 14:12:13 | jgilaber | ok I see | |
| 14:12:21 | sean-k-mooney | [15:11:16]> cat /sys/bus/cpu/devices/cpu0/cpufreq/scaling_governor | |
| 14:12:23 | sean-k-mooney | ─────┬───────────────────────────────────────────────────────────────────────────────────────────── | |
| 14:12:25 | sean-k-mooney | │ File: /sys/bus/cpu/devices/cpu0/cpufreq/scaling_governor | |
| 14:12:27 | sean-k-mooney | ─────┼───────────────────────────────────────────────────────────────────────────────────────────── | |
| 14:12:29 | sean-k-mooney | 1 │ powersave | |
| 14:12:31 | sean-k-mooney | ─────┴───────────────────────────────────────────────────────────────────────────────────────────── | |
| 14:12:33 | sean-k-mooney | ~ via 🐹 v1.25.12 via python: v3.14.6 on ☁️ smooney@redhat.com | |
| 14:12:35 | sean-k-mooney | [15:12:08]> cat /sys/bus/cpu/devices/cpu0/cpufreq/scaling_available_governors | |
| 14:12:37 | sean-k-mooney | ─────┬───────────────────────────────────────────────────────────────────────────────────────────── | |
| 14:12:39 | sean-k-mooney | │ File: /sys/bus/cpu/devices/cpu0/cpufreq/scaling_available_governors | |
| 14:12:40 | jgilaber | from UX side it seems a bit more cumbersome | |
| 14:12:41 | sean-k-mooney | ─────┼───────────────────────────────────────────────────────────────────────────────────────────── | |
| 14:12:43 | sean-k-mooney | 1 │ performance powersave | |
| 14:12:45 | sean-k-mooney | ─────┴──────────── | |
| 14:12:57 | sean-k-mooney | the module approch forces you do reload the module to change it | |
| 14:13:08 | sean-k-mooney | and it does not allwo you to eaislly do this on a per device basis | |
| 14:13:21 | jgilaber | that's true | |
| 14:13:41 | chandankumar | sean-k-mooney: jgilaber I was trying to come out like this https://gistpreview.github.io/?9cf18df5754db0948330f0cc5110c652 but not yet implemented | |
| 14:14:43 | sean-k-mooney | skiming that that is mostly aligned on the sysfs interface to configure this | |
| 14:14:54 | sean-k-mooney | basiclly secitn 6 | |
| 14:15:31 | sean-k-mooney | we will need to document this in the contibutor docs for the module and cpature the logic as part of this work | |
| 14:15:49 | sean-k-mooney | jgilaber: the other option woudl be to have multiple modules that we load seperatly | |
| 14:15:53 | sean-k-mooney | with a shared core | |
| 14:15:59 | sean-k-mooney | i.e. one per fake device type | |
| 14:16:13 | sean-k-mooney | but i was kind fo hoping to only need one module for htis in general | |
| 14:16:47 | jgilaber | yeah, I'll try to rework the patch with this new approach | |
| 14:17:02 | jgilaber | and we can discuss again | |
| 14:17:17 | sean-k-mooney | so i think we need to spit this out into its won commit | |