| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-08 | |||
| 18:46:20 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add SRBAC-specific policy tests for new defaults https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992331 | |
| 18:48:37 | opendevreview | sean mooney proposed openstack/cyborg master: Add cyborg-tempest-srbac Zuul job https://review.opendev.org/c/openstack/cyborg/+/992332 | |
| 18:54:48 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Migrate API tests to least-privilege client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330 | |
| 18:54:48 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add enforce_scope config and multi-persona client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992329 | |
| 18:54:49 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add SRBAC-specific policy tests for new defaults https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992331 | |
| 19:55:29 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add pre-commit config with generic hooks https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992348 | |
| 19:55:29 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add .pi/ to .gitignore https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992347 | |
| 19:55:30 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Enable pyupgrade rules and modernize super() calls https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992350 | |
| 19:55:30 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add ruff check with baseline pycodestyle and pyflakes rules https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992349 | |
| 19:55:31 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Replace flake8 with hacking pre-commit hook, enable ruff format https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992352 | |
| 19:55:31 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Enable logging, security, warning, and complexity checks https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992351 | |
| 19:55:32 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Enable ruff isort and disable overlapping hacking checks https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992353 | |
| 19:55:33 | opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Migrate setup.cfg metadata to pyproject.toml https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992354 | |
| #openstack-cyborg - 2026-06-09 | |||
| 07:10:59 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add scenario test for PCI device discovery and profile creation https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | |
| 07:16:58 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 09:13:37 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add scenario test for PCI device discovery and profile creation https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | |
| 13:06:56 | chandankumar | Cyborg team meeting starts in another 53 mins, Feel free to add your agenda here: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L42 | |
| 13:10:42 | sean-k-mooney | chandankumar: thansk for taking a look at my cyborg tempest fixes by the way | |
| 13:11:17 | sean-k-mooney | i have one minor bug to fix on the cybrog side but i think im more or less feature complete on the srbac work | |
| 13:11:49 | sean-k-mooney | obvioulys we still need to review an approve the spec but i have a poc of it workign now with test more or less end to end | |
| 13:35:28 | chandankumar | I have not looked at the srbac tempest tests, only focusing on ruff patches | |
| 13:42:46 | sean-k-mooney | yep i got annored by the fact i had to manually fix pep8 | |
| 13:42:58 | sean-k-mooney | also i ment ot do that a few months ago | |
| 13:43:09 | sean-k-mooney | but i the cve work took precendce | |
| 13:43:40 | sean-k-mooney | i still need to follwo ups wiht the client and some of the watcher repos but its not very hard to do | |
| 14:00:48 | chandankumar | Hello all, Team meeting time | |
| 14:00:54 | rlandy | o/ | |
| 14:00:59 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:00:59 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:00:59 | opendevmeet | Meeting started Tue Jun 9 14:00:59 2026 UTC and is due to finish in 60 minutes. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 14:00:59 | chandankumar | #startmeeting cyborg | |
| 14:01:08 | chandankumar | who all are aroud? | |
| 14:01:10 | chandankumar | o/ | |
| 14:01:28 | chandankumar | #courtsey ping sean-k-mooney dviroel amoralej bogdando[m] jgilaber_ | |
| 14:01:33 | sean-k-mooney | o/ | |
| 14:01:35 | rlandy | o/ | |
| 14:01:39 | jgilaber_ | o/ | |
| 14:01:59 | chandankumar | Here is the link to etherpad https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L42, feel free to add your topic here. | |
| 14:04:47 | chandankumar | Since We donot have any topic to discuss I will directly jump to review topic | |
| 14:04:55 | chandankumar | #topic Reviews | |
| 14:05:13 | chandankumar | #link Remove glance image verification: https://review.opendev.org/q/topic:%22remove-glance-sig%22 | |
| 14:05:42 | chandankumar | In last meeting, we discussed about dropping glance image verification code and respective parameters | |
| 14:05:51 | chandankumar | I broke the patches into 3 patches | |
| 14:06:15 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/991027: Remove broken image signature verification | |
| 14:06:24 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/991598: Use GenericDriver.update() as the FPGA programming interface | |
| 14:06:39 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/991579: Fix FakeDriver to support FPGA programming in CI | |
| 14:06:57 | chandankumar | and tested here: https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/991081: Add scenario test for FPGA programming with FakeDriver | |
| 14:07:26 | sean-k-mooney | the midle patch is still kidn fo incorrect | |
| 14:07:49 | sean-k-mooney | the fake driver shoudl not inherit form the fpga one | |
| 14:08:00 | sean-k-mooney | so really the thrid patch should be the secodn one | |
| 14:08:32 | chandankumar | I pasted it in wrong order | |
| 14:08:47 | sean-k-mooney | the order you pasted is correct | |
| 14:08:50 | jgilaber | can't we just skip the second patch in the chain? | |
| 14:08:52 | sean-k-mooney | the order you have them in gerrit is not | |
| 14:08:58 | sean-k-mooney | jgilaber: right | |
| 14:09:07 | sean-k-mooney | we shoudl not need https://review.opendev.org/c/openstack/cyborg/+/991579/2 | |
| 14:09:15 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/991598 | |
| 14:09:34 | chandankumar | I will do that | |
| 14:09:56 | sean-k-mooney | the main think we need to make it work is https://review.opendev.org/c/openstack/cyborg/+/991598/2/cyborg/agent/manager.py | |
| 14:10:09 | sean-k-mooney | and oblviy uypdatign the fpga dirver to use update internally | |
| 14:10:30 | sean-k-mooney | im sure there are other places in teh manger we need to fix | |
| 14:10:42 | sean-k-mooney | but evenutlly we shoudl nto call any driver spcific method | |
| 14:10:51 | sean-k-mooney | only the methods on the base driver | |
| 14:11:06 | sean-k-mooney | anyway the ci is green | |
| 14:11:08 | jgilaber | the last patch looks ok from a quick glance | |
| 14:11:13 | sean-k-mooney | and you updated tempest as well | |
| 14:11:26 | sean-k-mooney | so if you remove the midel patch i think we are good directionally | |
| 14:11:30 | sean-k-mooney | and can pick this up in gerrit | |
| 14:12:03 | jgilaber | +1, since update is already in the base driver, this looks correct | |
| 14:12:21 | chandankumar | https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/modules/generic.py#L82 | |
| 14:12:22 | sean-k-mooney | if you read the docsting/comment for it | |
| 14:12:32 | sean-k-mooney | its pretty clear that it was alwasy inteded for progrming the device | |
| 14:12:48 | chandankumar | one more thing https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/modules/generic.py#L82 and https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/driver.py also | |
| 14:13:07 | chandankumar | Do we want to keep both of the files? or keep one | |
| 14:13:08 | sean-k-mooney | yes so https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/driver.py | |
| 14:13:11 | sean-k-mooney | defiens the interface | |
| 14:13:31 | chandankumar | yes | |
| 14:13:40 | sean-k-mooney | we may combine them into one file | |
| 14:13:54 | sean-k-mooney | but lets do that eperatly | |
| 14:13:58 | sean-k-mooney | *seperatly | |
| 14:14:03 | chandankumar | yes that would be much better | |
| 14:14:25 | chandankumar | Ok I will propose a patch after the meeting | |
| 14:14:55 | sean-k-mooney | well we need to decied on the direction | |
| 14:15:02 | sean-k-mooney | my linclition is to delet this entirly | |
| 14:15:08 | sean-k-mooney | but i want to look a the history and usage | |
| 14:15:15 | sean-k-mooney | before we make a code change for it | |
| 14:15:57 | sean-k-mooney | id ont think https://github.com/openstack/cyborg/blob/master/cyborg/accelerator/drivers/modules/generic.py#L82 | |
| 14:15:59 | sean-k-mooney | is currently used | |
| 14:16:19 | sean-k-mooney | so we will liekly delete the enfire https://github.com/openstack/cyborg/tree/master/cyborg/accelerator/drivers/modules directory | |
| 14:16:36 | sean-k-mooney | but as i said we need to investigate if this is infact dead code or not | |
| 14:16:39 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/473186 - added the generic driver | |
| 14:17:30 | jgilaber | it looks like not all drivers inherit from the base one e.g https://github.com/openstack/cyborg/blob/233f2a5b7e396c24bf15ae22f8cf64a480666f5c/cyborg/accelerator/drivers/gpu/base.py#L30 | |
| 14:17:51 | sean-k-mooney | chandankumar: yes but that generic dirver was lated remvoed | |
| 14:18:46 | sean-k-mooney | jgilaber: ya that also a problem kind of | |
| 14:18:59 | sean-k-mooney | it will work but id dont want to chagne thie picemeal | |
| 14:19:00 | chandankumar | I will look into the history | |
| 14:19:32 | chandankumar | I can see two specs also https://github.com/openstack/cyborg-specs/blob/981405a8938ed786feaaf63ea1955aa7174a367b/specs/pike/implemented/cyborg-driver-proposal.rst#L8 and https://github.com/openstack/cyborg-specs/blob/981405a8938ed786feaaf63ea1955aa7174a367b/specs/train/implemented/cyborg-accelerator-driver.rst#L8 around generic driver | |
| 14:19:44 | sean-k-mooney | chandankumar: ok | |
| 14:20:03 | sean-k-mooney | so i have been hining at the fact that this might need a spec or mor thought then a quick bugfix | |
| 14:20:21 | sean-k-mooney | i.e. tha that you shoudl proably not focus on this right now | |