| *** mhen_ is now known as mhen | 02:13 | |
| kkaptanoglu | morning, could someone please review this patch https://review.opendev.org/c/openstack/nova/+/1000415 | 06:18 |
|---|---|---|
| opendevreview | Lucas HENRY proposed openstack/nova master: Accept ws and wss as forwarded console origin protocols https://review.opendev.org/c/openstack/nova/+/1014034 | 08:39 |
| opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Add test for MTU overwrite during live migration https://review.opendev.org/c/openstack/nova/+/995803 | 09:11 |
| opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve source MTU during live migration https://review.opendev.org/c/openstack/nova/+/995809 | 09:11 |
| opendevreview | Lajos Katona proposed openstack/nova master: Add regression test to repoduce bug 1854844 https://review.opendev.org/c/openstack/nova/+/991772 | 09:11 |
| opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve queue_size during live migration https://review.opendev.org/c/openstack/nova/+/995841 | 09:11 |
| polyedre | Can someone point me to where is the agenda of the Nova Team meeting from https://meetings.opendev.org/#Nova_Team_Meeting? Current link points to https://wiki.openstack.org/wiki/Meetings/Nova, which does not work | 10:38 |
| opendevreview | Pierre Riteau proposed openstack/nova stable/2026.2: Fix ironic instance rebuild with API >=2.93 https://review.opendev.org/c/openstack/nova/+/1014066 | 11:15 |
| *** ykarel_ is now known as ykarel | 12:49 | |
| abhishekk | greetings o/, need some help in moving this forward, https://review.opendev.org/c/openstack/nova/+/1011470 Thank you!! | 13:45 |
| *** haleyb|out is now known as haleyb | 14:08 | |
| tkajinam | o/ could anyone review this to restore CI coverage ? 997768: Revert "disable ovn based testing of test_live_migration_with_trunk" | https://review.opendev.org/c/openstack/nova/+/997768 | 14:11 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [DEFAULT] vcpu_pin_set https://review.opendev.org/c/openstack/nova/+/996016 | 14:12 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Select the concurrency backend only once https://review.opendev.org/c/openstack/nova/+/1014096 | 14:36 |
| opendevreview | Kamil Sambor proposed openstack/placement master: DNM: Verify nova concurrency backend fix https://review.opendev.org/c/openstack/placement/+/1014100 | 14:42 |
| stephenfin | abhishekk: small docs-related request on that change | 14:50 |
| abhishekk | stephenfin: thank you, i will fix those and upload new set ASAP | 14:51 |
| opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | 15:17 |
| abhishekk | stephenfin: sorry, need to push new PS as changes needed in unit tests as well | 15:32 |
| stephenfin | ack | 15:33 |
| opendevreview | Abhishek Kekane proposed openstack/nova master: Add Glance preferred stores support https://review.opendev.org/c/openstack/nova/+/1011470 | 15:42 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Restrict valid url scheme for console proxy base URL https://review.opendev.org/c/openstack/nova/+/1014119 | 16:05 |
| opendevreview | Leo Jenkins proposed openstack/nova-specs master: Support for Hostname-based Ceph Mon References https://review.opendev.org/c/openstack/nova-specs/+/1000098 | 16:10 |
| melwitt | tkajinam: on test_live_migration_with_trunk, it looks like it's still being skipped in the job run on the proposed patch? https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4c3/openstack/4c3859706ffb4868974d28ba28d2c814/testr_results.html | 16:17 |
| tkajinam | oh, I'll check it but at least LiveMigrationWithoutHostTest.test_live_migration_with_trunk runs and passes iiuc | 16:21 |
| melwitt | ah interesting. I see the skip with tempest.api.compute.admin.test_live_migration.LiveMigrationTest.test_live_migration_with_trunk | 16:22 |
| melwitt | tempest.api.compute.admin.test_live_migration.LiveAutoBlockMigrationV225Test is also skipping. not sure if the test is only valid without host maybe, so it's possible this is expected and I didn't know | 16:24 |
| tkajinam | I guess the expectation from tempest perspective is that it is skipped in all scenarios so LiveMigrationWithoutHostTest running it is unexpected (seeing no override there) | 16:25 |
| melwitt | test_live_migration_with_trunk ... SKIPPED: Marked as unstable and skipped because of bug: https://launchpad.net/bugs/2024160 | 16:25 |
| melwitt | different bug | 16:25 |
| tkajinam | but both are fixed IIUC | 16:26 |
| tkajinam | let me propose a tempest change to remove the skip decorator and I'll update the nova change to make it dependent on that change | 16:26 |
| melwitt | yeah, not yet sure where the above skip is coming from | 16:26 |
| melwitt | tkajinam: fwiw it sounds like it's unrelated to your nova change so I think you don't need to depend on it. it was just the first thing I checked is if the test is now running so I mentioned it | 16:28 |
| tkajinam | ok | 16:28 |
| tkajinam | if the tests are skipped in tempest then we may not need to maintain that exclude so we can anyway drop it imho. | 16:31 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: testing migration with network trunk https://review.opendev.org/c/openstack/nova/+/1014130 | 16:31 |
| tkajinam | ^^^ but I'll check this later ... | 16:31 |
| tkajinam | so v5 rpc api is being removed now. does it mean I can restore my old changes to remove compatibility code for nova-compute <= wallaby ? | 16:33 |
| tkajinam | eg. https://review.opendev.org/c/openstack/nova/+/909190 | 16:33 |
| opendevreview | Dan Smith proposed openstack/nova-specs master: Add cross-cell-live-migration spec https://review.opendev.org/c/openstack/nova-specs/+/1008081 | 18:53 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!