| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-24 | |||
| 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 | :) | |
| 17:44:05 | melwitt | heh | |
| 17:44:20 | melwitt | it would be nice if we could detect those and prevent | |
| 17:45:41 | sean-k-mooney | hum its python so anything is possible but it normally does not cause use problems | |
| 23:54:28 | opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | |
| #openstack-nova - 2026-06-25 | |||
| 01:35:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 08:06:52 | opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | |
| 13:11:15 | opendevreview | Merged openstack/os-resource-classes master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-resource-classes/+/994678 | |
| 13:29:40 | opendevreview | Stephen Finucane proposed openstack/placement master: Mandate policy scope enforcement https://review.opendev.org/c/openstack/placement/+/994892 | |
| 14:02:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Limit the port fields during security_group query https://review.opendev.org/c/openstack/nova/+/994896 | |
| 14:11:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 14:50:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 14:50:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Support launchSecurity element for SEV-SNP https://review.opendev.org/c/openstack/nova/+/994920 | |
| 14:56:35 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 15:32:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: AMD SEV: omit iommu='on' for virtio devices https://review.opendev.org/c/openstack/nova/+/909635 | |
| 15:33:55 | tkajinam | would be nice to get this one reviewed early because this likely conflicts with AMD SEV-SNP support ^^^ | |
| 15:34:04 | tkajinam | (due to a few cleanups I'm adding to the chain) | |
| 15:35:04 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 15:35:05 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 15:35:05 | opendevreview | Takashi Kajinami proposed openstack/nova master: launch: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |
| 15:46:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |
| 15:46:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 16:03:29 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 16:03:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 16:03:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Support launchSecurity element for SEV-SNP https://review.opendev.org/c/openstack/nova/+/994920 | |
| 16:03:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 16:03:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |
| 16:29:53 | opendevreview | Stephen Finucane proposed openstack/placement master: Mandate policy scope enforcement https://review.opendev.org/c/openstack/placement/+/994892 | |
| 18:25:50 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 18:28:45 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 18:33:39 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 18:33:40 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/nova/+/994950 | |
| 23:26:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Support launchSecurity element for SEV-SNP https://review.opendev.org/c/openstack/nova/+/994920 | |
| 23:26:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |
| 23:26:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 23:26:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 23:42:24 | opendevreview | Merged openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | |
| #openstack-nova - 2026-06-26 | |||
| 01:59:40 | gmaan | stephenfin: melwitt nova change for enforce_scope testing also ready https://review.opendev.org/c/openstack/nova/+/986946 | |
| 10:44:09 | opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | |
| 11:42:06 | chandank` | sean-k-mooney: Hello, Are you still reviewing the nvme spec? Reason I am asking because I need to update the cleanup state diagram and few nit pics. Or I will wait for your feedback. | |
| 11:49:46 | chandank` | sorry wrong channel | |
| 12:42:11 | opendevreview | Udayendu Kar proposed openstack/python-novaclient master: tests: skip osprofiler test when dependency is not installed https://review.opendev.org/c/openstack/python-novaclient/+/995048 | |
| 13:40:57 | opendevreview | mitya-eremeev-2 proposed openstack/nova stable/2025.1: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/995063 | |
| 13:41:14 | stephenfin | gibi: sean-k-mooney: Could one of you sanity check this for me? https://review.opendev.org/c/openstack/python-openstackclient/+/995003/ | |
| 13:41:52 | stephenfin | I assume it's fine but I've never done anything dual-stack'y like that | |
| 13:43:17 | sean-k-mooney | you can have both if you have subnets for both on the same network | |
| 13:43:28 | sean-k-mooney | but ill take a look now | |
| 13:44:15 | sean-k-mooney | i am not sure if we can spcify both in nova api but if you pass a newuton port or network the the port can be assigne both | |
| 13:45:21 | sean-k-mooney | stephenfin: hum so accordign to the api ref | |
| 13:45:22 | gibi | I will rely on sean-k-mooney's eyes as I also never did dual stack | |
| 13:45:23 | sean-k-mooney | networks.fixed_ip | |
| 13:45:27 | sean-k-mooney | is ipv4 only | |
| 13:45:33 | sean-k-mooney | A fixed IPv4 address for the NIC. Valid with a neutron or nova-networks network. | |
| 13:46:14 | sean-k-mooney | https://docs.openstack.org/api-ref/compute/#id11 | |
| 13:47:02 | sean-k-mooney | i will have to check the code but lookign at the fixed_ip filed there i dont see evnicne we have ever supprot ipv6 in server create | |
| 13:47:16 | sean-k-mooney | i.e where we provide an ipv6 adress to use | |
| 13:47:48 | sean-k-mooney | it will owrk if you pass port or netork that has an ipv6 subnet | |