| *** tbachman has joined #openstack-nova | 00:02 | |
| *** yingjun has joined #openstack-nova | 00:07 | |
| *** yangyape_ has quit IRC | 00:08 | |
| *** yangyapeng has joined #openstack-nova | 00:09 | |
| *** kylek3h has quit IRC | 00:10 | |
| *** yangyapeng has quit IRC | 00:13 | |
| *** catintheroof has joined #openstack-nova | 00:19 | |
| *** thorst has joined #openstack-nova | 00:20 | |
| *** mingyu has joined #openstack-nova | 00:27 | |
| *** mingyu has quit IRC | 00:31 | |
| *** esberglu has joined #openstack-nova | 00:33 | |
| *** Shunli has joined #openstack-nova | 00:33 | |
| *** vladikr has quit IRC | 00:36 | |
| *** esberglu has quit IRC | 00:37 | |
| *** Apoorva has joined #openstack-nova | 00:41 | |
| *** hieulq has joined #openstack-nova | 00:42 | |
| *** markvoelker has joined #openstack-nova | 00:42 | |
| *** zhurong has joined #openstack-nova | 01:02 | |
| *** yingjun has quit IRC | 01:03 | |
| *** edmondsw has joined #openstack-nova | 01:06 | |
| *** penick has joined #openstack-nova | 01:07 | |
| *** edmondsw has quit IRC | 01:10 | |
| *** yangyapeng has joined #openstack-nova | 01:12 | |
| *** liusheng has quit IRC | 01:14 | |
| *** markvoelker has quit IRC | 01:15 | |
| *** Apoorva has quit IRC | 01:15 | |
| *** yangyapeng has quit IRC | 01:16 | |
| *** tiendc has joined #openstack-nova | 01:19 | |
| *** liusheng has joined #openstack-nova | 01:22 | |
| *** thorst has quit IRC | 01:22 | |
| *** thorst has joined #openstack-nova | 01:23 | |
| *** sree has joined #openstack-nova | 01:27 | |
| *** phuongnh has joined #openstack-nova | 01:28 | |
| *** trungnv has joined #openstack-nova | 01:31 | |
| *** sree has quit IRC | 01:31 | |
| *** slaweq has joined #openstack-nova | 01:33 | |
| *** vladikr has joined #openstack-nova | 01:33 | |
| *** mingyu has joined #openstack-nova | 01:35 | |
| *** thorst has quit IRC | 01:35 | |
| *** catintheroof has quit IRC | 01:37 | |
| *** slaweq has quit IRC | 01:37 | |
| *** yufei has joined #openstack-nova | 01:38 | |
| *** catintheroof has joined #openstack-nova | 01:38 | |
| *** liuyulong has joined #openstack-nova | 01:45 | |
| *** sapd has joined #openstack-nova | 01:49 | |
| *** yamamoto has joined #openstack-nova | 01:54 | |
| *** catintheroof has quit IRC | 01:57 | |
| *** hongbin has joined #openstack-nova | 02:00 | |
| *** penick_ has joined #openstack-nova | 02:11 | |
| *** Tom_ has joined #openstack-nova | 02:11 | |
| *** penick has quit IRC | 02:12 | |
| *** slaweq has joined #openstack-nova | 02:13 | |
| *** slaweq has quit IRC | 02:18 | |
| *** esberglu has joined #openstack-nova | 02:21 | |
| *** yangyapeng has joined #openstack-nova | 02:22 | |
| *** zhouyaguo has joined #openstack-nova | 02:25 | |
| *** yangyapeng has quit IRC | 02:25 | |
| *** esberglu has quit IRC | 02:25 | |
| *** yangyapeng has joined #openstack-nova | 02:25 | |
| *** thorst has joined #openstack-nova | 02:36 | |
| *** itlinux has joined #openstack-nova | 02:37 | |
| *** david-lyle has joined #openstack-nova | 02:38 | |
| *** kevi9132 has joined #openstack-nova | 02:41 | |
| *** kevi9132_ has quit IRC | 02:41 | |
| *** kevi9132 has quit IRC | 02:46 | |
| *** mingyu has quit IRC | 02:51 | |
| *** edmondsw has joined #openstack-nova | 02:54 | |
| *** slaweq has joined #openstack-nova | 02:55 | |
| *** yangyapeng has quit IRC | 02:55 | |
| *** yangyapeng has joined #openstack-nova | 02:56 | |
| *** edmondsw has quit IRC | 02:58 | |
| *** slaweq has quit IRC | 02:59 | |
| *** penick_ has quit IRC | 03:03 | |
| *** baoli has quit IRC | 03:03 | |
| *** baoli has joined #openstack-nova | 03:04 | |
| *** vladikr has quit IRC | 03:06 | |
| *** mingyu has joined #openstack-nova | 03:08 | |
| *** zhurong has quit IRC | 03:08 | |
| *** zhurong has joined #openstack-nova | 03:11 | |
| *** markvoelker has joined #openstack-nova | 03:12 | |
| *** esberglu has joined #openstack-nova | 03:15 | |
| *** thorst has quit IRC | 03:17 | |
| *** thorst has joined #openstack-nova | 03:17 | |
| *** esberglu has quit IRC | 03:20 | |
| *** hamzy has quit IRC | 03:20 | |
| *** mingyu has quit IRC | 03:23 | |
| *** thorst has quit IRC | 03:24 | |
| *** baoli has quit IRC | 03:25 | |
| *** vladikr has joined #openstack-nova | 03:27 | |
| *** itlinux has quit IRC | 03:30 | |
| *** Shunli has quit IRC | 03:35 | |
| *** slaweq has joined #openstack-nova | 03:36 | |
| *** mingyu has joined #openstack-nova | 03:39 | |
| *** gbarros has joined #openstack-nova | 03:39 | |
| *** slaweq has quit IRC | 03:40 | |
| *** gbarros has quit IRC | 03:44 | |
| *** markvoelker has quit IRC | 03:47 | |
| *** trungnv has quit IRC | 03:48 | |
| *** slaweq has joined #openstack-nova | 03:56 | |
| *** trinaths has joined #openstack-nova | 03:59 | |
| *** slaweq has quit IRC | 04:01 | |
| *** hongbin has quit IRC | 04:04 | |
| *** trungnv has joined #openstack-nova | 04:05 | |
| *** Tom_ has quit IRC | 04:06 | |
| *** esberglu has joined #openstack-nova | 04:09 | |
| *** esberglu has quit IRC | 04:13 | |
| *** udesale has joined #openstack-nova | 04:13 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:14 | |
| *** crushil has joined #openstack-nova | 04:14 | |
| *** claudiub has joined #openstack-nova | 04:22 | |
| *** gbarros has joined #openstack-nova | 04:24 | |
| *** thorst has joined #openstack-nova | 04:25 | |
| *** links has joined #openstack-nova | 04:25 | |
| *** Tom has joined #openstack-nova | 04:26 | |
| *** sree has joined #openstack-nova | 04:27 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova-specs master: Strict isolation of group of hosts for image and flavor https://review.openstack.org/381912 | 04:27 |
|---|---|---|
| *** thorst has quit IRC | 04:29 | |
| *** Tom has quit IRC | 04:31 | |
| *** gbarros has quit IRC | 04:31 | |
| *** psachin has joined #openstack-nova | 04:31 | |
| *** sree has quit IRC | 04:31 | |
| *** gbarros has joined #openstack-nova | 04:31 | |
| *** gbarros has quit IRC | 04:36 | |
| *** gbarros has joined #openstack-nova | 04:37 | |
| *** slaweq has joined #openstack-nova | 04:37 | |
| *** mdnadeem has joined #openstack-nova | 04:41 | |
| *** crushil has quit IRC | 04:42 | |
| *** slaweq has quit IRC | 04:42 | |
| *** sree has joined #openstack-nova | 04:44 | |
| *** markvoelker has joined #openstack-nova | 04:44 | |
| *** udesale has quit IRC | 04:48 | |
| *** udesale has joined #openstack-nova | 04:49 | |
| *** gbarros has quit IRC | 04:56 | |
| *** gbarros has joined #openstack-nova | 04:56 | |
| *** gbarros has quit IRC | 05:01 | |
| *** esberglu has joined #openstack-nova | 05:03 | |
| *** mingyu has quit IRC | 05:07 | |
| *** esberglu has quit IRC | 05:08 | |
| *** ircuser-1 has joined #openstack-nova | 05:08 | |
| *** Tom has joined #openstack-nova | 05:09 | |
| *** markvoelker has quit IRC | 05:17 | |
| *** slaweq has joined #openstack-nova | 05:19 | |
| *** zhurong has quit IRC | 05:21 | |
| *** slaweq has quit IRC | 05:23 | |
| *** thorst has joined #openstack-nova | 05:25 | |
| *** ratailor has joined #openstack-nova | 05:30 | |
| *** Shunli has joined #openstack-nova | 05:33 | |
| *** mingyu has joined #openstack-nova | 05:34 | |
| *** mingyu has quit IRC | 05:49 | |
| *** zsli_ has joined #openstack-nova | 05:49 | |
| *** Shunli has quit IRC | 05:52 | |
| *** litao__ has joined #openstack-nova | 05:53 | |
| *** Eran_Kuris has joined #openstack-nova | 05:57 | |
| *** esberglu has joined #openstack-nova | 05:58 | |
| *** slaweq has joined #openstack-nova | 06:00 | |
| *** lajoskatona has joined #openstack-nova | 06:02 | |
| *** esberglu has quit IRC | 06:02 | |
| *** slaweq has quit IRC | 06:05 | |
| *** mingyu has joined #openstack-nova | 06:06 | |
| *** Oku_OS-away is now known as Oku_OS | 06:08 | |
| *** zsli_ has quit IRC | 06:09 | |
| *** yingjun has joined #openstack-nova | 06:11 | |
| *** markvoelker has joined #openstack-nova | 06:14 | |
| *** zhurong has joined #openstack-nova | 06:15 | |
| *** gbarros has joined #openstack-nova | 06:21 | |
| *** gbarros_ has joined #openstack-nova | 06:26 | |
| *** gbarros has quit IRC | 06:26 | |
| *** gbarros has joined #openstack-nova | 06:29 | |
| *** gbarros_ has quit IRC | 06:30 | |
| *** josecastroleon has joined #openstack-nova | 06:33 | |
| *** gbarros has quit IRC | 06:34 | |
| *** yamamoto_ has joined #openstack-nova | 06:36 | |
| *** yamamoto has quit IRC | 06:36 | |
| *** gbarros has joined #openstack-nova | 06:40 | |
| *** rcernin has joined #openstack-nova | 06:41 | |
| *** slaweq has joined #openstack-nova | 06:41 | |
| *** kevi9132 has joined #openstack-nova | 06:43 | |
| *** sahid has joined #openstack-nova | 06:44 | |
| *** slaweq has quit IRC | 06:45 | |
| *** avolkov has joined #openstack-nova | 06:46 | |
| *** gbarros has quit IRC | 06:47 | |
| *** kevi9132 has quit IRC | 06:47 | |
| *** slaweq has joined #openstack-nova | 06:48 | |
| *** markvoelker has quit IRC | 06:48 | |
| *** esberglu has joined #openstack-nova | 06:52 | |
| *** thorst has quit IRC | 06:52 | |
| *** yingjun has quit IRC | 06:52 | |
| *** andreas_s has joined #openstack-nova | 06:54 | |
| *** esberglu has quit IRC | 06:56 | |
| *** pcaruana has joined #openstack-nova | 07:06 | |
| *** sshwarts has joined #openstack-nova | 07:10 | |
| *** tssurya has joined #openstack-nova | 07:12 | |
| *** yamamoto_ has quit IRC | 07:14 | |
| *** ragiman has joined #openstack-nova | 07:14 | |
| *** zhurong has quit IRC | 07:14 | |
| *** yangzhenyu has quit IRC | 07:14 | |
| *** yamamoto has joined #openstack-nova | 07:16 | |
| *** tesseract has joined #openstack-nova | 07:23 | |
| *** gszasz has joined #openstack-nova | 07:24 | |
| *** alexchadin has joined #openstack-nova | 07:24 | |
| openstackgerrit | jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492 | 07:25 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Deprecate idle_timeout in api_database https://review.openstack.org/507006 | 07:27 |
| *** yangzhenyu has joined #openstack-nova | 07:27 | |
| *** moshele has joined #openstack-nova | 07:30 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: don't overwrite binding-profile https://review.openstack.org/505613 | 07:31 |
| *** zhurong has joined #openstack-nova | 07:32 | |
| *** yamamoto_ has joined #openstack-nova | 07:33 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Move assertEqual order of test_services.py https://review.openstack.org/507007 | 07:36 |
| *** OctopusZhang has joined #openstack-nova | 07:36 | |
| *** yamamoto has quit IRC | 07:36 | |
| *** OctopusZhang_ has joined #openstack-nova | 07:38 | |
| *** alexchadin has quit IRC | 07:39 | |
| *** yufei has quit IRC | 07:40 | |
| *** alexchadin has joined #openstack-nova | 07:40 | |
| *** OctopusZhang_ is now known as yufei | 07:40 | |
| *** jpena|off is now known as jpena | 07:40 | |
| *** yamamoto_ has quit IRC | 07:41 | |
| *** OctopusZhang has quit IRC | 07:42 | |
| *** chyka has joined #openstack-nova | 07:43 | |
| *** markvoelker has joined #openstack-nova | 07:45 | |
| *** hieulq has quit IRC | 07:45 | |
| *** ralonsoh has joined #openstack-nova | 07:46 | |
| *** esberglu has joined #openstack-nova | 07:46 | |
| *** chyka has quit IRC | 07:48 | |
| *** thorst has joined #openstack-nova | 07:49 | |
| *** esberglu has quit IRC | 07:50 | |
| *** trinaths has quit IRC | 07:51 | |
| *** rgerganov has joined #openstack-nova | 07:52 | |
| *** chyka has joined #openstack-nova | 07:53 | |
| *** rgerganov has quit IRC | 07:53 | |
| *** rgerganov has joined #openstack-nova | 07:53 | |
| *** chyka has quit IRC | 07:57 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029 | 08:04 |
| *** sshwarts has quit IRC | 08:04 | |
| *** Shunli has joined #openstack-nova | 08:06 | |
| *** yamamoto has joined #openstack-nova | 08:10 | |
| *** markvoelker has quit IRC | 08:18 | |
| *** mikal_ has joined #openstack-nova | 08:29 | |
| *** zhurong has quit IRC | 08:32 | |
| *** mikal has quit IRC | 08:32 | |
| *** yamamoto has quit IRC | 08:32 | |
| *** phuongnh has quit IRC | 08:36 | |
| *** esberglu has joined #openstack-nova | 08:40 | |
| *** derekh has joined #openstack-nova | 08:42 | |
| *** udesale__ has joined #openstack-nova | 08:44 | |
| *** mvk has joined #openstack-nova | 08:44 | |
| *** esberglu has quit IRC | 08:44 | |
| *** udesale has quit IRC | 08:46 | |
| *** alexchadin has quit IRC | 08:46 | |
| *** alexchadin has joined #openstack-nova | 08:47 | |
| *** alexchadin has quit IRC | 08:47 | |
| *** finucannot is now known as stephenfin | 08:47 | |
| *** alexchadin has joined #openstack-nova | 08:48 | |
| *** alexchadin has quit IRC | 08:48 | |
| *** alexchadin has joined #openstack-nova | 08:49 | |
| *** Shunli has quit IRC | 08:51 | |
| *** gszasz has quit IRC | 08:51 | |
| *** hferenc has quit IRC | 08:51 | |
| *** alexchadin has quit IRC | 08:52 | |
| *** zhurong has joined #openstack-nova | 08:52 | |
| *** udesale has joined #openstack-nova | 08:52 | |
| *** udesale__ has quit IRC | 08:54 | |
| *** armax has joined #openstack-nova | 08:56 | |
| *** thorst has quit IRC | 09:07 | |
| openstackgerrit | Lei Zhang proposed openstack/nova-specs master: Request traits in Nova https://review.openstack.org/468797 | 09:08 |
| *** markvoelker has joined #openstack-nova | 09:15 | |
| openstackgerrit | Merged openstack/nova master: Make 'fault' a valid joined query field for Instance https://review.openstack.org/506774 | 09:16 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: [WIP] Add ability for OVMF Secure Boot https://review.openstack.org/506720 | 09:18 |
| *** chyka has joined #openstack-nova | 09:19 | |
| *** armax has quit IRC | 09:19 | |
| *** lpetrut has joined #openstack-nova | 09:20 | |
| openstackgerrit | Merged openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417 | 09:20 |
| *** chyka has quit IRC | 09:23 | |
| *** yamahata has joined #openstack-nova | 09:23 | |
| *** gszasz has joined #openstack-nova | 09:24 | |
| *** mingyu has quit IRC | 09:25 | |
| *** hshiina has joined #openstack-nova | 09:27 | |
| *** bauwser is now known as bauzas | 09:27 | |
| *** jaosorior has joined #openstack-nova | 09:28 | |
| *** mvk has quit IRC | 09:28 | |
| *** armax has joined #openstack-nova | 09:32 | |
| *** esberglu has joined #openstack-nova | 09:34 | |
| *** esberglu has quit IRC | 09:38 | |
| openstackgerrit | jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492 | 09:40 |
| *** yamamoto has joined #openstack-nova | 09:40 | |
| *** takashin has left #openstack-nova | 09:41 | |
| *** armax has quit IRC | 09:43 | |
| *** belmoreira has joined #openstack-nova | 09:47 | |
| *** markvoelker has quit IRC | 09:48 | |
| *** mingyu has joined #openstack-nova | 09:52 | |
| *** TuanLA has joined #openstack-nova | 09:53 | |
| *** mvk has joined #openstack-nova | 09:57 | |
| *** sapcc-bot1 has quit IRC | 09:57 | |
| *** Tom has quit IRC | 09:58 | |
| *** yamamoto has quit IRC | 10:00 | |
| *** nicolasbock has joined #openstack-nova | 10:02 | |
| *** yamahata has quit IRC | 10:04 | |
| *** thorst has joined #openstack-nova | 10:04 | |
| *** belmoreira has quit IRC | 10:06 | |
| *** yufei has quit IRC | 10:09 | |
| *** trungnv has quit IRC | 10:10 | |
| *** tiendc has quit IRC | 10:10 | |
| *** TuanLA has quit IRC | 10:10 | |
| *** trungnv has joined #openstack-nova | 10:11 | |
| *** tiendc has joined #openstack-nova | 10:11 | |
| *** TuanLA has joined #openstack-nova | 10:11 | |
| *** nicolasbock has quit IRC | 10:12 | |
| *** armax has joined #openstack-nova | 10:19 | |
| *** zhurong has quit IRC | 10:21 | |
| *** nicolasbock has joined #openstack-nova | 10:24 | |
| *** zhurong has joined #openstack-nova | 10:25 | |
| *** esberglu has joined #openstack-nova | 10:28 | |
| *** gszasz has quit IRC | 10:29 | |
| *** jaypipes has joined #openstack-nova | 10:30 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Support traits in the Ironic driver https://review.openstack.org/507052 | 10:32 |
| *** esberglu has quit IRC | 10:33 | |
| *** gcb has joined #openstack-nova | 10:33 | |
| *** rgerganov has quit IRC | 10:36 | |
| *** udesale has quit IRC | 10:37 | |
| *** zhouyaguo has quit IRC | 10:40 | |
| *** zhouyaguo has joined #openstack-nova | 10:40 | |
| *** alexchadin has joined #openstack-nova | 10:41 | |
| *** hshiina has quit IRC | 10:42 | |
| *** kevi9132 has joined #openstack-nova | 10:45 | |
| *** markvoelker has joined #openstack-nova | 10:45 | |
| *** sshwarts has joined #openstack-nova | 10:45 | |
| openstackgerrit | Merged openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892 | 10:47 |
| *** cdent has joined #openstack-nova | 10:47 | |
| *** kevi9132 has quit IRC | 10:49 | |
| *** hieulq has joined #openstack-nova | 10:50 | |
| *** TuanLA has quit IRC | 10:52 | |
| *** gcb has quit IRC | 10:53 | |
| *** gszasz has joined #openstack-nova | 10:55 | |
| *** armax has quit IRC | 10:57 | |
| *** sdague has joined #openstack-nova | 10:59 | |
| *** armax has joined #openstack-nova | 11:02 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 11:02 |
| *** armax has quit IRC | 11:03 | |
| *** armax has joined #openstack-nova | 11:03 | |
| *** abhishek_k has joined #openstack-nova | 11:03 | |
| *** NehaAlhat has joined #openstack-nova | 11:04 | |
| *** armax has quit IRC | 11:04 | |
| *** bhagyashri_s has joined #openstack-nova | 11:04 | |
| *** ps_jadhav has joined #openstack-nova | 11:04 | |
| *** slaweq has quit IRC | 11:04 | |
| *** yangyapeng has quit IRC | 11:05 | |
| *** abhishekk has quit IRC | 11:06 | |
| *** bhagyashris has quit IRC | 11:07 | |
| *** neha_alhat has quit IRC | 11:07 | |
| *** pooja_jadhav has quit IRC | 11:07 | |
| *** sshwarts has quit IRC | 11:10 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec https://review.openstack.org/492026 | 11:11 |
| *** thorst has quit IRC | 11:14 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 11:17 |
| *** markvoelker has quit IRC | 11:18 | |
| *** sree has quit IRC | 11:20 | |
| *** trinaths has joined #openstack-nova | 11:20 | |
| *** sree has joined #openstack-nova | 11:20 | |
| *** slaweq has joined #openstack-nova | 11:21 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Add error notification for instance.interface_attach https://review.openstack.org/506643 | 11:21 |
| *** esberglu has joined #openstack-nova | 11:22 | |
| *** sree has quit IRC | 11:24 | |
| *** sree has joined #openstack-nova | 11:24 | |
| *** jpena is now known as jpena|lunch | 11:24 | |
| *** clayton has quit IRC | 11:25 | |
| *** dtantsur|afk is now known as dtantsur | 11:26 | |
| *** esberglu has quit IRC | 11:27 | |
| openstackgerrit | caoyuan proposed openstack/nova master: cleanup test-requirements https://review.openstack.org/507063 | 11:29 |
| *** clayton has joined #openstack-nova | 11:30 | |
| *** yamamoto has joined #openstack-nova | 11:30 | |
| *** tiendc has quit IRC | 11:34 | |
| *** belmoreira has joined #openstack-nova | 11:36 | |
| *** mingyu has quit IRC | 11:39 | |
| *** sree has quit IRC | 11:41 | |
| *** sree has joined #openstack-nova | 11:42 | |
| *** sree has quit IRC | 11:42 | |
| *** sree has joined #openstack-nova | 11:43 | |
| *** ps_jadhav is now known as pooja_jadhav | 11:45 | |
| *** smatzek has joined #openstack-nova | 11:45 | |
| *** gszasz has quit IRC | 11:47 | |
| *** abhishek_k has quit IRC | 11:47 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Add error notification for instance.interface_attach https://review.openstack.org/506643 | 11:51 |
| *** litao__ has quit IRC | 11:52 | |
| *** yamamoto has quit IRC | 12:00 | |
| *** thorst has joined #openstack-nova | 12:01 | |
| *** mingyu has joined #openstack-nova | 12:01 | |
| *** thorst_ has joined #openstack-nova | 12:04 | |
| *** rgerganov has joined #openstack-nova | 12:04 | |
| *** thorst has quit IRC | 12:05 | |
| *** alexchadin has quit IRC | 12:07 | |
| *** alexchadin has joined #openstack-nova | 12:07 | |
| *** gszasz has joined #openstack-nova | 12:07 | |
| *** edmondsw has joined #openstack-nova | 12:09 | |
| *** jmlowe has quit IRC | 12:10 | |
| *** mingyu has quit IRC | 12:10 | |
| *** mingyu has joined #openstack-nova | 12:10 | |
| *** MVenesio has joined #openstack-nova | 12:11 | |
| *** bhagyashri_s is now known as bhagyashris | 12:11 | |
| *** sambetts|afk is now known as sambetts | 12:13 | |
| openstackgerrit | Merged openstack/nova master: Add default configuration files to data_files https://review.openstack.org/506188 | 12:13 |
| *** markvoelker has joined #openstack-nova | 12:16 | |
| *** esberglu has joined #openstack-nova | 12:16 | |
| *** yamamoto has joined #openstack-nova | 12:18 | |
| *** yangyapeng has joined #openstack-nova | 12:20 | |
| *** MasterofJOKers_ has joined #openstack-nova | 12:20 | |
| *** esberglu has quit IRC | 12:21 | |
| *** sree has quit IRC | 12:21 | |
| *** mingyu has quit IRC | 12:21 | |
| *** sree has joined #openstack-nova | 12:21 | |
| *** mingyu has joined #openstack-nova | 12:21 | |
| *** MasterofJOKers has quit IRC | 12:22 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 12:22 | |
| *** alexchadin has quit IRC | 12:22 | |
| *** gszasz has quit IRC | 12:23 | |
| *** markvoelker has quit IRC | 12:23 | |
| *** markvoelker has joined #openstack-nova | 12:24 | |
| *** belmoreira has quit IRC | 12:24 | |
| *** gszasz has joined #openstack-nova | 12:24 | |
| *** jpena|lunch is now known as jpena | 12:25 | |
| openstackgerrit | Merged openstack/nova master: Add PowerVM hypervisor configuration doc https://review.openstack.org/505665 | 12:25 |
| *** sree has quit IRC | 12:26 | |
| *** trinaths has quit IRC | 12:26 | |
| *** belmoreira has joined #openstack-nova | 12:28 | |
| *** Eran_Kuris has quit IRC | 12:32 | |
| sdague | anyone else want to bring this one home - https://review.openstack.org/#/c/454323/ - it enables live snapshot by default | 12:32 |
| *** yamamoto has quit IRC | 12:33 | |
| cdent | sdague: I approve of your subtle s/Garbage// | 12:35 |
| *** erlon has joined #openstack-nova | 12:38 | |
| *** lyan has joined #openstack-nova | 12:38 | |
| *** derekh has quit IRC | 12:40 | |
| *** derekh has joined #openstack-nova | 12:40 | |
| *** kenperkins has joined #openstack-nova | 12:45 | |
| stephenfin | sdague: Done | 12:46 |
| sdague | stephenfin: thanks! | 12:46 |
| *** kevi9132 has joined #openstack-nova | 12:46 | |
| sdague | stephenfin: it would also be cool to get your eyes on this one - https://review.openstack.org/#/c/505673/ - so that we could enable the pike ppa on ubuntu, as we don't work with qemu 2.10 right now | 12:47 |
| *** Eran_Kuris has joined #openstack-nova | 12:47 | |
| *** udesale has joined #openstack-nova | 12:48 | |
| stephenfin | sdague: Yup, that was on my list for today. One last question - why 'operator.ge' in [1]? https://review.openstack.org/#/c/505673/5/nova/virt/images.py | 12:49 |
| stephenfin | (More out of curiosity than anything) | 12:49 |
| *** zhurong has quit IRC | 12:50 | |
| *** kevi9132 has quit IRC | 12:51 | |
| *** belmoreira has quit IRC | 12:51 | |
| *** yamamoto has joined #openstack-nova | 12:52 | |
| sdague | stephenfin: that was in mriedem's original patch, it seemed fine to stay. It could be regular operators as well | 12:52 |
| stephenfin | sdague: Figured. It looked good to me anyway so +2d | 12:53 |
| *** belmoreira has joined #openstack-nova | 12:53 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 12:53 |
| efried | jaypipes https://review.openstack.org/#/c/497713/4/specs/queens/approved/add-trait-support-in-allocation-candidates.rst@72 -- Are you suggesting that we be able to model an optional trait as an inventory-less resource class? | 12:54 |
| sdague | stephenfin: ty | 12:57 |
| *** lajoskatona has quit IRC | 12:58 | |
| *** udesale has quit IRC | 13:00 | |
| *** mriedem has joined #openstack-nova | 13:00 | |
| *** gszasz has quit IRC | 13:01 | |
| *** liverpooler has joined #openstack-nova | 13:02 | |
| *** jistr is now known as jistr|mtg | 13:03 | |
| *** sree has joined #openstack-nova | 13:05 | |
| *** jmlowe has joined #openstack-nova | 13:07 | |
| *** sshwarts has joined #openstack-nova | 13:08 | |
| *** gcb has joined #openstack-nova | 13:10 | |
| *** esberglu has joined #openstack-nova | 13:11 | |
| *** baoli has joined #openstack-nova | 13:12 | |
| *** gszasz has joined #openstack-nova | 13:13 | |
| *** Tom has joined #openstack-nova | 13:19 | |
| bauzas | stephenfin: do we have in Pike a config options page that shows what tox -genconfig does ? | 13:19 |
| bauzas | I mean in the docs tree | 13:19 |
| *** swebster has joined #openstack-nova | 13:20 | |
| bauzas | stephenfin: nevermind, https://docs.openstack.org/nova/latest/configuration/config.html | 13:20 |
| *** larsks has quit IRC | 13:20 | |
| mriedem | simple change here that unblocks a few other changes https://review.openstack.org/#/c/506104/ | 13:20 |
| mriedem | bauzas: this is more like what the nova.conf.sample would look like https://docs.openstack.org/nova/latest/configuration/sample-config.html | 13:21 |
| *** larsks has joined #openstack-nova | 13:23 | |
| *** links has quit IRC | 13:23 | |
| *** josecastroleon has quit IRC | 13:25 | |
| jaypipes | efried: err, no... | 13:27 |
| *** gouthamr has joined #openstack-nova | 13:27 | |
| *** josecastroleon has joined #openstack-nova | 13:27 | |
| efried | jaypipes I added a comment with an example use case. Generally, curious how that class of use case is supposed to be handled. | 13:28 |
| *** lbragstad has joined #openstack-nova | 13:28 | |
| jianghuaw | bauzas, jaypipes: If don't cover NUMA, is it possible to support vGPU with the approach of resource provider in queens? I understand the nested RP won't be completed in in queens. But maybe we don't need the full functions within nested RP for vGPU. As it's much straight. Only put the vGPU resource provider as the compute's children. | 13:28 |
| mriedem | jianghuaw: we plan on implemented nested resource providers in queens | 13:29 |
| *** gouthamr has quit IRC | 13:29 | |
| *** jistr|mtg is now known as jistr | 13:29 | |
| mriedem | but i don't think we plan on focusing on numa use cases | 13:29 |
| *** tssurya has quit IRC | 13:29 | |
| mriedem | *implementing | 13:29 |
| bauzas | jianghuaw: I don't think we should worry on NUMA resources now | 13:29 |
| *** gouthamr has joined #openstack-nova | 13:30 | |
| bauzas | jianghuaw: because once compute nodes will report NUMA cells using nested RPs, we'll get both vGPU support and NUMA handling tied together | 13:30 |
| jianghuaw | mriedem, bauzas, thanks. In that case, I believe we will proceed to implement vGPU with resource provider. right? | 13:30 |
| jianghuaw | bauzas, cool. | 13:30 |
| bauzas | jianghuaw: for the moment, I see vGPU resources as just traits and resource classes attached to a specific RP | 13:31 |
| bauzas | if one implements NUMA reporting with nested RPs, that will just mean that vGPU resource classes will be attached to the child RP that is providing the NUMA cell | 13:31 |
| bauzas | if that's a PCI device | 13:32 |
| jianghuaw | yeah, fair enough. | 13:32 |
| bauzas | jianghuaw: I actually raised that point in a comment in your change | 13:32 |
| bauzas | jianghuaw: but I was waiting for jaypipes acking or not that | 13:33 |
| jaypipes | bauzas: ack | 13:33 |
| bauzas | cool then | 13:33 |
| jianghuaw | :-) | 13:33 |
| bauzas | then, no need to wait for NUMA-isms | 13:33 |
| jianghuaw | that's great. | 13:33 |
| bauzas | jianghuaw: I actually owe you a new review of your spec | 13:34 |
| bauzas | and I owe jaypipes a serious review of his nested RP series | 13:34 |
| bauzas | jaypipes: you said you were about to rebase, right? | 13:34 |
| bauzas | mriedem: interesting bug https://bugs.launchpad.net/nova/+bug/1717915 | 13:35 |
| openstack | Launchpad bug 1717915 in oslo.messaging "nova services and transport_url, cannot connect to vhost if specified" [Undecided,New] | 13:35 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Live Migration sequence diagram https://review.openstack.org/506370 | 13:35 |
| efried | alex_xu Like-a-this? ^ | 13:35 |
| jianghuaw | bauzas, Thanks for have reviewed my spec for may rounds and given may useful comments. | 13:35 |
| jaypipes | bauzas: done last week: https://review.openstack.org/#/q/topic:bp/nested-resource-providers | 13:35 |
| bauzas | jaypipes: cool, that's then my top prio | 13:36 |
| jaypipes | efried: answered on spec. | 13:36 |
| efried | thx | 13:36 |
| jianghuaw | bauzas, jaypipes: I do need your help on how to define that options to restrict only one vGPU type is exposed. | 13:36 |
| jaypipes | jianghuaw: sorry, not following you... is this a patch you've proposed? | 13:37 |
| *** belmoreira has quit IRC | 13:37 | |
| mriedem | bauzas: yes, you'll see i was already triaging it | 13:37 |
| *** smatzek has quit IRC | 13:38 | |
| jianghuaw | jaypipes, I mean this patch:https://review.openstack.org/#/c/450122 | 13:38 |
| jianghuaw | sahid suggested to make the option to be general. | 13:39 |
| bauzas | mriedem: yup, I saw hence my ping | 13:39 |
| bauzas | mriedem: it was an implicit 'good call, but what should we do next" ? | 13:39 |
| *** Eran_Kuris has quit IRC | 13:39 | |
| mriedem | idk, i hoped that the rhops guys would know since you guys run with clustered rabbit | 13:39 |
| bauzas | because looks like it's something that worked in the past but we never officially supported it | 13:39 |
| mriedem | *rhosp | 13:40 |
| bauzas | owalsh: around ? | 13:40 |
| *** dave-mccowan has joined #openstack-nova | 13:41 | |
| bauzas | jaypipes: the only point that is still concerning me about https://review.openstack.org/#/c/450122 is how we draft the whitelist | 13:41 |
| jaypipes | bauzas: I had specifically asked jianghuaw to make the config option *not* the pci_passthrough_whitelist | 13:42 |
| bauzas | jaypipes: as it can be different for each virt driver | 13:42 |
| jaypipes | bauzas: how so? | 13:42 |
| mriedem | jianghuaw: bauzas: jaypipes: ew, yeah, was just going to ask if this is a new pci whitelist but for gpus | 13:42 |
| jaypipes | mriedem: yes, I asked for that. | 13:42 |
| bauzas | jaypipes: well, jianghuaw made a very explicit pci-tied config option | 13:43 |
| bauzas | I probably missed your point then | 13:43 |
| *** tssurya has joined #openstack-nova | 13:43 | |
| bauzas | but I'm fine with just a ListOpt containing strings that would match device IDs | 13:43 |
| bauzas | it would be up to the driver to find the right GPU that matches the string | 13:43 |
| jaypipes | bauzas: are you referring to the fact that the proposed enabled_vgpu_types CONF option has vendor_id and product_id keys? | 13:44 |
| bauzas | like enabled_vgpu_types = ['nvidia-11', 'nvidia-10'] | 13:44 |
| bauzas | yeah, that is too specific | 13:44 |
| bauzas | jianghuaw: ^ | 13:45 |
| edleafe | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | 13:45 |
| jianghuaw | how about use the vGPU type's name instead of the id? by considering the type name is common from both XenServer and kvm? enabled_vgpu_types = ['GRID K160Q'...] | 13:45 |
| jianghuaw | As I explained in the comment, both virts can get the type name - "GRID K160" | 13:46 |
| jianghuaw | And that's also the name recorded in the user guide. | 13:47 |
| sdague | mriedem: with you and I both with fingers in the qemu 2.10 patch, you want a 4th core to look into it - https://review.openstack.org/#/c/505673/ - or you want to just make sure my update isn't crazy and put it in | 13:48 |
| bauzas | jianghuaw: and what if I have two exact same cards ? | 13:48 |
| sahid | mriedem: did you notice my request, continuing development on /pci and in parallel working on porting virt features on RP | 13:48 |
| mriedem | sdague: i looked at it last week, which is how we went down the paused rabbit hole | 13:49 |
| mriedem | sdague: but we have plenty of cores, what's one more to review | 13:49 |
| sahid | by this way we could still provide proction-ready features and have at some point a deprecating phease | 13:49 |
| *** jheroux has joined #openstack-nova | 13:49 | |
| sdague | mriedem: right, that's fixed now | 13:49 |
| mriedem | sahid: this? https://review.openstack.org/#/c/485522/ | 13:49 |
| *** dpawlik has quit IRC | 13:49 | |
| mriedem | sahid: sorry no i'm not sure what you're referring to | 13:50 |
| sahid | mriedem: no worries let me give you a pointer | 13:50 |
| *** danpawlik has joined #openstack-nova | 13:50 | |
| *** bswartz has quit IRC | 13:50 | |
| sahid | mriedem: http://lists.openstack.org/pipermail/openstack-dev/2017-September/122591.html | 13:50 |
| jianghuaw | bauzas, hmmm. yes, that means we can only expose one type per compute node. | 13:50 |
| jianghuaw | I think that's the reason I used a pci-tied config option; | 13:51 |
| jianghuaw | we can specify the vGPU enabled a a specific PGPU keyed by pci address. | 13:52 |
| openstackgerrit | Merged openstack/nova master: Add some inline code docs tracing the cold migrate flow https://review.openstack.org/496861 | 13:52 |
| mriedem | jaypipes: "The decision of whether to allow an approach that adds more to the existing /pci module is ultimately Matt's." | 13:52 |
| mriedem | jaypipes: i didn't even see that bus coming | 13:52 |
| jaypipes | mriedem: lol. well, sorry, but you are the PTL :) | 13:53 |
| bauzas | jianghuaw: sec, reading your last comments then | 13:54 |
| *** kevi9132 has joined #openstack-nova | 13:54 | |
| *** kevi9132 has quit IRC | 13:56 | |
| bauzas | jianghuaw: so, in case of libvirt, we have the capabilities XML that shows up different device IDs for two same cards (or types if you prefer) | 13:56 |
| efried | jianghuaw Is it possible you have multiple identical GPUs (or mulitple pGPUs on the same card) where you would want to whitelist some but not others? | 13:56 |
| efried | :) | 13:57 |
| sahid | mriedem: jaypipes, my point is to continue make our users happy, have a deprecating phase to fix any issues for the work ongoing with RP and at some point, switch | 13:57 |
| bauzas | jianghuaw: because each of them is corresponding to a mediated device | 13:57 |
| jaypipes | sahid: well, we're never going to make users happy :) | 13:57 |
| bauzas | jianghuaw: and that's the deployer's responsibility to configure libvirt so that the GPUs are shown as mediated devices | 13:57 |
| openstackgerrit | Ed Leafe proposed openstack/nova-specs master: Return Alternate Hosts https://review.openstack.org/504275 | 13:58 |
| jaypipes | sahid: but anyway, like I said, I'm fine with you adding to the existing code. | 13:58 |
| jaypipes | sahid: I just will be focusing on the n-r-p stuff, that's all | 13:58 |
| bauzas | it's open-source development, anyone can just propose | 13:58 |
| sahid | jaypipes: yes i understand that which it make sense and i will be happy to help and review some part if i can | 13:58 |
| bauzas | I just felt we discussed on what could be achievable for Queens in parallel of the nested RPs implementation | 13:59 |
| *** takashin has joined #openstack-nova | 13:59 | |
| *** smatzek has joined #openstack-nova | 13:59 | |
| bauzas | the main focus for Queens was to provide the libvirt feature to returning RPs that were augmented by the VGPU resources | 14:00 |
| *** belmoreira has joined #openstack-nova | 14:00 | |
| bauzas | jianghuaw: can you clarify how Xen would see those GPU types ? | 14:00 |
| edleafe | Scheduler subteam meeting running now in #openstack-meeting-alt | 14:00 |
| sahid | bauzas: so you want to push on top of something under heavy developpement a feature which we are going to be used by large industries ? i think we should be reasonable | 14:00 |
| *** ratailor has quit IRC | 14:01 | |
| *** sshwarts has quit IRC | 14:01 | |
| bauzas | sahid: I'm just saying it's orthogonal :) | 14:01 |
| sahid | RP is doind a lof of things, we need a deprecating phase, where some users can migrate and so we can fix the issues | 14:01 |
| jianghuaw | bauzas, ok. XenServer will detect the pGPU types and make the same type of pGPUs into a single group. | 14:01 |
| *** MasterofJOKers has quit IRC | 14:01 | |
| bauzas | vGPU tracking is done by providing a new set of resource classes | 14:01 |
| bauzas | and traits | 14:01 |
| *** beekneemech is now known as bnemec | 14:02 | |
| bauzas | while nested resource providers is focusing on providing a tree of resource providers with dependencies | 14:02 |
| bauzas | those don't overlap | 14:02 |
| jianghuaw | And expose the vGPU types supported by the pGPU group. | 14:02 |
| *** superdan is now known as dansmith | 14:02 | |
| jaypipes | sahid: for the record, we *have* been doing that deprecation/migration period with resource providers. for example, we had a deprecation/migration period for tracking Ironic nodes as atomic resources. | 14:02 |
| bauzas | snap, scheduler meeting | 14:02 |
| jianghuaw | when requesting a vGPU, xenserver will schedule a PGPU and create a vGPU on it. | 14:03 |
| *** MasterofJOKers has joined #openstack-nova | 14:03 | |
| jaypipes | sahid: so it's not true that we're just imposing resource providers modeling without providing a migration path for existing resource classes. | 14:03 |
| *** sree has quit IRC | 14:03 | |
| jianghuaw | bauzas, so we can't define whitelist per pGPU by per pGPU group. | 14:03 |
| bauzas | sahid: jaypipes: probably a good call for discussing that in the scheduler meeting | 14:03 |
| sahid | jaypipes: oh yes right, but, what about NUMA, CPU Pinning, Huge Pages and all the virt specific features? | 14:03 |
| *** sree has joined #openstack-nova | 14:04 | |
| sahid | don't we have to migr incrementally? | 14:04 |
| jaypipes | sahid: yes, absolutely. | 14:04 |
| jaypipes | sahid: none of those resources are planned to migrate in Queens, BTW. | 14:04 |
| sahid | jaypipes: yes and what is the ETA for sometging production-ready? | 14:04 |
| sahid | ok cool that is my point | 14:04 |
| owalsh | bauzas: hey | 14:04 |
| bauzas | well, with the fact that CPU pinning isn't targeted to be pushed to Placement, right? | 14:04 |
| jianghuaw | bauzas, that's why we make the pGPU group as the vGPU's resource provider. | 14:04 |
| bauzas | jianghuaw: so, say I have two same cards, I'd only see one pGPU group right? | 14:05 |
| jianghuaw | true. | 14:05 |
| sahid | jaypipes: it will take at least 2 or 3 releases and during that time we can't make any development? | 14:05 |
| bauzas | jianghuaw: okay, then that's a bit different from libvirt | 14:05 |
| sahid | i just suggest we work in parallel | 14:06 |
| jianghuaw | bauzas, could you give an example for 'that shows up different device IDs for two same cards.' | 14:06 |
| jaypipes | sahid: depends on what you mean by "production-ready". (I personally don't think the existing pci manager is well-written or maintainable, but I guess there are different defninitions of "production-ready") | 14:06 |
| sahid | since the /pci is kind of production-ready and provide eveyrything we need | 14:06 |
| jaypipes | sahid: again, I'm not disagreeing with you... | 14:06 |
| sahid | jaypipes: yes yes :) | 14:06 |
| jaypipes | not sure why you're acting like I am :) | 14:07 |
| sahid | oops sorry really | 14:07 |
| sahid | it's just that you are the only who are paying attention at me :) | 14:07 |
| dansmith | jaypipes: we can provide pretty simple vgpu support via resource classes today right? without building more into the pci infrastructure we have | 14:08 |
| jaypipes | sahid: we're paying attention but also in scheduler IRC meeting :) | 14:08 |
| dansmith | jaypipes: if we just let virt drivers see that a vgpu was requested (i.e. see the flavor) and just configure the guest with an available one | 14:08 |
| jianghuaw | bauzas, will the "GRID M60-0B" have different type id <type id='nvidia-11'> for two same pGPUs? | 14:08 |
| *** sree has quit IRC | 14:08 | |
| bauzas | jianghuaw: I haven't tested yet, but I bet it | 14:09 |
| jaypipes | dansmith: very simple resources, yes. we could add a (custom) resource class for the VGPU resources and have a flavor consume some amount of those | 14:09 |
| dansmith | jaypipes: right, that'd be my preference for the first go-round | 14:09 |
| jianghuaw | bauzas, I think if the type id is different, it will be a good solution to use 'type-id' for libvirt. | 14:10 |
| bauzas | dansmith: jaypipes: I was only seeing the virt drivers adding the new VGPU resource class to the existing RPs as a first round, that's it :) | 14:10 |
| dansmith | jaypipes: let people segregate different types of gpus into aggregates, and just assume equal portions of vgpu per instance without any smarts, which I think would be a perfectly reasonable first stab, which we can iterate on when we have traits and things | 14:10 |
| dansmith | bauzas: ++ | 14:10 |
| jaypipes | dansmith: agreed. | 14:11 |
| bauzas | jianghuaw: yeah, so you agree with the fact that the listOpt can have different semantics based on the driver? | 14:11 |
| jianghuaw | bauzas, yes. libvirt uses the type-id but XenServer continue to use the type name. As you suggested, virt driver should able to mapping them into real resources. | 14:11 |
| bauzas | that would be strings for each of the drivers, but in the libvirt case, it would be device IDs while it would be pGPU resource type names for xen | 14:12 |
| jianghuaw | but we need confirm if the type-id will be different. | 14:12 |
| bauzas | jianghuaw: what I need is a testbed :p | 14:12 |
| jianghuaw | bauzas, thanks. | 14:13 |
| bauzas | jianghuaw: fancy updating the spec so I can do a pass once again ? | 14:13 |
| jianghuaw | bauzas, sure. I will update it soon. | 14:14 |
| bauzas | cool | 14:14 |
| sahid | dansmith: basically you can't do more than that right now and it's exactly what i'm pointing where we could have better support | 14:14 |
| jianghuaw | bauzas, so I think the option name as enabled_vgpu_types is fine. right? | 14:14 |
| *** celebrate has joined #openstack-nova | 14:14 | |
| jianghuaw | just confirm if I can keep this option name:-) | 14:14 |
| *** pchavva has joined #openstack-nova | 14:15 | |
| celebrate | Hi guys! Do you know where the "availability_zone" attribute of instance is stored? | 14:15 |
| dansmith | sahid: building more on the current pci stuff isn't "better" in any way, IMHO :) | 14:15 |
| *** liuyulong_ has joined #openstack-nova | 14:15 | |
| dansmith | sahid: and I just think that providing a basic building block initially and moving on from there avoids us building more baggage that we have to migrate, adapt, and remove later | 14:15 |
| mriedem | celebrate: instance.availability_zone, or as metadata on a host aggregate the instance is in | 14:16 |
| *** mdnadeem has quit IRC | 14:16 | |
| bauzas | celebrate: the scheduler queries hosts based on request_spec.availability_zone | 14:16 |
| bauzas | instance.az is only set at boot time | 14:16 |
| celebrate | yeah, in database i see AZ=nova, but scheduler thinks its "None" | 14:16 |
| sahid | dansmith: i'm not agree with you the /pci is good and do what we need | 14:17 |
| dansmith | sahid: okay :) | 14:17 |
| *** Yingxin has quit IRC | 14:17 | |
| sahid | dansmith: in all cases we will have to migrate the /pci | 14:17 |
| celebrate | Do you know how to make scheduler understand that AZ is "nova"? | 14:17 |
| *** Yingxin has joined #openstack-nova | 14:17 | |
| mriedem | sahid: we don't have to migrate vgpus if we don't implement it in /pci | 14:17 |
| sahid | the support of mdev is not so heavy | 14:17 |
| sahid | mriedem: mdev is just pci devices | 14:18 |
| bauzas | celebrate: well, nova AZ is actually equal to None | 14:18 |
| sahid | so when you will migrate sriov it will be the same work | 14:18 |
| *** armax has joined #openstack-nova | 14:18 | |
| bauzas | celebrate: that's a convention, or rather a config option default value | 14:18 |
| *** josecastroleon has quit IRC | 14:19 | |
| mriedem | celebrate: https://docs.openstack.org/nova/latest/user/aggregates.html | 14:19 |
| bauzas | namely [DEFAULT]/default_availability_zone | 14:19 |
| celebrate | I've already set [DEFAULT]/default_availability_zone to "nova" | 14:19 |
| celebrate | however scheduler thinks its None and refuses to respect AZ from aggregates | 14:19 |
| *** hongbin has joined #openstack-nova | 14:20 | |
| bauzas | celebrate: I will need to disappear in a couple of minutes, but here are a few things to know | 14:20 |
| bauzas | celebrate: #1 if you didn't ask a specific AZ at boot time, that means the scheduler won't care about placing your instance to a specific AZ | 14:20 |
| bauzas | celebrate: if you did specified a AZ at boot, AZFilter in the scheduler will check the AZ for each of the hosts | 14:21 |
| celebrate | can I somehow make scheduler change its mind? | 14:21 |
| *** READ10 has joined #openstack-nova | 14:21 | |
| bauzas | celebrate: not sure I get why :) | 14:21 |
| *** crushil has joined #openstack-nova | 14:21 | |
| celebrate | because I have different AZ and scheduler sends VM there during live-migration | 14:22 |
| celebrate | where this information is stored in DB? | 14:22 |
| *** BlartVersenwald is now known as NobodyCam | 14:22 | |
| *** tidwellr has joined #openstack-nova | 14:22 | |
| *** tssurya has quit IRC | 14:23 | |
| bauzas | celebrate: could you please describe your problem ? | 14:23 |
| celebrate | I have 3 different AZ and during live-migration VMs travel from one AZ to another | 14:23 |
| celebrate | and I want VMs to stay inside their AZ | 14:23 |
| bauzas | celebrate: if the instance wasn't boot by specifying an AZ with --availability_zone' then the answer is "no, it's not possible to modify that later" | 14:25 |
| bauzas | and tweaking the DB is fairly discouraged | 14:25 |
| bauzas | rather, you should consider making use of "default_schedule_zone" config opt which sets a default AZ to any instance | 14:25 |
| bauzas | I really need to disappear for a couple of mins (kids school thingy) | 14:26 |
| celebrate | please tell me where it is stored in DB | 14:28 |
| celebrate | please ) | 14:28 |
| liuyulong_ | mriedem, sdague, hello, http://lists.openstack.org/pipermail/openstack-dev/2017-September/122567.html, I sent the issue of rebuild-keypair-updating to the mail list openstack-dev. So what do you guys think about it? | 14:28 |
| mriedem | sahid: jaypipes: replied to the ML thread on vgpus | 14:29 |
| liuyulong_ | Are there any others related to this discussion? PTG members? | 14:29 |
| *** awaugama has joined #openstack-nova | 14:31 | |
| celebrate | so anybody knows where in DB "availability_zone" attribute is located? | 14:36 |
| celebrate | the one attribute that is respected by scheduler | 14:36 |
| *** belmoreira has quit IRC | 14:38 | |
| *** tssurya has joined #openstack-nova | 14:38 | |
| *** trinaths has joined #openstack-nova | 14:38 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Field network.multi_host is now deprecated. https://review.openstack.org/507151 | 14:39 |
| *** belmoreira has joined #openstack-nova | 14:40 | |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Update oslo version in requirements https://review.openstack.org/507153 | 14:41 |
| jaypipes | mriedem: danke\ | 14:43 |
| *** takedakn has joined #openstack-nova | 14:46 | |
| *** armax has quit IRC | 14:47 | |
| *** READ10 has quit IRC | 14:47 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613 | 14:48 |
| *** celebrate has quit IRC | 14:49 | |
| *** gyee has joined #openstack-nova | 14:51 | |
| *** belmoreira has quit IRC | 14:54 | |
| *** gcb has quit IRC | 14:55 | |
| mriedem | bauzas: don't forget https://review.openstack.org/#/c/506414/ | 14:58 |
| bauzas | mriedem: I'm on it | 14:58 |
| bauzas | just needs moar time | 14:58 |
| *** felipemonteiro has joined #openstack-nova | 14:58 | |
| *** READ10 has joined #openstack-nova | 15:00 | |
| *** takashin has left #openstack-nova | 15:01 | |
| sahid | mriedem: ok thanks it's fair :) | 15:01 |
| sahid | just have replied | 15:01 |
| *** felipemonteiro_ has joined #openstack-nova | 15:02 | |
| openstackgerrit | Merged openstack/nova master: Change livesnapshot to true by default https://review.openstack.org/454323 | 15:04 |
| jianghuaw | bauzas, sahid, do you have a link where I can get the list of vGPU type ids for mdev? | 15:04 |
| *** felipemonteiro has quit IRC | 15:05 | |
| jianghuaw | per this link: https://libvirt.org/drvnodedev.html | 15:05 |
| openstackgerrit | Merged openstack/nova master: Add live.migration.force.complete to the legacy notification whitelist https://review.openstack.org/506104 | 15:05 |
| jianghuaw | 'The element has one attribute id which holds an official vendor-supplied identifier for the type' | 15:06 |
| openstackgerrit | Merged openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | 15:06 |
| jianghuaw | I guess the type id is exactly mapped into the type name. | 15:07 |
| *** shaner has quit IRC | 15:07 | |
| *** shaner has joined #openstack-nova | 15:07 | |
| *** links has joined #openstack-nova | 15:08 | |
| *** links has quit IRC | 15:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Deprecate idle_timeout in api_database https://review.openstack.org/507006 | 15:22 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Deprecate idle_timeout in api_database https://review.openstack.org/507006 | 15:22 |
| openstackgerrit | Merged openstack/nova-specs master: Clarify wording in Searchlight spec - error conditions https://review.openstack.org/463388 | 15:23 |
| *** moshele has quit IRC | 15:25 | |
| bauzas | jianghuaw: AFAIK, that's vendor specific templates | 15:27 |
| bauzas | jianghuaw: like described in https://www.kraxel.org/blog/2017/01/virtual-gpu-support-landing-upstream/ | 15:27 |
| bauzas | jianghuaw: that's basically the driver which will provide mediated devices | 15:27 |
| bauzas | jianghuaw: well, it will provide the mdev types, and operators will have to create the mdev as stated in the bottom of the page you pointed me | 15:28 |
| *** links has joined #openstack-nova | 15:29 | |
| *** shaner has quit IRC | 15:30 | |
| openstackgerrit | Merged openstack/nova-specs master: Fix pike spec names to match blueprints https://review.openstack.org/500368 | 15:30 |
| mriedem | dansmith: replied in https://review.openstack.org/#/c/505661/ - i think we don't need it now, we should drop it | 15:30 |
| *** shaner has joined #openstack-nova | 15:30 | |
| mriedem | *if we don't need it now | 15:30 |
| dansmith | okay, I'll snip it out the next time I push | 15:31 |
| mriedem | ok, i guess the next patch for me to review is the big kahuna | 15:32 |
| *** takedakn has quit IRC | 15:32 | |
| *** Oku_OS is now known as Oku_OS-away | 15:33 | |
| *** pcaruana has quit IRC | 15:33 | |
| *** pcaruana has joined #openstack-nova | 15:34 | |
| *** ociuhandu has joined #openstack-nova | 15:35 | |
| *** chyka has joined #openstack-nova | 15:36 | |
| dansmith | mriedem: it's not particularly big, but it is the kahuna yeah :) | 15:36 |
| dansmith | mriedem: just think of all the awesome we get once that is in:) | 15:36 |
| dansmith | -353, +91 | 15:37 |
| dansmith | pretty good ratio | 15:37 |
| *** jmlowe has quit IRC | 15:37 | |
| mriedem | if you like that | 15:38 |
| mriedem | oh nvm | 15:39 |
| mriedem | it already merged https://review.openstack.org/#/c/460377/ :) | 15:39 |
| *** dtantsur is now known as dtantsur|bbl | 15:39 | |
| mriedem | stephenfin: an excellent +1 +W there ^ | 15:39 |
| *** ociuhandu has quit IRC | 15:40 | |
| dansmith | oh, I thought you mean the patch that uses the instance_list bit | 15:40 |
| stephenfin | mriedem: God damn it | 15:40 |
| stephenfin | dansmith: Fancy taking a look through it anyway? If I missed something, I imagine we can revert | 15:40 |
| *** trinaths has left #openstack-nova | 15:41 | |
| dansmith | stephenfin: in a bit, I'm in the middle of rebase hell with my migration uuid set | 15:41 |
| stephenfin | (y) | 15:41 |
| *** jmlowe has joined #openstack-nova | 15:44 | |
| *** andreas_s has quit IRC | 15:46 | |
| *** rcernin has quit IRC | 15:48 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122 | 15:52 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: cover migration cases with functional tests https://review.openstack.org/493865 | 15:54 |
| mriedem | ooo pike 16.0.1 was released | 15:54 |
| *** artom has joined #openstack-nova | 15:54 | |
| bauzas | jaypipes: mriedem: dansmith: as we discussed previously, if we say we could just imagine implementing baby steps for VGPU tracking by just amending the current RP that is returned by the virt driver and just add some resource class about how many VGPUs are available, that would mean it would be limited to only one single track of the virtual GPUs, whatever the number of devices we have | 15:55 |
| jaypipes | bauzas: yeah | 15:55 |
| dansmith | bauzas: which would be fine I think | 15:55 |
| bauzas | jaypipes: mriedem: dansmith: to make it clear, once nested RPs would be a thing, we would have the virt driver reporting the inventory per pGPU | 15:55 |
| dansmith | yes | 15:55 |
| jaypipes | bauzas: until n-r-p there would only be a single provider to associate a trait to, so yeah | 15:55 |
| bauzas | okay, we're all in violent agreement then | 15:55 |
| jaypipes | VIOLENCE! | 15:56 |
| mriedem | bauzas: are you going to implement get_inventory for the virt driver then? or munge it into get_available_resources? | 15:56 |
| * dansmith stabs jaypipes in the resource provider | 15:56 | |
| mriedem | i think only ironic and libvirt have the get_inventory method implemented | 15:56 |
| bauzas | mriedem: I was planning to use the new call | 15:56 |
| mriedem | ok | 15:56 |
| dansmith | yes | 15:56 |
| mriedem | jianghuaw would do it for xen, | 15:56 |
| * jaypipes scrapes dansmith's head with a rusty trait | 15:56 | |
| bauzas | mriedem: yeah, libvirt already uses the new interface | 15:56 |
| mriedem | and either cdent or rado (sp?) is i think doing it for vmware | 15:56 |
| mriedem | radu? | 15:57 |
| mriedem | and according to claudiub, there is nothing hyperv *can't* do, so i'm sure he's up for implementing that | 15:57 |
| jaypipes | heh | 15:57 |
| * mriedem will always remember that from the ATL PTG | 15:57 | |
| bauzas | HAH | 15:57 |
| *** baoli_ has joined #openstack-nova | 15:58 | |
| bauzas | honestly, I just feel having vGPU support in Nova would be an interesting usecase for showing how Placement can help :) | 15:58 |
| bauzas | and no longer do any crap stuff like PCIDeviceTracker | 15:58 |
| jianghuaw | mriedem, absolutely I'm happy to take the needed work for xen. | 15:58 |
| *** ociuhandu has joined #openstack-nova | 15:58 | |
| mriedem | cdent: edleafe: one of you want to update this for nova? http://specs.openstack.org/openstack/api-wg/liaisons.html#liaisons | 15:58 |
| jaypipes | lyarwood: I can't believe you didn't list my proposed name of "ffs" for fast-forward skips ;) | 15:59 |
| dansmith | jaypipes: because skip can't be in the name | 15:59 |
| edleafe | mriedem: who should the new liaison be? | 15:59 |
| dansmith | but if we could come up with another word for the s | 15:59 |
| jaypipes | dansmith: I know I was only kidding. :) | 15:59 |
| *** baoli has quit IRC | 15:59 | |
| *** sbezverk has quit IRC | 16:00 | |
| mriedem | how about slop | 16:00 |
| mriedem | skip-level offline (upgrade) process | 16:00 |
| jianghuaw | bauzas, jaypipes: please help to check if you are happy with the revised vGPU spec: https://review.openstack.org/#/c/450122/ | 16:00 |
| dansmith | can't have skip in the name | 16:00 |
| jaypipes | jianghuaw: yep, will do. | 16:00 |
| bauzas | jianghuaw: I'm already on it | 16:00 |
| claudiub | what do I have to do? | 16:01 |
| jaypipes | dansmith: I still like the package repository - integrated clustered kubernetes | 16:01 |
| mriedem | claudiub: make the hyperv driver implement the get_inventory method | 16:01 |
| mriedem | claudiub: to set the stage for vgpu support https://review.openstack.org/#/c/450122/ | 16:01 |
| claudiub | awesome | 16:02 |
| *** bhagyashris has quit IRC | 16:03 | |
| *** pooja_jadhav has quit IRC | 16:03 | |
| jianghuaw | jaypipes, bauzas: thanks both:-) | 16:03 |
| *** psachin has quit IRC | 16:03 | |
| *** NehaAlhat has quit IRC | 16:03 | |
| claudiub | welp, at the moment hyper-v reports some vgpu resources / stats, so i guess we'll have to move that around a bit. | 16:04 |
| *** artom has quit IRC | 16:06 | |
| *** Dinesh_Bhor has quit IRC | 16:07 | |
| *** Apoorva has joined #openstack-nova | 16:09 | |
| *** Apoorva has quit IRC | 16:09 | |
| jianghuaw | bauzas, cool. thanks for +2 on the spec:-) | 16:10 |
| *** Apoorva has joined #openstack-nova | 16:10 | |
| *** links has quit IRC | 16:10 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418 | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456 | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312 | 16:11 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | 16:11 |
| *** itlinux has joined #openstack-nova | 16:12 | |
| *** yufei has joined #openstack-nova | 16:12 | |
| dansmith | mriedem: I just pushed ^ so that that functional change isn't in the series | 16:13 |
| mriedem | are you going to abandon that one? | 16:13 |
| dansmith | sure | 16:13 |
| yufei | could someone help review this, a small patch which remove re-auth logic from ironic client wrapper https://review.openstack.org/#/c/502382/ | 16:14 |
| *** penick has joined #openstack-nova | 16:14 | |
| yufei | keystone session supports to re-auth for 401 error, so re-auth in ironic client is unnecessary now. | 16:15 |
| *** pcaruana has quit IRC | 16:15 | |
| openstackgerrit | Merged openstack/nova master: Add recreate test for unshelve offloaded instance spawn fail https://review.openstack.org/506414 | 16:15 |
| *** itlinux has quit IRC | 16:15 | |
| *** lpetrut has quit IRC | 16:16 | |
| cdent | mriedem: rgerganov is doing get_inventory for vmware, already running into some interesting wrinkles | 16:18 |
| *** sahid has quit IRC | 16:19 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Live Migration sequence diagram https://review.openstack.org/506370 | 16:19 |
| *** ociuhandu has quit IRC | 16:20 | |
| mriedem | yufei: you don't need to +1 your own change :) | 16:20 |
| mriedem | yufei: how is that different from the [service_user] group of config options? | 16:21 |
| *** Tom has quit IRC | 16:21 | |
| *** Tom_____ has joined #openstack-nova | 16:21 | |
| *** liuyulong_ has quit IRC | 16:22 | |
| bauzas | mmm, firefox loves me when I github blame the compute manager file | 16:27 |
| *** itlinux has joined #openstack-nova | 16:27 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add get_node_uuid() helper to ResourceTracker https://review.openstack.org/506730 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Move allocation manipulation out of drop_move_claim() https://review.openstack.org/498947 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make allocation cleanup honor new by-migration rules https://review.openstack.org/498948 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 16:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 16:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Refactor ServerMovingTests for non-move tests https://review.openstack.org/507195 | 16:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add recreate test for unshelve offloaded instance spawn fail https://review.openstack.org/507196 | 16:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Remove allocations when unshelve fails on host https://review.openstack.org/507197 | 16:29 |
| *** jpena is now known as jpena|brb | 16:31 | |
| yufei | @mriedem, thanks for review, keystone session support to re-auth for 401 error no matter what is configred in [service_user], the code is at https://github.com/openstack/python-keystoneclient/blob/b29f478f28c4989156cfe87392cbd308e3f55c1e/keystoneclient/session.py#L432 | 16:31 |
| yufei | I’m not sure that whether I understand your question correctly | 16:31 |
| *** ralonsoh has quit IRC | 16:33 | |
| mriedem | yufei: i'm trying to sort out when we can rely on ksa to re-auth and when we need to configure a service user token for re-auth | 16:33 |
| mriedem | for example, if ksa is going to re-auth for us, then why do we even need service user tokens? | 16:34 |
| yufei | the reauth logic was useful when we don’t use keystone session to build ironic client in the old version of nova, I forget the actual version of that, let me find | 16:35 |
| mriedem | yufei: https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/use-service-tokens.html if you're not familiar | 16:36 |
| openstackgerrit | Merged openstack/nova master: Remove allocations when unshelve fails on host https://review.openstack.org/506458 | 16:38 |
| openstackgerrit | Merged openstack/nova master: Add _wait_for_action_fail_completion to InstanceHelperMixin https://review.openstack.org/506660 | 16:38 |
| yufei | I remember that, when user use a token to build keystone client, keystone is unable to reauth for 401 error, but nova service use username and password to build keystone session client to call neutron or neutron, keystone will help to reauth in this way | 16:40 |
| mriedem | maybe edmondsw can reply in the patch, he knows more about ksa than i do | 16:41 |
| edleafe | mriedem: do you have a sucke^H^H^H^H volunteer for API-SIG liaison for Nova? | 16:42 |
| mriedem | edleafe: it's either you or cdent | 16:43 |
| yufei | for live-migration or some other actions, user send a request to nova service with token is context, if token expired, action failed. | 16:43 |
| edleafe | mriedem: it can't be | 16:43 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | 16:43 |
| yufei | thanks, mriedem | 16:43 |
| edleafe | the liaisons are the ones who review changes *after* the cores have approved them | 16:43 |
| mriedem | edleafe: then we just remove that entry, since gilliard isn't working on openstack | 16:43 |
| edleafe | mriedem: ok, I can remove it now, and maybe you can find someone at the next Nova meeting | 16:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 16:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | 16:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 16:44 |
| *** vishwanathj has joined #openstack-nova | 16:45 | |
| yufei | @edmondsw could you please help take a look at this patch when you are free? this is a patch about nova call ironic with keystone session, https://review.openstack.org/#/c/502382/ | 16:45 |
| *** namnh has joined #openstack-nova | 16:45 | |
| *** Tom_____ has quit IRC | 16:46 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 16:47 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | 16:47 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 16:47 |
| *** Swami has joined #openstack-nova | 16:47 | |
| openstackgerrit | Masanori Itoh proposed openstack/nova master: Fix IoOpsFilter test case class name. https://review.openstack.org/507205 | 16:49 |
| *** mvk has quit IRC | 16:49 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | 16:51 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 16:51 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 16:51 |
| *** sree has joined #openstack-nova | 16:52 | |
| *** crushil has quit IRC | 16:55 | |
| *** dtantsur|bbl is now known as dtantsur | 16:58 | |
| *** derekh has quit IRC | 16:59 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add functional for live migrate delete https://review.openstack.org/499583 | 17:00 |
| *** lpetrut has joined #openstack-nova | 17:00 | |
| *** yufei has quit IRC | 17:00 | |
| gibi | mriedem: Can I fast approve https://review.openstack.org/#/c/499583? It was already on the gate but got merge conflicts. So I resolved a trivial merge conflict and pushed it back | 17:01 |
| *** namnh has quit IRC | 17:04 | |
| mriedem | yeah | 17:04 |
| *** crushil has joined #openstack-nova | 17:04 | |
| *** namnh has joined #openstack-nova | 17:04 | |
| *** avolkov has quit IRC | 17:05 | |
| *** slaweq_ has joined #openstack-nova | 17:05 | |
| gibi | mriedem: thanks. This was my first fast approve so I wanted to make sure :) | 17:06 |
| *** slaweq_ has quit IRC | 17:07 | |
| *** namnh has quit IRC | 17:08 | |
| *** sbezverk has joined #openstack-nova | 17:09 | |
| *** masber has quit IRC | 17:10 | |
| jianghuaw | jaypipes, the default value 1 for display heads is requested by bauzas to cover the case where the head-counts don't return. | 17:13 |
| jaypipes | jianghuaw: understood, but that's wrong. :) | 17:13 |
| jaypipes | jianghuaw: if the virt driver cannot determine head counts, it simply should not write an inventory record for the VGPU_DISPLAY_HEAD resource class | 17:14 |
| jianghuaw | it's possible if the vGPU doesn't support multiple-heads. By default at least it should support one head? | 17:14 |
| jianghuaw | Then does it mean these vGPU can't be scheduled if it's requesting for one display head? | 17:15 |
| openstackgerrit | Merged openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442 | 17:15 |
| jaypipes | jianghuaw: no, the opposite. by not having an inventory record for VGPU_DISPLAY_HEAD, that means all the pGPUs (or GPU groups) would be schedulable. | 17:16 |
| cdent | jaypipes: in a choice between another process (just for placement-periodics) and cron, I’d definitely choose cron | 17:16 |
| jaypipes | jianghuaw: if the hypervisor cannot determine display heads, then the flavor should not request VGPU_DISPLAY_HEAD resources | 17:16 |
| cdent | mriedem, edleafe, dansmith: your thoughts on jay’s comments on https://review.openstack.org/#/c/504540/ encouraged | 17:17 |
| jaypipes | jianghuaw: the problem of having an inventory of VGPU_DISPLAY_HEAD=1 would mean only a single instance requesting VGPU_DISPLAY_HEAD:1 would be possible on the host. | 17:17 |
| *** pchavva has quit IRC | 17:18 | |
| jianghuaw | jaypipes, ah, I see. | 17:18 |
| jianghuaw | yes, you're correct. | 17:18 |
| jianghuaw | jaypipes, thanks for pointing it out:-) | 17:19 |
| *** slaweq_ has joined #openstack-nova | 17:20 | |
| mriedem | gibi: oops something failed in the rebase here https://review.openstack.org/#/c/499539/9 | 17:20 |
| *** harlowja has joined #openstack-nova | 17:20 | |
| gibi | gibi: looking | 17:21 |
| gibi | mriedem: looking | 17:21 |
| mriedem | https://review.openstack.org/#/c/499539/9 | 17:21 |
| mriedem | oops | 17:21 |
| mriedem | TypeError: _delete_and_check_allocations() takes exactly 2 arguments (4 given) | 17:21 |
| gibi | OK I can fix that up quickly | 17:21 |
| *** yamahata has joined #openstack-nova | 17:23 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 17:24 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | 17:24 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 17:24 |
| *** gszasz has quit IRC | 17:24 | |
| gibi | mriedem: it should be better now | 17:24 |
| mriedem | ok | 17:25 |
| *** pchavva has joined #openstack-nova | 17:26 | |
| *** mvk has joined #openstack-nova | 17:27 | |
| *** jpena|brb is now known as jpena | 17:29 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122 | 17:31 |
| *** moshele has joined #openstack-nova | 17:31 | |
| jianghuaw | jaypipes, see the revised spec ^ | 17:32 |
| jianghuaw | thanks. | 17:32 |
| *** egonzalez has joined #openstack-nova | 17:34 | |
| *** crushil_ has joined #openstack-nova | 17:34 | |
| *** moshele has quit IRC | 17:34 | |
| *** ijw has joined #openstack-nova | 17:35 | |
| *** baoli_ has quit IRC | 17:35 | |
| jianghuaw | bauzas, I did some minor change basing on jaypipes' comments. Need your help to review it again. thanks. https://review.openstack.org/#/c/450122 | 17:35 |
| *** baoli has joined #openstack-nova | 17:38 | |
| *** chyka has quit IRC | 17:39 | |
| *** chyka has joined #openstack-nova | 17:39 | |
| *** itlinux has quit IRC | 17:48 | |
| *** slaweq_ has quit IRC | 17:52 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | 17:53 |
| *** tesseract has quit IRC | 17:53 | |
| jaypipes | bauzas: k, I'm +2 on jianghuaw's spec | 17:53 |
| edmondsw | mriedem yufei replied in https://review.openstack.org/#/c/502382 | 17:55 |
| jianghuaw | jaypipes, thanks very much:-) | 17:55 |
| edmondsw | mriedem yufei I mean that I replied... | 17:56 |
| *** felipemonteiro_ has quit IRC | 17:58 | |
| *** itlinux has joined #openstack-nova | 17:59 | |
| *** baoli has quit IRC | 18:00 | |
| *** sree has quit IRC | 18:00 | |
| *** sree has joined #openstack-nova | 18:00 | |
| *** sree has quit IRC | 18:00 | |
| *** baoli has joined #openstack-nova | 18:01 | |
| *** ijw has quit IRC | 18:05 | |
| *** namnh has joined #openstack-nova | 18:05 | |
| *** ijw has joined #openstack-nova | 18:05 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410 | 18:06 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 18:06 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 18:06 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224 | 18:06 |
| *** Sukhdev has joined #openstack-nova | 18:07 | |
| *** sambetts is now known as sambetts|afk | 18:07 | |
| mriedem | py35 unit test job seems to be jacked all of a sudden http://logs.openstack.org/47/498947/6/check/gate-nova-python35/15aee2f/console.html#_2017-09-25_16_45_38_542562 | 18:09 |
| *** namnh has quit IRC | 18:10 | |
| *** rcernin has joined #openstack-nova | 18:11 | |
| *** lpetrut has quit IRC | 18:11 | |
| *** jpena is now known as jpena|off | 18:11 | |
| *** Sukhdev has quit IRC | 18:12 | |
| dansmith | mriedem: that's odd | 18:13 |
| *** crushil_ has quit IRC | 18:14 | |
| *** jmlowe has quit IRC | 18:15 | |
| mriedem | i've seen that in 2 jobs now | 18:16 |
| mriedem | 2 changes i mean | 18:16 |
| *** penick has quit IRC | 18:16 | |
| openstackgerrit | Merged openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122 | 18:17 |
| dansmith | is there something we can do about the quadrupling of the output on py35 due to warnings? | 18:18 |
| mriedem | sdague was working on slimming some of those down already | 18:18 |
| dansmith | okay | 18:18 |
| sdague | I never got over to the py35 side | 18:18 |
| sdague | I was trying to trim on the other side | 18:18 |
| dansmith | mriedem: so one of my patches failed with that, but it does show a test fail in the testr results | 18:18 |
| sdague | I could take a whack at the py35 one though | 18:18 |
| mriedem | dansmith: yeah i was looking at your migration one | 18:19 |
| mriedem | that's the link | 18:19 |
| mriedem | Failed: 0 | 18:19 |
| dansmith | ah okay | 18:19 |
| mriedem | {0} nova.tests.unit.test_rpc.TestRPC.test_cleanup_legacy_notifier_null [] ... inprogress | 18:19 |
| mriedem | timeout? | 18:19 |
| mriedem | i think that's a known one | 18:19 |
| sdague | mriedem: yeh | 18:19 |
| mriedem | yeah | 18:19 |
| sdague | that's the issue that mtreinish is going to have to look at | 18:19 |
| sdague | because that's something about the worker not returning | 18:20 |
| sdague | it could be subunit parsing | 18:20 |
| dansmith | does it actually take a while? | 18:20 |
| dansmith | maybe that's the bug I'm seeing locally where some test worker takes a long time | 18:21 |
| *** slaweq_ has joined #openstack-nova | 18:21 | |
| *** ijw has quit IRC | 18:21 | |
| sdague | dansmith: I don't know, I've only seen it in the gate | 18:21 |
| *** dtantsur is now known as dtantsur|afk | 18:21 | |
| dansmith | sdague: I only see it on my 24T boxes | 18:21 |
| mriedem | i've seen that TestRPC class timeout in lots of unit test jobs randomly | 18:21 |
| mriedem | probably because there is global state involved in those | 18:22 |
| dansmith | although in this gate run, it shows one worker as N/A which is not what I see | 18:22 |
| dansmith | is it getting killed? | 18:22 |
| mriedem | i just got this locally too | 18:22 |
| mriedem | - Worker 1 (2790 tests) => N/A | 18:22 |
| *** vladikr has quit IRC | 18:24 | |
| mriedem | oh gdi, TestRPC doesn't use nova.test.TestCase so we can't run it serialized either | 18:24 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1685333 | 18:24 |
| openstack | Launchpad bug 1685333 in OpenStack Compute (nova) "Fatal Python error: Cannot recover from stack overflow. - in py35 unit test job" [High,Confirmed] | 18:24 |
| *** ijw has joined #openstack-nova | 18:25 | |
| mriedem | sec, working on something | 18:25 |
| *** penick has joined #openstack-nova | 18:28 | |
| *** jpena|off has quit IRC | 18:30 | |
| sdague | mriedem: oh neat, if it doesn't inherit from nova.test.TestCase, that means that it doesn't have timeout support | 18:31 |
| *** ijw has quit IRC | 18:31 | |
| *** ijw has joined #openstack-nova | 18:31 | |
| *** vladikr has joined #openstack-nova | 18:32 | |
| mriedem | yeah, twiddling this a bit | 18:32 |
| sdague | mriedem: I could split up the base test class into some better slices so we don't loose that | 18:32 |
| mriedem | that's kind of what i'm doing | 18:32 |
| sdague | ok | 18:32 |
| mriedem | but also looking at the oslo.concurrency lock fixtures | 18:32 |
| mriedem | these tests muck with the global nova.rpc notification variables, so would need to be locked i think | 18:33 |
| sdague | ah... fun | 18:33 |
| sdague | well the segmentation is needed anyways | 18:33 |
| *** ltomasbo has quit IRC | 18:35 | |
| bauzas | jaypipes: actually, when reading the nested RP spec, efried made me think of something | 18:36 |
| *** ltomasbo has joined #openstack-nova | 18:37 | |
| bauzas | jaypipes: say I have two children RPs with eahc of them providing a specific inventory | 18:37 |
| *** jpena|off has joined #openstack-nova | 18:37 | |
| bauzas | jaypipes: do you see a way to ask for something like 'take me all of that', or just "take me that one" | 18:37 |
| bauzas | the problem I have in mind is with resource usage that is sharded between children RPs | 18:38 |
| *** slaweq_ has quit IRC | 18:41 | |
| efried | bauzas ++. If you recall, jaypipes volunteered to write a spec for the new syntax for this. I started scribbling some scenarios it needed to cover, and this was one of them. Take a look at the last two chunks starting L81 here: https://etherpad.openstack.org/p/nova-multi-alloc-request-syntax-brainstorm | 18:42 |
| efried | bauzas This is in contrast to the scenario on L71 where we explicitly want the inventory to come from *different* RPs. | 18:44 |
| *** itlinux has quit IRC | 18:51 | |
| *** yikun_jiang has joined #openstack-nova | 18:53 | |
| *** yangyape_ has joined #openstack-nova | 18:54 | |
| *** yangyapeng has quit IRC | 18:55 | |
| *** yikun has quit IRC | 18:55 | |
| *** dave-mccowan has quit IRC | 18:56 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make TestRPC inherit from the base nova TestCase https://review.openstack.org/507239 | 18:59 |
| mriedem | sdague: dansmith: ^ got distracted | 18:59 |
| mriedem | damn pep8 | 19:01 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make TestRPC inherit from the base nova TestCase https://review.openstack.org/507239 | 19:01 |
| *** tssurya has quit IRC | 19:08 | |
| *** felipemonteiro has joined #openstack-nova | 19:08 | |
| *** jose-phillips has quit IRC | 19:08 | |
| *** lpetrut has joined #openstack-nova | 19:09 | |
| *** baoli has quit IRC | 19:13 | |
| *** jose-phillips has joined #openstack-nova | 19:13 | |
| *** lpetrut has quit IRC | 19:14 | |
| mriedem | alright danny boy, i've got a fresh devstack with the fake virt driver and noop quota driver, time to create 100 VMs | 19:14 |
| *** lifeless has quit IRC | 19:14 | |
| *** lifeless has joined #openstack-nova | 19:15 | |
| sdague | mriedem: so, honestly, I wouldn't do it that way | 19:19 |
| *** jmlowe has joined #openstack-nova | 19:19 | |
| sdague | mriedem: let me take a different approach | 19:19 |
| *** dave-mccowan has joined #openstack-nova | 19:24 | |
| mriedem | i wonder if multi-create 100 vms will give me an http timeout | 19:26 |
| cfriesen | policy question....would nova ever backport a microversion bump to a stable release? | 19:27 |
| *** READ10 has quit IRC | 19:27 | |
| mriedem | nope | 19:27 |
| mriedem | you want your embedded flavor thing in ocata don't you | 19:27 |
| cfriesen | heh...nope. was just planning something | 19:27 |
| sdague | cfriesen: microversions aren't really backportable | 19:28 |
| cfriesen | was just wondering if I'd have to plan to be able to handle microversion bumps in bugfixes. | 19:28 |
| cfriesen | this makes it easier | 19:29 |
| *** dave-mccowan has quit IRC | 19:32 | |
| *** dave-mccowan has joined #openstack-nova | 19:32 | |
| *** baoli has joined #openstack-nova | 19:34 | |
| *** itlinux has joined #openstack-nova | 19:35 | |
| *** slaweq_ has joined #openstack-nova | 19:36 | |
| *** awaugama has quit IRC | 19:37 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | 19:37 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 19:37 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Set the Pike release version for scheduler RPC https://review.openstack.org/507245 | 19:37 |
| *** felipemonteiro_ has joined #openstack-nova | 19:37 | |
| *** slaweq_ has quit IRC | 19:38 | |
| *** slaweq_ has joined #openstack-nova | 19:38 | |
| *** baoli has quit IRC | 19:39 | |
| *** baoli has joined #openstack-nova | 19:40 | |
| *** slaweq_ has quit IRC | 19:41 | |
| *** felipemonteiro has quit IRC | 19:41 | |
| *** baoli has quit IRC | 19:41 | |
| dansmith | mriedem: you're doing the fake virt devstack testing on my switch-to-the-new-thing patch? | 19:42 |
| *** baoli has joined #openstack-nova | 19:42 | |
| mriedem | dansmith: trying to get a baseline | 19:42 |
| mriedem | straight curl with 100 instances is trivial | 19:42 |
| mriedem | .004s | 19:42 |
| *** jmlowe has quit IRC | 19:42 | |
| mriedem | so booting 500 more | 19:42 |
| mriedem | had to switch to curl to take out extra 5 seconds novaclient adds to nova list | 19:43 |
| dansmith | heh | 19:43 |
| openstackgerrit | Sean Dague proposed openstack/nova master: Break out BasicTestCase https://review.openstack.org/507253 | 19:45 |
| sdague | mriedem: that's an alternative | 19:45 |
| *** sapd_ has joined #openstack-nova | 19:45 | |
| *** slaweq_ has joined #openstack-nova | 19:46 | |
| *** sapd has quit IRC | 19:49 | |
| *** jmlowe has joined #openstack-nova | 19:50 | |
| *** cdent has quit IRC | 19:53 | |
| mriedem | commented | 19:53 |
| mriedem | i like mine better but i'm biased | 19:53 |
| openstackgerrit | ayoung proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148 | 19:55 |
| dansmith | I would rather not have another base-base-base test case class | 19:55 |
| dansmith | I know it's not very OO, but we have a lot already | 19:55 |
| mriedem | booting another 400 to get me to 1000 | 19:59 |
| mriedem | i figure that's a nice round number, and is our default limit | 20:00 |
| dansmith | mriedem: is that because you're not getting a large number to measure? | 20:00 |
| mriedem | with 600 i'm getting up around 2 seconds | 20:00 |
| melwitt | you're getting hundreds booted without any in ERROR state? | 20:01 |
| dansmith | oh okay that's pretty reasonable | 20:01 |
| mriedem | melwitt: f yeah | 20:01 |
| mriedem | i'm da bomb | 20:01 |
| melwitt | that's coolness | 20:01 |
| mriedem | i have figured out how to hyperconverge this 8gb devstack vm | 20:01 |
| mriedem | for hyper throughput | 20:01 |
| mriedem | using state of the art best of breed solutions | 20:02 |
| mriedem | melwitt: i can share my secret with you if you'll keep it quiet | 20:02 |
| melwitt | heh | 20:03 |
| mriedem | http://paste.openstack.org/show/621867/ | 20:03 |
| * melwitt clicks expecting to see MOGAN | 20:03 | |
| mriedem | ha | 20:03 |
| *** ragiman has quit IRC | 20:04 | |
| *** liverpooler has quit IRC | 20:04 | |
| *** baoli has quit IRC | 20:04 | |
| *** egonzalez has quit IRC | 20:05 | |
| melwitt | ah, FakeDriver. good thinkin | 20:05 |
| *** namnh has joined #openstack-nova | 20:07 | |
| mriedem | also | 20:08 |
| mriedem | (1) fake driver, (2) no quotas (3) no networking | 20:08 |
| mriedem | because there are port quotas in neutron | 20:08 |
| mriedem | default is 50 | 20:08 |
| mriedem | so specify --nic none when doing nova boot | 20:09 |
| mriedem | dansmith: when i pull your change down, i just have to restart the api right? | 20:10 |
| melwitt | ah, right. cool | 20:10 |
| dansmith | mriedem: yeah I would think so | 20:10 |
| dansmith | er, yeah, definitely | 20:11 |
| mriedem | stack@devstack:~/devstack$ nova list | grep -c ACTIVE | 20:11 |
| mriedem | 1000 | 20:11 |
| *** rcernin has quit IRC | 20:11 | |
| mriedem | alright, i guess i'll do curl like 10 times to get an average | 20:11 |
| *** namnh has quit IRC | 20:11 | |
| *** esberglu has quit IRC | 20:13 | |
| *** baoli has joined #openstack-nova | 20:13 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova-specs master: Add multiattach support to Nova https://review.openstack.org/499777 | 20:14 |
| *** jpena|off has quit IRC | 20:14 | |
| *** ltomasbo has quit IRC | 20:14 | |
| *** esberglu_ has joined #openstack-nova | 20:16 | |
| *** ltomasbo has joined #openstack-nova | 20:17 | |
| *** jpena|off has joined #openstack-nova | 20:18 | |
| *** acormier has joined #openstack-nova | 20:18 | |
| *** esberglu_ has quit IRC | 20:21 | |
| mriedem | dansmith: https://etherpad.openstack.org/p/nova-instance-list | 20:23 |
| mriedem | takes 4.495s on average | 20:23 |
| mriedem | to list 1000 instances | 20:23 |
| mriedem | take a look at ^ and see if there is anything i'm doing wrong or should get before i pull your change down | 20:23 |
| dansmith | you should multiply that value by ten | 20:24 |
| mriedem | the average? | 20:24 |
| dansmith | yes. | 20:24 |
| mriedem | why | 20:24 |
| dansmith | so that it will look really bad compared to whatever the number is that you get with my patch | 20:24 |
| melwitt | haha | 20:25 |
| mriedem | oh yeah server/details too | 20:27 |
| mriedem | for the uber joins | 20:27 |
| melwitt | +1 | 20:28 |
| dansmith | I would hope that details would be the same for both | 20:28 |
| dansmith | based on the internals | 20:28 |
| mriedem | it's not | 20:28 |
| dansmith | no, I mean, between this and my patch | 20:28 |
| dansmith | mriedem: you won' | 20:28 |
| dansmith | mriedem: you won't have two cells in your test with my patch right? | 20:28 |
| *** smatzek has quit IRC | 20:29 | |
| *** penick has quit IRC | 20:29 | |
| mriedem | correct, just cell0 and cell1 | 20:29 |
| dansmith | mriedem: something you might try to do is create 500 active and 500 that fail to schedule so you get them in cell0 and cell1 | 20:29 |
| dansmith | and the compare those numbers | 20:29 |
| *** esberglu has joined #openstack-nova | 20:29 | |
| mriedem | not sure how easy it would be to get them to fail to schedule w/o hacking the scheduler to just raise NoValidHost | 20:30 |
| mriedem | not that that is hard | 20:30 |
| dansmith | mriedem: you could create a flavor that has a billion vcpus or something right? | 20:30 |
| *** pchavva has quit IRC | 20:30 | |
| mriedem | fake driver and noop quota driver | 20:31 |
| mriedem | so i don't think that would matter | 20:31 |
| mriedem | and w/o the core/ram/disk filter we don't check limits in the compute claim | 20:31 |
| mriedem | although that woudn't help since then they'd be in the cell1 db | 20:32 |
| mriedem | i could just stop nova-scheduler :) | 20:32 |
| *** penick has joined #openstack-nova | 20:33 | |
| *** nicolasbock has quit IRC | 20:35 | |
| dansmith | mriedem: more vcpus than the compute has will make it fail | 20:36 |
| dansmith | I'm just saying, I expect we might see a performance _gain_ with my stuff if you can arrange to have some instances in cell0 | 20:36 |
| mriedem | true | 20:36 |
| dansmith | on top of the correctness thing, since right now cell0 always sorts above | 20:36 |
| mriedem | vcpus = 1000 is what's in the fake driver, so yeah | 20:37 |
| mriedem | i'll create a flavor with 2000 CPUs | 20:37 |
| mriedem | would need to delete 500 of these and then archive them | 20:37 |
| dansmith | mriedem: remember the overcommit value | 20:38 |
| dansmith | if you haven't set that then, 16x | 20:38 |
| mriedem | so flavor-create -vcpu 32000 mainframe | 20:39 |
| mriedem | got it | 20:39 |
| dansmith | haha | 20:39 |
| mriedem | it's for my HPC flavor | 20:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 20:40 |
| mriedem | 8.55s average for GET /servers/detail | 20:41 |
| mriedem | so nearly double | 20:41 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_certificates to REST API https://review.openstack.org/486204 | 20:42 |
| dansmith | tbh, I'm surprised it's not worse than that given what I know of the internals | 20:42 |
| *** felipemonteiro_ has quit IRC | 20:43 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:43 | |
| mriedem | that reminds me of a thread i meant to pull at one point, but i thought based on microversion we could be lazy-loading other things in the server view | 20:44 |
| mriedem | like tags | 20:44 |
| mriedem | oh, AND, | 20:44 |
| mriedem | if you have instances in error state, we'll lazy-load fault | 20:44 |
| mriedem | from the api | 20:44 |
| dansmith | not anymore | 20:44 |
| dansmith | this set rips out the fault lazy-loading remember | 20:45 |
| mriedem | https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/views/servers.py#L148 | 20:45 |
| mriedem | https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/views/servers.py#L164 | 20:45 |
| mriedem | yeah, | 20:45 |
| mriedem | just thinking about before that | 20:45 |
| dansmith | mriedem: yeah I rip that out | 20:45 |
| dansmith | hard. | 20:45 |
| mriedem | how about tags? | 20:45 |
| dansmith | tags are joined, AFAIK | 20:45 |
| mriedem | i should probably be running this with the latest microversion... | 20:45 |
| dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2229-L2229 | 20:47 |
| mriedem | i'm not using filters | 20:47 |
| *** jmlowe has quit IRC | 20:47 | |
| mriedem | so if you're listing servers with details with microversion>=2.26, we lazy-load tags on all of them... | 20:47 |
| mriedem | https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/views/servers.py#L164 | 20:47 |
| mriedem | let me do that quick and see if i'm right | 20:48 |
| jaypipes | efried: tried to answer your query in the traits spec as best I can. this stuff is icky. :) | 20:48 |
| efried | jaypipes Looking... | 20:48 |
| jaypipes | efried: all good questions/comments from you, btw. | 20:48 |
| efried | jaypipes Generally speaking, I'm fine with whatever answer; just would like it to be stated in the spec (docs, whatever) | 20:49 |
| jaypipes | totes | 20:49 |
| efried | jaypipes Thanks for that. I feel like I'm being a PITA. | 20:49 |
| jaypipes | no, not at all. | 20:49 |
| jaypipes | I agree that more explicit is better. it's good you're raising these points. | 20:49 |
| *** penick_ has joined #openstack-nova | 20:49 | |
| jaypipes | sshh, everybody quiet! penick has joined.. | 20:50 |
| -openstackstatus- NOTICE: The infra team is bringing Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information. | 20:50 | |
| * jaypipes provides hugops to clarkb, fungi, and jeblair as they embark upon a Zuul-journey | 20:51 | |
| *** penick has quit IRC | 20:52 | |
| *** liusheng has quit IRC | 20:52 | |
| *** liusheng has joined #openstack-nova | 20:53 | |
| fungi | hugops as a service! | 20:53 |
| jaypipes | fungi: :) | 20:54 |
| *** burt has quit IRC | 20:55 | |
| *** itlinux has quit IRC | 20:56 | |
| mriedem | hmm, very weird, OpenStack-API-Version doesn't seem to work | 20:59 |
| mriedem | sdague: ^ | 20:59 |
| mriedem | i'm doing a curl request with that on a devstack i created today, it doesn't pick up the microversion | 20:59 |
| mriedem | i have to use X-OpenStack-Nova-API-Version | 21:00 |
| openstackgerrit | Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | 21:00 |
| mriedem | oh nvm, i see, need to specify compute | 21:03 |
| mriedem | OpenStack-API-Version: compute 2.53 | 21:03 |
| dansmith | mriedem: actually, I wonder if you can create your broken instances by specifying a target host that doesn't exist? | 21:03 |
| mriedem | yeah probably | 21:03 |
| mriedem | i'm not running under the admin tenant... | 21:04 |
| mriedem | which might have other wrinkles...like checking the host status | 21:04 |
| *** gouthamr has quit IRC | 21:04 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 21:04 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 21:04 |
| mriedem | https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id14 | 21:04 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 21:04 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 21:04 |
| dansmith | jaypipes: ready for another flogging I think ^ | 21:05 |
| mriedem | microversion >= 2.16 GET /servers/detail will lookup host status if you're admin | 21:05 |
| mriedem | per instance | 21:05 |
| jaypipes | dansmith: I'll ready my cat-o-nine-tails then | 21:05 |
| dansmith | jaypipes: oooh, you know what I liiiiike | 21:06 |
| jaypipes | lol | 21:06 |
| *** edmondsw has quit IRC | 21:07 | |
| *** namnh has joined #openstack-nova | 21:07 | |
| *** edmondsw has joined #openstack-nova | 21:10 | |
| mriedem | dansmith: 15.73s average to list servers with details at microversion 2.53 | 21:11 |
| mriedem | double that of listing servers with microversion 2.1 | 21:11 |
| *** ijw has quit IRC | 21:11 | |
| dansmith | that sucks | 21:11 |
| dansmith | this is all baseline still right? | 21:12 |
| *** namnh has quit IRC | 21:12 | |
| *** ijw has joined #openstack-nova | 21:12 | |
| mriedem | ypu | 21:12 |
| mriedem | *up | 21:12 |
| mriedem | gdi | 21:12 |
| mriedem | YES | 21:12 |
| mriedem | and that's demo tenant | 21:12 |
| mriedem | fuck man, looking at the code for microversion 2.16 if you're an admin, we aren't caching host status | 21:13 |
| *** edmondsw has quit IRC | 21:14 | |
| mriedem | so i think if you're an admin, listing instances from all tenants, with microversion >= 2.16, we're doing a lazy-load of instance.services on every instance | 21:15 |
| mriedem | and that does a server group is_up call per service if it's not forced_down or disabled | 21:16 |
| mriedem | jesus | 21:16 |
| mriedem | oh nvm, for instance list it's a bit smarter | 21:17 |
| mriedem | whew | 21:17 |
| *** thorst_ has quit IRC | 21:19 | |
| *** tssurya has joined #openstack-nova | 21:24 | |
| *** jmlowe has joined #openstack-nova | 21:25 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Don't monkey patch eventlet in functional or unit tests https://review.openstack.org/506668 | 21:26 |
| mriedem | ah we don't lazy-load tags https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/servers.py#L297 | 21:26 |
| mriedem | we join those in the db query | 21:26 |
| mriedem | same for services | 21:27 |
| dansmith | mriedem: I shot you that link right? | 21:27 |
| mriedem | https://github.com/openstack/nova/blob/29ef2474d9d6a59ae6859b5b01caad350252c23e/nova/api/openstack/compute/servers.py#L295 should probably be version constrained | 21:27 |
| mriedem | dansmith: you send me something else | 21:27 |
| mriedem | you sent me https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2229-L2229 | 21:27 |
| mriedem | but that's in a conditional block | 21:27 |
| dansmith | oh I see, but actually I guess it's because they end up in columns_to_join_new | 21:28 |
| mriedem | https://review.openstack.org/#/c/355558/ | 21:28 |
| mriedem | ^ doesn't go far enough | 21:28 |
| dansmith | that's services | 21:29 |
| mriedem | i know, | 21:29 |
| dansmith | but yeah, that won't do anything I think, | 21:29 |
| dansmith | because it's not in the _expected_cols() thing in instance | 21:30 |
| *** tssurya has quit IRC | 21:31 | |
| mriedem | it worked because it was in _INSTANCE_OPTIONAL_JOINED_FIELDS | 21:32 |
| mriedem | for services | 21:32 |
| mriedem | fault wasn't before queens | 21:32 |
| dansmith | services is in optional? | 21:32 |
| * dansmith looks | 21:32 | |
| mriedem | yup | 21:32 |
| dansmith | oh we actually have a .services on instance | 21:33 |
| dansmith | I didn't remember | 21:33 |
| mriedem | yeah | 21:35 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1719460 | 21:35 |
| openstack | Launchpad bug 1719460 in OpenStack Compute (nova) "instance.services loading when listing instances regardless of microversion" [Medium,Triaged] | 21:35 |
| mriedem | i think ^ sums it up | 21:35 |
| openstackgerrit | Ed Leafe proposed openstack/nova-specs master: Re-propose nested resource providers spec https://review.openstack.org/505209 | 21:36 |
| *** jheroux has quit IRC | 21:36 | |
| *** MVenesio has quit IRC | 21:37 | |
| mriedem | btw, deleting 1000 instances via the API takes awhile | 21:37 |
| *** MVenesio has joined #openstack-nova | 21:37 | |
| *** thorst has joined #openstack-nova | 21:39 | |
| *** MVenesio has quit IRC | 21:41 | |
| *** dave-mccowan has quit IRC | 21:43 | |
| *** tidwellr has quit IRC | 21:43 | |
| *** thorst has quit IRC | 21:45 | |
| *** jose-phillips has quit IRC | 21:46 | |
| *** jose-phillips has joined #openstack-nova | 21:47 | |
| *** jose-phillips has quit IRC | 21:48 | |
| openstackgerrit | Merged openstack/nova master: [placement] correct error on bad resource class in allocation https://review.openstack.org/502205 | 21:49 |
| *** armax has joined #openstack-nova | 21:49 | |
| efried | mriedem Can I get some love on https://review.openstack.org/#/c/488137/ so I can get the rest of the series moving? | 21:52 |
| mriedem | how about tough love | 21:53 |
| efried | I expect nothing less from you | 21:53 |
| *** liverpooler has joined #openstack-nova | 21:55 | |
| *** crushil has quit IRC | 21:57 | |
| *** john51 has joined #openstack-nova | 21:59 | |
| *** claudiub has quit IRC | 22:00 | |
| mriedem | efried: your unit tests failed | 22:01 |
| *** john51_ has quit IRC | 22:01 | |
| efried | sonofa... Okay, I think I know what happened. | 22:02 |
| efried | Okay, since I'm going to need an edit anyway, I guess I might as well rebase... | 22:03 |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 22:04 |
| *** slaweq_ has quit IRC | 22:04 | |
| *** thorst has joined #openstack-nova | 22:06 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 22:06 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 22:06 |
| efried | mriedem That ought to fix it ^ | 22:06 |
| *** jose-phillips has joined #openstack-nova | 22:07 | |
| *** jose-phillips has quit IRC | 22:07 | |
| mriedem | well, meanwhile +2 on https://review.openstack.org/#/c/506370/ again | 22:08 |
| *** namnh has joined #openstack-nova | 22:08 | |
| efried | whee! | 22:08 |
| mriedem | good god it takes a long time to delete 1000 vms | 22:09 |
| dansmith | mriedem: is it actually deleting things or hung? | 22:10 |
| mriedem | down to 512 left | 22:11 |
| *** gouthamr has joined #openstack-nova | 22:12 | |
| *** namnh has quit IRC | 22:13 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 22:15 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Don't fix protocol-less glance api_servers anymore https://review.openstack.org/505317 | 22:15 |
| *** jpena|off has quit IRC | 22:17 | |
| jaypipes | mriedem: but don't you know? each launch of a VM does 400 calls to the database... | 22:17 |
| *** jpena|off has joined #openstack-nova | 22:23 | |
| jaypipes | dansmith: easy one from cdent: https://review.openstack.org/#/c/495913/ | 22:23 |
| *** yamahata has quit IRC | 22:29 | |
| *** moshele has joined #openstack-nova | 22:29 | |
| dansmith | jaypipes: I'm pretty meh on stuff like that in o.vo | 22:32 |
| dansmith | doesn't save anything having that in the library and makes it harder to trace why you have these two fields in your object | 22:32 |
| dansmith | but whatever | 22:32 |
| jaypipes | dansmith: yeah? seems like it's generally useful, no? | 22:32 |
| dansmith | no, it "saves" two lines from a single object in a project, or four for multiples, but makes chasing down the definitions a cross-repo venture | 22:33 |
| jaypipes | dansmith: I had asked cdent to submit that to ovo, so blame me :) | 22:33 |
| dansmith | I blame you. | 22:33 |
| jaypipes | :) | 22:33 |
| *** acormier has quit IRC | 22:35 | |
| *** acormier has joined #openstack-nova | 22:35 | |
| *** acormier has quit IRC | 22:40 | |
| mriedem | jaypipes: this is delete | 22:40 |
| mriedem | your callback joke doesn't fly with me | 22:41 |
| jaypipes | mriedem: :) | 22:42 |
| *** edmondsw has joined #openstack-nova | 22:42 | |
| mriedem | ooo cpu pinning failure in the ML | 22:43 |
| *** baoli has quit IRC | 22:43 | |
| *** dave-mccowan has joined #openstack-nova | 22:44 | |
| mriedem | how many db calls are made when we create a server? | 22:44 |
| *** lyan has quit IRC | 22:45 | |
| mriedem | 375? | 22:45 |
| dansmith | 375, 376, however many it takes | 22:46 |
| *** penick_ has quit IRC | 22:46 | |
| *** edmondsw has quit IRC | 22:47 | |
| *** armax has quit IRC | 22:47 | |
| *** namnh has joined #openstack-nova | 22:48 | |
| mriedem | 7.5 deletes per minute, just counted | 22:49 |
| mriedem | blazing | 22:49 |
| dansmith | you could restack quicker | 22:49 |
| *** namnh has quit IRC | 22:49 | |
| mriedem | probably | 22:49 |
| mriedem | do unstack/stack work? or restack? | 22:50 |
| openstackgerrit | Merged openstack/nova master: Add get_node_uuid() helper to ResourceTracker https://review.openstack.org/506730 | 22:53 |
| dansmith | reboot, stack works :) | 22:53 |
| dansmith | I don't know what still works for teardown after the systemd thing | 22:54 |
| cfriesen | mriedem: do you know where the cost is? cuz that seems higher than it should be. | 22:54 |
| *** chyka has quit IRC | 22:56 | |
| *** crushil has joined #openstack-nova | 22:58 | |
| jaypipes | edleafe: hopefully I explained the whole server-side PUT /allocations/{consumer_uuid} API microversion mismatch scenario properly in that last comment on the alternate hosts spec. | 22:58 |
| jaypipes | efried: we get all your questions answered on the nested resource providers spec? | 23:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 23:04 |
| *** dave-mcc_ has joined #openstack-nova | 23:05 | |
| *** hieulq has quit IRC | 23:07 | |
| *** dave-mccowan has quit IRC | 23:07 | |
| *** takashin has joined #openstack-nova | 23:07 | |
| *** trungnv has quit IRC | 23:08 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | 23:11 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 23:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 23:12 |
| *** felipemonteiro_ has quit IRC | 23:12 | |
| *** acormier has joined #openstack-nova | 23:14 | |
| *** acormier has quit IRC | 23:18 | |
| openstackgerrit | Merged openstack/nova master: Deprecate idle_timeout in api_database https://review.openstack.org/507006 | 23:19 |
| mriedem | cfriesen: this is a devstack env fwiw | 23:24 |
| edleafe | jaypipes: "++ let's ship it. :)" doesn't explain that much :) | 23:29 |
| *** hongbin has quit IRC | 23:29 | |
| jaypipes | edleafe: sorry, meant the selection object spec | 23:30 |
| *** trungnv has joined #openstack-nova | 23:30 | |
| edleafe | jaypipes: just funnin' ya | 23:31 |
| *** itlinux has joined #openstack-nova | 23:36 | |
| *** trungnv has quit IRC | 23:36 | |
| *** chyka has joined #openstack-nova | 23:39 | |
| *** sdague has quit IRC | 23:39 | |
| *** gyee has quit IRC | 23:46 | |
| *** masber has joined #openstack-nova | 23:48 | |
| *** gbarros has joined #openstack-nova | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!