Earlier  
Posted Nick Remark
#openstack-cyborg - 2026-06-26
14:44:09 chandankumar but it is needed for showing nvme on host
14:44:18 sean-k-mooney not nessiarly
14:44:30 chandankumar I was not getting other idea so borrowed from nvmevirt
14:46:13 sean-k-mooney i belive we can allcoate a much smaller ammoutn fo memroy and then back the nvme deevice with files on disk
14:47:00 sean-k-mooney chandankumar: anyway push it for review when you think its ready for that
14:47:08 sean-k-mooney and we can dicuss that in more detail
14:47:54 chandankumar min memmap reservation: 112 KB of reserved physical RAM, one 16 KB NVMe BAR0 region per VF
14:48:10 chandankumar ok pushing
14:49:18 sean-k-mooney so the pci-sim today does not preacllotate resreved memory
14:49:35 sean-k-mooney it allcoates a small amount at runtime for the pci config space and minimal bars
14:50:19 sean-k-mooney i expect we shoudl be able ot do the same for the nvem emulation (basiclly the contole regeiseter and command/data queues)
14:50:35 sean-k-mooney but keep the data storate just as a raw file on disk
14:50:53 sean-k-mooney so we are not using ram for the nvme capscity jsut the hardware emulation
14:53:29 sean-k-mooney chandankumar: https://paste.opendev.org/show/bjZuArbpAxmE35uoUUoV/ was my design chat on this topic from a week or to ago
14:53:37 sean-k-mooney https://paste.opendev.org/show/bjZuArbpAxmE35uoUUoV/
14:53:56 sean-k-mooney was the draft planing prom i was creating to help design this work
14:54:42 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
14:54:45 sean-k-mooney you will noteice that still reference https://github.com/SeanMooney/cyborg-extra/blob/master/pci-sim/fake_pci_sriov.c
14:54:53 sean-k-mooney since that was over a month ago
14:56:29 sean-k-mooney May 17, 12:20 AM
14:57:17 chandankumar ah thank you for sharing that , I will go over that
14:58:08 sean-k-mooney the paste was the final promt i had seelte on as the requriemet for the next plannign session to design the overall vf_personality feature
14:58:36 sean-k-mooney but i parked that work on until the intial pahse was merged and hook up into tempest testing as a phase 2 work
15:05:25 chandankumar ok
15:05:42 chandankumar see ya on monday, have a good weekend!
15:05:56 sean-k-mooney o/
#openstack-cyborg - 2026-06-28
07:25: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
09:17:57 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
09:47:24 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
14:25:20 opendevreview Takashi Kajinami proposed openstack/cyborg master: Refactor RequestContext https://review.opendev.org/c/openstack/cyborg/+/995195
16:56:52 opendevreview Takashi Kajinami proposed openstack/cyborg master: Refactor RequestContext https://review.opendev.org/c/openstack/cyborg/+/995195
16:58:08 opendevreview Takashi Kajinami proposed openstack/cyborg master: Refactor RequestContext https://review.opendev.org/c/openstack/cyborg/+/995195
17:03:31 opendevreview Takashi Kajinami proposed openstack/cyborg master: Remove unused context properties https://review.opendev.org/c/openstack/cyborg/+/995205
17:11:56 opendevreview Takashi Kajinami proposed openstack/cyborg master: Add shebang to mark bash files https://review.opendev.org/c/openstack/cyborg/+/995206
#openstack-cyborg - 2026-06-29
00:41:13 opendevreview Takashi Kajinami proposed openstack/cyborg master: Add shebang to mark bash files https://review.opendev.org/c/openstack/cyborg/+/995206
04:58:46 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
07:53:53 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
08:17:11 opendevreview chandan kumar proposed openstack/cyborg master: [WIP]pci-sim: add NVMe 1.4 controller emulation with host visibility https://review.opendev.org/c/openstack/cyborg/+/995079
13:08:21 opendevreview chandan kumar proposed openstack/cyborg-tempest-plugin master: [DNM] testing pci-sim nvme life cycle https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/995328
13:53:51 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
14:28:40 opendevreview chandan kumar proposed openstack/cyborg-tempest-plugin master: [DNM] testing pci-sim nvme life cycle https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/995328
15:30:23 chandankumar sean-k-mooney: Hello, this time all the tests passed https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/995328 (with some modification), https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5ac/openstack/5ac2a8a4b85843cfa8aea0aa2a697a9c/testr_results.html
15:30:40 chandankumar in lsblk output nvme block is not there
15:30:43 chandankumar in the guest
15:34:49 chandankumar ah, I need to add memmap params for that, I need a baremetal job there it will work, but I can test that tomororw
15:50:09 opendevreview chandan kumar proposed openstack/cyborg-tempest-plugin master: [DNM] testing pci-sim nvme life cycle https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/995328
16:12:14 sean-k-mooney ack ill try and take a look. however i would prefer if we did nto have a seperate job for the nvme work
16:12:45 sean-k-mooney so we might start that way but i woudl like to have a single job that test all 3 of pci,nvme and mdev drivers before teh end of the release
#openstack-cyborg - 2026-06-30
11:06:19 opendevreview sean mooney proposed openstack/cyborg master: Use request context as policy target https://review.opendev.org/c/openstack/cyborg/+/992722
11:06:20 opendevreview sean mooney proposed openstack/cyborg master: Add SRBAC base rules and enforce_new_defaults override https://review.opendev.org/c/openstack/cyborg/+/992292
11:06:20 opendevreview sean mooney proposed openstack/cyborg master: Migrate ARQ policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992293
11:06:21 opendevreview sean mooney proposed openstack/cyborg master: Migrate device policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992294
11:06:21 opendevreview sean mooney proposed openstack/cyborg master: Migrate deployable policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992295
11:06:22 opendevreview sean mooney proposed openstack/cyborg master: Migrate attribute policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992296
11:06:22 opendevreview sean mooney proposed openstack/cyborg master: Add SRBAC policy concepts documentation https://review.opendev.org/c/openstack/cyborg/+/992297
11:06:23 opendevreview sean mooney proposed openstack/cyborg master: Add cyborg-tempest-srbac Zuul job https://review.opendev.org/c/openstack/cyborg/+/992332
11:06:23 opendevreview sean mooney proposed openstack/cyborg master: Remove no-op deprecated policy bridges https://review.opendev.org/c/openstack/cyborg/+/995462
12:03:49 opendevreview Merged openstack/cyborg master: Fix incorrect set syntax in placement client logging https://review.opendev.org/c/openstack/cyborg/+/994851
12:03:51 opendevreview Merged openstack/cyborg master: Rename devstack CYBORG_CONF_FILE to CYBORG_CONF for consistency https://review.opendev.org/c/openstack/cyborg/+/979805
12:12:51 chandankumar sean-k-mooney: Hello, can you take a look at https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330 and https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992328/ in your free time, not super urgent, thank you!
12:29:34 sean-k-mooney ill update https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330/3 a little later today
12:31:26 sean-k-mooney i would liek to merge the srbac serives this week assuming there are no outstandign issues. after that i plan ot adress teh ruff/per-commit changes next on the tempest plugin side but ill update https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992347/1 to cover all ai agents first
12:32:20 chandankumar no all good from my side
12:32:30 chandankumar I will take a look at docs tomorrow
12:32:40 sean-k-mooney chandankumar: do you have any patch for teh tempest plugin you think woudl be good to land sooner rather then later?
12:33:27 sean-k-mooney i need to swap to spec review soon and hopefully we can move forward whith those today
12:33:37 sean-k-mooney so that next on my list
12:34:00 chandankumar sean-k-mooney: I think I can drop https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 in favor of your pci scenario tests.
12:34:17 chandankumar No other patch from my side on cyborg-tempest-plugin
12:34:51 sean-k-mooney oh the placement ones
12:35:21 sean-k-mooney those are not nessisarly reduntance but they are also not actully secnairo tests in the normal sense
12:35:45 chandankumar yes, but it is alos more than api tests
12:36:19 sean-k-mooney ya so i think we done really need to have placement assert the provier toplogy
12:36:30 sean-k-mooney but its up to you.
12:36:39 sean-k-mooney they add soem addtion coverage but not a lot
12:36:44 chandankumar Ok I will fine that patch and move it to api test
12:37:15 sean-k-mooney have you tested those with the pci-sim?
12:37:23 chandankumar I tested it with pci-sim
12:37:59 sean-k-mooney ack so ya if you move them to api i think that woudl be fien. we could keep them in secario to as you said its somewhere in between
12:38:01 chandankumar sorry let me double check
12:38:31 sean-k-mooney i think it shoudl work as long as you set the correct adreses
12:38:39 chandankumar tested here: https://review.opendev.org/c/openstack/cyborg/+/977083/51/.zuul.yaml
12:38:46 chandankumar nvmevirt patch
12:39:19 chandankumar https://123b844dd0bc940cd071-d2975a8f065f970ca29e743c021b8a36.ssl.cf1.rackcdn.com/openstack/5f698c9d53e44e52bbd22509f59b8b25/testr_results.html
12:39:30 chandankumar test_pci_device_discovery[id-b3d5e7f9-1a2c-4e6f-8b0d-2c4e6f8b0d2e]
12:39:33 sean-k-mooney whats the plan for that by the way
12:39:52 sean-k-mooney are we goign to abandon the nvmevirt patches in favor of your new nvme pci-sim series
12:40:05 chandankumar yes correct
12:40:10 chandankumar drop nvmevirt patches
12:40:41 chandankumar since we can achieve almost everyhting in first party CI
12:41:29 chandankumar sean-k-mooney: I am not sure you saw this message https://review.opendev.org/c/openstack/cyborg/+/995079/6#message-fd9b5227f7e11e2defeda4a79e4b1b19d0f3c0b2
12:41:49 chandankumar we can see nvme block in lsblk output in guest vm but we need a ubuntu image
12:42:17 sean-k-mooney well i guess it just need an image with the kernel nvme driver
12:42:26 chandankumar yes
12:42:27 sean-k-mooney i might add that to the list of cirrors changes i have in flight
12:42:41 chandankumar cool, that would be awesome
12:42:52 sean-k-mooney we coudl use lspci without the driver by the way
12:43:07 sean-k-mooney as an inital test
12:43:16 sean-k-mooney we just wont be able to run the lsblk part
12:43:18 chandankumar yup, in cirros we can see nvme in lspci output

Earlier   Later