| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-06-30 | |||
| 15:32:57 | gmaan | i have that in my list but I think I will not be able to start that for next 2 weeks, feel free to do it if you can before that | |
| 15:33:18 | melwitt | ah cool :) | |
| 15:35:48 | gmaan | sean-k-mooney: melwitt I was thinking to add it in meeting but missed. as you both are here, please check/test the enforce_scope removal for cyborg, I have proposed the example changes for other projects, melwitt has reviewed the nova/placement one already. https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/436HHMRVNUY3OSXE4ULPQ4P4DSHQEMES/ | |
| 15:36:36 | melwitt | ok cool yes I will look | |
| 15:36:41 | gmaan | thanks | |
| 15:38:03 | gmaan | i can see only a few reference of that but you can add testing change with oslo.policy change and test if anything else other than those few test changes is needed https://github.com/search?q=repo%3Aopenstack%2Fcyborg%20enforce_scope&type=code | |
| 15:38:16 | melwitt | oh yikes the deadline is july 3 | |
| 15:44:51 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205 | |
| 15:46:38 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205 | |
| 16:02:17 | opendevreview | melanie witt proposed openstack/cyborg master: Fix api-ref inaccuracies and add missing program endpoint https://review.opendev.org/c/openstack/cyborg/+/995533 | |
| 16:02:46 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused _extract_attributes_v2 https://review.opendev.org/c/openstack/cyborg/+/995534 | |
| 16:02:55 | sean-k-mooney | gmaan: yes i have been folowing that form afar | |
| 16:03:13 | sean-k-mooney | gmaan: basiclly waitign for you and stephen to figure out the way forward | |
| 16:03:37 | gmaan | sean-k-mooney: ack, thanks | |
| 16:03:42 | opendevreview | Takashi Kajinami proposed openstack/cyborg master: Remove unused method for image v1 API https://review.opendev.org/c/openstack/cyborg/+/995534 | |
| 16:03:53 | sean-k-mooney | gmaan: so i need to remove my overried where i explicly default that to true | |
| 16:04:15 | sean-k-mooney | but i wrote teh securigy bug fixes assuming the enfoce scope | |
| 16:04:18 | gmaan | yes, true or false, basically all usage of enforce_scope | |
| 16:04:34 | sean-k-mooney | and that whaat my srbac serise is built around as well | |
| 16:04:50 | gmaan | enforce_new_defaults stay same to toggle the old/new defaults but not the scope | |
| 16:04:58 | sean-k-mooney | yep | |
| 16:05:31 | sean-k-mooney | ill tack a patch on the end of my srbac serise to drop our usage | |
| 16:05:54 | sean-k-mooney | gmaan: im hopign to merge that this week as well | |
| 16:06:09 | gmaan | ++ | |
| 16:06:35 | sean-k-mooney | melwitt: sorry was on a call | |
| 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 | |