| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-04 | |||
| 13:18:04 | sean-k-mooney | ideally we woudl be able to express that on the nodeset definitoin "node_must_be_routeable" or somethign like that, i assuem we cant assuem all nodes have ipv6 yet | |
| 13:18:43 | sean-k-mooney | if we coudl we coudl have the exisitng multi node role that create the tunnel messh do that over ipv6 and then use the ipv4 addreses it provides for our jobs | |
| 13:19:14 | sean-k-mooney | i.e. instead of using the ip form the default route use the one for the multi node bridge | |
| 13:19:47 | sean-k-mooney | im referint ot https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-bridge just ot be clear | |
| 13:20:35 | sean-k-mooney | if we could rely on that mesh working across providers we woudl not have to care for the most part as that vxlan tunnle should resolve any firewall issues we have | |
| 13:21:05 | sean-k-mooney | although runing ceph rbd over that likely wotn be a fun time | |
| 13:21:55 | sean-k-mooney | lets just see how things go once that patch lands | |
| 13:24:35 | fungi | another option is an early acceptance check in the job to confirm the nodes are able to reach one another satisfactorally over the interfaces it wants them to use, and then the nodeset can be discarded and the build automatically retried if not. we do something similar in a common role to make sure nodes are able to reach internet resources, for example | |
| 13:28:03 | sean-k-mooney | ya thats not a bad idea we coudl do it as a pre-playbook that would alow the job to potetally rescdhule to a diffetn node set | |
| 13:28:26 | sean-k-mooney | thats honestly the best of both worlds as we may be able to avoid recheckign | |
| 13:28:56 | sean-k-mooney | and if we do it as a role we coudl selectivly add it to the jobs that actully need it | |
| 13:29:25 | sean-k-mooney | that may end up beign all the multi node jobs but perhaps not | |
| 14:17:18 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2025.2 Flamingo https://review.opendev.org/c/openstack/nova/+/944603 | |
| 14:19:52 | gboutry | Hello nova, is there anything else required from this review (a part from reviews): https://review.opendev.org/c/openstack/nova/+/953737? I'd be quite interested for the backport in 2025.1 | |
| 14:27:34 | gibi | gboutry: no, I think we just waiting for reviews there | |
| 14:28:11 | gibi | but the US is out today and our resident stable branch reviewer elodilles is on PTO | |
| 14:28:28 | gibi | bauzas: sean-k-mooney: ^^ could you hit https://review.opendev.org/c/openstack/nova/+/953737 | |
| 14:28:46 | sean-k-mooney | oh the backport sure | |
| 14:30:12 | sean-k-mooney | done | |
| 14:30:36 | sean-k-mooney | stephenfin: ^ if your still here could you also look at that | |
| 14:30:53 | gibi | thanks | |
| 14:49:46 | gboutry | Thank you very much! | |
| 15:11:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pci]Keep used dev in Placement regardless of dev_spec https://review.opendev.org/c/openstack/nova/+/954149 | |
| 15:14:06 | sean-k-mooney | gibi: so i did somethign similr to that for the pci device table a long time ago | |
| 15:14:34 | sean-k-mooney | gibi: is that only for the placment side or was there a regression on the device table as well | |
| 15:14:49 | gibi | it is only the PCI in Placement isde | |
| 15:14:50 | gibi | sdie | |
| 15:14:52 | gibi | side | |
| 15:14:55 | sean-k-mooney | ack | |
| 15:15:08 | sean-k-mooney | i guess we missed that edge case so in the orginal implemeation | |
| 15:15:36 | gibi | btw the pci tracker side is also a bit buggy as after the inconsistency handled by deleting the VM the device that is not in the devspec any more is not marked deleted until the compute is restarted again | |
| 15:16:13 | gibi | we had a bug in the edge case handling and we have an edge case of an edge case during VM deletion that was not really handled well | |
| 15:16:14 | sean-k-mooney | that becasue of the cacheing we have i think | |
| 15:16:31 | gibi | yepp, we don't re-read the hypervisor view and remove the dev | |
| 15:16:41 | sean-k-mooney | right | |
| 15:16:54 | sean-k-mooney | so the actual correct behavior is to refuse to start the comptue at all | |
| 15:17:08 | sean-k-mooney | i.e. if a deivce is in use by a vm and you remvoe it form the dev spec | |
| 15:17:16 | sean-k-mooney | that shoudl be a hard error | |
| 15:17:18 | gibi | we can change to that but that is a hard block if one PCI device died | |
| 15:17:29 | sean-k-mooney | but we didnt want to do that when we were fixing the orginal bug | |
| 15:18:00 | gibi | I have to drop for the weekend but I have to go back to the bugfix to do the proper tests anyhow next week | |
| 15:18:12 | gibi | o/ | |
| 15:18:15 | sean-k-mooney | gibi: yep although if we chang this now we probaly shoudl add a workaroudn flag | |
| 15:18:22 | sean-k-mooney | gibi: o/ | |
| 15:19:54 | gmaan | sean-k-mooney: you asked me to remind about manager role series review, whenever you have time https://review.opendev.org/q/topic:%22bp/policy-manager-role-default%22+status:open | |
| 15:20:30 | gmaan | I migrated all existing migration (live and cold) tempest tests to use manager role, changes are linked in same topic | |
| 15:21:16 | gmaan | one tempest change adding abort/force complete test is in progress which I am working in parallel but those things are covered in nova side tests also but I just want to write tempest test also if we can | |
| 15:22:12 | sean-k-mooney | cool, im trying to start wrappign up for the week so i likely wont get to it till monday | |
| 15:22:58 | gmaan | no issue, thanks | |
| 15:23:40 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/953847/6/tempest/lib/api_schema/response/compute/v2_1/servers.py so we just didnt have tempest schema for that before? | |
| 15:24:14 | sean-k-mooney | i guess tempest never tested those api in the past then? | |
| 15:24:48 | sean-k-mooney | i feel like any test that try to use abort/force complete woudl eb racy | |
| 15:25:29 | sean-k-mooney | unless you plannig to add somthign to slow down the migration by creating memory load | |
| 15:25:43 | sean-k-mooney | btu even then that seam hard to properly test | |
| 15:26:35 | gmaan | yeah, we did not had tests/schema for list migration (in-progress live migration list), tempest has migration list (list all migration) schema though | |
| 15:27:27 | gmaan | yeah, I am trying in try block if migration is still going on then perform the force complet/abort | |
| 15:27:49 | gmaan | I know that is best we can test and log if test is not able to perform the operation | |
| 15:28:28 | gmaan | I am not 100% convince if that will help or make a good tempest test but in some cases it can perform the things it want | |
| 15:31:23 | gmaan | for abort, it is like if test able to perform the abort migration it pass otherwise skip test saying 'abort is not performed bcz migration was completed before that' | |
| 18:00:26 | opendevreview | Merged openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| #openstack-nova - 2025-07-05 | |||
| 08:24:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utf8 utility https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:28:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use oslo.utils to escape IPv6 address https://review.opendev.org/c/openstack/nova/+/954175 | |
| 08:35:01 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:35:59 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:37:33 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:39:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:41:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 08:47:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate ExceptionHelper to test utilities https://review.opendev.org/c/openstack/nova/+/954176 | |
| 08:47:57 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate ExceptionHelper to test utilities https://review.opendev.org/c/openstack/nova/+/954176 | |
| 11:13:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use oslo.utils to escape IPv6 address https://review.opendev.org/c/openstack/nova/+/954175 | |
| 11:31:54 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use oslo.utils to escape IPv6 address https://review.opendev.org/c/openstack/nova/+/954175 | |
| 13:05:39 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| #openstack-nova - 2025-07-06 | |||
| 06:00:14 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| 06:17:44 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| 07:02:37 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| 20:20:38 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 20:20:39 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 20:20:39 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 20:20:40 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| #openstack-nova - 2025-07-07 | |||
| 01:39:49 | masahito | sean-k-mooney: late ack. Thanks. yes, I'm thinking to push the PoC code early as much as possible and trying to go the features in the early next cycle. | |
| 02:25:55 | mikal | Oh nice, because rebasing a patch series now involves adding a signed off header, it also invalidates all votes against that review, even if the code hasn't changed. | |
| 04:52:25 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 04:52:26 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 04:52:26 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 04:52:27 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 07:53:01 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 11:12:09 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 14:43:39 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 15:11:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pci]Keep used dev in Placement regardless of dev_spec https://review.opendev.org/c/openstack/nova/+/954149 | |
| 15:50:00 | opendevreview | Rajesh Tailor proposed openstack/nova master: Clarify that [pci]device_spec={} matches all devices https://review.opendev.org/c/openstack/nova/+/954274 | |
| 18:01:05 | fungi | sean-k-mooney: https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/2069607 was finally switched to public on friday, and contains the rationale behind cloud-init's decision to break that functionality we were discussing | |
| 19:29:19 | sean-k-mooney | fungi: ack ill skim it but the problem is that even on x86_64 using dmi is not a correct way to determin if the openstack data souce shoudl be enabled | |
| 19:29:55 | mikal | How does AWS handle this given they use exactly the same IP mechanism (we copied from them)? | |
| 19:30:05 | sean-k-mooney | for it to be somethign they could rely on it would have to be somethign we suport for all virt driever includign ironic | |
| 19:30:25 | sean-k-mooney | mikal: it handeled by you confiruting the backedn in the image | |
| 19:30:35 | sean-k-mooney | you are ment ot have custom images per cloud | |
| 19:30:52 | sean-k-mooney | so the uec images are the amazon flavor | |
| 19:30:54 | mikal | Also, its not strictly true that someone can man in the middle that IP address, assuming they're actually on OpenStack. We intercept that traffic and special case it. | |
| 19:31:18 | sean-k-mooney | well the 169 address were chosen because those are not allowed to be routed | |
| 19:31:22 | fungi | yeah, where this presents a challenge is e.g. distros producing official bootable cloud images that are intended for use in multiple environments | |