| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-05-12 | |||
| 14:01:54 | jgilaber | courtesy ping: amoralej bogdando rlandy chandankumar | |
| 14:01:59 | rlandy | o/ | |
| 14:02:04 | amoralej | o/ | |
| 14:02:25 | jgilaber | while we give folks a minute, feel free to add last minute topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L51 | |
| 14:03:34 | jgilaber | let's get started | |
| 14:03:52 | jgilaber | #topic cyborg tempest job definition | |
| 14:04:22 | jgilaber | I noticed last week that the tempest job definition is stored in the plugin repo instead of the service repo | |
| 14:04:35 | jgilaber | which I think is and odd pattern | |
| 14:04:49 | sean-k-mooney | it a pattern that exists in some plugins | |
| 14:04:54 | jgilaber | I proposed a patch to move the definition to the main cyborg repo | |
| 14:04:54 | sean-k-mooney | but its not my perfered approch | |
| 14:05:01 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/987620 | |
| 14:05:18 | sean-k-mooney | so we may need to rename them | |
| 14:05:25 | jgilaber | I wanted to ask for opinions before going forward and propose the same change in the stable branches | |
| 14:05:34 | sean-k-mooney | because we may not be able ot remvoe the defintion in the tempest plugin due to stable branches | |
| 14:06:22 | jgilaber | there are several options, what I did was to copy the job definition but name it cyborg-tempest-base | |
| 14:06:43 | jgilaber | and then move the cyborg-tempest job as a child of this new base job | |
| 14:07:22 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/987621 | |
| 14:10:36 | sean-k-mooney | ack | |
| 14:10:41 | sean-k-mooney | sorry was distracted | |
| 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 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.log.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 | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.html | |
| 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 | jgilaber | #endmeeting | |
| #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 | |