Earlier  
Posted Nick Remark
#openstack-nova - 2017-10-11
09:12:14 openstackgerrit sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188
09:42:53 openstackgerrit jichenjc proposed openstack/nova master: Remove duplicate error info https://review.openstack.org/510719
09:44:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
09:48:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
09:48:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
09:59:20 openstackgerrit Hironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration https://review.openstack.org/500677
09:59:58 openstackgerrit jichenjc proposed openstack/nova master: Remove duplicate error info https://review.openstack.org/510719
10:00:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
10:06:42 lyarwood mdbooth: so the stable rescue spec LGTM. I've been umming and ahhing about the sync vs async failure paths for a while, the only alternative I can see is making the rescue_instance rpc cast a call but that's a horrid change.
10:07:13 mdbooth lyarwood: On a call. Sync up in a bit.
10:10:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
10:12:54 openstackgerrit Andrey Volkov proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/509206
10:59:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
11:01:42 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839
11:05:53 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot https://review.openstack.org/511209
11:18:47 avolkov dansmith sdague bauzas cdent: hi, do you remember a case with updating AZ? I posted an alternative solution https://review.openstack.org/#/c/509206/ which restricts api actions. Basically, you can't rename AZ, add or delete host from
11:18:47 avolkov aggregate with AZ if host has VMs. Is it something we can move further with?
11:19:04 openstackgerrit Merged openstack/nova master: api-ref: note the microversions for GET /resource_providers query params https://review.openstack.org/502168
11:20:09 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001
11:36:06 openstackgerrit Matthew Booth proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices https://review.openstack.org/510106
11:36:14 mdbooth lyarwood: Thanks!
11:36:38 mdbooth lyarwood: Do you recall who approved it previously, btw?
11:38:09 mdbooth lyarwood: I updated for the work already done, btw, and fixed a couple of typos as it was changing anyway. If you have a sec, could you put your +1 back?
11:39:17 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
11:42:52 lyarwood mdbooth: Matt AFAIK
11:43:09 mdbooth lyarwood: Thanks. Makes sense as he asked for the microversion.
11:45:10 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892
12:29:31 mdbooth mriedem: This is a spec which will hopefully be familiar to you: https://review.openstack.org/#/c/510106/
12:32:14 openstackgerrit Matthew Booth proposed openstack/nova master: Move is_block_device_mapping to new_format_is_volume https://review.openstack.org/511217
12:32:14 openstackgerrit Matthew Booth proposed openstack/nova master: Remove driver.block_device.get_swap https://review.openstack.org/511218
12:38:35 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
12:44:18 bauzas avolkov: ta. FWIW, I wrote a very old spec for that but looks like the consensus is that it's not needing a microversion
12:45:39 stephenfin sahid: Want to take a look at https://review.openstack.org/#/c/361140/ ?
12:50:41 alex_xu s/15 mins/10 mins/...
12:50:51 sahid stephenfin: yes i will, ok
12:53:22 stephenfin lyarwood, mdbooth: Fancy taking a look at this? Looks sane to me, at least https://review.openstack.org/#/c/290793/
12:55:28 mdbooth stephenfin: Looking
12:58:33 stephenfin ralonsoh_: About? Got a question on https://review.openstack.org/#/c/484726/
13:06:56 ralonsoh_ stephenfin: sure! but this is an ongoing patch. Once we have the os-vif migration patches uploaded, you'll see why this object is needed
13:07:40 ralonsoh_ stephenfin: as soon as I have the rest of the code uploaded, I'll complete the bug description and I'll link this patch to them
13:07:52 ralonsoh_ I'm on it]
13:08:34 stephenfin ralonsoh_: Cool cool :) Maybe stick it in WIP until then?
13:08:50 ralonsoh_ sure! I'll do this now
13:09:16 efried gibi I posted another response on https://review.openstack.org/#/c/510244/ -- please let me know if you disagree or would like to discuss further. Thanks!
13:09:30 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Open vSwitch patch port https://review.openstack.org/485228
13:09:42 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Virtual Ethernet device pair https://review.openstack.org/484726
13:12:20 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
13:28:20 mriedem gibi: https://review.openstack.org/#/c/510938/
13:33:11 sahid stephenfin: it's not clear for me how are you make working "preferred", I mean in a technical POV
13:33:17 sahid https://review.openstack.org/#/c/361140/28/specs/queens/approved/share-pci-between-numa-nodes.rst
13:33:42 sahid stephenfin: you are probably going to use a weigher?
13:33:49 gmann mriedem: is it ok for you now - https://review.openstack.org/#/c/459483/33/nova/api/openstack/compute/server_migrations.py@58
13:34:07 gmann mriedem: not having whitelist to add 'type' in response
13:34:37 stephenfin sahid: What aspect of it?
13:34:47 sahid scheduling
13:36:02 sahid stephenfin: from how i see that you need two pass, and i don't think it"s something that the current scheduler provides
13:37:19 stephenfin sahid: Yeah, I think you might be right there. We're probably going to stick with making it configurable at the host leve
13:37:44 stephenfin i.e. you use the first host that is given to you by the scheduler
13:38:02 sahid stephenfin: seems more reasonable yes
13:38:06 stephenfin I don't think the two-pass solution is possible either. jaypipes might have suggestions though
13:38:19 stephenfin sahid: Cool. I can update the spec accordingly
13:38:58 sahid mriedem, jaypipes can one of you look ACK this this https://review.openstack.org/#/c/497457/ ?
13:41:38 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
13:43:40 mriedem sahid: need to see the live migration job pass on that first
13:43:46 mriedem stephenfin: ^
13:47:07 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
13:49:34 sahid jaypipes: i replied to you about the sriov-trusted-vfs (in case you did not notice it)
14:02:37 mdbooth stephenfin: Can't say I'm a fan of that change. Haven't put a -1 only because it's *probably* better than what we have now. Not sure if the added complexity is worth it, though.
14:04:48 mriedem sahid: which hypervisors don't support this flag? https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_MIGRATE_PARAM_BANDWIDTH
14:08:11 sahid mriedem: hum.. let me check that point
14:09:47 openstackgerrit Merged openstack/nova master: Don't fix protocol-less glance api_servers anymore https://review.openstack.org/505317
14:12:02 sahid mriedem: according to a simple grep in libvirt source it seems that vz and QEMU are supporting it
14:12:31 sahid but but nothing about vmware, hyperv..
14:13:22 mriedem sahid: well we wouldn't care about vmware or hyperv in the libvirt driver
14:13:37 mriedem but we care about xen, although i'm not sure if libvirt+xen is listed as supporting live migration anyway
14:13:57 openstackgerrit Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax https://review.openstack.org/510244
14:13:59 mriedem according to this it says we do https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_live_migrate
14:14:31 openstackgerrit Merged openstack/nova master: Move user_data max length check to schema https://review.openstack.org/510704
14:14:55 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
14:18:51 sahid mriedem: we always use that parameter (bandwiddh) for live-migration
14:19:09 sahid do you have a suggestion on how the code should be?
14:19:26 mriedem it must work for xen then
14:19:33 sahid I guess yes
14:19:33 mriedem libvirt+xen i mean
14:19:37 sahid but I can't ensure that
14:19:54 mriedem anthonyper: do you know what the current state of libvirt+xen live migration is? working?
14:19:58 mriedem has anyone tested lately?
14:25:41 anthonyper mriedem: I have no idea :(
14:33:46 anthonyper mriedem: I'll ask other people to try to find out if they have tested live migration recently.
14:34:36 mriedem anthonyper: ok thanks
14:36:09 sahid dansmith: if the wait_for_instance_event has been desgined has you are suggesting so it's not working
14:38:04 dansmith sahid: I'm not sure what you're saying
14:38:27 jaypipes sahid: yep, noticed it. sorry, been spending my last days rebasing the nested resource providers series.
14:38:42 jaypipes sahid: I'll review that slow live-migration patch this morning.
14:38:50 openstackgerrit Merged openstack/nova master: Add connection pool size to vSphere settings https://review.openstack.org/414085
14:39:12 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
14:39:20 sahid jaypipes: no worries and thanks if you can have a look
14:39:34 openstackgerrit Merged openstack/nova master: Clarify conf/compute.py help text for ListOpts https://review.openstack.org/468538
14:39:53 sahid dansmith: related to one of your comment on https://review.openstack.org/#/c/497457/
14:40:20 sahid the other one are just nits, right?

Earlier   Later