Wednesday, 2017-10-11

*** Oku_OS-away has quit IRC00:00
*** andreas_s has quit IRC00:01
*** jgriffith has quit IRC00:01
*** felipemonteiro has joined #openstack-nova00:01
*** chyka has joined #openstack-nova00:01
*** Oku_OS-away has joined #openstack-nova00:01
*** John has joined #openstack-nova00:02
*** John is now known as Guest6337800:02
*** felipemonteiro_ has joined #openstack-nova00:02
*** Guest63378 is now known as jgriffith00:03
*** Tom has quit IRC00:03
*** felipemonteiro has quit IRC00:06
*** chyka has quit IRC00:06
*** acormier has joined #openstack-nova00:09
*** suresh12 has joined #openstack-nova00:10
*** Apoorva_ has quit IRC00:11
*** yamahata has quit IRC00:18
*** tetsuro has joined #openstack-nova00:20
*** Apoorva has joined #openstack-nova00:20
*** felipemonteiro_ has quit IRC00:21
*** xinliang has quit IRC00:23
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements  https://review.openstack.org/49314600:27
*** andreas_s has joined #openstack-nova00:28
*** hshiina has joined #openstack-nova00:29
*** crushil has quit IRC00:31
*** crushil has joined #openstack-nova00:31
*** andreas_s has quit IRC00:32
*** acormier has quit IRC00:33
*** acormier has joined #openstack-nova00:34
*** xinliang has joined #openstack-nova00:35
*** crushil has quit IRC00:36
*** acormier has quit IRC00:38
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements  https://review.openstack.org/49314600:39
*** acormier has joined #openstack-nova00:43
*** acormier has quit IRC00:44
*** acormier has joined #openstack-nova00:45
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements  https://review.openstack.org/49314600:52
*** thorst has joined #openstack-nova00:53
*** mingyu has joined #openstack-nova00:53
*** thorst has quit IRC00:54
*** AlexeyAbashkin has joined #openstack-nova00:55
*** AlexeyAbashkin has quit IRC00:59
*** mingyu has quit IRC00:59
*** acormier_ has joined #openstack-nova01:03
*** andreas_s has joined #openstack-nova01:04
*** Apoorva_ has joined #openstack-nova01:05
*** acormier has quit IRC01:06
*** gyee has quit IRC01:08
*** phuongnh has joined #openstack-nova01:08
*** Apoorva has quit IRC01:08
*** andreas_s has quit IRC01:09
*** Apoorva_ has quit IRC01:09
*** liverpooler has quit IRC01:10
Kevin_Zhengmriedem, hi, for https://review.openstack.org/#/c/509326/01:12
*** yangyapeng has joined #openstack-nova01:14
Kevin_ZhengI 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
openstackLaunchpad bug 1718605 in neutron "[RFE] Support regex matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu)01:16
openstackgerritJay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client  https://review.openstack.org/41592101:17
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713801:17
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721501:17
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480701:17
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569301:17
*** jaypipes has quit IRC01:18
*** penick has joined #openstack-nova01:22
*** Tom_ has joined #openstack-nova01:24
*** Shunli has joined #openstack-nova01:25
*** litao__ has joined #openstack-nova01:30
*** hemna_ has quit IRC01:31
*** jwcroppe has joined #openstack-nova01:32
*** hongbin has joined #openstack-nova01:35
*** andreas_s has joined #openstack-nova01:37
*** acormier_ has quit IRC01:40
*** andreas_s has quit IRC01:41
*** suresh12 has quit IRC01:46
*** psachin has joined #openstack-nova01:51
*** Tom_ has quit IRC01:53
*** jwcroppe has quit IRC01:54
*** READ10 has quit IRC01:58
*** takashin has quit IRC02:01
*** owalsh_ has joined #openstack-nova02:08
*** penick has quit IRC02:11
*** owalsh has quit IRC02:11
*** thorst has joined #openstack-nova02:17
*** thorst has quit IRC02:18
*** catintheroof has joined #openstack-nova02:20
*** dave-mccowan has joined #openstack-nova02:20
*** suresh12 has joined #openstack-nova02:21
*** jlvillal has quit IRC02:21
*** jlvillal has joined #openstack-nova02:21
*** dave-mcc_ has quit IRC02:22
*** catintheroof has quit IRC02:22
*** liusheng has joined #openstack-nova02:25
*** suresh12 has quit IRC02:26
openstackgerritMerged openstack/nova master: Fix minor input items from previous patches  https://review.openstack.org/50641602:32
*** rgerganov has quit IRC02:33
*** thorst has joined #openstack-nova02:33
*** rgerganov has joined #openstack-nova02:33
*** ijw has quit IRC02:34
*** takashin has joined #openstack-nova02:34
*** thorst has quit IRC02:38
*** rmk has quit IRC02:42
*** rmk has joined #openstack-nova02:45
*** rmk has quit IRC02:45
*** rmk has joined #openstack-nova02:45
*** dave-mccowan has quit IRC02:51
*** erlon has quit IRC02:54
*** markvoelker has quit IRC02:54
*** chyka has joined #openstack-nova02:55
*** mingyu has joined #openstack-nova02:56
*** chyka has quit IRC02:56
*** mingyu has quit IRC02:56
*** links has joined #openstack-nova02:57
*** mingyu has joined #openstack-nova02:59
*** mingyu has quit IRC02:59
*** sree has joined #openstack-nova03:01
*** mingyu has joined #openstack-nova03:04
*** nicolasbock has quit IRC03:06
*** sree has quit IRC03:06
*** mingyu has quit IRC03:10
*** mingyu has joined #openstack-nova03:13
*** mingyu has quit IRC03:14
*** mingyu has joined #openstack-nova03:14
*** yamahata has joined #openstack-nova03:17
*** jmlowe has joined #openstack-nova03:24
*** yamamoto has quit IRC03:25
*** chyka has joined #openstack-nova03:27
*** chyka has quit IRC03:29
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/51102103:39
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103503:41
*** hshiina has quit IRC03:42
*** vivsoni has quit IRC03:51
*** suresh12 has joined #openstack-nova03:54
*** jwcroppe has joined #openstack-nova03:57
*** hshiina has joined #openstack-nova04:01
*** jichen has joined #openstack-nova04:02
*** ratailor has joined #openstack-nova04:05
*** mdnadeem has joined #openstack-nova04:06
*** ratailor_ has joined #openstack-nova04:07
*** mingyu has quit IRC04:09
*** ratailor has quit IRC04:11
*** hongbin has quit IRC04:12
*** yamamoto has joined #openstack-nova04:13
*** mingyu has joined #openstack-nova04:15
*** diga has joined #openstack-nova04:17
*** ratailor__ has joined #openstack-nova04:21
*** ratailor_ has quit IRC04:22
*** khamtamtun has joined #openstack-nova04:23
*** khamtamtun has quit IRC04:25
*** claudiub|2 has joined #openstack-nova04:25
*** gbarros has joined #openstack-nova04:28
*** vks1 has joined #openstack-nova04:31
*** chyka has joined #openstack-nova04:32
*** ratailor_ has joined #openstack-nova04:32
*** mingyu has quit IRC04:32
*** mingyu has joined #openstack-nova04:32
*** jmlowe has quit IRC04:33
*** gyee has joined #openstack-nova04:34
*** zen369 has joined #openstack-nova04:35
*** ratailor__ has quit IRC04:36
*** crushil has joined #openstack-nova04:37
*** gbarros has quit IRC04:37
*** vivsoni has joined #openstack-nova04:38
*** thorst has joined #openstack-nova04:39
*** chyka has quit IRC04:40
*** yamamoto has quit IRC04:43
*** thorst has quit IRC04:44
*** Tom has joined #openstack-nova04:46
*** Tom has quit IRC04:47
*** liusheng has quit IRC04:51
*** markvoelker has joined #openstack-nova04:55
*** armax has quit IRC05:01
*** cristicalin has joined #openstack-nova05:01
*** cristicalin has quit IRC05:04
*** sridharg has joined #openstack-nova05:07
*** cristicalin has joined #openstack-nova05:07
*** mdnadeem_ has joined #openstack-nova05:08
*** cristicalin has quit IRC05:09
*** gyee has quit IRC05:10
*** mdnadeem has quit IRC05:10
*** sree_ has joined #openstack-nova05:11
*** mingyu has quit IRC05:11
*** sree_ is now known as Guest7213805:11
*** edand has joined #openstack-nova05:12
*** mingyu has joined #openstack-nova05:12
*** crushil has quit IRC05:12
*** gouthamr has quit IRC05:12
*** Guest2849 is now known as sdake05:20
*** sdake is now known as Guest9543005:21
*** Guest95430 is now known as sdake_fixing05:23
*** ratailor__ has joined #openstack-nova05:23
*** mingyu has quit IRC05:25
*** ratailor_ has quit IRC05:26
*** suresh12 has quit IRC05:29
*** sdake_fixing has quit IRC05:29
*** sdake_fixing has joined #openstack-nova05:29
*** markvoelker has quit IRC05:30
*** lajoskatona has joined #openstack-nova05:30
*** mingyu has joined #openstack-nova05:32
*** Tom has joined #openstack-nova05:33
*** diga has quit IRC05:35
*** sdake_fixing has quit IRC05:35
*** mingyu has quit IRC05:36
*** Tom has quit IRC05:37
*** udesale has joined #openstack-nova05:38
*** yamamoto has joined #openstack-nova05:43
*** mdnadeem_ has quit IRC05:44
*** lpetrut has joined #openstack-nova05:49
*** yamamoto has quit IRC05:49
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/51102105:50
*** Tom has joined #openstack-nova05:50
*** lpetrut has quit IRC05:50
*** lpetrut has joined #openstack-nova05:51
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103505:52
*** yamamoto has joined #openstack-nova05:54
*** Tom has quit IRC05:54
*** josecastroleon has quit IRC05:57
*** yamamoto has quit IRC05:58
*** jwcroppe has quit IRC05:59
*** mdnadeem_ has joined #openstack-nova06:00
*** yamamoto has joined #openstack-nova06:02
*** yamamoto has quit IRC06:04
*** abhishek has joined #openstack-nova06:05
*** Tom has joined #openstack-nova06:06
*** abhi89 has quit IRC06:07
*** trinaths has joined #openstack-nova06:08
*** Oku_OS-away is now known as Oku_OS06:10
*** Tom has quit IRC06:11
*** andreas_s has joined #openstack-nova06:11
*** diga has joined #openstack-nova06:11
*** liusheng has joined #openstack-nova06:14
*** fragatina has joined #openstack-nova06:16
*** hoonetorg has quit IRC06:17
*** belmoreira has joined #openstack-nova06:18
*** yamamoto has joined #openstack-nova06:21
*** markvoelker has joined #openstack-nova06:26
*** avolkov has joined #openstack-nova06:27
*** spectr has quit IRC06:27
*** spectr has joined #openstack-nova06:28
*** zerda3 has joined #openstack-nova06:28
*** hoonetorg has joined #openstack-nova06:31
*** zsli_ has joined #openstack-nova06:32
*** ratailor_ has joined #openstack-nova06:32
*** mingyu has joined #openstack-nova06:33
*** Shunli has quit IRC06:35
*** ratailor__ has quit IRC06:35
*** mingyu has quit IRC06:38
*** thorst has joined #openstack-nova06:40
*** sshwarts has joined #openstack-nova06:40
*** zsli__ has joined #openstack-nova06:41
*** sahid has joined #openstack-nova06:42
*** markus_z has joined #openstack-nova06:43
*** zsli_ has quit IRC06:44
*** thorst has quit IRC06:44
*** suresh12 has joined #openstack-nova06:45
*** TuanLA has joined #openstack-nova06:46
*** abhi89 has joined #openstack-nova06:49
*** owalsh_ is now known as owalsh06:49
*** lpetrut has quit IRC06:49
*** suresh12 has quit IRC06:50
*** abhishek has quit IRC06:51
openstackgerritElod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification  https://review.openstack.org/50850606:53
*** vvargaszte has joined #openstack-nova06:55
*** zerda3 has quit IRC06:59
*** markvoelker has quit IRC07:00
*** sdake has joined #openstack-nova07:01
*** sdake has quit IRC07:01
*** sdake has joined #openstack-nova07:01
*** vvargaszte has quit IRC07:03
*** openstackgerrit has quit IRC07:03
*** abhi89 has quit IRC07:03
*** abhi89 has joined #openstack-nova07:03
*** rcernin has joined #openstack-nova07:05
*** jpena|off is now known as jpena07:08
*** edand has quit IRC07:11
*** tojuvone has quit IRC07:12
*** pcaruana has joined #openstack-nova07:13
*** lpetrut has joined #openstack-nova07:16
*** ralonsoh has joined #openstack-nova07:18
*** trinaths has quit IRC07:18
*** tesseract has joined #openstack-nova07:19
*** ratailor__ has joined #openstack-nova07:21
*** openstackgerrit has joined #openstack-nova07:22
openstackgerritElod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification  https://review.openstack.org/50850607:22
*** tojuvone has joined #openstack-nova07:23
*** ratailor_ has quit IRC07:24
*** edand has joined #openstack-nova07:27
*** trinaths has joined #openstack-nova07:27
*** yangyapeng has quit IRC07:29
*** yangyapeng has joined #openstack-nova07:29
*** tinwood has quit IRC07:30
*** tinwood has joined #openstack-nova07:30
openstackgerritMerged openstack/nova master: Target context for build notification in conductor  https://review.openstack.org/50996707:30
*** nikhil has quit IRC07:34
*** yamamoto has quit IRC07:35
*** yamamoto has joined #openstack-nova07:36
*** yamamoto has quit IRC07:36
*** ratailor__ has quit IRC07:36
*** AlexeyAbashkin has joined #openstack-nova07:37
*** vvargaszte has joined #openstack-nova07:38
*** thorst has joined #openstack-nova07:41
*** vvargaszte has quit IRC07:42
*** thorst has quit IRC07:45
*** openstackgerrit has quit IRC07:48
*** lpetrut has quit IRC07:49
*** itlinux has joined #openstack-nova07:52
*** tojuvone has quit IRC07:55
*** markvoelker has joined #openstack-nova07:57
*** hferenc has quit IRC08:04
*** itlinux has quit IRC08:04
*** tojuvone has joined #openstack-nova08:07
*** alexchadin has joined #openstack-nova08:12
*** yamahata has quit IRC08:19
*** alexchadin has quit IRC08:22
*** alexchadin has joined #openstack-nova08:22
*** yamamoto has joined #openstack-nova08:23
*** claudiub|2 has quit IRC08:26
*** pcaruana has quit IRC08:27
*** openstackgerrit has joined #openstack-nova08:28
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084408:28
*** markvoelker has quit IRC08:30
*** pcaruana has joined #openstack-nova08:31
*** openstackgerrit has quit IRC08:33
*** openstackgerrit has joined #openstack-nova08:33
openstackgerritHuan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources  https://review.openstack.org/51118008:33
*** suresh12 has joined #openstack-nova08:34
openstackgerritHuan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources  https://review.openstack.org/51118008:36
*** suresh12 has quit IRC08:39
*** spectr has quit IRC08:40
*** edand_ has joined #openstack-nova08:41
openstackgerritHuan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources  https://review.openstack.org/51118008:41
openstackgerritsahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus  https://review.openstack.org/51089708:41
*** edand has quit IRC08:43
*** danpawlik has quit IRC08:44
openstackgerritHuan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources  https://review.openstack.org/51118008:45
kashyapmdbooth: Hi there, when you get a moment:08:49
mdboothkashyap: Wassup?08:50
kashyapmdbooth: 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 mean08:50
*** spectr has joined #openstack-nova08:50
kashyapTake a look at the ouput of this command: virsh qemu-monitor-command f26-devstack --pretty '{"execute":"query-migrate-parameters"}'08:51
kashyaphttp://paste.openstack.org/show/623292/08:51
stephenfinkashyap: to the best of my knowledge, if it can't be done via libvirt then we don't do it08:52
* kashyap thinks we just let libvirt do the querying of migration parameters08:52
*** derekh has joined #openstack-nova08:52
mdboothkashyap: nova never talks to qemu directly if that's what you mean?08:52
kashyapstephenfin: Yep.  I know we don't "do direct QEMU", that's the whole frelling point of explicitly modelling in libvirt XML aspects08:53
mdboothIn fact, I don't think qemu allows more than one monitor, does it?08:53
mdboothi.e. it's not even possible08:53
kashyapmdbooth: I know Nova never talks to QEMU08:54
kashyapmdbooth: And QEMU does allow more than one monitor.08:54
mdboothkashyap: I didn't understand the question, then08:54
kashyapmdbooth: Let me give a bit more context:08:54
mdboothNova does set some of those values explicitly, via libvirt08:55
kashyapmdbooth: What made me ask the question is, one of the QEMU migration devs asked me this morning what migration parameters Nova uses08:55
*** lucas-afk is now known as lucasagomes08:55
kashyapmdbooth: You (libvirt could) set migration parameters via `migrate-set-parameters` QMP command, _except_ this "xbzrle-cache-size"08:56
kashyapAnd they were wondering does Nova use it in anyway.  So I went digging...08:56
mdboothYes, absolutely. Nova tweaks a bunch of those.08:57
*** yamamoto has quit IRC08:58
*** alexchadin has quit IRC08:58
*** belmoreira has quit IRC08:58
*** alexchadin has joined #openstack-nova08:59
* kashyap looks09:00
kashyapmdbooth: Meanwhile, here's an example for multiple monitors:09:00
kashyap    $ qemu-system-x86_64 [...] -monitor stdio -qmp unix:./qmp-sock,server,nowait09:00
kashyapHMP (Human Monitor Protocol) on 'stdio', and QMP on a UNIX socket.09:01
*** yamamoto has joined #openstack-nova09:01
*** tetsuro has quit IRC09:06
openstackgerritAndrey Volkov proposed openstack/osc-placement master: [WIP] CLI for resource classes  https://review.openstack.org/51118209:06
openstackgerritAndrey Volkov proposed openstack/osc-placement master: [WIP] RP list: member_of and resources parameters  https://review.openstack.org/51118309:06
*** yamamoto has quit IRC09:06
openstackgerritAlex Xu proposed openstack/nova master: [DNM] Add _get_provider_ids_with_any_trait  https://review.openstack.org/51118409:06
openstackgerritAlex Xu proposed openstack/nova master: [DNM] placement: add traits support in the AllocationCandidates object  https://review.openstack.org/51118509:06
*** danpawlik has joined #openstack-nova09:06
*** belmoreira has joined #openstack-nova09:08
*** belmoreira has quit IRC09:08
*** belmoreira has joined #openstack-nova09:10
*** lpetrut has joined #openstack-nova09:10
*** sambetts|afk is now known as sambetts09:11
*** dtantsur|afk is now known as dtantsur09:11
openstackgerritsahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host  https://review.openstack.org/51118809:12
*** ratailor has joined #openstack-nova09:14
*** dr_gogeta86 has joined #openstack-nova09:16
*** dr_gogeta86 has quit IRC09:16
*** dr_gogeta86 has joined #openstack-nova09:16
*** priteau has joined #openstack-nova09:20
*** lpetrut_ has joined #openstack-nova09:25
*** markvoelker has joined #openstack-nova09:28
*** zsli__ has quit IRC09:28
*** lpetrut has quit IRC09:28
*** ralonsoh_ has joined #openstack-nova09:32
*** tiendc has joined #openstack-nova09:33
*** ralonsoh has quit IRC09:36
*** jaosorior has quit IRC09:36
*** fragatina has quit IRC09:37
*** mdnadeem has joined #openstack-nova09:38
*** mdnadeem_ has quit IRC09:39
*** rmart04 has joined #openstack-nova09:40
*** mvk has quit IRC09:40
*** thorst has joined #openstack-nova09:42
openstackgerritjichenjc proposed openstack/nova master: Remove duplicate error info  https://review.openstack.org/51071909:42
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895509:44
*** jaosorior has joined #openstack-nova09:47
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895509:48
*** thorst has quit IRC09:48
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896409:48
*** tbachman has quit IRC09:49
*** tbachman has joined #openstack-nova09:50
*** avolkov has quit IRC09:53
*** belmoreira has quit IRC09:53
*** avolkov has joined #openstack-nova09:55
*** gszasz has joined #openstack-nova09:55
*** udesale has quit IRC09:58
*** alexchadin has quit IRC09:58
*** yangyapeng has quit IRC09:58
*** alexchadin has joined #openstack-nova09:58
*** yangyapeng has joined #openstack-nova09:58
openstackgerritHironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration  https://review.openstack.org/50067709:59
*** yangyapeng has quit IRC09:59
openstackgerritjichenjc proposed openstack/nova master: Remove duplicate error info  https://review.openstack.org/51071909:59
*** TuanLA has quit IRC10:00
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060810:00
*** markvoelker has quit IRC10:01
*** yangyapeng has joined #openstack-nova10:01
*** jichen has quit IRC10:03
*** sdague has joined #openstack-nova10:03
*** trinaths has quit IRC10:06
lyarwoodmdbooth: 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:06
mdboothlyarwood: On a call. Sync up in a bit.10:07
-openstackstatus- NOTICE: The CI system will be offline starting at 11:00 UTC (in just under an hour) for Zuul v3 rollout: http://lists.openstack.org/pipermail/openstack-dev/2017-October/123337.html10:08
*** andreas_s has quit IRC10:09
*** andreas_s has joined #openstack-nova10:10
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060810:10
*** mdnadeem has quit IRC10:12
openstackgerritAndrey Volkov proposed openstack/nova master: AZ operations: check host has no instances  https://review.openstack.org/50920610:12
*** andreas_s has quit IRC10:14
*** ratailor has quit IRC10:21
*** ratailor has joined #openstack-nova10:22
*** suresh12 has joined #openstack-nova10:22
*** mdnadeem has joined #openstack-nova10:26
*** vks1 has quit IRC10:26
*** suresh12 has quit IRC10:26
*** yangyapeng has quit IRC10:29
*** inara has quit IRC10:29
*** alexchadin has quit IRC10:31
*** inara has joined #openstack-nova10:31
*** Qiming has quit IRC10:32
*** ratailor_ has joined #openstack-nova10:41
*** smatzek has joined #openstack-nova10:43
*** ratailor has quit IRC10:45
*** tiendc has quit IRC10:50
*** Qiming has joined #openstack-nova10:54
*** edand_ has quit IRC10:55
*** erlon has joined #openstack-nova10:57
*** markvoelker has joined #openstack-nova10:58
*** Guest72138 has quit IRC10:59
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948310:59
*** phuongnh has quit IRC11:00
*** AlexeyAbashkin has quit IRC11:00
*** AlexeyAbashkin has joined #openstack-nova11:01
*** cali_boxer has quit IRC11:01
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types  https://review.openstack.org/43083911:01
*** cali_boxer has joined #openstack-nova11:02
*** nicolasbock has joined #openstack-nova11:03
*** sergek_ has quit IRC11:03
*** brault has quit IRC11:04
*** masuberu has joined #openstack-nova11:05
*** itlinux has joined #openstack-nova11:05
openstackgerritIldiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot  https://review.openstack.org/51120911:05
*** sergek_ has joined #openstack-nova11:06
*** masber has quit IRC11:07
*** andreas_s has joined #openstack-nova11:09
*** vks1 has joined #openstack-nova11:10
*** mvk has joined #openstack-nova11:12
*** belmoreira has joined #openstack-nova11:16
*** chyka has joined #openstack-nova11:16
avolkovdansmith 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 from11:18
avolkovaggregate with AZ if host has VMs. Is it something we can move further with?11:18
openstackgerritMerged openstack/nova master: api-ref: note the microversions for GET /resource_providers query params  https://review.openstack.org/50216811:19
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP)  https://review.openstack.org/50300111:20
*** hoonetorg has quit IRC11:21
*** chyka has quit IRC11:21
*** udesale has joined #openstack-nova11:21
*** acormier has joined #openstack-nova11:22
*** abhishek has joined #openstack-nova11:22
*** lucasagomes is now known as lucas-hungry11:23
*** larainema has quit IRC11:25
*** abhi89 has quit IRC11:26
*** markvoelker has quit IRC11:30
*** efried has joined #openstack-nova11:32
*** acormier has quit IRC11:33
*** avtar_ has joined #openstack-nova11:33
*** hoonetorg has joined #openstack-nova11:34
*** alexchadin has joined #openstack-nova11:35
*** yangyapeng has joined #openstack-nova11:35
openstackgerritMatthew Booth proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices  https://review.openstack.org/51010611:36
mdboothlyarwood: Thanks!11:36
mdboothlyarwood: Do you recall who approved it previously, btw?11:36
*** takashin has left #openstack-nova11:37
mdboothlyarwood: 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:38
*** ratailor__ has joined #openstack-nova11:38
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084411:39
*** rmart04 has quit IRC11:41
*** ratailor_ has quit IRC11:41
*** jpena is now known as jpena|lunch11:42
lyarwoodmdbooth: Matt AFAIK11:42
mdboothlyarwood: Thanks. Makes sense as he asked for the microversion.11:43
*** acormier has joined #openstack-nova11:43
*** spectr has quit IRC11:44
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for neutron client  https://review.openstack.org/50989211:45
*** acormier has quit IRC11:45
*** spectr has joined #openstack-nova11:46
*** ratailor__ has quit IRC11:49
*** spectr-RH has joined #openstack-nova11:54
*** claudiub|2 has joined #openstack-nova11:55
*** litao__ has quit IRC11:55
*** yamamoto has joined #openstack-nova11:55
*** eharney has joined #openstack-nova11:57
*** spectr has quit IRC11:57
*** smatzek has quit IRC11:58
*** yamamoto has quit IRC11:59
*** larainema has joined #openstack-nova12:00
*** dave-mccowan has joined #openstack-nova12:00
*** lucas-hungry is now known as lucasagomes12:01
*** josecastroleon has joined #openstack-nova12:02
*** andreas_s has quit IRC12:02
*** andreas_s has joined #openstack-nova12:02
*** edmondsw has joined #openstack-nova12:04
*** thorst has joined #openstack-nova12:06
*** yamamoto has joined #openstack-nova12:08
*** Yikun has joined #openstack-nova12:10
*** liusheng has quit IRC12:10
*** yikun_jiang has quit IRC12:10
*** suresh12 has joined #openstack-nova12:10
*** itlinux has quit IRC12:10
*** liusheng has joined #openstack-nova12:10
*** andreas_s has quit IRC12:11
*** spectr-RH has quit IRC12:13
*** spectr has joined #openstack-nova12:13
*** itlinux has joined #openstack-nova12:14
*** spectr-RH has joined #openstack-nova12:14
*** suresh12 has quit IRC12:15
*** liusheng has quit IRC12:15
*** smatzek has joined #openstack-nova12:16
*** andreas_s has joined #openstack-nova12:16
*** spectr has quit IRC12:18
*** openstackgerrit has quit IRC12:18
*** markvoelker has joined #openstack-nova12:18
*** diga has quit IRC12:19
*** andreas_s has quit IRC12:21
*** andreas_s has joined #openstack-nova12:21
*** andreas_s has quit IRC12:25
*** andreas_s has joined #openstack-nova12:26
mdboothmriedem: This is a spec which will hopefully be familiar to you: https://review.openstack.org/#/c/510106/12:29
*** vks1 has quit IRC12:29
*** andreas_s has quit IRC12:30
*** priteau has quit IRC12:30
*** andreas_s has joined #openstack-nova12:31
*** yamamoto has quit IRC12:31
*** openstackgerrit has joined #openstack-nova12:32
openstackgerritMatthew Booth proposed openstack/nova master: Move is_block_device_mapping to new_format_is_volume  https://review.openstack.org/51121712:32
openstackgerritMatthew Booth proposed openstack/nova master: Remove driver.block_device.get_swap  https://review.openstack.org/51121812:32
*** liverpooler has joined #openstack-nova12:33
*** sshwarts has quit IRC12:34
*** itlinux has quit IRC12:35
*** jpena|lunch is now known as jpena12:35
*** itlinux has joined #openstack-nova12:35
*** pchavva has joined #openstack-nova12:35
openstackgerritElod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification  https://review.openstack.org/50850612:38
*** thorst_ has joined #openstack-nova12:38
*** esberglu has quit IRC12:38
*** thorst has quit IRC12:39
*** andreas_s has quit IRC12:40
*** belmoreira has quit IRC12:41
*** catintheroof has joined #openstack-nova12:43
bauzasavolkov: ta. FWIW, I wrote a very old spec for that but looks like the consensus is that it's not needing a microversion12:44
*** rodolof has joined #openstack-nova12:44
stephenfinsahid: Want to take a look at https://review.openstack.org/#/c/361140/ ?12:45
*** andreas_s has joined #openstack-nova12:45
*** alexchadin has quit IRC12:46
*** lyan has joined #openstack-nova12:46
* alex_xu nova api meeting is in 15 mins at #openstack-meeting-412:50
alex_xus/15 mins/10 mins/...12:50
sahidstephenfin: yes i will, ok12:50
*** sridharg has quit IRC12:51
*** fragatina has joined #openstack-nova12:51
*** sapcc-bot has quit IRC12:52
*** sapcc-bot has joined #openstack-nova12:52
*** mkoderer_ has joined #openstack-nova12:52
*** dgonzalez_ has joined #openstack-nova12:52
*** tpatzig_ has joined #openstack-nova12:52
*** carthaca_ has joined #openstack-nova12:52
*** seife_ has joined #openstack-nova12:52
*** david_1 has joined #openstack-nova12:52
stephenfinlyarwood, mdbooth: Fancy taking a look at this? Looks sane to me, at least https://review.openstack.org/#/c/290793/12:53
*** smatzek has quit IRC12:53
*** seife_ has quit IRC12:54
*** dgonzalez_ has quit IRC12:54
*** mkoderer_ has quit IRC12:54
*** carthaca_ has quit IRC12:54
*** tpatzig_ has quit IRC12:54
*** david_1 has quit IRC12:54
*** takashin has joined #openstack-nova12:54
*** andreas_s has quit IRC12:55
mdboothstephenfin: Looking12:55
*** alexchadin has joined #openstack-nova12:55
*** esberglu has joined #openstack-nova12:57
stephenfinralonsoh_: About? Got a question on https://review.openstack.org/#/c/484726/12:58
*** andreas_s has joined #openstack-nova12:59
*** penick has joined #openstack-nova13:00
*** vladikr has joined #openstack-nova13:01
*** vladikr_ has joined #openstack-nova13:01
*** vladikr_ has quit IRC13:02
*** vladikr_ has joined #openstack-nova13:02
*** openstackgerrit has quit IRC13:03
*** takashin has quit IRC13:04
-openstackstatus- NOTICE: Due to unrelated emergencies, the Zuul v3 rollout has not started yet; stay tuned for further updates13:06
*** takashin has joined #openstack-nova13:06
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 needed13:06
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 them13:07
ralonsoh_I'm on it]13:07
stephenfinralonsoh_: Cool cool :) Maybe stick it in WIP until then?13:08
ralonsoh_sure! I'll do this now13:08
efriedgibi 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
*** openstackgerrit has joined #openstack-nova13:09
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Open vSwitch patch port  https://review.openstack.org/48522813:09
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Virtual Ethernet device pair  https://review.openstack.org/48472613:09
*** takedakn has joined #openstack-nova13:10
*** sdake has quit IRC13:11
*** sdake has joined #openstack-nova13:11
*** sdake has quit IRC13:11
*** sdake has joined #openstack-nova13:11
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084413:12
*** mriedem has joined #openstack-nova13:13
*** sdake has quit IRC13:15
*** acormier has joined #openstack-nova13:15
*** acormier has quit IRC13:18
*** acormier has joined #openstack-nova13:18
*** sdake has joined #openstack-nova13:19
*** sdake has quit IRC13:19
*** sdake has joined #openstack-nova13:19
*** acormier has quit IRC13:20
*** acormier has joined #openstack-nova13:20
*** zhurong has joined #openstack-nova13:20
*** smatzek has joined #openstack-nova13:22
*** baoli has joined #openstack-nova13:22
*** gouthamr has joined #openstack-nova13:23
*** mpaolino has joined #openstack-nova13:23
*** smatzek has quit IRC13:24
*** smatzek has joined #openstack-nova13:25
*** takashin has left #openstack-nova13:27
mriedemgibi: https://review.openstack.org/#/c/510938/13:28
*** smatzek has quit IRC13:29
*** takedakn has quit IRC13:30
sahidstephenfin: it's not clear for me how are you make working "preferred", I mean in a technical POV13:33
sahidhttps://review.openstack.org/#/c/361140/28/specs/queens/approved/share-pci-between-numa-nodes.rst13:33
sahidstephenfin: you are probably going to use a weigher?13:33
gmannmriedem: is it ok for you now - https://review.openstack.org/#/c/459483/33/nova/api/openstack/compute/server_migrations.py@5813:33
gmannmriedem: not having whitelist to add 'type' in response13:34
stephenfinsahid: What aspect of it?13:34
sahidscheduling13:34
*** penick has quit IRC13:35
*** jaypipes has joined #openstack-nova13:35
*** priteau has joined #openstack-nova13:35
sahidstephenfin: from how i see that you need two pass, and i don't think it"s something that the current scheduler provides13:36
*** sree has joined #openstack-nova13:36
*** itlinux has quit IRC13:36
stephenfinsahid: Yeah, I think you might be right there. We're probably going to stick with making it configurable at the host leve13:37
stephenfini.e. you use the first host that is given to you by the scheduler13:37
*** penick has joined #openstack-nova13:37
sahidstephenfin: seems more reasonable yes13:38
stephenfinI don't think the two-pass solution is possible either. jaypipes might have suggestions though13:38
stephenfinsahid: Cool. I can update the spec accordingly13:38
*** jistr is now known as jistr|mtg13:38
sahidmriedem, jaypipes can one of you look ACK this this https://review.openstack.org/#/c/497457/ ?13:38
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084413:41
*** hshiina has quit IRC13:43
*** udesale has quit IRC13:43
mriedemsahid: need to see the live migration job pass on that first13:43
mriedemstephenfin: ^13:43
*** mdnadeem has quit IRC13:46
*** sballe_ has joined #openstack-nova13:47
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084413:47
*** burt has joined #openstack-nova13:48
*** avtar_ has quit IRC13:49
sahidjaypipes: i replied to you about the sriov-trusted-vfs (in case you did not notice it)13:49
*** armax has joined #openstack-nova13:49
*** crushil has joined #openstack-nova13:51
*** udesale has joined #openstack-nova13:55
*** jmlowe has joined #openstack-nova13:55
*** awaugama has joined #openstack-nova13:56
*** links has quit IRC13:56
*** alexchadin has quit IRC13:58
*** jistr|mtg is now known as jistr14:00
mdboothstephenfin: 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:02
* mdbooth likes data constraints.14:02
*** gmann is now known as gmann_afk14:03
*** spectr-RH has quit IRC14:04
mriedemsahid: which hypervisors don't support this flag? https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_MIGRATE_PARAM_BANDWIDTH14:04
*** spectr has joined #openstack-nova14:04
*** jwcroppe has joined #openstack-nova14:07
sahidmriedem: hum.. let me check that point14:08
*** slaweq_ has quit IRC14:08
*** chyka has joined #openstack-nova14:09
openstackgerritMerged openstack/nova master: Don't fix protocol-less glance api_servers anymore  https://review.openstack.org/50531714:09
sahidmriedem: according to a simple grep in libvirt source it seems that vz and QEMU are supporting it14:12
sahidbut but nothing about vmware, hyperv..14:12
*** gbarros has joined #openstack-nova14:12
*** lyan has quit IRC14:13
*** chyka has quit IRC14:13
mriedemsahid: well we wouldn't care about vmware or hyperv in the libvirt driver14:13
*** eharney has quit IRC14:13
mriedembut we care about xen, although i'm not sure if libvirt+xen is listed as supporting live migration anyway14:13
openstackgerritEric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax  https://review.openstack.org/51024414:13
mriedemaccording to this it says we do https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_live_migrate14:13
*** rajinir-afk is now known as rajinir14:14
*** rmart04 has joined #openstack-nova14:14
openstackgerritMerged openstack/nova master: Move user_data max length check to schema  https://review.openstack.org/51070414:14
*** penick has quit IRC14:14
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084414:14
*** artom has joined #openstack-nova14:14
*** gbarros has quit IRC14:17
*** lyan has joined #openstack-nova14:17
*** gbarros has joined #openstack-nova14:17
sahidmriedem: we always use that parameter (bandwiddh) for live-migration14:18
*** takedakn has joined #openstack-nova14:18
sahiddo you have a suggestion on how the code should be?14:19
mriedemit must work for xen then14:19
sahidI guess yes14:19
mriedemlibvirt+xen i mean14:19
sahidbut I can't ensure that14:19
mriedemanthonyper: do you know what the current state of libvirt+xen live migration is? working?14:19
mriedemhas anyone tested lately?14:19
anthonypermriedem: I have no idea :(14:25
*** takedakn has quit IRC14:25
*** zhurong has quit IRC14:29
*** acormier has quit IRC14:30
*** trinaths has joined #openstack-nova14:30
*** acormier has joined #openstack-nova14:31
*** lajoskatona has quit IRC14:32
anthonypermriedem: I'll ask other people to try to find out if they have tested live migration recently.14:33
*** udesale has quit IRC14:34
mriedemanthonyper: ok thanks14:34
*** acormier has quit IRC14:35
sahiddansmith: if the wait_for_instance_event has been desgined has you are suggesting so it's not working14:36
*** acormier has joined #openstack-nova14:37
dansmithsahid: I'm not sure what you're saying14:38
jaypipessahid: yep, noticed it. sorry, been spending my last days rebasing the nested resource providers series.14:38
jaypipessahid: I'll review that slow live-migration patch this morning.14:38
openstackgerritMerged openstack/nova master: Add connection pool size to vSphere settings  https://review.openstack.org/41408514:38
*** felipemonteiro has joined #openstack-nova14:38
openstackgerritSurya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database  https://review.openstack.org/51084414:39
sahidjaypipes: no worries and thanks if you can have a look14:39
*** mvk has quit IRC14:39
openstackgerritMerged openstack/nova master: Clarify conf/compute.py help text for ListOpts  https://review.openstack.org/46853814:39
sahiddansmith: related to one of your comment on https://review.openstack.org/#/c/497457/14:39
sahidthe other one are just nits, right?14:40
*** felipemonteiro_ has joined #openstack-nova14:40
dansmithsahid: no, breaking n-net migrations is not a nit :)14:41
*** acormier has quit IRC14:42
sahiddansmith: can you clearly put a suggestion? do you me to add a condition in the code?14:42
dansmithsahid: you can handle it like the spawn code does for if neutron isn't enabled14:43
mriedemif (self._conn_supports_start_paused and14:43
mriedem            utils.is_neutron() and not14:43
mriedem            vifs_already_plugged and power_on and timeout):14:43
mriedem            events = self._get_neutron_events(network_info)14:43
mriedem        else:14:43
mriedem            events = []14:43
mriedemyeah14:43
mriedemif this and that and the other thing else something different14:43
*** spectr has quit IRC14:43
sahiddansmith: seems reasonable14:43
*** felipemonteiro has quit IRC14:44
dansmithsahid: I also want to know which specific action starts the chain of events that will result in neutron sending an event to nova, so that you can properly wrap that thing with the waiter14:44
sahiddansmith: depending of the mech used, i added a comment in the code14:45
dansmithsahid: ah, so two different places depending on OVS or linuxbridge? that's going to be a mess to handle properly14:47
dansmithbut what you have isn't right for either14:47
dansmithalthough it's much much wronger for OVS14:47
*** suresh12 has joined #openstack-nova14:47
sahiddansmith: i guess you are going to elaborate?14:50
dansmithsahid: I did in my comments14:50
dansmithnova events are not queued14:50
dansmithif you don't have a waiter running before an event arrives, it is dropped14:50
dansmithif you don't want to race, you need to have a waiter set up before you do the thing that triggers the event from the other service14:51
*** suresh12 has quit IRC14:51
dansmithdelaying live migration for five minutes, timeout, log error and then continue is not a good thing to do just because you raced with neutron14:51
mdboothmriedem: To spec, or not to spec: https://blueprints.launchpad.net/nova/+spec/local-disk-serial-numbers ?14:54
mdboothdansmith: Same question ^^^14:55
sahiddansmith: i see so we will have to add this specificly for ovs during pre-live-migration when we are creating the bridge on dest14:55
sahidand when we start calling libvirt.migrateToURI for LB14:55
*** penick has joined #openstack-nova14:56
dansmithsahid: neither option seems very ideal to me, but yes, that's the issue with what you have here14:56
dansmithmdbooth: probably a spec so we can argue about your id generation method14:57
mdboothdansmith: Yeay! I do love a good opportunity to argue about stuff.14:57
mdboothOk14:57
*** spectr has joined #openstack-nova14:58
mriedemgmann_afk: commented in that change and the one below it14:58
*** penick has quit IRC15:00
*** penick has joined #openstack-nova15:00
*** psachin has quit IRC15:02
*** trinaths has left #openstack-nova15:03
*** larainema has quit IRC15:05
*** sree has quit IRC15:06
*** sree has joined #openstack-nova15:07
sahiddansmith: since we are now using the events, do you see a blocker at moving plug_vifs() from pre-live-migration to migration step?15:07
*** eharney has joined #openstack-nova15:07
mriedemsahid: i looked at one of the live migration job runs on your patch and it's never waiting for any events15:07
openstackgerritEric Fried proposed openstack/nova master: Nix bug msg from ConfGroupForServiceTypeNotFound  https://review.openstack.org/51126115:08
efriedmriedem ^ cleanup as requested.15:08
*** Oku_OS is now known as Oku_OS-away15:08
sahidmriedem: what is the mech? for linux bridge we need that patch on neutron side15:08
sahidhttps://review.openstack.org/#/c/506182/15:08
mriedemsahid: the depends-on will pull the change in via zuul15:09
mriedemsahid: it's literally building an empty list of events15:09
mriedemmeaning there are no vifs in the instance.info_cache15:09
mriedemthis would be ovs15:09
*** markus_z has quit IRC15:09
mriedemhttp://logs.openstack.org/57/497457/11/check/legacy-tempest-dsvm-multinode-live-migration/9cd842e/logs/screen-n-cpu.txt.gz#_Oct_02_11_04_06_17267515:10
sahidmriedem: hm... if there is no vifs so yes i guess it's normal15:10
*** sree has quit IRC15:11
mriedemhmm wtf http://logs.openstack.org/57/497457/11/check/legacy-tempest-dsvm-multinode-live-migration/9cd842e/logs/screen-n-cpu.txt.gz#_Oct_02_11_04_01_31535415:11
*** links has joined #openstack-nova15:11
mriedemcfriesen: ^ this probably explains why i never saw any improvement in the network-changed event queueing patch for live migration15:11
sahidhum...15:12
mriedemcfriesen: this one https://review.openstack.org/#/c/465787/15:12
*** jwcroppe has quit IRC15:15
*** smatzek has joined #openstack-nova15:16
*** jianghuaw_ has joined #openstack-nova15:16
*** jwcroppe has joined #openstack-nova15:16
efriedWe needing to pay attention to bogus Zuul merge conflicts at this point?15:17
mriedemno15:17
mriedemwell, idk, but i'm not15:17
dansmithtoday we do15:17
efriedThought they cut over this morning.15:17
dansmithonce the cutover starts15:17
*** ralonsoh_ has quit IRC15:18
*** smatzek has quit IRC15:18
*** suresh12 has joined #openstack-nova15:19
*** smatzek has joined #openstack-nova15:19
*** kylek3h has joined #openstack-nova15:20
*** jwcroppe has quit IRC15:20
*** andreas_s has quit IRC15:29
*** andreas_s has joined #openstack-nova15:29
*** gyee has joined #openstack-nova15:33
jianghuaw_jaypipes, per the previous, we should populate the inventory for vgpu in the function of get_inventory. So we should re-open this abandoned patch: https://review.openstack.org/#/c/339204/.  Is it correct?15:35
*** zen369 has quit IRC15:38
*** andreas_s has quit IRC15:38
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/51102115:38
*** rodolof has quit IRC15:39
bauzasjianghuaw: I tend to say yes15:40
jaypipesjianghuaw_: yes15:40
bauzasjianghuaw: FWIW, I'm working on the libvirt side15:40
jaypipesjianghuaw_: feel free to take it.15:40
jaypipesjianghuaw_: I restored it just now. feel free to git review -d 339204 and take over ownership15:41
jaypipesjianghuaw_: or cherry-pick it as you want\15:41
jianghuaw_bauzas, jaypipes: great. Thanks.15:41
*** smatzek has quit IRC15:41
*** smatzek has joined #openstack-nova15:42
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005715:42
*** hongbin has joined #openstack-nova15:43
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for keystone conf & requests  https://review.openstack.org/50769315:44
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for cinder client  https://review.openstack.org/50834515:44
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for neutron client  https://review.openstack.org/50989215:44
openstackgerritmelanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells  https://review.openstack.org/51069115:45
openstackgerritmelanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage  https://review.openstack.org/50996815:45
openstackgerritmelanie witt proposed openstack/nova master: Make expected notifications output easier to read in tests  https://review.openstack.org/51082315:45
openstackgerritmelanie witt proposed openstack/nova master: Improve the CellDatabases test fixture and usage  https://review.openstack.org/50843215:46
*** jianghuaw_ has quit IRC15:46
*** jianghuaw_ has joined #openstack-nova15:47
*** smatzek has quit IRC15:47
openstackgerritIldiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot  https://review.openstack.org/51120915:48
*** penick_ has joined #openstack-nova15:51
*** smatzek has joined #openstack-nova15:51
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028515:52
openstackgerritIldiko Vancsa proposed openstack/nova master: DNM: Fail if we try to update volume connections during reboot  https://review.openstack.org/51103115:52
ildikovmriedem: attempt to fix the attachment_update mystery ^^15:52
*** AlexeyAbashkin has quit IRC15:53
*** penick has quit IRC15:53
*** ratailor has joined #openstack-nova15:54
mriedemildikov: HA!15:54
mriedemholy shit nice catch15:54
*** ralonsoh has joined #openstack-nova15:54
ildikovmriedem: my pleasure :)15:55
ildikovjgriffith: it seems we're off the hook this time :) ^^15:56
mriedemthere is another one like that in here15:56
mriedemmight as well fix that one too15:56
*** nicolasbock has quit IRC15:56
*** abhi89 has joined #openstack-nova15:56
*** slaweq has joined #openstack-nova15:57
*** pcaruana has quit IRC15:59
*** rcernin has quit IRC15:59
*** abhishek has quit IRC15:59
*** kylek3h has quit IRC16:01
*** kylek3h has joined #openstack-nova16:02
*** rodolof has joined #openstack-nova16:02
*** lucasagomes is now known as lucas-afk16:02
*** ratailor has quit IRC16:03
*** jianghuaw_ has quit IRC16:03
openstackgerritJay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class  https://review.openstack.org/50902916:03
openstackgerritJay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid()  https://review.openstack.org/50903016:03
openstackgerritJay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db()  https://review.openstack.org/50903116:03
openstackgerritJay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record  https://review.openstack.org/50903216:03
openstackgerritJay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid  https://review.openstack.org/50903316:04
openstackgerritJay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id()  https://review.openstack.org/50903516:04
openstackgerritJay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree  https://review.openstack.org/47057516:04
openstackgerritJay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client  https://review.openstack.org/41592116:04
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713816:04
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721516:04
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480716:04
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569316:04
*** rodolof has quit IRC16:04
*** sahid has quit IRC16:04
*** rodolof has joined #openstack-nova16:04
*** slaweq has quit IRC16:05
*** rodolof has quit IRC16:06
*** kylek3h has quit IRC16:06
*** rodolof has joined #openstack-nova16:06
ildikovmriedem: will do, thanks16:07
*** rodolof has quit IRC16:08
ildikovmriedem: I was too eager to get a clean run on the new attach patch :)16:08
*** rodolof has joined #openstack-nova16:08
*** nicolasbock has joined #openstack-nova16:09
*** kylek3h has joined #openstack-nova16:09
*** kylek3h has quit IRC16:09
*** hemna_ has joined #openstack-nova16:10
*** itlinux has joined #openstack-nova16:11
openstackgerritsean mooney proposed openstack/nova-specs master: Use neutron's new port binding API  https://review.openstack.org/37558016:11
dansmithmdbooth: if you want that bdm uuid patch for the serial number thing, I think we should slam it in soonly16:12
*** hemna_ has quit IRC16:13
dansmithI think everyone knows it's a good idea and uncontentious, it's just never been in the critical path for anyone for long enough16:13
dansmithso I say get it cleaned up and we should be able to get it in, IMHO16:13
mdboothdansmith: I'm going to write a spec for this first thing in the morning. Can I combine it with the serial number thing in 1 spec?16:13
dansmithmdbooth: the spec is the serial number thing, uuid on bdm is just a work item in that spec16:14
openstackgerritEric Fried proposed openstack/nova master: DNM: CI check  https://review.openstack.org/51100616:14
mdboothIncidentally, having a DriverBlockDevice object for the root disk is also something which is useful for a bunch of other stuff.16:14
mdboothOk. Look for me adding you to it tomorrow :)16:15
*** rmart04 has quit IRC16:15
*** kylek3h has joined #openstack-nova16:15
*** hemna_ has joined #openstack-nova16:16
mriedemmelwitt: dansmith: see comment #16, i thought we had this fixed already? https://bugs.launchpad.net/nova/+bug/168242316:18
openstackLaunchpad bug 1682423 in OpenStack Compute (nova) "create instance with dashbaord / 404 returned" [Undecided,Confirmed]16:18
dansmithmriedem: afaik yeah16:21
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_image_certificates  https://review.openstack.org/50039616:21
dansmithmriedem: "latest available ocata packages" doesn't mean much to me16:21
mriedemi meant 15.0.716:22
mriedemnot whatever ubuntu has currently16:22
dansmithI mean the person who said it's still happening for them16:22
mriedemoh16:22
*** rodolof has quit IRC16:23
mriedemi seem to remember some tempest bugs we were trying to fix around the time of this change https://review.openstack.org/#/c/427782/16:23
*** kylek3h has quit IRC16:25
mdboothmriedem: Any chance you could look at https://review.openstack.org/#/c/510106/ ? It's a reapproval. You asked for some changes during Newton, which I've added to this version of the spec.16:25
*** kylek3h has joined #openstack-nova16:25
mriedemnot right now16:26
mdboothlyarwood's stable virt16:26
mdboothmriedem: Ok. Is it likely to make next week?16:26
mriedemidk16:26
mriedemlots of people can review specs16:26
mriedemi've got my own to write and update as well16:26
mriedemkind of concerned with a bunch of people saying they are busted on ocata atm16:27
mdboothmriedem: np, just pinging you specifically as you've been involved with it previously.16:27
*** kylek3h has quit IRC16:29
*** yamahata has joined #openstack-nova16:30
mriedemdansmith: wonder if these people are running with uwsgi? :)16:32
*** acormier has joined #openstack-nova16:32
mriedemremember that defeated the service version check16:32
mriedemfor tripleo16:32
dansmithmriedem: ocata so no right?16:32
dansmithoh16:32
mriedembut they were hitting something similar,16:32
mriedemunti lwe told them not to do that16:32
*** weshay is now known as weshay|ruck16:32
dansmithhmm16:32
openstackgerritIldiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot  https://review.openstack.org/51120916:33
dansmithmriedem: see comment 916:33
dansmithmriedem: makes reference to having everything upgraded16:33
*** Swami has joined #openstack-nova16:35
mriedemyeah i think there are two different groups of people with different issues16:35
mriedemonce is talking about having old computes, which shouldn't have anything to do with looking up the instnace16:35
mriedemi think the others are talking about a race to lookup the instance while it's building16:36
*** mpaolino has quit IRC16:36
mriedemwhich is why they added a sleep in there i guess and said it's working16:36
dansmithwell, one unupdated api will wreck it right?16:36
mriedemb/c of the api version check?16:36
dansmithyeah16:36
dansmithit'll go straight to the cell db and not look at the BR16:37
*** abhishek has joined #openstack-nova16:37
mriedemyeah16:37
mriedembut, someone said they have a full new install16:37
mriedemand that was the person that put the time.sleep(1) workaround in16:37
dansmithall they need is one api service record that reported zero for the version16:38
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider  https://review.openstack.org/50230616:38
*** ralonsoh has quit IRC16:39
*** dtantsur is now known as dtantsur|afk16:39
dansmithand they did it before that check16:39
dansmiththe sleep I mean16:39
*** felipemonteiro_ has quit IRC16:40
*** abhi89 has quit IRC16:40
*** chyka has joined #openstack-nova16:41
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider  https://review.openstack.org/50230616:41
*** chyka has quit IRC16:45
*** jmlowe has quit IRC16:47
mriedemassuming their api services are all updated, i don't see how they could hit this really16:48
mriedemwe delete the build request after the instance mapping is updated,16:48
mriedemwe update the instance mapping after the instance is created in a cell16:48
openstackgerritmelanie witt proposed openstack/nova stable/pike: Target context for build notification in conductor  https://review.openstack.org/51129216:49
mriedemif the instance mapping doesn't have the cell mapping in it, we get the instance from the build request - and if that's gone by the time we look, we check the instance mapping again16:49
dansmithmriedem: aye16:49
mriedemgdi this is like 2 days i've spent all day chasing weird bugs16:49
*** links has quit IRC16:54
*** gszasz has quit IRC16:57
*** derekh has quit IRC16:58
*** andreas_s has joined #openstack-nova16:59
*** baoli has quit IRC16:59
*** felipemonteiro has joined #openstack-nova17:00
*** baoli has joined #openstack-nova17:00
*** Apoorva has joined #openstack-nova17:01
*** felipemonteiro_ has joined #openstack-nova17:01
*** gouthamr has quit IRC17:04
*** andreas_s has quit IRC17:05
*** felipemonteiro has quit IRC17:06
*** andreas_s has joined #openstack-nova17:07
*** jpena is now known as jpena|off17:09
*** suresh12 has quit IRC17:13
*** nikhil has joined #openstack-nova17:14
*** tesseract has quit IRC17:15
*** andreas_s has quit IRC17:15
*** andreas_s has joined #openstack-nova17:16
*** sambetts is now known as sambetts|afk17:17
mriedemmelwitt: does https://review.openstack.org/#/c/511292/ also need to go to ocata?17:21
*** AlexeyAbashkin has joined #openstack-nova17:21
*** READ10 has joined #openstack-nova17:22
melwitthm, lemme check. I had thought not but maybe this code was from ocata17:22
mriedemlooks like yes https://github.com/openstack/nova/blob/stable/ocata/nova/conductor/manager.py#L92117:22
mriedemit also looks like ocata is not targeting the instance action in the cell, so that's getting created in cell017:23
*** gjayavelu has joined #openstack-nova17:23
melwittyeah, was about to say that too17:23
openstackgerritDan Smith proposed openstack/nova master: Revert allocations by migration uuid  https://review.openstack.org/49894917:23
openstackgerritDan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations  https://review.openstack.org/50641917:23
openstackgerritDan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances  https://review.openstack.org/50642017:23
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763817:23
openstackgerritDan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits  https://review.openstack.org/51130617:23
mriedemhttps://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca17:24
dansmithmriedem: ^ updated to determine the source compute from allocations instead of trying to look it up17:24
mriedemmelwitt: https://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca fixed the instance action thing in pike, we'll want to backport that as well...17:24
mriedemdidn't have a bug o nit17:24
melwittah, crap17:24
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Make conductor create InstanceAction in the proper cell  https://review.openstack.org/51130717:25
mriedemthar she blar17:25
melwittk, cool17:25
melwittI should stack the other backport on that I think17:25
mriedemyup17:26
mriedemdansmith: ok17:26
melwittto increase the chance of clean backport. k17:26
mriedemdansmith: +2 on mel's pike backport https://review.openstack.org/#/c/511292/17:26
*** AlexeyAbashkin has quit IRC17:26
mriedemi need food in my belly17:26
*** awaugama has quit IRC17:27
*** links has joined #openstack-nova17:27
*** andreas_s has quit IRC17:29
*** AlexeyAbashkin has joined #openstack-nova17:29
*** tbachman has quit IRC17:29
jaypipesdansmith: I think zuul might be lying about the merge failure messages.17:32
dansmithjaypipes: I don't think they've actually started the cutover yet17:32
dansmithwhich means, yes17:32
*** baoli_ has joined #openstack-nova17:33
*** AlexeyAbashkin has quit IRC17:34
*** tbachman has joined #openstack-nova17:34
melwittit's expected to emit false merge conflicts prior to cutover? I guess I am missing something obvious17:34
dansmithit's been doing that all week17:34
melwittI know17:34
*** fragatina has quit IRC17:35
*** baoli has quit IRC17:35
dansmithI'm just saying, it's been doing it all week, and the changes they have planned today haven't started yet AFAIK, so I think we'd expect the same behavior17:35
dansmithI would assume that whatever has been giving zuul merge conflict heartburn over the last week is factored into the switchover17:36
melwittI had been thinking they were fixing zuul v3 before the cutover so maybe those would get fixed prior to the cutover17:36
melwittyeah. I guess I should ask17:37
dansmithunless the failure is because they're not having it do the post-merge conflict checks because of resources or something17:37
dansmithsince it's highly constrained at the moment, and not doing the actual merges17:37
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713817:37
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721517:37
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480717:37
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569317:37
dansmithI'm not sure what jaypipes' point was I guess17:38
jaypipesefried: k, updated the n-r-p spec to address your comments ^^17:38
efriedThanks jaypipes17:38
jaypipesdansmith: generally, you can assume I am pointless.17:38
efrieddansmith jaypipes They figured out why the bogus merge conflicts were happening.17:38
efriedBut no, they haven't cut over yet.17:38
efriedhttp://eavesdrop.openstack.org/irclogs/%23openstack-infra-incident/%23openstack-infra-incident.2017-10-11.log in case you're interested.17:39
melwittindeed they know about it in #infra and they're working on it17:39
dansmithso yeah, like I said, it's getting old data because it's not doing the actual merges17:40
*** itlinux has quit IRC17:40
melwittokay. I didn't know how it worked17:40
jaypipesefried, dansmith: k, thx for the info.17:41
openstackgerritDan Smith proposed openstack/nova master: Revert allocations by migration uuid  https://review.openstack.org/49894917:41
openstackgerritDan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations  https://review.openstack.org/50641917:41
openstackgerritDan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances  https://review.openstack.org/50642017:41
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763817:42
*** andreas_s has joined #openstack-nova17:43
*** smatzek has quit IRC17:44
*** smatzek has joined #openstack-nova17:45
*** lpetrut_ has quit IRC17:45
abhishekHi all.. can someone please review https://review.openstack.org/#/c/505546/17:45
*** andreas_s has quit IRC17:47
*** smatzek_ has joined #openstack-nova17:48
*** andreas_s has joined #openstack-nova17:48
*** slaweq has joined #openstack-nova17:48
*** smatzek has quit IRC17:49
*** avolkov has quit IRC17:50
*** smatzek_ has quit IRC17:52
efriedjaypipes https://review.openstack.org/#/c/377138/ not sure if I'm off base there...17:52
*** slaweq has quit IRC17:53
*** links has quit IRC17:53
*** andreas_s has quit IRC17:56
*** ijw has joined #openstack-nova17:57
*** kylek3h has joined #openstack-nova17:59
*** smatzek has joined #openstack-nova17:59
*** ijw has quit IRC18:04
*** suresh12 has joined #openstack-nova18:04
*** ijw has joined #openstack-nova18:05
*** andreas_s has joined #openstack-nova18:06
*** suresh12 has quit IRC18:09
*** jwcroppe has joined #openstack-nova18:10
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713818:10
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721518:10
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480718:10
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569318:10
*** lpetrut_ has joined #openstack-nova18:10
jaypipesefried: good feedback, thanks. fixed ^^18:10
efriedjaypipes rgr18:11
efriedsdague https://review.openstack.org/#/c/490031/ succeeded, so that ought to mean https://review.openstack.org/#/c/490057/ is good to go.  (To be extra sure, I'm still waiting for jenkins to vote on https://review.openstack.org/#/c/511006/, which runs the Nova CI with everything in place.)18:11
*** andreas_s has quit IRC18:11
efriedjaypipes Cool man, I think I'm +1 on the whole stack.  Let me know if I missed any.18:12
*** kylek3h has quit IRC18:12
jaypipesefried: thanks for the great reviews. appreciated.18:12
efriedyahyoubetcha :)18:13
*** acormier has quit IRC18:13
*** suresh12 has joined #openstack-nova18:14
openstackgerritPeter Hamilton proposed openstack/nova-specs master: Add support for certificate validation  https://review.openstack.org/48854118:16
*** efried is now known as efried_nomnom18:17
*** jmlowe has joined #openstack-nova18:18
*** brault has joined #openstack-nova18:19
*** artom_ has joined #openstack-nova18:20
*** vvargaszte has joined #openstack-nova18:22
*** artom has quit IRC18:23
*** brault has quit IRC18:23
*** baoli_ has quit IRC18:24
*** baoli has joined #openstack-nova18:24
*** gyee has quit IRC18:27
openstackgerritDan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000  https://review.openstack.org/50859518:28
openstackgerritDan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method  https://review.openstack.org/50900218:28
openstackgerritDan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells  https://review.openstack.org/50900318:28
*** gyee has joined #openstack-nova18:29
mriedemdansmith: if you're going to fix the indent issue in the bottom change in the series, you could add a todo i pointed out in here https://review.openstack.org/#/c/498949/18:32
mriedemthat would be great mmmkay?18:32
*** vvargaszte has quit IRC18:32
melwittthaaaaanks18:32
dansmithmriedem: wonkified? honestly matt...18:33
dansmithmriedem: so, on the shared storage thing,18:34
dansmithmriedem: I think that when we do that we'll have the instance hold its shared storage allocation on the new host,18:35
dansmithnot duplicate it for the migration one18:35
dansmiththe only case would be if we're root-on-shared-ephemeral I guess18:35
dansmithso, I dunno, I'll put a NOTE as I don't really know what the TODO is until later18:36
*** gyee has quit IRC18:37
openstackgerritDan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits  https://review.openstack.org/51130618:37
openstackgerritDan Smith proposed openstack/nova master: Revert allocations by migration uuid  https://review.openstack.org/49894918:37
openstackgerritDan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations  https://review.openstack.org/50641918:37
openstackgerritDan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances  https://review.openstack.org/50642018:37
openstackgerritDan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation  https://review.openstack.org/50763818:37
*** gyee has joined #openstack-nova18:38
mriedemok18:39
mriedemwonkification of the SchedulerReportClient 2.018:39
*** gyee has quit IRC18:39
dansmithheh18:40
*** gyee has joined #openstack-nova18:45
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Spec for flavor description  https://review.openstack.org/50101718:55
mriedemsdague: made those changes on restricting the id and name fields ^18:55
sdaguemriedem: will look18:55
*** efried_nomnom is now known as efried18:58
*** smatzek has quit IRC18:58
*** smatzek has joined #openstack-nova19:00
*** smatzek has quit IRC19:00
*** smatzek has joined #openstack-nova19:01
*** masuberu has quit IRC19:01
*** smatzek_ has joined #openstack-nova19:02
*** smatzek_ has quit IRC19:02
*** smatzek_ has joined #openstack-nova19:02
*** smatzek has quit IRC19:05
openstackgerritMerged openstack/nova master: Update cinder in RequestContext service catalog  https://review.openstack.org/51094719:07
*** brault has joined #openstack-nova19:09
sdaguemriedem: that looks completely sane to me19:12
*** brault has quit IRC19:13
mriedemnot only sane, but completely sane?19:13
mriedemnice19:13
mriedemhuh https://developer.openstack.org/api-ref/placement/#resource-provider-inventories19:17
mriedemis missing POST19:17
*** tbachman has quit IRC19:18
*** mvk has joined #openstack-nova19:18
*** slaweq has joined #openstack-nova19:18
*** READ10 has quit IRC19:19
*** brault has joined #openstack-nova19:22
*** rmk has quit IRC19:23
*** rmk has joined #openstack-nova19:26
*** rmk has quit IRC19:26
*** rmk has joined #openstack-nova19:26
*** acormier has joined #openstack-nova19:26
*** jamesdenton has joined #openstack-nova19:29
*** jmlowe has quit IRC19:30
*** gyee has quit IRC19:31
*** jwcroppe has quit IRC19:31
*** vvargaszte has joined #openstack-nova19:34
*** vvargaszte has quit IRC19:35
openstackgerritsean mooney proposed openstack/nova master: WIP reorder destination host setup  https://review.openstack.org/51065819:37
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Target context for build notification in conductor  https://review.openstack.org/51132519:41
mriedemeasy fix here: https://review.openstack.org/#/c/511209/19:47
mriedemhot dogs and cracker jacks here!19:47
*** jmlowe has joined #openstack-nova19:48
melwitt+W19:49
mriedemhttps://bugs.launchpad.net/nova/+bug/1722891 in case someone wants it19:50
openstackLaunchpad bug 1722891 in OpenStack Compute (nova) "api-ref: POST /resource_providers/{uuid}/inventories is missing" [Medium,Confirmed] - Assigned to hongbin (hongbin034)19:50
openstackgerritHongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot  https://review.openstack.org/51107419:50
*** lyan has quit IRC19:51
*** gyee has joined #openstack-nova19:51
mriedemgrenade jobs seem totally borked in zuulv319:52
*** lyan has joined #openstack-nova19:52
melwittmriedem: looks like hongbin is already gonna work on the bug you linked, so that's cool19:53
mriedemsomething really weird happens with our request id log handlin19:55
mriedemseems to be not thread safe or something19:55
mriedemi'm tracing a server create request through the n-cpu logs and see the normal stuff i'd expect while creating the instance,19:55
mriedembut then i see the same request id showing up in log messages for periodic tasks that are running19:55
melwitthm, I can't say I've seen that before19:56
melwittwhich makes me think it's a relatively new issue19:57
*** gyee has quit IRC19:57
*** Apoorva_ has joined #openstack-nova19:58
*** harlowja has quit IRC20:00
*** liverpooler has quit IRC20:00
*** Apoorva has quit IRC20:02
*** ociuhandu has quit IRC20:02
*** itlinux has joined #openstack-nova20:03
*** sree has joined #openstack-nova20:07
*** edmondsw_ has joined #openstack-nova20:08
*** edmondsw has quit IRC20:08
*** edmondsw_ is now known as edmondsw20:12
*** sree has quit IRC20:12
*** tbachman has joined #openstack-nova20:19
*** tbachman has quit IRC20:22
*** slaweq has quit IRC20:24
*** armax has quit IRC20:25
*** smatzek_ has quit IRC20:28
*** crushil has quit IRC20:28
*** smatzek has joined #openstack-nova20:30
*** andreas_s has joined #openstack-nova20:30
*** fragatina has joined #openstack-nova20:31
*** slaweq has joined #openstack-nova20:32
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix server status values in GET /servers docs  https://review.openstack.org/51069620:34
openstackgerritMatt Riedemann proposed openstack/nova master: Update "SHUTOFF" description in API guide  https://review.openstack.org/51069720:34
*** andreas_s has quit IRC20:35
*** crushil has joined #openstack-nova20:36
*** priteau has quit IRC20:38
*** jmlowe has quit IRC20:38
*** crushil has quit IRC20:41
mriedemuh, filter(~models.Migration.status.in_ means NOT IN right?20:43
mriedemthe ~20:43
*** smatzek has quit IRC20:43
*** smatzek has joined #openstack-nova20:44
*** smatzek_ has joined #openstack-nova20:44
efriedsdague https://review.openstack.org/#/c/511006/ basically passed - the one failure seems unrelated and I'd rather not choke the already-seemingly-choking gate further.20:47
*** gbarros has quit IRC20:47
*** smatzek has quit IRC20:48
*** smatzek_ has quit IRC20:49
*** pchavva has quit IRC20:50
*** tbachman has joined #openstack-nova20:53
mriedemdansmith: want to double check my thoughts about migration.status = 'done' in here? https://review.openstack.org/#/c/506419/ otherwise i think it's ok20:54
*** acormier has quit IRC20:55
*** thorst_ has quit IRC20:55
*** rodolof has joined #openstack-nova20:55
*** jmlowe has joined #openstack-nova20:57
*** thorst has joined #openstack-nova20:58
*** penick_ has quit IRC20:59
*** penick has joined #openstack-nova21:00
mriedemthis also seems like it could bite us https://github.com/openstack/nova/blob/64635ad4a5f60a79e1ec2d5369a8f84bfeccb7e4/nova/db/sqlalchemy/api.py#L4859-L486221:01
mriedemwhen the RT pulls migration records, it's for all migrations where either the source or dest is our local node21:01
*** catintheroof has quit IRC21:02
*** rodolof has quit IRC21:02
*** esberglu has quit IRC21:03
*** thorst has quit IRC21:03
*** rodolof has joined #openstack-nova21:03
*** edmondsw has quit IRC21:07
*** harlowja has joined #openstack-nova21:07
*** rodolof has quit IRC21:08
*** rodolof has joined #openstack-nova21:08
dansmithmriedem: yep will in a sec21:08
*** jmlowe has quit IRC21:12
*** rodolof has quit IRC21:13
*** rodolof has joined #openstack-nova21:14
*** edmondsw has joined #openstack-nova21:14
*** acormier has joined #openstack-nova21:16
*** crushil has joined #openstack-nova21:17
*** edmondsw has quit IRC21:19
*** acormier has quit IRC21:20
dansmithmriedem: replied21:20
*** slaweq has quit IRC21:20
mriedemok didn't think about not having migration allocations for evacs21:22
mriedembut makes sense to not put the source node allocations on the migration record for an evacuation since the source node should be dead21:22
dansmithand you're never going back there.. no rollback21:22
*** slaweq has joined #openstack-nova21:22
mriedembut...are we doubling today?21:22
mriedemin the scheduler when we evacuate we must be doubling21:23
dansmithif we are it's not for any useful reason I think21:23
*** smatzek has joined #openstack-nova21:23
mriedemi think it's just because we do it generically21:23
dansmithI think we delete the allocation for the old node first anyway right?21:23
* dansmith checks21:23
*** smatzek has quit IRC21:24
*** smatzek has joined #openstack-nova21:24
*** smatzek has quit IRC21:24
*** smatzek has joined #openstack-nova21:25
*** acormier has joined #openstack-nova21:25
dansmithallocate _for_evacuate_dest_host21:25
*** acormier has quit IRC21:25
*** crushil has quit IRC21:25
*** jmlowe has joined #openstack-nova21:25
*** acormier has joined #openstack-nova21:26
mriedemthat's when we're forcing the host during evacuate and bypassing the scheduler21:26
dansmithyeah, we end up calling claim_resources and doubling21:26
dansmithno, that method calls the scheduler21:26
mriedem_allocate_for_evacuate_dest_host?21:26
mriedemit doesn't21:26
mriedemit calls scheduler utils21:27
dansmithhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L782-L78221:27
mriedemb/c we do the same util thing for live migration with a forced host21:27
dansmithwhich calls claim_resources21:27
*** slaweq has quit IRC21:27
dansmithsorry, maybe it does't call scheduler, but it does the doubling, which is what I meant21:27
mriedemright, that's not select_destinations21:27
mriedemyeah21:27
dansmithsure, I was focused on the allocs21:27
mriedemthis is the thing i want to remove with a skip_filters flag to select_destinations21:27
*** rodolof has quit IRC21:27
mriedemso getting back to my original question i guess,21:27
*** rodolof has joined #openstack-nova21:27
dansmithregardless, I don't think this affects the new-world path21:27
mriedemwe never remove the allocation from the source node for an evacuation21:28
*** armax has joined #openstack-nova21:28
mriedemexcept when the source node comes back, if it does21:28
openstackgerritHongbin Lu proposed openstack/nova master: placement: add API reference for create inventory  https://review.openstack.org/51134221:28
dansmithyou think we never do it now?21:28
dansmithor you think we never will with the new-world way?21:28
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L68021:28
dansmithah, maybe your point is,21:28
mriedemit's going to be a weird side thing21:29
dansmithin the new world way we'll still end up calling the doubler?21:29
mriedemit's a move operation where the source allocation isn't on the migration uuid21:29
mriedemunlike live migrate, cold migrate/resize21:29
dansmithcan't we just stop doing the doubling across the board?21:29
dansmithjust delete it before we call claim_resources21:29
*** Apoorva_ has quit IRC21:29
mriedemfor evac?21:29
dansmiththat should work for both old and new paths21:30
dansmithyeah21:30
mriedemwell, unless evac fails on the dest21:30
mriedemand the instance never moved21:30
dansmithwhat does it matter? you're not going back21:30
*** Apoorva has joined #openstack-nova21:30
dansmithwe've created the migration, the source node is going to nuke it when it wakes up, per the rules21:30
mriedemnot necessarily21:30
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L64321:30
*** ociuhandu has joined #openstack-nova21:30
mriedemthe source only cleans up if the move completed, or is in progress21:30
dansmithum21:32
dansmiththe whole point of that robustification was to not do that,21:32
*** penick has quit IRC21:32
dansmithelse we'll race with the operation finishing21:32
dansmithI don't see where we're setting =accepted anymore actually21:32
mriedemconductor21:32
mriedemer api21:32
dansmithI don't see it21:33
dansmithah I see21:33
mriedemyeah so the api creates the migration record in 'accepted' status,21:33
dansmithright, so that happens synchronously with the api call21:33
dansmithright?21:33
*** gbarros has joined #openstack-nova21:33
mriedemif the evac is successful on the dest, the status goes to 'done'21:33
*** rodolof has quit IRC21:33
*** rodolof has joined #openstack-nova21:33
dansmithso if the evac api call returns, you know it's never starting on the source again right?21:33
dansmithwe do that before we create the new allocation,21:34
dansmithso if we have saved that in the db,21:34
dansmithwe know we can delete the old allocation without caring21:34
*** esberglu has joined #openstack-nova21:34
dansmithbecause at that point, the source host will never take it back21:34
mriedemnot necessarily :)21:34
mriedemif conductor/scheduler fails to find a host, the migration status for the evac is marked 'error'21:34
*** tbachman has quit IRC21:34
mriedemand the instance is still on the source21:35
mriedemwith no allocation on the dest21:35
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L87121:35
*** vladikr_ has quit IRC21:35
mriedemthis is all related to that thing gibi and i were talking about before the ptg21:35
*** vladikr has quit IRC21:35
mriedemhandling allocations for failed evacs21:35
dansmithright and I was trying to say we shouldn't ever let it go back21:35
dansmithso we should include error in the list of states we use in compute right?21:35
*** penick has joined #openstack-nova21:36
*** vladikr_ has joined #openstack-nova21:36
dansmithbecause the recovery path should be re-rebuilding that instance21:36
mriedemrebuilding the instance on the source won't recreate the allocations in placement21:36
mriedemb/c rebuild skips the scheduler, which skips the claim21:37
dansmithokay, I guess, yeah21:37
dansmithregardless,21:37
mriedemso if you didn't move the instance, but you delete the allocations on the source when starting up, rebuild won't help you with the allocations21:37
dansmithI really think we should avoid the source coming back up and re-owning the instance because I think it's going to be a likely source of races21:37
*** vladikr has joined #openstack-nova21:38
*** rodolof has quit IRC21:38
*** rodolof has joined #openstack-nova21:38
dansmiththe people that use this for HA stuff hammer on this pretty hard from scripts and I think the behavior needs to be as predictable and linear as possible21:38
mriedemok, well, it seems that requires more thought to determine if we change how evac works if the source comes back up and the instance didn't move, or if we should move source node allocations to a migration record during evac,21:39
mriedemall things i can't really sort out in my head right now with maya in my office reading her homework...21:40
dansmithso, we don't clear the instance.host in evacuate for some reason21:40
*** thorst has joined #openstack-nova21:40
mriedemwe just update it once evac is successful21:40
dansmithif we did, it would make it easier to just let people retry with an evac again if it failed the first time21:41
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L284121:41
dansmithyeah21:41
mriedemyou can already retry an evac if the first one failed21:41
dansmithnot if thecompute host came backup21:41
dansmithbecause we still have instance.host set and require it be down21:41
dansmithright?21:42
dansmiththe source I mean21:42
mriedemwell, if we nulled out instance.host, shit gets all sorts of wonky because you then have to deal with local delete in the api bullshit21:42
mriedemwonkified21:42
mriedemi think i've overshot the original problem we were talking about that started this :)21:43
*** rodolof has quit IRC21:43
*** rodolof has joined #openstack-nova21:43
mriedemthe assertion that RT doesn't have to care about migration allocations during evac21:43
dansmithso, in your case,21:43
dansmithlet's say we evac, fail, leave a doubled allocation today,21:43
dansmiththen evac again, what do we do, triple it?21:43
*** smatzek has quit IRC21:44
mriedemno, we cleanup the allocations for the failed dest host21:44
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L283121:44
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L281721:45
mriedemthose are for if the actual spawn failed, or the claim on dest failed21:45
mriedemremember me freaking out on you and jay about this a couple of weeks ago? :)21:45
mriedemso i think we've got that covered21:45
mriedemthe cleanup i mean21:45
mriedemand if we failed to pick a host or claim in conductor, we don't have anything to cleanup anyway21:45
dansmithI've lost sight of the thing you think is wrong in the new case21:46
mriedemi'm not saying it's wrong,21:46
mriedembut this code in the RT can pull migrations that are 'done'21:46
*** jmlowe has quit IRC21:47
mriedemand that db api query is not making a distinction on if the current node is the migration's source or dest21:47
dansmithwhich would cause it to skip allocations at most, yes?21:47
mriedemmaybe we should just fix the db api filter to include 'done'?21:47
*** eharney has quit IRC21:47
dansmithuntil the source comes up and marks it finished or whatever21:47
mriedemi think so...21:48
mriedemwell, that's why i asked for you to double check my thinking here21:48
*** rodolof has quit IRC21:48
mriedembecause this is weird21:48
dansmithbut it's an instance on our host, so we would skip it for that reason anyway21:48
dansmithmriedem: yeah, how'd that work out for you? :)21:48
mriedemi feel better21:48
*** rodolof has joined #openstack-nova21:49
dansmithwe could just filter out migrations that don't have us as the source or the right state since we have to iterate them anyway,21:49
dansmithbut I'm not sure it's required21:49
dansmithare you concerned about inbound or outbound migrations being included?21:50
dansmithyou said "source and dest" but didn't emphasize either, so I'm not sure which you're concerned about21:50
mriedemyeah i'm unclear on which we care about here too21:53
mriedemi would think source since migration allocations should only be for source21:53
*** gbarros has quit IRC21:53
dansmithsource of an evacuation, coming back up early?21:53
*** jmlowe has joined #openstack-nova21:54
dansmithcurrently it'll still count those instances as its own and not delete their allocations until .host flips21:54
*** Yingxin has quit IRC21:55
*** Yingxin has joined #openstack-nova21:56
mriedemok so in https://review.openstack.org/#/c/506419/19/nova/compute/resource_tracker.py21:57
mriedemif we assume we're on the dest node, because the source is dead during an evac21:58
*** rodolof has quit IRC21:58
*** rodolof has joined #openstack-nova21:58
mriedemand the migration status is done21:58
dansmithmigration is in accepted early,21:58
dansmiththen done once it's on our node21:58
mriedemmigration_get_in_progress_by_host_and_node could return it and the migration.dest_compute would be our local RT21:59
dansmiththen we're not the dest node anymore because instance.host=$us21:59
dansmithi.e. if this runs before evac completes, it's still in accepted state21:59
mriedemcorrect, and the migration wouldn't be returned21:59
mriedemfrom migration_get_in_progress_by_host_and_node21:59
mriedembecause: filter(~models.Migration.status.in_(['accepted', 'confirmed',22:00
*** ijw has quit IRC22:00
*** ijw has joined #openstack-nova22:00
mriedemif the migration status is 'done', then we've already set instance.host=$us22:00
dansmithyeah, so if it runs early, we might nuke the doubled allocation yeah?22:00
mriedemwell i'm looking in _remove_deleted_instances_allocations now22:01
dansmithnot sure how we're not doing that already today though22:01
mriedemif it runs before instance.host is set to the dest node, the instance shouldn't be in tracked_instances22:01
dansmithright22:02
mriedemand we'd get to if instance.host != cn.host:22:02
dansmithbut we have an allocation already because of conductor22:02
mriedemand we'd log22:02
mriedemLOG.debug("Instance %s has been moved to another host %s(%s). "22:02
mriedem                          "There are allocations remaining against the source "22:02
mriedem                          "host that might need to be removed: %s.",22:02
mriedemand those allocatoins remaining on the source would get cleaned up if/when the source comes back up and deletes them22:03
mriedemand if it doesn't come back up, we don't care22:03
*** rodolof has quit IRC22:03
dansmithack yeah22:03
mriedemif this runs after instance.host=$us, then we hit if consumer_uuid in known_instances: and we don't remove allocations22:03
*** jaypipes has quit IRC22:03
*** rodolof has joined #openstack-nova22:04
*** claudiub|2 has quit IRC22:04
mriedemso i think we're ok22:04
mriedemand i think we should probably add 'done' to https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4863 in some other unrelated change, probably just because someone forgot about it before, but 'done' is definitely not in-progres22:05
dansmithmriedem: so, we _are_ passing the functional tests that cover evac.. the only thing that might be missing is source compute coming back or something22:05
mriedemi know we have some functional tests for ^ because they are asserting the source removes the allocations from the source after it's moved22:06
dansmithI'm not really sure why accepted should be in there, tbh22:06
dansmithsince it's very much in progress22:06
mriedemhttps://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L165822:06
mriedemyeah not sure about accepted being in there either22:07
mriedemso ^ is the test that once you have evacuated to the dest and start up the source, the allocations are removed from the source22:07
dansmithah yeah okay22:07
mriedemwhich is actually currently broken :)22:07
mriedemhttps://review.openstack.org/#/c/510176/22:08
mriedemmayhap you'd like to +W that22:08
mriedemour code to remove the source allocations on startup after evac is not working22:08
dansmithso explain to me again how you can evacuate from a source, go to error, and then start back up on the source successfully?22:09
dansmithwe should be caught by the vm_state==ERROR skip clause no?22:09
*** lpetrut_ has quit IRC22:09
SamYapleif its in an errored state and you reset it to state active then do a hard-reboot on it (assuming shared storage) it should come back22:10
*** lyan has quit IRC22:10
*** baoli has quit IRC22:11
dansmithSamYaple: yeah, reset-state means you're cheating and other things are likely going to be out of sync22:11
SamYapledansmith: fair enough. im just saying what would typically get done in ops22:12
SamYapleim not advocating for this22:12
dansmithSamYaple: ack, yeah, I was thinking mriedem was saying there's a path out of the forest for this by a user22:12
*** tbachman has joined #openstack-nova22:12
mriedemdansmith: do we set the instance to error state on failed evac?22:12
dansmithmriedem: if we didn't you couldn't call evac on it again22:13
dansmithand you said we could22:13
mriedemi don't see that we change the vm_state on evac22:13
dansmithso we leave it as rebuilding?22:13
mriedemno22:13
mriedem    @check_instance_state(vm_state=[vm_states.ACTIVE, vm_states.STOPPED,22:13
mriedem                                    vm_states.ERROR])22:13
mriedem    def evacuate(self, context, instance, host, on_shared_storage,22:13
mriedemi think it would be active22:13
mriedemwell, on a successful evac it would be active22:14
*** smatzek has joined #openstack-nova22:14
mriedemif it started as error and evac failed, it'd stay in error22:14
dansmithoh task_state=rebuilding22:14
dansmithso um22:14
mriedemyar22:14
dansmiththat means I can re-evac while an evac is going?22:14
dansmiththat'd be fun22:14
mriedemno22:14
mriedemi think @check_instance_state defaults to not allow doing things if the task_state is set22:15
mriedemand we set the task_state to rebuilding in the api once you've started an evac22:15
dansmithdoes that decorator imply no task state if not specified?22:15
dansmithokay22:15
mriedemyeah it's weird and confusing to read22:15
mriedemtask_state=(None,)22:15
*** rodolof has quit IRC22:16
mriedemif (task_state is not None and22:16
mriedem                    instance.task_state not in task_state):22:16
dansmithokay22:16
mriedemso unrelated,22:16
*** burt has quit IRC22:17
mriedembut now i realize that if your instance was stopped and it was successfully evacuated, it's now running22:17
mriedemwe don't stop it again like we do for resizing/cold migrating a stopped instance22:17
mriedemwhich, as it turns out, was my first major bug fix in nova :)22:17
mriedemremember?!22:18
dansmithoh I remember22:18
dansmithokay so nothing to change on this?22:20
dansmithalso I'd like to note that this migration unfuckification has been the least fun thing to work on since, oh I dunno, the great flavor migration of 201522:22
mriedemyeah i just +Wed22:22
mriedemappears CI just exploded22:23
*** smatzek has quit IRC22:24
*** edmondsw has joined #openstack-nova22:25
dansmithwe _can_ do a migration-holding source allocation if you want,22:25
dansmithbut it's work for no good reason, IMHO,22:26
mriedemyeah, that's why i said if/when we need to22:26
dansmithokay22:26
mriedembecause that would mean the cleanup if source comes back up thing would have to deal with whether or not the allocation is on the migration or not22:26
dansmithright22:27
*** tbachman has quit IRC22:27
mriedemand that would definitely be22:27
mriedemsay it with me now22:27
dansmithI'm a little concerned that we have this reset-state-hard-reboot path to keep these on the source even after an evac has been triggered once22:27
* dansmith doesn't know what he's supposed to say in chorus22:28
mriedemwonky22:28
dansmithheh22:28
mriedemit would be wonky22:28
*** edmondsw has quit IRC22:30
*** felipemonteiro_ has quit IRC22:34
*** felipemonteiro_ has joined #openstack-nova22:34
mriedemwtf, why do we joinedload instance.system_metadata when pulling migration records out of the db? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L486722:35
*** priteau has joined #openstack-nova22:39
dansmithbecause migration.instance.system_metadata22:40
dansmithalthough I'm sure we don't need that anymore22:40
mriedemyou know what else is super confusing,22:41
mriedemwe have a 'finished' migration status,22:41
mriedembut it doesn't mean the migration is actually done, it means the instance is done being resized, but it's in verify_resize state22:41
mriedemso it's not an end state :/22:41
dansmithyup22:42
mriedemoh, an22:43
mriedem*and22:43
mriedemwhen reverting a resize,22:44
mriedemwe set the migration status to 'reverted' on the dest host before casting back to the source host to complete the revert22:44
*** priteau has quit IRC22:44
mriedemso technically the migration is still in progress at that point22:44
mriedemi sure hope nothing like resource tracking depends on us not fucking that all up :)22:44
*** AlexeyAbashkin has joined #openstack-nova22:47
*** tbachman has joined #openstack-nova22:48
*** edmondsw has joined #openstack-nova22:48
mriedemcheck out this fun stuff http://paste.openstack.org/show/623387/22:50
mriedemcreating rp inventory on the fly like a mad dog22:50
*** AlexeyAbashkin has quit IRC22:52
*** jmlowe has quit IRC22:53
*** edmondsw has quit IRC22:54
*** penick has quit IRC22:57
*** priteau has joined #openstack-nova23:01
openstackgerritMatt Riedemann proposed openstack/nova master: Fix migration_get_in_progress_by_host_and_node status list  https://review.openstack.org/51135123:05
*** priteau has quit IRC23:06
openstackgerritEric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax  https://review.openstack.org/51024423:10
*** hongbin has quit IRC23:11
*** felipemonteiro_ has quit IRC23:12
efriedMan Jenkins is a mess.23:12
*** masber has joined #openstack-nova23:14
*** ttx has quit IRC23:20
*** ttx has joined #openstack-nova23:20
*** crushil has joined #openstack-nova23:22
*** crushil has quit IRC23:27
*** yamamoto has joined #openstack-nova23:28
*** suresh12 has quit IRC23:30
*** suresh12 has joined #openstack-nova23:31
*** thorst has quit IRC23:33
*** suresh12_ has joined #openstack-nova23:34
mriedemdansmith: i guess the 'accepted' filter is intentional https://review.openstack.org/#/c/244486/23:35
*** suresh12 has quit IRC23:35
*** suresh12_ has quit IRC23:35
*** suresh12 has joined #openstack-nova23:37
openstackgerritmelanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume  https://review.openstack.org/49898323:38
*** suresh12 has quit IRC23:42
openstackgerritmelanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume  https://review.openstack.org/49898323:43
openstackgerritMatt Riedemann proposed openstack/nova master: Add 'done' to migration_get_in_progress_by_host_and_node filter  https://review.openstack.org/51135123:47
*** mriedem has quit IRC23:48
openstackgerritEd Leafe proposed openstack/nova master: Add alternate hosts  https://review.openstack.org/48621523:52
openstackgerritEd Leafe proposed openstack/nova master: Add Selection objects  https://review.openstack.org/49923923:52
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585423:52
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51015923:52
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135723:52
openstackgerritEd Leafe proposed openstack/nova master: WIP - Make conductor pass and use host_lists  https://review.openstack.org/51135823:52
*** fragatina has quit IRC23:52
*** itlinux has quit IRC23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!