| opendevreview | chandan kumar proposed openstack/cyborg master: Add NVMeVirt emulated NVMe device support for devstack https://review.opendev.org/c/openstack/cyborg/+/977083 | 05:54 |
|---|---|---|
| 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 |
| 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 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Added ansible-lint and bashate pre-commit hook https://review.opendev.org/c/openstack/cyborg/+/979788 | 07:12 |
| opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | 10:12 |
| opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | 10:52 |
| 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:30 |
| 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 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Added ansible-lint and bashate pre-commit hook https://review.opendev.org/c/openstack/cyborg/+/979788 | 11:32 |
| opendevreview | chandan kumar proposed openstack/cyborg master: [WIP] Add cyborg greande job https://review.opendev.org/c/openstack/cyborg/+/979802 | 11:51 |
| sean-k-mooney | o/ | 13:04 |
| sean-k-mooney | actully the irc call is not for an hour | 13:05 |
| sean-k-mooney | we moved our internal meeeting an hour early but the upstream irc call is at its normal time | 13:05 |
| sean-k-mooney | so we normally dont have a break in between but we do today | 13:05 |
| sean-k-mooney | rlandy: ^ | 13:05 |
| rlandy | yes - correct | 13:07 |
| skovili | ack | 13:09 |
| chandankumar | Cyborg meeting starts in 27 mins, Please add your agenda here: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | 13:33 |
| 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 | 13:59 |
| chandankumar | Hello, who all are around for cyborg meeting? | 14:01 |
| rlandy | o/ | 14:02 |
| jgilaber | o/ | 14:02 |
| chandankumar | #startmeeting cyborg | 14:02 |
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'cyborg' | 14:02 |
| bogdando[m] | o/ | 14:02 |
| chandankumar | #courtesy ping sean-k-mooney amoralej jgilaber | 14:03 |
| chandankumar | feel free to add topics to the agenda | 14:03 |
| amoralej | o/ | 14:03 |
| chandankumar | #link https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | 14:04 |
| sean-k-mooney | hi o/ | 14:05 |
| chandankumar | starting with the first topic | 14:05 |
| chandankumar | #topic testing nvme pci device tempest tests in Upstream CI | 14:05 |
| 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:06 |
| 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 |
| 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:07 |
| 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 |
| chandankumar | Later on I realized that I need to enable IOMMU on the host to make sure pci passthroug works | 14:08 |
| sean-k-mooney | althernitivly i think there are 2 other options | 14:08 |
| sean-k-mooney | 1 enable the softiommu on the kernle command line | 14:08 |
| sean-k-mooney | 2 set teh mdoule option for vfio-pci to allow "unsafemode" | 14:08 |
| sean-k-mooney | modprobe vfio enable_unsafe_noiommu_mode=1. | 14:09 |
| chandankumar | thanks sean-k-mooney Do we have any job upstream using any of these options? | 14:09 |
| sean-k-mooney | no | 14:09 |
| chandankumar | Do we need to change the nodeset label also? or current one will work fine | 14:09 |
| sean-k-mooney | without those we could still do some limited testing | 14:09 |
| sean-k-mooney | just not passhtough to a vm | 14:09 |
| sean-k-mooney | i..e we could bind an arq and unbindi it formn tempet | 14:10 |
| 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 |
| sean-k-mooney | it would be preferable to test it via nova howver | 14:10 |
| sean-k-mooney | with standard tempest | 14:10 |
| chandankumar | bind and unbind arg sounds good | 14:12 |
| 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 |
| chandankumar | Thank you! I will update it in next meeting how it goes. | 14:13 |
| bogdando[m] | a naive question, can't we reboot zuul node in CI? that breaks devstack, right? | 14:14 |
| sean-k-mooney | yes and no | 14:14 |
| sean-k-mooney | reboots dont break devstack | 14:14 |
| sean-k-mooney | and we can remboot the vms | 14:14 |
| sean-k-mooney | we woudl do that before installing devstack however | 14:14 |
| sean-k-mooney | if the kernel is compiled with supprot for it | 14:16 |
| sean-k-mooney | echo 1 | sudo tee /sys/module/vfio/parameters/enable_unsafe_noiommu_mode | 14:16 |
| bogdando[m] | so worth exploring? | 14:16 |
| sean-k-mooney | is the simples option in the exsitng pre playbook | 14:16 |
| 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 |
| sean-k-mooney | if not we can revisit how we approch this | 14:17 |
| sean-k-mooney | liek addign a new image to the ci and nodepool lable with the viommu enabled | 14:18 |
| sean-k-mooney | we dont wnat to do that by defualt as there is a non trivial performace hti to that | 14:18 |
| chandankumar | I will check with my existing nvmevirt preplaybook | 14:18 |
| chandankumar | yes, we can revisit this topic in next meeting based on testing how it goes. | 14:19 |
| chandankumar | Any more questions on this topic? | 14:20 |
| sean-k-mooney | ubuntu@openclaw:~$ grep CONFIG_VFIO_NOIOMMU /boot/config-$(uname -r) | 14:20 |
| sean-k-mooney | CONFIG_VFIO_NOIOMMU=y | 14:20 |
| sean-k-mooney | so ubuntu 24.04 appears to have it enabeld at least if we install the mainline kernel | 14:20 |
| bogdando[m] | \o/ | 14:20 |
| sean-k-mooney | but now lets see what your ci run shows | 14: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:21 |
| chandankumar | yup, I will verify it locally with mainline kernel | 14:22 |
| chandankumar | good point! | 14:22 |
| chandankumar | moving to next topic | 14:23 |
| chandankumar | #topic Reviews | 14:23 |
| chandankumar | there is one review | 14:23 |
| chandankumar | #link 979805: Rename devstack CYBORG_CONF_FILE to CYBORG_CONF for consistency | https://review.opendev.org/c/openstack/cyborg/+/979805 | 14:23 |
| 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 |
| sean-k-mooney | hum i think its currently CYBORG_CONF_FILE to disambiguate with CYBORG_CONF_DIR | 14:24 |
| chandankumar | but in Nova or other services we use <SERVICE>_CONF, so I thought about changing it to nova format | 14:24 |
| sean-k-mooney | well nova has more then one conf it uses with mutlipe variables | 14:25 |
| sean-k-mooney | i.e. NOVA_CPU_CONF | 14:25 |
| sean-k-mooney | i think | 14:25 |
| sean-k-mooney | we could make this change im just not sure it by us much | 14:26 |
| sean-k-mooney | if we do this i woudl make it fallback to the old name | 14:27 |
| sean-k-mooney | so CYBORG_CONF=${CYBORG_CONF:-${CYBORG_CONF_FILE}} | 14:28 |
| chandankumar | ok make sense, It will not break existing user | 14:28 |
| chandankumar | I will update the review after this call. | 14:28 |
| sean-k-mooney | ack | 14:28 |
| chandankumar | since there is no more reviews moving to next topic | 14:29 |
| chandankumar | #topic Bugs | 14:29 |
| chandankumar | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1954888 Docs: state how to use bound accelerators | 14:29 |
| chandankumar | we discussed this bug in the last meeting | 14:29 |
| sean-k-mooney | ya so im incliend to ether use this to track descirbing how to create the nova flavor and device profile | 14:30 |
| bogdando[m] | looks like it should stay deferred unless standalone cyborg gets on the radar | 14:30 |
| sean-k-mooney | and that workflow | 14:30 |
| sean-k-mooney | or using it to track a contibutor doc or closing as wont fix | 14:31 |
| sean-k-mooney | right standalon cyborg is not really a thing today so we shoudl nto have end user docs for that | 14:31 |
| sean-k-mooney | so either we have a conceptal do for devleoper/contibutors | 14:31 |
| sean-k-mooney | or we docuemnt the nova flow for end admins ro end users | 14:32 |
| sean-k-mooney | but i dont think we shoudl spend time on stand alone | 14:32 |
| sean-k-mooney | we could close this and create 2 seperate bugs for those other two docs gaps | 14:32 |
| jgilaber | +1 I think there is value in documenting the standard workflow using nova, but I would not go further than that for now | 14:32 |
| sean-k-mooney | i.e. 1 for admin doc for configuing and testign the standard wrokflow and 2 a contibutor doc to serve as a technial deep dive/refence of howt he services interact? | 14:33 |
| chandankumar | Do we have a reference for nova flow doc link which we can add as a reference in the bug? | 14:34 |
| sean-k-mooney | i dont think this really exist out side fo the spec today | 14:35 |
| sean-k-mooney | i dont recall htere being a nova doc for ti but there may be | 14:35 |
| chandankumar | ok | 14:36 |
| sean-k-mooney | i will file 2 seperate bugs and close this when done | 14:36 |
| chandankumar | thanks sean-k-mooney! | 14:37 |
| sean-k-mooney | i can likely create a draft for the docs in the next few weeks if no one else does | 14:37 |
| chandankumar | ok | 14:37 |
| chandankumar | any more question on this bug. | 14:38 |
| chandankumar | moving to next one | 14:38 |
| chandankumar | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1955285 arq delete no valid | 14:38 |
| sean-k-mooney | that is just invliad | 14:39 |
| sean-k-mooney | openstack accelerator arq delete c48f5ac0-0bff-44c0-9106-1850b355c5ca woudl be deliting arq c48f5ac0-0bff-44c0-9106-1850b355c5ca | 14:39 |
| sean-k-mooney | not the arq for instance c48f5ac0-0bff-44c0-9106-1850b355c5ca | 14:39 |
| sean-k-mooney | we might have a docs bug or a help text bug if it imples its the vm uuid | 14:40 |
| bogdando[m] | how reasonable is having this in CLI assuming only service user should be allowed to do that low level things for/via nova? | 14:41 |
| sean-k-mooney | but that is not what "openstack accelerator arq delete" means per the openstack client naming conventions | 14:41 |
| sean-k-mooney | bogdando[m]: it some thing we will need to enfoce going forard | 14:41 |
| jgilaber | there is an api call to delete arqs using the instance uuid Ie1894311b0e384ab52b1b3dfe0eb50618eef6c9f | 14:41 |
| jgilaber | sorry pasted wrong | 14:41 |
| jgilaber | https://docs.openstack.org/api-ref/accelerator/#delete-accelerator-requests-by-instance-uuid | 14:42 |
| sean-k-mooney | its only vaild for a user to delete an arq if its not asssocated with an instance | 14:42 |
| sean-k-mooney | ack let me review that seperatly | 14:42 |
| sean-k-mooney | this clie shoudl be callign the other one | 14:42 |
| sean-k-mooney | /v2/accelerator_requests?arqs={accelerator_request_uuid} not /v2/accelerator_requests?instance={instance_uuid} | 14:43 |
| sean-k-mooney | both exist and the nameing convetion of osc woudl requrie we call the first path | 14:43 |
| sean-k-mooney | to call the second we woudl need --instance or similar | 14:43 |
| sean-k-mooney | but the secodn is invlid from my persective | 14:43 |
| sean-k-mooney | im saying that as a nova core not a cybrog one | 14:44 |
| bogdando[m] | the original use case seemed to be remove all arqs for a running instance | 14:44 |
| sean-k-mooney | yep that not supproted by nova | 14:45 |
| jgilaber | for what is worth the usage test of openstack accelerator arq delete is clear enough | 14:45 |
| jgilaber | usage: openstack accelerator arq delete [-h] <uuid> [<uuid> ... | 14:45 |
| jgilaber | <uuid> UUID(s) of the accelerator_request(s) to delete. | 14:45 |
| sean-k-mooney | +1 | 14:45 |
| jgilaber | no mentions of instance uuids | 14:45 |
| sean-k-mooney | that confims the expecation in the bug reprot is incorect | 14:45 |
| jgilaber | yep, we can close it | 14:46 |
| bogdando[m] | at least for now | 14:46 |
| sean-k-mooney | i willl likely need to adress /v2/accelerator_requests?instance={instance_uuid} in my srback spec | 14:46 |
| sean-k-mooney | but i suspect wwe will need to remove it in a future microversion | 14:46 |
| sean-k-mooney | ill dig into the history of it | 14:47 |
| bogdando[m] | may be it makes sense in future, as housekeeping things, like defragment vram to allow repartitioning | 14:47 |
| sean-k-mooney | lets not specualte on it now | 14:47 |
| bogdando[m] | ack | 14:48 |
| sean-k-mooney | jgilaber: do you want to leave a comment on the bug or will i | 14:48 |
| sean-k-mooney | i woudl liek to link to the doc sting when closing it | 14:48 |
| chandankumar | https://github.com/openstack/cyborg/commit/1f9a808f1d965063a2744c6ea6ddeb76a5d57061 adds some does around second option | 14:48 |
| jgilaber | I can do it | 14:48 |
| sean-k-mooney | jgilaber: ++ | 14:48 |
| sean-k-mooney | chandankumar nova only supprots 1 ARQ today | 14:49 |
| sean-k-mooney | i think this was added in preperation for cybrog device via neuton ports | 14:50 |
| sean-k-mooney | since that fits the time frame but that never happend in nova or neutron | 14:50 |
| chandankumar | ok | 14:51 |
| sean-k-mooney | shall we move on to the next bug or finish early? | 14:51 |
| chandankumar | yes | 14:51 |
| chandankumar | #link https://bugs.launchpad.net/openstack-cyborg/+bug/1955286 cyborg cmd exec meeting keystone authentication fail | 14:51 |
| sean-k-mooney | that looks like a deployment isseu | 14:53 |
| sean-k-mooney | they are gettign a 401 form the micoversion endpoint | 14:53 |
| sean-k-mooney | which shoudl not requrie auth | 14:53 |
| sean-k-mooney | so without more info mation i think we close this as incompl.ete | 14:54 |
| sean-k-mooney | bases on the path this looks liek a devstack deployment not production | 14:54 |
| sean-k-mooney | /opt/stack/data/CA/int-ca/ca-chain.pem | 14:54 |
| sean-k-mooney | /opt/stack/data is where devstack store service state | 14:54 |
| chandankumar | I will take care of this one | 14:55 |
| sean-k-mooney | ack | 14:55 |
| chandankumar | there is one more bug, but I will move it to next meeting | 14:55 |
| sean-k-mooney | we could see if we have the same bevhior in our local devstack but i dont think this si a real issue | 14:56 |
| sean-k-mooney | ack | 14:56 |
| chandankumar | moving to last topic | 14:56 |
| chandankumar | #topic volunteer for next meeting | 14:56 |
| rlandy | I'll do it since I skipped my last round | 14:56 |
| chandankumar | thanks rlandy ! | 14:56 |
| chandankumar | #topic open discussion | 14:56 |
| chandankumar | Anyone wants to bring any topic before closing the meeting. | 14:57 |
| chandankumar | if not, then let me close the meeting | 14:58 |
| chandankumar | thank you all for joining the meeting! | 14:58 |
| chandankumar | #endmeeting | 14:59 |
| opendevmeet | Meeting ended Tue Mar 10 14:59:02 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-03-10-14.02.html | 14:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-03-10-14.02.txt | 14:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-03-10-14.02.log.html | 14:59 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!