Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-11
01:17:38 openstackgerrit Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
01:17:38 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
01:17:39 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
01:17:39 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
01:17:40 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
02:32:41 openstackgerrit Merged openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416
03:39:07 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
03:41:11 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
05:50:02 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021
05:52:13 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
06:53:48 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
07:22:24 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
07:30:51 openstackgerrit Merged openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967
08:28:13 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
08:33:41 openstackgerrit Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180
08:36:08 openstackgerrit Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180
08:41:39 openstackgerrit Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180
08:41:50 openstackgerrit sahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus https://review.openstack.org/510897
08:45:08 openstackgerrit Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180
08:49:54 kashyap mdbooth: Hi there, when you get a moment:
08:50:06 mdbooth kashyap: Wassup?
08:50:30 kashyap mdbooth: Do you recall top off your head, if Nova sets any QEMU migration parameters? Or do we just rely on libvirt? Let me give an example of what I mean
08:51:28 kashyap Take a look at the ouput of this command: virsh qemu-monitor-command f26-devstack --pretty '{"execute":"query-migrate-parameters"}'
08:51:32 kashyap http://paste.openstack.org/show/623292/
08:52:29 stephenfin kashyap: to the best of my knowledge, if it can't be done via libvirt then we don't do it
08:52:56 mdbooth kashyap: nova never talks to qemu directly if that's what you mean?
08:53:00 kashyap stephenfin: Yep. I know we don't "do direct QEMU", that's the whole frelling point of explicitly modelling in libvirt XML aspects
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

Earlier   Later