| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-07 | |||
| 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 | |
| 15:08:32 | jgilaber | yep, I'll create the [DNM] now to check | |
| 15:10:37 | sean-k-mooney | thanks | |
| 15:16:55 | jgilaber | we don't get notifications of stable branches patches in this channel? | |
| 15:23:48 | chandankumar | nope | |
| 16:51:57 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 16:53:40 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove redundant doc8 https://review.opendev.org/c/openstack/cyborg/+/996318 | |
| 18:58:28 | opendevreview | Merged openstack/cyborg master: Remove redundant doc8 https://review.opendev.org/c/openstack/cyborg/+/996318 | |
| #openstack-cyborg - 2026-07-08 | |||
| 05:11:25 | opendevreview | melanie witt proposed openstack/cyborg master: Add API sample structural validation tests https://review.opendev.org/c/openstack/cyborg/+/994830 | |
| 05:21:40 | opendevreview | chandan kumar proposed openstack/cyborg master: pci-sim: Add CentOS Stream 10 support https://review.opendev.org/c/openstack/cyborg/+/995935 | |
| 05:29:51 | opendevreview | melanie witt proposed openstack/cyborg master: Add functional test job to Zuul CI https://review.opendev.org/c/openstack/cyborg/+/996389 | |
| 05:34:00 | opendevreview | melanie witt proposed openstack/cyborg master: Add contributor docs for API sample validation tests https://review.opendev.org/c/openstack/cyborg/+/996390 | |
| 05:39:51 | opendevreview | melanie witt proposed openstack/cyborg master: Fix api-ref inaccuracies and add missing program endpoint https://review.opendev.org/c/openstack/cyborg/+/995533 | |
| 07:34:30 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 08:17:29 | chandankumar | jgilaber: Hi, https://review.opendev.org/c/openstack/project-config/+/996426 will enable stable branch gerrit review notification on cyborg channel | |
| 08:53:19 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Use cyborg-tempest-py3 for master job https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | |
| 08:54:10 | jgilaber | thanks chandankumar! I wanted to look for that | |
| 12:00:46 | opendevreview | chandan kumar proposed openstack/cyborg master: pci-sim: Add CentOS Stream 10 support https://review.opendev.org/c/openstack/cyborg/+/995935 | |
| 12:38:33 | sean-k-mooney | oh i was not aware we hadd a brnach filter applied | |
| 15:24:15 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove [glance] debug https://review.opendev.org/c/openstack/cyborg/+/995707 | |
| 15:24:28 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534 | |
| 16:01:38 | melwitt | anyone else care to take a gander at a local hacking check for logging calls? https://review.opendev.org/c/openstack/cyborg/+/995561 sean-k-mooney has reviewed it so far (thanks) | |
| 16:15:21 | sean-k-mooney | by the way i am undescided if i have broken teim-reivew on monday | |