| opendevreview | sean mooney proposed openstack/cyborg master: Fix check-cherry-picks.sh for numeric stable branches https://review.opendev.org/c/openstack/cyborg/+/977793 | 13:08 |
|---|---|---|
| jgilaber | Hi! The IRC meeting will start in ~30 minutes, feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L45 | 13:30 |
| jgilaber | #startmeeting | 14:00 |
| opendevmeet | jgilaber: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee' | 14:00 |
| jgilaber | #startmeeting Cyborg | 14:00 |
| opendevmeet | Meeting started Tue Feb 24 14:00:48 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. 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' | 14:00 |
| jgilaber | Hi all! Who is around? | 14:01 |
| sean-k-mooney | o/ | 14:01 |
| rlandy | o/ | 14:01 |
| dviroel | o/ | 14:01 |
| jgilaber | Courtesy ping: amoralej bogdando | 14:01 |
| amoralej | o/ | 14:01 |
| jgilaber | I think we can start, feel free to add topics to the agenda | 14:02 |
| jgilaber | #link https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L45 | 14:02 |
| jgilaber | first topic is from sean-k-mooney | 14:02 |
| jgilaber | #topic linters | 14:02 |
| sean-k-mooney | we can probly just note the titles | 14:02 |
| sean-k-mooney | so the first topic is the usage and adoption of linters in cybrog | 14:03 |
| sean-k-mooney | it is my intent to review quite alot of the code over the next 2 weeks and focus on impoving the automated linitign and code quality issues | 14:03 |
| sean-k-mooney | first by adding pre-commit and tools we have used like codespell to the repo | 14:04 |
| sean-k-mooney | then i woudl like to add ruff-check to and fix a set of liniting issues and finally adopt ruff for formating | 14:04 |
| sean-k-mooney | my intent is to automat many of the sytel and correctness topics we currently have to review for and modernise the code base | 14:05 |
| sean-k-mooney | i woudl like to know how others feel about this | 14:05 |
| jgilaber | based on the experience of similar work for watcher I think it's worth it | 14:06 |
| amoralej | +1 | 14:06 |
| amoralej | did you try to run linters with current codebase? will it need many changes? | 14:06 |
| sean-k-mooney | not yet and i expect yes | 14:06 |
| sean-k-mooney | the side benifit of this is it will force me to read a large protion of the code base | 14:06 |
| sean-k-mooney | and gain familarity with it by proxy | 14:07 |
| amoralej | anyway, it will be needed, so it's a good time to go for it | 14:07 |
| sean-k-mooney | ya i tought doing this early will reduce the firction for future work as we reviee the project adn start extending it | 14:07 |
| sean-k-mooney | are there any other comments/request on this topic? | 14:08 |
| sean-k-mooney | if not we can move to ci | 14:08 |
| jgilaber | I think we can move on | 14:09 |
| jgilaber | #topic ci jobs | 14:09 |
| sean-k-mooney | so here there is some good news and bad neews | 14:10 |
| sean-k-mooney | the good news is chandan fixed the ipv6 job last week | 14:10 |
| sean-k-mooney | and that is merges so i plan to propes makign that voting again adn adding it to gate in the next few days | 14:10 |
| sean-k-mooney | the bad news is the cherry-pick check is broken on the stable branches | 14:11 |
| sean-k-mooney | so i have a fix on master https://review.opendev.org/c/openstack/cyborg/+/977793 that i need to revise but that will need to be backport before we can fully unblock the tempest plugin repo | 14:11 |
| sean-k-mooney | we can cover that more in the review section | 14:12 |
| sean-k-mooney | the other change i was plannign to make is to combine the bandite tox job into the pep8 one which will become oru single linter job | 14:12 |
| sean-k-mooney | that will bring our jobs closer to how nova/watcher ectra are set up | 14:12 |
| sean-k-mooney | the other gap i noticed is we currently do not have a greade job | 14:13 |
| sean-k-mooney | im not sure we will have time to add one this cycle before RC1 | 14:13 |
| sean-k-mooney | my question is do you think its ok to do that early in the next cycle | 14:13 |
| sean-k-mooney | or shoudl we priorites adding grenade before rc1? | 14:13 |
| jgilaber | I think it's ok to defer it, it's not likely we'll see many big code changes until next cycle | 14:15 |
| sean-k-mooney | ack my current plan was to look at it after we cut rc1 | 14:17 |
| sean-k-mooney | assumign other dont take a look before then | 14:17 |
| sean-k-mooney | if there is no other comment on ci job priorty we could move to the last topic which is more of an annouchment | 14:18 |
| jgilaber | let's move on | 14:19 |
| jgilaber | #topic bugs | 14:19 |
| sean-k-mooney | so as noted in the ether pad | 14:19 |
| sean-k-mooney | i created https://launchpad.net/~cyborg-bug-team as an open team so folks interested in triage can now join that team | 14:19 |
| sean-k-mooney | i have configure it in lauchpad as the bug supervior | 14:20 |
| sean-k-mooney | meaning member of the team can add tag, set imporantce and update teh status of the bugs | 14:20 |
| sean-k-mooney | i have configured it to expire membership after 12 months but you will get a renewal email and you will be auto appoved for joining/renewal | 14:20 |
| sean-k-mooney | so if you plan to help with cyborg but triage please join that when you have time | 14:21 |
| sean-k-mooney | thats all i had | 14:21 |
| jgilaber | thanks sean-k-mooney, I just joined the team | 14:21 |
| jgilaber | if there are no more comments, we can move to the reviews section | 14:22 |
| jgilaber | #topic Reviews | 14:22 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/970153/13 | 14:22 |
| sean-k-mooney | so this review fixes a long running issue with cyborg where internally it used CONF.host for both the rpc endpoint and compute node placement resouce provider name | 14:23 |
| sean-k-mooney | in some evns they are the saem such as in our downstream installer where we write the fqdn into /etc/hostname | 14:24 |
| sean-k-mooney | btu in general they value of conf.HOST is the short hostnaem and the resouce provider name is the hypervior_hostname whch tends to be the fqdn | 14:24 |
| sean-k-mooney | this change will try to use both to find the resouce provier, it add config options so you can explcity sepcify it | 14:25 |
| sean-k-mooney | and has an expenital backoff and retry up to 3 times | 14:25 |
| sean-k-mooney | i think this change is ready to merged | 14:25 |
| sean-k-mooney | but wanted to ask if there are any other issue folks woudl liek me to adress before i proceed (after the meeting) | 14:26 |
| jgilaber | +1 from me | 14:26 |
| sean-k-mooney | jgilaber: and bogdando[m] have reviewd eariler versions fo this change. | 14:26 |
| sean-k-mooney | if no ohter comments we can move on to the next review | 14:28 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/977177/3 | 14:28 |
| sean-k-mooney | so ^ is me shamelessly stealing the release guide i wrote for watcher adn cleaning it up slightly for cyborg | 14:28 |
| sean-k-mooney | the tl;dr is it calls out the major release milesotnes and the actions we need to do for each of them | 14:29 |
| sean-k-mooney | #link https://e7462b45848db14b956a-a5016a94c30782ba95b6339cf8a222f0.ssl.cf5.rackcdn.com/openstack/c0eb5346b7434ee6a0ec7583d0197a09/docs/contributor/release-guide.html | 14:30 |
| sean-k-mooney | is a render | 14:30 |
| sean-k-mooney | we can refine this more as we go but again if there are no concerns with that i propose to merge that after the meeting as a baseline | 14:30 |
| jgilaber | I've only skimmed the patch but I can vouch for the usefulness of the guide as one of watcher's release liasons this last cycle | 14:31 |
| sean-k-mooney | i can leave it open for a day or two if folks want to review more | 14:31 |
| sean-k-mooney | i woudl like to merge the review after it this week | 14:31 |
| sean-k-mooney | but we are not in a super rush here | 14:31 |
| jgilaber | I'll review it after the meeting | 14:32 |
| sean-k-mooney | ack ill check back then thrusday or friday adn respond to any feedback if there is any | 14:32 |
| jgilaber | thanks, I think we can move to the next one | 14:33 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/977020/3 | 14:33 |
| sean-k-mooney | so the next review is the release note prelude | 14:33 |
| sean-k-mooney | #link https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_dfc/openstack/dfcb37a19d884363afaee29efe3c877d/docs/unreleased.html | 14:33 |
| sean-k-mooney | this will become the 2026.1 prelude when we cut the release | 14:34 |
| sean-k-mooney | rlandy: suggested i add a refence to docuemantion so i added it in the last revision | 14:34 |
| sean-k-mooney | im happy to refine it more if there is addtional feedback but i would like to merge this by the end of the week if no outstanding issues are raised | 14:35 |
| rlandy | I'll review again | 14:35 |
| sean-k-mooney | the hard deadlien for merging this is before rc1 | 14:35 |
| sean-k-mooney | so we have 3 weeks | 14:35 |
| sean-k-mooney | rlandy: ack thanks | 14:35 |
| sean-k-mooney | the only ohter comment i have on that is we have the opertunity to add a marketing highlights as part of the release | 14:37 |
| sean-k-mooney | i do not know if there is anythign we woudl ike to do in that regard but now is a good time to start thinking about that | 14:37 |
| sean-k-mooney | i was thinkign fo reuseing the prelude to some degree adn put more of a focus into that for next cycle | 14:38 |
| rlandy | +1 | 14:38 |
| sean-k-mooney | lets keep moving ot get though the adgenda unless there is anything else on this topic. ^ is technially the last change we need for the release | 14:39 |
| jgilaber | ack thanks sean-k-mooney | 14:40 |
| jgilaber | next review | 14:40 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/977083 | 14:41 |
| sean-k-mooney | so ^ is a work in progress change that chandan has been working on | 14:41 |
| sean-k-mooney | it is using a project https://github.com/snu-csl/nvmevirt | 14:41 |
| sean-k-mooney | to provide a kernel module to emulate nvme devices | 14:41 |
| sean-k-mooney | this we hope to use to test the pci driver and a future nvme/ssd driver | 14:42 |
| sean-k-mooney | this is making good progress but need more work before it is ready | 14:42 |
| sean-k-mooney | the next review in the list was https://review.opendev.org/c/openstack/project-config/+/977194 | 14:42 |
| sean-k-mooney | which add the project to our zuul tenant | 14:43 |
| sean-k-mooney | so that we cna have zuul prepare the git repo for us | 14:43 |
| sean-k-mooney | that is now merged | 14:43 |
| sean-k-mooney | #link https://review.opendev.org/c/openstack/project-config/+/977194 | 14:43 |
| sean-k-mooney | if folks have tiem to review over the next week or two we will likely hold htat until after RC1 is cut but its good to review | 14:44 |
| sean-k-mooney | my proposal is to review the branch status and review if we are ready to crate the rc1 tag during next weeks meeting | 14:45 |
| sean-k-mooney | any comment or questions? | 14:46 |
| sean-k-mooney | if not the last 2 reviews also go togehter | 14:47 |
| jgilaber | next one is | 14:47 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/976215 | 14:47 |
| sean-k-mooney | so this unfortually depend on https://review.opendev.org/c/openstack/cyborg/+/976224 | 14:48 |
| sean-k-mooney | which is currently blocked by a bug in the cherry-pick check which is fixed by https://review.opendev.org/c/openstack/cyborg/+/977793 | 14:48 |
| sean-k-mooney | so we will have to merge ^ on master and cherry-pick it back | 14:48 |
| sean-k-mooney | before we can fix the package name and then update the jobs in the tempest plugin | 14:49 |
| sean-k-mooney | none of that is hard but we just need to do it in the corerct order | 14:49 |
| sean-k-mooney | i hope we can resolve that and complete updating the master branch of the tempest plugin in the next week or so | 14:49 |
| sean-k-mooney | that effeitlvy it for reviews form me for now | 14:50 |
| jgilaber | sounds reasonable, any other comments on the reviews? If not we can do some bug triaging | 14:50 |
| jgilaber | #topic Bugs | 14:51 |
| jgilaber | first one | 14:51 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1987380 | 14:51 |
| sean-k-mooney | so this is a valid bug IMO, i marked it as medium for now | 14:51 |
| sean-k-mooney | the current nvidia driver only supprot full gpu passthough or mdevs | 14:52 |
| sean-k-mooney | unfortuletly the way it enmerate devices also include the VFs that mdevs are creted form for sriov capable gpus | 14:52 |
| sean-k-mooney | that a bug and the correct fix for now is to filter out any nvida pci device that is a VF | 14:52 |
| sean-k-mooney | long term we wil want to supprot mig and extendign this driver is one option but right now you cannot use this driver with mig VFs | 14:53 |
| sean-k-mooney | it could crash the host kernel on older verions fo rhel for example | 14:53 |
| sean-k-mooney | so this is a relitlivy easy bug for someone to fix but we do not need to fix it imiedealy | 14:54 |
| sean-k-mooney | any comments/concerns with trating this as medium | 14:54 |
| jgilaber | I'm not too familiar with the nvidia driver, but based on the context mediums seems ok | 14:55 |
| jgilaber | any other comment? if not we can take another bug | 14:56 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1954886 | 14:56 |
| sean-k-mooney | this i have triaged as high. the tl;dr is we have a issue with teh current api policy that allows you do detach a arch form a nova vm via the cyborg api | 14:57 |
| sean-k-mooney | that is a mild security issue but since you can only break your own vms less so | 14:57 |
| sean-k-mooney | the fix for this is we need to implement SRBAC next cycle and i plan to spen the next week or so investigatin that and preparing a spec | 14:57 |
| sean-k-mooney | i have set this as high to reflect that but im unsure fi there is a backportable solution to this | 14:58 |
| jgilaber | looking at the last comment, do you think we'll we need to anything else on top of the SRBAC implementation for this bug? | 14:58 |
| sean-k-mooney | so this may be a master only fix | 14:58 |
| sean-k-mooney | i dont think we do but we may make this a ptg topic to dicussi t futuer | 14:58 |
| sean-k-mooney | for now we are aware of i ti and we can revist. do you want to do the last 2 quickly? | 14:59 |
| jgilaber | ack, so high seems appropiate | 14:59 |
| jgilaber | let's try | 14:59 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2020178 | 14:59 |
| jgilaber | as noted, this one seems to be an exact duplicated of an already closed bug, I think it's safe to close | 15:00 |
| sean-k-mooney | i think so too | 15:00 |
| sean-k-mooney | and there is a comment to that effect on the bug | 15:01 |
| jgilaber | any objection to closing it? | 15:01 |
| sean-k-mooney | nope +1 | 15:01 |
| jgilaber | done, last one | 15:01 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1933399 | 15:01 |
| sean-k-mooney | ah i see | 15:03 |
| sean-k-mooney | took a second | 15:03 |
| sean-k-mooney | http://cyborg.openstack.svc.region-stackdev.myinspurcloud.com/v2 vs http://cyborg.openstack.svc.region-stackdev.myinspurcloud.com/accelerator/v2 | 15:03 |
| sean-k-mooney | it shoudl be http://cyborg.openstack.svc.region-stackdev.myinspurcloud.com/v2/accelerator | 15:03 |
| sean-k-mooney | i think | 15:03 |
| sean-k-mooney | we likely need to verify if this stilll hapens and then restore the orgian proposed fix | 15:04 |
| jgilaber | sounds right, move it back to triaged and medium? | 15:04 |
| sean-k-mooney | im not sure the propsoed fix is correct | 15:04 |
| sean-k-mooney | ya that sounds correct to me | 15:05 |
| jgilaber | ack, so that's all the bugs we had for today | 15:05 |
| jgilaber | final topic | 15:05 |
| jgilaber | #topic Volunteers to chair next meeting | 15:05 |
| jgilaber | any volunteers? | 15:05 |
| rlandy | I'll do it | 15:05 |
| jgilaber | thanks rlandy | 15:05 |
| sean-k-mooney | :) | 15:05 |
| jgilaber | and thanks everyone for participating, see you next week! | 15:06 |
| sean-k-mooney | i guess we should ask if there are any last miniut topics | 15:06 |
| rlandy | oops actually I am out | 15:06 |
| sean-k-mooney | i can do it of otehrs are not free | 15:06 |
| rlandy | I will find someone else | 15:06 |
| sean-k-mooney | ack we can figure it out offline | 15:06 |
| sean-k-mooney | or rather out of hte meeting | 15:06 |
| jgilaber | I can chair it also if needed | 15:06 |
| jgilaber | any last minute topics? | 15:07 |
| rlandy | ok - I'll take the week after | 15:07 |
| jgilaber | that is all then, thanks again! | 15:07 |
| jgilaber | #endmeeting | 15:07 |
| opendevmeet | Meeting ended Tue Feb 24 15:07:57 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:07 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-02-24-14.00.html | 15:07 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-02-24-14.00.txt | 15:07 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-02-24-14.00.log.html | 15:07 |
| sean-k-mooney | ill try not to info dump as much next time but there si a lot to get though for the first couple of meetings | 15:09 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix check-cherry-picks.sh for numeric stable branches https://review.opendev.org/c/openstack/cyborg/+/977793 | 16:24 |
| *** gmaan is now known as gmaan_afk | 16:29 | |
| opendevreview | Merged openstack/cyborg master: Fix resource provider hostname mismatch https://review.opendev.org/c/openstack/cyborg/+/970153 | 17:07 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix check-cherry-picks.sh for numeric stable branches https://review.opendev.org/c/openstack/cyborg/+/977793 | 17:08 |
| *** gmaan_afk is now known as gmaan | 17:37 | |
| opendevreview | sean mooney proposed openstack/cyborg master: update pre-commit hook version and fix issues https://review.opendev.org/c/openstack/cyborg/+/977859 | 19:54 |
| opendevreview | sean mooney proposed openstack/cyborg master: Update pre-commit hooks and fix violations https://review.opendev.org/c/openstack/cyborg/+/977860 | 19:54 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix codespell violations across the codebase https://review.opendev.org/c/openstack/cyborg/+/977861 | 19:55 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix sphinx-lint and doc8 violations in RST docs https://review.opendev.org/c/openstack/cyborg/+/977862 | 19:55 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate project metadata to pyproject.toml https://review.opendev.org/c/openstack/cyborg/+/977886 | 21:52 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix pyproject.toml deprecations and remove wsgi_scripts https://review.opendev.org/c/openstack/cyborg/+/977887 | 21:52 |
| opendevreview | sean mooney proposed openstack/cyborg master: Add ruff config and fix pre-commit hook failures https://review.opendev.org/c/openstack/cyborg/+/977888 | 21:52 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix E402 in Alembic migrations and template https://review.opendev.org/c/openstack/cyborg/+/977889 | 21:52 |
| opendevreview | sean mooney proposed openstack/cyborg master: Fix S311 and S603 ruff security violations https://review.opendev.org/c/openstack/cyborg/+/977890 | 21:52 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate to ruff-format + ruff lint; drop hacking plugin https://review.opendev.org/c/openstack/cyborg/+/977891 | 21:52 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!