Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-11
08:53:14 mdbooth In fact, I don't think qemu allows more than one monitor, does it?
08:53:18 mdbooth i.e. it's not even possible
08:54:14 kashyap mdbooth: I know Nova never talks to QEMU
08:54:22 kashyap mdbooth: And QEMU does allow more than one monitor.
08:54:45 mdbooth kashyap: I didn't understand the question, then
08:54:57 kashyap mdbooth: Let me give a bit more context:
08:55:06 mdbooth Nova does set some of those values explicitly, via libvirt
08:55:28 kashyap mdbooth: What made me ask the question is, one of the QEMU migration devs asked me this morning what migration parameters Nova uses
08:56:27 kashyap mdbooth: You (libvirt could) set migration parameters via `migrate-set-parameters` QMP command, _except_ this "xbzrle-cache-size"
08:56:49 kashyap And they were wondering does Nova use it in anyway. So I went digging...
08:57:15 mdbooth Yes, absolutely. Nova tweaks a bunch of those.
09:00:38 kashyap mdbooth: Meanwhile, here's an example for multiple monitors:
09:00:42 kashyap $ qemu-system-x86_64 [...] -monitor stdio -qmp unix:./qmp-sock,server,nowait
09:01:00 kashyap HMP (Human Monitor Protocol) on 'stdio', and QMP on a UNIX socket.
09:06:19 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: [WIP] CLI for resource classes https://review.openstack.org/511182
09:06:19 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: [WIP] RP list: member_of and resources parameters https://review.openstack.org/511183
09:06:49 openstackgerrit Alex Xu proposed openstack/nova master: [DNM] Add _get_provider_ids_with_any_trait https://review.openstack.org/511184
09:06:49 openstackgerrit Alex Xu proposed openstack/nova master: [DNM] placement: add traits support in the AllocationCandidates object https://review.openstack.org/511185
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

Earlier   Later