| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-02 | |||
| 14:42:20 | jgilaber | I don't understand how the current version of chandankumar patches passed in ci with the new test | |
| 14:42:47 | jgilaber | the agent manager calls explicitely the fpga driver program method | |
| 14:42:51 | chandankumar | jgilaber: it is just faking the device path, there is no real check | |
| 14:42:55 | sean-k-mooney | jgilaber: yes it does | |
| 14:43:30 | sean-k-mooney | jgilaber: well technially it doesnt | |
| 14:43:38 | sean-k-mooney | it called the program funciton on the driver object | |
| 14:43:49 | sean-k-mooney | jgilaber: by inheriting form the genic fgpa class | |
| 14:43:56 | sean-k-mooney | the fake driver gained a program funciton | |
| 14:44:12 | sean-k-mooney | which is why its workign but that not really the correct way this should work | |
| 14:44:39 | sean-k-mooney | jgilaber: does that make sense | |
| 14:45:00 | jgilaber | sort of, but I need to stare at it for a bit | |
| 14:45:07 | jgilaber | I'll do that offline though | |
| 14:45:12 | jgilaber | we can continue | |
| 14:45:26 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/991027/2/cyborg/accelerator/drivers/fake.py | |
| 14:45:43 | sean-k-mooney | so progam call update and update redutr True | |
| 14:46:04 | sean-k-mooney | and because that added program the agened didnt explode when it did driver.program | |
| 14:46:24 | sean-k-mooney | here https://github.com/openstack/cyborg/blob/2875d3c12d4484e9336ba5084f32f2acf83a2366/cyborg/agent/manager.py#L184 | |
| 14:46:30 | sean-k-mooney | but cool lets move on | |
| 14:47:07 | jgilaber | oh I misread the variable, I get it now thanks! | |
| 14:47:13 | chandankumar | I will propose a patch to make program method public and remove update , can back on that patch again | |
| 14:47:16 | sean-k-mooney | chandankumar: so meta comment | |
| 14:47:22 | sean-k-mooney | you need to split this patch in 2 | |
| 14:47:37 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/991027 shoudl now bopth remvoe the image signiture verficaion | |
| 14:47:48 | sean-k-mooney | and try and hook up programing supprot in the fake dirver | |
| 14:48:01 | sean-k-mooney | thsoe are 2 compeletely diffent activeties so they shoudl be in 2 diffent commits | |
| 14:48:26 | chandankumar | yes, sure! | |
| 14:48:44 | sean-k-mooney | and based on the disucss abvoe we may want to fix how the agent calls the driver ectra as well in a sperate commit | |
| 14:49:04 | chandankumar | ok | |
| 14:49:52 | chandankumar | Now I have next course of action on these | |
| 14:50:06 | chandankumar | thank you! | |
| 14:50:15 | jgilaber | thanks chandankumar! | |
| 14:50:33 | jgilaber | sean-k-mooney, you mentioned before you wanted to hightlight some patches? | |
| 14:51:15 | sean-k-mooney | yes i think im now happy enough with the current content to move forward with https://review.opendev.org/c/openstack/cyborg/+/989470 | |
| 14:51:39 | sean-k-mooney | i do plan ot add a followup with more detailed docuemeation on exactly how the kernel modlule works | |
| 14:51:53 | sean-k-mooney | but i want to work on that in parallel with creating some tempest tests to test the pci driver | |
| 14:52:25 | sean-k-mooney | as part of that change i have future tweaked the ci jobs | |
| 14:52:27 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/989470/5/.zuul.yaml | |
| 14:52:47 | sean-k-mooney | so the default tempest job will now be multi node (the ipv6 one will be single node) | |
| 14:52:57 | chandankumar | thank you for taking care of multinode changes in that patch | |
| 14:53:00 | sean-k-mooney | the jobs are also mvoed to debian 13 | |
| 14:53:23 | sean-k-mooney | i may add ubuntu 24.04 at some point btu it shoudl already work on ubuntu 26.04 | |
| 14:53:33 | sean-k-mooney | so im hoping ot avoid athat instead | |
| 14:53:45 | jgilaber | do we have a requirement to have some job on ubuntu? | |
| 14:53:46 | sean-k-mooney | its also tbd if i will add centos 10 stream supprot ro not | |
| 14:53:58 | sean-k-mooney | if we need that for some reason we can consider it in the future | |
| 14:53:58 | jgilaber | without the new module I mean | |
| 14:54:05 | sean-k-mooney | jgilaber: no | |
| 14:54:13 | sean-k-mooney | we can but we dont | |
| 14:54:22 | jgilaber | ack, then debian it's fine | |
| 14:54:22 | sean-k-mooney | at least not based on the current runtims | |
| 14:54:31 | sean-k-mooney | but the grendae josb are kep on ubuntu | |
| 14:54:41 | sean-k-mooney | with the module disabled | |
| 14:54:46 | sean-k-mooney | so we actully still have coverage | |
| 14:54:52 | jgilaber | perfect then | |
| 14:55:25 | sean-k-mooney | ideally once 26.04 is workign properly in the ci | |
| 14:55:33 | sean-k-mooney | i can move the ipv6 job to that | |
| 14:56:08 | sean-k-mooney | getting 24.04 to work likely would not be hard but ofr now i just wanted to minimise the kernel spread | |
| 14:56:45 | sean-k-mooney | i also want o highligh one other fix | |
| 14:56:48 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/989470/5/devstack/lib/cyborg | |
| 14:57:01 | sean-k-mooney | its pretty minor but cybrog shoudl not have nova's password | |
| 14:57:25 | chandankumar | ah yes correct | |
| 14:57:29 | sean-k-mooney | so we shoudl be using the cybrog account to talk to nova and placemetn | |
| 14:57:35 | sean-k-mooney | so i just fixe that here | |
| 14:58:00 | jgilaber | +1 good catch | |
| 14:58:05 | sean-k-mooney | any way im going to leave this open for a few days for ye to take a look | |
| 14:58:22 | sean-k-mooney | and then ill move forward with it at the end fo the week if ye dont find anythign concerning | |
| 14:58:41 | jgilaber | thanks, I'll prioritise this review after I'm done with the specs | |
| 14:58:46 | sean-k-mooney | the other time very quickly is https://review.opendev.org/c/openstack/cyborg-specs/+/989003 | |
| 14:58:59 | sean-k-mooney | ill update the assingee to be me | |
| 14:59:18 | sean-k-mooney | jgilaber has already teken a look | |
| 14:59:46 | sean-k-mooney | if there is no other feedback on that ill proceed with that spec again later in teh week or early next week | |
| 15:00:30 | sean-k-mooney | my over all plan is, get the pci driver testable in teh gate, add some tempest test for it, then start working on adding this fucntionality after | |
| 15:01:00 | sean-k-mooney | so ya if there is any other feedback let me know that all i had for reviews | |
| 15:01:25 | jgilaber | thanks! | |
| 15:01:35 | jgilaber | we're just over time | |
| 15:01:45 | jgilaber | is there any last minute topic that you want to raise quickly | |
| 15:01:47 | jgilaber | ? | |
| 15:02:04 | sean-k-mooney | not cirtical but i notice the nova spec for mdevs | |
| 15:02:06 | sean-k-mooney | merged | |
| 15:02:14 | jgilaber | yes, it merged yesterday | |
| 15:02:16 | sean-k-mooney | so ill also try an loop back to yoru cyborg one | |
| 15:02:30 | jgilaber | I'll try to get started on that work this week | |
| 15:02:42 | jgilaber | thanks! | |
| 15:02:53 | jgilaber | final topic | |
| 15:02:57 | jgilaber | #topic Volunteers to chair next meeting | |
| 15:03:00 | jgilaber | any volunteer? | |
| 15:03:13 | chandankumar | i will take care of next meeting | |
| 15:03:19 | sean-k-mooney | +1 | |
| 15:03:19 | jgilaber | thanks! | |
| 15:03:33 | jgilaber | we can leave it here for today, thanks all! | |
| 15:03:35 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-02-14.00.log.html | |
| 15:03:35 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-02-14.00.txt | |
| 15:03:35 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-02-14.00.html | |
| 15:03:35 | opendevmeet | Meeting ended Tue Jun 2 15:03:35 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 15:03:35 | jgilaber | #endmeeting | |
| 15:06:10 | sean-k-mooney | jgilaber: quick question | |
| 15:06:20 | sean-k-mooney | are you planning to start on the nvoa side fo that spec first | |
| 15:06:55 | sean-k-mooney | i.e. reviging the patchs for the mdev ci job and then the owner traits | |
| 15:07:05 | jgilaber | yes, I was planning to start on the nova side | |
| 15:07:30 | sean-k-mooney | ack that what i was going to suggest since i feel like we will have less of a reivew bottelneck on the cybrog side | |