| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2026-05-26 | |||
| 14:24:02 | jgilaber | I got feedback on the companion nova spec that I need to go over | |
| 14:24:48 | sean-k-mooney | yes i disagreed with some of that feedback | |
| 14:24:56 | sean-k-mooney | bu ti have not had time to see if they replied | |
| 14:25:34 | sean-k-mooney | #link https://review.opendev.org/c/openstack/nova-specs/+/967515 | |
| 14:25:39 | sean-k-mooney | that is the nova conterpart | |
| 14:27:54 | sean-k-mooney | i will try and reivwe the cybrog side this week | |
| 14:28:08 | jgilaber | ack thanks | |
| 14:28:53 | jgilaber | that seems to be all | |
| 14:29:02 | jgilaber | we don't have any new bugs | |
| 14:30:01 | sean-k-mooney | we can wrap early htis week fi we have no other items | |
| 14:30:14 | sean-k-mooney | i didnt really prepare antying to dicss for this week so it was light on my end | |
| 14:31:00 | jgilaber | I don't have anything else, so if there are no last minute topics we can end here for today | |
| 14:34:04 | jgilaber | thanks for the discussion, see you next week! | |
| 14:34:07 | jgilaber | #endmeeting | |
| 14:34:07 | opendevmeet | Meeting ended Tue May 26 14:34:07 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:34:07 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-26-14.00.html | |
| 14:34:07 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-26-14.00.txt | |
| 14:34:07 | opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-05-26-14.00.log.html | |
| 14:40:29 | opendevreview | Joan Gilabert proposed openstack/cyborg master: Move cyborg-tempest job definitions to cyborg https://review.opendev.org/c/openstack/cyborg/+/987620 | |
| 14:45:23 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Move job definitions to cyborg repo https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/987621 | |
| 14:52:18 | opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Move job definitions to cyborg repo https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/987621 | |
| 15:43:43 | jgilaber | sean-k-mooney ^^ I've updated the patches for the tempest job definition as discussed in the irc meeting | |
| 15:43:50 | jgilaber | it seems to work well | |
| 16:19:53 | sean-k-mooney | cool ill look shortly | |
| 20:21:37 | opendevreview | Merged openstack/cyborg master: Move cyborg-tempest job definitions to cyborg https://review.opendev.org/c/openstack/cyborg/+/987620 | |
| 20:22:59 | opendevreview | sean mooney proposed openstack/cyborg master: [WIP] devstack: add pci-sim fake SR-IOV PCI passthrough test fixture https://review.opendev.org/c/openstack/cyborg/+/989470 | |
| #openstack-cyborg - 2026-05-27 | |||
| 14:04:02 | opendevreview | Joan Gilabert proposed openstack/cyborg master: Extract shared Zuul config into abstract base job https://review.opendev.org/c/openstack/cyborg/+/990296 | |
| 15:30:44 | sean-k-mooney | jgilaber: i actlly have ^ in my change as well | |
| 16:02:41 | jgilaber | I did not see that | |
| 16:02:58 | jgilaber | I can abandon my patch if you prefer to do the change in your patch | |
| 21:00:13 | opendevreview | sean mooney proposed openstack/cyborg master: [WIP] devstack: add pci-sim fake SR-IOV PCI passthrough test fixture https://review.opendev.org/c/openstack/cyborg/+/989470 | |
| #openstack-cyborg - 2026-05-29 | |||
| 10:45:23 | opendevreview | Joan Gilabert proposed openstack/cyborg master: Extract shared Zuul config into abstract base job https://review.opendev.org/c/openstack/cyborg/+/990296 | |
| 12:01:44 | jgilaber | chandankumar, I just noticed the "cyborg-multinode-tempest" is not running anywhere, is that intentional? | |
| 12:04:03 | chandankumar | jgilaber: good catch, https://github.com/openstack/cyborg/commit/a2bbd5ac278dfd8d57dfd6b4a8f2aaa903a5ec33, i forgot to add it in the zuul pipeline | |
| 12:04:07 | chandankumar | let me propose a patch | |
| 12:07:15 | jgilaber | ack no problem | |
| 12:08:56 | opendevreview | chandan kumar proposed openstack/cyborg master: Add cyborg-multinode-tempest job to check pipeline https://review.opendev.org/c/openstack/cyborg/+/990628 | |
| 16:17:57 | opendevreview | Merged openstack/cyborg master: Extract shared Zuul config into abstract base job https://review.opendev.org/c/openstack/cyborg/+/990296 | |
| #openstack-cyborg - 2026-05-31 | |||
| 15:25:06 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: add pci-sim fake SR-IOV PCI passthrough test fixture https://review.opendev.org/c/openstack/cyborg/+/989470 | |
| #openstack-cyborg - 2026-06-01 | |||
| 07:24:34 | opendevreview | sean mooney proposed openstack/cyborg master: devstack: add pci-sim fake SR-IOV PCI passthrough test fixture https://review.opendev.org/c/openstack/cyborg/+/989470 | |
| #openstack-cyborg - 2026-06-02 | |||
| 08:36:06 | opendevreview | chandan kumar proposed openstack/cyborg master: Remove broken image signature verification https://review.opendev.org/c/openstack/cyborg/+/991027 | |
| 08:42:52 | opendevreview | chandan kumar proposed openstack/cyborg master: Remove broken image signature verification https://review.opendev.org/c/openstack/cyborg/+/991027 | |
| 13:23:30 | opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add scenario test for FPGA programming with FakeDriver https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/991081 | |
| 14:00:27 | jgilaber | #startmeeting cyborg | |
| 14:00:27 | opendevmeet | Meeting started Tue Jun 2 14:00:27 2026 UTC and is due to finish in 60 minutes. The chair is jgilaber. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 14:00:27 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 14:00:27 | opendevmeet | The meeting name has been set to 'cyborg' | |
| 14:00:39 | jgilaber | Hi all! Who is around today? | |
| 14:01:01 | jgilaber | While we gather, feel free to add topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L48 | |
| 14:01:13 | jgilaber | courtesy ping: sean-k-mooney amoralej bogdando rlandy chandankumar | |
| 14:01:22 | chandankumar | o/ | |
| 14:01:30 | sean-k-mooney | o/ | |
| 14:02:20 | jgilaber | let's give folks a minute to join and then we can start | |
| 14:03:23 | jgilaber | ok let's get started | |
| 14:03:30 | jgilaber | we have a topic from chandankumar | |
| 14:03:39 | jgilaber | #topic nvme cleanup stages spec proposal discussion | |
| 14:03:45 | jgilaber | #link nvme cleanup stages spec proposal discussion | |
| 14:03:49 | jgilaber | go ahead chandankumar | |
| 14:04:02 | chandankumar | sure | |
| 14:04:14 | chandankumar | Here is the current spec for nvme secure cleanup https://review.opendev.org/c/openstack/cyborg-specs/+/985349/10/specs/2026.2/approved/generic-nvme-driver-with-secure-cleanup.rst | |
| 14:04:20 | chandankumar | Thank you everyone for reviewing the spec. | |
| 14:04:27 | chandankumar | Below is the cleanup flow for nvme device once user deletes an instance: | |
| 14:04:33 | chandankumar | 1. Instance deletion and unbind process goes seperately. | |
| 14:04:42 | chandankumar | 2. During unbind process, cyborg will disable the nvme device with maintaining status by default and set resevered=total in placement | |
| 14:04:48 | chandankumar | 3. Unbind process will finish and cleanup will run async. | |
| 14:04:53 | chandankumar | 4. if the cleanup finishes successfully, we set reserved=0 in placement and add a new flag cleanup_fail to false and enable the device | |
| 14:04:59 | chandankumar | 5. if cleanup fails, keep the device disabled with maintaing state and set cleanup_fail to true and reserved=total in placement. | |
| 14:05:06 | chandankumar | 6. Operator can list the device with cleanup_failed and run cleanup manually on those devices. | |
| 14:05:17 | chandankumar | Earlier in previous patch iteration, I went with cleaning and cleaning_failed status message but there are too many state. | |
| 14:05:23 | chandankumar | So I sticked with | |
| 14:05:25 | chandankumar | default maintaining state and cleanup_failed flag to keep the flow simpler. | |
| 14:05:36 | chandankumar | Do we want to add a cleaning state to know the exact state? and if cleanup failed, we will stick with maintaining? | |
| 14:05:37 | sean-k-mooney | so im not ok with reusing disabled for cleaning failed | |
| 14:05:51 | chandankumar | How do we want to handle that? | |
| 14:05:52 | sean-k-mooney | that was one of the feedack i gave on irc when you pushed the sepc initally | |
| 14:05:56 | amoralej | o/ | |
| 14:06:21 | chandankumar | sean-k-mooney: sorry I am not getting | |
| 14:06:51 | sean-k-mooney | i want to intoduce a sperate device_state filed which will taransation between avilable -> allcoated -> cleaning and hten to eitehr error or aviabel dependign on if cleanign succeeded | |
| 14:07:34 | chandankumar | ok | |
| 14:07:49 | sean-k-mooney | we also need to set reserved=total durign arq bind not durign unbind | |
| 14:08:18 | sean-k-mooney | so on bind we woudl transiation the device to allcoated and set reserved=total | |
| 14:08:55 | sean-k-mooney | on unbind it moved to cleaning keeping reserved=total | |
| 14:09:14 | chandankumar | above device state sounds good, I was focus earloer on cleanup | |
| 14:09:16 | sean-k-mooney | and ether end in error (reserved=total) or aviable (reseting reserved=0) | |
| 14:10:01 | chandankumar | yes that approach seems better | |
| 14:10:33 | chandankumar | One more thing, Does operator can toggle the device state manually? once clenaup finishes successfully | |
| 14:10:42 | sean-k-mooney | im also wondering if we want to add a /device/<uuid>/clean endpoint ot manually trigger cleaning | |
| 14:10:51 | sean-k-mooney | that would be an admin operator similar to program | |
| 14:11:03 | sean-k-mooney | chandankumar: no | |
| 14:11:07 | sean-k-mooney | the admin cannot | |
| 14:11:22 | sean-k-mooney | if we add clean as a deivce action | |
| 14:11:30 | sean-k-mooney | that is how they would recover it | |
| 14:11:47 | sean-k-mooney | we coudl perhaps consider a way to force it | |
| 14:12:02 | jgilaber | that endpoint would only work for devices that are in error? | |
| 14:12:04 | sean-k-mooney | but this si really internal stant that an admin shoudl not have to change in a normal workflow | |
| 14:12:31 | sean-k-mooney | jgilaber: i would say error or aviableale woudl be ok | |
| 14:12:47 | sean-k-mooney | but a 409 for allcoated/cleaning | |
| 14:13:06 | jgilaber | right, available would be fine, but redundant | |
| 14:13:17 | sean-k-mooney | we coudl dicusssi fi cleanign an allcoated fpga before repogrammign it shoudl eb allowed or not in the spec | |
| 14:14:12 | sean-k-mooney | the other approch here for erroed device woudl eb a cyborg-manage command or simialr for operator to manualy update the state | |