| *** mhen_ is now known as mhen | 01:57 | |
| *** ykarel_ is now known as ykarel | 07:00 | |
| opendevreview | Merged openstack/nova master: Add Flamingo prelude section https://review.opendev.org/c/openstack/nova/+/959188 | 09:03 |
|---|---|---|
| opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Test patch removing glance standalone https://review.opendev.org/c/openstack/nova/+/960446 | 09:32 |
| 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 |
| 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 |
| opendevreview | OpenStack Release Bot proposed openstack/placement master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/placement/+/960509 | 12:32 |
| 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 |
| zigo | sean-k-mooney: It's the first time I'm writing a spec, I hope it's ok... | 12:34 |
| 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 |
| jlejeune | my test works fine. Have you ever seen this behavior before? | 13:09 |
| *** iurygregory_ is now known as iurygregory | 13:10 | |
| sean-k-mooney | zigo: you removed section instead of jsut say None so that is why pep8 is failing | 13:11 |
| zigo | Ok, will re-add it. | 13:12 |
| 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 |
| sean-k-mooney | my guess is member but we may decied to change it in the cycle after to admin | 13:12 |
| 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 |
| sean-k-mooney | basiclly do we have an upgrade impact or not | 13:15 |
| 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 |
| 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 |
| sean-k-mooney | or at least i dont think we do that exclusively | 13:16 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | 14:14 |
| 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:31 |
| jlejeune | sean-k-mooney: no worry, I saw that you're in discussion with zigo | 14:33 |
| sean-k-mooney | and on slack and oslo and watcher channel | 14:33 |
| jlejeune | :) | 14:33 |
| jlejeune | you're such a busy man :) | 14:34 |
| sean-k-mooney | that or i have the attention span fo a magpie | 14:34 |
| jlejeune | ^^ | 14:35 |
| 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 |
| jlejeune | exactly | 14:35 |
| sean-k-mooney | we sometimes need to disable case as call to get this to work as well | 14:36 |
| jlejeune | I didn't find any example of that, do you have some in mind ? | 14:37 |
| sean-k-mooney | no but i think we have or had example of causing live migration to fail in other tests | 14:37 |
| 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:38 |
| sean-k-mooney | i dont think that is the problem here | 14:39 |
| sean-k-mooney | but it can cause slightly diffent rpc behavior in teh regress test and sometime we have to turn it off | 14:39 |
| jlejeune | yes I saw that flag and I tried to turn it off, but without any success | 14:40 |
| 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:52 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | 14:53 |
| sean-k-mooney | im wonderign if there is a bette way to similate the failrue | 14:55 |
| sean-k-mooney | ah you now just raising a generic excption | 14:58 |
| 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:01 |
| 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:02 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/960349 | 15:09 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 16:17 |
| 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:32 |
| sean-k-mooney | jlejeune: we try to avoid `except Exception` in gernal | 16:33 |
| sean-k-mooney | because it can hide effort | 16:33 |
| sean-k-mooney | *errors | 16:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 16:52 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 18:27 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | 18:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 19:00 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 19:37 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | 20:02 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!