| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-23 | |||
| 21:07:13 | sean-k-mooney | https://github.com/SeanMooney/ard/blob/master/ansible/roles/ard_multinode_bridge/templates/ard-bridge-setup.sh.j2 | |
| 21:07:24 | clarkb | and that work is basically done so there isn't any reason not to | |
| 21:08:27 | sean-k-mooney | clarkb: ya https://review.opendev.org/c/zuul/zuul-jobs/+/959393 cerntely cleans up a lot | |
| 21:08:59 | clarkb | rereading things it stalled beacuse people using ovs as a hard dependency weren't properly installing/settting up that dep and relying on that role instead | |
| 21:09:08 | clarkb | I think we can basically say sorry you need to fix that and move forward | |
| 21:09:23 | clarkb | which is also what I basically said back when I was trying to move this forward | |
| 21:09:32 | sean-k-mooney | ya. is ther a reason it swapt to geneve? | |
| 21:09:56 | sean-k-mooney | i know it says "This is a more modern and flexible approach than the | |
| 21:09:58 | sean-k-mooney | previous vxlan implementation." | |
| 21:09:58 | clarkb | I think because geneve is the future and more flexible | |
| 21:10:08 | clarkb | right vxlan had a bunch of gotchas like the ipv6 support or lack of it | |
| 21:10:14 | clarkb | geneve exists to solve all those problems with a new protocol | |
| 21:10:21 | clarkb | so we should use it since those problems hit us previously | |
| 21:10:29 | sean-k-mooney | the main one that geneve solve is actully fragmentation | |
| 21:10:37 | sean-k-mooney | vxlan explity does not allow fragmentation | |
| 21:10:51 | sean-k-mooney | of the vxlan packet | |
| 21:10:58 | sean-k-mooney | geneve i belvie does | |
| 21:11:21 | sean-k-mooney | which makes it more flexible acorss l3 subnets | |
| 21:12:15 | sean-k-mooney | clarkb: but i guess i agree building on that patch woudl be good for the reason above | |
| 21:12:21 | sean-k-mooney | anyway dinner is ready o/ | |
| 21:13:02 | clarkb | ya I think the next step is to followup with ykarel to do what we can to properly capture ovs dependencies then proceed with the switch in zuul-jobs then add ipv6 support | |
| 21:13:09 | clarkb | probably don't even need to make ipv6 support optional just add it | |
| 21:14:05 | clarkb | ykarel: can you followup on https://review.opendev.org/c/zuul/zuul-jobs/+/959393 and indicate if properly managing jobs' dependency on ovs si a possibility? That really shouldn't be something we rely on a side effect from a base job role to handle | |
| 22:00:05 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 22:51:31 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 23:52:14 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| #openstack-nova - 2026-06-24 | |||
| 01:38:38 | opendevreview | sean mooney proposed openstack/nova master: doc: Add guide for NVIDIA vGPU with SR-IOV variant drivers https://review.opendev.org/c/openstack/nova/+/994160 | |
| 01:38:39 | opendevreview | sean mooney proposed openstack/nova master: libvirt: Support Cyborg PCI ARQ managed mode https://review.opendev.org/c/openstack/nova/+/994579 | |
| 02:23:15 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 03:07:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 03:07:59 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 03:11:31 | opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | |
| 03:16:47 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 03:17:13 | opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | |
| 03:17:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 04:25:02 | ykarel | clarkb, thx for reminding that, i had completely forgot about that :) | |
| 04:25:53 | ykarel | Will check on this in next few days and see how it can be unblocked | |
| 04:26:15 | ykarel | will start with the impacted job and see | |
| 06:44:00 | opendevreview | Seyeong Kim proposed openstack/nova master: Avoid serializing orphaned instance in log https://review.opendev.org/c/openstack/nova/+/992574 | |
| 11:22:05 | opendevreview | Stephen Finucane proposed openstack/os-resource-classes master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-resource-classes/+/994678 | |
| 11:22:06 | opendevreview | Stephen Finucane proposed openstack/os-vif master: pre-commit: Bump versions https://review.opendev.org/c/openstack/os-vif/+/994676 | |
| 11:22:06 | opendevreview | Stephen Finucane proposed openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | |
| 11:22:07 | opendevreview | Stephen Finucane proposed openstack/os-traits master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-traits/+/994680 | |
| 11:22:08 | opendevreview | Stephen Finucane proposed openstack/os-vif master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-vif/+/994677 | |
| 11:49:23 | opendevreview | Merged openstack/os-vif master: pre-commit: Bump versions https://review.opendev.org/c/openstack/os-vif/+/994676 | |
| 12:49:34 | opendevreview | Stephen Finucane proposed openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | |
| 13:22:19 | opendevreview | Elod Illes proposed openstack/python-novaclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-novaclient/+/994731 | |
| 13:27:09 | opendevreview | Merged openstack/osc-placement master: Update support python versions https://review.opendev.org/c/openstack/osc-placement/+/991603 | |
| 14:05:27 | opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | |
| 14:21:31 | Uggla | Note: I could not do the upstream triage this week. So cancelling it. | |
| 14:21:53 | gibi | Uggla: ack | |
| 15:12:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 15:12:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 16:03:29 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | |
| 16:04:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | |
| 16:37:38 | opendevreview | Merged openstack/os-vif master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-vif/+/994677 | |
| 16:38:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 16:39:24 | melwitt | sean-k-mooney: got the WIP ceph ipv6 passing and put it on top of the RBD live migration XML patch. the majority of the time was fighting with ipv6 brackets 🙄 the whole collection, if you're curious https://review.opendev.org/q/topic:%22ipv6-multinode%22 | |
| 16:40:23 | sean-k-mooney | ah so you needed the know host request to allow the live migraiton to complete | |
| 16:41:23 | sean-k-mooney | i wonder if it woul be worth having devstack or the devstack base job generate a tiny ~/.ssh/config file that auto accpate new hosts | |
| 16:41:58 | sean-k-mooney | https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/994583 is more correct in general but we dont really need to validate it in general for ci | |
| 16:43:15 | sean-k-mooney | basiclly we could also do | |
| 16:43:18 | sean-k-mooney | Host * | |
| 16:43:20 | sean-k-mooney | StrictHostKeyChecking accept-new | |
| 16:43:24 | sean-k-mooney | or set it to no | |
| 16:44:07 | melwitt | sean-k-mooney: I think known hosts was needed for cold migration rather than live. for a time I had live migration working but cold was broken due to lack of known hosts | |
| 16:44:22 | sean-k-mooney | its used for both | |
| 16:44:43 | sean-k-mooney | we dont use tls by defult in ci so we default to live mgiration over ssh | |
| 16:44:59 | melwitt | ok. I am not sure what combo of things made it so that live would work but cold not but I had that state for a bit until I fixed it | |
| 16:45:03 | sean-k-mooney | but your current patch is valid and more correct to refesh the know host after we create teh bridge | |
| 16:46:35 | melwitt | yeah it seemed to make sense | |
| 16:46:37 | sean-k-mooney | melwitt: so i guess the next step would be to rebase https://review.opendev.org/c/zuul/zuul-jobs/+/994521 on top of https://review.opendev.org/c/zuul/zuul-jobs/+/959393 | |
| 16:48:06 | melwitt | yeah, good point, I think ykarel said he might be able to pick up that patch again | |
| 16:48:12 | opendevreview | Merged openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | |
| 16:49:04 | opendevreview | Merged openstack/os-traits master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-traits/+/994680 | |
| 16:49:15 | melwitt | I cc myself | |
| 16:50:31 | melwitt | sean-k-mooney: note that the zuul-jobs refresh is not actually used in the openstack case bc the openstack-zuul-jobs playbook is used instead. but I put in both places for consistency | |
| 16:51:06 | melwitt | apparently if openstack-zuul-jobs did not override with its own playbook the base zuul-jobs playbook would be used | |
| 16:52:01 | sean-k-mooney | ah right i guess we were workign around a specific issue in our own ci env at somepoint | |
| 16:52:19 | melwitt | and I did note yalls comments that it's likely unnecessary or undesirable to make the ipv6 setup optional in zuul-jobs but I was wary of it breaking anything if there's something wrong with it that I'm not aware of | |
| 16:53:42 | sean-k-mooney | i woudl allow it to be optional i just proably would have turned it on by defualt | |
| 16:53:50 | sean-k-mooney | but im to overried that in our jobs in devstack | |
| 16:53:50 | melwitt | on the other hand making it always on would shake out issues better and also prevent new potential issues better. so either way is cool I think, just mentioning why I didn't change that right away | |
| 16:54:17 | melwitt | gotcha | |
| 16:55:57 | sean-k-mooney | we can either enable it here https://github.com/openstack/devstack/blob/master/.zuul.yaml#L378 or here https://github.com/openstack/devstack/blob/master/.zuul.yaml#L741 | |
| 16:56:37 | sean-k-mooney | i think you alray have that in the nova? job right it just addign the flag to the vars | |
| 16:58:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 17:06:02 | melwitt | sean-k-mooney: yeah just added to vars https://review.opendev.org/c/openstack/nova/+/984623/26/.zuul.yaml | |
| 17:08:00 | opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | |
| 17:19:53 | sean-k-mooney | melwitt: so im ok with the fix patch if your ok with https://review.opendev.org/c/openstack/nova/+/993597/4/nova/tests/functional/regressions/test_bug_1741364.py then im ok to upgrade to +2 | |
| 17:28:58 | opendevreview | Ghanshyam Maan proposed openstack/placement master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/placement/+/994790 | |
| 17:33:49 | gmaan | stephenfin: updated the enforce_scope removal changes for placement, separating the upgrade impact release notes with mentioning of oslo.policy new version with updating the same in requirements.txt. let me know if that works for you, holding the nova update till then https://review.opendev.org/c/openstack/placement/+/994790 | |
| 17:41:23 | melwitt | sean-k-mooney: I think it's a good point about the base class, I had missed the weirdness with imagebackend | |
| 17:41:43 | melwitt | I added a comment | |
| 17:42:06 | sean-k-mooney | i dont really have an issue using that as a base normally but when it reuqired use to mock or in this case unmock a mock i hesitate | |
| 17:42:35 | sean-k-mooney | melwitt: i remember gibi spending dime removing a bunch of double mocking in the past | |
| 17:42:53 | melwitt | sean-k-mooney: yeah I think that's totally valid | |
| 17:43:09 | sean-k-mooney | https://github.com/openstack/nova/commit/bf654e3a4a8f690ad0bec0955690bf4fadf98dba | |
| 17:43:18 | sean-k-mooney | "Remove double mocking... again" | |
| 17:43:20 | sean-k-mooney | :) | |