| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-03-06 | |||
| 11:00:30 | bogdando[m] | sean-k-mooney: hi, I wonder, if we should omit SUGGESTION level reviews from ai bot | |
| 12:53:24 | sean-k-mooney | in general no but i do plan to tweeak the tresholding and alther the reporting | |
| 12:54:29 | sean-k-mooney | bogdando[m]: in general i have found the suggetion level in put to be valid but sometime out of scope my general plan is that i will have it report less or have a higher barrires to report for the inline comments and keep all the observation in the html report | |
| 12:56:10 | sean-k-mooney | im alos planning to make it evetually report code-review +1 if there are no warning and above issue and cr+-1 only if it identifies a Crtical level issues, so suggestion would never prevent it leaving a +1 | |
| 12:57:43 | sean-k-mooney | bogdando[m]: the current treshold for reporting was intedt to be 0.8 for inline comment but its actully not filterint them right nwo so its using the 0.6 value for the html report for both | |
| 13:00:36 | sean-k-mooney | bogdando[m]: by the way ill evenutlly have the review job read HACKING.rst for per repo review guidance so we can also inflauce this in the repo if we chosoe too. i just havnt had time to spend on it but i plan ot pend a day on it in the next 2 wekks implementing many of these changes | |
| 13:21:10 | opendevreview | OpenStack Release Bot proposed openstack/python-cyborgclient master: Update master for stable/2026.1 https://review.opendev.org/c/openstack/python-cyborgclient/+/979173 | |
| 14:46:23 | opendevreview | Merged openstack/python-cyborgclient master: Update master for stable/2026.1 https://review.opendev.org/c/openstack/python-cyborgclient/+/979173 | |
| #openstack-cyborg - 2026-03-10 | |||
| 05:54:14 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 06:03:39 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 06:39:16 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 07:12:48 | opendevreview | chandan kumar proposed openstack/cyborg master: Added ansible-lint and bashate pre-commit hook https://review.opendev.org/c/openstack/cyborg/+/979788 | |
| 10:12:11 | opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | |
| 10:52:04 | opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | |
| 11:30:48 | opendevreview | chandan kumar proposed openstack/cyborg master: Rename devstack CYBORG_CONF_FILE to CYBORG_CONF for consistency https://review.opendev.org/c/openstack/cyborg/+/979805 | |
| 11:32:00 | opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 11:32:33 | opendevreview | chandan kumar proposed openstack/cyborg master: Added ansible-lint and bashate pre-commit hook https://review.opendev.org/c/openstack/cyborg/+/979788 | |
| 11:51:15 | opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | |
| 13:04:50 | sean-k-mooney | o/ | |
| 13:05:06 | sean-k-mooney | actully the irc call is not for an hour | |
| 13:05:28 | sean-k-mooney | we moved our internal meeeting an hour early but the upstream irc call is at its normal time | |
| 13:05:39 | sean-k-mooney | so we normally dont have a break in between but we do today | |
| 13:05:46 | sean-k-mooney | rlandy: ^ | |
| 13:07:41 | rlandy | yes - correct | |
| 13:09:47 | skovili | ack | |
| 13:33:44 | chandankumar | Cyborg meeting starts in 27 mins, Please add your agenda here: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | |
| 13:59:22 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: [WIP] Add cyborg pci nvme test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/979847 | |
| 14:01:55 | chandankumar | Hello, who all are around for cyborg meeting? | |
| 14:02:02 | rlandy | o/ | |
| 14:02:19 | jgilaber | o/ | |
| 14:02:23 | chandankumar | #startmeeting cyborg | |
| 14:02:23 | opendevmeet | Meeting started Tue Mar 10 14:02:23 2026 UTC and is due to finish in 60 minutes. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 14:02:23 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:02:23 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:02:29 | bogdando[m] | o/ | |
| 14:03:09 | chandankumar | #courtesy ping sean-k-mooney amoralej jgilaber | |
| 14:03:31 | chandankumar | feel free to add topics to the agenda | |
| 14:03:57 | amoralej | o/ | |
| 14:04:20 | chandankumar | #link https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | |
| 14:05:10 | sean-k-mooney | hi o/ | |
| 14:05:27 | chandankumar | starting with the first topic | |
| 14:05:50 | chandankumar | #topic testing nvme pci device tempest tests in Upstream CI | |
| 14:06:37 | chandankumar | I was working on adding nvmevirt devstack plugin for cyborg to test nvme pci device in CI https://review.opendev.org/c/openstack/cyborg/+/977083 | |
| 14:07:03 | chandankumar | I was working on adding the tempest scenario test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/979847 where we can expose the nvme pci device to the vm | |
| 14:07:28 | chandankumar | But while creating the vm, On nove compute side, I am hitting unsupported configuration: host doesn't support passthrough of host PCI devices | |
| 14:08:02 | sean-k-mooney | ya so nova can virutalize a iommu but you need to enable it in the image uploaded to the provider | |
| 14:08:03 | chandankumar | Later on I realized that I need to enable IOMMU on the host to make sure pci passthroug works | |
| 14:08:13 | sean-k-mooney | althernitivly i think there are 2 other options | |
| 14:08:26 | sean-k-mooney | 1 enable the softiommu on the kernle command line | |
| 14:08:41 | sean-k-mooney | 2 set teh mdoule option for vfio-pci to allow "unsafemode" | |
| 14:09:19 | sean-k-mooney | modprobe vfio enable_unsafe_noiommu_mode=1. | |
| 14:09:24 | chandankumar | thanks sean-k-mooney Do we have any job upstream using any of these options? | |
| 14:09:32 | sean-k-mooney | no | |
| 14:09:40 | chandankumar | Do we need to change the nodeset label also? or current one will work fine | |
| 14:09:46 | sean-k-mooney | without those we could still do some limited testing | |
| 14:09:50 | sean-k-mooney | just not passhtough to a vm | |
| 14:10:05 | sean-k-mooney | i..e we could bind an arq and unbindi it formn tempet | |
| 14:10:37 | sean-k-mooney | in a whitebox style plugin we coudl aslo ssh in write some data to the device and hten asert that its cleaned when unbound | |
| 14:10:45 | sean-k-mooney | it would be preferable to test it via nova howver | |
| 14:10:55 | sean-k-mooney | with standard tempest | |
| 14:12:29 | chandankumar | bind and unbind arg sounds good | |
| 14:13:28 | chandankumar | I will break the existing test into multiple tests to have bind/ubind as well as passthrough to vim via standard tempest tests | |
| 14:13:53 | chandankumar | Thank you! I will update it in next meeting how it goes. | |
| 14:14:10 | bogdando[m] | a naive question, can't we reboot zuul node in CI? that breaks devstack, right? | |
| 14:14:26 | sean-k-mooney | yes and no | |
| 14:14:31 | sean-k-mooney | reboots dont break devstack | |
| 14:14:36 | sean-k-mooney | and we can remboot the vms | |
| 14:14:43 | sean-k-mooney | we woudl do that before installing devstack however | |
| 14:16:07 | sean-k-mooney | if the kernel is compiled with supprot for it | |
| 14:16:09 | sean-k-mooney | echo 1 | sudo tee /sys/module/vfio/parameters/enable_unsafe_noiommu_mode | |
| 14:16:09 | bogdando[m] | so worth exploring? | |
| 14:16:21 | sean-k-mooney | is the simples option in the exsitng pre playbook | |
| 14:17:43 | sean-k-mooney | without that we can still do some testign via whtiebox we just can boot a vm. lets see if ubuntu 24.04 or debian 13 ship with that kernel module partmer | |
| 14:17:52 | sean-k-mooney | if not we can revisit how we approch this | |
| 14:18:06 | sean-k-mooney | liek addign a new image to the ci and nodepool lable with the viommu enabled | |
| 14:18:27 | sean-k-mooney | we dont wnat to do that by defualt as there is a non trivial performace hti to that | |
| 14:18:31 | chandankumar | I will check with my existing nvmevirt preplaybook | |
| 14:19:52 | chandankumar | yes, we can revisit this topic in next meeting based on testing how it goes. | |
| 14:20:06 | chandankumar | Any more questions on this topic? | |
| 14:20:23 | sean-k-mooney | ubuntu@openclaw:~$ grep CONFIG_VFIO_NOIOMMU /boot/config-$(uname -r) | |
| 14:20:25 | sean-k-mooney | CONFIG_VFIO_NOIOMMU=y | |
| 14:20:42 | sean-k-mooney | so ubuntu 24.04 appears to have it enabeld at least if we install the mainline kernel | |
| 14:20:50 | bogdando[m] | \o/ | |
| 14:20:51 | sean-k-mooney | but now lets see what your ci run shows | |
| 14:21:20 | sean-k-mooney | if we need to install the mainline (upstream kernel) we can do that but we need to verify it in a vanilla 24.04 | |
| 14:22:13 | chandankumar | yup, I will verify it locally with mainline kernel | |
| 14:22:22 | chandankumar | good point! | |
| 14:23:00 | chandankumar | moving to next topic | |
| 14:23:09 | chandankumar | #topic Reviews | |
| 14:23:25 | chandankumar | there is one review | |
| 14:23:29 | chandankumar | #link 979805: Rename devstack CYBORG_CONF_FILE to CYBORG_CONF for consistency | https://review.opendev.org/c/openstack/cyborg/+/979805 | |
| 14:24:18 | chandankumar | While I was working on nvmevirt devstack patch, I found that In cyborg we use CYBORG_CONF_FILE var for cyborg conf in devstack | |
| 14:24:41 | sean-k-mooney | hum i think its currently CYBORG_CONF_FILE to disambiguate with CYBORG_CONF_DIR | |
| 14:24:53 | chandankumar | but in Nova or other services we use <SERVICE>_CONF, so I thought about changing it to nova format | |
| 14:25:37 | sean-k-mooney | well nova has more then one conf it uses with mutlipe variables | |
| 14:25:48 | sean-k-mooney | i.e. NOVA_CPU_CONF | |
| 14:25:50 | sean-k-mooney | i think | |
| 14:26:14 | sean-k-mooney | we could make this change im just not sure it by us much | |
| 14:27:10 | sean-k-mooney | if we do this i woudl make it fallback to the old name | |
| 14:28:08 | sean-k-mooney | so CYBORG_CONF=${CYBORG_CONF:-${CYBORG_CONF_FILE}} | |