Earlier  
Posted Nick Remark
#openstack-cyborg - 2026-06-30
16:06:49 sean-k-mooney melwitt: so there are a couple of whitebox tests that restart the agent
16:07:15 sean-k-mooney melwitt: we used ot have test that reconfigre the nova.conf setting to test spefic feature that were mutally exclivie
16:07:46 sean-k-mooney so for vtpm ya there is one that need special handleing but the test for offliniing cpus and some fo the pining test used to restart agents too
16:08:01 melwitt yeah
16:08:36 sean-k-mooney so if you have time to work on making whitebox work with the serial decorator that woudl be cool
16:08:45 sean-k-mooney but no pressure either way
16:09:41 sean-k-mooney my current plan is wrap up srbac this week. then finish the pci-mangege mode work in then 1-2 weeks and then loook at addign whitebox tests
16:09:51 sean-k-mooney so if you get to it before the end fo the month the cool
16:10:00 sean-k-mooney if not ill look at it in august
16:10:12 sean-k-mooney *end of the month being july
16:11:29 sean-k-mooney im happy with the simple lifecyle test i have that ssh into the vm and run lspci are enouch for the pci driver testing im using right now. but i want to add a whitebox test that will check the xml actuly has manged=no later
16:12:25 sean-k-mooney i also verifyed it on real hardware. turns out if managed mode is not set with an l4 booting a vm will hard reboot the host which was fun to debug
16:12:37 sean-k-mooney so nvidia still have not fixed that kernel bug :)
16:13:09 sean-k-mooney the pci-sim wont crash the same way
16:17:29 melwitt oh wow hah
16:25:52 sean-k-mooney we have a note about that in the nova docs
16:26:02 sean-k-mooney it really really shoudl not happen
16:26:31 sean-k-mooney but ya i was like why is my ssh conenction droping when i boot a vm
16:26:39 sean-k-mooney host reboot will do that :)
16:38:02 melwitt crazy haha
16:46:44 opendevreview melanie witt proposed openstack/cyborg master: Fix api-ref inaccuracies and add missing program endpoint https://review.opendev.org/c/openstack/cyborg/+/995533
17:19:13 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205
17:28:52 opendevreview Joan Gilabert proposed openstack/cyborg-specs master: Add generic mdev driver spec for 2026.2 https://review.opendev.org/c/openstack/cyborg-specs/+/982276
18:15:54 opendevreview melanie witt proposed openstack/cyborg master: Add C300 hacking check for set literal in logging calls https://review.opendev.org/c/openstack/cyborg/+/995561
18:38:09 opendevreview melanie witt proposed openstack/cyborg master: Add C300 hacking check for set literal in logging calls https://review.opendev.org/c/openstack/cyborg/+/995561
20:52:29 sean-k-mooney melwitt: i goign to cal lit a day but i jsut re reviewd the hackign check and i was incorrect about removig pycodestyle
20:52:35 sean-k-mooney just pushed updated comemnts
20:54:01 melwitt sean-k-mooney: ok cool thanks. I was just having the same confusion :)
20:55:41 sean-k-mooney i didnt realsie you were using it in the unit test
20:56:00 sean-k-mooney so its a driect depency of the unit tests so you were right
20:56:22 sean-k-mooney and ya i have not seen teim recommend addign teh liscen here to the __init__.py
20:56:31 sean-k-mooney im not sure if that is becaue i just updated the model
20:56:41 sean-k-mooney or if its a chagne i hacking
20:56:59 sean-k-mooney but ya with the current hackign setting that is an error ill correct that
20:57:37 sean-k-mooney teim-ci will read HACKING.rst or AGENTS.md if they are present
20:57:57 melwitt ah ok
20:58:03 sean-k-mooney so i can add a note in HACKING.rst about that
20:58:20 sean-k-mooney or tweak its prompt for this edgecase
20:58:53 sean-k-mooney ok chat to you tomorrow o/
20:59:22 melwitt cool thanks, seeya tomorrow o/
21:42:12 opendevreview melanie witt proposed openstack/cyborg master: Add C300 hacking check for set literal in logging calls https://review.opendev.org/c/openstack/cyborg/+/995561
#openstack-cyborg - 2026-07-01
07:36:40 opendevreview chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349
08:12:20 opendevreview chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349
11:39:51 opendevreview chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349
12:03:54 opendevreview chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349
15:06:14 sean-k-mooney jgilaber: chandankumar melwitt just an fyi im +2 on both specs. i wil leave them open in cases there are any more coment but i i plan to merge them tomorrow otherwise
15:06:35 sean-k-mooney let me knwo if you want more time to reveiw othersie ill proceed in my morning
15:07:05 jgilaber sounds good to me, I'll do a last pass on the nvme spec now
15:08:00 chandankumar sure
15:09:30 melwitt ++
15:30:59 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove [glance] debug https://review.opendev.org/c/openstack/cyborg/+/995707
15:47:44 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534
15:50:39 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534
15:53:55 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused [keystone] group https://review.opendev.org/c/openstack/cyborg/+/995716
16:12:44 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534
17:17:10 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused image utilities https://review.opendev.org/c/openstack/cyborg/+/995534
17:55:15 opendevreview Merged openstack/cyborg master: Remove unused [keystone] group https://review.opendev.org/c/openstack/cyborg/+/995716
#openstack-cyborg - 2026-07-02
04:23:43 opendevreview chandan kumar proposed openstack/cyborg-specs master: Add generic NVMe driver spec with secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349
08:23:58 chandankumar sean-k-mooney: hello, Feel free to take a look at https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330 and https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992339 when free.
08:24:09 chandankumar Rest of the patches in this series looks good to merge!
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

Earlier   Later