| chandankumar | sean-k-mooney: Hello, We can remove depends-On: https://review.opendev.org/c/openstack/cyborg/+/992297 (srbac doc patch) from https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992329, it will help to merge tempest series independently. | 04:16 |
|---|---|---|
| rlandy | cccccccccccccccc | 08:43 |
| rlandy | Hello all, reminder the Cyborg IRC meeting is today at 2pm UTC. Please add any topic/bugs/reviews to https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L48. Thank you | 08:44 |
| *** chandank` is now known as chandankumar | 09:15 | |
| opendevreview | Ronelle Landy proposed openstack/python-cyborgclient master: Remove deprecation section in API v1 removal release note https://review.opendev.org/c/openstack/python-cyborgclient/+/996234 | 09:42 |
| opendevreview | Merged openstack/python-cyborgclient master: Remove deprecation section in API v1 removal release note https://review.opendev.org/c/openstack/python-cyborgclient/+/996234 | 10:06 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Ensure unique device profile names and cleanup in API tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992339 | 11:17 |
| opendevreview | Merged openstack/cyborg master: Fix api-ref request samples containing curl commands instead of JSON https://review.opendev.org/c/openstack/cyborg/+/994815 | 11:52 |
| opendevreview | Merged openstack/cyborg master: Refactor RequestContext https://review.opendev.org/c/openstack/cyborg/+/995195 | 11:53 |
| opendevreview | Merged openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205 | 12:07 |
| rlandy | reminder the Cyborg IRC meeting is today at 2pm UTC ( 1 hr). Please add any topic/bugs/reviews to https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L48 | 13:04 |
| rlandy | #startmeeting Cyborg IRC meeting 07 July 2026 | 14:00 |
| opendevmeet | Meeting started Tue Jul 7 14:00:43 2026 UTC and is due to finish in 60 minutes. The chair is rlandy. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cyborg_irc_meeting_07_july_2026' | 14:00 |
| rlandy | hello all - who is around? | 14:00 |
| chandankumar | o/ | 14:01 |
| morenod | o/ | 14:01 |
| rlandy | Courtesy ping: sean-k-mooney bogdando jgilaber skovili | 14:01 |
| jgilaber | o/ | 14:01 |
| sean-k-mooney | o/ | 14:02 |
| rlandy | k - let's begin | 14:03 |
| rlandy | pls add any additional topics/reviews/bugs to https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting | 14:03 |
| melwitt | o/ | 14:03 |
| skovili | hi | 14:03 |
| rlandy | we have the crowd expected ... starting topics | 14:04 |
| rlandy | #topic: Cyborg documentation review and restructure | 14:04 |
| rlandy | that is my topic | 14:04 |
| rlandy | I started looking at the current Cyborg documentation | 14:04 |
| rlandy | it appears that there are some sections we may want to rewrite completely | 14:05 |
| rlandy | ans some we may want to review and update | 14:05 |
| sean-k-mooney | yep i have been incremently impoving them over time | 14:06 |
| rlandy | The first question was if we want to keep the current structure - and the answer in the etherpad is yes | 14:06 |
| sean-k-mooney | the overall stcuture of the docs is more or less standarised across services | 14:06 |
| rlandy | understood | 14:06 |
| sean-k-mooney | i.e. the user guide contibutor guide admin guide | 14:06 |
| sean-k-mooney | so we dont want to entrilly change the sturue but rather build out the content | 14:06 |
| rlandy | the next question is where to start tracking the content the team wants to change | 14:07 |
| chandankumar | we can use launchpad itself. Like this one https://bugs.launchpad.net/openstack-cyborg/+bug/2150086 | 14:07 |
| sean-k-mooney | a full rework fo the docs to build them out is really a multi cycle effort but we coudl use a blueprint or wishlist bug | 14:07 |
| melwitt | could make an etherpad for it too | 14:08 |
| rlandy | well, do we want a full rework? | 14:08 |
| sean-k-mooney | yep i was about to say the sam | 14:08 |
| rlandy | minimally we would need a new introduction | 14:08 |
| sean-k-mooney | i think an ether pad is likely the better direction | 14:08 |
| rlandy | and then a review of the reference sections | 14:08 |
| rlandy | including API, support matric, install, config | 14:08 |
| jgilaber | +1, an etherpad will be more flexible for larger tasks like this | 14:09 |
| skovili | +1 | 14:09 |
| sean-k-mooney | the intoduction in the user guide is honestly one of the least impoatn section as that doc in its current form coudl honestly be deleted | 14:09 |
| sean-k-mooney | https://docs.openstack.org/cyborg/latest/index.html is actully ment to be the main docs landing page | 14:10 |
| sean-k-mooney | the intoduciton of what the service is | 14:10 |
| rlandy | except we would want to clarify the purpose of the service | 14:10 |
| rlandy | since that seems to confuse people | 14:10 |
| sean-k-mooney | is soming i wanted to eveutlly handel vai the scope of the project doc and the archterure doc | 14:10 |
| sean-k-mooney | yes so i started workign on https://docs.openstack.org/nova/latest/contributor/project-scope.html for cybrog twice this week and stopped | 14:11 |
| sean-k-mooney | a similar do to that is what i want to replace the intoduction with | 14:11 |
| rlandy | +1 | 14:11 |
| rlandy | ok - so I think the team is agreed an etherpad would be best to start dividing and prioritizing the sections to rewrite and review/update, correct? | 14:12 |
| melwitt | +1 agree having an introduction (overview of what the service is) is important | 14:12 |
| sean-k-mooney | it is but i dont think its sould be in the user guide | 14:13 |
| sean-k-mooney | more at the top level | 14:13 |
| melwitt | yes agree top level | 14:13 |
| sean-k-mooney | so in the doc index we shoudl have a what is cyborg section like https://docs.openstack.org/nova/latest/index.html#what-is-nova | 14:13 |
| sean-k-mooney | then you can drill down into the differnt docs seciosn for more detail depenign on what you want | 14:14 |
| rlandy | ok - so going with next steps here: | 14:15 |
| rlandy | #action: rlandy to put together etherpad for the team to start working on doc review/additions | 14:16 |
| rlandy | team can comment/vote on which sections to prioritize/pick tasks from there | 14:16 |
| rlandy | do we have a target date for this work? like the next PTG? | 14:17 |
| sean-k-mooney | i dont want to have that level of process or planning honestly. | 14:17 |
| sean-k-mooney | what i woudl prefer is for us to create a list of the current gaps | 14:17 |
| sean-k-mooney | adn organise around reviews in gerrit | 14:18 |
| sean-k-mooney | and then more review the progress at the next ptg | 14:18 |
| sean-k-mooney | then wait for it | 14:18 |
| sean-k-mooney | as we develop feature and close bugs i hope we will impvoe the docs as we go | 14:18 |
| 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 |
| sean-k-mooney | melwitt: you have started to add api sample tests | 14:19 |
| sean-k-mooney | oen of the request i had was to add a followup to the contibutros docs that explains how they work | 14:19 |
| melwitt | ++ agree that would be good. I will do it | 14:20 |
| rlandy | any other comments or suggestions on the documentation topic? | 14:21 |
| 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 |
| 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:21 |
| sean-k-mooney | what we could do is sue a cybrog-docs-update or similar topic to idenfiy related patches | 14:22 |
| 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 |
| melwitt | yes | 14:23 |
| rlandy | ok - let's start with the etherpad and shared gerrit topic: cybrog-docs-update | 14:24 |
| rlandy | we can see how that progress goes | 14:24 |
| rlandy | thank you all for the feedback | 14:26 |
| rlandy | moving on ... | 14:26 |
| rlandy | that's the end of listed topics - so we are moving on to reviews | 14:27 |
| rlandy | #topic: reviews | 14:27 |
| rlandy | There are no reviews listed on the meeting etherpad. Does anyone have any reviews to raise here? | 14:28 |
| sean-k-mooney | i guess a quick update form me on that front | 14:28 |
| sean-k-mooney | i have merge the change to adapt cybrog to the removal fo enforce_scope in oslo.policy | 14:29 |
| sean-k-mooney | beacuse of that i will need to rework the srbac serise slightly | 14:29 |
| 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 |
| sean-k-mooney | i had to do the same in watcher | 14:30 |
| 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:30 |
| sean-k-mooney | so i had to quickly fix that in the cyborg-tempest-plugin repo | 14:31 |
| sean-k-mooney | i had a patch for that so i just brougth it to the front of my srbac serise | 14:31 |
| sean-k-mooney | those are the main changes since the last meeting | 14:31 |
| sean-k-mooney | but i did start reviewing melwitt's api sample test serise this morning | 14:31 |
| sean-k-mooney | that about it for reviewd form me | 14:32 |
| sean-k-mooney | did anyoen want to raise any? | 14:33 |
| 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 |
| chandankumar | Just wanted to highlight here | 14:33 |
| sean-k-mooney | oh ok whiel we proably could merge my patch that was an inital version | 14:34 |
| 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 |
| 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 |
| sean-k-mooney | what is not done yet in my patch is a zuul job | 14:35 |
| chandankumar | Add Cyborg PCI driver lifecycle scenario tests | 14:35 |
| sean-k-mooney | technially yes | 14:35 |
| sean-k-mooney | the depend on was there becasue to use the uart you need the managed mode flag | 14:36 |
| sean-k-mooney | without it ti will be in the guest but you cant write to it and read back the data | 14:36 |
| sean-k-mooney | but technially i coudl drop it since the tests dont assert that currently | 14:36 |
| sean-k-mooney | it just does an ssh to the vm and runs lspci | 14:36 |
| 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:37 |
| 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:38 |
| 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 |
| chandankumar | that's why wanted to get that patch in | 14:39 |
| sean-k-mooney | sure i can rebase it out of merge conflcit later to day | 14:40 |
| chandankumar | so that we can add a single job in future to test pci, nvme adn other devices in a single job | 14:40 |
| sean-k-mooney | i think i need to also update the test exectuion too to enabel it in the job however | 14:40 |
| chandankumar | I will propose a patch for the job enable | 14:40 |
| sean-k-mooney | by the wya i dont knwo why your adding cyborg_sriov | 14:40 |
| chandankumar | It was just a brainstorming | 14:41 |
| sean-k-mooney | we dont have a sriov driver and wont that is wht the pci driver is | 14:41 |
| sean-k-mooney | ack ill pretenet that cyborg_mdev | 14:41 |
| 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 |
| sean-k-mooney | anythign else to bring up on reviews ? | 14:41 |
| rlandy | ok - then let's move on ... | 14:42 |
| rlandy | #topic: bugs ... | 14:42 |
| rlandy | Attribute API POST requires internal database integer for deployable_id: | 14:43 |
| rlandy | #link: https://bugs.launchpad.net/openstack-cyborg/+bug/2158996 | 14:43 |
| rlandy | melwitt: is this yours? | 14:43 |
| melwitt | I opened that one yes | 14:43 |
| chandankumar | I have added that bug to the meeting as it is untriaged | 14:43 |
| rlandy | ah ok | 14:44 |
| melwitt | this got revealed when I proposed some openstacksdk functional tests for cyborg | 14:44 |
| rlandy | do you want to suggest a status? importance? | 14:45 |
| 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:45 |
| sean-k-mooney | basiclly the merged code for the atibutes api | 14:46 |
| sean-k-mooney | does nto folow the approved spec | 14:46 |
| 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 |
| sean-k-mooney | and as impletned cant work unless you know internal db info | 14:46 |
| sean-k-mooney | so fundementally the api was broken form day 0 | 14:46 |
| 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 |
| rlandy | importance: medium? | 14:47 |
| sean-k-mooney | i woudl almost say low but medium works | 14:47 |
| rlandy | staying with that? | 14:47 |
| rlandy | ok - updating | 14:48 |
| rlandy | anything more on this bug? | 14:48 |
| sean-k-mooney | the minium fix is likely goign to be start accpating and returnign the deployable uuid | 14:48 |
| sean-k-mooney | and then next cycle we will have a new microverison and fix this properly | 14:48 |
| sean-k-mooney | but not in a backportable way | 14:49 |
| rlandy | ok ... any other bugs to raise here? | 14:50 |
| 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 |
| rlandy | now, let's open the floor | 14:51 |
| rlandy | #topic: open discussion | 14:52 |
| jgilaber | I wanted to raise this patch I just submitted | 14:52 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 | 14:52 |
| jgilaber | this finishes moving the cyborg-tempest job definition to the cyborg service repo | 14:52 |
| jgilaber | I started this move but I kinda forgot to finish it | 14:52 |
| jgilaber | It has a depends-on to a nova patch | 14:53 |
| jgilaber | #link https://review.opendev.org/c/openstack/nova/+/996297 | 14:53 |
| jgilaber | to also change the job that is running there | 14:53 |
| sean-k-mooney | yep i was lookign for https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/996298 this morning | 14:54 |
| sean-k-mooney | or rather to see if you had a verion of that already | 14:54 |
| sean-k-mooney | since i need that for the pci serise as well | 14:54 |
| sean-k-mooney | lets see fi that actully passes ci | 14:55 |
| 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 |
| jgilaber | I forgot to delete the ipv6 job as well | 14:55 |
| jgilaber | updated now | 14:55 |
| rlandy | thanks jgilaber - any other discussion points? | 14:55 |
| 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 |
| melwitt | sure | 14:56 |
| jgilaber | that is all for me, thanks | 14:56 |
| rlandy | ok ... let's move on | 14:57 |
| rlandy | #topic | 14:57 |
| rlandy | Volunteers to chair next meeting | 14:57 |
| rlandy | anyone open to running next week's meeting? | 14:57 |
| jgilaber | I can do it | 14:58 |
| rlandy | thank you jgilaber | 14:58 |
| rlandy | thanks all ... we will close out for today | 14:58 |
| rlandy | #endmeeting | 14:58 |
| 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 |
| 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 |
| 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 |
| 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 | 14:58 |
| 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 |
| sean-k-mooney | because cybrog on older stable branch refer to cyborg-tempest | 15:02 |
| sean-k-mooney | it fails | 15:02 |
| jgilaber | yep | 15:03 |
| jgilaber | I'm changing the patch to keep the cyborg-tempest as a child of cyborg-tempest-py3 | 15:04 |
| jgilaber | and once we have ci results we can decide if we want to do that or modify the stable branches | 15:04 |
| 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:05 |
| sean-k-mooney | i comemnted inien but we might need to keep the old defintion with `branches: ^stable/202[5,6].*` | 15:06 |
| sean-k-mooney | but sure lets see if the alias works | 15:06 |
| 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 |
| sean-k-mooney | just to make sure this works if we are backportign in cyborg | 15:07 |
| jgilaber | yep, I'll create the [DNM] now to check | 15:08 |
| sean-k-mooney | thanks | 15:10 |
| jgilaber | we don't get notifications of stable branches patches in this channel? | 15:16 |
| chandankumar | nope | 15:23 |
| *** opendevstatus_ is now known as opendevstatus__ | 16:38 | |
| *** opendevstatus__ is now known as opendevstatus | 16:38 | |
| *** opendevstatus is now known as Guest12850 | 16:38 | |
| 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:51 |
| opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove redundant doc8 https://review.opendev.org/c/openstack/cyborg/+/996318 | 16:53 |
| opendevreview | Merged openstack/cyborg master: Remove redundant doc8 https://review.opendev.org/c/openstack/cyborg/+/996318 | 18:58 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!