| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-26 | |||
| 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 | sean-k-mooney | its more that we only supprot 1 adress per port that we create | |
| 14:00:18 | frickler | while without a fixed-ip specified, I do get both addr types | |
| 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: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 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: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: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: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 | |
| 15:21:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: AMD SEV: omit iommu='on' for virtio devices https://review.opendev.org/c/openstack/nova/+/909635 | |
| 15:21:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 15:51:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 16:13:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 17:22:27 | opendevreview | Merged openstack/nova master: Add regression test for bug 2062127 https://review.opendev.org/c/openstack/nova/+/976654 | |
| 17:44:33 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 17:44:33 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 18:02:13 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 18:02:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 19:06:42 | opendevreview | Adam Harwell proposed openstack/placement master: Add ASGI application entry point for Placement API https://review.opendev.org/c/openstack/placement/+/995129 | |
| 19:10:38 | opendevreview | Adam Harwell proposed openstack/placement master: Add ASGI application entry point for Placement API https://review.opendev.org/c/openstack/placement/+/995133 | |
| 19:11:06 | rm_work[m] | erk | |
| 19:54:10 | sean-k-mooney | frickler: osc is not sending both. nova will accpet an ip in v4 or v6 format but it wont accpet multiple of them | |
| 19:54:43 | sean-k-mooney | frickler: i have not tested it with curl but i woudl expect a 400 band request if you tried to repoeate the file of make it a list instaed of a stirng | |
| 19:54:49 | sean-k-mooney | in the json payload | |
| 20:02:07 | melwitt | gmaan: question on https://review.opendev.org/c/openstack/nova/+/986946 let me know if I am wrong :) | |
| 20:17:21 | melwitt | sean-k-mooney: why did you +2 this? :) it's phrased as if it's an addition to someone else's patch but I don't see other related patches? oh, it's that you requested testing in the patch above and added it below https://review.opendev.org/c/openstack/nova/+/994437 | |
| 20:19:05 | gmaan | melwitt: checking, I am not sure why that test does not exist in base test class and only in scope testing | |
| 20:19:41 | melwitt | yeah, it was different than everything else seems like | |
| 20:21:14 | opendevreview | Adam Harwell proposed openstack/placement master: Add ASGI application entry point for Placement API https://review.opendev.org/c/openstack/placement/+/995133 | |
| 20:27:00 | gmaan | melwitt: nice catch, replied the reason. it was added when policy was SYSTEM_READER and we did not move it to base class when removed the system scope from our policy | |
| 20:27:04 | gmaan | I will fix it | |
| 20:27:32 | melwitt | ah gotcha. cool I will look again after you update | |
| 20:38:10 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 20:40:55 | opendevreview | Sylvain Desgrais proposed openstack/nova master: Allow security group names with leading/trailing spaces https://review.opendev.org/c/openstack/nova/+/995073 | |
| 21:43:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 22:29:59 | opendevreview | Merged openstack/nova master: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/994437 | |
| 22:35:02 | opendevreview | Merged openstack/nova master: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/936093 | |
| #openstack-nova - 2026-06-27 | |||
| 00:57:50 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 01:04:52 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for remote managed ports https://review.opendev.org/c/openstack/nova/+/995160 | |
| 01:27:02 | opendevreview | Adam Harwell proposed openstack/nova master: Add ASGI application entry points for Nova API https://review.opendev.org/c/openstack/nova/+/995163 | |
| 01:29:28 | opendevreview | Adam Harwell proposed openstack/nova master: Add ASGI application entry points for Nova API https://review.opendev.org/c/openstack/nova/+/995163 | |
| 01:58:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 01:58:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 01:58:24 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 03:18:52 | opendevreview | Merged openstack/nova master: libvirt: Fix logging type error https://review.opendev.org/c/openstack/nova/+/978432 | |
| 03:34:28 | opendevreview | Adam Harwell proposed openstack/placement master: Add ASGI application entry point for Placement API https://review.opendev.org/c/openstack/placement/+/995133 | |
| 03:34:59 | opendevreview | Adam Harwell proposed openstack/nova master: Add ASGI application entry points for Nova API https://review.opendev.org/c/openstack/nova/+/995163 | |
| 03:51:41 | opendevreview | Merged openstack/nova master: pyproject: Update packaging configuration https://review.opendev.org/c/openstack/nova/+/988598 | |
| 04:23:50 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 04:23:51 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 04:23:51 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 08:26:13 | frickler | hmm, if I try server create with just IPv6, I get a schema error (I think from OSC). also the api-ref is confusing | |
| 08:42:29 | sean-k-mooney | the api ref asys only ipv4 is allowd. i dont know that we ever inteantionl change the api to allow ipv6 but it may have happended as a sideffect of other work | |
| 08:43:21 | sean-k-mooney | whenever ipv6 became allowed the api ref and presumable the test for that were not updated to reflect that | |
| 08:44:07 | sean-k-mooney | in this case im refering to api sample test to cover passing an ipv6 adress ectra | |
| 12:08:11 | opendevreview | Merged openstack/nova master: utils: Pass string to lock https://review.opendev.org/c/openstack/nova/+/978433 | |
| 12:16:57 | opendevreview | Merged openstack/nova master: privsep: Pass strings, not ints https://review.opendev.org/c/openstack/nova/+/978434 | |
| 12:28:08 | opendevreview | Merged openstack/nova master: privsep: Correct arg order https://review.opendev.org/c/openstack/nova/+/978435 | |
| 12:40:21 | opendevreview | Merged openstack/nova master: privsep: Remove dead code https://review.opendev.org/c/openstack/nova/+/977111 | |
| 16:56:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 16:56:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-SNP memory encryption https://review.opendev.org/c/openstack/nova/+/994930 | |
| 16:56:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 17:19:15 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for remote managed ports https://review.opendev.org/c/openstack/nova/+/995160 | |
| #openstack-nova - 2026-06-28 | |||
| 05:13:12 | opendevreview | Merged openstack/nova master: Remove more dead code https://review.opendev.org/c/openstack/nova/+/978436 | |
| 11:47:34 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused 'shutdown_terminate' flag https://review.opendev.org/c/openstack/nova/+/773156 | |
| 13:26:00 | opendevreview | Merged openstack/nova master: Avoid stacktrace at CLI exit https://review.opendev.org/c/openstack/nova/+/993153 | |
| 13:26:13 | opendevreview | Merged openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | |
| 13:26:27 | opendevreview | Merged openstack/nova master: Fix wrong mock for properties https://review.opendev.org/c/openstack/nova/+/994505 | |
| 13:26:41 | opendevreview | Merged openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | |
| 13:26:55 | opendevreview | Merged openstack/nova master: Add regression test for bug #2153425 https://review.opendev.org/c/openstack/nova/+/988777 | |
| 13:27:10 | opendevreview | Merged openstack/nova master: live-migration: Ensure limits are set before _call_livem_checks_on_host https://review.opendev.org/c/openstack/nova/+/990212 | |
| 13:27:24 | opendevreview | Merged openstack/nova master: [novnc]Only log traffic if debug=True https://review.opendev.org/c/openstack/nova/+/994320 | |
| 13:49:11 | opendevreview | Merged openstack/nova master: compute: Pre-load flavor and system_metadata in init_host https://review.opendev.org/c/openstack/nova/+/977037 | |