| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-07 | |||
| 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 | |
| 14:55:42 | jgilaber | updated now | |
| 14:55:43 | rlandy | thanks jgilaber - any other discussion points? | |
| 14:56:06 | sean-k-mooney | and ya we shoudl merge the nova change first melwitt can you take a look at that later once ci reports back | |
| 14:56:28 | melwitt | sure | |
| 14:56:56 | jgilaber | that is all for me, thanks | |
| 14:57:04 | rlandy | ok ... let's move on | |
| 14:57:08 | rlandy | #topic | |
| 14:57:19 | rlandy | Volunteers to chair next meeting | |
| 14:57:32 | rlandy | anyone open to running next week's meeting? | |
| 14:58:00 | jgilaber | I can do it | |
| 14:58:06 | rlandy | thank you jgilaber | |
| 14:58:15 | rlandy | thanks all ... we will close out for today | |
| 14:58:27 | rlandy | #endmeeting | |
| 14:58:27 | opendevmeet | Meeting ended Tue Jul 7 14:58:27 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:58:27 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg_irc_meeting_07_july_2026/2026/cyborg_irc_meeting_07_july_2026.2026-07-07-14.00.html | |
| 14:58:27 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg_irc_meeting_07_july_2026/2026/cyborg_irc_meeting_07_july_2026.2026-07-07-14.00.txt | |
| 14:58:27 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg_irc_meeting_07_july_2026/2026/cyborg_irc_meeting_07_july_2026.2026-07-07-14.00.log.html | |
| 15:02:06 | sean-k-mooney | jgilaber: so https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 failed the way i was expecting it to fail | |
| 15:02:40 | sean-k-mooney | because cybrog on older stable branch refer to cyborg-tempest | |
| 15:02:43 | sean-k-mooney | it fails | |
| 15:03:56 | jgilaber | yep | |
| 15:04:30 | jgilaber | I'm changing the patch to keep the cyborg-tempest as a child of cyborg-tempest-py3 | |
| 15:04:49 | jgilaber | and once we have ci results we can decide if we want to do that or modify the stable branches | |
| 15:05:02 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Remove cyborg-tempest job definition https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 15:06:44 | sean-k-mooney | i comemnted inien but we might need to keep the old defintion with `branches: ^stable/202[5,6].*` | |
| 15:06:52 | sean-k-mooney | but sure lets see if the alias works | |
| 15:07:32 | sean-k-mooney | jgilaber: we may want to also push a DNM patch ot each of the stable branch that depend on this change | |
| 15:07:45 | sean-k-mooney | just to make sure this works if we are backportign in cyborg | |