Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-05-19
10:44:09 sean-k-mooney this was from friday yes
10:44:09 gibi let me check a more recent run...
10:44:09 sean-k-mooney this was from friday yes
10:44:47 gibi hm a todays run also has q-agt.txt
10:44:47 gibi hm a todays run also has q-agt.txt
10:45:13 gibi but I don't see in the zuul config where we set the Q_AGENT option to be openvswitch for this job
10:45:13 gibi but I don't see in the zuul config where we set the Q_AGENT option to be openvswitch for this job
10:45:14 sean-k-mooney yep also looking at it
10:45:14 sean-k-mooney yep also looking at it
10:45:30 sean-k-mooney is this still zullv2?
10:45:30 sean-k-mooney is this still zullv2?
10:46:18 sean-k-mooney no? https://github.com/openstack/nova/blob/master/.zuul.yaml#L53-L87
10:46:18 sean-k-mooney no? https://github.com/openstack/nova/blob/master/.zuul.yaml#L53-L87
10:46:24 sean-k-mooney that looks like it v3
10:46:24 sean-k-mooney that looks like it v3
10:46:58 sean-k-mooney ok well that is a different mistery to look at after
10:46:58 sean-k-mooney ok well that is a different mistery to look at after
10:48:03 sean-k-mooney maybe it has something to do with neutron-trunk: true but that seams odd if it does
10:48:03 sean-k-mooney maybe it has something to do with neutron-trunk: true but that seams odd if it does
10:48:24 gibi yeah, that is the only odd thing in the job config
10:48:24 gibi yeah, that is the only odd thing in the job config
10:48:54 sean-k-mooney do you think the previos live_migation_evetns patch would fix evacuate?
10:48:54 sean-k-mooney do you think the previos live_migation_evetns patch would fix evacuate?
10:49:22 sean-k-mooney we were planning to backport that to train as the race exits basically since netuuron has been a thing
10:49:22 sean-k-mooney we were planning to backport that to train as the race exits basically since netuuron has been a thing
10:50:12 gibi I don't think it fixes the evacuate as we see the error on master _after_ the live migration event patch merged
10:50:12 gibi I don't think it fixes the evacuate as we see the error on master _after_ the live migration event patch merged
10:50:23 gibi but the logic behind the failure can be the same
10:50:23 gibi but the logic behind the failure can be the same
10:50:38 sean-k-mooney gibi: well its disabled by default
10:50:38 sean-k-mooney gibi: well its disabled by default
10:51:12 sean-k-mooney because it need the nova patch to be present to enable it
10:51:12 sean-k-mooney because it need the nova patch to be present to enable it
10:51:19 gibi also it only changes _get_neutron_events_for_live_migration() which is fairly live migration specific
10:51:19 gibi also it only changes _get_neutron_events_for_live_migration() which is fairly live migration specific
10:51:55 gibi ohh you talk about the neutron fixes
10:51:55 gibi ohh you talk about the neutron fixes
10:52:04 sean-k-mooney yes
10:52:11 gibi we can try that
10:52:11 gibi we can try that
10:52:25 gibi as you said we want that to default to true anyhow
10:52:25 gibi as you said we want that to default to true anyhow
10:52:26 sean-k-mooney the nova patch was just so that the neutron fix would not break live migraiton
10:52:27 sean-k-mooney the nova patch was just so that the neutron fix would not break live migraiton
10:52:51 sean-k-mooney and because we needed the nova fix the neutron one defaulted to false for last cycle
10:52:51 sean-k-mooney and because we needed the nova fix the neutron one defaulted to false for last cycle
10:53:14 sean-k-mooney but now it can go to ture since the nova patch is present and backported
10:53:14 sean-k-mooney but now it can go to ture since the nova patch is present and backported
10:54:07 gibi lets switch it to true on master
10:54:07 gibi lets switch it to true on master
10:54:25 sean-k-mooney would you like me to submit a patch to change the default and note the evacuation bug you filed as a related bug? or will i leave that to you
10:54:25 sean-k-mooney would you like me to submit a patch to change the default and note the evacuation bug you filed as a related bug? or will i leave that to you
10:54:58 gibi you have a bit more context on that neutron flag so if you have time please propose the switch
10:54:58 gibi you have a bit more context on that neutron flag so if you have time please propose the switch
10:55:39 sean-k-mooney ok ill go do that shortly i have a docs meeting at the top of the hour so ill likely do it after that
10:55:39 sean-k-mooney ok ill go do that shortly i have a docs meeting at the top of the hour so ill likely do it after that
10:59:01 gibi sean-k-mooney: thanks
11:00:18 gibi sean-k-mooney: it seems the devstack default OVN change has been reverted https://review.opendev.org/c/openstack/devstack/+/791104
11:00:18 gibi sean-k-mooney: it seems the devstack default OVN change has been reverted https://review.opendev.org/c/openstack/devstack/+/791104
11:01:03 sean-k-mooney oh ok that would explain why we are seeing ml2/ovs
11:01:03 sean-k-mooney oh ok that would explain why we are seeing ml2/ovs
11:01:10 sean-k-mooney things exploded?
11:01:10 sean-k-mooney things exploded?
11:01:25 gibi based on the revert commit message yes there are extra things to fix
11:01:25 gibi based on the revert commit message yes there are extra things to fix
11:01:47 sean-k-mooney im really happy we did not make the default change during feature freeze then :)
11:01:48 sean-k-mooney im really happy we did not make the default change during feature freeze then :)
11:02:35 gibi yes :)
11:02:35 gibi yes :)
11:02:40 gibi good judgement
11:02:40 gibi good judgement
11:23:29 bauzas * bauzas goes afk for the afternoon if you look at me (getting a new electric car ;) 
11:24:08 gibi bauzas: congrats!
11:24:08 gibi bauzas: congrats!
11:24:16 gibi is it a tesla?
11:24:17 gibi is it a tesla?
11:24:31 bauzas gibi: nope, for my wife, a Peugeot 208
11:24:31 bauzas gibi: nope, for my wife, a Peugeot 208
11:25:17 bauzas I already have a PHEV (Skoda Superb iV) for me :)
11:25:17 bauzas I already have a PHEV (Skoda Superb iV) for me :)
11:26:03 gibi sounds cool :)
11:26:03 gibi sounds cool :)
11:39:04 lyarwood elod: https://review.opendev.org/c/openstack/nova/+/788720 - thoughts on landing this series?
11:39:04 lyarwood elod: https://review.opendev.org/c/openstack/nova/+/788720 - thoughts on landing this series?
11:42:09 elod lyarwood: sorry, yes, I started to look it yesterday, but want to see the "whole picture" first o:)
11:42:09 elod lyarwood: sorry, yes, I started to look it yesterday, but want to see the "whole picture" first o:)
11:43:13 elod lyarwood: it is (again) a bit too big for my taste for stable, but it's more or less clean if I'm not mistaken....
11:43:13 elod lyarwood: it is (again) a bit too big for my taste for stable, but it's more or less clean if I'm not mistaken....
11:46:43 lyarwood elod: yeah agreed it's pretty large but while it's clean I thought it would be useful
11:46:43 lyarwood elod: yeah agreed it's pretty large but while it's clean I thought it would be useful
11:46:51 lyarwood elod: it's something we wanted downstream for Wallaby either way
11:46:52 lyarwood elod: it's something we wanted downstream for Wallaby either way
11:47:06 lyarwood elod: so if we can squeeze it in early this cycle it would be great :)
11:47:06 lyarwood elod: so if we can squeeze it in early this cycle it would be great :)
11:48:48 elod lyarwood: understood :) I'll try to go through the patches today
11:48:48 elod lyarwood: understood :) I'll try to go through the patches today
11:50:58 lyarwood many thanks
11:50:58 lyarwood many thanks
11:52:57 gibi elod: as far as I remember it is a clean cherr-pick all the way. I did a mistake when I first tried to cherry-pick it as I used some old unmereged version of a patch
11:52:57 gibi elod: as far as I remember it is a clean cherr-pick all the way. I did a mistake when I first tried to cherry-pick it as I used some old unmereged version of a patch

Earlier   Later