| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-14 | |||
| 14:24:30 | sean-k-mooney | it could yes | |
| 14:25:01 | sean-k-mooney | im just pointing out that there is no actual depenonc eon the microversion supprot for ti to work to my knowldage | |
| 14:25:14 | sean-k-mooney | if you dont pass any microverion header its ment to defautl to 2.0 anyway | |
| 14:25:55 | sean-k-mooney | anyway we can proceed with it in this order | |
| 14:26:13 | sean-k-mooney | but it shoudl work without the microverison patch | |
| 14:26:29 | jgilaber | I see what you mean | |
| 14:26:49 | jgilaber | I'll try to review both the pci tests and the cyborg bug fix patch later today | |
| 14:27:03 | jgilaber | does anyone have any question comment on this patches? | |
| 14:28:05 | jgilaber | ok, moving on then | |
| 14:28:14 | jgilaber | #topci Generic Mdev driver | |
| 14:28:20 | jgilaber | #undo | |
| 14:28:20 | opendevmeet | Removing item from minutes: #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996821 | |
| 14:28:39 | jgilaber | ahh I did not want to do that sorry | |
| 14:28:46 | jgilaber | #topic Generic Mdev driver | |
| 14:29:12 | jgilaber | I want to mention as fyi that I have a series of patches to add the generic mdev driver | |
| 14:29:14 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/997047/1 | |
| 14:29:24 | jgilaber | it also adds mdevs to pci-sim | |
| 14:29:48 | jgilaber | I'm not asking for reviews yet, but I've tested it manually on a devstack env and it seems to work well | |
| 14:30:05 | jgilaber | I've been able to create an instance with Nova | |
| 14:30:21 | jgilaber | I plan to start with tempests test before asking for reviews | |
| 14:32:02 | sean-k-mooney | cool | |
| 14:32:25 | sean-k-mooney | did you hit any roadblocks or painpoints along the way | |
| 14:32:40 | sean-k-mooney | i.e. anything that could/should be impvoed or highlighted | |
| 14:33:29 | jgilaber | not really no, it's been quite quick | |
| 14:33:59 | jgilaber | I had claude write the pci-sim part since I did not know much about the kernel side and it did not get it right on the first try | |
| 14:34:12 | jgilaber | but it fixed it quickly | |
| 14:35:09 | sean-k-mooney | so until we add the ablity ot have diffent pf types | |
| 14:35:20 | sean-k-mooney | there is gong to be a conflcit between your work and chandans | |
| 14:35:32 | sean-k-mooney | that oen of the thing that need to be doen with care | |
| 14:35:50 | sean-k-mooney | as chandands inital verison was leaking device speific info into the generic stuctures | |
| 14:36:10 | sean-k-mooney | currently your adding enable_mdev=1 | |
| 14:36:14 | sean-k-mooney | as a module parmater | |
| 14:36:27 | sean-k-mooney | buyt that not compatible with teh per PF device personaltiy approch | |
| 14:36:43 | sean-k-mooney | still the poc is still useful | |
| 14:36:57 | sean-k-mooney | but we will have to rework the kernel part before it merges | |
| 14:36:57 | jgilaber | ack, that's not a problem I can rebase and improve the approach | |
| 14:37:16 | jgilaber | my goal was to have something working end to end | |
| 14:37:28 | sean-k-mooney | yep and in the mean time we can still use the poc to validate things | |
| 14:37:44 | jgilaber | I was not expecting the pci-sim part to be ready to merge at all | |
| 14:38:00 | sean-k-mooney | well i have just read the commit messages so far | |
| 14:38:07 | sean-k-mooney | so i cant say how close or far away it is | |
| 14:38:24 | sean-k-mooney | i was not expecting addign medev supprot to be that hard | |
| 14:38:41 | sean-k-mooney | given the pci-sim is derived form the mtty sample kernel module to begin with | |
| 14:39:26 | sean-k-mooney | jgilaber: i dont really have any other questions for now. do others? | |
| 14:39:50 | jgilaber | thanks sean-k-mooney, if there are no more questions we can move on | |
| 14:40:46 | jgilaber | we don't have anything else on the agenda for today, any last minute topics? | |
| 14:41:50 | melwitt | just thanks chandan and joan for reviews on API samples tests, still need a look from sean-k-mooney at your convenience | |
| 14:42:22 | sean-k-mooney | for the api samples | |
| 14:42:38 | sean-k-mooney | sure i can try and take a look later | |
| 14:42:53 | melwitt | thanks | |
| 14:43:46 | sean-k-mooney | i guess one other thing | |
| 14:43:54 | sean-k-mooney | i likely wont be here next week for this call | |
| 14:44:00 | sean-k-mooney | so feel free to proceed without me | |
| 14:44:43 | jgilaber | ack, noted sean-k-mooney | |
| 14:45:11 | jgilaber | anything else from anyone? if not I think we can leave it here for today | |
| 14:46:20 | jgilaber | thanks all! | |
| 14:46:23 | jgilaber | #endmeeting | |
| 14:46:23 | opendevmeet | Meeting ended Tue Jul 14 14:46:23 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:46:23 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-14-14.01.html | |
| 14:46:23 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-14-14.01.txt | |
| 14:46:23 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-14-14.01.log.html | |
| 17:11:20 | opendevreview | Takashi Kajinami proposed openstack/python-cyborgclient master: Drop explicit dependency on python-subunit https://review.opendev.org/c/openstack/python-cyborgclient/+/997251 | |
| 18:06:26 | opendevreview | Merged openstack/python-cyborgclient master: Drop explicit dependency on python-subunit https://review.opendev.org/c/openstack/python-cyborgclient/+/997251 | |
| #openstack-cyborg - 2026-07-15 | |||
| 09:09:41 | opendevreview | chandan kumar proposed openstack/cyborg master: Fix microversion header parsing for service type prefix https://review.opendev.org/c/openstack/cyborg/+/994574 | |
| 09:17:16 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add Cyborg PCI driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 | |
| 10:08:22 | 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 | |
| 10:41:05 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 10:45:34 | opendevreview | chandan kumar proposed openstack/cyborg master: Enable pci lifecycle test in cyborg tempest job https://review.opendev.org/c/openstack/cyborg/+/997365 | |
| 10:48:15 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 11:20:00 | 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 | |
| 11:21:01 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 12:02:11 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 12:57:11 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add Cyborg PCI driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 | |
| 12:58:40 | chandankumar | jgilaber: Hello, on pci life cycle test, I rewrote the get_headers method https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580/6/cyborg_tempest_plugin/services/cyborg_rest_client.py after https://github.com/openstack/cyborg-tempest-plugin/commit/0722965e6755d9c28f530d3d1a4b5e0d97e39c91 got merged | |
| 12:58:45 | chandankumar | Now tempest tests will work | |
| 13:07:25 | jgilaber | ack thanks chandankumar will take a look | |
| 13:10:57 | 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:12:26 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 13:57:22 | sean-k-mooney | chandankumar: i have not looked but i would not be surprised if tempest base client had native supprot for microverions | |
| 14:42:57 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 14:46:05 | chandankumar | sean-k-mooney: I checked the tempest repo and each of the service client have their own microversion implementation | |
| 14:46:06 | chandankumar | https://github.com/openstack/tempest/blob/48141b5d7a1377cc4bd5e779c6e6355663fe816d/tempest/lib/services/compute/base_compute_client.py#L39 | |
| 14:46:48 | chandankumar | https://github.com/openstack/tempest/blob/48141b5d7a1377cc4bd5e779c6e6355663fe816d/tempest/lib/services/placement/base_placement_client.py#L26 | |
| 14:48:17 | chandankumar | yes and you are correct | |
| 15:00:46 | sean-k-mooney | ok cool so what your propsoing is effectivly the same | |
| 15:00:52 | sean-k-mooney | or at least very similar | |
| 15:01:26 | sean-k-mooney | its directly adding the header when needed rather then passign it as a parmater to the cleints post or get method ectra | |
| 15:05:21 | sean-k-mooney | hum | |
| 15:05:35 | sean-k-mooney | but they do not have good exampel of using diffent microverion fo ra specific call | |
| 15:05:49 | sean-k-mooney | they are mearly setting the min version at the class level | |
| 15:05:52 | sean-k-mooney | so that not ideal | |
| 15:25:18 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add Cyborg PCI driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 | |
| 15:26:21 | opendevreview | chandan kumar proposed openstack/cyborg master: Enable pci lifecycle test in cyborg tempest job https://review.opendev.org/c/openstack/cyborg/+/997365 | |
| 15:26:51 | chandankumar | I have updated the tempest plugin review with cyborg_pci config section | |
| 15:28:30 | chandankumar | will check results tomorrow o/ | |
| 16:05:11 | opendevreview | Joan Gilabert proposed openstack/cyborg master: [WIP] Configure mdev driver lifecycle tests https://review.opendev.org/c/openstack/cyborg/+/997364 | |
| 17:01:02 | opendevreview | Merged openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534 | |
| 18:19:35 | opendevreview | Merged openstack/cyborg master: Fix microversion header parsing for service type prefix https://review.opendev.org/c/openstack/cyborg/+/994574 | |
| #openstack-cyborg - 2026-07-16 | |||
| 04:49:51 | opendevreview | chandan kumar proposed openstack/cyborg master: Configure pci lifecycle tempest settings via devstack plugin https://review.opendev.org/c/openstack/cyborg/+/997365 | |
| 05:36:35 | opendevreview | chandan kumar proposed openstack/cyborg master: pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079 | |