| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-23 | |||
| 14:49:25 | chandankumar | ok | |
| 14:49:37 | chandankumar | I will propose a patch | |
| 14:49:45 | sean-k-mooney | we coudl decied if we ant to check with the placment api in teh implemation review | |
| 14:49:55 | sean-k-mooney | woudl folks agree this shoudl be medium? | |
| 14:50:01 | chandankumar | yes | |
| 14:50:37 | jgilaber | yes, since this can be worked around | |
| 14:50:40 | sean-k-mooney | ok done feel free to assign this to yourself if you start working on it | |
| 14:51:10 | sean-k-mooney | next bug | |
| 14:51:31 | sean-k-mooney | #topic Set literals used instead of dicts in placement_client.py LOG calls | |
| 14:51:41 | sean-k-mooney | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2157892 | |
| 14:51:57 | sean-k-mooney | melwitt: you found this one | |
| 14:52:29 | melwitt | oh yes that was opened by me yesterday. just something I noticed while going over code in general | |
| 14:53:28 | sean-k-mooney | https://github.com/openstack/cyborg/blob/master/cyborg/common/placement_client.py#L313 | |
| 14:53:33 | sean-k-mooney | that looks pretty valid to me | |
| 14:53:55 | jgilaber | yes, this looks valid and a good find | |
| 14:53:56 | melwitt | yeah that should also be "%(rc_name)s" there too | |
| 14:54:07 | sean-k-mooney | its even correct like 4 lines up | |
| 14:54:09 | sean-k-mooney | https://github.com/openstack/cyborg/blob/master/cyborg/common/placement_client.py#L308 | |
| 14:54:25 | melwitt | heh yeah. not sure what happened there | |
| 14:54:42 | sean-k-mooney | im feeling like traged, low | |
| 14:54:51 | sean-k-mooney | given its only a log issue | |
| 14:54:53 | melwitt | ++ | |
| 14:55:10 | jgilaber | +1 | |
| 14:55:25 | sean-k-mooney | cool i tagged it low-haning-fruit as well | |
| 14:55:52 | sean-k-mooney | melwitt: so ya if you paln to work on it feel free to assign it to your self as normal but anything else on this? | |
| 14:56:34 | sean-k-mooney | if not we will move on | |
| 14:56:44 | melwitt | sure, no that was all on this | |
| 14:56:51 | sean-k-mooney | i need to file a bug for this but ill just note it as an fyi | |
| 14:57:00 | sean-k-mooney | i found out that our api microverison parsing is incorrect | |
| 14:57:09 | sean-k-mooney | i have the fix locally | |
| 14:57:35 | sean-k-mooney | but tl;dr cybrog was incorrectly updated to expec Openstack-Api-Version: 2.x | |
| 14:57:52 | sean-k-mooney | instead of Openstack-Api-Version: accelerators 2.x | |
| 14:58:02 | sean-k-mooney | and we later removed microvsrion pasrse as a dep | |
| 14:58:12 | sean-k-mooney | that was change 2 years ago | |
| 14:58:18 | sean-k-mooney | that was wrong and it breaks the sdk ectra | |
| 14:58:41 | 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 | |
| 14:58:47 | sean-k-mooney | so ill file a bug to track that | |
| 14:58:54 | sean-k-mooney | last itme for today | |
| 14:59:08 | sean-k-mooney | we have notign in open dicuss | |
| 14:59:25 | melwitt | ouch. so I guess you mentioned this before but that means we lack openstacksdk job running on cyborg repo right | |
| 14:59:26 | sean-k-mooney | are there any volunteers for the next meeting | |
| 14:59:44 | sean-k-mooney | so the sdk case might have a workaround | |
| 15:00:05 | sean-k-mooney | i noticed that nova is not using the sdk its using the wraw keystone auth client | |
| 15:00:13 | sean-k-mooney | and when i tried to use a new microveriosn it broek | |
| 15:00:36 | sean-k-mooney | so i need to validate if the sdk has a workaroudn or if its only broken for keystonatuth client | |
| 15:00:57 | sean-k-mooney | but we ill need to support both shapes for the forseable future | |
| 15:01:30 | jgilaber | I can chair next week's meeting | |
| 15:01:41 | sean-k-mooney | #link https://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html#client-interaction | |
| 15:01:51 | sean-k-mooney | #agreed jgilaber will chair the next meeting | |
| 15:01:55 | sean-k-mooney | jgilaber: thanks | |
| 15:02:05 | sean-k-mooney | ok we are over time so lets wrap there | |
| 15:02:09 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-23-14.03.log.html | |
| 15:02:09 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-23-14.03.txt | |
| 15:02:09 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-06-23-14.03.html | |
| 15:02:09 | opendevmeet | Meeting ended Tue Jun 23 15:02:09 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 15:02:09 | sean-k-mooney | #endmeeting | |
| 15:02:38 | sean-k-mooney | melwitt: ya we lack the job validitaing it but also we have no microverison test in tempest | |
| 15:02:54 | melwitt | ah gotcha | |
| 15:02:56 | sean-k-mooney | melwitt: and well the unit test were changed to match the new code when it was broken ... | |
| 15:03:51 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/905646 | |
| 15:04:06 | sean-k-mooney | i dont know really what the orgianl error was https://launchpad.net/bugs/2049489 | |
| 15:04:12 | sean-k-mooney | sicne that is all that is in the bug | |
| 15:04:18 | melwitt | ah yeah ... that is a fairly common thing I have seen, when the test just mirrors the code so much | |
| 15:04:56 | sean-k-mooney | so we have the downstream call now. but we dont have any api sample tests or juctiona or tempst tests for microverions | |
| 15:05:11 | sean-k-mooney | so we will need to build out those going forward | |
| 15:05:24 | sean-k-mooney | we have api sampels for the api-ref | |
| 15:05:31 | melwitt | I think i saw "some" api samples. oh ok | |
| 15:05:50 | sean-k-mooney | ya some exist but they are not hooked into tests | |
| 15:05:56 | sean-k-mooney | so they wont catch if the shape changes | |
| 15:06:40 | melwitt | ah ok | |
| 15:07:32 | sean-k-mooney | so we also dont have any of the openapi stuff that stpen did that would also adress this gap | |
| 15:08:01 | sean-k-mooney | but i think for next cycle i want to prospe a genral api refacotrign spec to core some ot this | |
| 15:08:11 | sean-k-mooney | with that said we can also jsut impvoe this when we have time | |
| 15:08:18 | sean-k-mooney | we dont need a spec for adding more tests | |
| 15:08:28 | melwitt | cool | |
| 15:08:36 | sean-k-mooney | as long as we update the contibutor docs to document how the tests shoudl work | |
| #openstack-cyborg - 2026-06-24 | |||
| 01:27:55 | opendevreview | sean mooney proposed openstack/cyborg master: Add managed mode support to generic PCI driver https://review.opendev.org/c/openstack/cyborg/+/994575 | |
| 01:27:55 | opendevreview | sean mooney proposed openstack/cyborg master: Fix microversion header parsing for service type prefix https://review.opendev.org/c/openstack/cyborg/+/994574 | |
| 01:27:56 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Configure pci-sim Cyborg PCI testing https://review.opendev.org/c/openstack/cyborg/+/994577 | |
| 01:27:56 | opendevreview | sean mooney proposed openstack/cyborg master: doc: Add generic PCI driver guide https://review.opendev.org/c/openstack/cyborg/+/994576 | |
| 01:27:57 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Add NVIDIA vGPU variant-driver setup helpers https://review.opendev.org/c/openstack/cyborg/+/994578 | |
| 01:44:30 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add Cyborg PCI driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 | |
| 01:44:45 | opendevreview | sean mooney proposed openstack/cyborg master: Add managed mode support to generic PCI driver https://review.opendev.org/c/openstack/cyborg/+/994575 | |
| 01:44:46 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Configure pci-sim Cyborg PCI testing https://review.opendev.org/c/openstack/cyborg/+/994577 | |
| 01:44:46 | opendevreview | sean mooney proposed openstack/cyborg master: doc: Add generic PCI driver guide https://review.opendev.org/c/openstack/cyborg/+/994576 | |
| 01:44:47 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Add NVIDIA vGPU variant-driver setup helpers https://review.opendev.org/c/openstack/cyborg/+/994578 | |
| 01:46:20 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Configure pci-sim Cyborg PCI testing https://review.opendev.org/c/openstack/cyborg/+/994577 | |
| 01:46:21 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: Add NVIDIA vGPU variant-driver setup helpers https://review.opendev.org/c/openstack/cyborg/+/994578 | |
| 06:00:02 | 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 | |
| 12:33:38 | opendevreview | Ronelle Landy proposed openstack/python-cyborgclient master: WIP: Drop v1 API from cyborgclient https://review.opendev.org/c/openstack/python-cyborgclient/+/994696 | |
| 13:03:04 | opendevreview | Ronelle Landy proposed openstack/python-cyborgclient master: Drop v1 API from cyborgclient https://review.opendev.org/c/openstack/python-cyborgclient/+/994696 | |
| 13:21:01 | opendevreview | Elod Illes proposed openstack/python-cyborgclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-cyborgclient/+/994725 | |
| 17:47:50 | sean-k-mooney | melwitt: so on a releated unrelated note | |
| 17:48:35 | sean-k-mooney | were you thinking of lookign at functional tests for cybrog in the next few weeks? if not i will towrad the end of july | |
| 17:48:58 | sean-k-mooney | well proably assuming something mroe imporant doesnt come up | |
| 17:50:33 | melwitt | sean-k-mooney: no I wasn't thinking about anything specific and wouldn't want to take if you have something in mind you'd like to do | |
| 17:51:12 | sean-k-mooney | it was more i was missing them while doing some of the feature dev | |
| 17:51:20 | melwitt | in general, I think putting testing in place earlier than later is nice bc subsequent work all can make use of it | |
| 17:51:30 | melwitt | yeah, that's also my concern | |
| 17:51:51 | sean-k-mooney | so not speciifcly but when i fhish up the 2 things i have in flight i wanted to switch back to some testing tech debt | |