Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-28
14:20:30 bauzas heh
14:20:46 bauzas I'm just not used to see this channel so quiet
14:20:53 bauzas so let's make random noises
14:37:09 bhagyashris eric-young: Hi,
15:48:53 cfriesen__ stephenfin: bauzas: if you're still on, I just had a thought. given that we *know* it's broken, should we consider preventing live migration with pinned CPUs, numa nodes, etc? Or is there enough history that we need to preserve the likely-broken current behaviour?
15:49:29 cfriesen I'm looking at Artom's WIP live-migration patches, and just had the above thought...
15:49:46 artom cfriesen, shoot
15:50:02 artom Oh
15:50:04 artom Hrmm
15:50:53 cfriesen artom: given that in the BP we're talking about preventing live migration unless it's "new enough" on both computes, should we just go ahead and block it now?
15:51:00 artom I feel explicitly saying "we can't do this" rather than doing it and potentially breaking the SLA we have in place (many:1 vCPU:pCPU pin mappings) is better
15:51:17 artom cfriesen, so, that patch is actually full done
15:51:22 artom Well - "done"
15:51:44 artom It's WIP because Matt asked me to move it above the "real work" patch, which is only in its very infancy
15:52:10 artom We could technically merge it as is - it's an "empty" service version bump, and a corresponding check in the conductor
15:52:36 artom But... historically, we've merged the "real work" patch first, and then the version checking
15:52:58 artom In any case, unless someone is running master in production, it doesn't really matter, since both patches will end up in the same release
15:53:05 cfriesen artom: yeah...I was just wondering about merging an unconditional "we can't live-migrate this" patch now, then refine the check later once we can actually do it
15:53:28 cfriesen artom: that assumes we actually merge the real one. :)
15:53:40 artom cfriesen, are you doubting my capabilities to push code through? ;)
15:54:03 artom sahid might know more about this, IIRC he had a thing similar to that: just block it for now, until we get the real solution in place
15:54:11 artom We ended up doing that downstream
15:54:22 cfriesen this functionality has been missing for so long, it's like the cursed behaviour or something. :)
15:54:30 artom Which... I mean, we found a bug in that, so, it's "fun" :)
15:55:01 artom We ended up doing that downstream because it got shut down upstream, I should clarify
15:55:09 cfriesen ah, okay.
15:55:09 artom .... this is all from memory, I may be waaay off
15:56:55 cfriesen anyway, I'm not strongly opinionated, just had a thought.
15:57:35 sahid cfriesen: we have a patch downstream so instead of changing guest pinning, we try to find a host which can handle it
15:59:02 sahid artom: is now working on a real solution so i think we should make all our efforts behind him
15:59:14 artom Wind in my sails baby
15:59:19 cfriesen sahid: if you've got enough compute nodes I could see that. We had to change the pinning because we support a hyperconverged dual-node configuration.
15:59:48 cfriesen sahid: for sure, I'm looking at artom's patches now.
15:59:56 sahid cfriesen: so you have to wait for artom's work i guess
16:00:21 sahid oh i did not know that, artom you already have some patches to review?
16:00:42 artom sahid, https://review.openstack.org/#/c/566723/8
16:00:46 artom And the two below
16:01:02 sahid artom: thanks
16:22:33 bhagyashris eric-young: hi,
23:51:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058
#openstack-nova - 2018-05-29
01:04:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/569320
02:58:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (4) https://review.openstack.org/570750
05:06:02 yumapath hi anyone there who can help with nova PCI passthrough
05:06:20 yumapath am hitting issues with the same, not able to successfully put a pci card in passthrough mode
06:12:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842
06:42:20 openstackgerrit Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090
06:58:22 jichen hi dansmith: sorry to disturb again, could you please help to approve https://review.openstack.org/#/c/562154/ as most info already in the ML ? Thanks
07:07:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018
07:07:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840
07:07:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
07:07:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
07:07:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
07:07:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
07:07:31 gibi good morning nova
07:17:26 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
07:32:00 Multiple_ https://review.openstack.org/#/c/563418/
07:32:42 Multiple_ Please have a look at the review.. only a minor change of adding a retry decorator
08:05:33 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:41:33 openstackgerrit jichenjc proposed openstack/nova master: Remove unused function https://review.openstack.org/570865
09:04:38 lyarwood stephenfin, mdbooth_, bauzas : https://review.openstack.org/#/c/569062/ - would you mind taking a look at this bugfix today if you have time?
09:04:52 stephenfin sure
09:38:07 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832
09:38:08 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
09:52:06 stephenfin lyarwood: Did you say something about us planning to rework how volume drivers (or something to do with volumes?) are implemented in nova?
09:54:09 stephenfin lyarwood: In relation to https://review.openstack.org/#/c/482640/
09:57:25 lyarwood stephenfin: yeah, I wasn't sure if this was going to be an imagebackend or volume driver, we don't want the former and this isn't so it's cool :)
09:57:42 stephenfin Ah, image backends. Gotcha
10:06:56 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559032
10:32:17 lyarwood kashyap: https://review.openstack.org/#/c/569442/ - can you close these backport out given https://review.openstack.org/#/c/570271/ has landed in stable?
10:32:39 kashyap lyarwood: I will; that fated forced thing
10:33:14 kashyap Done.
10:33:17 lyarwood kashyap: thanks
10:45:32 openstackgerrit zhuangzejian proposed openstack/nova master: Fix doc mistakes https://review.openstack.org/570894
10:50:21 stephenfin bauzas: Can you send this docs patch on its way https://review.openstack.org/#/c/540887/
11:05:25 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
11:17:07 openstackgerrit sean mooney proposed openstack/os-vif master: add noop plugin https://review.openstack.org/567924
11:26:01 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
11:53:18 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
12:01:39 openstackgerrit Chen proposed openstack/nova master: Fix some inconsistencies in doc https://review.openstack.org/570407
12:02:17 openstackgerrit Chen proposed openstack/nova master: Fix some inconsistencies in doc https://review.openstack.org/570407
12:05:28 openstackgerrit Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072
12:05:29 openstack bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood)
12:05:30 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Ignore active evacuations during _init_instance https://review.openstack.org/562284
12:38:58 openstackgerrit Merged openstack/nova stable/pike: Add ssbd and virt-ssbd flags to cpu_model_extra_flags whitelist https://review.openstack.org/570513
12:39:04 openstackgerrit Merged openstack/nova stable/queens: Don't reschedule on RequestedVRamTooHigh errors https://review.openstack.org/568642
12:46:44 efried ō/
12:47:37 gibi efried: o/
12:48:02 gibi efried: thankd for the comments on the bandwidth patch series, I'm going through those right now
12:48:17 efried cool
12:51:46 openstackgerrit Merged openstack/nova stable/queens: Add random sleep between retry calls to placement https://review.openstack.org/568218
12:51:52 openstackgerrit Merged openstack/nova stable/queens: xenapi: handle InstanceNotFound in detach_interface() https://review.openstack.org/564468
12:51:58 openstackgerrit Merged openstack/nova stable/queens: Improve performance when list instances with IP filter https://review.openstack.org/563972
12:52:04 openstackgerrit Merged openstack/nova stable/queens: Add nova-status check for ironic flavor migration https://review.openstack.org/563250
12:52:10 openstackgerrit Merged openstack/nova stable/queens: Log a warning and add nova-status check for old API service versions https://review.openstack.org/563251
13:09:15 efried mriedem: What was [1] supposed to be here: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130768.html ? Maybe jaypipes' https://review.openstack.org/#/c/553597/ ?
13:25:38 mriedem known issue?
13:25:39 mriedem chmod: cannot access '/home/zuul/workspace/logs/reproduce.sh': No such file or directory
13:26:42 sahid mriedem: about the vftrusted feature, i think i have addressed all you comments
13:26:50 sahid did i missed something you want me to add?
13:35:39 mriedem sahid: i just need to go through it again

Earlier   Later