| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-30 | |||
| 12:36:39 | sean-k-mooney | they add soem addtion coverage but not a lot | |
| 12:36:44 | chandankumar | Ok I will fine that patch and move it to api test | |
| 12:37:15 | sean-k-mooney | have you tested those with the pci-sim? | |
| 12:37:23 | chandankumar | I tested it with pci-sim | |
| 12:37:59 | sean-k-mooney | ack so ya if you move them to api i think that woudl be fien. we could keep them in secario to as you said its somewhere in between | |
| 12:38:01 | chandankumar | sorry let me double check | |
| 12:38:31 | sean-k-mooney | i think it shoudl work as long as you set the correct adreses | |
| 12:38:39 | chandankumar | tested here: https://review.opendev.org/c/openstack/cyborg/+/977083/51/.zuul.yaml | |
| 12:38:46 | chandankumar | nvmevirt patch | |
| 12:39:19 | chandankumar | https://123b844dd0bc940cd071-d2975a8f065f970ca29e743c021b8a36.ssl.cf1.rackcdn.com/openstack/5f698c9d53e44e52bbd22509f59b8b25/testr_results.html | |
| 12:39:30 | chandankumar | test_pci_device_discovery[id-b3d5e7f9-1a2c-4e6f-8b0d-2c4e6f8b0d2e] | |
| 12:39:33 | sean-k-mooney | whats the plan for that by the way | |
| 12:39:52 | sean-k-mooney | are we goign to abandon the nvmevirt patches in favor of your new nvme pci-sim series | |
| 12:40:05 | chandankumar | yes correct | |
| 12:40:10 | chandankumar | drop nvmevirt patches | |
| 12:40:41 | chandankumar | since we can achieve almost everyhting in first party CI | |
| 12:41:29 | chandankumar | sean-k-mooney: I am not sure you saw this message https://review.opendev.org/c/openstack/cyborg/+/995079/6#message-fd9b5227f7e11e2defeda4a79e4b1b19d0f3c0b2 | |
| 12:41:49 | chandankumar | we can see nvme block in lsblk output in guest vm but we need a ubuntu image | |
| 12:42:17 | sean-k-mooney | well i guess it just need an image with the kernel nvme driver | |
| 12:42:26 | chandankumar | yes | |
| 12:42:27 | sean-k-mooney | i might add that to the list of cirrors changes i have in flight | |
| 12:42:41 | chandankumar | cool, that would be awesome | |
| 12:42:52 | sean-k-mooney | we coudl use lspci without the driver by the way | |
| 12:43:07 | sean-k-mooney | as an inital test | |
| 12:43:16 | sean-k-mooney | we just wont be able to run the lsblk part | |
| 12:43:18 | chandankumar | yup, in cirros we can see nvme in lspci output | |
| 12:43:55 | sean-k-mooney | i dont realy have an issue with using a alternitive guest image for this test but ya we coudl make that part optional. | |
| 12:44:21 | chandankumar | that also sounds good | |
| 12:44:23 | sean-k-mooney | did you see my comment about needing to be abel to have 1 job with both nvme and uart pci-sim pfs | |
| 12:44:37 | chandankumar | yes, will work on that | |
| 12:44:55 | sean-k-mooney | cool, my hope is when we add mdev supprot to pci-sim | |
| 12:45:04 | sean-k-mooney | we will be able to validate teh 3 generic driver in one job | |
| 12:45:27 | sean-k-mooney | that will require makign the devstack pluging a little smarter but i think its quite doable | |
| 12:46:17 | sean-k-mooney | also i dont know if you read the review guide i created. i need to actully do a deepre review myslef but im goign to go look at the spec shortly before the irc meeting | |
| 12:49:01 | chandankumar | paste.opendev.org/show/bDXSGmjCs9TH0NRXcVaK/ not yet, it is in my list. | |
| 12:50:50 | opendevreview | chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349 | |
| 12:53:10 | chandankumar | sean-k-mooney: two more reviews good to merge https://review.opendev.org/c/openstack/cyborg/+/994816 and https://review.opendev.org/c/openstack/cyborg/+/994830 from Melanie. | |
| 12:58:11 | sean-k-mooney | ah the api sample tests and api refe set ya i skimmed them breifly but ill try and review those later in the week | |
| 12:58:37 | chandankumar | sure | |
| 13:00:42 | jgilaber | Hi all! IRC meeting will start in ~ 1 hour, add your topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L48 | |
| 13:37:15 | opendevreview | chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349 | |
| 13:48:07 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Refactor RequestContext https://review.opendev.org/c/openstack/cyborg/+/995195 | |
| 13:52:08 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205 | |
| 14:00:33 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:00:33 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:00:33 | opendevmeet | Meeting started Tue Jun 30 14:00:33 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 14:00:33 | jgilaber | #startmeeting cyborg | |
| 14:00:47 | jgilaber | Hello! Who is around today? | |
| 14:00:47 | sean-k-mooney | o/ | |
| 14:00:51 | chandankumar | o/ | |
| 14:01:04 | jgilaber | While we gather, feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L48 | |
| 14:01:24 | rlandy | o/ | |
| 14:01:27 | jgilaber | courtesy ping: amoralej bogdando | |
| 14:01:33 | morenod | o/ | |
| 14:01:38 | amoralej | o/ | |
| 14:01:53 | jgilaber | let's give folks a minute and we'll start with the first topic | |
| 14:02:18 | jgilaber | #topic Reminder: Spec freeze on July 2 | |
| 14:02:29 | jgilaber | we have two specs open | |
| 14:02:32 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg-specs/+/982276 | |
| 14:02:42 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg-specs/+/985349 | |
| 14:03:18 | melwitt | o/ | |
| 14:03:19 | jgilaber | I'll do another pass of the nvme spec after the meeting but I think that one is good to go | |
| 14:03:37 | chandankumar | I will take a look at mdev spec tomorrow | |
| 14:03:57 | sean-k-mooney | jgilaber: i just left comments on yours +1 with minor change im happy to merge it | |
| 14:04:06 | sean-k-mooney | chandankumar: yours is next on my list | |
| 14:04:09 | sean-k-mooney | for after the meeting | |
| 14:04:19 | jgilaber | ack thanks sean-k-mooney I'll look after the meeting | |
| 14:04:23 | chandankumar | thank you! | |
| 14:04:47 | jgilaber | looks like we're in good shape, anything else on this topic? | |
| 14:05:30 | jgilaber | ok, next topic | |
| 14:05:33 | jgilaber | #topic Freezing python-cyborgclient improvements | |
| 14:05:45 | jgilaber | this one if from rlandy I believe | |
| 14:05:51 | jgilaber | #link https://review.opendev.org/c/openstack/python-cyborgclient/+/994696 | |
| 14:05:54 | rlandy | yes... | |
| 14:06:18 | rlandy | this change started with bug: https://bugs.launchpad.net/python-cyborgclient/+bug/2146351 | |
| 14:06:39 | rlandy | which requires removing the old v1 API | |
| 14:06:48 | rlandy | from cyborgclient | |
| 14:07:09 | rlandy | but there is also the piece of sdk accelerator v2 support | |
| 14:07:44 | rlandy | looking at the description, the bug now contains many parts | |
| 14:08:02 | rlandy | and we need to rethink the approach here | |
| 14:08:15 | rlandy | as initially the plan was the keep using the openstacksdk | |
| 14:08:36 | rlandy | so chandankumar and I are bringing this to the team to establish a direction | |
| 14:08:47 | rlandy | because this is probably bigger than one bug | |
| 14:09:08 | rlandy | and we need to decide what should be addressed immediately vs taking a longer design | |
| 14:09:36 | jgilaber | problem 3 in the bug report seems backwards to me | |
| 14:09:56 | jgilaber | I don't think we want to move the accelerator proxy to the client | |
| 14:10:10 | chandankumar | yes, that needs to be fixed | |
| 14:10:15 | sean-k-mooney | rlandy: so the python-cybrogclient | |
| 14:10:19 | sean-k-mooney | alrady uses the sdk | |
| 14:10:22 | sean-k-mooney | for v2 | |
| 14:10:34 | sean-k-mooney | so this was orgianly intened to track only the v1 removal | |
| 14:10:50 | melwitt | yeah what's in the description there looks like specless blueprint material | |
| 14:11:08 | rlandy | ok - so that is possibly the right direction - to make the bug just cover v1 removal | |
| 14:11:23 | rlandy | and move the rest to some spec/blueprint etc. | |
| 14:11:23 | sean-k-mooney | yes | |
| 14:11:37 | sean-k-mooney | we did dicss this in the ptg | |
| 14:11:49 | sean-k-mooney | so the propsoal was to remove v1 this cycle and deprecate teh shell | |
| 14:12:00 | rlandy | so I would propose to abandon this change - and it complicates things | |
| 14:12:10 | rlandy | and just start over with the v1 removal | |
| 14:12:12 | sean-k-mooney | by shell i mean cybrog command | |