| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-10 | |||
| 13:02:48 | sean-k-mooney | anyway nova zeros out the root disk storage form the flavor for bfv guest in palcement so if you wanted too you shoudl be able ot sue that to bill more acurratly until a new policy rule is intorduced | |
| 13:03:20 | sean-k-mooney | zigo: would you mind crating a blueprint for this or adding a ptg topic so we have somethign to dicuss and track this request | |
| 13:04:13 | sean-k-mooney | i dont know if other woudl want a small spec for this but we can dicuss that in the irc meeting once we have a blueprint. | |
| 13:28:01 | zigo | sean-k-mooney: Will do, thanks for your input ! | |
| 14:36:23 | dansmith | gibi: so on the the s-g parallelize thing.. in fixing the last few tests, I think I've realized that the accounting for the "batches" test(s) was so off because of the sequential processing | |
| 14:36:36 | dansmith | sort of a "yeah, duh" moment | |
| 14:38:55 | opendevreview | Merged openstack/nova master: Fix bug 2114951 https://review.opendev.org/c/openstack/nova/+/952895 | |
| 14:48:06 | gibi | dansmith: :) | |
| 14:50:33 | dansmith | the downside is that the behavior becomes a little less deterministic (as expected) so I'm having to change assertions a bit | |
| 15:29:30 | sean-k-mooney | gibi: so lookign at https://docs.openstack.org/project-team-guide/release-management.html#milestone-3-feature-freeze the requirements freeze starts at m3 and is unfozent on master after the stable branch of the requirement repos created at https://docs.openstack.org/project-team-guide/release-management.html#release-2 | |
| 15:31:22 | sean-k-mooney | gibi: so taht woudl imply its too late to rolback pyroute2 however i have no idea regarding your other question fo can this happen on stable/2025.2, my gut feeling is no becuase it could break released packages based on rc1 or the ga release, or at the very least cause the pyroute2 package to need a downgrade | |
| 15:32:11 | sean-k-mooney | with that said im not sure so thanks for sending that email again | |
| 16:14:09 | gibi | yeah I feel we need to roll forward once we have a pyroute2 fix both on master and stable/2025.2 I hope rolling forward will be OK on stable | |
| 16:32:56 | opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Drop GLANCE_STANDALONE https://review.opendev.org/c/openstack/nova/+/960404 | |
| 17:00:33 | sean-k-mooney | gibi: my understanding is we are not allowed to raise minversion but we can increase max | |
| 17:01:04 | sean-k-mooney | so it shoudl be ok as long as theyre is no breaking change | |
| 17:01:41 | sean-k-mooney | its not something i have seen us do often however so im not very familar with the critia that is applied | |
| 18:30:20 | opendevreview | sean mooney proposed openstack/nova stable/2025.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960409 | |
| 18:34:11 | opendevreview | sean mooney proposed openstack/nova stable/2024.2: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960410 | |
| 18:35:00 | opendevreview | sean mooney proposed openstack/nova stable/2024.1: hypervisors: Optimize uptime retrieval for better performance https://review.opendev.org/c/openstack/nova/+/960411 | |
| 21:12:24 | opendevreview | Cyril Roelandt proposed openstack/nova master: DNM: Drop GLANCE_STANDALONE https://review.opendev.org/c/openstack/nova/+/960404 | |
| #openstack-nova - 2025-09-11 | |||
| 09:03:17 | opendevreview | Merged openstack/nova master: Add Flamingo prelude section https://review.opendev.org/c/openstack/nova/+/959188 | |
| 09:32:32 | opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Test patch removing glance standalone https://review.opendev.org/c/openstack/nova/+/960446 | |
| 12:32:38 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960507 | |
| 12:32:40 | opendevreview | OpenStack Release Bot proposed openstack/placement stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960508 | |
| 12:32:41 | opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960509 | |
| 12:34:15 | opendevreview | Thomas Goirand proposed openstack/nova-specs master: Restrict booting from volume with disk https://review.opendev.org/c/openstack/nova-specs/+/960513 | |
| 12:34:38 | zigo | sean-k-mooney: It's the first time I'm writing a spec, I hope it's ok... | |
| 13:09:42 | jlejeune | sean-k-mooney: gibi: Hello, I recently resumed my work on bug #2044235. You asked me to create a regression test to confirm the bug, but I'm having trouble reproducing it. When I raise the messagingTimeout exception from the ‘check_can_live_migrate_source’ method, it seems like this exception is caught by the fake transport driver, causing my test to fail. When I try with any other exception, | |
| 13:09:48 | jlejeune | my test works fine. Have you ever seen this behavior before? | |
| 13:11:50 | sean-k-mooney | zigo: you removed section instead of jsut say None so that is why pep8 is failing | |
| 13:12:08 | zigo | Ok, will re-add it. | |
| 13:12:43 | sean-k-mooney | zigo: ill take a look later but thansk for creating one. we can discuss in the spec what the default for the policy shoudl be | |
| 13:12:58 | sean-k-mooney | my guess is member but we may decied to change it in the cycle after to admin | |
| 13:15:16 | sean-k-mooney | zigo: the approch we normally take is add a new policy maintianing the same beahvior then update it after, however that in sope to dicuss in the spec | |
| 13:15:31 | sean-k-mooney | basiclly do we have an upgrade impact or not | |
| 13:16:00 | opendevreview | Thomas Goirand proposed openstack/nova-specs master: Restrict booting from volume with disk https://review.opendev.org/c/openstack/nova-specs/+/960513 | |
| 13:16:11 | sean-k-mooney | my expection is if we changed it to admin it would break tempest tests as we dont use speical falvors with 0 disk to test BFV in general | |
| 13:16:32 | sean-k-mooney | or at least i dont think we do that exclusively | |
| 14:14:33 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | |
| 14:31:36 | sean-k-mooney | jlejeune: sorry i am in a few converstaion at once. i see you pushed the tst so i can try and reveiw that patch and play with it loccally | |
| 14:33:00 | jlejeune | sean-k-mooney: no worry, I saw that you're in discussion with zigo | |
| 14:33:26 | sean-k-mooney | and on slack and oslo and watcher channel | |
| 14:33:42 | jlejeune | :) | |
| 14:34:09 | jlejeune | you're such a busy man :) | |
| 14:34:33 | sean-k-mooney | that or i have the attention span fo a magpie | |
| 14:35:26 | jlejeune | ^^ | |
| 14:35:46 | sean-k-mooney | so to your question atbout messagingTimeout it might be because of how we mock our messaage bus using the olso in memory oslo.messaging driver | |
| 14:35:58 | jlejeune | exactly | |
| 14:36:33 | sean-k-mooney | we sometimes need to disable case as call to get this to work as well | |
| 14:37:11 | jlejeune | I didn't find any example of that, do you have some in mind ? | |
| 14:37:54 | sean-k-mooney | no but i think we have or had example of causing live migration to fail in other tests | |
| 14:38:53 | sean-k-mooney | im not sure if your familar with the CAST_AS_CALL fixture as the name suggest it turn rpc casts which are fire and forget into a sync call that run in place and returns only when its complete | |
| 14:39:10 | sean-k-mooney | i dont think that is the problem here | |
| 14:39:31 | sean-k-mooney | but it can cause slightly diffent rpc behavior in teh regress test and sometime we have to turn it off | |
| 14:40:32 | jlejeune | yes I saw that flag and I tried to turn it off, but without any success | |
| 14:52:00 | sean-k-mooney | so we do have some regression tests for live migration roolback https://github.com/openstack/nova/blob/759e03c35d91dac4a04237d58493665c01952c9f/nova/tests/functional/regressions/test_bug_1848343.py#L105 | |
| 14:53:35 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | |
| 14:55:44 | sean-k-mooney | im wonderign if there is a bette way to similate the failrue | |
| 14:58:03 | sean-k-mooney | ah you now just raising a generic excption | |
| 15:01:19 | jlejeune | yes because I can't raise the MessagingTimeout exception, if I do so, it's caught by the secheduler when it tries to find a destination host | |
| 15:02:54 | jlejeune | but after thinking about the issue, for my point of view, it's not very a big deal to catch all exceptions coming from that rpc call during the pre live migration check, what do you think about it ? | |
| 15:09:43 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | |
| 16:17:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 16:32:31 | sean-k-mooney | jlejeune: am it depend we do have some logic that may care but lets look at it in the fix patch | |
| 16:33:01 | sean-k-mooney | jlejeune: we try to avoid `except Exception` in gernal | |
| 16:33:10 | sean-k-mooney | because it can hide effort | |
| 16:33:21 | sean-k-mooney | *errors | |
| 16:52:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 18:27:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 18:47:00 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 19:00:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 19:37:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 20:02:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| #openstack-nova - 2025-09-12 | |||
| 06:59:04 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 07:05:55 | jlejeune | sean-k-mooney: hello, yeah I know that catch 'Exception' is not necessarily a good idea, but in that case if I only put the try/except statement around the 'check_can_live_migrate_source' call, it will catch every exceptions which may happen during that specific rpc call | |
| 07:09:19 | ratailor | elodilles, could you please review when you have time https://review.opendev.org/c/openstack/nova/+/958834 and https://review.opendev.org/q/owner:ratailor@redhat.com+branch:stable/2024.2 | |
| 08:17:48 | opendevreview | Julien LE JEUNE proposed openstack/nova master: nova-condutor puts instance in error state https://review.opendev.org/c/openstack/nova/+/901655 | |
| 08:37:32 | elodilles | ratailor: ACK, added them to my TODO list | |
| 08:41:29 | opendevreview | Merged openstack/placement stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960507 | |
| 08:41:31 | opendevreview | Merged openstack/placement stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960508 | |
| 08:54:02 | opendevreview | OpenStack Release Bot proposed openstack/nova stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/nova/+/960743 | |
| 08:54:07 | opendevreview | OpenStack Release Bot proposed openstack/nova stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/nova/+/960744 | |
| 08:54:12 | opendevreview | OpenStack Release Bot proposed openstack/nova master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/nova/+/960745 | |
| 09:01:30 | gibi | sean-k-mooney: dansmith: I have the first comparable rally results between eventlet and native threading. See my top level comment in https://review.opendev.org/c/openstack/nova/+/960130/16#message-e6a461d92172aea570e8f18405d47ee00b9c300a It links to both rally results with timing and I manually pulled out the pool usage and the memory usage of the scheduler (I have the intention to script some of it | |
| 09:01:36 | gibi | in CI) | |
| 09:49:23 | ratailor | elodilles, Thanks! | |
| 09:52:52 | opendevreview | Taketani Ryo proposed openstack/nova-specs master: Add a spec for 2026.1 for libvirt launching Arm CCA instances https://review.opendev.org/c/openstack/nova-specs/+/960777 | |
| 09:54:36 | sean-k-mooney | gibi do you happen ot know if they ran on the same provider | |
| 09:54:59 | sean-k-mooney | gibi: i.e. are the host vms comparible | |
| 09:58:06 | gibi | the two rally execution happen within the same CI job on the same CI worker. So it is always comparable :) | |
| 09:59:02 | gibi | the base job runs rally then in the post-run I reconfigure to native threading and run rally again | |
| 09:59:58 | sean-k-mooney | yep just reading the comment on the patch | |
| 10:00:17 | sean-k-mooney | so that is good because it elimiates the biggest variabel in our ci | |
| 10:00:26 | sean-k-mooney | run to run hardware deltas | |
| 10:00:37 | sean-k-mooney | looking atht eh results this is very promising | |
| 10:01:25 | sean-k-mooney | gibi: the only metric i woul have liket to see that i have not seen yet is the total cpu time of the nova process | |
| 10:01:48 | sean-k-mooney | i.e. is there any delter on cpu load with eventlet vs threading | |
| 10:02:18 | gibi | sean-k-mooney: how can we collect that? | |
| 10:02:25 | sean-k-mooney | in theory there shoudl not be since the eventlet shcdluer vs kernel shclduer selecting the next thread shoudl eb more or less a wash | |