| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-24 | |||
| 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 | |
| 14:20:47 | sean-k-mooney | https://paste.opendev.org/show/bzPsu7xQOJ0oc0fzJzSs/ | |
| 14:21:17 | sean-k-mooney | although the vms get both v4 and v6 adresses the v6 adress is just not the one i specified | |
| 14:24:12 | sean-k-mooney | looking at the reques boday this is what is being sent https://paste.opendev.org/show/baVOCdYizNOYiccHJmWe/ | |
| 14:24:27 | sean-k-mooney | stephenfin: so the patch does not actlly work | |
| 14:25:12 | sean-k-mooney | it remvoes the check but the client is still only seingi the ipv4 adress becuase i think we check the ipv4 first in the code and then dont check the ipv6 adress | |
| 14:33:21 | sean-k-mooney | stephenfin: i marked https://bugs.launchpad.net/python-openstackclient/+bug/2152032 as invlid feel free to change that if you think that is wrong | |
| 14:34:54 | sean-k-mooney | frickler: thanks for checking :) | |
| 14:37:39 | stephenfin | no, that's great. thanks for looking into it :) frickler++ sean-k-mooney++ | |
| 14:37:41 | opendevreview | Sylvain Desgrais proposed openstack/nova master: Allow security group names with leading/trailing spaces https://review.opendev.org/c/openstack/nova/+/995073 | |
| 14:37:42 | frickler | well ... if nova accepts both v4 and v6 in the API, but then ignores one of those, arguably that's a bug that could/should get fixed an not an API change? | |
| 14:38:59 | frickler | using a preconfigured port works, but has more drawbacks, too. like security groups also need to get applied manually to the port and are ignored during the server create call | |
| 14:39:21 | opendevreview | Sylvain Desgrais proposed openstack/nova master: Allow security group names with leading/trailing spaces https://review.opendev.org/c/openstack/nova/+/995073 | |
| 14:49:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Limit the port fields during security_group query https://review.opendev.org/c/openstack/nova/+/994896 | |
| 14:57:43 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Support launchSecurity element for SEV-SNP https://review.opendev.org/c/openstack/nova/+/994920 | |
| 14:57:43 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 14:57:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |
| 14:57:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | |
| 14:57:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 15:21:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | |