| opendevreview | sean mooney proposed openstack/cyborg master: Document Cyborg maintainer roles and conventions https://review.opendev.org/c/openstack/cyborg/+/1008481 | 02:38 |
|---|---|---|
| opendevreview | sean mooney proposed openstack/cyborg master: Define Cyborg project scope and boundaries https://review.opendev.org/c/openstack/cyborg/+/1008513 | 02:38 |
| opendevreview | chandan kumar proposed openstack/cyborg master: [DNM] opendev github-check testing https://review.opendev.org/c/openstack/cyborg/+/1012510 | 02:39 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Add bind-time FPGA programming scenario test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/1014182 | 13:34 |
| jgilaber | #startmeeting cyborg | 14:00 |
| opendevmeet | Meeting started Tue Nov 10 14:00:11 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 today? | 14:00 |
| rlandy | o/ | 14:00 |
| melwitt | o/ | 14:00 |
| chandankumar | o/ | 14:00 |
| jgilaber | while we wait, as always feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L53 | 14:00 |
| sean-k-mooney | o/ | 14:00 |
| jgilaber | courtesy ping: amoralej bogdando skovili | 14:01 |
| jgilaber | let's start with the first topic | 14:02 |
| jgilaber | #topic openstack wiki | 14:02 |
| jgilaber | sean-k-mooney, go ahead | 14:02 |
| sean-k-mooney | tl;dr is the offial openstack wiki is offline following the php CVE a few weeks ago | 14:02 |
| sean-k-mooney | its unclear if it will ever come back | 14:03 |
| sean-k-mooney | but apart form soem outdated information and some test reports https://wiki.openstack.org/wiki/Cyborg/TestReport/InspurNVMeSSD | 14:03 |
| sean-k-mooney | we have mostly moved away form usign the wiki anyway | 14:03 |
| sean-k-mooney | so i would like to remove all refences to it form our docs | 14:03 |
| sean-k-mooney | and continue just the etherpad for mettings adn work trackign documents | 14:03 |
| sean-k-mooney | and gerrit and our actual project docs | 14:04 |
| sean-k-mooney | for long lived content | 14:04 |
| sean-k-mooney | any concerns with that and removeign all usage of the wiki form the proejct? | 14:04 |
| chandankumar | +1 for removing all the wiki reference | 14:04 |
| sean-k-mooney | i belive that is what ironic did as well | 14:04 |
| jgilaber | +1 from me as well | 14:05 |
| jgilaber | as noted we have very few links to the wiki | 14:05 |
| jgilaber | https://github.com/search?q=repo%3Aopenstack%2Fcyborg+wiki&type=code | 14:05 |
| melwitt | sounds reasonable to me | 14:05 |
| jgilaber | from a quick search only 5 links | 14:05 |
| sean-k-mooney | ya there may be others but i was not expecting many | 14:06 |
| sean-k-mooney | ok if we are agreed with the driection ill generate a docs update patch to remove them | 14:06 |
| jgilaber | #agreed remove openstack wiki links from cyborg docs | 14:07 |
| sean-k-mooney | ill also check if there is a more up to date location that we shoudl replace it with or if we need to add it to our docs gaps | 14:07 |
| jgilaber | sean-k-mooney++ thanks | 14:07 |
| jgilaber | others comments on this topic before moving on? | 14:08 |
| chandankumar | there is no reference of wiki in cyborg-tempest-plugin https://github.com/search?q=repo%3Aopenstack%2Fcyborg-tempest-plugin+wiki&type=code | 14:08 |
| jgilaber | thanks chandankumar | 14:08 |
| jgilaber | next topic also from sean-k-mooney | 14:08 |
| jgilaber | #topic tech-debt: deprecations and removals | 14:09 |
| sean-k-mooney | yep this is partly followup form the ptg | 14:09 |
| sean-k-mooney | the first one is pretty simple | 14:09 |
| sean-k-mooney | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2143943 | 14:09 |
| sean-k-mooney | teh v1 api which has been removed breily had quota supprot | 14:09 |
| sean-k-mooney | but it never existed for v2 | 14:09 |
| sean-k-mooney | last cycle i fixed nova to partly supprot unified limits with cybrog | 14:10 |
| sean-k-mooney | this bug just tracks removign the dead code | 14:10 |
| sean-k-mooney | so before we get too far into the cycle i just want to finally delete that dead code | 14:10 |
| sean-k-mooney | so i gues i had to points. one any concerns with doing that now | 14:11 |
| jgilaber | none from me, cleanup is always good | 14:11 |
| sean-k-mooney | and two ill create a patch to do this ebfore the next meeting unless someone else wants to pick it up | 14:11 |
| * sean-k-mooney is tring to burn down my ptg action item list | 14:12 | |
| sean-k-mooney | ok if no questions ill add that to my todo list and we can review in gerrit | 14:12 |
| sean-k-mooney | the next 2 topics are also form teh ptg | 14:13 |
| sean-k-mooney | updatign the driver supprot matix to move experimatal dreiver to deprecated for lack of testing | 14:13 |
| sean-k-mooney | and freezing and deprecating python-cybrogclient | 14:13 |
| sean-k-mooney | unless someone else want to pick either of these up i intened to create and inital patch for both before the next irc meeting | 14:14 |
| sean-k-mooney | any concerns or questions on either? | 14:14 |
| sean-k-mooney | melwitt: we have partity with osc/sdk now yes? | 14:14 |
| sean-k-mooney | my goal is to resolve all the actionable ptg Action items before the end of the month | 14:15 |
| melwitt | sean-k-mooney: I need to check on whether sdk has had a release since the microversion negotiation patch landed. that's the only thing I'm aware of that behaves differently regarding the default version on the CLI | 14:15 |
| sean-k-mooney | ack, the deprecation patch wont actully change anything beyond the readme and a release note but we can also hold it until there is a release if we want too | 14:16 |
| melwitt | looks like not released yet, fwiw | 14:16 |
| sean-k-mooney | would you prefer to handel that or will i just propose it and cc you | 14:16 |
| melwitt | feel free to propose it, either way is good with me | 14:17 |
| sean-k-mooney | cool any other questions or concerns on ^ | 14:17 |
| jgilaber | sounds good to me | 14:18 |
| jgilaber | let's move to reviews | 14:19 |
| jgilaber | #topic Reviews | 14:19 |
| jgilaber | sean-k-mooney, you have a few links, do you want to go one by one? | 14:19 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/1008481 | 14:19 |
| sean-k-mooney | either works | 14:19 |
| sean-k-mooney | the first is the maintianer guide | 14:19 |
| sean-k-mooney | this was created partly in responce to the grovernace resolution for projects | 14:20 |
| sean-k-mooney | to document there review confvtions, ways of working and path to core | 14:20 |
| sean-k-mooney | that pairs with | 14:20 |
| sean-k-mooney | #link https://review.opendev.org/c/openstack/cyborg/+/1008513 | 14:20 |
| sean-k-mooney | the scope fo the project docuemnt | 14:20 |
| sean-k-mooney | which is a high level docuemnt indeded to help guide scoping decsions for future and current work | 14:21 |
| sean-k-mooney | based on similer docs in nova,keystone,glance and cidner | 14:21 |
| sean-k-mooney | togherer this is an effort from me to write down some of the tribal knowldage | 14:22 |
| sean-k-mooney | so that future maintianer dont have to resurrect this for themsleve form pass reviews | 14:22 |
| sean-k-mooney | and to help new contibotrs knwo what the expections are | 14:22 |
| sean-k-mooney | if there are no other objects or open question on those i would like to merge them this week | 14:23 |
| sean-k-mooney | ill pause there for a sec before movign to the next too any comments? | 14:23 |
| jgilaber | I have the docs patches open to review, hopefully today | 14:23 |
| jgilaber | the specs are good to go I think | 14:24 |
| sean-k-mooney | thats what i was going to go to next | 14:24 |
| sean-k-mooney | ill leave the scope and maintienr guide open for another few days and merge on friday if there are no objections | 14:25 |
| sean-k-mooney | i guess we can move on to the specs | 14:25 |
| sean-k-mooney | #link https://review.opendev.org/c/openstack/cyborg-specs/+/1012031 | 14:25 |
| sean-k-mooney | is the ovo object indriection api spec | 14:25 |
| sean-k-mooney | this will be a prequist to both the nvme and pci manged mode sepcs | 14:26 |
| sean-k-mooney | #link https://review.opendev.org/c/openstack/cyborg-specs/+/1012084 | 14:26 |
| sean-k-mooney | is the pci managed mode spec | 14:26 |
| sean-k-mooney | as jgilaber noted i also belive these are good to go | 14:27 |
| sean-k-mooney | are there any question on either | 14:27 |
| sean-k-mooney | or woudl anyone like more time? | 14:27 |
| chandankumar | from myside, both are good to go | 14:27 |
| sean-k-mooney | if not ill merge them after the meeting otherwise ill wait till firday for any new comemnts | 14:27 |
| sean-k-mooney | obviously if there are open question we have tiem to discuss them | 14:28 |
| sean-k-mooney | melwitt: rlandy any input? if not we can move on and ill appvoe them after the meeting | 14:29 |
| melwitt | no, sorry I did not review them properly | 14:30 |
| sean-k-mooney | would you like me to keep them open for time to allow you to reveiw? | 14:30 |
| rlandy | ok by me | 14:31 |
| rlandy | I can look at them tomorrow morning | 14:31 |
| melwitt | sure I can do the same | 14:31 |
| sean-k-mooney | im partly trying to reduce the number of patches i have in flight but im also not in a large rush | 14:31 |
| sean-k-mooney | ok ill leave them open till firday for any other comment and we can dicuss more next week if needed | 14:32 |
| sean-k-mooney | thats all i had | 14:32 |
| jgilaber | thanks sean-k-mooney | 14:32 |
| jgilaber | chandankumar, you also wanted to highlight some patches | 14:33 |
| chandankumar | yes | 14:33 |
| jgilaber | #link https://review.opendev.org/q/topic:%22bug/2157586%22 | 14:33 |
| chandankumar | I was working on fixing https://bugs.launchpad.net/openstack-cyborg/+bug/2157586: Cyborg Device Profile API rejects OWNER_CYBORG trait. | 14:33 |
| chandankumar | 997945: Validate device profile traits against os-traits library | https://review.opendev.org/c/openstack/cyborg/+/997945 and allow to pass CUSTOM_ traits while | 14:33 |
| chandankumar | creating device profile. | 14:33 |
| chandankumar | It depends on 992210: Add per-driver Cyborg-to-Placement device test | https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | 14:33 |
| chandankumar | it adds device key to pass vendor and product id under cyborg_ group in tempest.conf | 14:34 |
| chandankumar | and then validates the placement resource provider. | 14:34 |
| sean-k-mooney | so i have one dbout regarding https://review.opendev.org/c/openstack/cyborg/+/997945 | 14:35 |
| sean-k-mooney | Validate device profile traits against os-traits library is not wrong | 14:35 |
| sean-k-mooney | however | 14:35 |
| sean-k-mooney | it is assumign that cybrog and placment are usign the saem version fo os-traits | 14:35 |
| sean-k-mooney | which in many cases will be correct | 14:36 |
| sean-k-mooney | but not nessiarly in all cases. | 14:36 |
| sean-k-mooney | part of me wonders if we shoudl validate agaisnt placments reported traits instead | 14:36 |
| sean-k-mooney | the issue with that is i dont think our rest api shoudl really have to call placements as part of its validation | 14:37 |
| sean-k-mooney | im inclided to proceed with your patch as proposed | 14:38 |
| sean-k-mooney | using os-traits for now | 14:38 |
| sean-k-mooney | but we may want to condier if we shoudl instead check the live traits from placmenet in a followup | 14:38 |
| sean-k-mooney | what do others think? | 14:39 |
| chandankumar | I totally agree here, os-traits version used by placement and cyborg might be different | 14:39 |
| jgilaber | I'm undecided | 14:39 |
| melwitt | hm yeah ... my first thought is I'm curious if there is precedent in any other project regarding such validation | 14:40 |
| jgilaber | not sure we should reject device profiles with a valid trait just because there is no rp that has them at the moment | 14:40 |
| melwitt | I would be wary to assume same os-traits version as well | 14:40 |
| sean-k-mooney | melwitt: i think nova jsut takes what you set in teh flavor/image and passes it through | 14:40 |
| sean-k-mooney | so we coudl also jsut remove the validation | 14:40 |
| sean-k-mooney | we just say its operator error if you request a trait that does not exist | 14:41 |
| sean-k-mooney | you will just get a no valid host | 14:41 |
| sean-k-mooney | which is exactly how this woudl fail too | 14:41 |
| sean-k-mooney | palceemnt will return no allcoation candiates | 14:41 |
| sean-k-mooney | im very tempted to simplfy the code and remove the check and "trust the operator" to get this write | 14:42 |
| sean-k-mooney | *right | 14:43 |
| melwitt | yeah I guess, given that versions can be different, if we keep the validation maybe it should not be a blocker but rather some sort of warning? but even that is probably overkill as warning means operator action is needed usually. I dunno | 14:43 |
| * sean-k-mooney does not trust me to spell anything right | 14:43 | |
| sean-k-mooney | so the validation is for device profile create | 14:43 |
| sean-k-mooney | i feel like logging a warning is not very discoverable | 14:44 |
| sean-k-mooney | like sure we told you it might be wrong but we ran into the other room and wispered it to you | 14:44 |
| melwitt | yeah, that's why my train of thought kind of stalled out there | 14:45 |
| sean-k-mooney | we do not have a clean way to acppcte the profile adn return the warning to you in the api responce | 14:45 |
| jgilaber | I would prefer removing the check before checking with placement personally, better cost/benefit | 14:45 |
| sean-k-mooney | thats where im leanign too | 14:45 |
| melwitt | same | 14:45 |
| sean-k-mooney | in anycase i think we can proceed with https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | 14:46 |
| sean-k-mooney | ill review that after the meeting | 14:46 |
| sean-k-mooney | then we can revist the actual validation fix | 14:46 |
| sean-k-mooney | chandankumar: what do you think are you ok with reworkign it to remove the validation? | 14:46 |
| chandankumar | Ok I will update the patch to remove the validation | 14:46 |
| sean-k-mooney | ok cool lets see what that looks like and we can hopefully close out that bug in the next few days | 14:47 |
| chandankumar | yup | 14:47 |
| chandankumar | that's it from my side on these reviews | 14:48 |
| jgilaber | ok, that is the end of the list of reviews | 14:49 |
| jgilaber | and we have no new bugs | 14:49 |
| jgilaber | any last minute topic? | 14:49 |
| sean-k-mooney | i have a short one | 14:49 |
| sean-k-mooney | https://etherpad.opendev.org/p/cyborg-2026.2-status | 14:49 |
| sean-k-mooney | so i need to replace that | 14:49 |
| sean-k-mooney | should i a create a new one for 2027.1 | 14:49 |
| sean-k-mooney | or B drop the release and have a roling document | 14:50 |
| sean-k-mooney | im thinking A | 14:50 |
| sean-k-mooney | but i i just said i woudl ask before i do it and update the topic link | 14:50 |
| melwitt | probably A | 14:50 |
| jgilaber | we already have one https://etherpad.opendev.org/p/cyborg-2027.1-status | 14:50 |
| melwitt | that rolling document would be rolling pretty far | 14:50 |
| sean-k-mooney | ya and cool ok ill just update the link | 14:51 |
| sean-k-mooney | it also useful to have this per release | 14:51 |
| sean-k-mooney | when it comes to the release notes and marketing highlights | 14:51 |
| sean-k-mooney | ok that was quick ill lookup the irc bot comamdn and fix the topic soon | 14:52 |
| jgilaber | ack thanks sean-k-mooney | 14:54 |
| jgilaber | last topic | 14:54 |
| jgilaber | #topic Volunteers to chair next meeting | 14:55 |
| jgilaber | any volunteer? | 14:55 |
| *** ChanServ changes topic to "OpenStack Cyborg development | planning: https://etherpad.opendev.org/p/cyborg-2027.1-status | meeting: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting" | 14:55 | |
| sean-k-mooney | i guess i can if no one else wants too | 14:57 |
| jgilaber | ok thanks sean-k-mooney | 14:57 |
| jgilaber | that is all for today thanks everyone! | 14:57 |
| jgilaber | #endmeeting | 14:57 |
| opendevmeet | Meeting ended Tue Nov 10 14:57:40 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:57 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-11-10-14.00.html | 14:57 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-11-10-14.00.txt | 14:57 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-11-10-14.00.log.html | 14:57 |
| chandankumar | thanks jgilaber for chairing the meeting. | 14:57 |
| chandankumar | I will be not around from Nov 16 to Nov 24. | 14:58 |
| sean-k-mooney | chandankumar: ack | 15:01 |
| *** gmaan is now known as gmaan_afk | 17:34 | |
| *** gmaan_afk is now known as gmaan | 19:04 | |
| opendevreview | melanie witt proposed openstack/cyborg-specs master: Add services API spec for 2027.1 https://review.opendev.org/c/openstack/cyborg-specs/+/1001999 | 23:49 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!