| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-07 | |||
| 14:18:16 | sean-k-mooney | and then more review the progress at the next ptg | |
| 14:18:20 | sean-k-mooney | then wait for it | |
| 14:18:43 | sean-k-mooney | as we develop feature and close bugs i hope we will impvoe the docs as we go | |
| 14:19:11 | sean-k-mooney | for example im execting the new drivers to be acocmpianed by a driver specific docuemt that describs how to confugre and use them | |
| 14:19:30 | sean-k-mooney | melwitt: you have started to add api sample tests | |
| 14:19:47 | sean-k-mooney | oen of the request i had was to add a followup to the contibutros docs that explains how they work | |
| 14:20:18 | melwitt | ++ agree that would be good. I will do it | |
| 14:21:24 | rlandy | any other comments or suggestions on the documentation topic? | |
| 14:21:39 | melwitt | yeah for the docs work, I think we could agree as a team we are looking to get it done in the near term, if ppl like the idea. +1 to organize around reviews on gerrit | |
| 14:21:42 | sean-k-mooney | i woudl be include to start with https://docs.openstack.org/cyborg/latest/reference/support-matrix.html adn refactor it to add teh supprot level or work on the main docs landing page first https://docs.openstack.org/cyborg/latest/index.html | |
| 14:22:46 | sean-k-mooney | what we could do is sue a cybrog-docs-update or similar topic to idenfiy related patches | |
| 14:23:09 | sean-k-mooney | it woudl be nice ot maintian a list of those patches and there are in teh ether pad as well | |
| 14:23:57 | melwitt | yes | |
| 14:24:46 | rlandy | ok - let's start with the etherpad and shared gerrit topic: cybrog-docs-update | |
| 14:24:55 | rlandy | we can see how that progress goes | |
| 14:26:11 | rlandy | thank you all for the feedback | |
| 14:26:49 | rlandy | moving on ... | |
| 14:27:10 | rlandy | that's the end of listed topics - so we are moving on to reviews | |
| 14:27:17 | rlandy | #topic: reviews | |
| 14:28:02 | rlandy | There are no reviews listed on the meeting etherpad. Does anyone have any reviews to raise here? | |
| 14:28:52 | sean-k-mooney | i guess a quick update form me on that front | |
| 14:29:19 | sean-k-mooney | i have merge the change to adapt cybrog to the removal fo enforce_scope in oslo.policy | |
| 14:29:45 | sean-k-mooney | beacuse of that i will need to rework the srbac serise slightly | |
| 14:30:12 | sean-k-mooney | i have also merged some clean up patches form takashi removing nova code that was never used form our context object | |
| 14:30:20 | sean-k-mooney | i had to do the same in watcher | |
| 14:30:51 | sean-k-mooney | the other thng to not is the requirements-check job now fails if a project does not have a vailid pyproject.toml | |
| 14:31:03 | sean-k-mooney | so i had to quickly fix that in the cyborg-tempest-plugin repo | |
| 14:31:20 | sean-k-mooney | i had a patch for that so i just brougth it to the front of my srbac serise | |
| 14:31:31 | sean-k-mooney | those are the main changes since the last meeting | |
| 14:31:47 | sean-k-mooney | but i did start reviewing melwitt's api sample test serise this morning | |
| 14:32:56 | sean-k-mooney | that about it for reviewd form me | |
| 14:33:02 | sean-k-mooney | did anyoen want to raise any? | |
| 14:33:40 | chandankumar | https://review.opendev.org/c/openstack/cyborg/+/995935 adds cs10 support in pci-sim but i will rebase on top of sean pci-sim validation patch https://review.opendev.org/c/openstack/cyborg/+/996101 | |
| 14:33:48 | chandankumar | Just wanted to highlight here | |
| 14:34:24 | sean-k-mooney | oh ok whiel we proably could merge my patch that was an inital version | |
| 14:35:00 | sean-k-mooney | for context i started working on some linting for the kernel module inclduign a port of the kernesl checkpatch script and adding clang format for code formating | |
| 14:35:18 | chandankumar | one more thing, can we drop depends on from https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580 so that we can merge it independently | |
| 14:35:20 | sean-k-mooney | what is not done yet in my patch is a zuul job | |
| 14:35:29 | chandankumar | Add Cyborg PCI driver lifecycle scenario tests | |
| 14:35:57 | sean-k-mooney | technially yes | |
| 14:36:13 | sean-k-mooney | the depend on was there becasue to use the uart you need the managed mode flag | |
| 14:36:26 | sean-k-mooney | without it ti will be in the guest but you cant write to it and read back the data | |
| 14:36:38 | sean-k-mooney | but technially i coudl drop it since the tests dont assert that currently | |
| 14:36:54 | sean-k-mooney | it just does an ssh to the vm and runs lspci | |
| 14:37:35 | sean-k-mooney | well not quite but basiclly https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/994580/1/cyborg_tempest_plugin/tests/scenario/test_pci_driver_lifecycle.py#101 | |
| 14:38:14 | sean-k-mooney | the lspci testing was in your follow up https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/995328/3/cyborg_tempest_plugin/tests/scenario/test_pci_driver_lifecycle.py#103 | |
| 14:39:03 | chandankumar | yes I want to restructure the test based on mixing class, Here is my ongoing thought on that https://paste.openstack.org/raw/bQVCnAE5DfkR8Tky99g7/ | |
| 14:39:27 | chandankumar | that's why wanted to get that patch in | |
| 14:40:02 | sean-k-mooney | sure i can rebase it out of merge conflcit later to day | |
| 14:40:23 | chandankumar | so that we can add a single job in future to test pci, nvme adn other devices in a single job | |
| 14:40:24 | sean-k-mooney | i think i need to also update the test exectuion too to enabel it in the job however | |
| 14:40:41 | chandankumar | I will propose a patch for the job enable | |
| 14:40:44 | sean-k-mooney | by the wya i dont knwo why your adding cyborg_sriov | |
| 14:41:06 | chandankumar | It was just a brainstorming | |
| 14:41:08 | sean-k-mooney | we dont have a sriov driver and wont that is wht the pci driver is | |
| 14:41:35 | sean-k-mooney | ack ill pretenet that cyborg_mdev | |
| 14:41:50 | chandankumar | above is not the final design, I am thinking in that direction if support multiple drivers and how we can run those tests in a single job | |
| 14:41:51 | sean-k-mooney | anythign else to bring up on reviews ? | |
| 14:42:17 | rlandy | ok - then let's move on ... | |
| 14:42:44 | rlandy | #topic: bugs ... | |
| 14:43:03 | rlandy | Attribute API POST requires internal database integer for deployable_id: | |
| 14:43:13 | rlandy | #link: https://bugs.launchpad.net/openstack-cyborg/+bug/2158996 | |
| 14:43:30 | rlandy | melwitt: is this yours? | |
| 14:43:52 | melwitt | I opened that one yes | |
| 14:43:54 | chandankumar | I have added that bug to the meeting as it is untriaged | |
| 14:44:02 | rlandy | ah ok | |
| 14:44:19 | melwitt | this got revealed when I proposed some openstacksdk functional tests for cyborg | |
| 14:45:04 | rlandy | do you want to suggest a status? importance? | |
| 14:45:38 | melwitt | chatted a bit about it with sean-k-mooney last week, there are a few options for fixing. I will add a comment to summarize what we chatted | |
| 14:46:03 | sean-k-mooney | basiclly the merged code for the atibutes api | |
| 14:46:08 | sean-k-mooney | does nto folow the approved spec | |
| 14:46:16 | melwitt | I would say status of Confirmed as chandankumar has also repro'd it and then importance probably of Medium considering it's been broken from day one | |
| 14:46:23 | sean-k-mooney | and as impletned cant work unless you know internal db info | |
| 14:46:48 | sean-k-mooney | so fundementally the api was broken form day 0 | |
| 14:47:17 | sean-k-mooney | so we could fix it, or remove it or defer this to next cyels as aprt of a larger api refactor spec | |
| 14:47:46 | rlandy | importance: medium? | |
| 14:47:58 | sean-k-mooney | i woudl almost say low but medium works | |
| 14:47:59 | rlandy | staying with that? | |
| 14:48:05 | rlandy | ok - updating | |
| 14:48:29 | rlandy | anything more on this bug? | |
| 14:48:43 | sean-k-mooney | the minium fix is likely goign to be start accpating and returnign the deployable uuid | |
| 14:48:58 | sean-k-mooney | and then next cycle we will have a new microverison and fix this properly | |
| 14:49:06 | sean-k-mooney | but not in a backportable way | |
| 14:50:56 | rlandy | ok ... any other bugs to raise here? | |
| 14:51:43 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Remove cyborg-tempest job definition https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 14:51:49 | rlandy | now, let's open the floor | |
| 14:52:03 | rlandy | #topic: open discussion | |
| 14:52:05 | jgilaber | I wanted to raise this patch I just submitted | |
| 14:52:11 | jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 14:52:34 | jgilaber | this finishes moving the cyborg-tempest job definition to the cyborg service repo | |
| 14:52:49 | jgilaber | I started this move but I kinda forgot to finish it | |
| 14:53:06 | jgilaber | It has a depends-on to a nova patch | |
| 14:53:08 | jgilaber | #link https://review.opendev.org/c/openstack/nova/+/996297 | |
| 14:53:14 | jgilaber | to also change the job that is running there | |
| 14:54:06 | sean-k-mooney | yep i was lookign for https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 this morning | |
| 14:54:46 | sean-k-mooney | or rather to see if you had a verion of that already | |
| 14:54:58 | sean-k-mooney | since i need that for the pci serise as well | |
| 14:55:08 | sean-k-mooney | lets see fi that actully passes ci | |
| 14:55:18 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Remove cyborg-tempest job definition https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 14:55:28 | jgilaber | I forgot to delete the ipv6 job as well | |