| opendevreview | chandan kumar proposed openstack/cyborg-tempest-plugin master: Add Cyborg-to-Placement integration scenario test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | 04:04 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/cyborg master: Allow non-CUSTOM traits in device profile groups https://review.opendev.org/c/openstack/cyborg/+/997945 | 04:04 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Move create() from FPGADriver to GenericDriver https://review.opendev.org/c/openstack/cyborg/+/998004 | 04:43 |
| 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 | 04:48 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Verify guest reboot started in PCI lifecycle test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997751 | 07:53 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 07:53 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Add Cyborg MDEV driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997352 | 07:53 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 08:50 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Add Cyborg MDEV driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997352 | 08:50 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Drop postgres migration unit tests https://review.opendev.org/c/openstack/cyborg/+/998101 | 09:45 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Add bashate to pre-commit and fix violations https://review.opendev.org/c/openstack/cyborg/+/984027 | 10:03 |
| opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add new-defaults config and multi-persona client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992329 | 10:24 |
| opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Migrate API tests to least-privilege client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330 | 10:24 |
| opendevreview | sean mooney proposed openstack/cyborg-tempest-plugin master: Add focused SRBAC policy tests for new defaults https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992331 | 10:24 |
| opendevreview | sean mooney proposed openstack/cyborg master: Use request context as policy target https://review.opendev.org/c/openstack/cyborg/+/992722 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Add SRBAC base rules and enforce_new_defaults override https://review.opendev.org/c/openstack/cyborg/+/992292 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate ARQ policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992293 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate device policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992294 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate deployable policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992295 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Migrate attribute policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992296 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Add SRBAC policy concepts documentation https://review.opendev.org/c/openstack/cyborg/+/992297 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Add cyborg-tempest-srbac Zuul job https://review.opendev.org/c/openstack/cyborg/+/992332 | 10:34 |
| opendevreview | sean mooney proposed openstack/cyborg master: Remove no-op deprecated policy bridges https://review.opendev.org/c/openstack/cyborg/+/995462 | 10:34 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Add bashate to pre-commit and fix violations https://review.opendev.org/c/openstack/cyborg/+/984027 | 12:11 |
| opendevreview | Merged openstack/cyborg master: Use request context as policy target https://review.opendev.org/c/openstack/cyborg/+/992722 | 12:45 |
| opendevreview | Merged openstack/cyborg master: Add SRBAC base rules and enforce_new_defaults override https://review.opendev.org/c/openstack/cyborg/+/992292 | 12:49 |
| opendevreview | Merged openstack/cyborg master: Migrate ARQ policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992293 | 12:49 |
| opendevreview | Merged openstack/cyborg master: Migrate device policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992294 | 12:49 |
| opendevreview | Merged openstack/cyborg master: Migrate deployable policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992295 | 12:55 |
| opendevreview | Merged openstack/cyborg master: Migrate attribute policies to DocumentedRuleDefault https://review.opendev.org/c/openstack/cyborg/+/992296 | 12:55 |
| opendevreview | Merged openstack/cyborg master: Add SRBAC policy concepts documentation https://review.opendev.org/c/openstack/cyborg/+/992297 | 12:55 |
| opendevreview | chandan kumar proposed openstack/cyborg master: Add bashate to pre-commit and fix violations https://review.opendev.org/c/openstack/cyborg/+/984027 | 13:18 |
| jgilaber | Hi all! IRC meeting starting in 10 minutes, add your topics to the agenda https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L47 | 13:51 |
| jgilaber | #startmeeting cyborg | 14:00 |
| opendevmeet | Meeting started Tue Jul 21 14:00:57 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cyborg' | 14:00 |
| jgilaber | Hi! Who is around today? | 14:01 |
| chandankumar | o/ | 14:01 |
| jgilaber | courtesy ping: sean-k-mooney amoralej bogdando rlandy skovili | 14:01 |
| amoralej | o/ | 14:01 |
| melwitt | I think o/ | 14:01 |
| jgilaber | while we wait for more people, feel free to add topics to the (currently empty) agenda | 14:01 |
| jgilaber | https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L47 | 14:02 |
| jgilaber | let's start | 14:02 |
| jgilaber | jumping directly to reviews since the agenda is empty | 14:03 |
| jgilaber | #topic Reviews | 14:03 |
| rlandy | o/ | 14:03 |
| jgilaber | first reviews are from chandankumar | 14:03 |
| jgilaber | #link https://review.opendev.org/q/topic:%22fpga_program%22 | 14:03 |
| chandankumar | yes | 14:03 |
| jgilaber | anything you want to highlight? | 14:03 |
| chandankumar | I was working on improving the tempest test coverage and fixes around fgpa program interface | 14:04 |
| chandankumar | I have proposed couple of reviews for rename and refactoring | 14:04 |
| chandankumar | Use GenericDriver.update() as the FPGA programming interface - https://review.opendev.org/c/openstack/cyborg/+/991598 | 14:04 |
| chandankumar | https://review.opendev.org/c/openstack/cyborg/+/998004: Move create() from FPGADriver to GenericDriver | 14:04 |
| chandankumar | both reviews are tested with https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/991081 | 14:05 |
| chandankumar | scenario test for FPGA programming with FakeDriver | 14:05 |
| chandankumar | It boots a server with an | 14:05 |
| chandankumar | FPGA device profile backed by FakeDriver, locates the fake deployable, and calls the program API to verify that it successed. | 14:05 |
| chandankumar | Feel free to have a look at these reviews. | 14:06 |
| jgilaber | ack, I had reviewed the first one in the chain, but not the other two, I'll add them to my review list | 14:06 |
| chandankumar | thank you jgilaber! | 14:06 |
| chandankumar | Next one is also mine | 14:06 |
| chandankumar | #link https://review.opendev.org/q/topic:%22bug/2157586%22 | 14:06 |
| chandankumar | few days back I opened https://bugs.launchpad.net/openstack-cyborg/+bug/2157586 | 14:07 |
| chandankumar | Cyborg Device Profile API rejects OWNER_CYBORG trait because it rejects non custom traits | 14:07 |
| chandankumar | https://review.opendev.org/c/openstack/cyborg/+/997945 removes the conditional for non custom traits and allow owner_cyborg traits to b used with device profole | 14:08 |
| jgilaber | that bug only affects traits but not resource classes right? | 14:08 |
| chandankumar | I have also added scenario test to validate cyborg placement integration https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992210 | 14:08 |
| chandankumar | Based on each device profile, it troes to find out triats based on vendor and product id | 14:09 |
| chandankumar | yes only traits | 14:10 |
| jgilaber | ack, this seems like an important bug to fix, thanks for working on it | 14:10 |
| chandankumar | that's it from my side on reviews | 14:11 |
| jgilaber | thanks chandankumar | 14:11 |
| jgilaber | any other question on these reviews? | 14:12 |
| jgilaber | moving on | 14:12 |
| jgilaber | I have a couple of patches to hightligth | 14:12 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997751 | 14:12 |
| jgilaber | This first one tries to improve the problems with the guest reboot test | 14:13 |
| chandankumar | yes, thank you for working on this one. | 14:14 |
| jgilaber | and the next one in the chain refactors the lifecycle tests | 14:15 |
| jgilaber | so that they can be reused for multiple drivers | 14:15 |
| chandankumar | Do we want to do more recheck on this one to make sure reboot test is not failing? | 14:15 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 14:15 |
| jgilaber | yes, I want to recheck it some more | 14:15 |
| jgilaber | and check if the failures in https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 are because the first patch does not solve the problem completely | 14:15 |
| jgilaber | or that I'm reintroducing the problem in the refactor somehow | 14:16 |
| jgilaber | I'll double check that after the meeting | 14:16 |
| jgilaber | any other question/comment on this? | 14:16 |
| chandankumar | on second patch reboot is still failing, it is timing out when instance is coming back after reboot | 14:16 |
| chandankumar | in temepst-py3 job. | 14:16 |
| jgilaber | yes, I'm not sure if it's the exact same failure or is a new problem from the refactor | 14:17 |
| chandankumar | I have not taken a look at the refactored code, will take a look tomorrow | 14:17 |
| chandankumar | here py3 job is faoling while ipv6 is passing | 14:17 |
| jgilaber | ack, thanks chandankumar | 14:18 |
| jgilaber | yes, the error is not very consistent | 14:18 |
| chandankumar | yes | 14:18 |
| jgilaber | anything else on this? | 14:18 |
| jgilaber | if not, I have one last review | 14:18 |
| jgilaber | #link https://review.opendev.org/c/openstack/cyborg/+/997979 | 14:19 |
| jgilaber | I noticed that in cyborg we had a test file to test database migrations with classes that were supposed to run them with both mariadb and postgres | 14:19 |
| jgilaber | but both were skipped since the dbs were not setup | 14:19 |
| jgilaber | and the checks for the migrations were not being called | 14:20 |
| jgilaber | so I added test requirements for mariadb in this patch to at least run them there | 14:20 |
| jgilaber | the proposal is to add mariadb for now and enable also postgres in the future if we want to | 14:20 |
| jgilaber | any questions on this? | 14:21 |
| chandankumar | I have added 998101: Drop postgres migration unit tests | https://review.opendev.org/c/openstack/cyborg/+/998101to drop prostes test | 14:21 |
| chandankumar | Based on code search https://codesearch.openstack.org/?q=TestMigrationsPostgreSQL&i=nope&literal=nope&files=&excludeFiles=&repos= I did not find except cyborg and ironic having postgres coverage | 14:22 |
| chandankumar | +1 for enabling it | 14:23 |
| jgilaber | thanks, lgtm | 14:24 |
| jgilaber | we did not enable them in watcher either | 14:24 |
| chandankumar | ok | 14:25 |
| jgilaber | anything else on this topic? | 14:25 |
| jgilaber | moving to bugs | 14:26 |
| jgilaber | #topic Bugs | 14:26 |
| jgilaber | #link https://bugs.launchpad.net/openstack-cyborg/+bug/2161354 | 14:26 |
| jgilaber | this seems like a valid bug and reading the report | 14:27 |
| jgilaber | adding support for this in the pci driver might be better in the long term | 14:27 |
| jgilaber | melwitt, do you know what would we need to change to support those devices? | 14:28 |
| jgilaber | in the meantime, I think we can triage this as medium? | 14:30 |
| chandankumar | yes medium sounds good | 14:31 |
| jgilaber | ack, done | 14:32 |
| jgilaber | that's the last thing we had in the agenda, any last minute topic? | 14:32 |
| jgilaber | moving to the last topic then | 14:33 |
| jgilaber | #topic Volunteers to chair next meeting | 14:33 |
| jgilaber | any volunterr? | 14:33 |
| chandankumar | I can volunteer for next meeting | 14:33 |
| jgilaber | thanks chandankumar ++ | 14:34 |
| jgilaber | that's all for today, thanks all! | 14:34 |
| jgilaber | #endmeeting | 14:34 |
| opendevmeet | Meeting ended Tue Jul 21 14:34:48 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:34 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-21-14.00.html | 14:34 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-21-14.00.txt | 14:34 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-21-14.00.log.html | 14:34 |
| chandankumar | thank you jgilaber ! | 14:35 |
| melwitt | jgilaber_: hi sorry I had to step away in the middle of the meeting. on https://bugs.launchpad.net/openstack-cyborg/+bug/2161354, for the shorter term it would be a small change to leverage the same exclude mechanism in the NIC driver (duplication), | 15:06 |
| melwitt | for the long term moving the NIC driver into the PCI driver would be a larger change with the main thing being we would need to move/add SRIOV VF enumeration into the PCI driver. along with device type awareness | 15:07 |
| *** jgilaber_ is now known as jgilaber | 15:19 | |
| jgilaber | melwitt, no worries I was just curious | 15:19 |
| jgilaber | the sriov is currently in progress, what kind of device type awareness would we need, do we need to do some setup on the device? | 15:20 |
| melwitt | just to be able to differentiate between say a gpu vs a nic | 15:22 |
| melwitt | so check the type to know whether to do the sriov VF enumeration for example | 15:23 |
| melwitt | (sorry I am not so knowledgeable in the drivers yet, maybe that is already there) | 15:24 |
| jgilaber | ah ok | 15:42 |
| jgilaber | no that is missing | 15:42 |
| jgilaber | I guess it was never implemented as it was prefered to have vendor specific drivers to provide such details | 15:42 |
| melwitt | yes seems so. it could be debated whether to consolidate into the generic pci driver | 15:44 |
| melwitt | for now for the bug, I think to do the simple duplication thing since I think more discussion is needed about the idea of consolidating | 15:45 |
| jgilaber | +1 to that | 15:50 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Add new-defaults config and multi-persona client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992329 | 16:20 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Migrate API tests to least-privilege client aliases https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992330 | 16:46 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Add focused SRBAC policy tests for new defaults https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/992331 | 16:46 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Verify guest reboot started in PCI lifecycle test https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997751 | 16:46 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 17:02 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Add Cyborg MDEV driver lifecycle scenario tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997352 | 17:02 |
| opendevreview | Merged openstack/cyborg master: Add bashate to pre-commit and fix violations https://review.opendev.org/c/openstack/cyborg/+/984027 | 20:44 |
| opendevreview | Merged openstack/cyborg master: Add cyborg-tempest-srbac Zuul job https://review.opendev.org/c/openstack/cyborg/+/992332 | 22:34 |
| opendevreview | Merged openstack/cyborg master: Remove no-op deprecated policy bridges https://review.opendev.org/c/openstack/cyborg/+/995462 | 22:35 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!