| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-04-16 | |||
| 20:11:50 | sean-k-mooney | we shoudl be but the pci stat pools need to prperly handel that grouping | |
| 20:12:40 | sean-k-mooney | it hilgy likely we never tested it and it broken as a result but its totally expressable today vai our config options | |
| 20:15:05 | sean-k-mooney | Uggla: i have not had time to add pci grouping to the nova ptg but if we wanted to talk about different VF in the same pf having diffent rsouce classes for gpus usecases ti woudl make sense to talk about it toghetehr with nova pci grouping | |
| 20:15:16 | sean-k-mooney | i had ot parly implemnt that for that feature anyway | |
| 20:15:37 | sean-k-mooney | since i want to supprot creating groups fo VFs from teh same or diffent PFs | |
| 20:19:04 | sean-k-mooney | i was mainly testing with takign 64 VF that i was subdeviding into groups | |
| 20:19:52 | sean-k-mooney | a groups of 8 and groups of 4 would have 2 diffent resocues classes even if they are effectivly the same device | |
| 20:21:06 | sean-k-mooney | as long as all host that report CUSTOM_PCI_GROUP_NICX8 provide the same VF type for that i.e. so they are interchangable form the schdluer point of view | |
| 20:21:45 | sean-k-mooney | its ok to report other VF form the same card with a differnt type. | |
| 20:21:58 | sean-k-mooney | *differnt resouce class. | |
| 20:23:40 | sean-k-mooney | if you had a gpu that you partioned into 3 Mig device 1 that used half the card an 2 that used 1/4 each | |
| 20:24:20 | sean-k-mooney | the correct way to model that in placment via nova ro cyborg would be 1 RP for the PF with an inveotyr of 2 of one resocue class and 1 of a diffent one | |
| 20:26:36 | sean-k-mooney | being very hand wavy that is the eventual plan for how to do that in cybrog but not right away. | |
| 21:22:11 | opendevreview | Merged openstack/nova master: Fix keypairs lost during cross-cell resize https://review.opendev.org/c/openstack/nova/+/981558 | |
| 21:22:28 | opendevreview | Merged openstack/nova master: Skip some unit tests on macOS https://review.opendev.org/c/openstack/nova/+/985014 | |
| #openstack-nova - 2026-04-17 | |||
| 00:27:47 | opendevreview | ChungWon Lee proposed openstack/nova master: Add regression test for bug #2134375 https://review.opendev.org/c/openstack/nova/+/984900 | |
| 02:58:11 | opendevreview | Sam Morrison proposed openstack/nova master: Filter out deleted instances when looking for build timouts https://review.opendev.org/c/openstack/nova/+/880125 | |
| 06:57:50 | opendevreview | chanyeol yoon proposed openstack/nova master: Fix infinite WARNING loop in _reclaim_queued_deletes https://review.opendev.org/c/openstack/nova/+/985068 | |
| 07:55:13 | opendevreview | Kamil Sambor proposed openstack/nova master: Enable threading mode for proxy services https://review.opendev.org/c/openstack/nova/+/976089 | |
| 09:31:01 | opendevreview | Kamil Sambor proposed openstack/nova master: Test nova CLI commands with native threading https://review.opendev.org/c/openstack/nova/+/984036 | |
| 09:43:38 | Uggla | sean-k-mooney, as discussed I will prepare the pci grouping topic because that's an internal high prio topic. | |
| 09:44:38 | sean-k-mooney | thanks i have not really had tiem to propsoe it bvut that partly becasue i dont really knwo what if any the open questiosn are | |
| 09:44:53 | sean-k-mooney | the spec has obvioulsy been up for a while and its jsut a refinement of the previoulsy appoved spec | |
| 09:45:35 | sean-k-mooney | so i dont really know what need ptg discussions vs what just need gerrit reivew | |
| 09:45:55 | bauzas | Uggla: sean-k-mooney: IMHO we should be discussing this in the cyborg-nova x-p session | |
| 09:46:02 | sean-k-mooney | nope | |
| 09:46:16 | sean-k-mooney | thi is not a cyborg related topic | |
| 09:46:50 | sean-k-mooney | this is a pure nova proposal and even if cyborg may have simialr functionaltiy in the future it is indepent of that | |
| 09:46:59 | gibi | it is hard to say what are the open question before loading context on the PCI grouping topic. I'm doing that loading now | |
| 09:47:16 | sean-k-mooney | gibi: thanks ++ | |
| 09:48:06 | sean-k-mooney | this is what im suggesting for the nova-cybrog topics https://etherpad.opendev.org/p/cyborg-2026.2-ptg#L294 | |
| 09:48:15 | sean-k-mooney | but honestly i feel like that already too much | |
| 09:48:33 | gibi | Uggla: on a related note, I found hints in the pci/stats code that also assumes that VFs under the same parent are equivalent https://github.com/openstack/nova/blob/525158521281cf643a9942522788efd15ba12d6b/nova/pci/stats.py#L164-L169 so this code need to be looked at when we want to remove the restriction of one RC for all VFs under the same PF | |
| 09:49:10 | Uggla | sean-k-mooney, Could you send me the link to the spec if you have it handy? | |
| 09:49:22 | sean-k-mooney | sure ill go grab it | |
| 09:49:57 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/975635 | |
| 09:50:17 | Uggla | I agree the cyborg content seems already too large. So definitively need a session for PCI grouping. | |
| 09:50:26 | sean-k-mooney | weired in my brian i pushed that in like december not febuary | |
| 09:51:06 | sean-k-mooney | i guess because i sated on it just before i went on pto and then picked it up agan breifly in late jan early febuary | |
| 09:51:19 | Uggla | sean-k-mooney thx | |
| 09:52:00 | sean-k-mooney | the other spec https://review.opendev.org/c/openstack/nova-specs/+/967515 will be taken over by joan thaat is the first cyborg sepcifc spec for nova for this cycle | |
| 09:52:32 | sean-k-mooney | im not sure if we will have another | |
| 09:52:44 | sean-k-mooney | it kind of depend on which feature/integration we priorites | |
| 09:55:29 | Uggla | Also, I’m likely lacking some knowledge on PCI grouping, but I’m counting on all of you to help me out. <3 | |
| 09:57:43 | bauzas | stephenfin: thanks a lot for your proposals ! Sorry I preferred to have some bootstrapped files but sure I'll review your patch and accept it once I doublecheck about the differences between the markdown file and the restructured one | |
| 09:57:44 | sean-k-mooney | in essence its just a way to say "provision these togeher and schdule them as a group" that the core idea, today you can have man pci alias request in a single flavor but they alwas provide you 1 device per aplis, pci grouping is just a way to say thses specific set of device are member of a group and i would like 1 of that type of group in a concise way | |
| 09:58:14 | sean-k-mooney | everything beyond that and the usecase that enabels are the details but that is the 10,000 foot view | |
| 09:59:59 | stephenfin | bauzas: Thanks. Can I get core on that too, please? I think sean-k-mooney is after it also | |
| 10:00:31 | sean-k-mooney | yes i would like to activly be inovled in the development fo that repo | |
| 10:00:34 | stephenfin | or better, just add various `*-core` projects for now, which we can extend with non-cores (ops, other end-users) over time | |
| 10:00:50 | stephenfin | *`*-core` groups from projects | |
| 10:01:11 | sean-k-mooney | im also reuickly reivewing the scafold patch | |
| 10:01:21 | stephenfin | sean-k-mooney: it's already merged :( | |
| 10:01:31 | sean-k-mooney | ok i can propsoe a revert :P | |
| 10:01:34 | sean-k-mooney | just kidding | |
| 10:01:35 | gibi | sean-k-mooney: where I feel complication (and digging into) is that today we schedule based on per device parameteres like live-migratable, traits, device_type, numa, etc. But when you have a group of such devices these parameters are not always easily inherited from a diverse set of device to a single group the devs are in | |
| 10:01:35 | stephenfin | bauzas slightly jumped the gun, I'm afraid | |
| 10:01:48 | sean-k-mooney | but i want to get some agreement on the repo layout ectra before we go too far | |
| 10:02:02 | stephenfin | (I understand his rationale, but it was still under very active review) | |
| 10:02:06 | sean-k-mooney | i was going to work on some liniting/precommit tooling too | |
| 10:02:52 | sean-k-mooney | if this is going to be a real repo it need at least basic zuul ci | |
| 10:02:59 | sean-k-mooney | and that shoudl be doen very early | |
| 10:05:44 | stephenfin | true | |
| 10:09:02 | gibi | sean-k-mooney: stephenfin: which repo are you talking about? :) | |
| 10:09:15 | stephenfin | https://review.opendev.org/q/project:openstack/agentic-workflows | |
| 10:09:27 | sean-k-mooney | https://opendev.org/openstack/agentic-workflows | |
| 10:10:34 | sean-k-mooney | we have a lot of thing to agree on like rst+sphink or markdonw ectra before we start commiting content there | |
| 10:12:00 | gibi | ohh, is it a grassroot movement, or was there already a decision on the tc / community level to have it? | |
| 10:12:26 | sean-k-mooney | it was discssed on the list and propsoed to buidl it under the tact sig | |
| 10:12:32 | sean-k-mooney | but litrlly in the last day | |
| 10:12:43 | gibi | very strange move before the PTG | |
| 10:12:48 | sean-k-mooney | which is why i was not expecting to really merge any content ther until after it was more widely dicssed at the ptg | |
| 10:12:52 | sean-k-mooney | right | |
| 10:13:06 | stephenfin | yeah | |
| 10:13:11 | sean-k-mooney | i didnt expect either the repo to be create or content to be merged untile after the tc slot | |
| 10:13:33 | sean-k-mooney | i fine with propsoing some review ectra | |
| 10:13:59 | gibi | I'm affraid that the content in this repo is seens as the communities agreement about this topic | |
| 10:17:57 | bauzas | sorry I was off for 15 mins due to a taxi need | |
| 10:18:23 | bauzas | sean-k-mooney: stephenfin: I had to merge some things but we can change anything for sure | |
| 10:18:38 | sean-k-mooney | well you not really | |
| 10:18:48 | sean-k-mooney | it could have statyed in gerrit | |
| 10:19:05 | stephenfin | yeah that's what I said 😅 | |
| 10:19:40 | sean-k-mooney | but ya we can change things | |
| 10:20:08 | bauzas | stephenfin: sean-k-mooney: I'm just adding you both as cores | |
| 10:20:25 | sean-k-mooney | i need to also go do taxi for "Freya" since my frid does not want to walk in the rain with the dog. | |
| 10:20:47 | sean-k-mooney | because of course teh "dog" does not want to walk :P | |
| 10:21:01 | bauzas | stephenfin: sean-k-mooney: added you both | |
| 10:21:07 | bauzas | stephenfin: now reviewing your change | |
| 10:22:53 | stephenfin | ty 🙏 | |
| 10:26:06 | bauzas | stephenfin: 2 nits and I'm cool | |
| 10:26:08 | bauzas | https://review.opendev.org/c/openstack/agentic-workflows/+/985078 | |
| 10:31:45 | bauzas | sean-k-mooney: sorry, as I said with my +W comment, it was a chicken-and-egg issue, but please review https://opendev.org/openstack/agentic-workflows/src/branch/master/AGENTS.md because of your concerns about the directories | |
| 10:33:09 | bauzas | stephenfin: sean-k-mooney: gibi: would you prefer I revert my merged changes actually ? | |
| 10:35:39 | stephenfin | bauzas: thanks, done | |
| 10:36:07 | gibi | bauzas: I prefer to have proper discussion on the PTG | |
| 10:36:23 | stephenfin | RE: changes, I don't see the point of reverting now, but I'd certainly hold off merging any skills yet. PTG is next week: let's wait until after that | |
| 10:36:32 | gibi | before sending an official looking message via a repo in the openstack namespec | |
| 10:36:37 | gibi | namespace | |
| 10:36:41 | bauzas | I'm like stephenfin here | |
| 10:37:04 | bauzas | I think we can stop merging anything but the AGENTS.md and the README.md | |