Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-07-06
11:15:17 openstackgerrit Elod Illes proposed openstack/nova stable/stein: Revert "Make greande jobs n-v for EM and oldest stable" https://review.opendev.org/739464
11:24:36 aarents lyarwood: When you will have time, can you confirm this one is ok for you: https://review.opendev.org/#/c/719008 thks !
11:31:58 lyarwood aarents: looking
11:33:57 openstack Launchpad bug 1740320 in OpenStack Compute (nova) "nova-scheduler does not honor max_instances_per_host set to a host aggregate" [Undecided,Confirmed] - Assigned to vinay harsha mitta (vinay7)
11:33:57 vinay_m hi can anyone help me in how to reproduce this bug: https://bugs.launchpad.net/nova/+bug/1740320
11:36:33 lyarwood aarents: ack'd
11:52:27 aarents lyarwood: thks
12:19:43 openstackgerrit Merged openstack/nova stable/stein: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732696
12:20:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: Repro gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704865
12:22:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reduce gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704866
12:23:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reduce gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704866
12:23:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: Repro gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704865
12:29:58 gibi lyarwood: ^ I've moved the regression test to the regression directory and reported a bug
12:30:53 lyarwood gibi: ack thanks, I'll take a look this afternoon
12:31:07 gibi thanks
12:33:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Test the state of VMware NSX 3pp CI https://review.opendev.org/734114
12:36:35 lseki Hi folks, are CI jobs `nova-next` and `nova-multi-cell` unstable? They're failing more often than not in my patch, but sometimes they pass...
12:39:19 sean-k-mooney lseki: in general no they are not
12:39:45 sean-k-mooney the ci has been a little unstable for the last week or so
12:39:57 sean-k-mooney but those jobs are generaly stable
12:43:53 lseki sean-k-mooney: thanks, I'll trigger a couple more times
12:44:04 sean-k-mooney which patch is it
12:44:19 sean-k-mooney did you chekc why they failed?
12:44:34 sean-k-mooney you technically are ment to do that when you recheck
12:44:51 lseki https://review.opendev.org/#/c/739077/
12:45:15 lseki the last time `tempest.api.compute.admin.test_volume_swap.TestVolumeSwap` failed by timeout
12:45:33 lseki so I guess it's something intermittent
12:46:25 sean-k-mooney the multi cell failure was with swap volume
12:47:30 sean-k-mooney ya so since you are modifying the volume extention code we need to be carful that its not related
12:49:10 lseki I believe it's not, because I modified nfs driver, and those jobs are using LVM
12:53:17 sean-k-mooney yep its unlikely
12:53:51 sean-k-mooney it looks like there are some issue with lvm currenlyt the nova lvm job failed with Details: {'code': 404, 'message': 'Image not found.'}
12:54:28 sean-k-mooney and the swap volume one has been failing intermitently although that is in a few jobs
12:54:56 sean-k-mooney normally we would add an elastic rechk query for the swap volume issue and file a bug
12:57:14 openstackgerrit sean mooney proposed openstack/nova master: lookup nic feature by pci address https://review.opendev.org/739131
12:57:14 openstackgerrit sean mooney proposed openstack/nova master: catch libvirt exception when nodedev not found. https://review.opendev.org/739017
13:00:18 sean-k-mooney bauzas: lyarwood ^ can ye take a look stephenfin are you around today?
13:00:44 bauzas sean-k-mooney: sure, already did this morning, can you please explain me what is the current consensus ?
13:00:58 sean-k-mooney aroud which?
13:01:31 sean-k-mooney you wanted to have both so i added the unit test you wanted and shortened the try catch
13:02:35 sean-k-mooney i plan to backport both patches to train upstream and downstream
13:03:27 sean-k-mooney ill do a direct backport to 16.1 downstream and then do the normal upstrem backport process
13:04:03 bauzas sean-k-mooney: you told me on Friday that you wanted to abandon https://review.opendev.org/#/c/739017/ but given my thoughts, you are okay with merging it ?
13:04:41 openstackgerrit Merged openstack/nova stable/ussuri: Fix cherry-pick check for merge patch https://review.opendev.org/739176
13:04:42 sean-k-mooney i never wanted to do that first patch but i wrote it because you and gibi seam to think it was sufficent
13:05:07 sean-k-mooney i would prefer to abandon it but its written so im fine with merging it
13:06:03 bauzas okay, thanks
13:06:36 sean-k-mooney if ye can weigh in on https://review.opendev.org/#/c/739131/1/nova/virt/libvirt/host.py@1181
13:07:06 sean-k-mooney i could technically move the typeing chnage into a third patch but i think that sets a bad precident
13:07:22 sean-k-mooney we talkabout this in the past with some of stephens patches too.
13:07:48 sean-k-mooney if pepole want me to split that i can but that is the only open question im aware of
13:09:49 sean-k-mooney bauzas: im not really sure why i could not get the unit test you asked for working on firday but i started from scratch this morning and it worked fine so i obvioulys was doing something dumb before
13:10:59 bauzas sean-k-mooney: honestly, I'm on the same page with melwitt about py3 typing
13:11:12 bauzas but I already said that here :)
13:11:43 sean-k-mooney bauzas: ya so i really dont want to set the percident that we dont use typing just because we are going to backport a patch
13:12:15 sean-k-mooney if i must i can split it but i will then be backportin that to ussuri but not to train
13:12:31 sean-k-mooney lyarwood: any input form a stable point of view ^
13:12:36 bauzas well, I'm not opiniated
13:13:23 lyarwood sean-k-mooney: yup that's fine from a stable / downstream point of view
13:13:44 sean-k-mooney lyarwood: which is. removing the typing annotation on backport
13:13:57 bauzas but honestly the balance in between risks and benefits is a a bit small
13:14:11 bauzas and I don't understand why we would *have to* provide types
13:14:21 bauzas but if someone wants to do it, then meh
13:14:23 sean-k-mooney we dont but i think we should
13:14:27 lyarwood sean-k-mooney: oh I missed that
13:14:38 openstackgerrit Merged openstack/nova master: libvirt: ensure disk_over_commit is not negative https://review.opendev.org/719008
13:14:40 lyarwood sean-k-mooney: I thought you were just talking about the direct downstream backport part
13:15:08 sean-k-mooney well input on that is fine too. that is more based on time constraints downstream
13:15:19 sean-k-mooney normally i would just do it upstream
13:32:54 sean-k-mooney gibi: thanks :)
13:33:03 sean-k-mooney ill brb going to grab lunch
13:41:00 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736957
13:53:36 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184
15:09:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: Prevent starting controller with older than N-1 computes https://review.opendev.org/738482
15:24:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Prevent starting controller with older than N-1 computes https://review.opendev.org/738482
15:31:04 openstackgerrit Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184
16:11:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix mock for 'autospec' and 'new' together https://review.opendev.org/739333
16:29:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: Prevent starting controller with older than N-1 computes https://review.opendev.org/738482
16:30:45 openstackgerrit Lin Yang proposed openstack/nova master: Add traits for new AVX512 CPU feature https://review.opendev.org/731288
16:31:38 gibi dansmith, melwitt, alex_xu: Thanks for the feedback in https://review.opendev.org/#/c/738482 (N-1 compute support). I've updated the patch and now it has tests too
17:28:26 dansmith gibi: I would be +2 except for this question: https://review.opendev.org/#/c/738482/4/nova/conductor/manager.py
17:28:30 dansmith am I missing why we still need that?
17:30:40 dansmith gibi: also, the max-concurrent-snapshots stuff is all merged I think, so it can come off of the runway list
17:48:10 openstackgerrit Merged openstack/nova master: catch libvirt exception when nodedev not found. https://review.opendev.org/739017
17:50:19 openstackgerrit Merged openstack/nova stable/ussuri: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736957
19:21:05 openstackgerrit sean mooney proposed openstack/nova stable/ussuri: catch libvirt exception when nodedev not found. https://review.opendev.org/739593
20:08:24 gmann dansmith: melwitt can you please check this easy review to make tests happy with oslotest 4.4.0 - https://review.opendev.org/#/c/739333/
20:14:42 dansmith gmann: got it
20:15:55 gmann thanks
21:22:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: nova-live-migration: Fix OS detection during Ceph tests https://review.opendev.org/739608
21:22:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708
21:22:24 openstack bug 1550919 in OpenStack Compute (nova) rocky "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood)
21:22:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Add functional test for bug 1550919 https://review.opendev.org/732707
21:23:31 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Make greande jobs n-v for EM and oldest stable https://review.opendev.org/739020
21:32:41 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: nova-live-migration: Fix OS detection during Ceph tests https://review.opendev.org/739608
21:32:41 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Make greande jobs n-v for EM and oldest stable https://review.opendev.org/739020
21:32:43 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708
21:32:43 openstack bug 1550919 in OpenStack Compute (nova) rocky "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood)
21:32:43 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Add functional test for bug 1550919 https://review.opendev.org/732707
21:32:46 lyarwood ^ okay that should work now, grenade and nova-live-migration disabled, nova-live-migration fixed, other changes landing on top.

Earlier   Later