| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-04-28 | |||
| 14:40:09 | sean-k-mooney | so its not stricly a bug | |
| 14:40:19 | sean-k-mooney | the suprpot for this was part of a new spec | |
| 14:40:25 | sean-k-mooney | that was not fully impmented | |
| 14:40:44 | sean-k-mooney | i.e. owner traits were only intoduced a long time after cybrog was creted | |
| 14:41:01 | sean-k-mooney | as part fo a feature that spaned nova and cybrog that was never impmented on teh nova side | |
| 14:41:07 | sean-k-mooney | and only partly doen in cyborg | |
| 14:41:29 | sean-k-mooney | so im wondering if this shoudl really be a bug at all given it was orgially a feature qruieding a spec | |
| 14:41:44 | chandankumar | https://github.com/search?q=repo%3Aopenstack%2Fnova-specs%20OWNER_CYBORG&type=code there are two specs | |
| 14:41:51 | sean-k-mooney | and given the nvova part was never done (its part of the nova spec for vgpus) | |
| 14:42:09 | chandankumar | this one https://github.com/openstack/nova-specs/blob/e53cb185c680e479a44a94ef0cfb7e1f52c762d6/specs/wallaby/approved/cyborg-vgpu-support.rst#L125 ? | |
| 14:42:29 | sean-k-mooney | yep both are appvoed not implemtned | |
| 14:42:41 | sean-k-mooney | so the nvida driver was partly updated in cybrog because of this | |
| 14:42:54 | sean-k-mooney | but it was never completed and the canges for the other cybrog driver were not done | |
| 14:43:54 | sean-k-mooney | https://github.com/openstack/cyborg-specs/blob/master/specs/wallaby/approved/vgpu-driver-proposal.rst | |
| 14:44:06 | sean-k-mooney | was the cyborg spec for this | |
| 14:45:02 | sean-k-mooney | that was finally implemtned in 2023.1 https://github.com/openstack/cyborg-specs/blob/master/specs/2023.1/implemented/vgpu-driver-proposal.rst | |
| 14:45:28 | sean-k-mooney | without the nova half and without updatign the other drivers to also use the cybrog owner trait | |
| 14:47:07 | chandankumar | when you say nova half, how much work is still left to implement to make it work properly? | |
| 14:47:27 | chandankumar | https://review.opendev.org/q/topic:%22bp/cyborg-vgpu-support%22 | |
| 14:47:47 | chandankumar | https://review.opendev.org/c/openstack/nova/+/812247: vGPU support by Cyborg is an open patch | |
| 14:47:48 | sean-k-mooney | none of the nova change merged | |
| 14:48:03 | sean-k-mooney | so on the nova side there are 2 related parts | |
| 14:48:34 | sean-k-mooney | first nova need to report all its resouce provider with OWNER_COMPUTE or OWNER_NOVA i dont recall | |
| 14:49:00 | sean-k-mooney | it thne need to have a prefilter to include that for its device requests that are not cyborg related | |
| 14:51:46 | chandankumar | Since it is not properly implemented. | |
| 14:52:04 | chandankumar | How do you suggest here next course of action? | |
| 14:52:19 | chandankumar | I am going to do a deep review of how placement nova and cyborg works on traits | |
| 14:52:49 | chandankumar | or we want to table this discussion for some other day? | |
| 14:53:38 | jgilaber | once we implement the owner trait in nova, after upgrading will the trait be added to an already existing resource class or does the operator need to do some action? | |
| 14:55:34 | sean-k-mooney | nova shoudl do it | |
| 14:55:39 | sean-k-mooney | for its resouce classes | |
| 14:55:56 | sean-k-mooney | but the requesting of it need to be gated by a min compute service version check | |
| 14:56:42 | chandankumar | can you share one example of min service version check? | |
| 14:56:47 | chandankumar | from other project | |
| 14:58:08 | sean-k-mooney | sure but i need to check if cybrog has the same concpet today | |
| 14:58:43 | sean-k-mooney | so in the service object | |
| 14:58:45 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/service.py#L40 | |
| 14:59:04 | sean-k-mooney | we have an integer constant that we increment when we add specific feautres | |
| 14:59:28 | sean-k-mooney | eacg compute serivce looks at there local constant and report that value to the db | |
| 14:59:43 | sean-k-mooney | nova maintian a history of what those are https://github.com/openstack/nova/blob/master/nova/objects/service.py#L65 | |
| 15:00:08 | sean-k-mooney | we use them both for feature discoverya and detecting nodes that are tooo old | |
| 15:01:31 | sean-k-mooney | in the compute api among other places we can then check the compute service version and block actions | |
| 15:01:33 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L373-L394 | |
| 15:01:48 | sean-k-mooney | so operation that dont work with cybrog are decorated with block_accelerators | |
| 15:01:56 | sean-k-mooney | which acppate a verion | |
| 15:02:20 | sean-k-mooney | if the min version fo any compute agent cant supprot it | |
| 15:02:22 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/service.py#L621-L686 | |
| 15:02:35 | sean-k-mooney | we block teh request | |
| 15:03:43 | sean-k-mooney | we also have other hleper https://github.com/openstack/nova/blob/master/nova/objects/service.py#L495-L518 | |
| 15:04:12 | sean-k-mooney | but the simple version of it is nova will refuse to starte some serices if a compute agent is older then it can supprot | |
| 15:05:53 | sean-k-mooney | nova/cyborg or any service with a distibuted compont officlaly only supprot the current release the prior release and the prior slup release | |
| 15:06:01 | sean-k-mooney | so at most 2 relases back | |
| 15:09:12 | chandankumar | I was aware about block_accelerators usage regarding controlling instance actions | |
| 15:11:17 | chandankumar | ah, thank you for explaining, Now i understand service version history | |
| 15:11:36 | chandankumar | I just looked into the cyborg codebase, there is no such mapping | |
| 15:14:38 | chandankumar | We need to implement similar minimum version check for cyborg that we can use it during upgrade. | |
| 15:21:31 | sean-k-mooney | its actully more invaidve then that because we do not currently track compute-agent isntace in the db at all | |
| 15:22:17 | sean-k-mooney | so for cybrog we need to isntead have a lot of infra before we coudl do this automaticly so we woudl have to gate this behind a config option i think | |
| #openstack-cyborg - 2026-04-29 | |||
| 14:43:43 | 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 | |
| 15:39:30 | opendevreview | Takashi Kajinami proposed openstack/cyborg-tempest-plugin master: zuul: Use ansible variable to configure tempest plugins https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/986675 | |
| 15:55:10 | opendevreview | Takashi Kajinami proposed openstack/cyborg-tempest-plugin master: zuul: Use tempest_plugins variable https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/986675 | |
| 18:05:24 | opendevreview | Merged openstack/cyborg-tempest-plugin master: clean up tempest job config for 2026.1 https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/983194 | |
| #openstack-cyborg - 2026-05-03 | |||
| 16:10:02 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/cyborg/+/987089 | |
| 17:40:51 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove ineffective static_root https://review.opendev.org/c/openstack/cyborg/+/987095 | |
| 17:59:28 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/cyborg/+/987089 | |
| 18:04:42 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused microversion-parse https://review.opendev.org/c/openstack/cyborg/+/987097 | |
| #openstack-cyborg - 2026-05-05 | |||
| 10:55:16 | opendevreview | chandan kumar proposed openstack/cyborg master: pci: add OWNER_CYBORG constant and fix trait https://review.opendev.org/c/openstack/cyborg/+/986536 | |
| 14:01:16 | jgilaber | #startmeeting cyborg | |
| 14:01:16 | opendevmeet | Meeting started Tue May 5 14:01:16 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:16 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:01:16 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:01:23 | chandankumar | o/ | |
| 14:01:28 | jgilaber | Hi all, who is around today? | |
| 14:01:46 | rlandy | o/ | |
| 14:01:55 | jgilaber | courtesy ping: sean-k-mooney amoralej bogdando | |
| 14:02:11 | amoralej | o/ | |
| 14:02:26 | jgilaber | while we give folks a minute to join, feel free to add any topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L50 | |
| 14:05:04 | jgilaber | I think we can get started, looks like we have a quite light agenda today | |
| 14:05:20 | jgilaber | #topic 2026.2 status etherpad | |
| 14:05:29 | jgilaber | #link https://etherpad.opendev.org/p/cyborg-2026.2-status | |
| 14:05:46 | jgilaber | I've created an etherpad to track the work of the current release | |
| 14:06:10 | jgilaber | it's empty but we can start adding our specs, blueprints and patches | |
| 14:06:58 | chandankumar | thank you jgilaber for creating the etherpad, will add the ongoing reviews and specs there. | |
| 14:08:10 | jgilaber | thanks chandankumar | |
| 14:08:39 | jgilaber | #topic 2024.2 branch is eol | |
| 14:09:00 | jgilaber | #link https://review.opendev.org/c/openstack/releases/+/984970 | |
| 14:09:20 | jgilaber | just a quick announcement that the 2024.2 branch was moved to eol last week | |
| 14:10:11 | jgilaber | it doesn't look like we had any patch open in that branch https://review.opendev.org/q/project:openstack/cyborg+branch:stable/2024.2 | |
| 14:10:19 | jgilaber | so we should be all set in that regard | |
| 14:10:38 | chandankumar | yes, https://review.opendev.org/q/project:openstack/python-cyborgclient+branch:stable/2024.2 open patches already abandoned also | |
| 14:10:52 | jgilaber | any comments on this topic? | |
| 14:10:54 | chandankumar | 2024.2 stable branch tempest job is also dropped https://github.com/openstack/cyborg-tempest-plugin/commit/80a60518b415c0eeb51e47c322eeb675f35482e9 | |
| 14:11:09 | chandankumar | I think we are all good there, No action needed | |
| 14:11:45 | chandankumar | https://launchpad.net/openstack-cyborg/2024.2 is also marked obselete | |
| 14:12:43 | jgilaber | yep, looks good | |
| 14:13:16 | jgilaber | that is all the topics in the agenda, does anyone have any last minute topic? | |
| 14:14:24 | jgilaber | we also do no have any patches listed for review, feel free to call out if you have any | |
| 14:14:53 | jgilaber | If not, I think we can triage some bugs | |
| 14:15:09 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/986536: pci: add OWNER_CYBORG constant and fix trait is ready for review | |
| 14:15:48 | jgilaber | thanks chandankumar I'll add it to my review list | |