Earlier  
Later
New Search
Posted Nick Remark
#openstack-cyborg - 2026-06-30
15:25:52 melwitt ++ I think that is wise
15:25:53 sean-k-mooney but im thinking of adding it to the new nova-whitebox job
15:27:01 melwitt speaking of that, we need your re-review on https://review.opendev.org/c/openstack/nova/+/898100 to land it
15:27:02 sean-k-mooney its pretty stable form what i can tell and compiling it is fast but i was goign to give it a month or two before goign town that route
15:27:21 sean-k-mooney oh yes cool ill look at that shortly
15:27:30 melwitt yeah ++ for adequate bake time
15:28:46 sean-k-mooney it also currently need debian 13 or ubuntu 26.04 which i dont think that whiteboxc job uses
15:29:00 melwitt oh right
15:29:07 sean-k-mooney so i want to get the whitebox tests running on the cybrog job first
15:29:21 sean-k-mooney and only look at updatign the nova oen after we have all the kinks worked out
15:29:36 melwitt ++
15:29:40 sean-k-mooney one of the optimisation we need to do for whitebox is use the serial decorator
15:29:47 sean-k-mooney instead of --concurancy=1
15:30:23 sean-k-mooney i think gmann was consideritng doign that refactor but if not i woudl like to do that before integrating pci-sim into it
15:30:54 melwitt ok so about that
15:31:30 melwitt there is one test in whitebox, a vtpm test, that restarts libvirtd. this is what messes up all ability to use parallel. we need to separate out that test for it to run on its own
15:32:13 melwitt I could work on that actually, I just didn't think about it at the time
15:32:56 melwitt I guess there might be other similar tests in whitebox too. if so, just all of those need to go in their own serial/ directory and run after everything else like normal tempest does
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

Earlier   Later