| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-14 | |||
| 14:18:40 | sean-k-mooney | sort of | |
| 14:18:47 | sean-k-mooney | it is | |
| 14:19:13 | sean-k-mooney | but there is also sort of this https://review.opendev.org/c/openstack/cyborg/+/994574 | |
| 14:19:51 | sean-k-mooney | cyborg microverion handling is incorrect | |
| 14:19:54 | sean-k-mooney | server side | |
| 14:20:32 | sean-k-mooney | it was correct but it was broken by https://opendev.org/openstack/cyborg/commit/279a260bb2d3b91136ce71c1feb29bc68a7aa962 | |
| 14:20:34 | jgilaber | is there a dependency between them? | |
| 14:20:38 | sean-k-mooney | for https://bugs.launchpad.net/openstack-cyborg/+bug/2049489 | |
| 14:21:15 | sean-k-mooney | yes and no | |
| 14:21:26 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996821 | |
| 14:21:38 | sean-k-mooney | is adding the infra to supprot micoversion but not actully using them | |
| 14:22:53 | sean-k-mooney | the pci life cycle test also do not use them | |
| 14:23:14 | sean-k-mooney | well not beyound declaring min_microversion = '2.0' | |
| 14:23:52 | jgilaber | but the plugin patch could merge before https://review.opendev.org/c/openstack/cyborg/+/994574 right? | |
| 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 | |