*** Oku_OS-away has quit IRC | 00:00 | |
*** andreas_s has quit IRC | 00:01 | |
*** jgriffith has quit IRC | 00:01 | |
*** felipemonteiro has joined #openstack-nova | 00:01 | |
*** chyka has joined #openstack-nova | 00:01 | |
*** Oku_OS-away has joined #openstack-nova | 00:01 | |
*** John has joined #openstack-nova | 00:02 | |
*** John is now known as Guest63378 | 00:02 | |
*** felipemonteiro_ has joined #openstack-nova | 00:02 | |
*** Guest63378 is now known as jgriffith | 00:03 | |
*** Tom has quit IRC | 00:03 | |
*** felipemonteiro has quit IRC | 00:06 | |
*** chyka has quit IRC | 00:06 | |
*** acormier has joined #openstack-nova | 00:09 | |
*** suresh12 has joined #openstack-nova | 00:10 | |
*** Apoorva_ has quit IRC | 00:11 | |
*** yamahata has quit IRC | 00:18 | |
*** tetsuro has joined #openstack-nova | 00:20 | |
*** Apoorva has joined #openstack-nova | 00:20 | |
*** felipemonteiro_ has quit IRC | 00:21 | |
*** xinliang has quit IRC | 00:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 00:27 |
---|---|---|
*** andreas_s has joined #openstack-nova | 00:28 | |
*** hshiina has joined #openstack-nova | 00:29 | |
*** crushil has quit IRC | 00:31 | |
*** crushil has joined #openstack-nova | 00:31 | |
*** andreas_s has quit IRC | 00:32 | |
*** acormier has quit IRC | 00:33 | |
*** acormier has joined #openstack-nova | 00:34 | |
*** xinliang has joined #openstack-nova | 00:35 | |
*** crushil has quit IRC | 00:36 | |
*** acormier has quit IRC | 00:38 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 00:39 |
*** acormier has joined #openstack-nova | 00:43 | |
*** acormier has quit IRC | 00:44 | |
*** acormier has joined #openstack-nova | 00:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 00:52 |
*** thorst has joined #openstack-nova | 00:53 | |
*** mingyu has joined #openstack-nova | 00:53 | |
*** thorst has quit IRC | 00:54 | |
*** AlexeyAbashkin has joined #openstack-nova | 00:55 | |
*** AlexeyAbashkin has quit IRC | 00:59 | |
*** mingyu has quit IRC | 00:59 | |
*** acormier_ has joined #openstack-nova | 01:03 | |
*** andreas_s has joined #openstack-nova | 01:04 | |
*** Apoorva_ has joined #openstack-nova | 01:05 | |
*** acormier has quit IRC | 01:06 | |
*** gyee has quit IRC | 01:08 | |
*** phuongnh has joined #openstack-nova | 01:08 | |
*** Apoorva has quit IRC | 01:08 | |
*** andreas_s has quit IRC | 01:09 | |
*** Apoorva_ has quit IRC | 01:09 | |
*** liverpooler has quit IRC | 01:10 | |
Kevin_Zheng | mriedem, hi, for https://review.openstack.org/#/c/509326/ | 01:12 |
*** yangyapeng has joined #openstack-nova | 01:14 | |
Kevin_Zheng | I responded some comment, as for the neutron part, I submitted a RFE and seems Amando is fine with it and they will discuss it at the driver meeting this friday: https://bugs.launchpad.net/neutron/+bug/1718605, how should I put it in the spec? | 01:16 |
openstack | Launchpad bug 1718605 in neutron "[RFE] Support regex matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu) | 01:16 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 01:17 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 01:17 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 01:17 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 01:17 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 01:17 |
*** jaypipes has quit IRC | 01:18 | |
*** penick has joined #openstack-nova | 01:22 | |
*** Tom_ has joined #openstack-nova | 01:24 | |
*** Shunli has joined #openstack-nova | 01:25 | |
*** litao__ has joined #openstack-nova | 01:30 | |
*** hemna_ has quit IRC | 01:31 | |
*** jwcroppe has joined #openstack-nova | 01:32 | |
*** hongbin has joined #openstack-nova | 01:35 | |
*** andreas_s has joined #openstack-nova | 01:37 | |
*** acormier_ has quit IRC | 01:40 | |
*** andreas_s has quit IRC | 01:41 | |
*** suresh12 has quit IRC | 01:46 | |
*** psachin has joined #openstack-nova | 01:51 | |
*** Tom_ has quit IRC | 01:53 | |
*** jwcroppe has quit IRC | 01:54 | |
*** READ10 has quit IRC | 01:58 | |
*** takashin has quit IRC | 02:01 | |
*** owalsh_ has joined #openstack-nova | 02:08 | |
*** penick has quit IRC | 02:11 | |
*** owalsh has quit IRC | 02:11 | |
*** thorst has joined #openstack-nova | 02:17 | |
*** thorst has quit IRC | 02:18 | |
*** catintheroof has joined #openstack-nova | 02:20 | |
*** dave-mccowan has joined #openstack-nova | 02:20 | |
*** suresh12 has joined #openstack-nova | 02:21 | |
*** jlvillal has quit IRC | 02:21 | |
*** jlvillal has joined #openstack-nova | 02:21 | |
*** dave-mcc_ has quit IRC | 02:22 | |
*** catintheroof has quit IRC | 02:22 | |
*** liusheng has joined #openstack-nova | 02:25 | |
*** suresh12 has quit IRC | 02:26 | |
openstackgerrit | Merged openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | 02:32 |
*** rgerganov has quit IRC | 02:33 | |
*** thorst has joined #openstack-nova | 02:33 | |
*** rgerganov has joined #openstack-nova | 02:33 | |
*** ijw has quit IRC | 02:34 | |
*** takashin has joined #openstack-nova | 02:34 | |
*** thorst has quit IRC | 02:38 | |
*** rmk has quit IRC | 02:42 | |
*** rmk has joined #openstack-nova | 02:45 | |
*** rmk has quit IRC | 02:45 | |
*** rmk has joined #openstack-nova | 02:45 | |
*** dave-mccowan has quit IRC | 02:51 | |
*** erlon has quit IRC | 02:54 | |
*** markvoelker has quit IRC | 02:54 | |
*** chyka has joined #openstack-nova | 02:55 | |
*** mingyu has joined #openstack-nova | 02:56 | |
*** chyka has quit IRC | 02:56 | |
*** mingyu has quit IRC | 02:56 | |
*** links has joined #openstack-nova | 02:57 | |
*** mingyu has joined #openstack-nova | 02:59 | |
*** mingyu has quit IRC | 02:59 | |
*** sree has joined #openstack-nova | 03:01 | |
*** mingyu has joined #openstack-nova | 03:04 | |
*** nicolasbock has quit IRC | 03:06 | |
*** sree has quit IRC | 03:06 | |
*** mingyu has quit IRC | 03:10 | |
*** mingyu has joined #openstack-nova | 03:13 | |
*** mingyu has quit IRC | 03:14 | |
*** mingyu has joined #openstack-nova | 03:14 | |
*** yamahata has joined #openstack-nova | 03:17 | |
*** jmlowe has joined #openstack-nova | 03:24 | |
*** yamamoto has quit IRC | 03:25 | |
*** chyka has joined #openstack-nova | 03:27 | |
*** chyka has quit IRC | 03:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021 | 03:39 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 03:41 |
*** hshiina has quit IRC | 03:42 | |
*** vivsoni has quit IRC | 03:51 | |
*** suresh12 has joined #openstack-nova | 03:54 | |
*** jwcroppe has joined #openstack-nova | 03:57 | |
*** hshiina has joined #openstack-nova | 04:01 | |
*** jichen has joined #openstack-nova | 04:02 | |
*** ratailor has joined #openstack-nova | 04:05 | |
*** mdnadeem has joined #openstack-nova | 04:06 | |
*** ratailor_ has joined #openstack-nova | 04:07 | |
*** mingyu has quit IRC | 04:09 | |
*** ratailor has quit IRC | 04:11 | |
*** hongbin has quit IRC | 04:12 | |
*** yamamoto has joined #openstack-nova | 04:13 | |
*** mingyu has joined #openstack-nova | 04:15 | |
*** diga has joined #openstack-nova | 04:17 | |
*** ratailor__ has joined #openstack-nova | 04:21 | |
*** ratailor_ has quit IRC | 04:22 | |
*** khamtamtun has joined #openstack-nova | 04:23 | |
*** khamtamtun has quit IRC | 04:25 | |
*** claudiub|2 has joined #openstack-nova | 04:25 | |
*** gbarros has joined #openstack-nova | 04:28 | |
*** vks1 has joined #openstack-nova | 04:31 | |
*** chyka has joined #openstack-nova | 04:32 | |
*** ratailor_ has joined #openstack-nova | 04:32 | |
*** mingyu has quit IRC | 04:32 | |
*** mingyu has joined #openstack-nova | 04:32 | |
*** jmlowe has quit IRC | 04:33 | |
*** gyee has joined #openstack-nova | 04:34 | |
*** zen369 has joined #openstack-nova | 04:35 | |
*** ratailor__ has quit IRC | 04:36 | |
*** crushil has joined #openstack-nova | 04:37 | |
*** gbarros has quit IRC | 04:37 | |
*** vivsoni has joined #openstack-nova | 04:38 | |
*** thorst has joined #openstack-nova | 04:39 | |
*** chyka has quit IRC | 04:40 | |
*** yamamoto has quit IRC | 04:43 | |
*** thorst has quit IRC | 04:44 | |
*** Tom has joined #openstack-nova | 04:46 | |
*** Tom has quit IRC | 04:47 | |
*** liusheng has quit IRC | 04:51 | |
*** markvoelker has joined #openstack-nova | 04:55 | |
*** armax has quit IRC | 05:01 | |
*** cristicalin has joined #openstack-nova | 05:01 | |
*** cristicalin has quit IRC | 05:04 | |
*** sridharg has joined #openstack-nova | 05:07 | |
*** cristicalin has joined #openstack-nova | 05:07 | |
*** mdnadeem_ has joined #openstack-nova | 05:08 | |
*** cristicalin has quit IRC | 05:09 | |
*** gyee has quit IRC | 05:10 | |
*** mdnadeem has quit IRC | 05:10 | |
*** sree_ has joined #openstack-nova | 05:11 | |
*** mingyu has quit IRC | 05:11 | |
*** sree_ is now known as Guest72138 | 05:11 | |
*** edand has joined #openstack-nova | 05:12 | |
*** mingyu has joined #openstack-nova | 05:12 | |
*** crushil has quit IRC | 05:12 | |
*** gouthamr has quit IRC | 05:12 | |
*** Guest2849 is now known as sdake | 05:20 | |
*** sdake is now known as Guest95430 | 05:21 | |
*** Guest95430 is now known as sdake_fixing | 05:23 | |
*** ratailor__ has joined #openstack-nova | 05:23 | |
*** mingyu has quit IRC | 05:25 | |
*** ratailor_ has quit IRC | 05:26 | |
*** suresh12 has quit IRC | 05:29 | |
*** sdake_fixing has quit IRC | 05:29 | |
*** sdake_fixing has joined #openstack-nova | 05:29 | |
*** markvoelker has quit IRC | 05:30 | |
*** lajoskatona has joined #openstack-nova | 05:30 | |
*** mingyu has joined #openstack-nova | 05:32 | |
*** Tom has joined #openstack-nova | 05:33 | |
*** diga has quit IRC | 05:35 | |
*** sdake_fixing has quit IRC | 05:35 | |
*** mingyu has quit IRC | 05:36 | |
*** Tom has quit IRC | 05:37 | |
*** udesale has joined #openstack-nova | 05:38 | |
*** yamamoto has joined #openstack-nova | 05:43 | |
*** mdnadeem_ has quit IRC | 05:44 | |
*** lpetrut has joined #openstack-nova | 05:49 | |
*** yamamoto has quit IRC | 05:49 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/511021 | 05:50 |
*** Tom has joined #openstack-nova | 05:50 | |
*** lpetrut has quit IRC | 05:50 | |
*** lpetrut has joined #openstack-nova | 05:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 05:52 |
*** yamamoto has joined #openstack-nova | 05:54 | |
*** Tom has quit IRC | 05:54 | |
*** josecastroleon has quit IRC | 05:57 | |
*** yamamoto has quit IRC | 05:58 | |
*** jwcroppe has quit IRC | 05:59 | |
*** mdnadeem_ has joined #openstack-nova | 06:00 | |
*** yamamoto has joined #openstack-nova | 06:02 | |
*** yamamoto has quit IRC | 06:04 | |
*** abhishek has joined #openstack-nova | 06:05 | |
*** Tom has joined #openstack-nova | 06:06 | |
*** abhi89 has quit IRC | 06:07 | |
*** trinaths has joined #openstack-nova | 06:08 | |
*** Oku_OS-away is now known as Oku_OS | 06:10 | |
*** Tom has quit IRC | 06:11 | |
*** andreas_s has joined #openstack-nova | 06:11 | |
*** diga has joined #openstack-nova | 06:11 | |
*** liusheng has joined #openstack-nova | 06:14 | |
*** fragatina has joined #openstack-nova | 06:16 | |
*** hoonetorg has quit IRC | 06:17 | |
*** belmoreira has joined #openstack-nova | 06:18 | |
*** yamamoto has joined #openstack-nova | 06:21 | |
*** markvoelker has joined #openstack-nova | 06:26 | |
*** avolkov has joined #openstack-nova | 06:27 | |
*** spectr has quit IRC | 06:27 | |
*** spectr has joined #openstack-nova | 06:28 | |
*** zerda3 has joined #openstack-nova | 06:28 | |
*** hoonetorg has joined #openstack-nova | 06:31 | |
*** zsli_ has joined #openstack-nova | 06:32 | |
*** ratailor_ has joined #openstack-nova | 06:32 | |
*** mingyu has joined #openstack-nova | 06:33 | |
*** Shunli has quit IRC | 06:35 | |
*** ratailor__ has quit IRC | 06:35 | |
*** mingyu has quit IRC | 06:38 | |
*** thorst has joined #openstack-nova | 06:40 | |
*** sshwarts has joined #openstack-nova | 06:40 | |
*** zsli__ has joined #openstack-nova | 06:41 | |
*** sahid has joined #openstack-nova | 06:42 | |
*** markus_z has joined #openstack-nova | 06:43 | |
*** zsli_ has quit IRC | 06:44 | |
*** thorst has quit IRC | 06:44 | |
*** suresh12 has joined #openstack-nova | 06:45 | |
*** TuanLA has joined #openstack-nova | 06:46 | |
*** abhi89 has joined #openstack-nova | 06:49 | |
*** owalsh_ is now known as owalsh | 06:49 | |
*** lpetrut has quit IRC | 06:49 | |
*** suresh12 has quit IRC | 06:50 | |
*** abhishek has quit IRC | 06:51 | |
openstackgerrit | Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506 | 06:53 |
*** vvargaszte has joined #openstack-nova | 06:55 | |
*** zerda3 has quit IRC | 06:59 | |
*** markvoelker has quit IRC | 07:00 | |
*** sdake has joined #openstack-nova | 07:01 | |
*** sdake has quit IRC | 07:01 | |
*** sdake has joined #openstack-nova | 07:01 | |
*** vvargaszte has quit IRC | 07:03 | |
*** openstackgerrit has quit IRC | 07:03 | |
*** abhi89 has quit IRC | 07:03 | |
*** abhi89 has joined #openstack-nova | 07:03 | |
*** rcernin has joined #openstack-nova | 07:05 | |
*** jpena|off is now known as jpena | 07:08 | |
*** edand has quit IRC | 07:11 | |
*** tojuvone has quit IRC | 07:12 | |
*** pcaruana has joined #openstack-nova | 07:13 | |
*** lpetrut has joined #openstack-nova | 07:16 | |
*** ralonsoh has joined #openstack-nova | 07:18 | |
*** trinaths has quit IRC | 07:18 | |
*** tesseract has joined #openstack-nova | 07:19 | |
*** ratailor__ has joined #openstack-nova | 07:21 | |
*** openstackgerrit has joined #openstack-nova | 07:22 | |
openstackgerrit | Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506 | 07:22 |
*** tojuvone has joined #openstack-nova | 07:23 | |
*** ratailor_ has quit IRC | 07:24 | |
*** edand has joined #openstack-nova | 07:27 | |
*** trinaths has joined #openstack-nova | 07:27 | |
*** yangyapeng has quit IRC | 07:29 | |
*** yangyapeng has joined #openstack-nova | 07:29 | |
*** tinwood has quit IRC | 07:30 | |
*** tinwood has joined #openstack-nova | 07:30 | |
openstackgerrit | Merged openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | 07:30 |
*** nikhil has quit IRC | 07:34 | |
*** yamamoto has quit IRC | 07:35 | |
*** yamamoto has joined #openstack-nova | 07:36 | |
*** yamamoto has quit IRC | 07:36 | |
*** ratailor__ has quit IRC | 07:36 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:37 | |
*** vvargaszte has joined #openstack-nova | 07:38 | |
*** thorst has joined #openstack-nova | 07:41 | |
*** vvargaszte has quit IRC | 07:42 | |
*** thorst has quit IRC | 07:45 | |
*** openstackgerrit has quit IRC | 07:48 | |
*** lpetrut has quit IRC | 07:49 | |
*** itlinux has joined #openstack-nova | 07:52 | |
*** tojuvone has quit IRC | 07:55 | |
*** markvoelker has joined #openstack-nova | 07:57 | |
*** hferenc has quit IRC | 08:04 | |
*** itlinux has quit IRC | 08:04 | |
*** tojuvone has joined #openstack-nova | 08:07 | |
*** alexchadin has joined #openstack-nova | 08:12 | |
*** yamahata has quit IRC | 08:19 | |
*** alexchadin has quit IRC | 08:22 | |
*** alexchadin has joined #openstack-nova | 08:22 | |
*** yamamoto has joined #openstack-nova | 08:23 | |
*** claudiub|2 has quit IRC | 08:26 | |
*** pcaruana has quit IRC | 08:27 | |
*** openstackgerrit has joined #openstack-nova | 08:28 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 08:28 |
*** markvoelker has quit IRC | 08:30 | |
*** pcaruana has joined #openstack-nova | 08:31 | |
*** openstackgerrit has quit IRC | 08:33 | |
*** openstackgerrit has joined #openstack-nova | 08:33 | |
openstackgerrit | Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180 | 08:33 |
*** suresh12 has joined #openstack-nova | 08:34 | |
openstackgerrit | Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180 | 08:36 |
*** suresh12 has quit IRC | 08:39 | |
*** spectr has quit IRC | 08:40 | |
*** edand_ has joined #openstack-nova | 08:41 | |
openstackgerrit | Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180 | 08:41 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus https://review.openstack.org/510897 | 08:41 |
*** edand has quit IRC | 08:43 | |
*** danpawlik has quit IRC | 08:44 | |
openstackgerrit | Huan Xie proposed openstack/nova master: [WIP] VGPU: Define vgpu related resources https://review.openstack.org/511180 | 08:45 |
kashyap | mdbooth: Hi there, when you get a moment: | 08:49 |
mdbooth | kashyap: Wassup? | 08:50 |
kashyap | mdbooth: Do you recall top off your head, if Nova sets any QEMU migration parameters? Or do we just rely on libvirt? Let me give an example of what I mean | 08:50 |
*** spectr has joined #openstack-nova | 08:50 | |
kashyap | Take a look at the ouput of this command: virsh qemu-monitor-command f26-devstack --pretty '{"execute":"query-migrate-parameters"}' | 08:51 |
kashyap | http://paste.openstack.org/show/623292/ | 08:51 |
stephenfin | kashyap: to the best of my knowledge, if it can't be done via libvirt then we don't do it | 08:52 |
* kashyap thinks we just let libvirt do the querying of migration parameters | 08:52 | |
*** derekh has joined #openstack-nova | 08:52 | |
mdbooth | kashyap: nova never talks to qemu directly if that's what you mean? | 08:52 |
kashyap | stephenfin: Yep. I know we don't "do direct QEMU", that's the whole frelling point of explicitly modelling in libvirt XML aspects | 08:53 |
mdbooth | In fact, I don't think qemu allows more than one monitor, does it? | 08:53 |
mdbooth | i.e. it's not even possible | 08:53 |
kashyap | mdbooth: I know Nova never talks to QEMU | 08:54 |
kashyap | mdbooth: And QEMU does allow more than one monitor. | 08:54 |
mdbooth | kashyap: I didn't understand the question, then | 08:54 |
kashyap | mdbooth: Let me give a bit more context: | 08:54 |
mdbooth | Nova does set some of those values explicitly, via libvirt | 08:55 |
kashyap | mdbooth: What made me ask the question is, one of the QEMU migration devs asked me this morning what migration parameters Nova uses | 08:55 |
*** lucas-afk is now known as lucasagomes | 08:55 | |
kashyap | mdbooth: You (libvirt could) set migration parameters via `migrate-set-parameters` QMP command, _except_ this "xbzrle-cache-size" | 08:56 |
kashyap | And they were wondering does Nova use it in anyway. So I went digging... | 08:56 |
mdbooth | Yes, absolutely. Nova tweaks a bunch of those. | 08:57 |
*** yamamoto has quit IRC | 08:58 | |
*** alexchadin has quit IRC | 08:58 | |
*** belmoreira has quit IRC | 08:58 | |
*** alexchadin has joined #openstack-nova | 08:59 | |
* kashyap looks | 09:00 | |
kashyap | mdbooth: Meanwhile, here's an example for multiple monitors: | 09:00 |
kashyap | $ qemu-system-x86_64 [...] -monitor stdio -qmp unix:./qmp-sock,server,nowait | 09:00 |
kashyap | HMP (Human Monitor Protocol) on 'stdio', and QMP on a UNIX socket. | 09:01 |
*** yamamoto has joined #openstack-nova | 09:01 | |
*** tetsuro has quit IRC | 09:06 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: [WIP] CLI for resource classes https://review.openstack.org/511182 | 09:06 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: [WIP] RP list: member_of and resources parameters https://review.openstack.org/511183 | 09:06 |
*** yamamoto has quit IRC | 09:06 | |
openstackgerrit | Alex Xu proposed openstack/nova master: [DNM] Add _get_provider_ids_with_any_trait https://review.openstack.org/511184 | 09:06 |
openstackgerrit | Alex Xu proposed openstack/nova master: [DNM] placement: add traits support in the AllocationCandidates object https://review.openstack.org/511185 | 09:06 |
*** danpawlik has joined #openstack-nova | 09:06 | |
*** belmoreira has joined #openstack-nova | 09:08 | |
*** belmoreira has quit IRC | 09:08 | |
*** belmoreira has joined #openstack-nova | 09:10 | |
*** lpetrut has joined #openstack-nova | 09:10 | |
*** sambetts|afk is now known as sambetts | 09:11 | |
*** dtantsur|afk is now known as dtantsur | 09:11 | |
openstackgerrit | sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188 | 09:12 |
*** ratailor has joined #openstack-nova | 09:14 | |
*** dr_gogeta86 has joined #openstack-nova | 09:16 | |
*** dr_gogeta86 has quit IRC | 09:16 | |
*** dr_gogeta86 has joined #openstack-nova | 09:16 | |
*** priteau has joined #openstack-nova | 09:20 | |
*** lpetrut_ has joined #openstack-nova | 09:25 | |
*** markvoelker has joined #openstack-nova | 09:28 | |
*** zsli__ has quit IRC | 09:28 | |
*** lpetrut has quit IRC | 09:28 | |
*** ralonsoh_ has joined #openstack-nova | 09:32 | |
*** tiendc has joined #openstack-nova | 09:33 | |
*** ralonsoh has quit IRC | 09:36 | |
*** jaosorior has quit IRC | 09:36 | |
*** fragatina has quit IRC | 09:37 | |
*** mdnadeem has joined #openstack-nova | 09:38 | |
*** mdnadeem_ has quit IRC | 09:39 | |
*** rmart04 has joined #openstack-nova | 09:40 | |
*** mvk has quit IRC | 09:40 | |
*** thorst has joined #openstack-nova | 09:42 | |
openstackgerrit | jichenjc proposed openstack/nova master: Remove duplicate error info https://review.openstack.org/510719 | 09:42 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 09:44 |
*** jaosorior has joined #openstack-nova | 09:47 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 09:48 |
*** thorst has quit IRC | 09:48 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 09:48 |
*** tbachman has quit IRC | 09:49 | |
*** tbachman has joined #openstack-nova | 09:50 | |
*** avolkov has quit IRC | 09:53 | |
*** belmoreira has quit IRC | 09:53 | |
*** avolkov has joined #openstack-nova | 09:55 | |
*** gszasz has joined #openstack-nova | 09:55 | |
*** udesale has quit IRC | 09:58 | |
*** alexchadin has quit IRC | 09:58 | |
*** yangyapeng has quit IRC | 09:58 | |
*** alexchadin has joined #openstack-nova | 09:58 | |
*** yangyapeng has joined #openstack-nova | 09:58 | |
openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration https://review.openstack.org/500677 | 09:59 |
*** yangyapeng has quit IRC | 09:59 | |
openstackgerrit | jichenjc proposed openstack/nova master: Remove duplicate error info https://review.openstack.org/510719 | 09:59 |
*** TuanLA has quit IRC | 10:00 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 10:00 |
*** markvoelker has quit IRC | 10:01 | |
*** yangyapeng has joined #openstack-nova | 10:01 | |
*** jichen has quit IRC | 10:03 | |
*** sdague has joined #openstack-nova | 10:03 | |
*** trinaths has quit IRC | 10:06 | |
lyarwood | mdbooth: so the stable rescue spec LGTM. I've been umming and ahhing about the sync vs async failure paths for a while, the only alternative I can see is making the rescue_instance rpc cast a call but that's a horrid change. | 10:06 |
mdbooth | lyarwood: 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.html | 10:08 | |
*** andreas_s has quit IRC | 10:09 | |
*** andreas_s has joined #openstack-nova | 10:10 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 10:10 |
*** mdnadeem has quit IRC | 10:12 | |
openstackgerrit | Andrey Volkov proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/509206 | 10:12 |
*** andreas_s has quit IRC | 10:14 | |
*** ratailor has quit IRC | 10:21 | |
*** ratailor has joined #openstack-nova | 10:22 | |
*** suresh12 has joined #openstack-nova | 10:22 | |
*** mdnadeem has joined #openstack-nova | 10:26 | |
*** vks1 has quit IRC | 10:26 | |
*** suresh12 has quit IRC | 10:26 | |
*** yangyapeng has quit IRC | 10:29 | |
*** inara has quit IRC | 10:29 | |
*** alexchadin has quit IRC | 10:31 | |
*** inara has joined #openstack-nova | 10:31 | |
*** Qiming has quit IRC | 10:32 | |
*** ratailor_ has joined #openstack-nova | 10:41 | |
*** smatzek has joined #openstack-nova | 10:43 | |
*** ratailor has quit IRC | 10:45 | |
*** tiendc has quit IRC | 10:50 | |
*** Qiming has joined #openstack-nova | 10:54 | |
*** edand_ has quit IRC | 10:55 | |
*** erlon has joined #openstack-nova | 10:57 | |
*** markvoelker has joined #openstack-nova | 10:58 | |
*** Guest72138 has quit IRC | 10:59 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 10:59 |
*** phuongnh has quit IRC | 11:00 | |
*** AlexeyAbashkin has quit IRC | 11:00 | |
*** AlexeyAbashkin has joined #openstack-nova | 11:01 | |
*** cali_boxer has quit IRC | 11:01 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839 | 11:01 |
*** cali_boxer has joined #openstack-nova | 11:02 | |
*** nicolasbock has joined #openstack-nova | 11:03 | |
*** sergek_ has quit IRC | 11:03 | |
*** brault has quit IRC | 11:04 | |
*** masuberu has joined #openstack-nova | 11:05 | |
*** itlinux has joined #openstack-nova | 11:05 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot https://review.openstack.org/511209 | 11:05 |
*** sergek_ has joined #openstack-nova | 11:06 | |
*** masber has quit IRC | 11:07 | |
*** andreas_s has joined #openstack-nova | 11:09 | |
*** vks1 has joined #openstack-nova | 11:10 | |
*** mvk has joined #openstack-nova | 11:12 | |
*** belmoreira has joined #openstack-nova | 11:16 | |
*** chyka has joined #openstack-nova | 11:16 | |
avolkov | dansmith sdague bauzas cdent: hi, do you remember a case with updating AZ? I posted an alternative solution https://review.openstack.org/#/c/509206/ which restricts api actions. Basically, you can't rename AZ, add or delete host from | 11:18 |
avolkov | aggregate with AZ if host has VMs. Is it something we can move further with? | 11:18 |
openstackgerrit | Merged openstack/nova master: api-ref: note the microversions for GET /resource_providers query params https://review.openstack.org/502168 | 11:19 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001 | 11:20 |
*** hoonetorg has quit IRC | 11:21 | |
*** chyka has quit IRC | 11:21 | |
*** udesale has joined #openstack-nova | 11:21 | |
*** acormier has joined #openstack-nova | 11:22 | |
*** abhishek has joined #openstack-nova | 11:22 | |
*** lucasagomes is now known as lucas-hungry | 11:23 | |
*** larainema has quit IRC | 11:25 | |
*** abhi89 has quit IRC | 11:26 | |
*** markvoelker has quit IRC | 11:30 | |
*** efried has joined #openstack-nova | 11:32 | |
*** acormier has quit IRC | 11:33 | |
*** avtar_ has joined #openstack-nova | 11:33 | |
*** hoonetorg has joined #openstack-nova | 11:34 | |
*** alexchadin has joined #openstack-nova | 11:35 | |
*** yangyapeng has joined #openstack-nova | 11:35 | |
openstackgerrit | Matthew Booth proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices https://review.openstack.org/510106 | 11:36 |
mdbooth | lyarwood: Thanks! | 11:36 |
mdbooth | lyarwood: Do you recall who approved it previously, btw? | 11:36 |
*** takashin has left #openstack-nova | 11:37 | |
mdbooth | lyarwood: I updated for the work already done, btw, and fixed a couple of typos as it was changing anyway. If you have a sec, could you put your +1 back? | 11:38 |
*** ratailor__ has joined #openstack-nova | 11:38 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 11:39 |
*** rmart04 has quit IRC | 11:41 | |
*** ratailor_ has quit IRC | 11:41 | |
*** jpena is now known as jpena|lunch | 11:42 | |
lyarwood | mdbooth: Matt AFAIK | 11:42 |
mdbooth | lyarwood: Thanks. Makes sense as he asked for the microversion. | 11:43 |
*** acormier has joined #openstack-nova | 11:43 | |
*** spectr has quit IRC | 11:44 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | 11:45 |
*** acormier has quit IRC | 11:45 | |
*** spectr has joined #openstack-nova | 11:46 | |
*** ratailor__ has quit IRC | 11:49 | |
*** spectr-RH has joined #openstack-nova | 11:54 | |
*** claudiub|2 has joined #openstack-nova | 11:55 | |
*** litao__ has quit IRC | 11:55 | |
*** yamamoto has joined #openstack-nova | 11:55 | |
*** eharney has joined #openstack-nova | 11:57 | |
*** spectr has quit IRC | 11:57 | |
*** smatzek has quit IRC | 11:58 | |
*** yamamoto has quit IRC | 11:59 | |
*** larainema has joined #openstack-nova | 12:00 | |
*** dave-mccowan has joined #openstack-nova | 12:00 | |
*** lucas-hungry is now known as lucasagomes | 12:01 | |
*** josecastroleon has joined #openstack-nova | 12:02 | |
*** andreas_s has quit IRC | 12:02 | |
*** andreas_s has joined #openstack-nova | 12:02 | |
*** edmondsw has joined #openstack-nova | 12:04 | |
*** thorst has joined #openstack-nova | 12:06 | |
*** yamamoto has joined #openstack-nova | 12:08 | |
*** Yikun has joined #openstack-nova | 12:10 | |
*** liusheng has quit IRC | 12:10 | |
*** yikun_jiang has quit IRC | 12:10 | |
*** suresh12 has joined #openstack-nova | 12:10 | |
*** itlinux has quit IRC | 12:10 | |
*** liusheng has joined #openstack-nova | 12:10 | |
*** andreas_s has quit IRC | 12:11 | |
*** spectr-RH has quit IRC | 12:13 | |
*** spectr has joined #openstack-nova | 12:13 | |
*** itlinux has joined #openstack-nova | 12:14 | |
*** spectr-RH has joined #openstack-nova | 12:14 | |
*** suresh12 has quit IRC | 12:15 | |
*** liusheng has quit IRC | 12:15 | |
*** smatzek has joined #openstack-nova | 12:16 | |
*** andreas_s has joined #openstack-nova | 12:16 | |
*** spectr has quit IRC | 12:18 | |
*** openstackgerrit has quit IRC | 12:18 | |
*** markvoelker has joined #openstack-nova | 12:18 | |
*** diga has quit IRC | 12:19 | |
*** andreas_s has quit IRC | 12:21 | |
*** andreas_s has joined #openstack-nova | 12:21 | |
*** andreas_s has quit IRC | 12:25 | |
*** andreas_s has joined #openstack-nova | 12:26 | |
mdbooth | mriedem: This is a spec which will hopefully be familiar to you: https://review.openstack.org/#/c/510106/ | 12:29 |
*** vks1 has quit IRC | 12:29 | |
*** andreas_s has quit IRC | 12:30 | |
*** priteau has quit IRC | 12:30 | |
*** andreas_s has joined #openstack-nova | 12:31 | |
*** yamamoto has quit IRC | 12:31 | |
*** openstackgerrit has joined #openstack-nova | 12:32 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Move is_block_device_mapping to new_format_is_volume https://review.openstack.org/511217 | 12:32 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Remove driver.block_device.get_swap https://review.openstack.org/511218 | 12:32 |
*** liverpooler has joined #openstack-nova | 12:33 | |
*** sshwarts has quit IRC | 12:34 | |
*** itlinux has quit IRC | 12:35 | |
*** jpena|lunch is now known as jpena | 12:35 | |
*** itlinux has joined #openstack-nova | 12:35 | |
*** pchavva has joined #openstack-nova | 12:35 | |
openstackgerrit | Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506 | 12:38 |
*** thorst_ has joined #openstack-nova | 12:38 | |
*** esberglu has quit IRC | 12:38 | |
*** thorst has quit IRC | 12:39 | |
*** andreas_s has quit IRC | 12:40 | |
*** belmoreira has quit IRC | 12:41 | |
*** catintheroof has joined #openstack-nova | 12:43 | |
bauzas | avolkov: ta. FWIW, I wrote a very old spec for that but looks like the consensus is that it's not needing a microversion | 12:44 |
*** rodolof has joined #openstack-nova | 12:44 | |
stephenfin | sahid: Want to take a look at https://review.openstack.org/#/c/361140/ ? | 12:45 |
*** andreas_s has joined #openstack-nova | 12:45 | |
*** alexchadin has quit IRC | 12:46 | |
*** lyan has joined #openstack-nova | 12:46 | |
* alex_xu nova api meeting is in 15 mins at #openstack-meeting-4 | 12:50 | |
alex_xu | s/15 mins/10 mins/... | 12:50 |
sahid | stephenfin: yes i will, ok | 12:50 |
*** sridharg has quit IRC | 12:51 | |
*** fragatina has joined #openstack-nova | 12:51 | |
*** sapcc-bot has quit IRC | 12:52 | |
*** sapcc-bot has joined #openstack-nova | 12:52 | |
*** mkoderer_ has joined #openstack-nova | 12:52 | |
*** dgonzalez_ has joined #openstack-nova | 12:52 | |
*** tpatzig_ has joined #openstack-nova | 12:52 | |
*** carthaca_ has joined #openstack-nova | 12:52 | |
*** seife_ has joined #openstack-nova | 12:52 | |
*** david_1 has joined #openstack-nova | 12:52 | |
stephenfin | lyarwood, mdbooth: Fancy taking a look at this? Looks sane to me, at least https://review.openstack.org/#/c/290793/ | 12:53 |
*** smatzek has quit IRC | 12:53 | |
*** seife_ has quit IRC | 12:54 | |
*** dgonzalez_ has quit IRC | 12:54 | |
*** mkoderer_ has quit IRC | 12:54 | |
*** carthaca_ has quit IRC | 12:54 | |
*** tpatzig_ has quit IRC | 12:54 | |
*** david_1 has quit IRC | 12:54 | |
*** takashin has joined #openstack-nova | 12:54 | |
*** andreas_s has quit IRC | 12:55 | |
mdbooth | stephenfin: Looking | 12:55 |
*** alexchadin has joined #openstack-nova | 12:55 | |
*** esberglu has joined #openstack-nova | 12:57 | |
stephenfin | ralonsoh_: About? Got a question on https://review.openstack.org/#/c/484726/ | 12:58 |
*** andreas_s has joined #openstack-nova | 12:59 | |
*** penick has joined #openstack-nova | 13:00 | |
*** vladikr has joined #openstack-nova | 13:01 | |
*** vladikr_ has joined #openstack-nova | 13:01 | |
*** vladikr_ has quit IRC | 13:02 | |
*** vladikr_ has joined #openstack-nova | 13:02 | |
*** openstackgerrit has quit IRC | 13:03 | |
*** takashin has quit IRC | 13:04 | |
-openstackstatus- NOTICE: Due to unrelated emergencies, the Zuul v3 rollout has not started yet; stay tuned for further updates | 13:06 | |
*** takashin has joined #openstack-nova | 13: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 needed | 13: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 them | 13:07 |
ralonsoh_ | I'm on it] | 13:07 |
stephenfin | ralonsoh_: Cool cool :) Maybe stick it in WIP until then? | 13:08 |
ralonsoh_ | sure! I'll do this now | 13:08 |
efried | gibi I posted another response on https://review.openstack.org/#/c/510244/ -- please let me know if you disagree or would like to discuss further. Thanks! | 13:09 |
*** openstackgerrit has joined #openstack-nova | 13:09 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Open vSwitch patch port https://review.openstack.org/485228 | 13:09 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add Virtual Ethernet device pair https://review.openstack.org/484726 | 13:09 |
*** takedakn has joined #openstack-nova | 13:10 | |
*** sdake has quit IRC | 13:11 | |
*** sdake has joined #openstack-nova | 13:11 | |
*** sdake has quit IRC | 13:11 | |
*** sdake has joined #openstack-nova | 13:11 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 13:12 |
*** mriedem has joined #openstack-nova | 13:13 | |
*** sdake has quit IRC | 13:15 | |
*** acormier has joined #openstack-nova | 13:15 | |
*** acormier has quit IRC | 13:18 | |
*** acormier has joined #openstack-nova | 13:18 | |
*** sdake has joined #openstack-nova | 13:19 | |
*** sdake has quit IRC | 13:19 | |
*** sdake has joined #openstack-nova | 13:19 | |
*** acormier has quit IRC | 13:20 | |
*** acormier has joined #openstack-nova | 13:20 | |
*** zhurong has joined #openstack-nova | 13:20 | |
*** smatzek has joined #openstack-nova | 13:22 | |
*** baoli has joined #openstack-nova | 13:22 | |
*** gouthamr has joined #openstack-nova | 13:23 | |
*** mpaolino has joined #openstack-nova | 13:23 | |
*** smatzek has quit IRC | 13:24 | |
*** smatzek has joined #openstack-nova | 13:25 | |
*** takashin has left #openstack-nova | 13:27 | |
mriedem | gibi: https://review.openstack.org/#/c/510938/ | 13:28 |
*** smatzek has quit IRC | 13:29 | |
*** takedakn has quit IRC | 13:30 | |
sahid | stephenfin: it's not clear for me how are you make working "preferred", I mean in a technical POV | 13:33 |
sahid | https://review.openstack.org/#/c/361140/28/specs/queens/approved/share-pci-between-numa-nodes.rst | 13:33 |
sahid | stephenfin: you are probably going to use a weigher? | 13:33 |
gmann | mriedem: is it ok for you now - https://review.openstack.org/#/c/459483/33/nova/api/openstack/compute/server_migrations.py@58 | 13:33 |
gmann | mriedem: not having whitelist to add 'type' in response | 13:34 |
stephenfin | sahid: What aspect of it? | 13:34 |
sahid | scheduling | 13:34 |
*** penick has quit IRC | 13:35 | |
*** jaypipes has joined #openstack-nova | 13:35 | |
*** priteau has joined #openstack-nova | 13:35 | |
sahid | stephenfin: from how i see that you need two pass, and i don't think it"s something that the current scheduler provides | 13:36 |
*** sree has joined #openstack-nova | 13:36 | |
*** itlinux has quit IRC | 13:36 | |
stephenfin | sahid: Yeah, I think you might be right there. We're probably going to stick with making it configurable at the host leve | 13:37 |
stephenfin | i.e. you use the first host that is given to you by the scheduler | 13:37 |
*** penick has joined #openstack-nova | 13:37 | |
sahid | stephenfin: seems more reasonable yes | 13:38 |
stephenfin | I don't think the two-pass solution is possible either. jaypipes might have suggestions though | 13:38 |
stephenfin | sahid: Cool. I can update the spec accordingly | 13:38 |
*** jistr is now known as jistr|mtg | 13:38 | |
sahid | mriedem, jaypipes can one of you look ACK this this https://review.openstack.org/#/c/497457/ ? | 13:38 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 13:41 |
*** hshiina has quit IRC | 13:43 | |
*** udesale has quit IRC | 13:43 | |
mriedem | sahid: need to see the live migration job pass on that first | 13:43 |
mriedem | stephenfin: ^ | 13:43 |
*** mdnadeem has quit IRC | 13:46 | |
*** sballe_ has joined #openstack-nova | 13:47 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 13:47 |
*** burt has joined #openstack-nova | 13:48 | |
*** avtar_ has quit IRC | 13:49 | |
sahid | jaypipes: i replied to you about the sriov-trusted-vfs (in case you did not notice it) | 13:49 |
*** armax has joined #openstack-nova | 13:49 | |
*** crushil has joined #openstack-nova | 13:51 | |
*** udesale has joined #openstack-nova | 13:55 | |
*** jmlowe has joined #openstack-nova | 13:55 | |
*** awaugama has joined #openstack-nova | 13:56 | |
*** links has quit IRC | 13:56 | |
*** alexchadin has quit IRC | 13:58 | |
*** jistr|mtg is now known as jistr | 14:00 | |
mdbooth | stephenfin: Can't say I'm a fan of that change. Haven't put a -1 only because it's *probably* better than what we have now. Not sure if the added complexity is worth it, though. | 14:02 |
* mdbooth likes data constraints. | 14:02 | |
*** gmann is now known as gmann_afk | 14:03 | |
*** spectr-RH has quit IRC | 14:04 | |
mriedem | sahid: which hypervisors don't support this flag? https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_MIGRATE_PARAM_BANDWIDTH | 14:04 |
*** spectr has joined #openstack-nova | 14:04 | |
*** jwcroppe has joined #openstack-nova | 14:07 | |
sahid | mriedem: hum.. let me check that point | 14:08 |
*** slaweq_ has quit IRC | 14:08 | |
*** chyka has joined #openstack-nova | 14:09 | |
openstackgerrit | Merged openstack/nova master: Don't fix protocol-less glance api_servers anymore https://review.openstack.org/505317 | 14:09 |
sahid | mriedem: according to a simple grep in libvirt source it seems that vz and QEMU are supporting it | 14:12 |
sahid | but but nothing about vmware, hyperv.. | 14:12 |
*** gbarros has joined #openstack-nova | 14:12 | |
*** lyan has quit IRC | 14:13 | |
*** chyka has quit IRC | 14:13 | |
mriedem | sahid: well we wouldn't care about vmware or hyperv in the libvirt driver | 14:13 |
*** eharney has quit IRC | 14:13 | |
mriedem | but we care about xen, although i'm not sure if libvirt+xen is listed as supporting live migration anyway | 14:13 |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax https://review.openstack.org/510244 | 14:13 |
mriedem | according to this it says we do https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_live_migrate | 14:13 |
*** rajinir-afk is now known as rajinir | 14:14 | |
*** rmart04 has joined #openstack-nova | 14:14 | |
openstackgerrit | Merged openstack/nova master: Move user_data max length check to schema https://review.openstack.org/510704 | 14:14 |
*** penick has quit IRC | 14:14 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 14:14 |
*** artom has joined #openstack-nova | 14:14 | |
*** gbarros has quit IRC | 14:17 | |
*** lyan has joined #openstack-nova | 14:17 | |
*** gbarros has joined #openstack-nova | 14:17 | |
sahid | mriedem: we always use that parameter (bandwiddh) for live-migration | 14:18 |
*** takedakn has joined #openstack-nova | 14:18 | |
sahid | do you have a suggestion on how the code should be? | 14:19 |
mriedem | it must work for xen then | 14:19 |
sahid | I guess yes | 14:19 |
mriedem | libvirt+xen i mean | 14:19 |
sahid | but I can't ensure that | 14:19 |
mriedem | anthonyper: do you know what the current state of libvirt+xen live migration is? working? | 14:19 |
mriedem | has anyone tested lately? | 14:19 |
anthonyper | mriedem: I have no idea :( | 14:25 |
*** takedakn has quit IRC | 14:25 | |
*** zhurong has quit IRC | 14:29 | |
*** acormier has quit IRC | 14:30 | |
*** trinaths has joined #openstack-nova | 14:30 | |
*** acormier has joined #openstack-nova | 14:31 | |
*** lajoskatona has quit IRC | 14:32 | |
anthonyper | mriedem: I'll ask other people to try to find out if they have tested live migration recently. | 14:33 |
*** udesale has quit IRC | 14:34 | |
mriedem | anthonyper: ok thanks | 14:34 |
*** acormier has quit IRC | 14:35 | |
sahid | dansmith: if the wait_for_instance_event has been desgined has you are suggesting so it's not working | 14:36 |
*** acormier has joined #openstack-nova | 14:37 | |
dansmith | sahid: I'm not sure what you're saying | 14:38 |
jaypipes | sahid: yep, noticed it. sorry, been spending my last days rebasing the nested resource providers series. | 14:38 |
jaypipes | sahid: I'll review that slow live-migration patch this morning. | 14:38 |
openstackgerrit | Merged openstack/nova master: Add connection pool size to vSphere settings https://review.openstack.org/414085 | 14:38 |
*** felipemonteiro has joined #openstack-nova | 14:38 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 14:39 |
sahid | jaypipes: no worries and thanks if you can have a look | 14:39 |
*** mvk has quit IRC | 14:39 | |
openstackgerrit | Merged openstack/nova master: Clarify conf/compute.py help text for ListOpts https://review.openstack.org/468538 | 14:39 |
sahid | dansmith: related to one of your comment on https://review.openstack.org/#/c/497457/ | 14:39 |
sahid | the other one are just nits, right? | 14:40 |
*** felipemonteiro_ has joined #openstack-nova | 14:40 | |
dansmith | sahid: no, breaking n-net migrations is not a nit :) | 14:41 |
*** acormier has quit IRC | 14:42 | |
sahid | dansmith: can you clearly put a suggestion? do you me to add a condition in the code? | 14:42 |
dansmith | sahid: you can handle it like the spawn code does for if neutron isn't enabled | 14:43 |
mriedem | if (self._conn_supports_start_paused and | 14:43 |
mriedem | utils.is_neutron() and not | 14: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 |
mriedem | yeah | 14:43 |
mriedem | if this and that and the other thing else something different | 14:43 |
*** spectr has quit IRC | 14:43 | |
sahid | dansmith: seems reasonable | 14:43 |
*** felipemonteiro has quit IRC | 14:44 | |
dansmith | sahid: 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 waiter | 14:44 |
sahid | dansmith: depending of the mech used, i added a comment in the code | 14:45 |
dansmith | sahid: ah, so two different places depending on OVS or linuxbridge? that's going to be a mess to handle properly | 14:47 |
dansmith | but what you have isn't right for either | 14:47 |
dansmith | although it's much much wronger for OVS | 14:47 |
*** suresh12 has joined #openstack-nova | 14:47 | |
sahid | dansmith: i guess you are going to elaborate? | 14:50 |
dansmith | sahid: I did in my comments | 14:50 |
dansmith | nova events are not queued | 14:50 |
dansmith | if you don't have a waiter running before an event arrives, it is dropped | 14:50 |
dansmith | if 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 service | 14:51 |
*** suresh12 has quit IRC | 14:51 | |
dansmith | delaying live migration for five minutes, timeout, log error and then continue is not a good thing to do just because you raced with neutron | 14:51 |
mdbooth | mriedem: To spec, or not to spec: https://blueprints.launchpad.net/nova/+spec/local-disk-serial-numbers ? | 14:54 |
mdbooth | dansmith: Same question ^^^ | 14:55 |
sahid | dansmith: i see so we will have to add this specificly for ovs during pre-live-migration when we are creating the bridge on dest | 14:55 |
sahid | and when we start calling libvirt.migrateToURI for LB | 14:55 |
*** penick has joined #openstack-nova | 14:56 | |
dansmith | sahid: neither option seems very ideal to me, but yes, that's the issue with what you have here | 14:56 |
dansmith | mdbooth: probably a spec so we can argue about your id generation method | 14:57 |
mdbooth | dansmith: Yeay! I do love a good opportunity to argue about stuff. | 14:57 |
mdbooth | Ok | 14:57 |
*** spectr has joined #openstack-nova | 14:58 | |
mriedem | gmann_afk: commented in that change and the one below it | 14:58 |
*** penick has quit IRC | 15:00 | |
*** penick has joined #openstack-nova | 15:00 | |
*** psachin has quit IRC | 15:02 | |
*** trinaths has left #openstack-nova | 15:03 | |
*** larainema has quit IRC | 15:05 | |
*** sree has quit IRC | 15:06 | |
*** sree has joined #openstack-nova | 15:07 | |
sahid | dansmith: 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-nova | 15:07 | |
mriedem | sahid: i looked at one of the live migration job runs on your patch and it's never waiting for any events | 15:07 |
openstackgerrit | Eric Fried proposed openstack/nova master: Nix bug msg from ConfGroupForServiceTypeNotFound https://review.openstack.org/511261 | 15:08 |
efried | mriedem ^ cleanup as requested. | 15:08 |
*** Oku_OS is now known as Oku_OS-away | 15:08 | |
sahid | mriedem: what is the mech? for linux bridge we need that patch on neutron side | 15:08 |
sahid | https://review.openstack.org/#/c/506182/ | 15:08 |
mriedem | sahid: the depends-on will pull the change in via zuul | 15:09 |
mriedem | sahid: it's literally building an empty list of events | 15:09 |
mriedem | meaning there are no vifs in the instance.info_cache | 15:09 |
mriedem | this would be ovs | 15:09 |
*** markus_z has quit IRC | 15:09 | |
mriedem | 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_06_172675 | 15:10 |
sahid | mriedem: hm... if there is no vifs so yes i guess it's normal | 15:10 |
*** sree has quit IRC | 15:11 | |
mriedem | hmm 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_315354 | 15:11 |
*** links has joined #openstack-nova | 15:11 | |
mriedem | cfriesen: ^ this probably explains why i never saw any improvement in the network-changed event queueing patch for live migration | 15:11 |
sahid | hum... | 15:12 |
mriedem | cfriesen: this one https://review.openstack.org/#/c/465787/ | 15:12 |
*** jwcroppe has quit IRC | 15:15 | |
*** smatzek has joined #openstack-nova | 15:16 | |
*** jianghuaw_ has joined #openstack-nova | 15:16 | |
*** jwcroppe has joined #openstack-nova | 15:16 | |
efried | We needing to pay attention to bogus Zuul merge conflicts at this point? | 15:17 |
mriedem | no | 15:17 |
mriedem | well, idk, but i'm not | 15:17 |
dansmith | today we do | 15:17 |
efried | Thought they cut over this morning. | 15:17 |
dansmith | once the cutover starts | 15:17 |
*** ralonsoh_ has quit IRC | 15:18 | |
*** smatzek has quit IRC | 15:18 | |
*** suresh12 has joined #openstack-nova | 15:19 | |
*** smatzek has joined #openstack-nova | 15:19 | |
*** kylek3h has joined #openstack-nova | 15:20 | |
*** jwcroppe has quit IRC | 15:20 | |
*** andreas_s has quit IRC | 15:29 | |
*** andreas_s has joined #openstack-nova | 15:29 | |
*** gyee has joined #openstack-nova | 15: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 IRC | 15:38 | |
*** andreas_s has quit IRC | 15:38 | |
openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/511021 | 15:38 |
*** rodolof has quit IRC | 15:39 | |
bauzas | jianghuaw: I tend to say yes | 15:40 |
jaypipes | jianghuaw_: yes | 15:40 |
bauzas | jianghuaw: FWIW, I'm working on the libvirt side | 15:40 |
jaypipes | jianghuaw_: feel free to take it. | 15:40 |
jaypipes | jianghuaw_: I restored it just now. feel free to git review -d 339204 and take over ownership | 15:41 |
jaypipes | jianghuaw_: or cherry-pick it as you want\ | 15:41 |
jianghuaw_ | bauzas, jaypipes: great. Thanks. | 15:41 |
*** smatzek has quit IRC | 15:41 | |
*** smatzek has joined #openstack-nova | 15:42 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 15:42 |
*** hongbin has joined #openstack-nova | 15:43 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for keystone conf & requests https://review.openstack.org/507693 | 15:44 |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | 15:44 |
openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | 15:44 |
openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | 15:45 |
openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | 15:45 |
openstackgerrit | melanie witt proposed openstack/nova master: Make expected notifications output easier to read in tests https://review.openstack.org/510823 | 15:45 |
openstackgerrit | melanie witt proposed openstack/nova master: Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | 15:46 |
*** jianghuaw_ has quit IRC | 15:46 | |
*** jianghuaw_ has joined #openstack-nova | 15:47 | |
*** smatzek has quit IRC | 15:47 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot https://review.openstack.org/511209 | 15:48 |
*** penick_ has joined #openstack-nova | 15:51 | |
*** smatzek has joined #openstack-nova | 15:51 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 15:52 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: DNM: Fail if we try to update volume connections during reboot https://review.openstack.org/511031 | 15:52 |
ildikov | mriedem: attempt to fix the attachment_update mystery ^^ | 15:52 |
*** AlexeyAbashkin has quit IRC | 15:53 | |
*** penick has quit IRC | 15:53 | |
*** ratailor has joined #openstack-nova | 15:54 | |
mriedem | ildikov: HA! | 15:54 |
mriedem | holy shit nice catch | 15:54 |
*** ralonsoh has joined #openstack-nova | 15:54 | |
ildikov | mriedem: my pleasure :) | 15:55 |
ildikov | jgriffith: it seems we're off the hook this time :) ^^ | 15:56 |
mriedem | there is another one like that in here | 15:56 |
mriedem | might as well fix that one too | 15:56 |
*** nicolasbock has quit IRC | 15:56 | |
*** abhi89 has joined #openstack-nova | 15:56 | |
*** slaweq has joined #openstack-nova | 15:57 | |
*** pcaruana has quit IRC | 15:59 | |
*** rcernin has quit IRC | 15:59 | |
*** abhishek has quit IRC | 15:59 | |
*** kylek3h has quit IRC | 16:01 | |
*** kylek3h has joined #openstack-nova | 16:02 | |
*** rodolof has joined #openstack-nova | 16:02 | |
*** lucasagomes is now known as lucas-afk | 16:02 | |
*** ratailor has quit IRC | 16:03 | |
*** jianghuaw_ has quit IRC | 16:03 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029 | 16:03 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030 | 16:03 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031 | 16:03 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032 | 16:03 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 16:04 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 16:04 |
*** rodolof has quit IRC | 16:04 | |
*** sahid has quit IRC | 16:04 | |
*** rodolof has joined #openstack-nova | 16:04 | |
*** slaweq has quit IRC | 16:05 | |
*** rodolof has quit IRC | 16:06 | |
*** kylek3h has quit IRC | 16:06 | |
*** rodolof has joined #openstack-nova | 16:06 | |
ildikov | mriedem: will do, thanks | 16:07 |
*** rodolof has quit IRC | 16:08 | |
ildikov | mriedem: I was too eager to get a clean run on the new attach patch :) | 16:08 |
*** rodolof has joined #openstack-nova | 16:08 | |
*** nicolasbock has joined #openstack-nova | 16:09 | |
*** kylek3h has joined #openstack-nova | 16:09 | |
*** kylek3h has quit IRC | 16:09 | |
*** hemna_ has joined #openstack-nova | 16:10 | |
*** itlinux has joined #openstack-nova | 16:11 | |
openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | 16:11 |
dansmith | mdbooth: if you want that bdm uuid patch for the serial number thing, I think we should slam it in soonly | 16:12 |
*** hemna_ has quit IRC | 16:13 | |
dansmith | I think everyone knows it's a good idea and uncontentious, it's just never been in the critical path for anyone for long enough | 16:13 |
dansmith | so I say get it cleaned up and we should be able to get it in, IMHO | 16:13 |
mdbooth | dansmith: 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 |
dansmith | mdbooth: the spec is the serial number thing, uuid on bdm is just a work item in that spec | 16:14 |
openstackgerrit | Eric Fried proposed openstack/nova master: DNM: CI check https://review.openstack.org/511006 | 16:14 |
mdbooth | Incidentally, having a DriverBlockDevice object for the root disk is also something which is useful for a bunch of other stuff. | 16:14 |
mdbooth | Ok. Look for me adding you to it tomorrow :) | 16:15 |
*** rmart04 has quit IRC | 16:15 | |
*** kylek3h has joined #openstack-nova | 16:15 | |
*** hemna_ has joined #openstack-nova | 16:16 | |
mriedem | melwitt: dansmith: see comment #16, i thought we had this fixed already? https://bugs.launchpad.net/nova/+bug/1682423 | 16:18 |
openstack | Launchpad bug 1682423 in OpenStack Compute (nova) "create instance with dashbaord / 404 returned" [Undecided,Confirmed] | 16:18 |
dansmith | mriedem: afaik yeah | 16:21 |
openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_image_certificates https://review.openstack.org/500396 | 16:21 |
dansmith | mriedem: "latest available ocata packages" doesn't mean much to me | 16:21 |
mriedem | i meant 15.0.7 | 16:22 |
mriedem | not whatever ubuntu has currently | 16:22 |
dansmith | I mean the person who said it's still happening for them | 16:22 |
mriedem | oh | 16:22 |
*** rodolof has quit IRC | 16:23 | |
mriedem | i 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 IRC | 16:25 | |
mdbooth | mriedem: 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-nova | 16:25 | |
mriedem | not right now | 16:26 |
mdbooth | lyarwood's stable virt | 16:26 |
mdbooth | mriedem: Ok. Is it likely to make next week? | 16:26 |
mriedem | idk | 16:26 |
mriedem | lots of people can review specs | 16:26 |
mriedem | i've got my own to write and update as well | 16:26 |
mriedem | kind of concerned with a bunch of people saying they are busted on ocata atm | 16:27 |
mdbooth | mriedem: np, just pinging you specifically as you've been involved with it previously. | 16:27 |
*** kylek3h has quit IRC | 16:29 | |
*** yamahata has joined #openstack-nova | 16:30 | |
mriedem | dansmith: wonder if these people are running with uwsgi? :) | 16:32 |
*** acormier has joined #openstack-nova | 16:32 | |
mriedem | remember that defeated the service version check | 16:32 |
mriedem | for tripleo | 16:32 |
dansmith | mriedem: ocata so no right? | 16:32 |
dansmith | oh | 16:32 |
mriedem | but they were hitting something similar, | 16:32 |
mriedem | unti lwe told them not to do that | 16:32 |
*** weshay is now known as weshay|ruck | 16:32 | |
dansmith | hmm | 16:32 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Fix connection info refresh for reboot https://review.openstack.org/511209 | 16:33 |
dansmith | mriedem: see comment 9 | 16:33 |
dansmith | mriedem: makes reference to having everything upgraded | 16:33 |
*** Swami has joined #openstack-nova | 16:35 | |
mriedem | yeah i think there are two different groups of people with different issues | 16:35 |
mriedem | once is talking about having old computes, which shouldn't have anything to do with looking up the instnace | 16:35 |
mriedem | i think the others are talking about a race to lookup the instance while it's building | 16:36 |
*** mpaolino has quit IRC | 16:36 | |
mriedem | which is why they added a sleep in there i guess and said it's working | 16:36 |
dansmith | well, one unupdated api will wreck it right? | 16:36 |
mriedem | b/c of the api version check? | 16:36 |
dansmith | yeah | 16:36 |
dansmith | it'll go straight to the cell db and not look at the BR | 16:37 |
*** abhishek has joined #openstack-nova | 16:37 | |
mriedem | yeah | 16:37 |
mriedem | but, someone said they have a full new install | 16:37 |
mriedem | and that was the person that put the time.sleep(1) workaround in | 16:37 |
dansmith | all they need is one api service record that reported zero for the version | 16:38 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 16:38 |
*** ralonsoh has quit IRC | 16:39 | |
*** dtantsur is now known as dtantsur|afk | 16:39 | |
dansmith | and they did it before that check | 16:39 |
dansmith | the sleep I mean | 16:39 |
*** felipemonteiro_ has quit IRC | 16:40 | |
*** abhi89 has quit IRC | 16:40 | |
*** chyka has joined #openstack-nova | 16:41 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 16:41 |
*** chyka has quit IRC | 16:45 | |
*** jmlowe has quit IRC | 16:47 | |
mriedem | assuming their api services are all updated, i don't see how they could hit this really | 16:48 |
mriedem | we delete the build request after the instance mapping is updated, | 16:48 |
mriedem | we update the instance mapping after the instance is created in a cell | 16:48 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Target context for build notification in conductor https://review.openstack.org/511292 | 16:49 |
mriedem | if 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 again | 16:49 |
dansmith | mriedem: aye | 16:49 |
mriedem | gdi this is like 2 days i've spent all day chasing weird bugs | 16:49 |
*** links has quit IRC | 16:54 | |
*** gszasz has quit IRC | 16:57 | |
*** derekh has quit IRC | 16:58 | |
*** andreas_s has joined #openstack-nova | 16:59 | |
*** baoli has quit IRC | 16:59 | |
*** felipemonteiro has joined #openstack-nova | 17:00 | |
*** baoli has joined #openstack-nova | 17:00 | |
*** Apoorva has joined #openstack-nova | 17:01 | |
*** felipemonteiro_ has joined #openstack-nova | 17:01 | |
*** gouthamr has quit IRC | 17:04 | |
*** andreas_s has quit IRC | 17:05 | |
*** felipemonteiro has quit IRC | 17:06 | |
*** andreas_s has joined #openstack-nova | 17:07 | |
*** jpena is now known as jpena|off | 17:09 | |
*** suresh12 has quit IRC | 17:13 | |
*** nikhil has joined #openstack-nova | 17:14 | |
*** tesseract has quit IRC | 17:15 | |
*** andreas_s has quit IRC | 17:15 | |
*** andreas_s has joined #openstack-nova | 17:16 | |
*** sambetts is now known as sambetts|afk | 17:17 | |
mriedem | melwitt: does https://review.openstack.org/#/c/511292/ also need to go to ocata? | 17:21 |
*** AlexeyAbashkin has joined #openstack-nova | 17:21 | |
*** READ10 has joined #openstack-nova | 17:22 | |
melwitt | hm, lemme check. I had thought not but maybe this code was from ocata | 17:22 |
mriedem | looks like yes https://github.com/openstack/nova/blob/stable/ocata/nova/conductor/manager.py#L921 | 17:22 |
mriedem | it also looks like ocata is not targeting the instance action in the cell, so that's getting created in cell0 | 17:23 |
*** gjayavelu has joined #openstack-nova | 17:23 | |
melwitt | yeah, was about to say that too | 17:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 17:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 17:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 17:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 17:23 |
openstackgerrit | Dan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306 | 17:23 |
mriedem | https://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca | 17:24 |
dansmith | mriedem: ^ updated to determine the source compute from allocations instead of trying to look it up | 17:24 |
mriedem | melwitt: https://github.com/openstack/nova/commit/ca51db7e2db72cb774c94fb039509f2b4e6c94ca fixed the instance action thing in pike, we'll want to backport that as well... | 17:24 |
mriedem | didn't have a bug o nit | 17:24 |
melwitt | ah, crap | 17:24 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Make conductor create InstanceAction in the proper cell https://review.openstack.org/511307 | 17:25 |
mriedem | thar she blar | 17:25 |
melwitt | k, cool | 17:25 |
melwitt | I should stack the other backport on that I think | 17:25 |
mriedem | yup | 17:26 |
mriedem | dansmith: ok | 17:26 |
melwitt | to increase the chance of clean backport. k | 17:26 |
mriedem | dansmith: +2 on mel's pike backport https://review.openstack.org/#/c/511292/ | 17:26 |
*** AlexeyAbashkin has quit IRC | 17:26 | |
mriedem | i need food in my belly | 17:26 |
*** awaugama has quit IRC | 17:27 | |
*** links has joined #openstack-nova | 17:27 | |
*** andreas_s has quit IRC | 17:29 | |
*** AlexeyAbashkin has joined #openstack-nova | 17:29 | |
*** tbachman has quit IRC | 17:29 | |
jaypipes | dansmith: I think zuul might be lying about the merge failure messages. | 17:32 |
dansmith | jaypipes: I don't think they've actually started the cutover yet | 17:32 |
dansmith | which means, yes | 17:32 |
*** baoli_ has joined #openstack-nova | 17:33 | |
*** AlexeyAbashkin has quit IRC | 17:34 | |
*** tbachman has joined #openstack-nova | 17:34 | |
melwitt | it's expected to emit false merge conflicts prior to cutover? I guess I am missing something obvious | 17:34 |
dansmith | it's been doing that all week | 17:34 |
melwitt | I know | 17:34 |
*** fragatina has quit IRC | 17:35 | |
*** baoli has quit IRC | 17:35 | |
dansmith | I'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 behavior | 17:35 |
dansmith | I would assume that whatever has been giving zuul merge conflict heartburn over the last week is factored into the switchover | 17:36 |
melwitt | I had been thinking they were fixing zuul v3 before the cutover so maybe those would get fixed prior to the cutover | 17:36 |
melwitt | yeah. I guess I should ask | 17:37 |
dansmith | unless the failure is because they're not having it do the post-merge conflict checks because of resources or something | 17:37 |
dansmith | since it's highly constrained at the moment, and not doing the actual merges | 17:37 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 17:37 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 17:37 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 17:37 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 17:37 |
dansmith | I'm not sure what jaypipes' point was I guess | 17:38 |
jaypipes | efried: k, updated the n-r-p spec to address your comments ^^ | 17:38 |
efried | Thanks jaypipes | 17:38 |
jaypipes | dansmith: generally, you can assume I am pointless. | 17:38 |
efried | dansmith jaypipes They figured out why the bogus merge conflicts were happening. | 17:38 |
efried | But no, they haven't cut over yet. | 17:38 |
efried | http://eavesdrop.openstack.org/irclogs/%23openstack-infra-incident/%23openstack-infra-incident.2017-10-11.log in case you're interested. | 17:39 |
melwitt | indeed they know about it in #infra and they're working on it | 17:39 |
dansmith | so yeah, like I said, it's getting old data because it's not doing the actual merges | 17:40 |
*** itlinux has quit IRC | 17:40 | |
melwitt | okay. I didn't know how it worked | 17:40 |
jaypipes | efried, dansmith: k, thx for the info. | 17:41 |
openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 17:41 |
openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 17:41 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 17:41 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 17:42 |
*** andreas_s has joined #openstack-nova | 17:43 | |
*** smatzek has quit IRC | 17:44 | |
*** smatzek has joined #openstack-nova | 17:45 | |
*** lpetrut_ has quit IRC | 17:45 | |
abhishek | Hi all.. can someone please review https://review.openstack.org/#/c/505546/ | 17:45 |
*** andreas_s has quit IRC | 17:47 | |
*** smatzek_ has joined #openstack-nova | 17:48 | |
*** andreas_s has joined #openstack-nova | 17:48 | |
*** slaweq has joined #openstack-nova | 17:48 | |
*** smatzek has quit IRC | 17:49 | |
*** avolkov has quit IRC | 17:50 | |
*** smatzek_ has quit IRC | 17:52 | |
efried | jaypipes https://review.openstack.org/#/c/377138/ not sure if I'm off base there... | 17:52 |
*** slaweq has quit IRC | 17:53 | |
*** links has quit IRC | 17:53 | |
*** andreas_s has quit IRC | 17:56 | |
*** ijw has joined #openstack-nova | 17:57 | |
*** kylek3h has joined #openstack-nova | 17:59 | |
*** smatzek has joined #openstack-nova | 17:59 | |
*** ijw has quit IRC | 18:04 | |
*** suresh12 has joined #openstack-nova | 18:04 | |
*** ijw has joined #openstack-nova | 18:05 | |
*** andreas_s has joined #openstack-nova | 18:06 | |
*** suresh12 has quit IRC | 18:09 | |
*** jwcroppe has joined #openstack-nova | 18:10 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 18:10 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 18:10 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 18:10 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 18:10 |
*** lpetrut_ has joined #openstack-nova | 18:10 | |
jaypipes | efried: good feedback, thanks. fixed ^^ | 18:10 |
efried | jaypipes rgr | 18:11 |
efried | sdague 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 IRC | 18:11 | |
efried | jaypipes Cool man, I think I'm +1 on the whole stack. Let me know if I missed any. | 18:12 |
*** kylek3h has quit IRC | 18:12 | |
jaypipes | efried: thanks for the great reviews. appreciated. | 18:12 |
efried | yahyoubetcha :) | 18:13 |
*** acormier has quit IRC | 18:13 | |
*** suresh12 has joined #openstack-nova | 18:14 | |
openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/488541 | 18:16 |
*** efried is now known as efried_nomnom | 18:17 | |
*** jmlowe has joined #openstack-nova | 18:18 | |
*** brault has joined #openstack-nova | 18:19 | |
*** artom_ has joined #openstack-nova | 18:20 | |
*** vvargaszte has joined #openstack-nova | 18:22 | |
*** artom has quit IRC | 18:23 | |
*** brault has quit IRC | 18:23 | |
*** baoli_ has quit IRC | 18:24 | |
*** baoli has joined #openstack-nova | 18:24 | |
*** gyee has quit IRC | 18:27 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595 | 18:28 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 18:28 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 18:28 |
*** gyee has joined #openstack-nova | 18:29 | |
mriedem | dansmith: 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 |
mriedem | that would be great mmmkay? | 18:32 |
*** vvargaszte has quit IRC | 18:32 | |
melwitt | thaaaaanks | 18:32 |
dansmith | mriedem: wonkified? honestly matt... | 18:33 |
dansmith | mriedem: so, on the shared storage thing, | 18:34 |
dansmith | mriedem: I think that when we do that we'll have the instance hold its shared storage allocation on the new host, | 18:35 |
dansmith | not duplicate it for the migration one | 18:35 |
dansmith | the only case would be if we're root-on-shared-ephemeral I guess | 18:35 |
dansmith | so, I dunno, I'll put a NOTE as I don't really know what the TODO is until later | 18:36 |
*** gyee has quit IRC | 18:37 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306 | 18:37 |
openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 18:37 |
openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 18:37 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 18:37 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 18:37 |
*** gyee has joined #openstack-nova | 18:38 | |
mriedem | ok | 18:39 |
mriedem | wonkification of the SchedulerReportClient 2.0 | 18:39 |
*** gyee has quit IRC | 18:39 | |
dansmith | heh | 18:40 |
*** gyee has joined #openstack-nova | 18:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec for flavor description https://review.openstack.org/501017 | 18:55 |
mriedem | sdague: made those changes on restricting the id and name fields ^ | 18:55 |
sdague | mriedem: will look | 18:55 |
*** efried_nomnom is now known as efried | 18:58 | |
*** smatzek has quit IRC | 18:58 | |
*** smatzek has joined #openstack-nova | 19:00 | |
*** smatzek has quit IRC | 19:00 | |
*** smatzek has joined #openstack-nova | 19:01 | |
*** masuberu has quit IRC | 19:01 | |
*** smatzek_ has joined #openstack-nova | 19:02 | |
*** smatzek_ has quit IRC | 19:02 | |
*** smatzek_ has joined #openstack-nova | 19:02 | |
*** smatzek has quit IRC | 19:05 | |
openstackgerrit | Merged openstack/nova master: Update cinder in RequestContext service catalog https://review.openstack.org/510947 | 19:07 |
*** brault has joined #openstack-nova | 19:09 | |
sdague | mriedem: that looks completely sane to me | 19:12 |
*** brault has quit IRC | 19:13 | |
mriedem | not only sane, but completely sane? | 19:13 |
mriedem | nice | 19:13 |
mriedem | huh https://developer.openstack.org/api-ref/placement/#resource-provider-inventories | 19:17 |
mriedem | is missing POST | 19:17 |
*** tbachman has quit IRC | 19:18 | |
*** mvk has joined #openstack-nova | 19:18 | |
*** slaweq has joined #openstack-nova | 19:18 | |
*** READ10 has quit IRC | 19:19 | |
*** brault has joined #openstack-nova | 19:22 | |
*** rmk has quit IRC | 19:23 | |
*** rmk has joined #openstack-nova | 19:26 | |
*** rmk has quit IRC | 19:26 | |
*** rmk has joined #openstack-nova | 19:26 | |
*** acormier has joined #openstack-nova | 19:26 | |
*** jamesdenton has joined #openstack-nova | 19:29 | |
*** jmlowe has quit IRC | 19:30 | |
*** gyee has quit IRC | 19:31 | |
*** jwcroppe has quit IRC | 19:31 | |
*** vvargaszte has joined #openstack-nova | 19:34 | |
*** vvargaszte has quit IRC | 19:35 | |
openstackgerrit | sean mooney proposed openstack/nova master: WIP reorder destination host setup https://review.openstack.org/510658 | 19:37 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Target context for build notification in conductor https://review.openstack.org/511325 | 19:41 |
mriedem | easy fix here: https://review.openstack.org/#/c/511209/ | 19:47 |
mriedem | hot dogs and cracker jacks here! | 19:47 |
*** jmlowe has joined #openstack-nova | 19:48 | |
melwitt | +W | 19:49 |
mriedem | https://bugs.launchpad.net/nova/+bug/1722891 in case someone wants it | 19:50 |
openstack | Launchpad bug 1722891 in OpenStack Compute (nova) "api-ref: POST /resource_providers/{uuid}/inventories is missing" [Medium,Confirmed] - Assigned to hongbin (hongbin034) | 19:50 |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot https://review.openstack.org/511074 | 19:50 |
*** lyan has quit IRC | 19:51 | |
*** gyee has joined #openstack-nova | 19:51 | |
mriedem | grenade jobs seem totally borked in zuulv3 | 19:52 |
*** lyan has joined #openstack-nova | 19:52 | |
melwitt | mriedem: looks like hongbin is already gonna work on the bug you linked, so that's cool | 19:53 |
mriedem | something really weird happens with our request id log handlin | 19:55 |
mriedem | seems to be not thread safe or something | 19:55 |
mriedem | i'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 |
mriedem | but then i see the same request id showing up in log messages for periodic tasks that are running | 19:55 |
melwitt | hm, I can't say I've seen that before | 19:56 |
melwitt | which makes me think it's a relatively new issue | 19:57 |
*** gyee has quit IRC | 19:57 | |
*** Apoorva_ has joined #openstack-nova | 19:58 | |
*** harlowja has quit IRC | 20:00 | |
*** liverpooler has quit IRC | 20:00 | |
*** Apoorva has quit IRC | 20:02 | |
*** ociuhandu has quit IRC | 20:02 | |
*** itlinux has joined #openstack-nova | 20:03 | |
*** sree has joined #openstack-nova | 20:07 | |
*** edmondsw_ has joined #openstack-nova | 20:08 | |
*** edmondsw has quit IRC | 20:08 | |
*** edmondsw_ is now known as edmondsw | 20:12 | |
*** sree has quit IRC | 20:12 | |
*** tbachman has joined #openstack-nova | 20:19 | |
*** tbachman has quit IRC | 20:22 | |
*** slaweq has quit IRC | 20:24 | |
*** armax has quit IRC | 20:25 | |
*** smatzek_ has quit IRC | 20:28 | |
*** crushil has quit IRC | 20:28 | |
*** smatzek has joined #openstack-nova | 20:30 | |
*** andreas_s has joined #openstack-nova | 20:30 | |
*** fragatina has joined #openstack-nova | 20:31 | |
*** slaweq has joined #openstack-nova | 20:32 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix server status values in GET /servers docs https://review.openstack.org/510696 | 20:34 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update "SHUTOFF" description in API guide https://review.openstack.org/510697 | 20:34 |
*** andreas_s has quit IRC | 20:35 | |
*** crushil has joined #openstack-nova | 20:36 | |
*** priteau has quit IRC | 20:38 | |
*** jmlowe has quit IRC | 20:38 | |
*** crushil has quit IRC | 20:41 | |
mriedem | uh, filter(~models.Migration.status.in_ means NOT IN right? | 20:43 |
mriedem | the ~ | 20:43 |
*** smatzek has quit IRC | 20:43 | |
*** smatzek has joined #openstack-nova | 20:44 | |
*** smatzek_ has joined #openstack-nova | 20:44 | |
efried | sdague 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 IRC | 20:47 | |
*** smatzek has quit IRC | 20:48 | |
*** smatzek_ has quit IRC | 20:49 | |
*** pchavva has quit IRC | 20:50 | |
*** tbachman has joined #openstack-nova | 20:53 | |
mriedem | dansmith: want to double check my thoughts about migration.status = 'done' in here? https://review.openstack.org/#/c/506419/ otherwise i think it's ok | 20:54 |
*** acormier has quit IRC | 20:55 | |
*** thorst_ has quit IRC | 20:55 | |
*** rodolof has joined #openstack-nova | 20:55 | |
*** jmlowe has joined #openstack-nova | 20:57 | |
*** thorst has joined #openstack-nova | 20:58 | |
*** penick_ has quit IRC | 20:59 | |
*** penick has joined #openstack-nova | 21:00 | |
mriedem | this also seems like it could bite us https://github.com/openstack/nova/blob/64635ad4a5f60a79e1ec2d5369a8f84bfeccb7e4/nova/db/sqlalchemy/api.py#L4859-L4862 | 21:01 |
mriedem | when the RT pulls migration records, it's for all migrations where either the source or dest is our local node | 21:01 |
*** catintheroof has quit IRC | 21:02 | |
*** rodolof has quit IRC | 21:02 | |
*** esberglu has quit IRC | 21:03 | |
*** thorst has quit IRC | 21:03 | |
*** rodolof has joined #openstack-nova | 21:03 | |
*** edmondsw has quit IRC | 21:07 | |
*** harlowja has joined #openstack-nova | 21:07 | |
*** rodolof has quit IRC | 21:08 | |
*** rodolof has joined #openstack-nova | 21:08 | |
dansmith | mriedem: yep will in a sec | 21:08 |
*** jmlowe has quit IRC | 21:12 | |
*** rodolof has quit IRC | 21:13 | |
*** rodolof has joined #openstack-nova | 21:14 | |
*** edmondsw has joined #openstack-nova | 21:14 | |
*** acormier has joined #openstack-nova | 21:16 | |
*** crushil has joined #openstack-nova | 21:17 | |
*** edmondsw has quit IRC | 21:19 | |
*** acormier has quit IRC | 21:20 | |
dansmith | mriedem: replied | 21:20 |
*** slaweq has quit IRC | 21:20 | |
mriedem | ok didn't think about not having migration allocations for evacs | 21:22 |
mriedem | but makes sense to not put the source node allocations on the migration record for an evacuation since the source node should be dead | 21:22 |
dansmith | and you're never going back there.. no rollback | 21:22 |
*** slaweq has joined #openstack-nova | 21:22 | |
mriedem | but...are we doubling today? | 21:22 |
mriedem | in the scheduler when we evacuate we must be doubling | 21:23 |
dansmith | if we are it's not for any useful reason I think | 21:23 |
*** smatzek has joined #openstack-nova | 21:23 | |
mriedem | i think it's just because we do it generically | 21:23 |
dansmith | I think we delete the allocation for the old node first anyway right? | 21:23 |
* dansmith checks | 21:23 | |
*** smatzek has quit IRC | 21:24 | |
*** smatzek has joined #openstack-nova | 21:24 | |
*** smatzek has quit IRC | 21:24 | |
*** smatzek has joined #openstack-nova | 21:25 | |
*** acormier has joined #openstack-nova | 21:25 | |
dansmith | allocate _for_evacuate_dest_host | 21:25 |
*** acormier has quit IRC | 21:25 | |
*** crushil has quit IRC | 21:25 | |
*** jmlowe has joined #openstack-nova | 21:25 | |
*** acormier has joined #openstack-nova | 21:26 | |
mriedem | that's when we're forcing the host during evacuate and bypassing the scheduler | 21:26 |
dansmith | yeah, we end up calling claim_resources and doubling | 21:26 |
dansmith | no, that method calls the scheduler | 21:26 |
mriedem | _allocate_for_evacuate_dest_host? | 21:26 |
mriedem | it doesn't | 21:26 |
mriedem | it calls scheduler utils | 21:27 |
dansmith | https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L782-L782 | 21:27 |
mriedem | b/c we do the same util thing for live migration with a forced host | 21:27 |
dansmith | which calls claim_resources | 21:27 |
*** slaweq has quit IRC | 21:27 | |
dansmith | sorry, maybe it does't call scheduler, but it does the doubling, which is what I meant | 21:27 |
mriedem | right, that's not select_destinations | 21:27 |
mriedem | yeah | 21:27 |
dansmith | sure, I was focused on the allocs | 21:27 |
mriedem | this is the thing i want to remove with a skip_filters flag to select_destinations | 21:27 |
*** rodolof has quit IRC | 21:27 | |
mriedem | so getting back to my original question i guess, | 21:27 |
*** rodolof has joined #openstack-nova | 21:27 | |
dansmith | regardless, I don't think this affects the new-world path | 21:27 |
mriedem | we never remove the allocation from the source node for an evacuation | 21:28 |
*** armax has joined #openstack-nova | 21:28 | |
mriedem | except when the source node comes back, if it does | 21:28 |
openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: add API reference for create inventory https://review.openstack.org/511342 | 21:28 |
dansmith | you think we never do it now? | 21:28 |
dansmith | or you think we never will with the new-world way? | 21:28 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L680 | 21:28 |
dansmith | ah, maybe your point is, | 21:28 |
mriedem | it's going to be a weird side thing | 21:29 |
dansmith | in the new world way we'll still end up calling the doubler? | 21:29 |
mriedem | it's a move operation where the source allocation isn't on the migration uuid | 21:29 |
mriedem | unlike live migrate, cold migrate/resize | 21:29 |
dansmith | can't we just stop doing the doubling across the board? | 21:29 |
dansmith | just delete it before we call claim_resources | 21:29 |
*** Apoorva_ has quit IRC | 21:29 | |
mriedem | for evac? | 21:29 |
dansmith | that should work for both old and new paths | 21:30 |
dansmith | yeah | 21:30 |
mriedem | well, unless evac fails on the dest | 21:30 |
mriedem | and the instance never moved | 21:30 |
dansmith | what does it matter? you're not going back | 21:30 |
*** Apoorva has joined #openstack-nova | 21:30 | |
dansmith | we've created the migration, the source node is going to nuke it when it wakes up, per the rules | 21:30 |
mriedem | not necessarily | 21:30 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L643 | 21:30 |
*** ociuhandu has joined #openstack-nova | 21:30 | |
mriedem | the source only cleans up if the move completed, or is in progress | 21:30 |
dansmith | um | 21:32 |
dansmith | the whole point of that robustification was to not do that, | 21:32 |
*** penick has quit IRC | 21:32 | |
dansmith | else we'll race with the operation finishing | 21:32 |
dansmith | I don't see where we're setting =accepted anymore actually | 21:32 |
mriedem | conductor | 21:32 |
mriedem | er api | 21:32 |
dansmith | I don't see it | 21:33 |
dansmith | ah I see | 21:33 |
mriedem | yeah so the api creates the migration record in 'accepted' status, | 21:33 |
dansmith | right, so that happens synchronously with the api call | 21:33 |
dansmith | right? | 21:33 |
*** gbarros has joined #openstack-nova | 21:33 | |
mriedem | if the evac is successful on the dest, the status goes to 'done' | 21:33 |
*** rodolof has quit IRC | 21:33 | |
*** rodolof has joined #openstack-nova | 21:33 | |
dansmith | so if the evac api call returns, you know it's never starting on the source again right? | 21:33 |
dansmith | we do that before we create the new allocation, | 21:34 |
dansmith | so if we have saved that in the db, | 21:34 |
dansmith | we know we can delete the old allocation without caring | 21:34 |
*** esberglu has joined #openstack-nova | 21:34 | |
dansmith | because at that point, the source host will never take it back | 21:34 |
mriedem | not necessarily :) | 21:34 |
mriedem | if conductor/scheduler fails to find a host, the migration status for the evac is marked 'error' | 21:34 |
*** tbachman has quit IRC | 21:34 | |
mriedem | and the instance is still on the source | 21:35 |
mriedem | with no allocation on the dest | 21:35 |
mriedem | https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L871 | 21:35 |
*** vladikr_ has quit IRC | 21:35 | |
mriedem | this is all related to that thing gibi and i were talking about before the ptg | 21:35 |
*** vladikr has quit IRC | 21:35 | |
mriedem | handling allocations for failed evacs | 21:35 |
dansmith | right and I was trying to say we shouldn't ever let it go back | 21:35 |
dansmith | so we should include error in the list of states we use in compute right? | 21:35 |
*** penick has joined #openstack-nova | 21:36 | |
*** vladikr_ has joined #openstack-nova | 21:36 | |
dansmith | because the recovery path should be re-rebuilding that instance | 21:36 |
mriedem | rebuilding the instance on the source won't recreate the allocations in placement | 21:36 |
mriedem | b/c rebuild skips the scheduler, which skips the claim | 21:37 |
dansmith | okay, I guess, yeah | 21:37 |
dansmith | regardless, | 21:37 |
mriedem | so 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 allocations | 21:37 |
dansmith | I 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 races | 21:37 |
*** vladikr has joined #openstack-nova | 21:38 | |
*** rodolof has quit IRC | 21:38 | |
*** rodolof has joined #openstack-nova | 21:38 | |
dansmith | the 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 possible | 21:38 |
mriedem | ok, 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 |
mriedem | all things i can't really sort out in my head right now with maya in my office reading her homework... | 21:40 |
dansmith | so, we don't clear the instance.host in evacuate for some reason | 21:40 |
*** thorst has joined #openstack-nova | 21:40 | |
mriedem | we just update it once evac is successful | 21:40 |
dansmith | if we did, it would make it easier to just let people retry with an evac again if it failed the first time | 21:41 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2841 | 21:41 |
dansmith | yeah | 21:41 |
mriedem | you can already retry an evac if the first one failed | 21:41 |
dansmith | not if thecompute host came backup | 21:41 |
dansmith | because we still have instance.host set and require it be down | 21:41 |
dansmith | right? | 21:42 |
dansmith | the source I mean | 21:42 |
mriedem | well, if we nulled out instance.host, shit gets all sorts of wonky because you then have to deal with local delete in the api bullshit | 21:42 |
mriedem | wonkified | 21:42 |
mriedem | i think i've overshot the original problem we were talking about that started this :) | 21:43 |
*** rodolof has quit IRC | 21:43 | |
*** rodolof has joined #openstack-nova | 21:43 | |
mriedem | the assertion that RT doesn't have to care about migration allocations during evac | 21:43 |
dansmith | so, in your case, | 21:43 |
dansmith | let's say we evac, fail, leave a doubled allocation today, | 21:43 |
dansmith | then evac again, what do we do, triple it? | 21:43 |
*** smatzek has quit IRC | 21:44 | |
mriedem | no, we cleanup the allocations for the failed dest host | 21:44 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2831 | 21:44 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2817 | 21:45 |
mriedem | those are for if the actual spawn failed, or the claim on dest failed | 21:45 |
mriedem | remember me freaking out on you and jay about this a couple of weeks ago? :) | 21:45 |
mriedem | so i think we've got that covered | 21:45 |
mriedem | the cleanup i mean | 21:45 |
mriedem | and if we failed to pick a host or claim in conductor, we don't have anything to cleanup anyway | 21:45 |
dansmith | I've lost sight of the thing you think is wrong in the new case | 21:46 |
mriedem | i'm not saying it's wrong, | 21:46 |
mriedem | but this code in the RT can pull migrations that are 'done' | 21:46 |
*** jmlowe has quit IRC | 21:47 | |
mriedem | and that db api query is not making a distinction on if the current node is the migration's source or dest | 21:47 |
dansmith | which would cause it to skip allocations at most, yes? | 21:47 |
mriedem | maybe we should just fix the db api filter to include 'done'? | 21:47 |
*** eharney has quit IRC | 21:47 | |
dansmith | until the source comes up and marks it finished or whatever | 21:47 |
mriedem | i think so... | 21:48 |
mriedem | well, that's why i asked for you to double check my thinking here | 21:48 |
*** rodolof has quit IRC | 21:48 | |
mriedem | because this is weird | 21:48 |
dansmith | but it's an instance on our host, so we would skip it for that reason anyway | 21:48 |
dansmith | mriedem: yeah, how'd that work out for you? :) | 21:48 |
mriedem | i feel better | 21:48 |
*** rodolof has joined #openstack-nova | 21:49 | |
dansmith | we 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 |
dansmith | but I'm not sure it's required | 21:49 |
dansmith | are you concerned about inbound or outbound migrations being included? | 21:50 |
dansmith | you said "source and dest" but didn't emphasize either, so I'm not sure which you're concerned about | 21:50 |
mriedem | yeah i'm unclear on which we care about here too | 21:53 |
mriedem | i would think source since migration allocations should only be for source | 21:53 |
*** gbarros has quit IRC | 21:53 | |
dansmith | source of an evacuation, coming back up early? | 21:53 |
*** jmlowe has joined #openstack-nova | 21:54 | |
dansmith | currently it'll still count those instances as its own and not delete their allocations until .host flips | 21:54 |
*** Yingxin has quit IRC | 21:55 | |
*** Yingxin has joined #openstack-nova | 21:56 | |
mriedem | ok so in https://review.openstack.org/#/c/506419/19/nova/compute/resource_tracker.py | 21:57 |
mriedem | if we assume we're on the dest node, because the source is dead during an evac | 21:58 |
*** rodolof has quit IRC | 21:58 | |
*** rodolof has joined #openstack-nova | 21:58 | |
mriedem | and the migration status is done | 21:58 |
dansmith | migration is in accepted early, | 21:58 |
dansmith | then done once it's on our node | 21:58 |
mriedem | migration_get_in_progress_by_host_and_node could return it and the migration.dest_compute would be our local RT | 21:59 |
dansmith | then we're not the dest node anymore because instance.host=$us | 21:59 |
dansmith | i.e. if this runs before evac completes, it's still in accepted state | 21:59 |
mriedem | correct, and the migration wouldn't be returned | 21:59 |
mriedem | from migration_get_in_progress_by_host_and_node | 21:59 |
mriedem | because: filter(~models.Migration.status.in_(['accepted', 'confirmed', | 22:00 |
*** ijw has quit IRC | 22:00 | |
*** ijw has joined #openstack-nova | 22:00 | |
mriedem | if the migration status is 'done', then we've already set instance.host=$us | 22:00 |
dansmith | yeah, so if it runs early, we might nuke the doubled allocation yeah? | 22:00 |
mriedem | well i'm looking in _remove_deleted_instances_allocations now | 22:01 |
dansmith | not sure how we're not doing that already today though | 22:01 |
mriedem | if it runs before instance.host is set to the dest node, the instance shouldn't be in tracked_instances | 22:01 |
dansmith | right | 22:02 |
mriedem | and we'd get to if instance.host != cn.host: | 22:02 |
dansmith | but we have an allocation already because of conductor | 22:02 |
mriedem | and we'd log | 22:02 |
mriedem | LOG.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 |
mriedem | and those allocatoins remaining on the source would get cleaned up if/when the source comes back up and deletes them | 22:03 |
mriedem | and if it doesn't come back up, we don't care | 22:03 |
*** rodolof has quit IRC | 22:03 | |
dansmith | ack yeah | 22:03 |
mriedem | if this runs after instance.host=$us, then we hit if consumer_uuid in known_instances: and we don't remove allocations | 22:03 |
*** jaypipes has quit IRC | 22:03 | |
*** rodolof has joined #openstack-nova | 22:04 | |
*** claudiub|2 has quit IRC | 22:04 | |
mriedem | so i think we're ok | 22:04 |
mriedem | and 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-progres | 22:05 |
dansmith | mriedem: so, we _are_ passing the functional tests that cover evac.. the only thing that might be missing is source compute coming back or something | 22:05 |
mriedem | i know we have some functional tests for ^ because they are asserting the source removes the allocations from the source after it's moved | 22:06 |
dansmith | I'm not really sure why accepted should be in there, tbh | 22:06 |
dansmith | since it's very much in progress | 22:06 |
mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1658 | 22:06 |
mriedem | yeah not sure about accepted being in there either | 22:07 |
mriedem | so ^ is the test that once you have evacuated to the dest and start up the source, the allocations are removed from the source | 22:07 |
dansmith | ah yeah okay | 22:07 |
mriedem | which is actually currently broken :) | 22:07 |
mriedem | https://review.openstack.org/#/c/510176/ | 22:08 |
mriedem | mayhap you'd like to +W that | 22:08 |
mriedem | our code to remove the source allocations on startup after evac is not working | 22:08 |
dansmith | so 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 |
dansmith | we should be caught by the vm_state==ERROR skip clause no? | 22:09 |
*** lpetrut_ has quit IRC | 22:09 | |
SamYaple | if 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 back | 22:10 |
*** lyan has quit IRC | 22:10 | |
*** baoli has quit IRC | 22:11 | |
dansmith | SamYaple: yeah, reset-state means you're cheating and other things are likely going to be out of sync | 22:11 |
SamYaple | dansmith: fair enough. im just saying what would typically get done in ops | 22:12 |
SamYaple | im not advocating for this | 22:12 |
dansmith | SamYaple: ack, yeah, I was thinking mriedem was saying there's a path out of the forest for this by a user | 22:12 |
*** tbachman has joined #openstack-nova | 22:12 | |
mriedem | dansmith: do we set the instance to error state on failed evac? | 22:12 |
dansmith | mriedem: if we didn't you couldn't call evac on it again | 22:13 |
dansmith | and you said we could | 22:13 |
mriedem | i don't see that we change the vm_state on evac | 22:13 |
dansmith | so we leave it as rebuilding? | 22:13 |
mriedem | no | 22: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 |
mriedem | i think it would be active | 22:13 |
mriedem | well, on a successful evac it would be active | 22:14 |
*** smatzek has joined #openstack-nova | 22:14 | |
mriedem | if it started as error and evac failed, it'd stay in error | 22:14 |
dansmith | oh task_state=rebuilding | 22:14 |
dansmith | so um | 22:14 |
mriedem | yar | 22:14 |
dansmith | that means I can re-evac while an evac is going? | 22:14 |
dansmith | that'd be fun | 22:14 |
mriedem | no | 22:14 |
mriedem | i think @check_instance_state defaults to not allow doing things if the task_state is set | 22:15 |
mriedem | and we set the task_state to rebuilding in the api once you've started an evac | 22:15 |
dansmith | does that decorator imply no task state if not specified? | 22:15 |
dansmith | okay | 22:15 |
mriedem | yeah it's weird and confusing to read | 22:15 |
mriedem | task_state=(None,) | 22:15 |
*** rodolof has quit IRC | 22:16 | |
mriedem | if (task_state is not None and | 22:16 |
mriedem | instance.task_state not in task_state): | 22:16 |
dansmith | okay | 22:16 |
mriedem | so unrelated, | 22:16 |
*** burt has quit IRC | 22:17 | |
mriedem | but now i realize that if your instance was stopped and it was successfully evacuated, it's now running | 22:17 |
mriedem | we don't stop it again like we do for resizing/cold migrating a stopped instance | 22:17 |
mriedem | which, as it turns out, was my first major bug fix in nova :) | 22:17 |
mriedem | remember?! | 22:18 |
dansmith | oh I remember | 22:18 |
dansmith | okay so nothing to change on this? | 22:20 |
dansmith | also 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 2015 | 22:22 |
mriedem | yeah i just +Wed | 22:22 |
mriedem | appears CI just exploded | 22:23 |
*** smatzek has quit IRC | 22:24 | |
*** edmondsw has joined #openstack-nova | 22:25 | |
dansmith | we _can_ do a migration-holding source allocation if you want, | 22:25 |
dansmith | but it's work for no good reason, IMHO, | 22:26 |
mriedem | yeah, that's why i said if/when we need to | 22:26 |
dansmith | okay | 22:26 |
mriedem | because 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 not | 22:26 |
dansmith | right | 22:27 |
*** tbachman has quit IRC | 22:27 | |
mriedem | and that would definitely be | 22:27 |
mriedem | say it with me now | 22:27 |
dansmith | I'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 once | 22:27 |
* dansmith doesn't know what he's supposed to say in chorus | 22:28 | |
mriedem | wonky | 22:28 |
dansmith | heh | 22:28 |
mriedem | it would be wonky | 22:28 |
*** edmondsw has quit IRC | 22:30 | |
*** felipemonteiro_ has quit IRC | 22:34 | |
*** felipemonteiro_ has joined #openstack-nova | 22:34 | |
mriedem | wtf, 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#L4867 | 22:35 |
*** priteau has joined #openstack-nova | 22:39 | |
dansmith | because migration.instance.system_metadata | 22:40 |
dansmith | although I'm sure we don't need that anymore | 22:40 |
mriedem | you know what else is super confusing, | 22:41 |
mriedem | we have a 'finished' migration status, | 22:41 |
mriedem | but it doesn't mean the migration is actually done, it means the instance is done being resized, but it's in verify_resize state | 22:41 |
mriedem | so it's not an end state :/ | 22:41 |
dansmith | yup | 22:42 |
mriedem | oh, an | 22:43 |
mriedem | *and | 22:43 |
mriedem | when reverting a resize, | 22:44 |
mriedem | we set the migration status to 'reverted' on the dest host before casting back to the source host to complete the revert | 22:44 |
*** priteau has quit IRC | 22:44 | |
mriedem | so technically the migration is still in progress at that point | 22:44 |
mriedem | i sure hope nothing like resource tracking depends on us not fucking that all up :) | 22:44 |
*** AlexeyAbashkin has joined #openstack-nova | 22:47 | |
*** tbachman has joined #openstack-nova | 22:48 | |
*** edmondsw has joined #openstack-nova | 22:48 | |
mriedem | check out this fun stuff http://paste.openstack.org/show/623387/ | 22:50 |
mriedem | creating rp inventory on the fly like a mad dog | 22:50 |
*** AlexeyAbashkin has quit IRC | 22:52 | |
*** jmlowe has quit IRC | 22:53 | |
*** edmondsw has quit IRC | 22:54 | |
*** penick has quit IRC | 22:57 | |
*** priteau has joined #openstack-nova | 23:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix migration_get_in_progress_by_host_and_node status list https://review.openstack.org/511351 | 23:05 |
*** priteau has quit IRC | 23:06 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax https://review.openstack.org/510244 | 23:10 |
*** hongbin has quit IRC | 23:11 | |
*** felipemonteiro_ has quit IRC | 23:12 | |
efried | Man Jenkins is a mess. | 23:12 |
*** masber has joined #openstack-nova | 23:14 | |
*** ttx has quit IRC | 23:20 | |
*** ttx has joined #openstack-nova | 23:20 | |
*** crushil has joined #openstack-nova | 23:22 | |
*** crushil has quit IRC | 23:27 | |
*** yamamoto has joined #openstack-nova | 23:28 | |
*** suresh12 has quit IRC | 23:30 | |
*** suresh12 has joined #openstack-nova | 23:31 | |
*** thorst has quit IRC | 23:33 | |
*** suresh12_ has joined #openstack-nova | 23:34 | |
mriedem | dansmith: i guess the 'accepted' filter is intentional https://review.openstack.org/#/c/244486/ | 23:35 |
*** suresh12 has quit IRC | 23:35 | |
*** suresh12_ has quit IRC | 23:35 | |
*** suresh12 has joined #openstack-nova | 23:37 | |
openstackgerrit | melanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume https://review.openstack.org/498983 | 23:38 |
*** suresh12 has quit IRC | 23:42 | |
openstackgerrit | melanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume https://review.openstack.org/498983 | 23:43 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add 'done' to migration_get_in_progress_by_host_and_node filter https://review.openstack.org/511351 | 23:47 |
*** mriedem has quit IRC | 23:48 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | 23:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 23:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 23:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159 | 23:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 23:52 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Make conductor pass and use host_lists https://review.openstack.org/511358 | 23:52 |
*** fragatina has quit IRC | 23:52 | |
*** itlinux has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!