| *** jgilaber_ is now known as jgilaber | 11:06 | |
| jgilaber | Hi all, kind reminder the IRC meeting will start in ~30 minutes, add your topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L76 | 13:31 |
|---|---|---|
| jgilaber | #startmeeting cyborg | 14:01 |
| opendevmeet | Meeting started Tue May 12 14:01:07 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'cyborg' | 14:01 |
| jgilaber | Hi! Who is around today? | 14:01 |
| sean-k-mooney | o/ | 14:01 |
| jgilaber | courtesy ping: amoralej bogdando rlandy chandankumar | 14:01 |
| rlandy | o/ | 14:01 |
| amoralej | o/ | 14:02 |
| 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:02 |
| jgilaber | let's get started | 14:03 |
| jgilaber | #topic cyborg tempest job definition | 14:03 |
| jgilaber | I noticed last week that the tempest job definition is stored in the plugin repo instead of the service repo | 14:04 |
| jgilaber | which I think is and odd pattern | 14:04 |
| sean-k-mooney | it a pattern that exists in some plugins | 14:04 |
| sean-k-mooney | but its not my perfered approch | 14:04 |
| jgilaber | I proposed a patch to move the definition to the main cyborg repo | 14:04 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/987620 | 14:05 |
| sean-k-mooney | so we may need to rename them | 14:05 |
| jgilaber | I wanted to ask for opinions before going forward and propose the same change in the stable branches | 14:05 |
| sean-k-mooney | because we may not be able ot remvoe the defintion in the tempest plugin due to stable branches | 14:05 |
| jgilaber | there are several options, what I did was to copy the job definition but name it cyborg-tempest-base | 14:06 |
| jgilaber | and then move the cyborg-tempest job as a child of this new base job | 14:06 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/987621 | 14:07 |
| sean-k-mooney | ack | 14:10 |
| sean-k-mooney | sorry was distracted | 14:10 |
| sean-k-mooney | lets continue to review this on gerrit | 14:10 |
| sean-k-mooney | but i think the overall move makes sense | 14:10 |
| sean-k-mooney | i woudl like to move all the job definiton severually | 14:11 |
| jgilaber | sure, if there are no further comment/objections let's move on | 14:11 |
| sean-k-mooney | as i said i dont know if we just want to keep the alis in the old place | 14:11 |
| sean-k-mooney | its more a case fo ensurign we dont brake the stable branches | 14:11 |
| sean-k-mooney | i dont really have any objections to the move | 14:12 |
| sean-k-mooney | we just need to figure out the sequcne to make it work | 14:12 |
| sean-k-mooney | the only one that really shoudl be in the tempest repo are the stable branch specific ones | 14:12 |
| 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:13 |
| jgilaber | #topic Bugs | 14:14 |
| sean-k-mooney | well that kind of my point | 14:14 |
| sean-k-mooney | we shoudlnot be modifyign the stable branches | 14:15 |
| sean-k-mooney | anyway yes bugs | 14:15 |
| jgilaber | we have a new bug to triage | 14:15 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2151792 | 14:15 |
| jgilaber | the bug report indicates two errors returned by the client | 14:17 |
| sean-k-mooney | ya so... | 14:17 |
| sean-k-mooney | that api feature was approvced in bobcat but never marked as implemtned | 14:17 |
| jgilaber | one was fixed in 2025.2 | 14:17 |
| sean-k-mooney | so im not sure if it actully works | 14:17 |
| sean-k-mooney | and yes there is a bug in the clinet <> sdk interaction as well | 14:17 |
| sean-k-mooney | looking at commente 2 par tof this seam to be verion dependent | 14:18 |
| sean-k-mooney | i.e it may require a newer verison fo the sdk then is allwoed in 2025.1 | 14:19 |
| 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 |
| sean-k-mooney | see if it works or not on master | 14:20 |
| sean-k-mooney | and then determim how to proceed | 14:20 |
| jgilaber | the "attribute list" seems to work for version 4.7.2 so it should work on master too | 14:21 |
| jgilaber | Chandan did reproduce the enable/disable error on master | 14:21 |
| 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:21 |
| sean-k-mooney | yes they are which shoudl be the last part of the feature | 14:22 |
| sean-k-mooney | but i dont know if the rest of the feature was compelted | 14:22 |
| sean-k-mooney | https://review.opendev.org/q/topic:%22bp/disable-enable-device%22 | 14:23 |
| sean-k-mooney | in theory the feature as added by https://review.opendev.org/c/openstack/cyborg/+/891309 | 14:23 |
| jgilaber | the code looks complete | 14:24 |
| sean-k-mooney | there are not tempest tests | 14:24 |
| sean-k-mooney | so we dont actully know if this worked end to end | 14:25 |
| 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 |
| sean-k-mooney | yes https://review.opendev.org/c/openstack/openstacksdk/+/894888 | 14:26 |
| jgilaber | no reference of any variable to track if the device is enabled or disabled | 14:26 |
| sean-k-mooney | was never merged | 14:26 |
| sean-k-mooney | https://review.opendev.org/c/openstack/python-cyborgclient/+/894887 didnt have a depens on | 14:26 |
| sean-k-mooney | but it requires the sdk change | 14:27 |
| jgilaber | that explains the bug | 14:27 |
| sean-k-mooney | so this feature may work at the the api level | 14:27 |
| sean-k-mooney | but we need to fix thet sdk, add tempest test and test this proeprly end to end in ci | 14:27 |
| sean-k-mooney | we can mark this as vaild and medium for the client | 14:28 |
| jgilaber | agreed | 14:28 |
| sean-k-mooney | but not for cyborg itself at least for now | 14:28 |
| sean-k-mooney | its really a feature not a bug for the sdk | 14:29 |
| sean-k-mooney | so we can deal with that seperately | 14:29 |
| sean-k-mooney | ill do that now | 14:29 |
| jgilaber | ack, thanks sean-k-mooney | 14:29 |
| jgilaber | that is everything we had in the agenda | 14:29 |
| jgilaber | any topics for the open discussion? | 14:30 |
| 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 |
| sean-k-mooney | so not form me | 14:32 |
| sean-k-mooney | i guess a quick update | 14:32 |
| sean-k-mooney | the cve's we found have now been fixed and backported | 14:32 |
| sean-k-mooney | and the release patch merged yesterday | 14:32 |
| sean-k-mooney | so the release shoudl be done | 14:32 |
| sean-k-mooney | i have some followups to work on like removing the test skip | 14:32 |
| sean-k-mooney | and the docs improvment we talksed about | 14:33 |
| sean-k-mooney | those are the thing i have not had time to refelct on yet | 14:33 |
| sean-k-mooney | so they are on my todo list jsut not someitng i have any really update on | 14:33 |
| sean-k-mooney | thats about it | 14:33 |
| jgilaber | thanks for all the work there sean-k-mooney | 14:33 |
| 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:34 |
| jgilaber | so last thing for today is a call for volunteers to chair next meeting | 14:35 |
| jgilaber | I can do it next week if there are none | 14:35 |
| sean-k-mooney | if you can that woudl be awsome | 14:36 |
| jgilaber | ok so it looks like we can close the meeting for today | 14:36 |
| jgilaber | any last minute comment? | 14:37 |
| jgilaber | closing then, thanks! | 14:37 |
| jgilaber | #endmeeting | 14:37 |
| 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 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.html | 14:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.txt | 14:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-12-14.01.log.html | 14:37 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!