Earlier  
Posted Nick Remark
#openstack-nova - 2017-10-10
21:44:02 mriedem dansmith: remind me, new style migration allocation things, the dest node allocation is held by the instance, and the source node allocation is held by the migration, correct?
21:44:12 dansmith yes
21:44:25 openstackgerrit Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_image_certificates https://review.openstack.org/500396
22:08:38 mriedem dansmith: ok comments in https://review.openstack.org/#/c/498949/
22:34:41 openstackgerrit Merged openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211
23:07:04 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Handle not found error on taking snapshot https://review.openstack.org/511074
23:09:12 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Handle not found error on taking snapshot https://review.openstack.org/511074
23:09:13 openstackgerrit Merged openstack/nova master: Ensure instance can migrate when launched concurrently https://review.openstack.org/506093
#openstack-nova - 2017-10-11
00:27:08 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
00:39:53 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
00:52:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
01:12:40 Kevin_Zheng mriedem, hi, for https://review.openstack.org/#/c/509326/
01:16:26 Kevin_Zheng I responded some comment, as for the neutron part, I submitted a RFE and seems Amando is fine with it and they will discuss it at the driver meeting this friday: https://bugs.launchpad.net/neutron/+bug/1718605, how should I put it in the spec?
01:16:27 openstack Launchpad bug 1718605 in neutron "[RFE] Support regex matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu)
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]

Earlier   Later