| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-09 | |||
| 14:49:45 | chandankumar | thank you all for attending! | |
| 14:49:49 | chandankumar | #endmeeting | |
| 14:49:49 | opendevmeet | Meeting ended Tue Jun 9 14:49:49 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:49:49 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-09-14.00.html | |
| 14:49:49 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-09-14.00.txt | |
| 14:49:49 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-09-14.00.log.html | |
| 14:49:57 | jgilaber | thanks chandankumar | |
| 14:51:51 | sean-k-mooney | jgilaber: to your point ealier https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/pci/base.py#L21 | |
| 14:51:59 | sean-k-mooney | most of the dirver dont inherit form base | |
| 14:52:35 | sean-k-mooney | one of of the openquetion i have is do we want ot use a base clss or a protocol to defien the internfce | |
| 14:52:55 | sean-k-mooney | either works and a common base class has its advangates as woudl a mixin for the common functionatliy | |
| 14:53:57 | sean-k-mooney | for the new medev dirver you shoudl inherit form https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/driver.py#L17 | |
| 14:55:04 | sean-k-mooney | but in genreal i think i want to spend some time thinking about restucture the repo a bit next cycel to normalise the code base a bit | |
| 14:55:45 | sean-k-mooney | for exampel we shoudl not ahve any config stuff https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/configuration.py outside fo https://github.com/openstack/cyborg/tree/master/cyborg/conf | |
| 14:56:00 | sean-k-mooney | and there shoudl not be any db code | |
| 14:56:02 | sean-k-mooney | https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/accelerator.py | |
| 14:56:18 | sean-k-mooney | outside of https://github.com/openstack/cyborg/tree/master/cyborg/db | |
| 14:56:32 | sean-k-mooney | but i suspect at elast some fo this code is deadcode | |
| 14:57:08 | sean-k-mooney | so the dirver not ineritign form the base driver is just one of the peices fo tehcnial debt we need to clean up in the repo layout | |
| 14:57:24 | jgilaber | I'll double check but I think I have it already in my spec, but yes either way it would work | |
| 14:57:41 | jgilaber | it was unexpected to have that base class but not use it | |
| 14:58:04 | sean-k-mooney | ya... so there si a lot of have completed code migration still in the repo | |
| 14:58:41 | sean-k-mooney | out side fo the work you and chandankumar are doing implanning to spend the rest fo the cycle doign these cleanups before the next ptg | |
| 14:59:05 | sean-k-mooney | so that we are in a better places to looks at larger architecurall isseus over the next cycle ot two | |
| 15:00:16 | sean-k-mooney | once the image verifcation cleanup is done im plannign to work on https://bugs.launchpad.net/openstack-cyborg/+bug/2143943 assuming you or chandan doen pick it up by then | |
| 15:02:20 | sean-k-mooney | its on my list to start after im happy with the srback work adn have the inital poc of the pci mangaged mode changes so my eta for that is about 2-3 weeks | |
| 15:06:26 | jgilaber | I'll be busy for a bit with the nova mtty and the the rest of the of the nova mdev work | |
| 15:06:42 | sean-k-mooney | ya no worreis this was more an fyi | |
| 15:06:46 | sean-k-mooney | of what i plan to work on next | |
| 15:07:46 | sean-k-mooney | in think i cna have the 2 "feature" i plan to delver for the cycel done by the end of june or early july, then im plannign to swap to buring down technial debt adn reviewing ye're sutff for the rest of the cycle | |
| 15:10:00 | sean-k-mooney | jgilaber: speaking of you lates mtty run is currenly runnign temspt | |
| 15:10:10 | sean-k-mooney | its goign to fail on some unrelated tests | |
| 15:10:31 | jgilaber | oh cool | |
| 15:10:53 | sean-k-mooney | but we should have results form it in the next hour or so | |
| 15:10:53 | jgilaber | yeah it's going to fail for sure since the whitebox tests have check for valid pci address | |
| 15:11:23 | sean-k-mooney | its acutlly failing on unrelated live mgiration tests | |
| 15:11:34 | jgilaber | I was testing the zuul config and see if I had the configuration right and then I push a patch to make that check accept mdev devices | |
| 15:11:37 | sean-k-mooney | so ill looint why ocne we have actul logs | |
| 15:11:47 | jgilaber | ack thanks! | |
| 15:13:02 | sean-k-mooney | my guess is the whitebox plugin may have confirued something tha tbrok live migrtion or some other unrelated issues but ill need to look at the nova-comptue logs to figure that out rather then just the tempest log stream | |
| 15:14:13 | sean-k-mooney | i know you didnt set the cpu config options in the josb def for whitebox but i dont recall what the devstack plugisn defaults are | |
| 15:15:08 | jgilaber | yes I saw your comment I probably missed a few settings, I tried to add the minimal set to get the test to run and then iteratively add the rest | |
| 15:15:19 | jgilaber | to make sure that I understood what each does | |
| 15:15:35 | sean-k-mooney | ya once we have a full run we can look at the rendered config files and then see what we need to tweak | |
| 15:15:50 | sean-k-mooney | i dont thin that will be hard | |
| 15:16:22 | sean-k-mooney | the ohter option we have is to write new tests in tempest iteslf | |
| 15:16:34 | sean-k-mooney | that done need whitebox | |
| 15:30:48 | 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 | |
| 17:06:07 | sean-k-mooney | jgilaber: so the live migtion fialed because the qemu process crashed | |
| 18:01:54 | jgilaber | sean-k-mooney, do you think it is related to the patch? | |
| 18:02:35 | sean-k-mooney | no | |
| 18:02:40 | sean-k-mooney | i think you jsut got unlucky | |
| 18:02:49 | jgilaber | ack, thanks | |
| 18:03:16 | jgilaber | I pushed a new version because the tempest regex was wrong | |
| 18:03:30 | jgilaber | and I saw no reference to the whitebox | |
| 18:06:53 | sean-k-mooney | that also failing i think but ill take a look when its done | |
| 18:07:38 | sean-k-mooney | we may want to disabel the other jobs while we are debuging this | |
| 18:08:07 | sean-k-mooney | just so it does not us as much gate resouces and or take as long | |
| 18:08:15 | sean-k-mooney | but we can do that tomorrow | |
| 18:14:46 | jgilaber | good point, I'll do that tomorrow | |
| 19:48:25 | blanson[m] | \\\=-0987654321KL;' | |
| #openstack-cyborg - 2026-06-10 | |||
| 01:54:26 | opendevreview | chandan kumar proposed openstack/cyborg master: Use GenericDriver.update() as the FPGA programming interface https://review.opendev.org/c/openstack/cyborg/+/991598 | |
| 02:00:53 | opendevreview | chandan kumar proposed openstack/cyborg master: Remove broken image signature verification https://review.opendev.org/c/openstack/cyborg/+/991027 | |
| 02:00:53 | opendevreview | chandan kumar proposed openstack/cyborg master: Use GenericDriver.update() as the FPGA programming interface https://review.opendev.org/c/openstack/cyborg/+/991598 | |
| 10:04:09 | sean-k-mooney | jgilaber: progress | |
| 10:04:12 | sean-k-mooney | Details: [{'status': 400, 'title': 'Bad Request', 'detail': "The server could not comply with the request since it is either malformed or otherwise incorrect.\n\n Invalid query string parameters: Additional properties are not allowed ('in_tree' was unexpected) Failed validating 'additionalProperties' in schema: {'type': 'object', 'properties': {'name': {'type': | |
| 10:04:14 | sean-k-mooney | 'string'}, 'uuid': {'type': 'string', 'format': 'uuid'}}, 'additionalProperties': False} On instance: {'in_tree': '58f9b863-0a2f-4e9b-a2e9-84d92707beab'} ", 'request_id': 'req-6792f268-11b3-4a14-ada4-f87e733e5058'}] | |
| 10:04:23 | sean-k-mooney | so the whitebox test ran | |
| 10:04:39 | sean-k-mooney | but apprently we are calling placment incorrectly | |
| 10:04:56 | sean-k-mooney | and or tempst has incorrect schma validation | |
| 10:05:31 | sean-k-mooney | 400 GET https://213.32.77.206/placement/resource_providers?in_tree=58f9b863-0a2f-4e9b-a2e9-84d92707beab 0.028s | |
| 10:05:47 | sean-k-mooney | that looks like placement rejecting it but my guess is this is a microversion issue | |
| 10:07:27 | sean-k-mooney | New in version 1.14 | |
| 10:07:40 | jgilaber | yes, I'm working on a patch for that | |
| 10:08:12 | sean-k-mooney | looking a tthe hearder we are not passign any header or are getting defaulted back to 'openstack-api-version': 'placement 1.0' | |
| 10:08:26 | jgilaber | I'm waiting to get the vendor id of the device in the guest vm to configure the test properly | |
| 10:08:42 | sean-k-mooney | ack | |
| 10:09:22 | sean-k-mooney | the vendor id and prouct ida are 4348:3253 | |
| 10:10:26 | sean-k-mooney | os i think you neeed to set that as hex so you want 4348 i think | |
| 10:11:05 | sean-k-mooney | its set here https://github.com/torvalds/linux/blob/master/samples/vfio-mdev/mtty.c#L252 in reviert byte order | |
| 10:11:34 | sean-k-mooney | well or here https://github.com/torvalds/linux/blob/master/samples/vfio-mdev/README.rst?plain=1#L59 | |
| 10:13:22 | sean-k-mooney | for mdpy which is actully predentign to be a gpu its using redhats vendor id | |
| 10:13:30 | sean-k-mooney | https://github.com/torvalds/linux/blob/master/samples/vfio-mdev/mdpy-defs.h#L12 | |
| 10:13:53 | sean-k-mooney | because it s using the allcoation redhat provide too qemu | |
| 10:27:55 | jgilaber | that seems right, I've pushed a patch to set the placement min microversion, let's see what happens next | |
| 10:29:32 | sean-k-mooney | it is set in some test but not in the smoke test | |
| 10:30:07 | sean-k-mooney | https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/test_vgpu.py#L316 | |
| 10:32:17 | sean-k-mooney | jgilaber: ah you setin it in the base class ya i was debating propsing you do that cool | |
| 10:32:35 | sean-k-mooney | we could clean up some of the other palces where its now beign set to the same value later | |
| 10:33:10 | sean-k-mooney | jgilaber: can you update the nova patch to depend on that one to test it end to end? | |
| 10:33:45 | sean-k-mooney | if it passes i cna fast appove that but if you need to respin it woudl be nice to remove the duplication | |
| 10:47:00 | jgilaber | already set the depends-on | |
| 10:47:20 | sean-k-mooney | cool | |
| 10:47:35 | jgilaber | ack, will remove the duplication after the current ci run finishes | |
| 15:56:32 | sean-k-mooney | jgilaber: so it booted correctly https://paste.opendev.org/show/b9TDtQ29jhj8jJfAy5Ge/ | |
| 15:57:03 | sean-k-mooney | but but ssh to the vm failed | |
| 15:57:19 | sean-k-mooney | 2026-06-10 14:15:22.306 107224 ERROR tempest.lib.common.ssh paramiko.ssh_exception.NoValidConnectionsError: [Errno None] Unable to connect to port 22 on 172.24.5.245 | |
| 15:57:54 | sean-k-mooney | the fallback password is also incorrect | |
| 15:57:56 | sean-k-mooney | User: cirros, Password: password | |
| 15:58:43 | sean-k-mooney | it shoudl be gocubsgo but its trying to use the ssh key anyway | |