| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-03-03 | |||
| 14:40:45 | jgilaber | ack thanks, any other comment on this review? | |
| 14:41:36 | jgilaber | let's move to bug triage then | |
| 14:41:43 | jgilaber | #topic Bugs | |
| 14:41:50 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1940511 | |
| 14:42:42 | sean-k-mooney | so this i need to build more context on | |
| 14:42:59 | sean-k-mooney | the api docs site is unfortunetlly down right now for me | |
| 14:43:19 | sean-k-mooney | openstack accelerator arq create command | |
| 14:43:36 | sean-k-mooney | isue used to create the Accleerator resouce request, this is normally only done by nova | |
| 14:43:45 | sean-k-mooney | so humans shoudl not really call this directly | |
| 14:44:48 | sean-k-mooney | https://github.com/openstack/cyborg/blob/master/api-ref/source/arqs.inc | |
| 14:45:53 | sean-k-mooney | the request looks somethign like this https://github.com/openstack/cyborg/blob/master/doc/api_samples/accelerator_requests/accelerator_requests-post-curl.json | |
| 14:45:53 | jgilaber | looking at the api the command is not correct, but the client does have a "--group-id" option | |
| 14:46:31 | sean-k-mooney | and this is the responce https://github.com/openstack/cyborg/blob/master/doc/api_samples/accelerator_requests/accelerator_requests-create-resp.json | |
| 14:46:46 | sean-k-mooney | so the comemnt is that https://github.com/openstack/cyborg/blob/master/doc/api_samples/accelerator_requests/accelerator_requests-create-resp.json#L14 | |
| 14:46:49 | sean-k-mooney | is always 0 | |
| 14:48:13 | sean-k-mooney | we likely shoudl create a repoducer | |
| 14:48:25 | sean-k-mooney | but i thinkw e need to better understand teh semantics of the api | |
| 14:48:30 | sean-k-mooney | by reviewing the related spec | |
| 14:49:40 | sean-k-mooney | i think for now we shoudl move on | |
| 14:49:53 | jgilaber | agreed, not sure if it's a real bug or not, do we mark as incomplete or we leave it as is? | |
| 14:50:11 | sean-k-mooney | lets mark it incomplete for now sure | |
| 14:50:16 | sean-k-mooney | i added the api tag | |
| 14:50:24 | jgilaber | ack thanks | |
| 14:50:26 | sean-k-mooney | it not clear if its a clinet or api bug | |
| 14:50:36 | sean-k-mooney | or neither | |
| 14:50:42 | jgilaber | let's move to the second one | |
| 14:50:43 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1941025 | |
| 14:51:19 | sean-k-mooney | so this is a feature reqeust | |
| 14:51:25 | sean-k-mooney | its also in the wrogn repo | |
| 14:51:35 | sean-k-mooney | it shoudl be in the python-cyborg client repo | |
| 14:51:52 | sean-k-mooney | its valid i think to supprot this in the future | |
| 14:52:35 | sean-k-mooney | i have fixed that | |
| 14:52:44 | sean-k-mooney | what shoudl we triage this ass | |
| 14:52:46 | sean-k-mooney | *as | |
| 14:53:04 | sean-k-mooney | wishlist since it a feature reqeust ro medium becuase its a partiy gap | |
| 14:53:10 | sean-k-mooney | between the cli and api | |
| 14:53:47 | jgilaber | I'm leaning towards wishlist for now | |
| 14:54:17 | jgilaber | but no strong opinion either way | |
| 14:54:47 | sean-k-mooney | works for me | |
| 14:55:11 | jgilaber | ack, let's move to the third one | |
| 14:55:16 | jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1954888 | |
| 14:55:38 | jgilaber | this is a doc request | |
| 14:56:29 | sean-k-mooney | so there im conflcited | |
| 14:56:42 | sean-k-mooney | only nova and zun should do that directly | |
| 14:56:58 | sean-k-mooney | as bidnign and unbinding the acclerator via ARQs shoudl reqruei the service role | |
| 14:57:16 | sean-k-mooney | with that said if you used cybrog in standaloen deployment | |
| 14:57:31 | sean-k-mooney | without either then it would not be a terible thing to document | |
| 14:57:42 | sean-k-mooney | im just not sure we want to supprot that usage mode | |
| 14:57:46 | sean-k-mooney | or at least not right now | |
| 14:58:01 | sean-k-mooney | so this feels low to wishlist if not invalid | |
| 14:58:18 | sean-k-mooney | im all for adding documeition fo how the bidnign flow shoudl work | |
| 14:58:22 | sean-k-mooney | in the contibutor docs | |
| 14:58:38 | sean-k-mooney | but i dont think we shoudl be documenting a manual end user/admin procedure | |
| 14:58:53 | jgilaber | I'm wondering if the intention was to document how to use nova with cyborg to bind accelerators? | |
| 14:59:07 | jgilaber | I'm not sure how well that is documented right now | |
| 14:59:16 | sean-k-mooney | it could be interepeted as a requce to docuemtn the function as a serivce flow | |
| 14:59:43 | sean-k-mooney | i think we set this to incomplete and come back to it next week | |
| 14:59:55 | sean-k-mooney | we need to basiclly refine the scope of it | |
| 15:00:03 | jgilaber | sounds good | |
| 15:00:43 | jgilaber | since we're at time, we can leave the last two bugs for the next meeting, any comment for the open discussion? | |
| 15:01:18 | jgilaber | if not we can move to the final topic | |
| 15:01:21 | jgilaber | #topic Volunteers to chair next meeting | |
| 15:01:24 | jgilaber | any volunteer? | |
| 15:01:26 | chandankumar | I can chair | |
| 15:01:33 | jgilaber | thanks chandankumar! | |
| 15:01:41 | jgilaber | that is all for today, thanks all! | |
| 15:01:43 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 15:01:47 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg___march_3rd/2026/cyborg___march_3rd.2026-03-03-14.00.log.html | |
| 15:01:47 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg___march_3rd/2026/cyborg___march_3rd.2026-03-03-14.00.txt | |
| 15:01:47 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg___march_3rd/2026/cyborg___march_3rd.2026-03-03-14.00.html | |
| 15:01:47 | opendevmeet | Meeting ended Tue Mar 3 15:01:47 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 15:01:47 | jgilaber | #endmeeting | |
| 15:03:18 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 15:03:48 | sean-k-mooney | i tried to capture what we discussed on the last bug here https://bugs.launchpad.net/openstack-cyborg/+bug/1954888/comments/1 | |
| 15:11:00 | opendevreview | sean mooney proposed openstack/cyborg master: adopt isort https://review.opendev.org/c/openstack/cyborg/+/978768 | |
| 15:11:39 | sean-k-mooney | ^ is the last patch in the linter serise by the way. there may be other work we do in the future but i do not expect to make other changes in the near term | |
| 15:22:30 | opendevreview | Bogdan Dobrelya proposed openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| 16:01:47 | opendevreview | Bogdan Dobrelya proposed openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| 16:23:54 | opendevreview | Bogdan Dobrelya proposed openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| 16:33:48 | opendevreview | Bogdan Dobrelya proposed openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| 16:47:38 | opendevreview | Bogdan Dobrelya proposed openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| #openstack-cyborg - 2026-03-05 | |||
| 09:54:05 | bogdando[m] | sean-k-mooney: hi, I am done with Filter out SR-IOV VF devices from NVIDIA GPU driver, seems no more ai assisted or humans provided feedback to address | |
| 11:06:04 | sean-k-mooney | bogdando[m]: ack ill take a look at it again today | |
| 11:06:42 | sean-k-mooney | ill probaly merge it then update my linter seriese on top | |
| 11:16:19 | sean-k-mooney | bogdando[m]: some nits inline but nothigh that prevents moving forward with it so +2w | |
| 12:01:26 | opendevreview | Merged openstack/cyborg master: Filter out SR-IOV VF devices from NVIDIA GPU driver https://review.opendev.org/c/openstack/cyborg/+/978725 | |
| 15:52:21 | opendevreview | Merged openstack/cyborg master: Migrate project metadata to pyproject.toml https://review.opendev.org/c/openstack/cyborg/+/977886 | |
| 15:55:37 | opendevreview | Merged openstack/cyborg master: Add ruff config and fix pre-commit hook failures https://review.opendev.org/c/openstack/cyborg/+/977888 | |
| 16:11:23 | opendevreview | Merged openstack/cyborg master: Fix E402 in Alembic migrations and template https://review.opendev.org/c/openstack/cyborg/+/977889 | |
| 16:16:51 | opendevreview | Merged openstack/cyborg master: Fix S311 and S603 ruff security violations https://review.opendev.org/c/openstack/cyborg/+/977890 | |
| 16:30:24 | opendevreview | sean mooney proposed openstack/cyborg master: adopt isort https://review.opendev.org/c/openstack/cyborg/+/978768 | |
| 16:30:24 | opendevreview | sean mooney proposed openstack/cyborg master: Migrate to ruff-format + ruff lint; drop hacking plugin https://review.opendev.org/c/openstack/cyborg/+/977891 | |
| 17:39:49 | opendevreview | Merged openstack/cyborg master: Migrate to ruff-format + ruff lint; drop hacking plugin https://review.opendev.org/c/openstack/cyborg/+/977891 | |
| 17:43:06 | opendevreview | Merged openstack/cyborg master: adopt isort https://review.opendev.org/c/openstack/cyborg/+/978768 | |
| #openstack-cyborg - 2026-03-06 | |||
| 05:34:26 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 11:00:30 | bogdando[m] | sean-k-mooney: hi, I wonder, if we should omit SUGGESTION level reviews from ai bot | |
| 12:53:24 | sean-k-mooney | in general no but i do plan to tweeak the tresholding and alther the reporting | |
| 12:54:29 | sean-k-mooney | bogdando[m]: in general i have found the suggetion level in put to be valid but sometime out of scope my general plan is that i will have it report less or have a higher barrires to report for the inline comments and keep all the observation in the html report | |
| 12:56:10 | sean-k-mooney | im alos planning to make it evetually report code-review +1 if there are no warning and above issue and cr+-1 only if it identifies a Crtical level issues, so suggestion would never prevent it leaving a +1 | |
| 12:57:43 | sean-k-mooney | bogdando[m]: the current treshold for reporting was intedt to be 0.8 for inline comment but its actully not filterint them right nwo so its using the 0.6 value for the html report for both | |