| opendevreview | chandan kumar proposed openstack/cyborg master: Allow non-CUSTOM traits in device profile groups https://review.opendev.org/c/openstack/cyborg/+/997945 | 07:16 |
|---|---|---|
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 10:31 |
| 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 | 10:31 |
| opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | 12:23 |
| opendevreview | Gihong Lee proposed openstack/cyborg master: Migrate accelerator drivers to shared lspci helpers https://review.opendev.org/c/openstack/cyborg/+/998817 | 12:23 |
| opendevreview | Joan Gilabert proposed openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 12:59 |
| 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 | 12:59 |
| chandankumar | cyborg irc meeting will start in another 50 mins, Add your agenda here: https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L47 | 13:09 |
| chandankumar | time for cyborg meeting! | 14:00 |
| chandankumar | #startmeeting cyborg | 14:01 |
| opendevmeet | Meeting started Tue Jul 28 14:01:01 2026 UTC and is due to finish in 60 minutes. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'cyborg' | 14:01 |
| jgilaber | o/ | 14:01 |
| morenod | o/ | 14:01 |
| chandankumar | who all are around? | 14:01 |
| rlandy | o/ | 14:01 |
| opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | 14:01 |
| chandankumar | #courtesy ping sean-k-mooney amoralej | 14:01 |
| chandankumar | Here is the link to agenda | 14:02 |
| chandankumar | #link https://etherpad.opendev.org/p/openstack-cyborg-irc-meeting#L43 | 14:02 |
| chandankumar | feel free to add there | 14:02 |
| sean-k-mooney | o/ | 14:03 |
| melwitt | o/ | 14:03 |
| chandankumar | Let me start with the first topic | 14:03 |
| chandankumar | #topic Proposal for pci-sim to support one device type per pf | 14:03 |
| chandankumar | jgilaber: added it Add per-PF device type via pf_mode parameter | https://review.opendev.org/c/openstack/cyborg/+/998603 | 14:04 |
| chandankumar | #link https://review.opendev.org/c/openstack/cyborg/+/998603 | 14:04 |
| jgilaber | yes, we've discussed about this in the past | 14:04 |
| jgilaber | since we plan to add mdevs and nvme emulation to pci-sim | 14:04 |
| jgilaber | we need some interface for configuring that | 14:04 |
| jgilaber | I've pushed that patch to propose one possibility | 14:04 |
| jgilaber | and I'm looking for feedback before rebasing my mdev work on top of it | 14:05 |
| jgilaber | it adds a 'pf_mode' parameter to the module | 14:05 |
| jgilaber | which is a comma-separated list of types, one per pf | 14:05 |
| jgilaber | e.g 'insmod fake_pci_sriov.ko num_pfs=2 pf_mode=sriov,mdev' would create two pfs, the first one sriov, the second with mdevs | 14:06 |
| sean-k-mooney | that shoudl not be via kernel args | 14:06 |
| sean-k-mooney | it shoudl be via per PF files in /sys | 14:07 |
| sean-k-mooney | that the same feedback i gave to chandankumar for the nvm devices as well | 14:07 |
| sean-k-mooney | each PF shoudl have a personality file that the pf reads to determin the type of VF to create | 14:08 |
| sean-k-mooney | which you set before you allocate teh VFs | 14:08 |
| sean-k-mooney | that is similar ot how vGPUs work and other hader deivce that supprot configurable types for there VFs | 14:08 |
| jgilaber | so the user/devstack plugin creates those files? | 14:10 |
| sean-k-mooney | basiclly /sys/bus/pci/devices/<PF>/vf_personality | 14:10 |
| sean-k-mooney | when you mode prope the module it will create teh file | 14:10 |
| opendevreview | Gihong Lee proposed openstack/cyborg master: Add shared lspci helpers to common utils https://review.opendev.org/c/openstack/cyborg/+/998726 | 14:10 |
| opendevreview | Gihong Lee proposed openstack/cyborg master: Migrate accelerator drivers to shared lspci helpers https://review.opendev.org/c/openstack/cyborg/+/998817 | 14:10 |
| sean-k-mooney | and you need to write the desired value to it | 14:10 |
| sean-k-mooney | we shoudl also have a /sys/bus/pci/devices/<PF>/vf_personalitys that list the avlaible type space seperated | 14:11 |
| sean-k-mooney | that or follow the cpu governors nameing convetion of adding _avaialble | 14:11 |
| jgilaber | ok I see | 14:12 |
| sean-k-mooney | [15:11:16]> cat /sys/bus/cpu/devices/cpu0/cpufreq/scaling_governor | 14:12 |
| sean-k-mooney | ─────┬───────────────────────────────────────────────────────────────────────────────────────────── | 14:12 |
| sean-k-mooney | │ File: /sys/bus/cpu/devices/cpu0/cpufreq/scaling_governor | 14:12 |
| sean-k-mooney | ─────┼───────────────────────────────────────────────────────────────────────────────────────────── | 14:12 |
| sean-k-mooney | 1 │ powersave | 14:12 |
| sean-k-mooney | ─────┴───────────────────────────────────────────────────────────────────────────────────────────── | 14:12 |
| sean-k-mooney | ~ via 🐹 v1.25.12 via python: v3.14.6 on ☁️ smooney@redhat.com | 14:12 |
| sean-k-mooney | [15:12:08]> cat /sys/bus/cpu/devices/cpu0/cpufreq/scaling_available_governors | 14:12 |
| sean-k-mooney | ─────┬───────────────────────────────────────────────────────────────────────────────────────────── | 14:12 |
| sean-k-mooney | │ File: /sys/bus/cpu/devices/cpu0/cpufreq/scaling_available_governors | 14:12 |
| jgilaber | from UX side it seems a bit more cumbersome | 14:12 |
| sean-k-mooney | ─────┼───────────────────────────────────────────────────────────────────────────────────────────── | 14:12 |
| sean-k-mooney | 1 │ performance powersave | 14:12 |
| sean-k-mooney | ─────┴──────────── | 14:12 |
| sean-k-mooney | the module approch forces you do reload the module to change it | 14:12 |
| sean-k-mooney | and it does not allwo you to eaislly do this on a per device basis | 14:13 |
| jgilaber | that's true | 14:13 |
| chandankumar | sean-k-mooney: jgilaber I was trying to come out like this https://gistpreview.github.io/?9cf18df5754db0948330f0cc5110c652 but not yet implemented | 14:13 |
| sean-k-mooney | skiming that that is mostly aligned on the sysfs interface to configure this | 14:14 |
| sean-k-mooney | basiclly secitn 6 | 14:14 |
| sean-k-mooney | we will need to document this in the contibutor docs for the module and cpature the logic as part of this work | 14:15 |
| sean-k-mooney | jgilaber: the other option woudl be to have multiple modules that we load seperatly | 14:15 |
| sean-k-mooney | with a shared core | 14:15 |
| sean-k-mooney | i.e. one per fake device type | 14:15 |
| sean-k-mooney | but i was kind fo hoping to only need one module for htis in general | 14:16 |
| jgilaber | yeah, I'll try to rework the patch with this new approach | 14:16 |
| jgilaber | and we can discuss again | 14:17 |
| sean-k-mooney | so i think we need to spit this out into its won commit | 14:17 |
| sean-k-mooney | i.e. one commit that intoduces the personatiy file reporting and configuration with just uart for now | 14:17 |
| sean-k-mooney | the you can chandan can hopefully build on that in parallel | 14:17 |
| sean-k-mooney | otherwise we will have to sequence both feature to not confilct | 14:18 |
| jgilaber | yes, that is what I did already. The linked patch does not have the mdev bits | 14:18 |
| sean-k-mooney | ah https://review.opendev.org/c/openstack/cyborg/+/998603 | 14:19 |
| sean-k-mooney | ok so that is just the config interface | 14:19 |
| sean-k-mooney | i woudl not be agaisn haveing module option to have a shorthand way to preallocate soem device fo a specifi tyep as well | 14:19 |
| sean-k-mooney | i.e. UART=2 NVME=4 | 14:19 |
| sean-k-mooney | buti still think the sysfs way is more flexible | 14:19 |
| chandankumar | what about bringing both the approach and revisit in next meeting? | 14:20 |
| jgilaber | ack, thanks for the feedback, I got what I need to continue | 14:20 |
| sean-k-mooney | basiclly i woudl like to have a clean way to be able to do soemtihng like pci_sim_toplogy=uart:nova,nvme:cyborg,mdev:cyborg | 14:20 |
| jgilaber | a note on timeline, I'll be on PTO starting Aug 5th | 14:20 |
| sean-k-mooney | and have devstack create 3 pf assign 1 to nova and 2 to cyborg ectra | 14:21 |
| jgilaber | so ideally it would be good to advance on this before the next meeting | 14:21 |
| sean-k-mooney | i think we can contiue the fonvcertion here and in gerrit | 14:21 |
| jgilaber | +1 | 14:21 |
| sean-k-mooney | we dont need to wait for the meeting to make progress | 14:21 |
| chandankumar | sounds good | 14:21 |
| chandankumar | Any more questions on this topic? | 14:22 |
| chandankumar | moving to next one | 14:22 |
| chandankumar | #topic Fix for guest reboot test | 14:23 |
| chandankumar | jgilaber is working on fixing guest reboot test | 14:23 |
| chandankumar | below are the reviews | 14:23 |
| chandankumar | #link 998283: Ensure guest is not reachable after reboot | https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/998283 | 14:23 |
| jgilaber | yes, I see sean-k-mooney has already reviewed it, thanks | 14:23 |
| chandankumar | #link Tested with pci and mdev tests in 997364: [WIP] Configure mdev driver lifecycle tests | https://review.opendev.org/c/openstack/cyborg/+/997364 | 14:23 |
| jgilaber | it seems to fix the problem, at least I haven't seen any failure with it | 14:24 |
| sean-k-mooney | ya i thinkt hat is ok ot go i was mainly waiting on ci on the followup | 14:24 |
| sean-k-mooney | basiclly to get more executions of the wait for unablable logic | 14:24 |
| sean-k-mooney | im going to do one more recheck but if that passes i think ill merge it later today | 14:25 |
| jgilaber | works for me, thanks! | 14:25 |
| chandankumar | thank you jgilaber for fixing the test. | 14:25 |
| chandankumar | Moving to next one | 14:25 |
| chandankumar | #topic Reviews | 14:25 |
| chandankumar | Any reviews needs attension please added it to https://etherpad.opendev.org/p/cyborg-2026.2-status | 14:26 |
| chandankumar | from etherpad I can see generic mdev driver is ready for review | 14:26 |
| chandankumar | #link https://review.opendev.org/q/topic:%22generic-mdev-driver%22 | 14:27 |
| sean-k-mooney | we need to review the nova part as well first right | 14:27 |
| chandankumar | Do we want to add the nova piece to the etherpad? | 14:27 |
| sean-k-mooney | at what point dose the depency start | 14:27 |
| sean-k-mooney | i guess we can | 14:28 |
| chandankumar | Here is the link to nova piece https://review.opendev.org/q/topic:%22cyborg-vgpu-support%22 | 14:28 |
| jgilaber | technically I guess at https://review.opendev.org/c/openstack/cyborg/+/997050 | 14:28 |
| melwitt | it's dependency that does not need or use Depends-On? | 14:29 |
| sean-k-mooney | well that what i was debating | 14:29 |
| jgilaber | but I added the depends-on at the last patch https://review.opendev.org/c/openstack/cyborg/+/997364/25 | 14:29 |
| sean-k-mooney | melwitt: so we can fully enabel the cybrog side without techinally mdoifying nova | 14:29 |
| sean-k-mooney | melwitt: but it wont actully work until we modify nova end ot end | 14:29 |
| melwitt | oh ok it's on the last patch | 14:29 |
| jgilaber | exactly | 14:29 |
| sean-k-mooney | the reporting and device binding will on the cybrog cide | 14:29 |
| melwitt | was just curious | 14:29 |
| sean-k-mooney | *cide | 14:30 |
| sean-k-mooney | ...side | 14:30 |
| jgilaber | the last one enables the tests so it needs the nova patches to work | 14:30 |
| melwitt | gotcha | 14:30 |
| sean-k-mooney | i think we may want to see the nova change merge beofre we merge https://review.opendev.org/c/openstack/cyborg/+/997050 | 14:31 |
| sean-k-mooney | but we can discuss it in teh review | 14:31 |
| jgilaber | yes, that would be a natural point as well | 14:31 |
| sean-k-mooney | its partly because fo the release note https://review.opendev.org/c/openstack/cyborg/+/997050/7/releasenotes/notes/generic-mdev-driver-af107e247cb1c815.yaml | 14:31 |
| sean-k-mooney | i dont really wanto to say "this is supproted" if only the cybrog part is done | 14:31 |
| melwitt | I think that could be ok as long as the last patch demonstrates everything working end-to-end (ensure we will not need to backtrack) | 14:31 |
| sean-k-mooney | melwitt: oh yes we can it just i want to make sure we have got some trackign on the nvoa side in paralle | 14:32 |
| jgilaber | that's fair, I could move the depends-on there | 14:32 |
| sean-k-mooney | no you dont need too | 14:32 |
| chandankumar | I have added nova dependency as well as final patch where all are tested on review etherpad | 14:32 |
| sean-k-mooney | this is more for use to have some check points | 14:32 |
| sean-k-mooney | lets review this as normal | 14:33 |
| sean-k-mooney | i jus tmay hold +w until we feel the nova changes are getting close | 14:33 |
| sean-k-mooney | i think you have already demonstrated a lot fo this via the later patches | 14:33 |
| melwitt | yeah I mean just saying that would be my criteria for merging the nova stuff first, seems ok idea to me as long as we have the end-to-end working demonstrated we know the nova stuff is 100% | 14:33 |
| jgilaber | yes the last does have everything working in tempest tests | 14:34 |
| melwitt | (that we won't need to change it) | 14:34 |
| sean-k-mooney | melwitt: so on the nova part | 14:34 |
| jgilaber | with the same lifecycle tests we have for pci devices | 14:34 |
| sean-k-mooney | cybrog alrady has one driver that supprot mdev based gpus | 14:34 |
| sean-k-mooney | and nova didnt merge teh nova change in the past | 14:34 |
| sean-k-mooney | which is why i want to see us make some progress there as well | 14:35 |
| melwitt | yes makes sense | 14:35 |
| sean-k-mooney | so we dotn end up with 2 driver that cant work at the end of the cycle | 14:35 |
| sean-k-mooney | by the way im not that worreid we have time | 14:35 |
| sean-k-mooney | just trying to level set | 14:35 |
| melwitt | are they all on the same gerrit topic? I find that helpful for not missing things personally | 14:36 |
| melwitt | yeah, all good | 14:36 |
| sean-k-mooney | the cybrog ones are the nova oens are on a diffent topic | 14:36 |
| chandankumar | we have two topics https://review.opendev.org/q/topic:%22generic-mdev-driver%22 and https://review.opendev.org/q/topic:%22cyborg-vgpu-support%22 | 14:36 |
| sean-k-mooney | we coudl use the same for both | 14:36 |
| sean-k-mooney | but technially the bluepritn names are diffent | 14:37 |
| sean-k-mooney | so two topic | 14:37 |
| melwitt | we don't have to, was just saying I like using it to tie together patches for the same effort that are across multiple projects. just a personal thing I like to do helps me keep track of all together | 14:37 |
| melwitt | https://review.opendev.org/q/topic:%22ipv6-multinode%22 is a good example haha | 14:38 |
| melwitt | ok, matching blueprint names, no problem. ignore me :) | 14:39 |
| sean-k-mooney | ya i like to do that too | 14:39 |
| sean-k-mooney | its only an issue if you have 2 difefnt blurpints for it | 14:39 |
| sean-k-mooney | then you have to chosoe wihc to use or use the hastag feature | 14:39 |
| melwitt | yeah, I see | 14:40 |
| chandankumar | https://review.opendev.org/q/topic:%22cyborg-vgpu-support%22+OR+topic:%22generic-mdev-driver%22 | 14:40 |
| sean-k-mooney | that worsk as well :) | 14:41 |
| chandankumar | any more questions on this review series | 14:41 |
| sean-k-mooney | unles there are other queston on this we can likely move on | 14:41 |
| jgilaber | yes, I think we can move one, thanks | 14:42 |
| sean-k-mooney | ill try to review the nova parts in the next day or two | 14:42 |
| chandankumar | Under Improvement section, we have guest reboot we already talked about that | 14:42 |
| melwitt | that doesn't fit how I use it fwiw, bc I click on the topic from one of the patches to discover the others. but the link is helpful nonetheless :) | 14:42 |
| chandankumar | Now jumping to bug | 14:42 |
| chandankumar | #topic Bugs | 14:42 |
| chandankumar | #link Devstack doesn't enable port_device_profile ML2 extension driver for Cyborg: https://bugs.launchpad.net/openstack-cyborg/+bug/2161365 | 14:42 |
| chandankumar | It was opened by melwitt | 14:43 |
| melwitt | I got some feedback on my patch for this, apparently not really a bug. so not sure if I should be thinking to close it? | 14:43 |
| sean-k-mooney | whishlist is fine | 14:43 |
| melwitt | ok | 14:43 |
| jgilaber | +1 seems fine as is | 14:43 |
| chandankumar | sean-k-mooney: chnaged it to wishlist | 14:43 |
| sean-k-mooney | ist a mini enhancment | 14:43 |
| melwitt | thanks | 14:43 |
| sean-k-mooney | melwitt: since we are on this topic | 14:44 |
| chandankumar | it is already triaged, thank you | 14:44 |
| sean-k-mooney | i ased for some enhacned docs in teh review i left | 14:44 |
| sean-k-mooney | but i was also thinking maybe we shoudl clean up https://github.com/openstack/cyborg/blob/master/cyborg.conf.intelnic.sample | 14:44 |
| sean-k-mooney | its really weird to have a random config example at the root of the repo | 14:45 |
| melwitt | ok sure | 14:45 |
| sean-k-mooney | my prefence woudl be to add a local.conf exmaple instead https://github.com/openstack/cyborg/blob/master/devstack/local-conf.pci-sim.sample | 14:45 |
| sean-k-mooney | but ya that basiclly what i said in the review | 14:46 |
| sean-k-mooney | i am fine with adding a speicif driver doc later | 14:46 |
| melwitt | I thhink that would make sense | 14:46 |
| melwitt | *think | 14:46 |
| sean-k-mooney | so im not really asking for fully docs just capture the impaont parts form the gist you rote | 14:46 |
| melwitt | ok, can do | 14:47 |
| sean-k-mooney | i.e. if your automating the cofniguration with devstack add a small devstack local.conf samepel adn a littel prose to descibe how to set it up. | 14:47 |
| sean-k-mooney | cool | 14:47 |
| melwitt | thanks | 14:47 |
| sean-k-mooney | by the way the reaon i said its not entirly a bug | 14:47 |
| sean-k-mooney | is until this cycle | 14:48 |
| sean-k-mooney | there really wast any devstack automation for any fo the drivers | 14:48 |
| sean-k-mooney | im fine with filign wishlist bugs as tracker but we are missing devstack supprot for most of the rest of the drivrs as well | 14:48 |
| sean-k-mooney | and i dont thnk fixing that when we have time shoudl need a lot of paper work | 14:49 |
| melwitt | yeah I mean for this, it's the general device_profile extension so seems required for any cyborg use no? | 14:49 |
| sean-k-mooney | only if using cyborg with neutron ports | 14:49 |
| melwitt | yeah, it's good we have Wishlist | 14:50 |
| melwitt | ah right. ok | 14:50 |
| sean-k-mooney | that why i was debting if this shoudl only be enabled if you enable the intel driver | 14:50 |
| sean-k-mooney | or sriov nic agent but its harmless to alwasy enabel | 14:50 |
| sean-k-mooney | neutron will ignore it if your usign ovn af far as im aware | 14:50 |
| melwitt | yeah. I think a guard would make sense. I guess intel is the only nic driver under nic/? I forgot already | 14:51 |
| sean-k-mooney | currently yes | 14:51 |
| sean-k-mooney | https://github.com/openstack/cyborg/blob/master/pyproject.toml#L52-L63 | 14:51 |
| sean-k-mooney | at least in tree | 14:51 |
| melwitt | ok cool. bc really it would be if any nic driver enabled but if we only have the one, then yeah | 14:52 |
| sean-k-mooney | its technically a stevador entriy point i think but i woudl have to check | 14:52 |
| chandankumar | we have one more bug to discuss, then we can talk more about that after meeting | 14:53 |
| sean-k-mooney | cool | 14:53 |
| chandankumar | #link | 14:53 |
| chandankumar | [cyborg-tempest-plugin] Cyborg tempest tests fail in role manager does not exist https://bugs.launchpad.net/openstack-cyborg/+bug/2161909 | 14:53 |
| chandankumar | amoralej: opened it. | 14:54 |
| sean-k-mooney | melwitt: https://github.com/openstack/cyborg/blob/master/cyborg/agent/resource_tracker.py#L53-L57 | 14:54 |
| chandankumar | It comes only when manager role does not exist in the environment | 14:54 |
| sean-k-mooney | chandankumar: ya so i dug into this a bit | 14:54 |
| sean-k-mooney | the reaosn this fails downtram is we are using keystone form antelope 2023.1 | 14:54 |
| sean-k-mooney | and the manager rule is created by default in bobcat 2023.2 | 14:55 |
| sean-k-mooney | so for all suprpoted branches we can unconditonly rely on it existing | 14:55 |
| sean-k-mooney | it would not be hardd to add supprot for older release but those are technially not supproted on master | 14:56 |
| chandankumar | for upstream stable branches, It does not seems to be a bug then | 14:56 |
| sean-k-mooney | right its not | 14:56 |
| sean-k-mooney | it just a question of do we wnat to add one off support for older branches | 14:57 |
| sean-k-mooney | in generall we should not try to supprot antelope | 14:57 |
| sean-k-mooney | but we coudl supprot it in this case if we choose too | 14:57 |
| jgilaber | how extensive would the changes be? | 14:58 |
| sean-k-mooney | like an if or two | 14:58 |
| sean-k-mooney | https://github.com/openstack/cyborg-tempest-plugin/blob/master/cyborg_tempest_plugin/tests/api/base.py#L73-L107 | 14:58 |
| sean-k-mooney | the fix is to only create the role specific client if we are using the new defaults | 14:59 |
| sean-k-mooney | so move | 14:59 |
| sean-k-mooney | https://github.com/openstack/cyborg-tempest-plugin/blob/master/cyborg_tempest_plugin/tests/api/base.py#L82-L94 | 14:59 |
| sean-k-mooney | under the if branches | 14:59 |
| sean-k-mooney | https://github.com/openstack/cyborg-tempest-plugin/blob/master/cyborg_tempest_plugin/tests/api/base.py#L100-L107 | 14:59 |
| jgilaber | ok, I think that is acceptable | 14:59 |
| chandankumar | ok | 15:00 |
| chandankumar | coming to bug importance to low? | 15:00 |
| sean-k-mooney | whishlist | 15:00 |
| sean-k-mooney | again its for an unsuprpoted release | 15:00 |
| jgilaber | yes, wishlist seems right | 15:00 |
| sean-k-mooney | but we can fix it up fairly eaislly this time | 15:00 |
| chandankumar | Done | 15:00 |
| sean-k-mooney | if the issue was usign a newer api provided by tempest | 15:00 |
| chandankumar | we are running out of time | 15:00 |
| sean-k-mooney | or somehtign liek that we woudl not fix it | 15:01 |
| chandankumar | #topic Open Discussion | 15:01 |
| melwitt | I just wanted to mention that I am thinking of proposing a spec for what I'm calling "services API" summary here https://gist.github.com/melwitt/66443c2606ae69280ea1effff7c30437 | 15:01 |
| melwitt | this will just mean adding a services table and heartbeat mechanism | 15:01 |
| melwitt | if anyone has any thoughts or concerns please let me know (doesn't have to be now given the time) | 15:02 |
| sean-k-mooney | i have one posisbel scope expantion to run by you | 15:02 |
| sean-k-mooney | but we can dicuss that later | 15:02 |
| chandankumar | melwitt: thank you! | 15:02 |
| chandankumar | #topic volunteer to chair for next meeting | 15:02 |
| chandankumar | Anyone up for chairing next meeting? | 15:03 |
| jgilaber | I can do it chandankumar | 15:03 |
| chandankumar | thank you jgilaber ! | 15:03 |
| chandankumar | thank you all for joining! | 15:03 |
| sean-k-mooney | ack i can if you need ot skip | 15:03 |
| chandankumar | #endmeeting | 15:03 |
| opendevmeet | Meeting ended Tue Jul 28 15:03:42 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-28-14.01.html | 15:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-28-14.01.txt | 15:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cyborg/2026/cyborg.2026-07-28-14.01.log.html | 15:03 |
| sean-k-mooney | melwitt: so you included https://gist.github.com/melwitt/66443c2606ae69280ea1effff7c30437#related-work-http-healthcheck-endpoint | 15:04 |
| sean-k-mooney | i dont think we shoudld supprot the oslo.middleware | 15:04 |
| sean-k-mooney | but i would love to supprot https://specs.openstack.org/openstack/nova-specs/specs/2024.2/approved/per-process-healthchecks.html | 15:04 |
| melwitt | ok cool I wondered about that | 15:04 |
| sean-k-mooney | for all of our service binaries | 15:04 |
| sean-k-mooney | so i have not issue with a healtcheck endpoint | 15:05 |
| sean-k-mooney | i just woudl prefer to have a more useful one | 15:05 |
| melwitt | yeah that's fair | 15:05 |
| sean-k-mooney | you can also say no | 15:06 |
| sean-k-mooney | but ya it woudl be cool to spike on that | 15:06 |
| sean-k-mooney | and see if it woudl fit with what you wanted to do or not | 15:06 |
| melwitt | haha no I put it there for completeness since it is a thing we could do but don't have to do | 15:06 |
| sean-k-mooney | i breing this up because manilla are adding one as well https://github.com/openstack/manila-specs/blob/master/specs/hibiscus/manila-per-process-healthchecks.rst | 15:07 |
| melwitt | I was already wary of it given that not all services use it, so I figured there must be a reason for that and might not be something we want to add to | 15:07 |
| melwitt | (not all services use oslo.middleware I mean) | 15:08 |
| sean-k-mooney | the reaosn i dont like it for the api is it does not tell you anthign hiting the microversion discovery endpoitn does nto provide | 15:08 |
| sean-k-mooney | but it leskes the gcc adn kernel verison in debug mode | 15:08 |
| sean-k-mooney | which si not exactly great | 15:08 |
| melwitt | yeah, I vaguely remember that from nova discussion | 15:09 |
| sean-k-mooney | if you actully hit the microversion endpoint its actully slightly better as it actully invokes the wsgi app vs just the middleware | 15:09 |
| melwitt | I have to run for a bit but will bbl | 15:09 |
| sean-k-mooney | cool o/ | 15:09 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Ensure guest is not reachable after reboot https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/998283 | 16:20 |
| opendevreview | Merged openstack/cyborg-tempest-plugin master: Refactor PCI lifecycle tests https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/997936 | 16:20 |
| 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 | 16:21 |
| 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 | 16:31 |
| opendevreview | Joan Gilabert proposed openstack/cyborg master: Add per-PF vf_personality sysfs interface https://review.opendev.org/c/openstack/cyborg/+/998603 | 16:54 |
| opendevreview | Joan Gilabert proposed openstack/cyborg master: Add per-PF vf_personality sysfs interface https://review.opendev.org/c/openstack/cyborg/+/998603 | 17:30 |
| opendevreview | Joan Gilabert proposed openstack/cyborg master: Add per-PF vf_personality sysfs interface https://review.opendev.org/c/openstack/cyborg/+/998603 | 17:37 |
| opendevreview | melanie witt proposed openstack/cyborg master: Enable port_device_profile ML2 extension driver for NIC drivers https://review.opendev.org/c/openstack/cyborg/+/998055 | 19:55 |
| opendevreview | melanie witt proposed openstack/cyborg master: Enable port_device_profile ML2 extension driver for NIC drivers https://review.opendev.org/c/openstack/cyborg/+/998055 | 20:17 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!