Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-04-29
10:13:37 sean-k-mooney gibi: so not a rebase but similar
10:13:37 sean-k-mooney gibi: so not a rebase but similar
10:13:46 gibi then it should have picked up my detach fixes
10:13:46 gibi then it should have picked up my detach fixes
10:14:12 lyarwood AFAICT it didn't
10:14:13 lyarwood AFAICT it didn't
10:14:20 lyarwood this is just in the check queue btw
10:14:20 lyarwood this is just in the check queue btw
10:14:27 lyarwood it didn't make it to the gate this run
10:14:27 lyarwood it didn't make it to the gate this run
10:14:31 sean-k-mooney yep check should do the same
10:14:31 sean-k-mooney yep check should do the same
10:14:48 sean-k-mooney it will speculetlivly merge and then run on the result
10:14:48 sean-k-mooney it will speculetlivly merge and then run on the result
10:15:15 sean-k-mooney its why sometimes test pass locally and fail in the gate
10:15:16 sean-k-mooney its why sometimes test pass locally and fail in the gate
10:15:33 sean-k-mooney and you have to do a local rebase to see the isssues locally
10:15:34 sean-k-mooney and you have to do a local rebase to see the isssues locally
10:16:02 sean-k-mooney could it have just been a timing thing?
10:16:02 sean-k-mooney could it have just been a timing thing?
10:16:23 sean-k-mooney check jobs dont restart if something merges
10:16:24 lyarwood https://zuul.opendev.org/t/openstack/build/02d0c22ca9334a1094782df2f9570c35/log/compute1/logs/screen-n-cpu.txt and req-d017e833-5df2-4c5e-b348-f2625a2a608d is the old flow as it's using loopingcall
10:16:24 sean-k-mooney check jobs dont restart if something merges
10:16:25 lyarwood https://zuul.opendev.org/t/openstack/build/02d0c22ca9334a1094782df2f9570c35/log/compute1/logs/screen-n-cpu.txt and req-d017e833-5df2-4c5e-b348-f2625a2a608d is the old flow as it's using loopingcall
10:16:26 sean-k-mooney unlike gate
10:16:26 sean-k-mooney unlike gate
10:17:40 gibi * gibi clicks
10:18:08 gibi yepp the logs are from the old code
10:18:08 gibi yepp the logs are from the old code
10:18:19 lyarwood anyway it's rebased now
10:18:19 lyarwood anyway it's rebased now
10:18:21 lyarwood so hopefully
10:18:21 lyarwood so hopefully
10:18:37 gibi fingers crossed
10:18:37 gibi fingers crossed
10:19:18 gibi lyarwood: it is a grenade run, so I guess before the upgrade it have to run the old code from Wallaby
10:19:18 gibi lyarwood: it is a grenade run, so I guess before the upgrade it have to run the old code from Wallaby
10:19:34 lyarwood aaaaaaaah
10:19:34 lyarwood aaaaaaaah
10:19:37 lyarwood right
10:19:37 lyarwood right
10:19:42 lyarwood I didn't even think
10:19:42 lyarwood I didn't even think
10:19:55 lyarwood so is this still v to w?
10:19:55 lyarwood so is this still v to w?
10:20:05 gibi I think it is w to master
10:20:05 gibi I think it is w to master
10:20:13 gibi but we only merged the fix to master
10:20:13 gibi but we only merged the fix to master
10:20:19 sean-k-mooney oh ya
10:20:19 sean-k-mooney oh ya
10:20:58 lyarwood we did talk about backporting your change to wallaby
10:20:59 lyarwood we did talk about backporting your change to wallaby
10:21:08 sean-k-mooney well grenade shoudl be wallaby to xena
10:21:08 sean-k-mooney well grenade shoudl be wallaby to xena
10:21:15 lyarwood * lyarwood needs to jump on a call brb
10:21:36 lyarwood sean-k-mooney: yeah I think this is the live migration test between the two
10:21:36 lyarwood sean-k-mooney: yeah I think this is the live migration test between the two
10:21:40 lyarwood sean-k-mooney: with the compute still on w
10:21:40 lyarwood sean-k-mooney: with the compute still on w
10:22:59 sean-k-mooney back porting the change for libvirt events to wallaby would be nice form a downstrem perspecitve too
10:23:00 sean-k-mooney back porting the change for libvirt events to wallaby would be nice form a downstrem perspecitve too
10:23:53 sean-k-mooney we talked about trying that downstream but while i like the idea of doing that i was not personally sure we wanted to diverge form upstream in this case
10:23:53 sean-k-mooney we talked about trying that downstream but while i like the idea of doing that i was not personally sure we wanted to diverge form upstream in this case
10:24:56 gibi lyarwood: the delta between w and current master should be small, so I can fire up the cherry-picks
10:24:56 gibi lyarwood: the delta between w and current master should be small, so I can fire up the cherry-picks
10:29:46 gibi ahh there are already merge conflicts :/
10:29:47 gibi ahh there are already merge conflicts :/
10:36:13 lyarwood yeah that might be my fault sorry
10:36:13 lyarwood yeah that might be my fault sorry
10:36:32 lyarwood actually yeah it is
10:36:33 lyarwood actually yeah it is
10:36:51 lyarwood sean-k-mooney: yeah I did bring it up upstream as well as it's also a bugfix IMHO
10:36:52 lyarwood sean-k-mooney: yeah I did bring it up upstream as well as it's also a bugfix IMHO
10:37:52 gibi it is nobody's fault, we needed those fixes
10:37:52 gibi it is nobody's fault, we needed those fixes
11:08:30 openstackgerrit Vlad Gusev proposed openstack/nova master: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/764435
11:08:30 openstackgerrit Vlad Gusev proposed openstack/nova master: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/764435
11:10:56 openstackgerrit Daniel Bengtsson proposed openstack/nova master: Use the new type HostDomainOpt. https://review.opendev.org/c/openstack/nova/+/788240
12:13:05 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Replace blind retry with libvirt event waiting in detach https://review.opendev.org/c/openstack/nova/+/788720
12:13:06 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Move the guest.get_disk test to test_guest https://review.opendev.org/c/openstack/nova/+/788721
12:13:07 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Enable mypy on libvirt/guest.py https://review.opendev.org/c/openstack/nova/+/788722
12:13:08 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Follow up type hints for a634103 https://review.opendev.org/c/openstack/nova/+/788723
12:13:09 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: libvirt: Remove dead error handling code https://review.opendev.org/c/openstack/nova/+/788724
12:13:10 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Move instance power state check to _detach_with_retry https://review.opendev.org/c/openstack/nova/+/788725
12:14:24 gibi lyarwood, sean-k-mooney: ^^ here is the backport series. I'm not 100% confident about it as there was hairy conflicts in the tests
12:14:25 gibi lyarwood, sean-k-mooney: ^^ here is the backport series. I'm not 100% confident about it as there was hairy conflicts in the tests
12:14:51 openstackgerrit Balazs Gibizer proposed openstack/nova stable/wallaby: Consolidate device detach error handling https://review.opendev.org/c/openstack/nova/+/788726
12:26:38 gibi lyarwood: fyi, there is a rescue + volume + RBD bug in our untriaged list https://bugs.launchpad.net/nova/+bug/1926601
12:26:40 openstack Launchpad bug 1926601 in OpenStack Compute (nova) "Rescuing RBD volume-backed instance does not work" [Undecided,New]
12:26:40 gibi lyarwood: fyi, there is a rescue + volume + RBD bug in our untriaged list https://bugs.launchpad.net/nova/+bug/1926601
12:29:04 sean-k-mooney this is hte rbd images backend
12:29:04 sean-k-mooney this is hte rbd images backend
12:29:31 sean-k-mooney and a bfv guest
12:29:32 sean-k-mooney and a bfv guest
12:29:55 lyarwood gibi: ack yeah just on a call but I'll triage it today
12:29:56 lyarwood gibi: ack yeah just on a call but I'll triage it today
12:30:41 gibi thanks
12:31:34 sean-k-mooney it sound like we are just not creating the rescue disk in ceph but trying to use it
12:31:34 sean-k-mooney it sound like we are just not creating the rescue disk in ceph but trying to use it

Earlier   Later