| *** rajathagasthya has joined #openstack-nova | 00:00 | |
| *** acormier has joined #openstack-nova | 00:02 | |
| *** claudiub has quit IRC | 00:03 | |
| *** rtjure has joined #openstack-nova | 00:03 | |
| *** slaweq has joined #openstack-nova | 00:07 | |
| *** acormier has quit IRC | 00:08 | |
| *** jmlowe has quit IRC | 00:10 | |
| *** jmlowe has joined #openstack-nova | 00:10 | |
| *** dave-mccowan has joined #openstack-nova | 00:11 | |
| *** slaweq has quit IRC | 00:12 | |
| *** Swami has quit IRC | 00:14 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160 | 00:18 |
|---|---|---|
| *** hongbin has quit IRC | 00:18 | |
| *** acormier has joined #openstack-nova | 00:20 | |
| mriedem | dansmith: excellent | 00:20 |
| mriedem | i'll have to start digging in tomorrow | 00:21 |
| *** Apoorva_ has joined #openstack-nova | 00:22 | |
| *** READ10 has quit IRC | 00:22 | |
| dansmith | mriedem: cool, I want to get this merged while I have context and stuff so I can get back to migration uuid stuff | 00:23 |
| mriedem | yeah | 00:23 |
| dansmith | mriedem: also, I think the live migration job is fairly fscked | 00:23 |
| mriedem | yup, see me bring that up earlier? | 00:23 |
| dansmith | oh no, sorry | 00:23 |
| mriedem | top fail right now http://status.openstack.org/elastic-recheck/#1718295 | 00:23 |
| dansmith | cool, sorry I didn't see it | 00:23 |
| mriedem | i'll send something to the dev list | 00:24 |
| *** acormier has quit IRC | 00:25 | |
| *** Apoorva has quit IRC | 00:26 | |
| *** Apoorva_ has quit IRC | 00:26 | |
| mriedem | oh shit | 00:26 |
| *** zhenq has joined #openstack-nova | 00:26 | |
| mriedem | https://review.openstack.org/#/c/501224/ | 00:26 |
| mriedem | i bet $7.75 it's that | 00:27 |
| *** zhenq has quit IRC | 00:27 | |
| dansmith | hoo boy | 00:27 |
| *** ijw has joined #openstack-nova | 00:29 | |
| mriedem | revert in progress | 00:30 |
| mriedem | although (1) gerrit is hung and (2) no devstack cores around | 00:30 |
| mriedem | https://review.openstack.org/#/c/505446/ | 00:30 |
| mriedem | nice, new gerrit includes the owner and reviewers from the change that you revert | 00:31 |
| *** ijw has quit IRC | 00:32 | |
| *** ijw has joined #openstack-nova | 00:32 | |
| *** thorst has joined #openstack-nova | 00:34 | |
| *** TuanLA has joined #openstack-nova | 00:37 | |
| mriedem | and with that, i'm going to slurp night time cold medicine tea and pass out | 00:39 |
| *** mriedem has quit IRC | 00:39 | |
| *** acormier has joined #openstack-nova | 00:39 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/490722 | 00:41 |
| *** jobewan has joined #openstack-nova | 00:42 | |
| *** zhenq has joined #openstack-nova | 00:43 | |
| *** zhenq has quit IRC | 00:43 | |
| *** maciejjo1 has joined #openstack-nova | 00:43 | |
| *** maciejjozefczyk has quit IRC | 00:45 | |
| *** Shunli has joined #openstack-nova | 00:45 | |
| *** dave-mcc_ has joined #openstack-nova | 00:46 | |
| *** yangyapeng has quit IRC | 00:48 | |
| *** dave-mccowan has quit IRC | 00:48 | |
| *** yangyapeng has joined #openstack-nova | 00:48 | |
| *** zhenq has joined #openstack-nova | 00:48 | |
| *** zhenq has quit IRC | 00:48 | |
| *** acormier has quit IRC | 00:48 | |
| *** catintheroof has quit IRC | 00:49 | |
| *** ijw has quit IRC | 00:49 | |
| *** rajathagasthya has quit IRC | 00:49 | |
| *** catintheroof has joined #openstack-nova | 00:49 | |
| *** catintheroof has quit IRC | 00:50 | |
| *** dpawlik has joined #openstack-nova | 00:50 | |
| *** bigdogstl has joined #openstack-nova | 00:50 | |
| *** danpawlik has quit IRC | 00:51 | |
| *** maciejjo1 has quit IRC | 00:51 | |
| *** maciejjozefczyk has joined #openstack-nova | 00:51 | |
| *** bigdogstl has quit IRC | 00:52 | |
| *** yangyapeng has quit IRC | 00:52 | |
| *** bigdogstl has joined #openstack-nova | 00:53 | |
| *** zhurong has joined #openstack-nova | 00:54 | |
| *** dave-mcc_ has quit IRC | 00:54 | |
| *** jaypipes has quit IRC | 00:54 | |
| *** zhenq has joined #openstack-nova | 00:55 | |
| *** nicolasbock has quit IRC | 00:55 | |
| *** zhenq has quit IRC | 00:55 | |
| *** zhenq has joined #openstack-nova | 00:56 | |
| *** zhenq has quit IRC | 00:56 | |
| *** ijw has joined #openstack-nova | 00:59 | |
| *** bigdogstl has quit IRC | 01:01 | |
| *** acormier has joined #openstack-nova | 01:03 | |
| *** ijw has quit IRC | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:05 | |
| *** jjahns has joined #openstack-nova | 01:06 | |
| *** slaweq has joined #openstack-nova | 01:08 | |
| openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Fix getting a wrong guest object https://review.openstack.org/498478 | 01:10 |
| *** owalsh has quit IRC | 01:11 | |
| *** acormier has quit IRC | 01:12 | |
| *** owalsh has joined #openstack-nova | 01:12 | |
| *** bigdogstl has joined #openstack-nova | 01:12 | |
| *** kevi9132 has quit IRC | 01:12 | |
| *** kevi9132 has joined #openstack-nova | 01:13 | |
| *** slaweq has quit IRC | 01:14 | |
| *** liusheng has quit IRC | 01:15 | |
| *** jichen has joined #openstack-nova | 01:15 | |
| *** gyee has quit IRC | 01:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 01:15 |
| *** chyka has quit IRC | 01:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 01:15 |
| jichen | alex_xu: hi , have time to take a look at https://review.openstack.org/#/c/485121/ ? | 01:16 |
| *** acormier has joined #openstack-nova | 01:17 | |
| *** bigdogstl has quit IRC | 01:17 | |
| *** gbarros has joined #openstack-nova | 01:17 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 01:19 |
| *** baoli has joined #openstack-nova | 01:20 | |
| *** acormier has quit IRC | 01:21 | |
| *** jobewan is now known as jobewan_away | 01:22 | |
| *** acormier has joined #openstack-nova | 01:26 | |
| *** slaweq has joined #openstack-nova | 01:30 | |
| *** bigdogstl has joined #openstack-nova | 01:30 | |
| *** acormier has quit IRC | 01:30 | |
| *** jobewan_away is now known as jobewan | 01:32 | |
| *** Kevin_Zheng has joined #openstack-nova | 01:33 | |
| *** zhenq has joined #openstack-nova | 01:33 | |
| *** zhenq has quit IRC | 01:33 | |
| *** zhenq has joined #openstack-nova | 01:35 | |
| *** bigdogstl has quit IRC | 01:35 | |
| *** zhenq has quit IRC | 01:35 | |
| *** slaweq has quit IRC | 01:36 | |
| *** Yikun has joined #openstack-nova | 01:38 | |
| *** acormier has joined #openstack-nova | 01:40 | |
| *** suresh12_ has quit IRC | 01:41 | |
| *** kevi9132_ has joined #openstack-nova | 01:41 | |
| *** jobewan has quit IRC | 01:44 | |
| *** kevi9132 has quit IRC | 01:45 | |
| *** yamahata has quit IRC | 01:46 | |
| *** suresh12 has joined #openstack-nova | 01:47 | |
| *** gbarros has quit IRC | 01:49 | |
| *** acormier has quit IRC | 01:50 | |
| *** mingyu_ has joined #openstack-nova | 01:51 | |
| *** suresh12 has quit IRC | 01:51 | |
| *** mingyu has quit IRC | 01:53 | |
| *** itlinux has joined #openstack-nova | 01:54 | |
| *** slaweq has joined #openstack-nova | 01:54 | |
| *** acormier has joined #openstack-nova | 01:55 | |
| *** Yikun has quit IRC | 01:55 | |
| *** hongbin has joined #openstack-nova | 01:55 | |
| *** litao__ has joined #openstack-nova | 01:56 | |
| *** yangyapeng has joined #openstack-nova | 01:58 | |
| *** slaweq has quit IRC | 01:58 | |
| *** tbachman has quit IRC | 01:58 | |
| *** kukacz has quit IRC | 02:00 | |
| *** kukacz_ has joined #openstack-nova | 02:01 | |
| *** Yikun has joined #openstack-nova | 02:01 | |
| *** Yikun has quit IRC | 02:02 | |
| *** zhenq has joined #openstack-nova | 02:03 | |
| *** Yikun has joined #openstack-nova | 02:03 | |
| *** zhenq has quit IRC | 02:03 | |
| *** acormier has quit IRC | 02:03 | |
| *** tbachman has joined #openstack-nova | 02:05 | |
| *** itlinux has quit IRC | 02:08 | |
| openstackgerrit | octopuszhang proposed openstack/nova master: remove re-auth logic for ironic client wrapper https://review.openstack.org/502382 | 02:08 |
| *** bigdogstl has joined #openstack-nova | 02:09 | |
| *** liusheng has joined #openstack-nova | 02:09 | |
| *** tinwood has quit IRC | 02:10 | |
| *** tinwood has joined #openstack-nova | 02:12 | |
| *** slaweq has joined #openstack-nova | 02:15 | |
| *** acormier has joined #openstack-nova | 02:18 | |
| *** dave-mccowan has joined #openstack-nova | 02:19 | |
| *** Yikun has quit IRC | 02:21 | |
| *** slaweq has quit IRC | 02:21 | |
| *** dave-mcc_ has joined #openstack-nova | 02:21 | |
| *** acormier has quit IRC | 02:22 | |
| *** dave-mccowan has quit IRC | 02:23 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471 | 02:23 |
| *** Sukhdev has quit IRC | 02:24 | |
| *** Yikun has joined #openstack-nova | 02:26 | |
| *** lbragstad has joined #openstack-nova | 02:27 | |
| *** bigdogstl has quit IRC | 02:27 | |
| *** acormier has joined #openstack-nova | 02:29 | |
| *** rajathagasthya has joined #openstack-nova | 02:29 | |
| *** chyka has joined #openstack-nova | 02:29 | |
| *** chyka has quit IRC | 02:31 | |
| *** chyka has joined #openstack-nova | 02:31 | |
| *** yangyapeng has quit IRC | 02:33 | |
| *** coreywright has quit IRC | 02:33 | |
| *** chyka has quit IRC | 02:35 | |
| *** bigdogstl has joined #openstack-nova | 02:36 | |
| *** bigdogstl has quit IRC | 02:36 | |
| *** acormier has quit IRC | 02:36 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | 02:37 |
| *** dave-mcc_ has quit IRC | 02:38 | |
| *** rajathagasthya has quit IRC | 02:38 | |
| *** zhenq has joined #openstack-nova | 02:38 | |
| *** zhenq has quit IRC | 02:38 | |
| *** rajathagasthya has joined #openstack-nova | 02:39 | |
| *** chyka has joined #openstack-nova | 02:39 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 02:41 |
| *** takashin has joined #openstack-nova | 02:43 | |
| *** rajathagasthya has quit IRC | 02:43 | |
| *** jmlowe has quit IRC | 02:43 | |
| *** acormier has joined #openstack-nova | 02:46 | |
| *** bigdogstl has joined #openstack-nova | 02:47 | |
| *** acormier has quit IRC | 02:50 | |
| *** bigdogstl has quit IRC | 02:51 | |
| *** coreywright has joined #openstack-nova | 02:52 | |
| *** slaweq has joined #openstack-nova | 02:53 | |
| *** prashkre has joined #openstack-nova | 02:54 | |
| *** zhenq has joined #openstack-nova | 02:56 | |
| *** zhenq has quit IRC | 02:56 | |
| *** slaweq has quit IRC | 02:59 | |
| *** bigdogstl has joined #openstack-nova | 02:59 | |
| *** acormier has joined #openstack-nova | 03:02 | |
| *** bigdogstl has quit IRC | 03:04 | |
| *** vladikr has quit IRC | 03:04 | |
| *** vladikr has joined #openstack-nova | 03:05 | |
| *** acormier has quit IRC | 03:07 | |
| *** crushil has joined #openstack-nova | 03:10 | |
| *** crushil has quit IRC | 03:13 | |
| *** sridharg has joined #openstack-nova | 03:15 | |
| *** crushil has joined #openstack-nova | 03:16 | |
| jichen | alex_xu: shuai ge, have time to take a look at https://review.openstack.org/#/c/485121/ ? | 03:22 |
| *** rajathagasthya has joined #openstack-nova | 03:23 | |
| *** thorst has quit IRC | 03:30 | |
| *** baoli has quit IRC | 03:32 | |
| *** baoli has joined #openstack-nova | 03:33 | |
| alex_xu | jichen: yea, I will check that after the meeting... | 03:36 |
| jichen | alex_xu: ok, perfect, thanks | 03:36 |
| jichen | by the way, which meeting? nova or nova-api? | 03:36 |
| *** slaweq has joined #openstack-nova | 03:37 | |
| alex_xu | jichen: intel internal meeting... | 03:37 |
| jichen | alex_xu: ok, thanks a lot | 03:38 |
| alex_xu | jichen: np | 03:38 |
| *** bigdogstl has joined #openstack-nova | 03:39 | |
| *** mfisch has quit IRC | 03:41 | |
| *** mfisch has joined #openstack-nova | 03:42 | |
| *** mfisch has quit IRC | 03:42 | |
| *** slaweq has quit IRC | 03:42 | |
| *** mfisch has joined #openstack-nova | 03:42 | |
| *** mfisch has quit IRC | 03:42 | |
| *** mfisch has joined #openstack-nova | 03:42 | |
| *** bigdogstl has quit IRC | 03:43 | |
| *** crushil has quit IRC | 03:45 | |
| *** bigdogstl has joined #openstack-nova | 03:47 | |
| *** suresh12 has joined #openstack-nova | 03:48 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/490722 | 03:48 |
| *** mfisch has quit IRC | 03:49 | |
| *** mfisch has joined #openstack-nova | 03:50 | |
| *** slaweq has joined #openstack-nova | 03:50 | |
| *** mfisch has quit IRC | 03:50 | |
| *** mfisch has joined #openstack-nova | 03:50 | |
| *** slaweq has quit IRC | 03:54 | |
| *** vladikr has quit IRC | 03:54 | |
| *** acormier has joined #openstack-nova | 03:55 | |
| *** vladikr has joined #openstack-nova | 03:55 | |
| *** bigdogstl has quit IRC | 03:55 | |
| *** acormier has quit IRC | 03:59 | |
| *** Sree has joined #openstack-nova | 04:00 | |
| *** huanxie has joined #openstack-nova | 04:01 | |
| *** bigdogstl has joined #openstack-nova | 04:01 | |
| *** slaweq has joined #openstack-nova | 04:02 | |
| *** vladikr has quit IRC | 04:03 | |
| *** mfisch` has joined #openstack-nova | 04:03 | |
| *** vladikr has joined #openstack-nova | 04:03 | |
| *** hongbin has quit IRC | 04:04 | |
| *** mfisch has quit IRC | 04:06 | |
| *** slaweq has quit IRC | 04:07 | |
| *** baoli has quit IRC | 04:08 | |
| *** vladikr has quit IRC | 04:09 | |
| *** diga has joined #openstack-nova | 04:09 | |
| *** vladikr has joined #openstack-nova | 04:09 | |
| *** rajathagasthya has quit IRC | 04:11 | |
| *** slaweq has joined #openstack-nova | 04:15 | |
| *** crushil has joined #openstack-nova | 04:18 | |
| *** huanxie has quit IRC | 04:19 | |
| *** slaweq has quit IRC | 04:21 | |
| *** gouthamr has quit IRC | 04:21 | |
| *** markvoelker has quit IRC | 04:22 | |
| *** markvoelker has joined #openstack-nova | 04:22 | |
| *** lbragstad has quit IRC | 04:23 | |
| *** markvoelker has quit IRC | 04:27 | |
| *** cfriesen has quit IRC | 04:29 | |
| *** tbh_ has joined #openstack-nova | 04:30 | |
| *** thorst has joined #openstack-nova | 04:31 | |
| *** zhenq has joined #openstack-nova | 04:33 | |
| *** zhenq has quit IRC | 04:33 | |
| *** thorst has quit IRC | 04:35 | |
| *** zhenq has joined #openstack-nova | 04:36 | |
| *** zhenq has quit IRC | 04:36 | |
| *** crushil has quit IRC | 04:39 | |
| *** zhenq has joined #openstack-nova | 04:39 | |
| *** lbragstad has joined #openstack-nova | 04:39 | |
| *** links has joined #openstack-nova | 04:39 | |
| *** zhenq has quit IRC | 04:39 | |
| *** links has quit IRC | 04:41 | |
| *** shan has joined #openstack-nova | 04:43 | |
| *** mingyu_ has quit IRC | 04:44 | |
| *** sbezverk has joined #openstack-nova | 04:45 | |
| *** bigdogstl has quit IRC | 04:51 | |
| *** kevi9132_ has quit IRC | 04:52 | |
| *** zhurong has quit IRC | 04:53 | |
| *** mdnadeem has joined #openstack-nova | 04:53 | |
| *** Tom has joined #openstack-nova | 04:56 | |
| *** takashin has quit IRC | 04:57 | |
| *** bigdogstl has joined #openstack-nova | 04:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/newton: Updated from global requirements https://review.openstack.org/373293 | 04:58 |
| *** links has joined #openstack-nova | 04:59 | |
| *** bigdogstl has quit IRC | 04:59 | |
| *** links has quit IRC | 05:00 | |
| *** moshele has joined #openstack-nova | 05:00 | |
| *** jaosorior has joined #openstack-nova | 05:01 | |
| *** acormier has joined #openstack-nova | 05:03 | |
| *** acormier has quit IRC | 05:07 | |
| *** avolkov has joined #openstack-nova | 05:12 | |
| *** lbragstad has quit IRC | 05:13 | |
| *** links has joined #openstack-nova | 05:16 | |
| *** links has quit IRC | 05:17 | |
| *** rajathagasthya has joined #openstack-nova | 05:17 | |
| *** udesale has joined #openstack-nova | 05:23 | |
| *** edand has joined #openstack-nova | 05:25 | |
| *** rajathagasthya has quit IRC | 05:29 | |
| *** suresh12 has quit IRC | 05:30 | |
| *** thorst has joined #openstack-nova | 05:31 | |
| *** gcb has joined #openstack-nova | 05:33 | |
| *** claudiub has joined #openstack-nova | 05:35 | |
| *** thorst has quit IRC | 05:36 | |
| *** psachin has joined #openstack-nova | 05:38 | |
| *** ratailor has joined #openstack-nova | 05:38 | |
| *** nmathew has joined #openstack-nova | 05:45 | |
| *** moshele has quit IRC | 05:50 | |
| *** chyka has quit IRC | 05:50 | |
| *** josecastroleon has quit IRC | 05:51 | |
| *** zhenq has joined #openstack-nova | 05:54 | |
| *** zhenq has quit IRC | 05:55 | |
| *** zhenq has joined #openstack-nova | 05:57 | |
| *** trungnv has quit IRC | 05:57 | |
| *** TuanLA has quit IRC | 05:57 | |
| *** hieulq has quit IRC | 05:57 | |
| *** zhenq has quit IRC | 05:57 | |
| *** trungnv has joined #openstack-nova | 05:57 | |
| *** TuanLA has joined #openstack-nova | 05:57 | |
| *** hieulq has joined #openstack-nova | 05:57 | |
| *** slaweq has joined #openstack-nova | 05:58 | |
| *** takashin has joined #openstack-nova | 06:01 | |
| *** slaweq has quit IRC | 06:04 | |
| *** Tom has quit IRC | 06:04 | |
| *** diga has quit IRC | 06:08 | |
| *** manasm has joined #openstack-nova | 06:08 | |
| *** jichen has quit IRC | 06:12 | |
| *** Oku_OS-away is now known as Oku_OS | 06:13 | |
| *** udesale has quit IRC | 06:16 | |
| *** pratapagoutham has joined #openstack-nova | 06:18 | |
| *** abalutoiu_ has joined #openstack-nova | 06:18 | |
| *** slaweq has joined #openstack-nova | 06:20 | |
| *** sahid has joined #openstack-nova | 06:22 | |
| *** markvoelker has joined #openstack-nova | 06:23 | |
| *** jichen has joined #openstack-nova | 06:23 | |
| *** lajoskatona has joined #openstack-nova | 06:23 | |
| *** slaweq has quit IRC | 06:25 | |
| *** slaweq has joined #openstack-nova | 06:30 | |
| *** thorst has joined #openstack-nova | 06:32 | |
| *** slaweq_ has joined #openstack-nova | 06:34 | |
| *** thorst has quit IRC | 06:36 | |
| *** slaweq has quit IRC | 06:37 | |
| *** yingjun has joined #openstack-nova | 06:43 | |
| *** rcernin has joined #openstack-nova | 06:43 | |
| *** yangyapeng has joined #openstack-nova | 06:44 | |
| *** zhurong has joined #openstack-nova | 06:45 | |
| *** abalutoiu_ has quit IRC | 06:47 | |
| *** andreas_s has joined #openstack-nova | 06:48 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Add trait support in the allocation candidates API https://review.openstack.org/497713 | 06:50 |
| *** slaweq has joined #openstack-nova | 06:52 | |
| *** kevi9132 has joined #openstack-nova | 06:53 | |
| *** rajathagasthya has joined #openstack-nova | 06:54 | |
| *** udesale has joined #openstack-nova | 06:56 | |
| *** markvoelker has quit IRC | 06:57 | |
| *** slaweq has quit IRC | 06:57 | |
| *** Tom has joined #openstack-nova | 06:58 | |
| *** tbh_ has quit IRC | 06:58 | |
| *** kevi9132 has quit IRC | 06:58 | |
| pratapagoutham | #openstack-tempest | 06:59 |
| *** pcaruana has joined #openstack-nova | 07:02 | |
| *** openstackgerrit has quit IRC | 07:02 | |
| *** slaweq has joined #openstack-nova | 07:03 | |
| *** tesseract has joined #openstack-nova | 07:05 | |
| *** sshwarts has joined #openstack-nova | 07:07 | |
| *** slaweq has quit IRC | 07:09 | |
| *** rajathagasthya has quit IRC | 07:10 | |
| *** gszasz has joined #openstack-nova | 07:10 | |
| *** bigdogstl has joined #openstack-nova | 07:11 | |
| *** jichen has quit IRC | 07:12 | |
| *** yangyapeng has quit IRC | 07:12 | |
| *** yangyapeng has joined #openstack-nova | 07:13 | |
| *** yangyapeng has quit IRC | 07:13 | |
| *** yangyapeng has joined #openstack-nova | 07:14 | |
| *** yangyapeng has quit IRC | 07:15 | |
| *** armax has quit IRC | 07:15 | |
| *** yangyapeng has joined #openstack-nova | 07:15 | |
| *** yangyapeng has quit IRC | 07:15 | |
| *** sgordon has quit IRC | 07:16 | |
| *** yangyapeng has joined #openstack-nova | 07:16 | |
| *** yangyapeng has quit IRC | 07:17 | |
| *** yangyapeng has joined #openstack-nova | 07:17 | |
| *** yangyapeng has quit IRC | 07:17 | |
| *** bigdogstl has quit IRC | 07:17 | |
| *** yangyapeng has joined #openstack-nova | 07:18 | |
| *** yangyapeng has quit IRC | 07:18 | |
| *** yangyapeng has joined #openstack-nova | 07:19 | |
| *** yangyapeng has quit IRC | 07:19 | |
| *** yangyapeng has joined #openstack-nova | 07:20 | |
| *** yangyapeng has quit IRC | 07:21 | |
| *** sgordon has joined #openstack-nova | 07:21 | |
| *** yangyapeng has joined #openstack-nova | 07:21 | |
| *** yamahata has joined #openstack-nova | 07:25 | |
| *** vladikr has quit IRC | 07:26 | |
| *** vladikr has joined #openstack-nova | 07:27 | |
| *** ralonsoh has joined #openstack-nova | 07:28 | |
| *** edand has quit IRC | 07:28 | |
| *** zhenq has joined #openstack-nova | 07:30 | |
| *** zhenq has quit IRC | 07:31 | |
| *** vladikr has quit IRC | 07:32 | |
| *** vladikr has joined #openstack-nova | 07:33 | |
| *** thorst has joined #openstack-nova | 07:33 | |
| *** huanxie has joined #openstack-nova | 07:34 | |
| *** thorst has quit IRC | 07:39 | |
| *** tetsuro has quit IRC | 07:40 | |
| *** zhenq has joined #openstack-nova | 07:40 | |
| *** tetsuro has joined #openstack-nova | 07:40 | |
| *** zhenq has quit IRC | 07:40 | |
| *** tetsuro has left #openstack-nova | 07:41 | |
| *** edand has joined #openstack-nova | 07:44 | |
| *** mingyu has joined #openstack-nova | 07:45 | |
| *** jpena|off has joined #openstack-nova | 07:45 | |
| *** jpena|off is now known as jpena | 07:45 | |
| *** vladikr has quit IRC | 07:46 | |
| *** vladikr has joined #openstack-nova | 07:47 | |
| *** zhurong has quit IRC | 07:48 | |
| *** ygl has joined #openstack-nova | 07:48 | |
| *** mingyu has quit IRC | 07:49 | |
| *** yangyapeng has quit IRC | 07:50 | |
| *** yangyapeng has joined #openstack-nova | 07:51 | |
| *** markvoelker has joined #openstack-nova | 07:54 | |
| *** vladikr has quit IRC | 07:54 | |
| *** jaosorior has quit IRC | 07:55 | |
| *** vladikr has joined #openstack-nova | 07:55 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** alexchadin has joined #openstack-nova | 08:02 | |
| *** rmart04 has joined #openstack-nova | 08:02 | |
| *** phuongnh has quit IRC | 08:02 | |
| *** phuongnh has joined #openstack-nova | 08:02 | |
| *** vladikr has quit IRC | 08:05 | |
| *** vladikr has joined #openstack-nova | 08:05 | |
| *** abalutoiu has joined #openstack-nova | 08:05 | |
| *** yangyapeng has quit IRC | 08:05 | |
| *** yangyapeng has joined #openstack-nova | 08:06 | |
| *** yamahata has quit IRC | 08:06 | |
| *** slaweq has joined #openstack-nova | 08:09 | |
| *** yangyapeng has quit IRC | 08:10 | |
| *** jichen has joined #openstack-nova | 08:10 | |
| *** zhenq has joined #openstack-nova | 08:11 | |
| *** zhenq has quit IRC | 08:12 | |
| *** Yikun__ has joined #openstack-nova | 08:12 | |
| *** slaweq has quit IRC | 08:13 | |
| *** alexchadin has quit IRC | 08:18 | |
| *** zhenq has joined #openstack-nova | 08:18 | |
| *** sridharg has quit IRC | 08:18 | |
| *** zhenq has quit IRC | 08:19 | |
| *** jaosorior has joined #openstack-nova | 08:19 | |
| *** slaweq has joined #openstack-nova | 08:19 | |
| *** alexchadin has joined #openstack-nova | 08:23 | |
| *** slaweq has quit IRC | 08:24 | |
| *** yangyapeng has joined #openstack-nova | 08:25 | |
| *** markvoelker has quit IRC | 08:28 | |
| *** openstackgerrit has joined #openstack-nova | 08:28 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | 08:28 |
| *** slaweq has joined #openstack-nova | 08:29 | |
| *** mvk has quit IRC | 08:30 | |
| *** yangyapeng has quit IRC | 08:30 | |
| *** yamamoto_ has quit IRC | 08:33 | |
| *** slaweq has quit IRC | 08:34 | |
| *** thorst has joined #openstack-nova | 08:35 | |
| *** avolkov has quit IRC | 08:36 | |
| *** avolkov has joined #openstack-nova | 08:36 | |
| *** prashkre_ has joined #openstack-nova | 08:37 | |
| *** prashkre has quit IRC | 08:37 | |
| *** yamamoto has joined #openstack-nova | 08:39 | |
| *** slaweq has joined #openstack-nova | 08:39 | |
| *** zhurong has joined #openstack-nova | 08:44 | |
| openstackgerrit | Merged openstack/nova master: Enable custom certificates for keystone communication https://review.openstack.org/485121 | 08:45 |
| *** slaweq has quit IRC | 08:45 | |
| *** prashkre__ has joined #openstack-nova | 08:47 | |
| *** prashkre_ has quit IRC | 08:47 | |
| *** derekh has joined #openstack-nova | 08:48 | |
| *** abalutoiu has quit IRC | 08:49 | |
| *** yamamoto has quit IRC | 08:52 | |
| openstackgerrit | jichenjc proposed openstack/nova stable/pike: Enable custom certificates for keystone communication https://review.openstack.org/505546 | 08:54 |
| *** yamamoto has joined #openstack-nova | 08:54 | |
| *** mvk has joined #openstack-nova | 09:00 | |
| *** slaweq_ has quit IRC | 09:00 | |
| *** slaweq has joined #openstack-nova | 09:01 | |
| *** pratapagoutham has quit IRC | 09:01 | |
| *** slaweq_ has joined #openstack-nova | 09:02 | |
| *** slaweq has quit IRC | 09:02 | |
| *** slaweq has joined #openstack-nova | 09:02 | |
| *** thorst has quit IRC | 09:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | 09:04 |
| *** slaweq has quit IRC | 09:06 | |
| *** slaweq has joined #openstack-nova | 09:12 | |
| *** bigdogstl has joined #openstack-nova | 09:14 | |
| *** sapcc-bot has quit IRC | 09:14 | |
| *** tpatzig_ has joined #openstack-nova | 09:15 | |
| *** carthaca_ has joined #openstack-nova | 09:15 | |
| *** seife_ has joined #openstack-nova | 09:15 | |
| *** dgonzalez_ has joined #openstack-nova | 09:15 | |
| *** david_1 has joined #openstack-nova | 09:15 | |
| *** phuongnh has quit IRC | 09:15 | |
| *** sapcc-bot has joined #openstack-nova | 09:15 | |
| *** gszasz has quit IRC | 09:16 | |
| *** yassine has quit IRC | 09:16 | |
| *** phuongnh has joined #openstack-nova | 09:16 | |
| *** dgonzalez_ has quit IRC | 09:17 | |
| *** carthaca_ has quit IRC | 09:17 | |
| *** seife_ has quit IRC | 09:17 | |
| *** tpatzig_ has quit IRC | 09:17 | |
| *** david_1 has quit IRC | 09:17 | |
| *** abhishekk has joined #openstack-nova | 09:17 | |
| *** zsli_ has joined #openstack-nova | 09:17 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485 | 09:17 |
| abhishekk | sean-k-mooney: hi you around? | 09:18 |
| *** bigdogstl has quit IRC | 09:18 | |
| *** slaweq has quit IRC | 09:18 | |
| *** Shunli has quit IRC | 09:21 | |
| *** prashkre__ has quit IRC | 09:24 | |
| *** Tahvok has quit IRC | 09:24 | |
| *** markvoelker has joined #openstack-nova | 09:25 | |
| *** bigdogstl has joined #openstack-nova | 09:26 | |
| *** Tahvok has joined #openstack-nova | 09:26 | |
| *** sambetts|afk is now known as sambetts | 09:28 | |
| *** bigdogstl has quit IRC | 09:30 | |
| *** zsli_ has quit IRC | 09:31 | |
| *** huanxie has quit IRC | 09:31 | |
| *** slaweq has joined #openstack-nova | 09:34 | |
| *** slaweq has quit IRC | 09:39 | |
| *** ygl has quit IRC | 09:39 | |
| *** slaweq has joined #openstack-nova | 09:44 | |
| *** slaweq has quit IRC | 09:50 | |
| *** dtantsur|afk is now known as dtantsur | 09:50 | |
| *** zhenq has joined #openstack-nova | 09:51 | |
| *** zhenq has quit IRC | 09:51 | |
| *** yingjun has quit IRC | 09:52 | |
| *** gszasz has joined #openstack-nova | 09:55 | |
| *** karthiks has quit IRC | 09:56 | |
| *** markvoelker has quit IRC | 09:57 | |
| *** ioggstream has joined #openstack-nova | 09:58 | |
| *** udesale has quit IRC | 10:01 | |
| *** claudiub|2 has joined #openstack-nova | 10:01 | |
| *** toabctl has quit IRC | 10:01 | |
| *** claudiub has quit IRC | 10:04 | |
| *** alexchadin has quit IRC | 10:06 | |
| *** trungnv has quit IRC | 10:12 | |
| *** jichen has quit IRC | 10:13 | |
| *** sdague has joined #openstack-nova | 10:14 | |
| *** zhurong has quit IRC | 10:22 | |
| *** Tom has quit IRC | 10:24 | |
| *** Sree has quit IRC | 10:25 | |
| *** bigdogstl has joined #openstack-nova | 10:27 | |
| *** phuongnh has quit IRC | 10:28 | |
| *** Yikun__ has quit IRC | 10:30 | |
| *** bigdogstl has quit IRC | 10:31 | |
| *** TuanLA has quit IRC | 10:34 | |
| *** huanxie has joined #openstack-nova | 10:36 | |
| *** huanxie has quit IRC | 10:40 | |
| *** nicolasbock has joined #openstack-nova | 10:42 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: Factor out relocate_vm() https://review.openstack.org/270115 | 10:43 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | 10:43 |
| *** yamamoto has quit IRC | 10:46 | |
| *** nicolasbock has quit IRC | 10:46 | |
| *** vladikr has quit IRC | 10:47 | |
| *** vladikr has joined #openstack-nova | 10:47 | |
| *** slaweq has joined #openstack-nova | 10:48 | |
| *** yamamoto has joined #openstack-nova | 10:51 | |
| *** edmondsw has joined #openstack-nova | 10:52 | |
| *** sridharg has joined #openstack-nova | 10:52 | |
| *** vladikr has quit IRC | 10:53 | |
| *** vladikr has joined #openstack-nova | 10:53 | |
| *** slaweq has quit IRC | 10:55 | |
| *** markvoelker has joined #openstack-nova | 10:55 | |
| *** kevi9132 has joined #openstack-nova | 10:55 | |
| *** edmondsw has quit IRC | 10:56 | |
| *** cdent has joined #openstack-nova | 10:56 | |
| *** sshwarts has quit IRC | 10:58 | |
| *** thorst has joined #openstack-nova | 10:59 | |
| *** kevi9132 has quit IRC | 10:59 | |
| *** nicolasbock has joined #openstack-nova | 10:59 | |
| *** edand has quit IRC | 11:00 | |
| openstackgerrit | garyk proposed openstack/nova master: VMware: ensure that live migration attaches to correct interface https://review.openstack.org/478797 | 11:01 |
| *** zhenq has joined #openstack-nova | 11:01 | |
| *** zhenq has quit IRC | 11:02 | |
| *** thorst has quit IRC | 11:04 | |
| *** lajoskatona has quit IRC | 11:06 | |
| *** karthiks has joined #openstack-nova | 11:06 | |
| migi | hi, I am facing issue with the rename of branch stable/liberty to liberty-eol tag. The liberty-eol contains tox.ini with wrong pointer to the upper-constraints.txt which I want to fix, but I do not know how to sent this for review cause liberty-eol it's tag and not a branch so can't use gerrit | 11:14 |
| migi | any hint will be appriciated | 11:14 |
| migi | jhesketh: ^^, since you created tags | 11:15 |
| *** dave-mccowan has joined #openstack-nova | 11:16 | |
| jhesketh | migi: firstly is the tag in the correct position? ie was the branch broken before the EOL? | 11:16 |
| *** toabctl has joined #openstack-nova | 11:17 | |
| migi | jhesketh: see here: https://github.com/openstack/nova/blob/liberty-eol/tox.ini#L12 | 11:18 |
| migi | jhesketh: before it was ok | 11:18 |
| jhesketh | migi: ah, I see the issue | 11:18 |
| jhesketh | migi: the tag is in the correct place though right? it's just that the code needs the branch | 11:18 |
| migi | jhesketh: so proper link should be: https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=liberty-eol | 11:18 |
| migi | jhesketh: tag is ok | 11:19 |
| jhesketh | okay, just checking :-) | 11:19 |
| robcresswell | Could anyone from Nova shed any light on this change https://review.openstack.org/#/c/501564/ ? It sounds like an old microversion change, but I'm unsure what the correct path here is. Does novaclient default to 2.1 if no version is supplied? | 11:19 |
| migi | jhesketh: same will happen with kilo/mitaka | 11:19 |
| jhesketh | right | 11:20 |
| jhesketh | so the point of EOL'ing is to not make any more changes to a release. And tags typically shouldn't be moved | 11:20 |
| jhesketh | so while there are a few technical ways of fixing it, I think this is more a discussion for the nova and stable team to have | 11:20 |
| migi | jhesketh: actually kilo is OK as it has no such pointer, just mitaka | 11:20 |
| *** slaweq has joined #openstack-nova | 11:20 | |
| migi | jhesketh: meaning who exactly? | 11:21 |
| jhesketh | migi: my suggestion would be to raise it on the dev mailing list either in the EOL thread or as a new one | 11:22 |
| *** rgerganov has joined #openstack-nova | 11:22 | |
| migi | jhesketh: yep, will do, thanks | 11:22 |
| jhesketh | there are pros and cons to these kind of changes and there is an argument to be had whether it's something to allow from the point of view of discontinued development | 11:22 |
| jhesketh | my personal thought is that if you needed to continue working on an EOL release that you could create a fork of the project and host the branch yourself somewhere | 11:23 |
| rgerganov | gibi, stephenfin: could you please take a look at https://review.openstack.org/#/c/450636/ ? This patch finishes the implementation of console log for VMware. Thanks! | 11:23 |
| jhesketh | but clearly that has huge implications and is a massive amount of work given the number of repos etc | 11:23 |
| *** karthiks has quit IRC | 11:24 | |
| *** slaweq has quit IRC | 11:26 | |
| *** markvoelker has quit IRC | 11:28 | |
| *** nmathew has quit IRC | 11:34 | |
| *** vladikr has quit IRC | 11:39 | |
| *** vladikr has joined #openstack-nova | 11:39 | |
| *** slaweq has joined #openstack-nova | 11:43 | |
| *** zhenq has joined #openstack-nova | 11:45 | |
| *** zhenq has quit IRC | 11:45 | |
| *** zhenq has joined #openstack-nova | 11:46 | |
| *** zhenq has quit IRC | 11:46 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | 11:46 |
| *** alexchadin has joined #openstack-nova | 11:47 | |
| *** slaweq has quit IRC | 11:47 | |
| *** Sree has joined #openstack-nova | 11:52 | |
| *** slaweq has joined #openstack-nova | 11:53 | |
| *** Sree has quit IRC | 11:56 | |
| *** thorst has joined #openstack-nova | 11:57 | |
| *** zhenq has joined #openstack-nova | 11:58 | |
| *** Sree has joined #openstack-nova | 11:58 | |
| *** zhenq has quit IRC | 11:58 | |
| *** litao__ has quit IRC | 11:59 | |
| *** slaweq has quit IRC | 11:59 | |
| *** karthiks has joined #openstack-nova | 12:00 | |
| *** jpena is now known as jpena|lunch | 12:04 | |
| *** alexchadin has quit IRC | 12:06 | |
| *** MVenesio has joined #openstack-nova | 12:06 | |
| *** alexchadin has joined #openstack-nova | 12:07 | |
| openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573 | 12:07 |
| *** edmondsw has joined #openstack-nova | 12:12 | |
| efried | alex_xu Would you mind if I posted an edit on https://review.openstack.org/#/c/497713/ just for spelling/grammar? | 12:13 |
| *** markvoelker has joined #openstack-nova | 12:17 | |
| *** markvoelker has quit IRC | 12:21 | |
| *** markvoelker has joined #openstack-nova | 12:22 | |
| *** MVenesio has quit IRC | 12:22 | |
| *** pchavva has joined #openstack-nova | 12:26 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: don't overwrite binding-profile https://review.openstack.org/505613 | 12:27 |
| *** slaweq has joined #openstack-nova | 12:27 | |
| *** slaweq has quit IRC | 12:32 | |
| *** jmlowe has joined #openstack-nova | 12:37 | |
| *** slaweq has joined #openstack-nova | 12:37 | |
| *** acormier has joined #openstack-nova | 12:38 | |
| *** jmlowe has quit IRC | 12:38 | |
| *** kylek3h has joined #openstack-nova | 12:38 | |
| *** gbarros has joined #openstack-nova | 12:39 | |
| *** udesale has joined #openstack-nova | 12:39 | |
| *** aarefiev has quit IRC | 12:40 | |
| *** gbarros has quit IRC | 12:40 | |
| *** yassine has joined #openstack-nova | 12:41 | |
| *** acormier has quit IRC | 12:41 | |
| *** gbarros has joined #openstack-nova | 12:42 | |
| *** acormier has joined #openstack-nova | 12:42 | |
| *** slaweq has quit IRC | 12:42 | |
| sdague | something seems to have gone pretty sideways on livemigration jobs http://logs.openstack.org/55/502155/5/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/52b53e8/logs/screen-n-api.txt.gz#_Sep_20_08_32_57_608522 | 12:43 |
| *** manasm_ has joined #openstack-nova | 12:44 | |
| * kashyap clicks | 12:44 | |
| *** manasm has quit IRC | 12:44 | |
| *** manasm_ is now known as manasm | 12:44 | |
| *** acormier has quit IRC | 12:46 | |
| *** slaweq has joined #openstack-nova | 12:47 | |
| efried | sdague - mriedem submitted a fix in devstack last night. | 12:48 |
| sdague | efried: url? | 12:49 |
| sdague | I guess it didn't hit my radar yet | 12:49 |
| efried | Yeah, getting it, stand by. | 12:49 |
| efried | sdague https://bugs.launchpad.net/nova/+bug/1718295 | 12:50 |
| openstack | Launchpad bug 1718295 in devstack "Unexpected exception in API method: MigrationError_Remote: Migration error: Disk info file is invalid: qemu-img failed to execute - Failed to get shared "write" lock\nIs another process using the image?" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | 12:50 |
| sdague | oh, the pike ppa went sideways? | 12:50 |
| efried | a>ppa<rently | 12:51 |
| sdague | good to know | 12:51 |
| sdague | "P | 12:51 |
| sdague | "P | 12:51 |
| sdague | :P (damn kept missing keys) | 12:51 |
| efried | I knew what you meant. | 12:51 |
| *** manasm has quit IRC | 12:52 | |
| *** kbaegis has joined #openstack-nova | 12:52 | |
| *** slaweq has quit IRC | 12:52 | |
| efried | sdague Looks like you juuuust missed the fix with that patch :) | 12:52 |
| efried | cdent gotcher back | 12:52 |
| cdent | efried: eh? wat? zzzzzzzzz | 12:53 |
| efried | cdent Your timely recheck on https://review.openstack.org/#/c/502155/ a couple hours ago. | 12:54 |
| sdague | efried: apparently, well, at least it's fixed now | 12:54 |
| sdague | I wish the ppa game wasn't such a coin flip | 12:54 |
| *** d9k has left #openstack-nova | 12:55 | |
| efried | sdague Reverted, anyway. It's possible an actual fix is in the works, based on jamespage comments in the bug. | 12:55 |
| *** ygl has joined #openstack-nova | 12:56 | |
| kashyap | Can someon remind, in what context are we using Ubuntu PPA (Personal Package Archives) | 12:56 |
| *** mriedem has joined #openstack-nova | 12:56 | |
| sdague | kashyap: to get newer libvirt on xenial | 12:57 |
| cdent | efried: thanks, I’m still relocating my landmarks | 12:57 |
| kashyap | sdague: Is PPA considered 'official' - as in upstream Ubuntu maintainers maintain it? | 12:57 |
| sdague | kashyap: yes | 12:57 |
| sdague | this is the ocata ppa we're working with | 12:57 |
| sdague | it's ubuntu supported | 12:57 |
| *** slaweq has joined #openstack-nova | 12:58 | |
| kashyap | Ah, okay. For a moment, I assumed PPA in this context meant - "random unsupported packages done by a contributor, which is used only for personal testing" | 12:58 |
| *** lyan has joined #openstack-nova | 12:59 | |
| *** felipemonteiro has joined #openstack-nova | 13:01 | |
| *** catintheroof has joined #openstack-nova | 13:02 | |
| stephenfin | sahid: RE: https://review.openstack.org/#/c/501132/, could you add a summary of the comments with sean-k-mooney to the commit message? After that, it's an easy +2 | 13:02 |
| *** felipemonteiro__ has joined #openstack-nova | 13:02 | |
| *** slaweq has quit IRC | 13:04 | |
| *** jpena|lunch is now known as jpena | 13:04 | |
| alex_xu | nova api meeting is running at #openstack-meeting-4 | 13:05 |
| *** felipemonteiro has quit IRC | 13:06 | |
| *** yingjun has joined #openstack-nova | 13:07 | |
| *** gouthamr has joined #openstack-nova | 13:07 | |
| *** abhishekk has quit IRC | 13:08 | |
| *** ociuhandu has joined #openstack-nova | 13:08 | |
| *** xyang1 has joined #openstack-nova | 13:08 | |
| *** esberglu has joined #openstack-nova | 13:09 | |
| *** manasm has joined #openstack-nova | 13:09 | |
| *** lajoskatona has joined #openstack-nova | 13:10 | |
| *** zhenq has joined #openstack-nova | 13:11 | |
| *** crushil_ has joined #openstack-nova | 13:11 | |
| *** ratailor has quit IRC | 13:12 | |
| *** zhenq has quit IRC | 13:12 | |
| openstackgerrit | Merged openstack/nova-specs master: Spec to remove the hide server address config options https://review.openstack.org/502516 | 13:14 |
| *** baoli has joined #openstack-nova | 13:15 | |
| *** sean-k-mooney has quit IRC | 13:18 | |
| *** slaweq has joined #openstack-nova | 13:19 | |
| *** alexchadin has quit IRC | 13:21 | |
| *** slaweq has quit IRC | 13:24 | |
| *** manasm has quit IRC | 13:27 | |
| *** burt has joined #openstack-nova | 13:27 | |
| *** yingjun has quit IRC | 13:28 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 13:28 |
| *** slaweq has joined #openstack-nova | 13:30 | |
| *** kevi9132 has joined #openstack-nova | 13:30 | |
| *** zhenq has joined #openstack-nova | 13:30 | |
| *** zhenq has quit IRC | 13:31 | |
| *** jheroux has joined #openstack-nova | 13:32 | |
| *** ygl has quit IRC | 13:32 | |
| *** kevi9132_ has joined #openstack-nova | 13:33 | |
| *** kevi9132 has quit IRC | 13:35 | |
| *** slaweq has quit IRC | 13:36 | |
| *** zhenq has joined #openstack-nova | 13:40 | |
| *** zhenq has quit IRC | 13:40 | |
| *** ygl has joined #openstack-nova | 13:44 | |
| *** rmart04 has quit IRC | 13:46 | |
| *** rmart04_ has joined #openstack-nova | 13:46 | |
| *** alexchadin has joined #openstack-nova | 13:46 | |
| *** cleong has joined #openstack-nova | 13:46 | |
| openstackgerrit | Merged openstack/nova master: Fix a typo https://review.openstack.org/505062 | 13:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | 13:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | 13:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add documentation for cpu_realtime, cpu_realtime_mask https://review.openstack.org/502056 | 13:49 |
| stephenfin | sdague: Could you take a look at the first of those? gibi has reviewed it a few times but it keeps getting into merge conflicts :( | 13:50 |
| sdague | +2 | 13:52 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate https://review.openstack.org/497455 | 13:52 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add method to configure migration speed https://review.openstack.org/497456 | 13:52 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 13:52 |
| sdague | that all looks very reasonable | 13:52 |
| stephenfin | sdague: Thank you, sir | 13:53 |
| gibi | stephenfin, sdague: I'm also checking that rebase with an intent to approve it | 13:53 |
| stephenfin | gibi: and you too :) | 13:53 |
| *** zhurong has joined #openstack-nova | 13:55 | |
| stephenfin | sahid: Could you take a look at https://review.openstack.org/#/c/502056/ again? | 13:55 |
| *** gbarros has quit IRC | 13:55 | |
| stephenfin | I get that we need to do more configuration that what's there, but there's a lot of stuff to do. I'd prefer to add an admin guide doc for that in the future | 13:56 |
| stephenfin | ...which I should do sometime in the next few weeks | 13:56 |
| *** gbarros has joined #openstack-nova | 13:56 | |
| * mdbooth is just looking at a corrupt customer log file containing lines like: | 13:56 | |
| mdbooth | 2017-09-14 15:54:39.689 120626 ERROR nova.s2017-09-14 15:54:39.690 120562 ERROR nova.servicegroup.drivers.db [-] Unexpected error while reporting service status | 13:57 |
| gibi | stephenfin: +2+W | 13:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 13:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 13:57 |
| stephenfin | gibi: Yay. Thanks :) | 13:57 |
| mdbooth | With the continuation of the first log later in the file without its timestamp | 13:57 |
| mdbooth | Are we not locking in the logger? | 13:57 |
| sahid | stephenfin: the admin is going to configure the flavor, it seems reasonable to me to add a note saying what i do have mentioned on the review | 13:58 |
| *** felipemonteiro__ has quit IRC | 13:58 | |
| mdbooth | Unfortunately, it also means the logs are not correctly sorted :/ | 13:58 |
| stephenfin | sahid: Right, but vcpu_pin_set is not a flavor property. I've mentioned pinned CPUs because it is | 13:58 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: [WIP] CLI for aggregates https://review.openstack.org/505643 | 13:58 |
| *** udesale has quit IRC | 13:59 | |
| mdbooth | This is not conducive to merge sorting | 13:59 |
| stephenfin | You also need to configure things like isolate the CPUs and use a properly configured guest, but I don't mention those there because they're nothing to do with configuring flavor properties | 13:59 |
| * mdbooth files a bug | 13:59 | |
| sahid | isolate the CPUs? | 13:59 |
| stephenfin | sahid: From the host? | 14:00 |
| stephenfin | Whatever it is that replace the isolcpus boot parameter | 14:00 |
| sahid | that is not related to Nova, i'm talking about an option which is related to Nova, that is why i think a small note is important | 14:00 |
| stephenfin | and vcpu_pin_set is not related to flavors. This is an flavor (extra_spec) overview doc | 14:01 |
| sahid | if you don't mention that an admin could just think that after to have configured the host, enabling cpu_realtime=yes in flavor is enough | 14:01 |
| *** manasm has joined #openstack-nova | 14:01 | |
| stephenfin | They could also boot an standard Linux guest kernel or forget to configure isolcpus | 14:02 |
| stephenfin | The point is that this is just an overview of the flavor extra specs available and the interactions between them | 14:03 |
| *** awaugama has joined #openstack-nova | 14:03 | |
| *** rajathagasthya has joined #openstack-nova | 14:03 | |
| stephenfin | We should add a real-time doc but it should be separate, like this: | 14:03 |
| stephenfin | https://docs.openstack.org/nova/latest/admin/cpu-topologies.html | 14:03 |
| *** slaweq has joined #openstack-nova | 14:04 | |
| sahid | stephenfin: ok, i just gave to you my point, i would have added that note but feel free to not mention it | 14:04 |
| *** Sree has quit IRC | 14:04 | |
| stephenfin | sahid: Yup, and I appreciate it :) I'm countering that I don't think it's necessary here, and would make more sense in the upcoming larger doc | 14:05 |
| *** shan has quit IRC | 14:05 | |
| stephenfin | ...where I'll definitely mention it | 14:05 |
| mriedem | mdbooth: it's likely a problem in the customers log config | 14:06 |
| mriedem | mdbooth: i saw something like that with our new super conductor logs in devstack, the fix for that was in devstack https://review.openstack.org/#/c/497944/1/lib/nova | 14:06 |
| mriedem | but you should checkout what devstack does for log config | 14:06 |
| mdbooth | mriedem: Looking. | 14:06 |
| *** gbarros has quit IRC | 14:06 | |
| mdbooth | mriedem: Are you sure that's the same? It looks like 2 threads are writing simultaneously to the same log file. | 14:08 |
| *** ijw has joined #openstack-nova | 14:08 | |
| mdbooth | Hence a new log starts in the middle of the previous one, rather than on a separate line | 14:08 |
| sdague | mdbooth: the python logger should handle that | 14:08 |
| *** slaweq has quit IRC | 14:09 | |
| *** armax has joined #openstack-nova | 14:09 | |
| sdague | callers of the logger should not be locking around it, that's all supposed to be handled within the logger itself | 14:09 |
| sdague | all oslo.log does is setup some common python logger patterns | 14:10 |
| mdbooth | sdague: Yeah, that's what I'd have thought... | 14:10 |
| mdbooth | I wonder how this has happened, though | 14:10 |
| mdbooth | There are a ton of examples of it in these logs | 14:10 |
| sdague | mdbooth: going through syslog? | 14:11 |
| sdague | because syslog has some challenges | 14:11 |
| *** chyka has joined #openstack-nova | 14:11 | |
| mdbooth | Looks like it was generating DB errors continuously for a period of time, so lots of opportunity for overlap | 14:11 |
| mdbooth | Do we normally log through syslog? | 14:11 |
| toabctl | mriedem, hey. could you please have another look at https://review.openstack.org/#/c/398308/ ? | 14:11 |
| sdague | no, we normally log to a file or stdout | 14:11 |
| *** zhenq has joined #openstack-nova | 14:12 | |
| *** zhenq has quit IRC | 14:12 | |
| sdague | now, that being said, I expect if you are blowing through the output buffer regularly by putting giant stack traces all the time, you might end up with the tails of those going weird | 14:12 |
| jamespage | sdague, efried: just to be clear, I don't think there is a fix to make in qemu - afaict its behaving as intended for the 2.10 release | 14:12 |
| sdague | but that's probably a more latent python logging issue | 14:12 |
| sdague | jamespage: yeh, it being another flag seems to indicate that | 14:13 |
| sdague | jamespage: do you all have a patch already for it for your pike ppa on nova? | 14:13 |
| mdbooth | Hmm, this is conductor not compute | 14:13 |
| mdbooth | Are the workers independent? | 14:13 |
| mdbooth | i.e. might they be separately opening the same log file? | 14:13 |
| *** gbarros has joined #openstack-nova | 14:14 | |
| sdague | mdbooth: yes, the workers are processes | 14:14 |
| jamespage | sdague: I have a simple patch to fix the nova package for Pike in Ubuntu and the UCA; that's not good for direct submission to nova as its not conditional i.e. its a blind add the flag (cause we know which qemu version will be in use for artful and xenial+ Pike UCA) | 14:14 |
| *** slaweq has joined #openstack-nova | 14:14 | |
| mdbooth | sdague: I'll bet that's it... | 14:14 |
| sdague | jamespage: gotcha | 14:14 |
| *** rmart04_ has quit IRC | 14:14 | |
| mriedem | toabctl: done | 14:14 |
| toabctl | mriedem, thx | 14:15 |
| sdague | jamespage: link for where you injected it might be good regardless | 14:15 |
| sdague | jamespage: then we can figure out the right conditional | 14:15 |
| jamespage | sdague: looking the libvirt driver + images module to figure out the best way to pass that in conditionally - most version checking is done in driver, not in images... | 14:15 |
| jamespage | sdague: | 14:15 |
| jamespage | sure | 14:15 |
| mriedem | jamespage: i was having the same problem when thinking about how to make that conditional | 14:15 |
| mriedem | the driver knows the version, but way down in the bowels of the image code it doesn't | 14:16 |
| *** chyka has quit IRC | 14:16 | |
| *** rmart04 has joined #openstack-nova | 14:16 | |
| jamespage | mriedem: yeah its awkward from that perspective | 14:16 |
| *** ijw has quit IRC | 14:16 | |
| jamespage | lemme attach my patch to the bug report | 14:16 |
| *** acormier has joined #openstack-nova | 14:18 | |
| mriedem | tasker: updated https://review.openstack.org/#/c/504260/ | 14:18 |
| *** slaweq has quit IRC | 14:19 | |
| *** zhenq has joined #openstack-nova | 14:19 | |
| *** zhenq has quit IRC | 14:20 | |
| jamespage | mriedem: my thinking was to pass that down from driver to the image code as an optional param | 14:20 |
| jamespage | testing that patch shortly | 14:20 |
| mriedem | jamespage: yeah that is probably what i'd do | 14:22 |
| mriedem | exit code is 1 when it fails, so that's not unique enough, | 14:23 |
| mriedem | we could scrape the stderr for the message, and retry with the flag, but that's not fun either | 14:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 14:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 14:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661 | 14:23 |
| sahid | stephenfin: about https://review.openstack.org/#/c/501132/, it seems to me the commit message well reflects what is done on the patch | 14:24 |
| sdague | mriedem: is there a reason to not set a CONST with the version on __init__ of the driver? | 14:24 |
| *** slaweq has joined #openstack-nova | 14:24 | |
| stephenfin | sahid: It reflects what but not _why_. The why is what I care about (I can parse the what from reading the code) | 14:25 |
| mriedem | sdague: and have the image code check the driver? | 14:25 |
| *** cfriesen has joined #openstack-nova | 14:25 | |
| stephenfin | If sean-k-mooney had questions that you took the time to address, the chances are that others will have the same question if they look at that patch in however many months/years time | 14:25 |
| dansmith | mriedem: I think he means a global, which seems less good to me | 14:26 |
| sdague | mriedem: well, I was thinking have the driver set a CONST in the image namespace | 14:26 |
| mdbooth | sdague: Thanks. I've filed a bug against Nova, but suspect it's only actually fixable by using an external logging service. | 14:26 |
| mdbooth | Or perhaps by having separate conductors writing to separate log files? | 14:26 |
| openstackgerrit | Merged openstack/nova master: Skip more racy rebuild failing tests with cells v1 https://review.openstack.org/499001 | 14:26 |
| stephenfin | sahid: See here for example https://review.openstack.org/#/c/479802/ | 14:26 |
| sdague | because I thought we wanted image to be distinct | 14:26 |
| mdbooth | We'd have to name them. | 14:26 |
| sdague | mdbooth: I'm not sure what help a nova bug does here, it's really not fixable in nova | 14:27 |
| cfriesen | is anyone aware of a scheduling issue in Pike when rescheduling instances that were originally booted as part of a multi-instance boot? There's a thread "Pike NOVA Disable and Live Migrate all instances" on the openstack list that seems to indicate a bug. | 14:27 |
| openstackgerrit | Merged openstack/nova master: conf: Rename two VNC options https://review.openstack.org/498387 | 14:27 |
| dansmith | mdbooth: python logging should be locking the fd, which is opened before the fork | 14:27 |
| *** chyka has joined #openstack-nova | 14:27 | |
| mdbooth | sdague: Well I found it in Nova, and it's definitely a thing. There's likely to be a better place to move it. | 14:27 |
| *** ygl has quit IRC | 14:27 | |
| mriedem | mdbooth: if you found it in nova, it's also then probably an issue in all other services | 14:28 |
| mriedem | if it's logging related | 14:28 |
| mdbooth | dansmith: That kind of locking could only work by preventing other conductors from opening the log file at all. | 14:28 |
| *** zhenq has joined #openstack-nova | 14:28 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc https://review.openstack.org/505665 | 14:28 |
| mdbooth | Which would mean that it would block all conductors beyond the first. | 14:28 |
| *** zhenq has quit IRC | 14:29 | |
| dansmith | mdbooth: the other processes don't open the log, they inherit it across the fork | 14:29 |
| *** slaweq has quit IRC | 14:29 | |
| mdbooth | dansmith: Oh, interesting. However, the locking would still be python thread locking. | 14:30 |
| *** andreas_s has quit IRC | 14:30 | |
| dansmith | not if logging is locking the file | 14:30 |
| cdent | cfriesen: I’ve been wondering if part of that was due somehow to the doubling stuff | 14:30 |
| mdbooth | Unless the logger is taking and releasing an os lock for every write? | 14:30 |
| *** rmart04 has quit IRC | 14:32 | |
| sdague | mdbooth: I really think that once you push sufficiently large writes through the python logging buffer, this is just the python behavior | 14:32 |
| sdague | and the only fix is don't do that | 14:32 |
| *** zhenq has joined #openstack-nova | 14:32 | |
| *** zhenq has quit IRC | 14:32 | |
| openstackgerrit | sahid proposed openstack/os-vif master: ovs-hybrid: should permanently keep MAC entries https://review.openstack.org/501132 | 14:33 |
| *** aarefiev has joined #openstack-nova | 14:34 | |
| *** slaweq has joined #openstack-nova | 14:34 | |
| mdbooth | https://docs.python.org/3/library/multiprocessing.html#module-multiprocessing | 14:35 |
| mdbooth | According to ^^^ in python 3 at least logging doesn't use external locks | 14:35 |
| mdbooth | I guess that would make it a bug in oslo.log | 14:36 |
| *** tidwellr has joined #openstack-nova | 14:36 | |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the ResourceClass object https://review.openstack.org/502155 | 14:36 |
| stephenfin | sahid: Lovely. +Wd | 14:36 |
| mdbooth | Same for python 2 | 14:36 |
| mdbooth | And I don't see oslo.log importing multiprocessing | 14:37 |
| mdbooth | Well, it does, but it doesn't seem to use it | 14:37 |
| mdbooth | That's pretty weird | 14:37 |
| sdague | mdbooth: I expect that when you don't overrun the python logging natural buffer it just works | 14:38 |
| *** mingyu has joined #openstack-nova | 14:38 | |
| sdague | and when you do, you get funkiness | 14:38 |
| sdague | and I agree, if you want to fix it, you have to do it down in oslo.log | 14:38 |
| mdbooth | sdague: We log exceptions, though, which are kinda arbitrarily large | 14:38 |
| mdbooth | I don't think we want to stop doing that | 14:38 |
| sdague | mdbooth: we do, but we've apparently been lucky thus far | 14:39 |
| mdbooth | I'll move the bug to oslo.log and mention the multiprocess thing | 14:39 |
| mdbooth | Assuming, that is, we don't want to open separate log files for conductor workers? | 14:39 |
| mdbooth | Because that would be a nova fix, and possibly useful in its own right | 14:40 |
| sdague | mdbooth: no, because that problem is equally theoretically a problem for all the other services as well | 14:40 |
| dansmith | I definitely don't want separate logs for conductor workers | 14:40 |
| sdague | except nova-compute | 14:40 |
| dansmith | because..holy crap | 14:40 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 14:40 |
| *** slaweq has quit IRC | 14:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 14:40 |
| mdbooth | dansmith: Merged logs, ftw! ;) | 14:41 |
| sdague | and it totally would wreck all the log injest systems people have | 14:41 |
| mdbooth | True. | 14:41 |
| mdbooth | oslo.log it is | 14:41 |
| *** Tom___ has joined #openstack-nova | 14:41 | |
| *** NostawRm has joined #openstack-nova | 14:42 | |
| *** zhenq has joined #openstack-nova | 14:42 | |
| jamespage | mriedem: working a fix now | 14:43 |
| cfriesen | cdent: that does seem to be the most likely suspect. points to a gap in our testing. | 14:43 |
| openstackgerrit | Merged openstack/nova master: Update docs for _destroy_evacuated_instances https://review.openstack.org/500144 | 14:45 |
| mriedem | jamespage: i'm testing sdague's idea too | 14:46 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: check qemu version when calling qemu-img info https://review.openstack.org/505673 | 14:46 |
| *** jpena is now known as jpena|brb | 14:46 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | 14:46 |
| cdent | cfriesen: gaps in testing is a bit of a trend, but gibi is fixing it ;) | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | 14:47 |
| mriedem | jamespage: testing here https://review.openstack.org/#/c/505674/ | 14:47 |
| sdague | mriedem: you got a devstack patch depends on that? | 14:48 |
| mriedem | yes ^ | 14:48 |
| sdague | ah cool | 14:48 |
| gibi | cdent: do you mean I should add a test case which boots multiple instances with a single boot command then try to migrate them? ;) | 14:49 |
| cdent | I meant you were fixing the trend more generally but if you’re feeling motivated :) | 14:49 |
| dansmith | mriedem: if you're okay with it I'm just going to fast approve all these unregister patches as they're just mechanical search/replace: https://review.openstack.org/#/c/502157/5 | 14:50 |
| *** rmart04 has joined #openstack-nova | 14:50 | |
| openstackgerrit | Merged openstack/nova master: Add @targets_cell for live_migrate_instance method in conductor https://review.openstack.org/503601 | 14:51 |
| mriedem | dansmith: i can go through them quick | 14:51 |
| mriedem | cfriesen: re that ML thread, he's disabling a host and live migrating off the source host - does he mean evacuating off the source host? | 14:52 |
| *** smatzek has joined #openstack-nova | 14:52 | |
| gibi | cdent: at least I made TODO on my desk about it but I'm not promising anything :) | 14:53 |
| dansmith | mriedem: okay | 14:54 |
| mriedem | dansmith: cdent: question in https://review.openstack.org/#/c/502157/5//COMMIT_MSG but just to make sure i know what we're doing in the series | 14:54 |
| dansmith | mriedem: I'm not sure I understand what you're asking | 14:55 |
| dansmith | oh I see, | 14:56 |
| dansmith | because there weren't any actual object references anywhere | 14:56 |
| dansmith | in that patch specifically | 14:56 |
| *** slaweq has joined #openstack-nova | 14:56 | |
| *** hongbin has joined #openstack-nova | 14:56 | |
| cdent | yeah, that’s a pasto | 14:57 |
| mriedem | yeah, ok | 14:57 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM hypervisor configuration doc https://review.openstack.org/505665 | 14:57 |
| mriedem | alright moving on | 14:57 |
| *** sbezverk has quit IRC | 14:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 14:58 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 14:58 |
| *** mdnadeem has quit IRC | 14:58 | |
| *** Tom___ has quit IRC | 14:58 | |
| *** Tom has joined #openstack-nova | 14:58 | |
| mriedem | cfriesen: on that migrate thread, i think in pike ed added the new stuff to pass the number of instances from the request spec (something like that) to the scheduler, so it could be why it's thinking it wants to migrate all 10 at once, from the persisted request spec | 14:58 |
| mriedem | might be good for edleafe to look at that | 14:58 |
| mriedem | https://review.openstack.org/#/c/465171/ | 14:59 |
| mriedem | but this passes just the single instance being live migrated https://review.openstack.org/#/c/465171/11/nova/conductor/tasks/live_migrate.py | 15:00 |
| *** slaweq has quit IRC | 15:01 | |
| *** alexchadin has quit IRC | 15:02 | |
| *** Tom has quit IRC | 15:03 | |
| sahid | melwitt: hum i do not understand your comment | 15:05 |
| mriedem | cfriesen: this looks like a latent issue https://github.com/openstack/nova/blob/8a386b055c82df67092a1abc683e7225ef80671e/nova/scheduler/filter_scheduler.py#L81 | 15:05 |
| sahid | https://review.openstack.org/#/c/334614/ | 15:05 |
| *** zhurong has quit IRC | 15:05 | |
| mriedem | we're checking the number of instances from the request spec | 15:05 |
| mriedem | which is persisted when we created the instance with multi-create | 15:06 |
| mriedem | and using that during the live migration of the single instance | 15:06 |
| cdent | bauzas: are you still invested in keeping your -1 on https://review.openstack.org/#/c/419502/ | 15:06 |
| sahid | not sure if you want me to comment an other time or if we can discuss about it | 15:06 |
| mriedem | cfriesen: "num_instances = spec_obj.num_instances" should probably be changed to "num_instances = len(instance_uuids)" now | 15:06 |
| *** Nil_ has joined #openstack-nova | 15:07 | |
| *** slaweq has joined #openstack-nova | 15:07 | |
| mriedem | cfriesen: i'm not subscribed to the general list but you might ask them to change that line and see if it works | 15:07 |
| bauzas | cdent: I think my concern is still valid | 15:07 |
| bauzas | cdent: renaming an AZ is having an user impact | 15:07 |
| dansmith | bauzas: cdent dear god | 15:08 |
| *** lajoskatona has quit IRC | 15:08 | |
| mriedem | you should just not be able to rename an az while there are instances in that az | 15:08 |
| cdent | bauzas: I was checking because as far as I could read the bug report, the “bad” solution was considered good enough | 15:08 |
| dansmith | that operation could be across cells, and across a loooot of instances. It could fail in the middle leaving things highly confusing | 15:08 |
| dansmith | mriedem: ++ | 15:08 |
| bauzas | mriedem: that was my point | 15:09 |
| mriedem | if you want to rename the az, migrate the instances out of it | 15:09 |
| mriedem | but i'm not sure if that's possible? | 15:09 |
| mriedem | unless you use the force flag... | 15:09 |
| *** rmart04 has quit IRC | 15:09 | |
| mriedem | but if the operator forces a live migration of an instance from az1 to az2, do we even update the instance.availability_zone field? | 15:09 |
| cdent | I think the implementation was following sean’s advice: https://bugs.launchpad.net/nova/+bug/1378904/comments/5 | 15:10 |
| openstack | Launchpad bug 1378904 in OpenStack Compute (nova) "renaming availability zone doesn't modify host's availability zone" [Low,In progress] - Assigned to Radoslav Gerganov (rgerganov) | 15:10 |
| cdent | but if there’s a bigger problem than that, would be great to see those problems on the review | 15:10 |
| bauzas | mriedem: I think migrating a VM requires some kind of discussion between the operator and the end user | 15:10 |
| *** manasm has quit IRC | 15:10 | |
| bauzas | not something magical per so | 15:10 |
| bauzas | se | 15:10 |
| dansmith | migrating across AZs without the user's input is dangerous too | 15:11 |
| mriedem | what if the host is going to fail? i guess live migrate to another host in the same az | 15:11 |
| bauzas | and yeah, you can't live migrate from one AZ to the other | 15:11 |
| bauzas | unless you force of course | 15:11 |
| mriedem | bauzas: with the force flag you can :) | 15:11 |
| mriedem | right? | 15:11 |
| bauzas | yup | 15:11 |
| bauzas | but that'd be a terrible experience again | 15:11 |
| bauzas | because I'm not sure we update the AZ record honestly | 15:11 |
| sdague | AZs really exist to bound failure domains so that you can HA across them correctly | 15:11 |
| sdague | moving an instance across AZ boundaries completely ruins that strategy | 15:12 |
| bauzas | so, the point is, if you made a typo, then you're screwed up if some users began to use your cloud, and you have to explain to them that you screwed up | 15:12 |
| *** Oku_OS is now known as Oku_OS-away | 15:12 | |
| bauzas | but you shouldn't magically fix your issue | 15:12 |
| bauzas | tl;dr: assume your mistakes | 15:13 |
| dansmith | sdague: that was my point yeah | 15:13 |
| *** slaweq has quit IRC | 15:13 | |
| cfriesen | mriedem: will pass on the suggestion | 15:13 |
| * edleafe reads back | 15:13 | |
| sdague | dansmith: ++ | 15:13 |
| mriedem | edleafe: think it's sorted out, but still looks like a latent bug | 15:13 |
| edleafe | mriedem: Agree on the change for num_instances | 15:14 |
| *** suresh12 has joined #openstack-nova | 15:14 | |
| edleafe | I can do that quickly - maybe for backport? | 15:14 |
| mriedem | don't use multi-create, and if you do, don't migrate any of those instances if len(hosts) < len(instances) | 15:14 |
| mriedem | edleafe: i think it would be good to have a functional test for thisfirst | 15:14 |
| mriedem | e.g. 2 computes, 2 instances created in a single boot request to compute 1, disable compute 1 and live migrate the isntances to compute 2 | 15:15 |
| mriedem | it should fail on the first live migration attempt since you're trying to move 2 instances and we have 1 host | 15:15 |
| sdague | dansmith: I'm +1 on your block of az renames if there are instances in them | 15:15 |
| sdague | honestly, these things should probably be idempotent like flavors | 15:15 |
| dansmith | sdague: cool | 15:15 |
| *** Tom has joined #openstack-nova | 15:15 | |
| dansmith | yeah, you'll still have to count instances across cells in the az in question to know whether or not to block it | 15:16 |
| dansmith | just not letting that happen at all is easier still | 15:16 |
| mriedem | edleafe: it's actually a weird check in the filter scheduler driver, i don't really understand why we compare the number of instances to the number of hosts, surely we can create more than one instance per host | 15:17 |
| openstackgerrit | Merged openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | 15:17 |
| edleafe | mriedem: are you refrerring to https://github.com/openstack/nova/blob/8a386b055c82df67092a1abc683e7225ef80671e/nova/scheduler/filter_scheduler.py#L86 ? | 15:18 |
| *** jpena|brb is now known as jpena | 15:18 | |
| openstackgerrit | Merged openstack/nova master: doc: Add documentation for emulator_thread_policy https://review.openstack.org/501721 | 15:18 |
| mriedem | edleafe: yes | 15:18 |
| edleafe | mriedem: if so, that's the number of *selected* hosts, not the total number of hosts | 15:18 |
| mriedem | cfriesen: edleafe: in fact https://review.openstack.org/#/c/491439/ | 15:18 |
| edleafe | IOW, we couldn't find hosts for all the instances | 15:18 |
| *** dave-mcc_ has joined #openstack-nova | 15:18 | |
| esberglu | sdague: Can you restore this for us? https://review.openstack.org/#/c/422512/ | 15:18 |
| esberglu | If I were to submit a new patch would it restore the changeset? | 15:18 |
| *** liverpooler has quit IRC | 15:18 | |
| sdague | esberglu: restored | 15:19 |
| esberglu | sdague: tx | 15:19 |
| *** dave-mccowan has quit IRC | 15:19 | |
| sdague | esberglu: the patch owner in gerrit, or a core can do restores | 15:19 |
| sdague | but you have to restore before pushing an updated patch | 15:19 |
| esberglu | sdague: Good to know thanks | 15:20 |
| *** links has joined #openstack-nova | 15:21 | |
| mriedem | claudiub|2: the master branch change for https://review.openstack.org/#/c/505285/ is merged now | 15:21 |
| edleafe | mriedem: I'm confused. If https://review.openstack.org/#/c/491439/ merged, why is the old code still in master? | 15:24 |
| *** Tom has quit IRC | 15:24 | |
| *** links has quit IRC | 15:24 | |
| *** Tom has joined #openstack-nova | 15:25 | |
| tasker | having trouble live-migrating my last instance out of compute-1. after a suggestion from melwitt, I looked into the scheduler logs and I see that it scheduling the instance, it states that it's looking at the target host, notes that the target host fails and is removed from contention, but doesn't state why it failed -- even with debug logging on. | 15:25 |
| *** yamahata has joined #openstack-nova | 15:25 | |
| tasker | any thoughts? or is this infrastructure problems? | 15:25 |
| tasker | it's obvious that nova is doing its job, it's just not being verbose enough. | 15:25 |
| mriedem | edleafe: different place | 15:26 |
| *** sridharg has quit IRC | 15:27 | |
| edleafe | mriedem: ah | 15:27 |
| mriedem | tasker: depends on if it's the pre-live migration check on the target host, those logs would either be in nova-conductor or nova-compute for the target host | 15:27 |
| mriedem | edleafe: this is why we probably need a functional test for this scenario | 15:27 |
| *** slaweq_ has quit IRC | 15:29 | |
| *** Tom has quit IRC | 15:29 | |
| cfriesen | mriedem: edleave: did bauzas' patch cause the problem? (because the mailing list thread didn't see the IndexError) | 15:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 15:32 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 15:32 |
| mriedem | cfriesen: not sure | 15:32 |
| *** bigdogstl has joined #openstack-nova | 15:32 | |
| mriedem | it's clear that we need to be consistent in both places though | 15:32 |
| cfriesen | mriedem: for sure | 15:32 |
| *** ijw has joined #openstack-nova | 15:34 | |
| *** bigdogstl has quit IRC | 15:36 | |
| mriedem | dansmith: ok that series is all +W | 15:38 |
| *** slaweq has joined #openstack-nova | 15:39 | |
| *** Tom has joined #openstack-nova | 15:39 | |
| *** slaweq has quit IRC | 15:45 | |
| *** gyee has joined #openstack-nova | 15:45 | |
| *** Tom has quit IRC | 15:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 15:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 15:48 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] manage cache headers https://review.openstack.org/495380 | 15:52 |
| *** crushil_ is now known as crushil | 15:52 | |
| *** Sukhdev has joined #openstack-nova | 15:53 | |
| cdent | Is now a good time to get this merged, since we are at the beginning of the cycle: https://review.openstack.org/#/c/486829/ | 15:53 |
| gibi | could one of the cores look at test only patch? https://review.openstack.org/#/c/496202/ only needs a second +2 | 15:54 |
| cdent | fix on MiniDNS | 15:54 |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova master: Vzstorage: synchronize volume connect/disconnect https://review.openstack.org/505708 | 15:55 |
| stephenfin | gibi: Looking | 15:57 |
| *** Sukhdev_ has joined #openstack-nova | 15:58 | |
| gibi | stephenfin: thanks | 15:59 |
| *** Jeffrey4l_ has joined #openstack-nova | 16:01 | |
| *** penick has joined #openstack-nova | 16:03 | |
| *** rajathagasthya has quit IRC | 16:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 16:05 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 16:05 |
| *** Yikun__ has joined #openstack-nova | 16:06 | |
| *** Jeffrey4l has quit IRC | 16:06 | |
| *** Jeffrey4l has joined #openstack-nova | 16:07 | |
| mriedem | gibi: stephenfin: commented on https://review.openstack.org/#/c/496202/ | 16:08 |
| *** Jeffrey4l_ has quit IRC | 16:08 | |
| mriedem | that seems to be munging together a few different things | 16:08 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | 16:08 |
| *** suresh12 has quit IRC | 16:09 | |
| stephenfin | mriedem: Aye, but they all seemed valid | 16:09 |
| stephenfin | There could be merit in splitting them out though, so I've rebased to take it out of the gate queue | 16:09 |
| mriedem | if we're going to whitelist legacy notifications because they were missed, the change that adds them in should have a test to make sure they are sent | 16:09 |
| mriedem | which i guess is the referenced patch...but those are linked together in any way | 16:10 |
| mriedem | here is an example https://review.openstack.org/#/c/504978/ | 16:10 |
| dansmith | mriedem: right on right on right on | 16:13 |
| gibi | mriedem: OK, let's make a separate bug for the missing force_complete test coverage and move the whitelisting into that | 16:13 |
| mriedem | sounds good to me | 16:15 |
| mriedem | dansmith: you're getting older and the patches are staying the same age? | 16:16 |
| gibi | sorry for the mess | 16:16 |
| dansmith | mriedem: lol | 16:16 |
| *** rcernin has quit IRC | 16:17 | |
| bauzas | dansmith: sdague: sorry, was running a meeting, but like I said in my comment in https://review.openstack.org/#/c/419502/2 "Modifying an AZ name should not be possible if instances are still in the AZ" | 16:19 |
| mriedem | sdague: jamespage: looks like the devstack patch with the qemu version hack is passing the neutron multinode job that runs live migration | 16:19 |
| bauzas | dansmith: sdague: the only point I have is that if we agree on that, should we have a microversion for that? | 16:19 |
| dansmith | bauzas: we can't support the old thing, so no, IMHO | 16:20 |
| bauzas | dansmith: sdague: I mean, say I wanna update my aggregate and modify the AZ value, should we need a microversion for returning a 40x now ? | 16:20 |
| bauzas | instead of a HTTP200 | 16:20 |
| bauzas | in theory, it should, but I guess it's an already broken behaviour | 16:20 |
| bauzas | also, it would require to get the full list of all per-cell instances when updating the aggregate, not a huge deal but still something to do | 16:21 |
| *** Sree has joined #openstack-nova | 16:22 | |
| *** gbarros has quit IRC | 16:23 | |
| gibi | mriedem, stephenfin: reported the bug https://bugs.launchpad.net/nova/+bug/1718485 | 16:23 |
| openstack | Launchpad bug 1718485 in OpenStack Compute (nova) "instance.live.migration.force.complete is not a versioned notification and not whitelisted" [Undecided,New] | 16:23 |
| mriedem | thanks | 16:24 |
| gibi | will push the fix soon | 16:25 |
| *** hoonetorg has quit IRC | 16:25 | |
| *** ioggstream has quit IRC | 16:27 | |
| *** suresh12 has joined #openstack-nova | 16:29 | |
| *** rajathagasthya has joined #openstack-nova | 16:31 | |
| stephenfin | gibi: Cool. Ping me when you do, sure | 16:32 |
| *** cdent has quit IRC | 16:33 | |
| *** sahid has quit IRC | 16:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: check qemu version when calling qemu-img info https://review.openstack.org/505673 | 16:34 |
| *** suresh12 has quit IRC | 16:34 | |
| *** ociuhandu has quit IRC | 16:36 | |
| *** cdent has joined #openstack-nova | 16:38 | |
| *** nicolasbock has quit IRC | 16:39 | |
| *** sbezverk has joined #openstack-nova | 16:40 | |
| mriedem | dansmith: i assume the -2 on https://review.openstack.org/#/c/504983/ is just a placeholder for the whole thing to be reviewed and approved? | 16:41 |
| dansmith | mriedem: it was because the top level wasn't wired in, but yeah I figure no reason to land things that aren't reachable until we have some reasonable reviews on the rest of it | 16:42 |
| *** vvargaszte has joined #openstack-nova | 16:42 | |
| mriedem | efried: fyi for your ksa endpoint discovery stuff https://review.openstack.org/#/c/485121/ | 16:42 |
| *** vvargaszte has quit IRC | 16:42 | |
| mriedem | dansmith: ok, will hit those after lunh | 16:42 |
| mriedem | *lunch | 16:42 |
| *** suresh12 has joined #openstack-nova | 16:43 | |
| dansmith | mriedem: cool thanks | 16:44 |
| dansmith | mriedem: note that I found a bug in a functional test for pagination with that series | 16:44 |
| dansmith | which wasn't an issue before because we were inefficient, but it's nice that it was a test bug and not a functional one | 16:44 |
| *** Sree has quit IRC | 16:44 | |
| *** sambetts is now known as sambetts|afk | 16:44 | |
| *** Sree has joined #openstack-nova | 16:45 | |
| mriedem | efried: edmondsw: if the operator has to configure something different because of https://review.openstack.org/#/c/505546/ that's a big no-no for a backport | 16:45 |
| edmondsw | mriedem they don't have to | 16:46 |
| edmondsw | mriedem they can... they don't have to | 16:46 |
| *** READ10 has joined #openstack-nova | 16:47 | |
| *** kaisers_ has joined #openstack-nova | 16:47 | |
| *** felipemonteiro has joined #openstack-nova | 16:48 | |
| *** ijw has quit IRC | 16:48 | |
| openstackgerrit | James Page proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505748 | 16:49 |
| *** Sree has quit IRC | 16:50 | |
| cdent | edleafe: a) o/ b) if bp/return-selection-objects the right topic for the real thing? | 16:50 |
| *** namnh has joined #openstack-nova | 16:51 | |
| cdent | s/if/is/ | 16:51 |
| *** felipemonteiro__ has joined #openstack-nova | 16:51 | |
| *** mvk has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the UsageList object https://review.openstack.org/502156 | 16:52 |
| *** Sukhdev_ has quit IRC | 16:52 | |
| jamespage | mriedem: https://review.openstack.org/505748 but I think I prefer sdague's approach - calls to qemu_img_info are in alot of places... | 16:53 |
| *** psachin has quit IRC | 16:53 | |
| *** suresh12 has quit IRC | 16:53 | |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the Usage object https://review.openstack.org/502157 | 16:54 |
| *** suresh12 has joined #openstack-nova | 16:54 | |
| *** felipemonteiro has quit IRC | 16:54 | |
| mriedem | jamespage: yeah just reviewed it | 16:55 |
| mriedem | left some comments | 16:55 |
| mriedem | there are other places it's going to fail because you're not passing that flag, like fetch_to_raw | 16:55 |
| openstackgerrit | Merged openstack/nova master: Use symbolic names for capabilities, expand sys_admin context. https://review.openstack.org/504193 | 16:55 |
| mriedem | it really becomes a lot of whack a mole | 16:55 |
| *** slaweq has joined #openstack-nova | 16:56 | |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the AllocationList object https://review.openstack.org/502158 | 16:56 |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the Allocation object https://review.openstack.org/502159 | 16:56 |
| dansmith | mriedem: jamespage: Is this the fix for the live migration job? | 16:57 |
| jamespage | mriedem: yeah - felt like pulling at a ball of string | 16:58 |
| jamespage | dansmith: a start on at least | 16:58 |
| dansmith | okay, I feel like we need a quicker resolution in the meantime.. are we reverting the repo for devstack in the interim or something? | 16:58 |
| dansmith | apologies if I missed it | 16:58 |
| *** gbarros has joined #openstack-nova | 16:59 | |
| *** Apoorva has joined #openstack-nova | 16:59 | |
| mriedem | dansmith: the revert was merged last night | 16:59 |
| dansmith | oh? I thought I saw fails from this morning | 17:00 |
| *** Apoorva has quit IRC | 17:00 | |
| *** slaweq has quit IRC | 17:00 | |
| mriedem | 3:37am i guess https://review.openstack.org/#/c/505446/ | 17:00 |
| dansmith | okay maybe these ran before that | 17:01 |
| *** rajathagasthya has quit IRC | 17:01 | |
| *** suresh12 has quit IRC | 17:02 | |
| *** rajathagasthya has joined #openstack-nova | 17:02 | |
| *** derekh has quit IRC | 17:03 | |
| *** vvargaszte has joined #openstack-nova | 17:04 | |
| *** slaweq has joined #openstack-nova | 17:06 | |
| openstackgerrit | James Page proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505748 | 17:09 |
| *** slaweq has quit IRC | 17:10 | |
| *** dave-mcc_ has quit IRC | 17:10 | |
| *** dave-mccowan has joined #openstack-nova | 17:11 | |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the InventoryList object https://review.openstack.org/502160 | 17:11 |
| edleafe | cdent: a_) \o b) don't understand the question | 17:12 |
| *** jmlowe has joined #openstack-nova | 17:12 | |
| openstackgerrit | Merged openstack/nova stable/pike: Add @targets_cell for live_migrate_instance method in conductor https://review.openstack.org/505285 | 17:12 |
| cdent | edleafe: I’m trying to confirm that that’s the correct for review | 17:12 |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the Inventory object https://review.openstack.org/502161 | 17:12 |
| edleafe | cdent: yes, that's the one | 17:12 |
| cdent | thanks | 17:13 |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the ResourceProviderList object https://review.openstack.org/502162 | 17:13 |
| openstackgerrit | Merged openstack/nova master: [placement] Unregister the ResourceProvider object https://review.openstack.org/502163 | 17:14 |
| *** slaweq has joined #openstack-nova | 17:16 | |
| openstackgerrit | Merged openstack/nova master: [placement] Removing versioning from resource_provider objects https://review.openstack.org/502164 | 17:16 |
| *** rajathagasthya has left #openstack-nova | 17:17 | |
| tasker | mriedem: the only entry in the nova-conductor is a reply from the scheduler saying "no valid host was found". the scheduler shows "host [u'compute-2'] fails" but doesn't explain _why_ it failed. and the computes have no record or log of the request because it never gets past the scheduler. | 17:19 |
| *** ociuhandu has joined #openstack-nova | 17:19 | |
| *** penick has quit IRC | 17:20 | |
| mriedem | tasker: the scheduler will dump the filter it failed on | 17:21 |
| mriedem | i'd have to see if that's logged at info or debug | 17:21 |
| melwitt | tasker: was the target host removed by a scheduler filter or are you saying the request was sent to the target compute host and then failed there? | 17:21 |
| *** slaweq has quit IRC | 17:22 | |
| melwitt | if the former, the debug logs should show which filter removed the host from consideration. if the latter, the nova-compute logs should contain some message about why the request failed | 17:22 |
| mriedem | tasker: you should see something like this at INFO level | 17:22 |
| mriedem | LOG.info(_LI("Filter %s returned 0 hosts"), cls_name) | 17:22 |
| mriedem | so for that request, figure out which filter kicked it out | 17:22 |
| mriedem | you should also see something like, "Filtering removed all hosts for the request with" | 17:23 |
| mriedem | if you have INFO level logging enabled | 17:24 |
| tasker | Filter results: ['RetryFilter: (start: 1, end: 0)'] | 17:24 |
| tasker | ok .. after your explanation, that line makes sense. | 17:24 |
| melwitt | it sounds like the request landed on a compute host and then failed and came back to the scheduler to retry a different host, and then it failed with NoValidHost | 17:25 |
| mriedem | i don't think live migration actually does a retry from the compute | 17:25 |
| mriedem | conductor will retry if the pre-migration checks on the chosen target host fail | 17:25 |
| *** penick has joined #openstack-nova | 17:25 | |
| mriedem | we only retry from compute -> conductor for (1) initial create and (2) cold migrate/resize | 17:26 |
| mriedem | this is where conductor is asking the scheduler for a host during live migrate https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L274 | 17:26 |
| tasker | other migrations succeed. | 17:27 |
| tasker | just not this one. | 17:27 |
| *** namnh has quit IRC | 17:27 | |
| melwitt | there should be some message about the request in the compute log. if filtering didn't remove the target host, then it must have gone to the host and then failed | 17:28 |
| tasker | on a successful migrtion, RetryFilter returns 1 host. | 17:28 |
| *** namnh has joined #openstack-nova | 17:28 | |
| mriedem | tasker: how many hosts do you have? does the instance have anything special about it, like pci requests or numa affinity? | 17:29 |
| tasker | on the instance that fails to live-migrate there is no log on the target host. filtering removes _all_ targets regardless of which host I send it to. | 17:29 |
| melwitt | RetryFilter removes previously tried hosts from consideration. so if it removes anything, that means it already tried to run it on the host it selected last time | 17:29 |
| tasker | 3; no. it and all other instances were built to the same requirements. | 17:29 |
| tasker | melwitt: does that have a memory? or is each migration request independent of the previous? | 17:30 |
| melwitt | each request is independent | 17:30 |
| *** vvargaszte has quit IRC | 17:30 | |
| *** mvk has joined #openstack-nova | 17:31 | |
| tasker | the only thing different is that this instance was built while the cluster was at M. now my cluster is at N. | 17:31 |
| *** Swami has joined #openstack-nova | 17:31 | |
| melwitt | I would grep for the request-id of the failing live migration request in nova-conductor and nova-compute logs and see if there's anything about it | 17:32 |
| *** jpena is now known as jpena|off | 17:32 | |
| *** namnh has quit IRC | 17:32 | |
| tasker | I am live watching all the logs. | 17:32 |
| tasker | conductor says "no valid host" because that's what scheduler says. | 17:32 |
| mriedem | i'm wondering if the retry filter shouldn't be ignored here... | 17:32 |
| tasker | if i send another instance to the same target, retryfilter passes the host. | 17:33 |
| melwitt | RetryFilter removing a host implies that it tried to run the pre-migration on a host already | 17:33 |
| *** bigdogstl has joined #openstack-nova | 17:33 | |
| melwitt | so something is weird here | 17:34 |
| tasker | nothing logged at the compute at INFO or lower. | 17:34 |
| mriedem | i'm wondering if the original instance request spec has retries set on it, and that's goofing up the retry filter during live migration | 17:34 |
| tasker | something I could see in the database? | 17:34 |
| mriedem | yeah, you should be able to pull the serialized json request spec out of the db | 17:35 |
| *** dtantsur is now known as dtantsur|afk | 17:35 | |
| mriedem | select spec from nova_api.request_specs where instance_uuid=xyz; | 17:36 |
| mriedem | melwitt: this is where we get the request spec during live migration https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L227 | 17:36 |
| *** gszasz has quit IRC | 17:36 | |
| mriedem | we don't set a retry attribute on the request spec | 17:36 |
| *** yassine has quit IRC | 17:36 | |
| mriedem | we pass it along from the api if we are able to look it up https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3860 | 17:37 |
| mriedem | which would be the original request spec from when we built the instance, which would have retry stuff set on it | 17:37 |
| mriedem | if that wasn't set, we'd ignore it here https://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L31 | 17:38 |
| *** slaweq has joined #openstack-nova | 17:38 | |
| *** bigdogstl has quit IRC | 17:38 | |
| mriedem | which, this is all kinds of weird because the conductor task is handling it's own retry logic https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L344 | 17:38 |
| mriedem | which tells me, when doing a live migration, we don't want to even ask the RetryFilter | 17:38 |
| mriedem | bauzas: are you around? | 17:39 |
| mriedem | tasker: if you can get that request spec json blob out of the db, that should tell us if the retry attribute is set and what hosts and how many times it's been retried during the initial build | 17:39 |
| *** sdague has quit IRC | 17:40 | |
| *** suresh12 has joined #openstack-nova | 17:41 | |
| tasker | mriedem: yes, "retry" is set and is populated with both hypervisors that I'm trying to send it to. | 17:41 |
| tasker | would you like to see it? | 17:42 |
| mriedem | sure, throw it in a paste | 17:42 |
| *** slaweq has quit IRC | 17:42 | |
| mriedem | i think this would actually fix the problem http://paste.openstack.org/show/621556/ | 17:46 |
| melwitt | mriedem: so the first time you build an instance, if it fails host X, it will save that to the RequestSpec so host X will never be tried again ever? I didn't know it persisted something like that | 17:46 |
| mriedem | oops one issue there | 17:46 |
| tasker | http://paste.openstack.org/show/621557/ | 17:47 |
| mriedem | melwitt: yeah | 17:47 |
| *** slaweq has joined #openstack-nova | 17:48 | |
| mriedem | melwitt: i think this https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L583 | 17:48 |
| mriedem | conductor build_instances is called from compute on a reschedule | 17:49 |
| melwitt | well, I learned a thing. I had thought retries were a locally tracked thing per request | 17:49 |
| mriedem | and will put the chosen host in the retry object's list of hosts it's tried | 17:49 |
| mriedem | request spec is the persisted object that keeps on giving | 17:49 |
| mriedem | even when you don't want the gift | 17:49 |
| mriedem | so i think this is the fix http://paste.openstack.org/show/621558/ | 17:49 |
| melwitt | yeah. I wouldn't have chosen to track retries _forever_. I thought RequestSpec would only contain the original request requirements so they could be honored in the future | 17:50 |
| mriedem | just like how we don't want the original forced host/node during live migration, we don't want the original retry hosts either probably | 17:50 |
| *** Apoorva has joined #openstack-nova | 17:50 | |
| tasker | mriedem: I'm going to try your patch out. | 17:51 |
| *** Apoorva has quit IRC | 17:51 | |
| mriedem | tasker: yeah so the request spec had 2 original attempts, on compute-3.openstack.local and compute-2.openstack.local | 17:52 |
| tasker | both targets I'm trying to send it to. | 17:52 |
| mriedem | ok | 17:52 |
| mriedem | and that's why this fails https://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L44 | 17:52 |
| *** yamahata has quit IRC | 17:52 | |
| tasker | which part of nova would I apply that to? compute, conductor, or scheduler? | 17:52 |
| mriedem | conductor, but note, that diff is against master branch code | 17:52 |
| tasker | let me see if I can adapt to the branch that I'm using. | 17:53 |
| mriedem | http://paste.openstack.org/show/621559/ is latest stable/newton | 17:54 |
| *** slaweq has quit IRC | 17:54 | |
| tasker | cool, thanks. I'm going to step out for some "fresh air" and then apply this. | 17:55 |
| mriedem | it does stand to reason that if this instance failed to build originally on those 2 hosts, that live migrating it there might fail too...but we don't know why it originally failed, could have been a resource claim issue at the time | 17:55 |
| tasker | that's a fair observation. | 17:55 |
| melwitt | yeah, often it's a failed claim. and also what if that compute host is eventually replaced over the lifetime of the cluster, making it a fresh candidate for several instances that might still avoid it because they once failed to build there back when it was a different machine | 17:57 |
| *** tesseract has quit IRC | 17:59 | |
| *** acormier has quit IRC | 18:01 | |
| tasker | live migration successful | 18:06 |
| tasker | mriedem and melwitt -- thank you so much for helping to figure out what was wrong. | 18:08 |
| *** gbarros has quit IRC | 18:10 | |
| *** slaweq has joined #openstack-nova | 18:10 | |
| mriedem | nice | 18:11 |
| mriedem | tasker: want to open a bug? i have a fix with the test locally | 18:11 |
| tasker | sure. let me collect my notes. | 18:11 |
| cdent | dansmith: responded to some of your comments on https://review.openstack.org/#/c/500410/ . you semi-accidentally identified a separate problem. The nullable thing I’m not quite sure how to proceed, depending on what we want to do. | 18:12 |
| *** ijw has joined #openstack-nova | 18:13 | |
| *** MVenesio has joined #openstack-nova | 18:14 | |
| mriedem | cdent: congratulations, you have the first complete blueprint in queens https://blueprints.launchpad.net/nova/+spec/placement-deregister-objects | 18:14 |
| *** ralonsoh has quit IRC | 18:14 | |
| *** slaweq has quit IRC | 18:15 | |
| dansmith | cdent: replied, I don't think there's an issue.. just make those not nullable and (separately) always set them and I think we're good | 18:15 |
| *** Yikun__ has quit IRC | 18:15 | |
| *** Apoorva has joined #openstack-nova | 18:16 | |
| *** yamamoto has quit IRC | 18:17 | |
| *** slaweq has joined #openstack-nova | 18:20 | |
| *** Sukhdev_ has joined #openstack-nova | 18:20 | |
| *** lucasxu has joined #openstack-nova | 18:21 | |
| cdent | dansmith: except that they are only ever used on input, never output, so why bother reading them? | 18:24 |
| tasker | mriedem: https://bugs.launchpad.net/nova/+bug/1718512 | 18:25 |
| openstack | Launchpad bug 1718512 in OpenStack Compute (nova) "migration fails if instance build failed on destination host" [Undecided,New] | 18:25 |
| mriedem | thanks | 18:25 |
| dansmith | cdent: because it's (effectively) free, it makes the object consistent | 18:25 |
| tasker | I hope that summary / description is clear enough. | 18:25 |
| *** felipemonteiro__ has quit IRC | 18:26 | |
| *** slaweq has quit IRC | 18:26 | |
| *** felipemonteiro__ has joined #openstack-nova | 18:26 | |
| *** READ10 has quit IRC | 18:28 | |
| cdent | dansmith: okay, I’m happy to do that. I’m not sure why consistency matters _now_ but if we’d like it as a general rule, that’s fine. | 18:28 |
| *** felipemonteiro has joined #openstack-nova | 18:28 | |
| *** namnh has joined #openstack-nova | 18:28 | |
| *** kbaegis has quit IRC | 18:29 | |
| dansmith | cdent: clearly it doesn't have a user now, but we're an abstract model on top of the data store and unless there is a reason not to, we should do that thing, IMHO | 18:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Ignore original retried hosts when live migrating https://review.openstack.org/505771 | 18:30 |
| mriedem | tasker: ^ i'm going to wip that and send something to the dev/ops lists for this, because if we change this for live migration, we also have to do it for cold migrate/evacuate and unshelve | 18:30 |
| mriedem | melwitt: ^ | 18:30 |
| melwitt | ack | 18:30 |
| cdent | dansmith: does YAGNI count as a reason not to? | 18:30 |
| *** felipemonteiro__ has quit IRC | 18:31 | |
| dansmith | cdent: I dunno what GNI is in there, but if you think there's a reason not to, then don't do it and we can just leave my -1 on there | 18:31 |
| cdent | I’ve just always been brought up on the idea of not doing things unless there is a reason to do so, but as I said, I’m happy to do it and will do it. | 18:32 |
| *** Sukhdev_ has quit IRC | 18:33 | |
| *** namnh has quit IRC | 18:33 | |
| mriedem | if they are in the same table it seems easy peasy to just load them, it'd be one thing if we were lazy loading a big table join or something, | 18:34 |
| mriedem | but if/when someone needs these, it's going to be weird debugging why they just aren't already in the object when it's read from the db | 18:34 |
| mriedem | like every other object we have | 18:34 |
| dansmith | mriedem: it's a join, but it's super tiny | 18:34 |
| mriedem | oh | 18:34 |
| mriedem | like the usages/consumers table join thing? | 18:35 |
| dansmith | mriedem: like a 1:1 integer join | 18:35 |
| dansmith | yes | 18:35 |
| dansmith | consumers | 18:35 |
| *** felipemonteiro has quit IRC | 18:35 | |
| mriedem | i'm assuming at some point we actually intend on using these fields? | 18:35 |
| *** felipemonteiro has joined #openstack-nova | 18:35 | |
| dansmith | mriedem: if they're fields they should be loadable, and if we're never going to use them that way then maybe we shouldn't have them be fields | 18:36 |
| mriedem | yeah i'd agree with that | 18:36 |
| mriedem | i don't have context here though clearly | 18:36 |
| dansmith | since consumer isn't a top level object in placement, getting them with allocations is the only way we can pull them out otherwise | 18:36 |
| dansmith | unless we're querying for a consumer already | 18:36 |
| dansmith | like if we needed to get all allocations against a resource provider | 18:37 |
| *** kbaegis has joined #openstack-nova | 18:37 | |
| *** slaweq has joined #openstack-nova | 18:42 | |
| *** bigdogstl has joined #openstack-nova | 18:44 | |
| *** Sree has joined #openstack-nova | 18:46 | |
| *** gbarros has joined #openstack-nova | 18:46 | |
| *** slaweq has quit IRC | 18:47 | |
| mriedem | tasker: http://lists.openstack.org/pipermail/openstack-operators/2017-September/014233.html | 18:48 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] manage cache headers https://review.openstack.org/495380 | 18:49 |
| *** Sree has quit IRC | 18:50 | |
| *** bigdogstl has quit IRC | 18:51 | |
| *** slaweq has joined #openstack-nova | 18:52 | |
| *** gbarros has quit IRC | 18:55 | |
| tasker | mriedem: thanks! I'll keep an eye out on these. | 18:55 |
| *** egonzalez has joined #openstack-nova | 18:56 | |
| tasker | mriedem and melwitt: I want to thank you two for taking the time to work with me on my issues for the last few days. your support has been a positive influence in my cloudy days. with your help, I've been able to progress with my upgrade trials. | 18:57 |
| tasker | If you two are ever in the Kansas City area, let me know. first round's on me. | 18:59 |
| *** slaweq has quit IRC | 18:59 | |
| *** felipemonteiro__ has joined #openstack-nova | 19:00 | |
| *** felipemonteiro has quit IRC | 19:04 | |
| mriedem | tasker: cool, you're welcome. you hit some weird issues so i'm glad we flushed those out and have fixes | 19:05 |
| *** yamahata has joined #openstack-nova | 19:09 | |
| bauzas | mriedem: back now, wazzup ? | 19:13 |
| mriedem | bauzas: http://lists.openstack.org/pipermail/openstack-operators/2017-September/014233.html | 19:14 |
| * bauzas looking | 19:16 | |
| *** yamamoto has joined #openstack-nova | 19:17 | |
| bauzas | holy shit | 19:18 |
| bauzas | mriedem: honestly, no reason to persist that | 19:18 |
| bauzas | mriedem: if some host wasn't accepted for an instance create, there is no reason it will be excluded for a live migration | 19:18 |
| bauzas | at least, we should still verify it by the scheduler | 19:19 |
| bauzas | mriedem: but here, the main problem is that I should possibly modify the ReqSpec object to have a specific way to say which fields are needed to be persisted and others not | 19:20 |
| bauzas | mriedem: that's something we didn't discussed with alaski when he provided the .save() method | 19:20 |
| melwitt | tasker: yep, thanks for helping us fix those bugs. and I learned yet more things I didn't know about live migration and retries | 19:21 |
| *** yamamoto has quit IRC | 19:26 | |
| mriedem | bauzas: yeah i was thinking about renaming the reset_forced_destinations() method to something like reset_for_move() and that would reset forced_hosts/forced_nodes/retry | 19:27 |
| mriedem | so it could be handled for live migrate/cold migrate/unshelve and evacuate in the same place | 19:28 |
| bauzas | mriedem: yeah, or just modify _get_update_primitives to disallow the non-needed fields | 19:29 |
| bauzas | mriedem: like using a propery that'd give you which fields are non-persistent | 19:29 |
| bauzas | that way a call to .create() or .save() would avoid persist those | 19:29 |
| bauzas | persisting* | 19:29 |
| mriedem | that won't help us for any existing req specs | 19:30 |
| bauzas | but that wouldn't solve existing specs | 19:30 |
| bauzas | yeah | 19:30 |
| bauzas | okay, I think we could just rename reset_forced_dests() | 19:30 |
| bauzas | and modify the comments of course | 19:30 |
| mriedem | yeah | 19:31 |
| mriedem | to fix all 4 move operations | 19:31 |
| mriedem | i'm going to want to backport this too | 19:31 |
| bauzas | want me to help on that or you feel enough brave ? | 19:31 |
| mriedem | i'm brave | 19:31 |
| bauzas | good boy | 19:31 |
| mriedem | anything that can help me procrastinate from reviewing dansmith's efficient multi-cell instance list / sort thing | 19:31 |
| mriedem | :) | 19:31 |
| bauzas | honestly, I think it would be a good opportunity for discussing which fields shouldn't be persisted | 19:32 |
| dansmith | mriedem: my set should be pretty easy to follow I think | 19:32 |
| bauzas | like requested_destination | 19:32 |
| dansmith | mriedem: I tried really hard to add functionality in layers | 19:32 |
| mriedem | dansmith: yeah, i'm going to hit it now - i'll wait to deal with this reqspec stuff for tomorrow morning | 19:32 |
| mriedem | gives me something to look forward to | 19:32 |
| *** ijw has quit IRC | 19:33 | |
| *** rmcall has joined #openstack-nova | 19:33 | |
| *** slaweq has joined #openstack-nova | 19:34 | |
| *** suresh12 has quit IRC | 19:35 | |
| *** awaugama has quit IRC | 19:37 | |
| cdent | dansmith: there’s loads of pre-existings tests lying around that don’t require AllocationList to have a project and user. That makes it feel like more effort than it is worth to start requiring it in the Allocation object, so what I’m thinking of doing is not requiring it, but if they are there in the DB, loading it up when reading the object. Concur? | 19:40 |
| dansmith | cdent: I'm not sure what you mean.. surely we require it from the API now right? | 19:41 |
| dansmith | I guess one of the early microversions didn't require it so we _could_ have things in the DB that don't? | 19:41 |
| mriedem | consumers requires project_id/user_id in the API in the latest microversion, | 19:41 |
| mriedem | allocations doesn't | 19:41 |
| *** slaweq has quit IRC | 19:42 | |
| melwitt | I thought PUT allocations required project/user | 19:42 |
| cdent | they do | 19:42 |
| cdent | but we don’t enforce that at the object level | 19:42 |
| cdent | only the http api level | 19:42 |
| dansmith | cdent: we should have migrated any existing allocations explicitly or incidentally | 19:43 |
| cdent | what I’m saying is: to make the change to requiring things will change a pile of tests | 19:43 |
| dansmith | cdent: if the http api always forces it (which it should) then we should raise an error when we hit one in the DB that doesn't, not just return an object with those things missing | 19:43 |
| melwitt | the existing allocations get overwritten upon update with newer code to have project/user | 19:43 |
| melwitt | (periodic update) | 19:43 |
| dansmith | melwitt: right, that's what I meant by incidentally | 19:44 |
| melwitt | k | 19:44 |
| cdent | so the question I have two things: should I change all the tests and make the enforcement all over the place, or should I just make sure that we read the data when the data is there (which it always will be for real-world allocations) | 19:45 |
| *** slaweq has joined #openstack-nova | 19:45 | |
| *** suresh12 has joined #openstack-nova | 19:45 | |
| dansmith | if the tests don't mirror reality, then we should update the tests I think | 19:45 |
| dansmith | and maybe we should have a blocker migration that ensures that all the things in the DB have gotten user/project fields going forward to force the issue or something | 19:46 |
| dansmith | so we can remove the nullable on the schema Imean | 19:46 |
| *** felipemonteiro__ has quit IRC | 19:47 | |
| cdent | out of curiosity: why’s that matter? | 19:47 |
| dansmith | we put constraints in the database to make sure we don't store data that violates the schema we want right? | 19:48 |
| *** liusheng has quit IRC | 19:48 | |
| dansmith | like, you can't have an info_cache for an instance that doesn't exist, and you can't have more than one instance with the same id | 19:48 |
| *** felipemonteiro has joined #openstack-nova | 19:48 | |
| *** liusheng has joined #openstack-nova | 19:48 | |
| cdent | right, and we have two forms that the database is happy to support | 19:48 |
| dansmith | if user/project are not optional, then they should not be optional | 19:48 |
| cdent | they are optional, for hosts that are long lived | 19:49 |
| cdent | as in, started life before all this | 19:49 |
| dansmith | that doesn't mean they're optional, that means some data is in an old format | 19:49 |
| dansmith | hence the migration, blocker, etc to make sure we don't have to deal with two formats forever | 19:50 |
| dansmith | if we want one format, we should get all our data into that one format | 19:50 |
| *** liverpooler has joined #openstack-nova | 19:52 | |
| cdent | Well, I guess I can do all that, but it’s not going to happen tonight I’ve run out of brain | 19:53 |
| *** bigdogstl has joined #openstack-nova | 19:58 | |
| *** gbarros has joined #openstack-nova | 20:00 | |
| *** smatzek has quit IRC | 20:01 | |
| *** bigdogstl has quit IRC | 20:02 | |
| *** pchavva has quit IRC | 20:06 | |
| *** gbarros has quit IRC | 20:06 | |
| bauzas | mriedem: dear god, related issue https://bugs.launchpad.net/nova/+bug/1718455 | 20:07 |
| openstack | Launchpad bug 1718455 in OpenStack Compute (nova) "[pike] Nova host disable and Live Migrate all instances fail." [Undecided,New] | 20:07 |
| bauzas | mriedem: I'm working on the fix now | 20:07 |
| bauzas | mriedem: but we should honestly just persist num_instances=1 for move ops | 20:07 |
| bauzas | given we get the ReqSpec by calling the instance UUID | 20:08 |
| mriedem | bauzas: talked at length about that bug this morning | 20:08 |
| mriedem | i would like to see a functional test for the actual scenario, since a related fix made in pike missed that part | 20:08 |
| bauzas | I know, I co-authored that fix | 20:09 |
| *** gouthamr has quit IRC | 20:09 | |
| mriedem | i think you authored it.. | 20:09 |
| bauzas | I just passed a new revision but meh | 20:09 |
| bauzas | and yeah, we could functional test it | 20:09 |
| mriedem | this https://review.openstack.org/#/c/491439/ | 20:09 |
| bauzas | lke a regression chnage | 20:10 |
| bauzas | oh fun, it was co-authored because of a pep8 fix :) | 20:10 |
| bauzas | ah nvm, got it :) | 20:11 |
| bauzas | anyway, yeah I can work on a regression test | 20:11 |
| bauzas | mriedem: or a func test, as you want | 20:12 |
| mriedem | i don't know if it was regressed in pike or not | 20:12 |
| mriedem | or if this was a latent bug before pike | 20:12 |
| bauzas | mriedem: that should have been regression when we merged the claims stuff | 20:15 |
| bauzas | mriedem: because before that, when you were asking for 10 instances, it was possibly returning you 10 times the same host | 20:15 |
| bauzas | mmm, wait | 20:16 |
| *** jmlowe has quit IRC | 20:22 | |
| *** armax has quit IRC | 20:23 | |
| *** crushil_ has joined #openstack-nova | 20:25 | |
| bauzas | mriedem: holy fuck, we introduced the problem with https://github.com/openstack/nova/commit/2bd7df84 | 20:26 |
| bauzas | whack-a-mole | 20:26 |
| bauzas | we changed _schedule to return the number of hosts per instances | 20:27 |
| bauzas | so it's now returning 1 host | 20:27 |
| bauzas | for a live-migration | 20:27 |
| *** crushil has quit IRC | 20:27 | |
| bauzas | but we haven't fixed the caller, so it's still awaiting 10 | 20:27 |
| bauzas | so we're fscked | 20:27 |
| bauzas | definitely a pike regression then | 20:27 |
| bauzas | but we could write a func test anyway | 20:28 |
| mriedem | yeah so just write a functional regression test like we have for others | 20:29 |
| *** kbaegis has quit IRC | 20:29 | |
| mriedem | should be pretty simple, create 2 computes and 2 instances forced to 1 compute, then live migrate one of the instances and it should fail with novalidhost | 20:29 |
| *** crushil_ has quit IRC | 20:30 | |
| *** namnh has joined #openstack-nova | 20:30 | |
| mriedem | melwitt: dansmith: cells meeting rodeo in 30 minutes | 20:30 |
| *** crushil_ has joined #openstack-nova | 20:30 | |
| dansmith | yup | 20:30 |
| mriedem | trying to wrap my head around this heapq craziness | 20:31 |
| dansmith | do you want to do this one as a hangout? | 20:31 |
| dansmith | I could do some dansplaining | 20:31 |
| dansmith | see what I did there? | 20:31 |
| mriedem | how could i not | 20:31 |
| melwitt | heh | 20:32 |
| *** catintheroof has quit IRC | 20:32 | |
| dansmith | mriedem: so, hangout? or have you seen enough of me for six months? | 20:33 |
| *** ijw has joined #openstack-nova | 20:33 | |
| *** namnh has quit IRC | 20:34 | |
| *** MVenesio has quit IRC | 20:35 | |
| *** ijw has quit IRC | 20:35 | |
| *** ijw has joined #openstack-nova | 20:36 | |
| *** crushil_ has quit IRC | 20:36 | |
| mriedem | i'm gearing up | 20:36 |
| melwitt | for a hangout? | 20:37 |
| mriedem | and the apocalypse | 20:37 |
| mriedem | but more a hangout right now yes | 20:37 |
| melwitt | :) | 20:37 |
| mriedem | https://hangouts.google.com/call/QDaiUUdRHaNIQmbJi5NVAAkE | 20:38 |
| dansmith | oh now? | 20:39 |
| mriedem | yeah | 20:39 |
| mriedem | my kid will be home in 20 making a bunch of noise | 20:39 |
| *** corey_ has joined #openstack-nova | 20:42 | |
| *** cleong has quit IRC | 20:43 | |
| mriedem | process_sort_params in the db api | 20:43 |
| mriedem | default_keys=['created_at', 'id'], | 20:43 |
| *** Sree has joined #openstack-nova | 20:46 | |
| *** csuttles has quit IRC | 20:47 | |
| *** Sree has quit IRC | 20:51 | |
| *** penick has quit IRC | 20:51 | |
| *** jistr has quit IRC | 20:52 | |
| bauzas | dansmith: mriedem: oh fun, rediscovered https://review.openstack.org/#/c/446446/5/specs/pike/approved/az-block-name-update.rst | 20:58 |
| bauzas | I should copyright that :) | 20:59 |
| *** pcaruana has quit IRC | 20:59 | |
| *** penick has joined #openstack-nova | 21:00 | |
| *** jheroux has quit IRC | 21:00 | |
| *** felipemonteiro__ has joined #openstack-nova | 21:02 | |
| *** Sukhdev has quit IRC | 21:03 | |
| *** thorst has quit IRC | 21:03 | |
| *** rmcall has quit IRC | 21:04 | |
| *** rmcall has joined #openstack-nova | 21:04 | |
| *** kbaegis has joined #openstack-nova | 21:05 | |
| *** felipemonteiro has quit IRC | 21:06 | |
| *** esberglu has quit IRC | 21:07 | |
| *** thorst has joined #openstack-nova | 21:07 | |
| *** xyang1 has quit IRC | 21:11 | |
| *** thorst has quit IRC | 21:12 | |
| *** tidwellr has quit IRC | 21:12 | |
| *** slaweq has quit IRC | 21:14 | |
| *** edmondsw has quit IRC | 21:14 | |
| *** edmondsw has joined #openstack-nova | 21:14 | |
| *** rmcall has quit IRC | 21:16 | |
| *** slaweq has joined #openstack-nova | 21:16 | |
| *** edmondsw_ has joined #openstack-nova | 21:17 | |
| *** corey_ has quit IRC | 21:17 | |
| *** rmcall has joined #openstack-nova | 21:17 | |
| *** edmondsw has quit IRC | 21:19 | |
| mriedem | dansmith: talking about this https://github.com/openstack/nova/commit/c4820305d2f9ee8d62bcc708baf3fa6dfe7ca960 | 21:20 |
| *** edmondsw_ has quit IRC | 21:21 | |
| *** esberglu has joined #openstack-nova | 21:21 | |
| *** esberglu_ has joined #openstack-nova | 21:24 | |
| *** esberglu has quit IRC | 21:24 | |
| *** armax has joined #openstack-nova | 21:28 | |
| *** lucasxu has quit IRC | 21:29 | |
| *** dave-mccowan has quit IRC | 21:32 | |
| *** Sukhdev has joined #openstack-nova | 21:34 | |
| *** Sukhdev_ has joined #openstack-nova | 21:34 | |
| *** csuttles has joined #openstack-nova | 21:37 | |
| *** kenperkins has joined #openstack-nova | 21:39 | |
| *** csuttles has quit IRC | 21:39 | |
| efried | stephenfin Ic05c2c8364e015f6878b0bc25449216624568ad5 ouch. This means folks who paid attention to the deprecation and moved to [vnc]vncserver_proxyclient_address are now busted, without a deprecation period on the old-name-in-the-new-group. | 21:42 |
| efried | Arguably the rename should have been done as part of the move. But it warn't. mriedem Can I get a ruling ^ ? (https://review.openstack.org/#/c/498387/) | 21:44 |
| *** edmondsw has joined #openstack-nova | 21:45 | |
| *** edmondsw has quit IRC | 21:49 | |
| mriedem | wuh | 21:49 |
| mriedem | (1) vncserver_listen was in the DEFAULT group, and moved to the [vnc] group, (2) vncserver_listen was in the [vnc] group and renamed to server_listen in the [vnc] group | 21:51 |
| mriedem | so now [DEFAULT]vncserver_listen just won't work, right? | 21:51 |
| mriedem | or will it? | 21:51 |
| melwitt | did anyone figure out how to get gerrit email notifications again? | 21:52 |
| mriedem | efried: i'm having a hard time following, do you have a bug report or a test to recreate the issue? | 21:53 |
| efried | mriedem Yeah, I'm writing up the bug report, and I have the fix. | 21:53 |
| *** avolkov has quit IRC | 21:54 | |
| efried | mriedem stephenfin https://bugs.launchpad.net/nova/+bug/1718545 | 21:58 |
| openstack | Launchpad bug 1718545 in OpenStack Compute (nova) "[vnc]vncserver_proxyclient_address and [vnc]vncserver_listen removed without deprecation" [Undecided,New] | 21:58 |
| *** burt has quit IRC | 22:02 | |
| *** slaweq has quit IRC | 22:03 | |
| *** thorst has joined #openstack-nova | 22:06 | |
| *** baoli has quit IRC | 22:06 | |
| mriedem | thanks | 22:09 |
| mriedem | i'd be fine with a revert honestly | 22:09 |
| mriedem | the constant random config option changes annoy me abit | 22:10 |
| *** thorst has quit IRC | 22:10 | |
| openstackgerrit | Merged openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194 | 22:10 |
| *** Apoorva_ has joined #openstack-nova | 22:11 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Restore [vnc]vnc*; remove [DEFAULT] equivalents https://review.openstack.org/505831 | 22:12 |
| efried | mriedem stephenfin ^ | 22:12 |
| *** slaweq has joined #openstack-nova | 22:13 | |
| *** Apoorva has quit IRC | 22:13 | |
| openstackgerrit | Merged openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195 | 22:14 |
| efried | cdent too, as a reviewer of the original. | 22:14 |
| openstackgerrit | Merged openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437 | 22:15 |
| *** catintheroof has joined #openstack-nova | 22:15 | |
| cdent | efried: I’m too asleep to give that a good review now, and I guess must have been too asleep when I reviewed it the first time | 22:15 |
| efried | You're not alone :) | 22:15 |
| mriedem | efried: commented | 22:16 |
| cdent | I’ve put it on my list for tomorrow | 22:16 |
| mriedem | dansmith: you hate methods with 100 kwargs?! | 22:17 |
| dansmith | mriedem: yes, yes I do | 22:17 |
| dansmith | mriedem: especially when they're ALL optional | 22:17 |
| mriedem | like _provision_instances, build_instances, build_and_run_instance, rebuild_instance | 22:17 |
| mriedem | ETC?! | 22:17 |
| dansmith | yeah, all of ours | 22:18 |
| *** slaweq has quit IRC | 22:18 | |
| mriedem | does java allow default values yet? | 22:18 |
| dansmith | heh idk | 22:19 |
| mriedem | guess not | 22:19 |
| mriedem | i remember missing that when going from c++ to java, and then being pleasantly surprised to see it again when going from java to python | 22:19 |
| mriedem | now, what would go do | 22:20 |
| tasker | require an interface. | 22:20 |
| openstackgerrit | Merged openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343 | 22:21 |
| dansmith | tasker: I dunno if you're making fun of python or go with that statement, but... don't tell me :) | 22:21 |
| openstackgerrit | Merged openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | 22:21 |
| tasker | . D | 22:22 |
| openstackgerrit | Merged openstack/nova master: Squash dacnet_admin privsep context. https://review.openstack.org/504805 | 22:22 |
| mriedem | now someone needs to backport all of these https://review.openstack.org/#/q/topic:ironic-offline-migration+(status:open+OR+status:merged) | 22:22 |
| *** slaweq has joined #openstack-nova | 22:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839 | 22:25 |
| efried | mriedem Replied :(letting you know since we don't seem to be getting emails): | 22:26 |
| *** esberglu_ has quit IRC | 22:27 | |
| *** chyka has quit IRC | 22:27 | |
| *** slaweq has quit IRC | 22:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646 | 22:28 |
| *** chyka has joined #openstack-nova | 22:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708 | 22:28 |
| *** cfriesen has quit IRC | 22:28 | |
| *** yamamoto has joined #openstack-nova | 22:28 | |
| *** cfriesen has joined #openstack-nova | 22:28 | |
| *** felipemonteiro__ has quit IRC | 22:29 | |
| *** namnh has joined #openstack-nova | 22:31 | |
| *** thorst has joined #openstack-nova | 22:33 | |
| *** namnh has quit IRC | 22:36 | |
| *** thorst has quit IRC | 22:38 | |
| *** thorst has joined #openstack-nova | 22:39 | |
| *** thorst has quit IRC | 22:40 | |
| *** Sree has joined #openstack-nova | 22:47 | |
| *** ijw has quit IRC | 22:47 | |
| *** ijw has joined #openstack-nova | 22:48 | |
| mriedem | i haven't read my gerrit email in a looong time | 22:49 |
| mriedem | couldn't keep up with it | 22:49 |
| *** lyan has quit IRC | 22:51 | |
| *** Sree has quit IRC | 22:52 | |
| *** jistr has joined #openstack-nova | 22:53 | |
| dansmith | yeah, gerrit emails are borked today, known issue | 22:57 |
| *** penick has quit IRC | 23:00 | |
| *** jistr has quit IRC | 23:02 | |
| dansmith | here comes | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort() https://review.openstack.org/505391 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 23:03 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 23:03 |
| *** jistr has joined #openstack-nova | 23:04 | |
| *** egonzalez has quit IRC | 23:10 | |
| *** claudiub|2 has quit IRC | 23:10 | |
| *** gouthamr has joined #openstack-nova | 23:11 | |
| cfriesen | from what I understand nova-api handles both the normal API endpoint and the metadata endpoint...how does the multiple calls to launcher.launch_service() end up using the same process(es)? | 23:12 |
| *** catintheroof has quit IRC | 23:19 | |
| *** ijw has quit IRC | 23:26 | |
| *** gouthamr has quit IRC | 23:27 | |
| tasker | did review.openstack just break? | 23:31 |
| tasker | nevermind - i can get to it again. | 23:34 |
| *** ijw has joined #openstack-nova | 23:39 | |
| *** thorst has joined #openstack-nova | 23:41 | |
| *** hongbin has quit IRC | 23:44 | |
| *** zhenq has quit IRC | 23:44 | |
| *** thorst has quit IRC | 23:46 | |
| *** bigdogstl has joined #openstack-nova | 23:49 | |
| *** namnh has joined #openstack-nova | 23:50 | |
| *** itlinux has joined #openstack-nova | 23:53 | |
| *** bigdogstl has quit IRC | 23:54 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410 | 23:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 23:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!