| 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 | 06:12 |
|---|---|---|
| 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 | 07:19 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-specs master: Add generic mdev driver spec for 2026.2 https://review.opendev.org/c/openstack/cyborg-specs/+/982276 | 12:53 |
| jgilaber | Hi all! Just a reminder that the weekly IRC meeting will start in ~1 hour, feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L51 | 13:00 |
| jgilaber | #startmeeting cyborg | 14:01 |
| opendevmeet | Meeting started Tue May 19 14:01:24 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 all, who is around today? | 14:01 |
| jgilaber | feel free to add last minute topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L51 | 14:01 |
| jgilaber | courtesy ping: sean-k-mooney amoralej bogdando rlandy chandankumar | 14:03 |
| sean-k-mooney | o/ | 14:05 |
| chandankumar | o/ | 14:06 |
| jgilaber | chandankumar, when you're ready, let's start with your topic | 14:07 |
| jgilaber | #topic accelerators not recconnected to vm after host reboots | 14:07 |
| chandankumar | jgilaber: sure | 14:08 |
| chandankumar | I was working on reproducing accelerators not reconnected to vm after host reboots: https://bugs.launchpad.net/openstack-cyborg/+bug/2011754 | 14:08 |
| chandankumar | I have reproduced this bug in devstack all in env after reboot | 14:08 |
| chandankumar | https://bugs.launchpad.net/openstack-cyborg/+bug/2011754/comments/2 - is the reproducer | 14:09 |
| chandankumar | I still need to find out why it is coming. | 14:10 |
| chandankumar | Feel free to take a look at the reproducer and mark this bug as a triaged | 14:10 |
| sean-k-mooney | chandan so looking at taht | 14:11 |
| jgilaber | so the problem IIUC is that the code path in nova for a host reboot does not take the arqs into account? | 14:11 |
| sean-k-mooney | sudo virsh dumpxml $(sudo virsh list --name | grep '^instance-') | 14:11 |
| sean-k-mooney | is that rebotting you vm that has devstack in it | 14:11 |
| sean-k-mooney | oh sorry | 14:11 |
| sean-k-mooney | i mess read that | 14:11 |
| sean-k-mooney | that checkign the nova vm after the reboot | 14:12 |
| sean-k-mooney | so this is likely a nova bug not a cyborg one | 14:12 |
| sean-k-mooney | jgilaber: yes that is more then likely the root issue | 14:12 |
| chandankumar | yes it is | 14:12 |
| jgilaber | ack, that is good find, thanks chandankumar ++ | 14:13 |
| chandankumar | codepath https://paste.openstack.org/raw/b80mmttpLa8EIMHCdSYq/ based | 14:13 |
| chandankumar | on cursor | 14:13 |
| sean-k-mooney | lets add nova to the bug and mark it invlid for cybrog | 14:13 |
| sean-k-mooney | i can do that now | 14:13 |
| chandankumar | I need to review this code path and add more comments around that | 14:13 |
| jgilaber | ack thanks sean-k-mooney | 14:14 |
| chandankumar | that's it from my side on this | 14:14 |
| jgilaber | let's move to reviews then | 14:15 |
| jgilaber | #topic Reviews | 14:15 |
| jgilaber | we have a few links | 14:15 |
| jgilaber | #link https://review.opendev.org/c/openstack/python-cyborgclient/+/988610 | 14:15 |
| chandankumar | I was trying to work on a fix for Image verification bug https://bugs.launchpad.net/openstack-cyborg/+bug/2144308 and found that the program cli was broken | 14:16 |
| chandankumar | I have taken a look at python-cyborgclient code base and opened thi https://bugs.launchpad.net/openstack-cyborg/+bug/2152616 | 14:16 |
| chandankumar | above cr fixes the program cli and with that, we can easily reproduce the image verificaiton bug | 14:17 |
| sean-k-mooney | we are goign to have to fiture out a way to test this end to end evenutally | 14:17 |
| sean-k-mooney | we likely shoud exetened the fake driver to supprot program as a noop | 14:18 |
| chandankumar | the image verification bug fix does not seems to be easy | 14:18 |
| sean-k-mooney | or similar just so we can eventully test this in some way | 14:18 |
| sean-k-mooney | chandankumar: its not really a bug | 14:18 |
| chandankumar | I think as sean said in the past we should drop that feature with no implemented | 14:18 |
| sean-k-mooney | its a fully feature | 14:18 |
| sean-k-mooney | ya it was broken form day one | 14:19 |
| chandankumar | The image verification piece I mean? Not the whole program feature | 14:19 |
| sean-k-mooney | so im inclidned ot remvoe it and start fresh | 14:19 |
| chandankumar | verify_glance_signatures = True this part | 14:19 |
| sean-k-mooney | right | 14:19 |
| sean-k-mooney | that has never worked so im inclidne to say we shoudl treat it as a new feature | 14:19 |
| sean-k-mooney | its not something we woudl beackport in my opiopion | 14:19 |
| chandankumar | if we check the config, https://github.com/openstack/cyborg/blob/186cdd2b76aff57233660042f71761383285e5e8/cyborg/conf/glance.py#L56 | 14:20 |
| chandankumar | it talks about key_manager which does not implemented in cyborg and deprecated config https://github.com/openstack/cyborg/blob/186cdd2b76aff57233660042f71761383285e5e8/cyborg/conf/glance.py#L63 | 14:20 |
| chandankumar | I was not sure how to remove the code | 14:20 |
| chandankumar | Do we want to drop the config or keep the config and raise a warning with no implemented? | 14:21 |
| sean-k-mooney | if we remove it we will baskiclly revert the pat | 14:21 |
| sean-k-mooney | as i said this never work | 14:21 |
| sean-k-mooney | so there cant be any exsitng usage | 14:22 |
| chandankumar | ok, I will work on the patch for removal | 14:22 |
| sean-k-mooney | we woudl need to have a release note for the removal of teh feature | 14:22 |
| chandankumar | sure | 14:23 |
| chandankumar | that's it on this review | 14:23 |
| jgilaber | ack let's move to the next one | 14:24 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/988597 | 14:24 |
| chandankumar | this one is also mine, It adds uuid to cyborg tempest plugin tests for all api and scenario tests | 14:24 |
| chandankumar | Please add this one to your review queue | 14:25 |
| jgilaber | I'll review properly after the meeting but from a quick look lgtm, thanks for adressing this chandankumar | 14:25 |
| jgilaber | moving on | 14:26 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/984508 | 14:26 |
| sean-k-mooney | chandankumar: +2w on the tepemst plugin patch | 14:27 |
| chandankumar | this one adds multinode devstack job for cyborg testing. | 14:27 |
| chandankumar | sean-k-mooney: thank you! | 14:27 |
| sean-k-mooney | ya i think that is also good to go ill loop back to in shortly | 14:27 |
| jgilaber | this lgtm as well | 14:28 |
| jgilaber | final one | 14:28 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/987097 | 14:28 |
| chandankumar | it is from takashi to drop microversion-parse unused deps | 14:28 |
| sean-k-mooney | is gerrit having issue loadign for ye | 14:29 |
| jgilaber | working fine for me | 14:30 |
| sean-k-mooney | the second change id is incroeect it shoudl have been for https://review.opendev.org/c/openstack/cyborg/+/905646 | 14:30 |
| sean-k-mooney | i just need to refresh | 14:30 |
| chandankumar | ok | 14:31 |
| chandankumar | that's it on review jgilaber you can move to the next section | 14:32 |
| jgilaber | ack, thanks | 14:32 |
| jgilaber | moving on to bugs | 14:32 |
| jgilaber | #topic Bugs | 14:32 |
| jgilaber | we have two today | 14:32 |
| jgilaber | first one | 14:32 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2152545 | 14:32 |
| chandankumar | I opened this one lspci Command Execution and Parsing Code Duplication last week to track lspci code parsing duplication | 14:33 |
| chandankumar | since it is a tech debt | 14:33 |
| chandankumar | sean-k-mooney is also looking to adding more tech debts | 14:33 |
| jgilaber | I think we can treat this as a wishlist bug | 14:33 |
| sean-k-mooney | triaged and whishlist | 14:34 |
| jgilaber | +1, doing that now | 14:34 |
| sean-k-mooney | i think having this as a tracker is fien i just dont know if we want to adress it as part of a larger effort | 14:34 |
| jgilaber | we could do it both ways | 14:35 |
| sean-k-mooney | we proably will want to do it as a genera targeted refactor | 14:35 |
| chandankumar | yes, do smaller fixes without breaking the whole | 14:35 |
| sean-k-mooney | with that said i have some action form the ptg to follwo up on inlcuding markign driver as experiemtnal or deprecated | 14:36 |
| sean-k-mooney | so dependign on how we proceed with that we may decied to not fix all of them | 14:36 |
| sean-k-mooney | i have not sat down and gone through the list of all fo them yet to see where they are but i may try to do that an propose an inital patch in the next week or two | 14:37 |
| chandankumar | we will also discussing about moving these driver https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/fpga/intel/driver.py#L42 to sub package which has binary ivolved? | 14:37 |
| chandankumar | ok | 14:38 |
| jgilaber | any other comment on this bug? | 14:39 |
| chandankumar | nope | 14:39 |
| jgilaber | ok, moving to the last one | 14:39 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2152616 | 14:39 |
| chandankumar | we discussed about this one during python-cyborgclient fix | 14:40 |
| jgilaber | right | 14:40 |
| chandankumar | It would be a feature not a bug | 14:40 |
| jgilaber | do we have a bug tracker for the client? | 14:40 |
| chandankumar | https://bugs.launchpad.net/python-cyborgclient/ | 14:41 |
| chandankumar | yes, my bad | 14:41 |
| chandankumar | we need to move this bug there | 14:41 |
| jgilaber | yes, and I think it's a valid bug for the client | 14:41 |
| jgilaber | medium importance? | 14:42 |
| chandankumar | yes | 14:42 |
| sean-k-mooney | medium seams fine | 14:42 |
| sean-k-mooney | and ya we can mark it involade for cybrog and triaged for the client | 14:42 |
| jgilaber | done, looks like we're set for today | 14:42 |
| jgilaber | any topci for the open discussion? | 14:42 |
| sean-k-mooney | i guess a quick update | 14:43 |
| chandankumar | https://review.opendev.org/c/openstack/releases/+/988076: Release python-cyborgclient for Hibiscus-1 milestone have review proposed | 14:43 |
| chandankumar | for hibiscus-1 milestone | 14:43 |
| sean-k-mooney | the m1 release were due last week | 14:44 |
| sean-k-mooney | we can proceed with it and we can do more release as and when needed | 14:44 |
| jgilaber | yes I saw today that I missed the notification for that patch | 14:44 |
| sean-k-mooney | on a seperate note i brought up https://blueprints.launchpad.net/nova/+spec/support-vfio-variant-driver-managed-mode-via-cyborg | 14:46 |
| sean-k-mooney | in the nova metting yesterday adn we aggeed to track that as a specless blueprint | 14:46 |
| sean-k-mooney | i have filet the corresponding cyborg blueprint https://blueprints.launchpad.net/openstack-cyborg/+spec/support-vfio-variant-driver-managed-mode and spec https://review.opendev.org/c/openstack/cyborg-specs/+/989003 | 14:46 |
| jgilaber | that is good news, thanks sean-k-mooney | 14:47 |
| sean-k-mooney | so this is just allowing cyborg managed pci device to use vfio-vairiant drivers | 14:47 |
| jgilaber | I'll try to review the spec this week | 14:47 |
| chandankumar | thank you sean-k-mooney for sharing it, I will go throgh it | 14:47 |
| sean-k-mooney | no rush | 14:48 |
| sean-k-mooney | i will try and review the nvme and generic mdev spec later in the week as well | 14:48 |
| sean-k-mooney | that about all i had for today | 14:48 |
| chandankumar | Just one more update: I will be out tomorrow till May 27. | 14:48 |
| sean-k-mooney | enjoy the break | 14:49 |
| jgilaber | ack, enjoy the time off chandankumar! | 14:49 |
| chandankumar | thank you! | 14:49 |
| jgilaber | before closing, any volunteers to chair next meeting? | 14:49 |
| jgilaber | if not, I can chair again, no problem | 14:50 |
| sean-k-mooney | i guess i can unless you want to | 14:50 |
| jgilaber | as you prefer, it's not a lot of trouble | 14:50 |
| jgilaber | that is all for today, thanks for the discussion! | 14:51 |
| jgilaber | #endmeeting | 14:51 |
| opendevmeet | Meeting ended Tue May 19 14:51:49 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-19-14.01.html | 14:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-19-14.01.txt | 14:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-19-14.01.log.html | 14:51 |
| chandankumar | I have also address all the comments on https://review.opendev.org/q/topic:%22cyborg-dev-doc%22, please add it to your list, thank you! | 14:52 |
| jgilaber | ack, will do chandankumar | 14:52 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Fix scenario import and add test UUIDs https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/988597 | 15:03 |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Drop reference to legacy keystonemiddleware options https://review.opendev.org/c/openstack/cyborg/+/989165 | 15:42 |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Skip grenade job for test code update https://review.opendev.org/c/openstack/cyborg/+/989166 | 15:43 |
| tkajinam | fyi. unit tests are affected by the new release of keystonemiddleware and https://review.opendev.org/c/openstack/cyborg/+/989165 fixes it | 17:51 |
| * tkajinam runs away | 17:51 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!