| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-24 | |||
| 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 | |
| 13:48:11 | sean-k-mooney | but we dont currently allow you to specify a subnet to create the prot on or pass an ipv6 ip | |
| 13:49:08 | sean-k-mooney | so either the api ref is wrong (*that is possibel) or ipv6 is not suprpot in genral so the issue is not about dualstack or not at that point | |
| 13:51:57 | sean-k-mooney | stephenfin: ok so looking at the openapi schmea you wote | |
| 13:52:17 | sean-k-mooney | teh api accpate networks.fixed_ip but we allow that to be an ipv4 or v6 adress | |
| 13:52:44 | sean-k-mooney | at least in the schema | |
| 13:53:25 | sean-k-mooney | but we only support one of fixed_ip prot or uuid per network | |
| 13:53:40 | sean-k-mooney | meainign you cant pass both for the same port | |
| 13:53:59 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L283-L299 | |
| 13:54:39 | stephenfin | I wonder can we test it somehow just to see what happens? | |
| 13:54:51 | stephenfin | I imagine deploying dual-stack devstack is not a trivial matter 😅 | |
| 13:55:15 | sean-k-mooney | i dont think its that terirbel you just need to create a newton networ with both | |
| 13:55:20 | sean-k-mooney | the host does not need it | |
| 13:55:31 | sean-k-mooney | but i can see if i have a devstack i can test it on quickly | |
| 13:59:44 | sean-k-mooney | i think the failure mode is we dont accpet muleipel fixed ip pre netowrk object in our api. we could be i think that is why that is there | |
| 13:59:54 | sean-k-mooney | so it snot someabout v4 or v6 | |
| 13:59:56 | frickler | I tested it locally and indeed the v6 address does not get set | |
| 14:00:18 | frickler | while without a fixed-ip specified, I do get both addr types | |
| 14:00:18 | sean-k-mooney | its more that we only supprot 1 adress per port that we create | |
| 14:00:40 | sean-k-mooney | frickler: ya it woudl change the subnet we attach too | |
| 14:01:15 | sean-k-mooney | i think before we make the client change we woudl need a new api microverion but in the past we said if you neded somethign like ethat you shoudl precreate the neutorn port | |
| 14:01:25 | sean-k-mooney | basiclly because we didnt wnat to keep ading proxy apis for that | |
| 14:01:54 | sean-k-mooney | frickler: i assume you created the vm with curl? | |
| 14:02:12 | sean-k-mooney | or did you just use the propsoed patch? | |
| 14:03:51 | frickler | I did use the patch. against a 2024.1 admittedly | |
| 14:03:55 | frickler | *cloud | |
| 14:04:27 | sean-k-mooney | devstack default to a dual stack deployment https://paste.opendev.org/show/bq6cSQIBviTCgXyGkvA7/ so il test it again on that deployment which si masteris with the patch | |
| 14:20:35 | sean-k-mooney | frickler: so what im seeitn tiwth the patch applied is regardless of the oder only the v4 adress gets applied | |