Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-05-19
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
12:04:07 sean-k-mooney bauzas: i have been debating about getting an eletric car for a while but 1 i like my mini and 2 i driver maybe 5000KMs per year so its har to justtify spending more then a few grand on a car
12:04:07 sean-k-mooney bauzas: i have been debating about getting an eletric car for a while but 1 i like my mini and 2 i driver maybe 5000KMs per year so its har to justtify spending more then a few grand on a car
12:08:29 elod gibi: that part is OK then :) just have to think whether the series is valid for backport or not o:) (by looking at the code and patches yesterday it was a bit "featurish" for me, but probably there's no risk to backport... but haven't looked all of the patches yet)
12:08:29 elod gibi: that part is OK then :) just have to think whether the series is valid for backport or not o:) (by looking at the code and patches yesterday it was a bit "featurish" for me, but probably there's no risk to backport... but haven't looked all of the patches yet)
12:32:25 gibi elod: it does not change any external interfaces except the two new config options, but those can be removed, if you wish, from the backport. There is also no externally visible behavior change except the fix of the failure
12:32:25 gibi elod: it does not change any external interfaces except the two new config options, but those can be removed, if you wish, from the backport. There is also no externally visible behavior change except the fix of the failure
12:33:51 gibi I guess you feel it as a feature becuase we started using a different mechanism to talk to libvirt regarding the attachment, we went from polling to waiting for events. And waiting for events needed extra preparation in the code
12:33:51 gibi I guess you feel it as a feature becuase we started using a different mechanism to talk to libvirt regarding the attachment, we went from polling to waiting for events. And waiting for events needed extra preparation in the code
12:35:00 sean-k-mooney i dont think its a featue its just a refacotiong
12:35:00 sean-k-mooney i dont think its a featue its just a refacotiong
12:35:24 gibi sean-k-mooney: yes, it needed a sizeable refactoring to properly fix that bug
12:35:24 gibi sean-k-mooney: yes, it needed a sizeable refactoring to properly fix that bug
12:35:32 sean-k-mooney the behaivior before and after modulo bugs is identical from blackbox perspctive
12:35:32 sean-k-mooney the behaivior before and after modulo bugs is identical from blackbox perspctive

Earlier   Later