| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-07-02 | |||
| 09:19:10 | opendevreview | Ronelle Landy proposed openstack/python-cyborgclient master: Remove deprecated v1 API and fix Python 3.12+ SSL compatibility https://review.opendev.org/c/openstack/python-cyborgclient/+/995789 | |
| 10:50:42 | sean-k-mooney | chandankumar: thanks for the reminder ill try and adress those today | |
| 11:24:38 | opendevreview | Merged openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349 | |
| 11:24:40 | opendevreview | Merged openstack/cyborg-specs master: Add generic mdev driver spec for 2026.2 https://review.opendev.org/c/openstack/cyborg-specs/+/982276 | |
| 11:45:58 | opendevreview | Ronelle Landy proposed openstack/python-cyborgclient master: Remove deprecated v1 API and fix Python 3.12+ SSL compatibility https://review.opendev.org/c/openstack/python-cyborgclient/+/995789 | |
| 14:32:32 | sean-k-mooney | chandankumar: oh your right https://review.opendev.org/c/openstack/cyborg/+/994577?tab=change-view-tab-header-zuul-results-summary is not enabling the new tests as exected beasue i have not added the depens on | |
| 14:32:51 | sean-k-mooney | i have doen that in my srbac serise so i need to fix that | |
| 14:33:24 | sean-k-mooney | https://review.opendev.org/c/openstack/cyborg/+/992332/6 | |
| 14:37:51 | chandankumar | sean-k-mooney: this week I am also thinking to port pci-sim on cs10 it will help to integrate it with install yamls and tests stuff using pci-driver | |
| 14:38:21 | sean-k-mooney | well im not sure we shoudl need to modify install yamls | |
| 14:38:25 | sean-k-mooney | but sure | |
| 14:38:50 | sean-k-mooney | i was exectign to just have a playbook for this in the nova-operator repo in the ci folder | |
| 14:39:22 | sean-k-mooney | i dont think addign c10s supprot will be hard persay just need to gernalise how we isntall the depenciences | |
| 14:40:24 | sean-k-mooney | chandankumar: when working on that it might make sense to add a c10s ro rocky/alma job to the expermiental and weekly lines | |
| 14:40:31 | sean-k-mooney | upsteam | |
| 14:40:59 | chandankumar | playbook idea is also fine | |
| 14:41:34 | chandankumar | sure, let me get the patch up. | |
| 14:41:41 | sean-k-mooney | so i was thinking a playbooks and or make target to do it but whatever works. i generally dont like useing insall_yamls for much | |
| 14:42:07 | sean-k-mooney | i mean its fine its just a prefernce | |
| 14:42:17 | sean-k-mooney | the reason i was thinkign a playbook | |
| 14:43:05 | sean-k-mooney | is my over all plan was to define a custom datapalen service that woudl run it on the edpm nodes as part of the dataplane deploy | |
| 14:44:13 | sean-k-mooney | so we woudl then just havea kustomization that added that custom datapalen service to the node set the way repo setup works | |
| 14:45:28 | sean-k-mooney | so rather then a freestanding playbook defiening it inline within the dataplaen sevice might be the best approch | |
| 14:45:51 | sean-k-mooney | or do it wih kustomize | |
| 14:46:27 | chandankumar | yes, custo kustomization repo setup seems to be much better | |
| 14:46:30 | sean-k-mooney | in either case wi twoudl be trivial to have a make target that just does an oc apply -k/-f to defien the serice and it easy to then add that to the dataplane node set | |
| 14:47:33 | sean-k-mooney | i assume you were thinking it woudl jsut clone the cybrog repo if not present on the edpm ndoe and then do the compile caling the devstack plugin directly or simialr. | |
| 14:47:48 | sean-k-mooney | actully it does nto really need to do that | |
| 14:47:55 | sean-k-mooney | it can just use the make targets | |
| 14:48:05 | chandankumar | yes, clone and run make target | |
| 14:48:12 | sean-k-mooney | ya ok | |
| 14:48:14 | sean-k-mooney | i think that works | |
| 14:48:44 | sean-k-mooney | if you can get that workign on c10s after you get the devstack version done that would be cool | |
| 14:48:54 | sean-k-mooney | amoralej: ^ | |
| 14:49:05 | sean-k-mooney | does that sound good to you | |
| 14:49:35 | amoralej | reading back | |
| 14:49:40 | sean-k-mooney | chandankumar: that woudl be a pr to the nova operator on top of amoralej pr | |
| 14:50:16 | chandankumar | ok | |
| 14:50:34 | sean-k-mooney | i woudl do it like this https://github.com/openstack-k8s-operators/nova-operator/blob/main/ci/nova-operator-tempest-multinode/pre_deploy_hook.yml | |
| 14:50:53 | melwitt | sean-k-mooney: re: the enforce_scope patch, if we can I think it would be better to move it as early in the SRBAC series as we can bc the supposedly deadline for landing it is tomorrow | |
| 14:51:24 | sean-k-mooney | melwitt: that just because it will break us when they do the new oslo release | |
| 14:51:38 | sean-k-mooney | melwitt: but ya let me just do ti against master now | |
| 14:51:44 | sean-k-mooney | melwitt: unless you want to? | |
| 14:51:50 | melwitt | ah gotcha. ok cool | |
| 14:51:51 | amoralej | but you mean out of ci-fw? | |
| 14:52:00 | amoralej | sorry, i'm not following the conversation | |
| 14:52:02 | melwitt | sean-k-mooney: no feel free to go ahead | |
| 14:52:17 | sean-k-mooney | amoralej: yes we shoudl not need to do this isn ci-fw | |
| 14:52:29 | sean-k-mooney | we coudl but why add the extra complexity | |
| 14:52:46 | sean-k-mooney | amoralej: that is the other option do it like the hci-prepare roles | |
| 14:53:10 | amoralej | this is what i did to run cyborg with tempest in edpm job https://github.com/openstack-k8s-operators/nova-operator/pull/1143/changes/689f12f927268a4e94a7e7f8445c149bc3c776ff | |
| 14:53:23 | sean-k-mooney | amoralej: if we do it in the operator its easy to then have a make target in nova-opertor to use it | |
| 14:53:28 | amoralej | i added a ci/nova-operator-cyborg-tempest-multinode/pre_deploy_hook.yml | |
| 14:53:37 | sean-k-mooney | if we do it in ci-fw that make usign the pci-sim lcoally harder | |
| 14:53:39 | amoralej | similar to what you pointed but adding cyborg service | |
| 14:54:24 | sean-k-mooney | amoralej: so yes my soggestion was to buidl on that | |
| 14:54:49 | sean-k-mooney | and add a new pci-sim OpenStackDataPlaneService | |
| 14:55:32 | sean-k-mooney | that will clone the cybrog repo onto the edpm node and compile the pci-sim for cybrog to then use | |
| 14:55:50 | amoralej | ah, got it | |
| 14:55:55 | sean-k-mooney | amoralej: so we can start iwth only the fake driver | |
| 14:56:09 | sean-k-mooney | then once chandankumar has c10s supprot working we can add in the pci-sim | |
| 14:57:13 | sean-k-mooney | chandankumar: amoralej: if we manage to get all that workign how feisbel woudl it be to add a operator job on opendev chagnes? | |
| 14:57:14 | amoralej | iirc, we are still using CS9 in the centos10 jobs used in operators because of some issue | |
| 14:57:41 | sean-k-mooney | oh good to know | |
| 14:58:04 | sean-k-mooney | i guess we will have to resolve that before the year is out one way or another | |
| 14:58:28 | sean-k-mooney | the priorty is devstack c10s supprot then operators after | |
| 14:58:36 | amoralej | yeah, it's CS9, i think chandankumar worked on that, he may remember | |
| 14:58:54 | amoralej | yes, it needs to be fixed somehow | |
| 14:59:46 | sean-k-mooney | i think the issue is realate to the repo setup | |
| 14:59:56 | sean-k-mooney | and or rdo/c10s package aviablity | |
| 15:00:31 | sean-k-mooney | i did try useing c10s recently but didnt debug why it was failing beyound repo-setup | |
| 15:02:14 | chandankumar | on cs10 some packages were not available that time | |
| 15:02:44 | chandankumar | and some config issue i think | |
| 15:03:11 | chandankumar | on dataplane node (I donot exactly remember the details) | |
| 15:04:30 | sean-k-mooney | ya i think it would not neesiarly be hard to fix it just wasnt impornat to fix for what i was doing at the time | |
| 15:05:08 | sean-k-mooney | chandankumar: so we may however defer the edpm supprot for pci-sim | |
| 15:05:13 | sean-k-mooney | until that is resolved | |
| 15:05:42 | sean-k-mooney | we shoudl follow up with that downstream and see if we can get that resolved by the relevent teams | |
| 15:06:04 | chandankumar | ok | |
| 16:43:26 | melwitt | I'm trying out some openstacksdk test stuff https://review.opendev.org/c/openstack/openstacksdk/+/995851 if it goes well I will add more tests and depends-on rlandy's patch | |
| 17:08:21 | sean-k-mooney | oh cool that woudl be cool | |
| 17:13:26 | opendevreview | sean mooney proposed openstack/cyborg master: Adapt Cyborg to always-enforced policy scope https://review.opendev.org/c/openstack/cyborg/+/995867 | |
| 17:13:51 | melwitt | actually I think I could use Needed-By | |
| 17:14:57 | sean-k-mooney | you can | |
| 17:15:06 | sean-k-mooney | we dont have tooling that really sues that | |
| 17:15:30 | sean-k-mooney | Needed-By is jsut for humans today | |
| 17:15:49 | melwitt | oh really? ok I saw it on some gmaan's patches | |
| 17:15:52 | sean-k-mooney | so wither you want to do needed-by or depend on in this case is kind of up to you on merge order | |
| 17:16:15 | sean-k-mooney | ya so we use it for thinks like tackign the steps in new project creation | |
| 17:16:21 | sean-k-mooney | but zuul wont use it for anything | |
| 17:16:34 | melwitt | yeah. I was thinking openstacksdk review pretty quick usually so that's what made me think about it | |
| 17:16:43 | melwitt | ok good to know | |
| 17:17:02 | sean-k-mooney | so it normally is but currently only artem i belive has merge access | |
| 17:17:07 | gmaan | depends-on block the merge but yes needed-by is more of giving a context in commit msg about its need | |
| 17:17:33 | sean-k-mooney | stphenfin is on peternitiy leave for the next 2 weeks then part time after that | |
| 17:17:39 | melwitt | oh ok I thought stephenfin was on there. ok | |
| 17:17:46 | melwitt | ahh ok | |
| 17:17:49 | sean-k-mooney | he is | |
| 17:17:55 | melwitt | I'll use Depends-On then | |
| 17:18:31 | sean-k-mooney | i have +2 rights i belive but no +w so when your happy with it im happy to review | |