| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-05-12 | |||
| 14:10:51 | sean-k-mooney | lets continue to review this on gerrit | |
| 14:10:58 | sean-k-mooney | but i think the overall move makes sense | |
| 14:11:23 | sean-k-mooney | i woudl like to move all the job definiton severually | |
| 14:11:38 | jgilaber | sure, if there are no further comment/objections let's move on | |
| 14:11:39 | sean-k-mooney | as i said i dont know if we just want to keep the alis in the old place | |
| 14:11:57 | sean-k-mooney | its more a case fo ensurign we dont brake the stable branches | |
| 14:12:05 | sean-k-mooney | i dont really have any objections to the move | |
| 14:12:18 | sean-k-mooney | we just need to figure out the sequcne to make it work | |
| 14:12:44 | sean-k-mooney | the only one that really shoudl be in the tempest repo are the stable branch specific ones | |
| 14:13:56 | jgilaber | agreed, we can discuss the details like the job names in the patches, I'll push changes to the stable branches after the meeting | |
| 14:14:54 | jgilaber | #topic Bugs | |
| 14:14:57 | sean-k-mooney | well that kind of my point | |
| 14:15:02 | sean-k-mooney | we shoudlnot be modifyign the stable branches | |
| 14:15:13 | sean-k-mooney | anyway yes bugs | |
| 14:15:32 | jgilaber | we have a new bug to triage | |
| 14:15:39 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2151792 | |
| 14:17:10 | jgilaber | the bug report indicates two errors returned by the client | |
| 14:17:11 | sean-k-mooney | ya so... | |
| 14:17:27 | sean-k-mooney | that api feature was approvced in bobcat but never marked as implemtned | |
| 14:17:29 | jgilaber | one was fixed in 2025.2 | |
| 14:17:33 | sean-k-mooney | so im not sure if it actully works | |
| 14:17:57 | sean-k-mooney | and yes there is a bug in the clinet <> sdk interaction as well | |
| 14:18:44 | sean-k-mooney | looking at commente 2 par tof this seam to be verion dependent | |
| 14:19:09 | sean-k-mooney | i.e it may require a newer verison fo the sdk then is allwoed in 2025.1 | |
| 14:20:02 | sean-k-mooney | i htink we are going to have to look at when the supprot was added for this in the sdk adn client | |
| 14:20:09 | sean-k-mooney | see if it works or not on master | |
| 14:20:14 | sean-k-mooney | and then determim how to proceed | |
| 14:21:02 | jgilaber | the "attribute list" seems to work for version 4.7.2 so it should work on master too | |
| 14:21:23 | jgilaber | Chandan did reproduce the enable/disable error on master | |
| 14:21:40 | jgilaber | looking at the code it seems the api controllers are implemented https://github.com/openstack/cyborg/blob/master/cyborg/api/controllers/v2/devices.py#L157-L183 | |
| 14:22:02 | sean-k-mooney | yes they are which shoudl be the last part of the feature | |
| 14:22:09 | sean-k-mooney | but i dont know if the rest of the feature was compelted | |
| 14:23:04 | sean-k-mooney | https://review.opendev.org/q/topic:%22bp/disable-enable-device%22 | |
| 14:23:37 | sean-k-mooney | in theory the feature as added by https://review.opendev.org/c/openstack/cyborg/+/891309 | |
| 14:24:12 | jgilaber | the code looks complete | |
| 14:24:54 | sean-k-mooney | there are not tempest tests | |
| 14:25:05 | sean-k-mooney | so we dont actully know if this worked end to end | |
| 14:26:04 | jgilaber | it looks like it's missing completely from the openstacksdk proxy https://github.com/openstack/openstacksdk/blob/master/openstack/accelerator/v2/device.py | |
| 14:26:24 | sean-k-mooney | yes https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 14:26:25 | jgilaber | no reference of any variable to track if the device is enabled or disabled | |
| 14:26:27 | sean-k-mooney | was never merged | |
| 14:26:53 | sean-k-mooney | https://review.opendev.org/c/openstack/python-cyborgclient/+/894887 didnt have a depens on | |
| 14:27:21 | sean-k-mooney | but it requires the sdk change | |
| 14:27:27 | jgilaber | that explains the bug | |
| 14:27:32 | sean-k-mooney | so this feature may work at the the api level | |
| 14:27:55 | sean-k-mooney | but we need to fix thet sdk, add tempest test and test this proeprly end to end in ci | |
| 14:28:27 | sean-k-mooney | we can mark this as vaild and medium for the client | |
| 14:28:41 | jgilaber | agreed | |
| 14:28:48 | sean-k-mooney | but not for cyborg itself at least for now | |
| 14:29:02 | sean-k-mooney | its really a feature not a bug for the sdk | |
| 14:29:08 | sean-k-mooney | so we can deal with that seperately | |
| 14:29:24 | sean-k-mooney | ill do that now | |
| 14:29:38 | jgilaber | ack, thanks sean-k-mooney | |
| 14:29:58 | jgilaber | that is everything we had in the agenda | |
| 14:30:04 | jgilaber | any topics for the open discussion? | |
| 14:32:04 | sean-k-mooney | am honestly tere are a few thing but i kind fo dont have the mental energy to look at them now | |
| 14:32:08 | sean-k-mooney | so not form me | |
| 14:32:16 | sean-k-mooney | i guess a quick update | |
| 14:32:34 | sean-k-mooney | the cve's we found have now been fixed and backported | |
| 14:32:40 | sean-k-mooney | and the release patch merged yesterday | |
| 14:32:44 | sean-k-mooney | so the release shoudl be done | |
| 14:32:56 | sean-k-mooney | i have some followups to work on like removing the test skip | |
| 14:33:07 | sean-k-mooney | and the docs improvment we talksed about | |
| 14:33:16 | sean-k-mooney | those are the thing i have not had time to refelct on yet | |
| 14:33:30 | sean-k-mooney | so they are on my todo list jsut not someitng i have any really update on | |
| 14:33:36 | sean-k-mooney | thats about it | |
| 14:33:54 | jgilaber | thanks for all the work there sean-k-mooney | |
| 14:34:24 | jgilaber | I got a notification in the bug reports that the fixes were included in new releases so it looks like everything is done | |
| 14:35:27 | jgilaber | so last thing for today is a call for volunteers to chair next meeting | |
| 14:35:58 | jgilaber | I can do it next week if there are none | |
| 14:36:43 | sean-k-mooney | if you can that woudl be awsome | |
| 14:36:59 | jgilaber | ok so it looks like we can close the meeting for today | |
| 14:37:06 | jgilaber | any last minute comment? | |
| 14:37:58 | jgilaber | closing then, thanks! | |
| 14:37:59 | jgilaber | #endmeeting | |
| 14:37:59 | opendevmeet | Meeting ended Tue May 12 14:37:59 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:37:59 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.html | |
| 14:37:59 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.txt | |
| 14:37:59 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.log.html | |
| #openstack-cyborg - 2026-05-13 | |||
| 08:12:36 | 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 | |
| 08:23:40 | 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 | |
| 08:29:35 | chandankumar | sean-k-mooney: jgilaber hello, https://review.opendev.org/c/openstack/cyborg-specs/+/985349: Add generic NVMe driver spec with secure cleanup is ready for review now, feel free to add it to your review queue. thank you! | |
| 09:01:42 | opendevreview | chandan kumar proposed openstack/cyborg master: Add VM setup documentation for Cyborg development https://review.opendev.org/c/openstack/cyborg/+/983964 | |
| 09:02:32 | opendevreview | chandan kumar proposed openstack/cyborg master: Add Cyborg NVMe driver development environment documentation https://review.opendev.org/c/openstack/cyborg/+/982711 | |
| 09:02:43 | opendevreview | chandan kumar proposed openstack/cyborg master: Add user guide for creating instances with accelerators https://review.opendev.org/c/openstack/cyborg/+/983027 | |
| 09:02:56 | opendevreview | chandan kumar proposed openstack/cyborg master: Add doc around cyborg tempest test and grenade testing https://review.opendev.org/c/openstack/cyborg/+/983030 | |
| 09:44:12 | sean-k-mooney[m] | [chandankumar](https://matrix.to/#/@_oftc_chandankumar:matrix.org) i have read over the spec. ill review it again later but there are a number of things i think we should change | |
| 09:46:03 | sean-k-mooney[m] | i do not want to reuse disable and enable to also track cleaning i also dont want to add deiver specific columns to the device table i.e. cleaning format | |
| 09:47:34 | sean-k-mooney[m] | i also dont thing pooling and parsing a sanatisation log is a good approch and i have reservation about the file locks being implemented in cyborg | |
| 09:49:07 | sean-k-mooney[m] | the core seams ok but there is a non trivial amount if duplication and some non relevent info in the spec that should be removed | |
| 09:49:45 | sean-k-mooney[m] | ill do a proper review and leave comments inline in the next day or two | |
| 10:01:13 | chandankumar | thank you sean-k-mooney[m] for the feedback, I will wait for your comments! | |
| 12:48:40 | chandankumar | sean-k-mooney: jgilaber based on discussion during PTG around PCI driver, I have added https://bugs.launchpad.net/openstack-cyborg/+bug/2152545 to track all the duplicate code around lspci and its parsing code. | |
| 12:49:03 | chandankumar | I will add this bug to the upcoming cyborg meeting for discussion on how to move forward. | |
| 12:49:18 | sean-k-mooney | ack | |
| 12:49:33 | jgilaber | ack, thanks for documenting it chandankumar | |
| 12:49:38 | sean-k-mooney | i have been debating if i will start on the v2 driver this release or not | |
| 12:49:50 | sean-k-mooney | i may owrk on this dedup | |
| 12:50:10 | sean-k-mooney | but im currently tyring ot blance soem pros and cons | |
| 12:50:45 | sean-k-mooney | one thing i want to look at is what external tools we use today | |