| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-04-28 | |||
| 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 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:01:16 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 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 | jgilaber | #startmeeting 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 | |
| 14:15:57 | chandankumar | this patch makes sure pci devices tracked by OWNER_CYBORG trait just like vgpu one | |
| 14:16:19 | chandankumar | contributor doc https://review.opendev.org/q/topic:%22cyborg-dev-doc%22 is also good for review | |
| 14:17:05 | chandankumar | that's it from my side on review | |
| 14:17:30 | jgilaber | thanks, I've added them to the meeting etherpad | |
| 14:18:00 | jgilaber | #topic Bugs | |
| 14:18:25 | jgilaber | I've noticed we had three untriaged bugs | |
| 14:18:32 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1970818 | |
| 14:20:43 | jgilaber | this looks like a valid bug to me, although we should verify if the problem still exists, since it was reported for a quite older version | |
| 14:21:06 | chandankumar | yes, I am working on verify that | |
| 14:22:22 | jgilaber | ack, thanks | |
| 14:22:54 | jgilaber | ok to mark it as incomplete and revisit once we have verified it? | |
| 14:24:22 | chandankumar | I just added the comment there, | |
| 14:25:18 | chandankumar | The traceback itself is too long, Not useful for end users | |
| 14:25:55 | jgilaber | indeed, this is valid then, thanks chandankumar! | |
| 14:26:00 | chandankumar | I think we can mark them it as a valid bug, i think it needs to be imprived | |
| 14:26:29 | jgilaber | +1, any thougs about the importance? medium/high? | |
| 14:26:57 | chandankumar | I would set medium as user can see the error message. UX needs to be improved here | |
| 14:27:34 | jgilaber | sounds good, done | |
| 14:27:51 | jgilaber | let's move to the second one | |
| 14:27:56 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2017513 | |
| 14:28:17 | chandankumar | Let me work on the reproducer for this bug and update it | |