| *** felipemonteiro has quit IRC | 00:02 | |
| *** baoli has joined #openstack-nova | 00:11 | |
| *** ujjain has quit IRC | 00:13 | |
| *** felipemonteiro_ has quit IRC | 00:14 | |
| *** ujjain has joined #openstack-nova | 00:16 | |
| *** ujjain has quit IRC | 00:16 | |
| *** ujjain has joined #openstack-nova | 00:16 | |
| *** baoli has quit IRC | 00:16 | |
| *** acormier has joined #openstack-nova | 00:20 | |
| *** slaweq has quit IRC | 00:23 | |
| *** slaweq has joined #openstack-nova | 00:27 | |
| *** baoli has joined #openstack-nova | 00:32 | |
| *** psachin has joined #openstack-nova | 00:34 | |
| *** huanxie has joined #openstack-nova | 00:40 | |
| *** jichen has joined #openstack-nova | 00:44 | |
| *** ijw has joined #openstack-nova | 00:48 | |
| *** sdague has quit IRC | 00:50 | |
| *** jwcroppe has quit IRC | 00:51 | |
| *** awaugama has joined #openstack-nova | 00:52 | |
| *** jwcroppe has joined #openstack-nova | 00:52 | |
| *** ijw_ has joined #openstack-nova | 00:54 | |
| *** ijw__ has joined #openstack-nova | 00:55 | |
| *** baoli has quit IRC | 00:56 | |
| *** ijw has quit IRC | 00:56 | |
| *** jmlowe has quit IRC | 00:57 | |
| *** armax has joined #openstack-nova | 00:57 | |
| *** acormier has quit IRC | 00:58 | |
| *** hieulq has joined #openstack-nova | 00:58 | |
| *** ijw_ has quit IRC | 00:58 | |
| *** jmlowe has joined #openstack-nova | 00:59 | |
| *** slaweq has quit IRC | 01:00 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 01:02 |
|---|---|---|
| *** baoli has joined #openstack-nova | 01:02 | |
| *** slaweq has joined #openstack-nova | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** baoli has quit IRC | 01:08 | |
| *** thorst has quit IRC | 01:09 | |
| *** yangyapeng has joined #openstack-nova | 01:12 | |
| *** jmlowe has quit IRC | 01:12 | |
| *** artom has quit IRC | 01:14 | |
| *** artom has joined #openstack-nova | 01:15 | |
| *** yamamoto has joined #openstack-nova | 01:21 | |
| *** ijw__ has quit IRC | 01:23 | |
| *** baoli has joined #openstack-nova | 01:26 | |
| *** trungnv has joined #openstack-nova | 01:26 | |
| *** Shunli has joined #openstack-nova | 01:27 | |
| *** jmlowe has joined #openstack-nova | 01:31 | |
| *** slaweq has quit IRC | 01:38 | |
| *** yangyapeng has quit IRC | 01:39 | |
| *** baoli has quit IRC | 01:41 | |
| *** yangyapeng has joined #openstack-nova | 01:41 | |
| *** sbezverk has quit IRC | 01:43 | |
| *** sbezverk has joined #openstack-nova | 01:44 | |
| *** jmlowe has quit IRC | 01:45 | |
| *** slaweq has joined #openstack-nova | 01:45 | |
| *** awaugama has quit IRC | 01:48 | |
| *** hshiina has joined #openstack-nova | 01:50 | |
| *** tbachman_ has joined #openstack-nova | 01:52 | |
| *** litao__ has joined #openstack-nova | 01:53 | |
| *** tbachman has quit IRC | 01:55 | |
| *** tbachman_ is now known as tbachman | 01:55 | |
| *** baoli has joined #openstack-nova | 02:01 | |
| *** slaweq has quit IRC | 02:17 | |
| *** gouthamr has quit IRC | 02:27 | |
| *** slaweq has joined #openstack-nova | 02:28 | |
| *** penick has quit IRC | 02:32 | |
| *** dave-mccowan has quit IRC | 02:35 | |
| *** yamamoto_ has joined #openstack-nova | 02:36 | |
| *** baoli has quit IRC | 02:37 | |
| *** adreznec has quit IRC | 02:37 | |
| *** felipemonteiro_ has joined #openstack-nova | 02:38 | |
| *** adreznec has joined #openstack-nova | 02:39 | |
| *** yamamoto has quit IRC | 02:39 | |
| *** esberglu has quit IRC | 02:43 | |
| *** gcb has joined #openstack-nova | 02:51 | |
| *** slaweq has quit IRC | 02:58 | |
| *** acormier has joined #openstack-nova | 02:58 | |
| *** slaweq has joined #openstack-nova | 02:59 | |
| *** udesale has joined #openstack-nova | 03:00 | |
| *** crushil has joined #openstack-nova | 03:02 | |
| *** crushil has left #openstack-nova | 03:02 | |
| *** crushil has joined #openstack-nova | 03:02 | |
| *** acormier has quit IRC | 03:03 | |
| *** crushil has left #openstack-nova | 03:04 | |
| *** crushil_ has joined #openstack-nova | 03:05 | |
| *** jmlowe has joined #openstack-nova | 03:06 | |
| *** thorst has joined #openstack-nova | 03:10 | |
| *** thorst has quit IRC | 03:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488 | 03:20 |
| *** masber has joined #openstack-nova | 03:20 | |
| *** felipemonteiro_ has quit IRC | 03:22 | |
| *** edand has joined #openstack-nova | 03:23 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: rt: Implement XenAPI get_inventory() method https://review.openstack.org/339204 | 03:23 |
| *** esberglu has joined #openstack-nova | 03:26 | |
| *** edand has quit IRC | 03:30 | |
| *** esberglu has quit IRC | 03:30 | |
| *** Tom_ has joined #openstack-nova | 03:38 | |
| *** hshiina has quit IRC | 03:39 | |
| *** hshiina has joined #openstack-nova | 03:42 | |
| *** coreywright has quit IRC | 03:42 | |
| *** mdnadeem has joined #openstack-nova | 03:43 | |
| *** diga has joined #openstack-nova | 03:44 | |
| *** udesale has quit IRC | 03:46 | |
| *** edmondsw has joined #openstack-nova | 03:53 | |
| *** TuanLA has joined #openstack-nova | 03:54 | |
| *** felipemonteiro_ has joined #openstack-nova | 03:55 | |
| *** coreywright has joined #openstack-nova | 03:55 | |
| *** suresh12 has joined #openstack-nova | 03:56 | |
| *** edmondsw has quit IRC | 03:57 | |
| *** udesale has joined #openstack-nova | 03:59 | |
| *** udesale has quit IRC | 04:00 | |
| *** udesale has joined #openstack-nova | 04:00 | |
| *** slaweq has quit IRC | 04:05 | |
| *** hoonetorg has quit IRC | 04:06 | |
| *** claudiub has joined #openstack-nova | 04:10 | |
| *** penick has joined #openstack-nova | 04:11 | |
| *** browne has joined #openstack-nova | 04:13 | |
| *** felipemonteiro__ has joined #openstack-nova | 04:15 | |
| *** penick has quit IRC | 04:15 | |
| *** browne has quit IRC | 04:15 | |
| *** slaweq has joined #openstack-nova | 04:17 | |
| *** felipemonteiro_ has quit IRC | 04:18 | |
| *** huanxie has quit IRC | 04:18 | |
| *** hoonetorg has joined #openstack-nova | 04:22 | |
| *** huanxie has joined #openstack-nova | 04:24 | |
| *** hongbin_ has quit IRC | 04:24 | |
| *** felipemonteiro__ has quit IRC | 04:27 | |
| *** penick has joined #openstack-nova | 04:38 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:42 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support https://review.openstack.org/449155 | 04:44 |
| *** vvargaszte has joined #openstack-nova | 04:44 | |
| *** vks1 has joined #openstack-nova | 04:46 | |
| *** slaweq has quit IRC | 04:48 | |
| *** sree has joined #openstack-nova | 04:48 | |
| *** vvargaszte has quit IRC | 04:52 | |
| *** Tom_ has quit IRC | 04:55 | |
| *** links has joined #openstack-nova | 04:57 | |
| *** markvoelker has quit IRC | 04:58 | |
| *** markvoelker has joined #openstack-nova | 04:58 | |
| *** slaweq has joined #openstack-nova | 04:59 | |
| *** crushil_ is now known as crushil | 05:05 | |
| *** armax has quit IRC | 05:07 | |
| *** armax has joined #openstack-nova | 05:07 | |
| *** vvargaszte has joined #openstack-nova | 05:07 | |
| *** armax has quit IRC | 05:07 | |
| *** armax has joined #openstack-nova | 05:08 | |
| *** armax has quit IRC | 05:08 | |
| *** armax has joined #openstack-nova | 05:09 | |
| *** armax has quit IRC | 05:09 | |
| *** armax has joined #openstack-nova | 05:10 | |
| *** armax has quit IRC | 05:10 | |
| *** armax has joined #openstack-nova | 05:10 | |
| *** armax has quit IRC | 05:11 | |
| *** armax has joined #openstack-nova | 05:11 | |
| *** armax has quit IRC | 05:11 | |
| *** thorst has joined #openstack-nova | 05:12 | |
| *** armax has joined #openstack-nova | 05:12 | |
| *** armax has quit IRC | 05:12 | |
| *** armax has joined #openstack-nova | 05:13 | |
| *** armax has quit IRC | 05:13 | |
| *** Tom_ has joined #openstack-nova | 05:13 | |
| *** armax has joined #openstack-nova | 05:14 | |
| *** armax has quit IRC | 05:14 | |
| *** armax has joined #openstack-nova | 05:14 | |
| *** armax has quit IRC | 05:15 | |
| *** Tom_ has quit IRC | 05:15 | |
| *** Tom_ has joined #openstack-nova | 05:15 | |
| *** thorst has quit IRC | 05:16 | |
| *** lpetrut_ has joined #openstack-nova | 05:23 | |
| *** suresh12 has quit IRC | 05:23 | |
| *** suresh12 has joined #openstack-nova | 05:24 | |
| *** suresh12 has quit IRC | 05:28 | |
| *** huanxie has quit IRC | 05:32 | |
| *** slaweq has quit IRC | 05:32 | |
| *** penick has quit IRC | 05:34 | |
| *** slaweq has joined #openstack-nova | 05:34 | |
| *** Tom_ has quit IRC | 05:35 | |
| *** pratapagoutham has joined #openstack-nova | 05:35 | |
| *** Tom_ has joined #openstack-nova | 05:36 | |
| *** kaisers1 has joined #openstack-nova | 05:36 | |
| *** edmondsw has joined #openstack-nova | 05:41 | |
| *** edmondsw has quit IRC | 05:45 | |
| *** markvoelker has quit IRC | 05:46 | |
| *** diga has quit IRC | 05:47 | |
| *** ircuser-1 has joined #openstack-nova | 05:57 | |
| *** zen369 has joined #openstack-nova | 05:59 | |
| *** huanxie has joined #openstack-nova | 06:01 | |
| *** lpetrut_ has quit IRC | 06:06 | |
| *** lpetrut_ has joined #openstack-nova | 06:06 | |
| *** slaweq has quit IRC | 06:07 | |
| *** edand has joined #openstack-nova | 06:07 | |
| *** slaweq has joined #openstack-nova | 06:10 | |
| *** Oku_OS-away is now known as Oku_OS | 06:11 | |
| *** lpetrut_ has quit IRC | 06:13 | |
| *** vivsoni has joined #openstack-nova | 06:14 | |
| vivsoni | Hi, when 'volume_use_multipath' is set to 'True' in nova.conf 'libvirt' section, the get_connector_properties() is not picking up this value.. | 06:16 |
| vivsoni | it always takes default hard-coded value from libvirt.py | 06:16 |
| *** lajoskatona has joined #openstack-nova | 06:19 | |
| *** diga has joined #openstack-nova | 06:21 | |
| *** yamahata has joined #openstack-nova | 06:21 | |
| *** lpetrut_ has joined #openstack-nova | 06:22 | |
| *** sridharg has joined #openstack-nova | 06:22 | |
| *** larainema has quit IRC | 06:24 | |
| *** serverascode has quit IRC | 06:24 | |
| *** ameade has quit IRC | 06:24 | |
| *** cargonza has quit IRC | 06:24 | |
| *** tomhambleton_ has quit IRC | 06:24 | |
| *** karlamrhein has quit IRC | 06:24 | |
| *** NobodyCam has quit IRC | 06:24 | |
| *** wolsen has quit IRC | 06:24 | |
| *** aweeks has quit IRC | 06:24 | |
| *** zul has quit IRC | 06:24 | |
| *** betherly has quit IRC | 06:24 | |
| *** carl_baldwin has quit IRC | 06:24 | |
| *** aarefiev has quit IRC | 06:24 | |
| *** csatari has quit IRC | 06:24 | |
| *** kmalloc has quit IRC | 06:24 | |
| *** hogepodge has quit IRC | 06:24 | |
| *** gcb has quit IRC | 06:24 | |
| *** DuncanT has quit IRC | 06:25 | |
| *** TheJulia has quit IRC | 06:25 | |
| *** samueldmq has quit IRC | 06:25 | |
| *** vdrok has quit IRC | 06:25 | |
| *** gmann has quit IRC | 06:25 | |
| *** jamespage has quit IRC | 06:25 | |
| *** fmccrthy has quit IRC | 06:25 | |
| *** fmccrthy has joined #openstack-nova | 06:26 | |
| *** NobodyCam has joined #openstack-nova | 06:26 | |
| *** aweeks has joined #openstack-nova | 06:26 | |
| *** carl_baldwin has joined #openstack-nova | 06:26 | |
| *** jamespage has joined #openstack-nova | 06:26 | |
| *** hogepodge has joined #openstack-nova | 06:26 | |
| *** kmalloc has joined #openstack-nova | 06:26 | |
| *** vdrok has joined #openstack-nova | 06:26 | |
| *** wolsen has joined #openstack-nova | 06:26 | |
| *** samueldmq has joined #openstack-nova | 06:26 | |
| *** cargonza has joined #openstack-nova | 06:26 | |
| *** karlamrhein has joined #openstack-nova | 06:26 | |
| *** ameade has joined #openstack-nova | 06:26 | |
| *** csatari has joined #openstack-nova | 06:26 | |
| *** ying_zuo has quit IRC | 06:26 | |
| *** serverascode has joined #openstack-nova | 06:27 | |
| *** TheJulia has joined #openstack-nova | 06:27 | |
| *** DuncanT has joined #openstack-nova | 06:27 | |
| *** aarefiev has joined #openstack-nova | 06:27 | |
| *** betherly has joined #openstack-nova | 06:27 | |
| *** karthiks has joined #openstack-nova | 06:28 | |
| *** gcb has joined #openstack-nova | 06:28 | |
| *** openstackgerrit has quit IRC | 06:28 | |
| *** Jeffrey4l has quit IRC | 06:28 | |
| *** Jeffrey4l has joined #openstack-nova | 06:29 | |
| *** psachin has quit IRC | 06:30 | |
| *** trinaths has joined #openstack-nova | 06:31 | |
| *** jwcroppe has quit IRC | 06:36 | |
| *** jwcroppe has joined #openstack-nova | 06:36 | |
| *** priya_ has joined #openstack-nova | 06:36 | |
| *** bkopilov_ has joined #openstack-nova | 06:37 | |
| *** priya_ has quit IRC | 06:38 | |
| *** esberglu has joined #openstack-nova | 06:38 | |
| *** mariusv has joined #openstack-nova | 06:39 | |
| *** mariusv has quit IRC | 06:39 | |
| *** mariusv has joined #openstack-nova | 06:39 | |
| *** AlexeyAbashkin has joined #openstack-nova | 06:39 | |
| *** yangyapeng has quit IRC | 06:42 | |
| *** slaweq has quit IRC | 06:42 | |
| *** yangyapeng has joined #openstack-nova | 06:42 | |
| *** esberglu has quit IRC | 06:43 | |
| *** slaweq has joined #openstack-nova | 06:45 | |
| *** markus_z has joined #openstack-nova | 06:45 | |
| *** AlexeyAbashkin has quit IRC | 06:46 | |
| *** psachin has joined #openstack-nova | 06:46 | |
| *** liusheng has quit IRC | 06:51 | |
| *** liusheng has joined #openstack-nova | 06:52 | |
| *** hferenc has joined #openstack-nova | 06:56 | |
| *** vks1 has quit IRC | 06:59 | |
| *** liusheng has quit IRC | 06:59 | |
| *** liusheng has joined #openstack-nova | 06:59 | |
| *** vvargaszte has quit IRC | 07:01 | |
| *** claudiub has quit IRC | 07:01 | |
| *** markvoelker has joined #openstack-nova | 07:02 | |
| *** openstackgerrit has joined #openstack-nova | 07:03 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 07:03 |
| *** priya_ has joined #openstack-nova | 07:03 | |
| priya_ | hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/ | 07:03 |
| *** lpetrut_ has quit IRC | 07:10 | |
| *** sahid has joined #openstack-nova | 07:10 | |
| *** vks1 has joined #openstack-nova | 07:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 07:14 |
| *** claudiub has joined #openstack-nova | 07:14 | |
| *** tesseract has joined #openstack-nova | 07:18 | |
| *** markvoelker has quit IRC | 07:18 | |
| *** slaweq has quit IRC | 07:19 | |
| *** markvoelker has joined #openstack-nova | 07:20 | |
| *** markvoelker has quit IRC | 07:20 | |
| *** markvoelker has joined #openstack-nova | 07:20 | |
| priya_ | hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/ | 07:22 |
| *** slaweq has joined #openstack-nova | 07:23 | |
| *** bkopilov has quit IRC | 07:27 | |
| *** bkopilov has joined #openstack-nova | 07:27 | |
| *** edmondsw has joined #openstack-nova | 07:29 | |
| *** yamahata has quit IRC | 07:30 | |
| *** toabctl has joined #openstack-nova | 07:32 | |
| *** efried has quit IRC | 07:32 | |
| *** edmondsw has quit IRC | 07:33 | |
| *** ragiman has joined #openstack-nova | 07:34 | |
| *** yamamoto_ has quit IRC | 07:35 | |
| *** xinliang has joined #openstack-nova | 07:37 | |
| *** xinliang has quit IRC | 07:37 | |
| *** xinliang has joined #openstack-nova | 07:37 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:38 | |
| *** tssurya has joined #openstack-nova | 07:41 | |
| *** slaweq_ has joined #openstack-nova | 07:44 | |
| *** efried has joined #openstack-nova | 07:44 | |
| *** lpetrut_ has joined #openstack-nova | 07:45 | |
| *** jpena|off is now known as jpena | 07:48 | |
| *** lpetrut_ has quit IRC | 07:53 | |
| *** slaweq has quit IRC | 07:57 | |
| *** Tom_ has quit IRC | 07:57 | |
| *** Tom_ has joined #openstack-nova | 07:58 | |
| *** Tom_ has quit IRC | 07:59 | |
| *** brault has joined #openstack-nova | 07:59 | |
| *** Tom_ has joined #openstack-nova | 07:59 | |
| *** slaweq has joined #openstack-nova | 08:01 | |
| *** trungnv has quit IRC | 08:06 | |
| *** TuanLA_ has joined #openstack-nova | 08:06 | |
| *** trungnv has joined #openstack-nova | 08:07 | |
| *** phuongnh has quit IRC | 08:07 | |
| *** TuanLA has quit IRC | 08:07 | |
| *** sridharg has quit IRC | 08:09 | |
| *** fragatina has joined #openstack-nova | 08:10 | |
| *** sridharg has joined #openstack-nova | 08:10 | |
| *** belmoreira has joined #openstack-nova | 08:13 | |
| *** derekh has joined #openstack-nova | 08:13 | |
| gibi | stephenfin: hi! Did you intentionally leave out the idamshift manpage from the latest version of https://review.openstack.org/#/c/502105 ? | 08:16 |
| *** slunkad has joined #openstack-nova | 08:16 | |
| stephenfin | gibi: I did, yes. mikal has removed that in another patch and I didn't want to introduce a conflict | 08:17 |
| *** fragatina has quit IRC | 08:17 | |
| *** fragatina has joined #openstack-nova | 08:17 | |
| belmoreira | I'm trying to change the default nova log level to ERROR. After some googling I can't find a solution... errrr... How can I do this? any pointer is appreciated | 08:17 |
| gibi | stephenfin: ahh OK, then I +2 on the manpage patch | 08:19 |
| openstackgerrit | Merged openstack/nova master: Move test_uuid_sentinels to NoDBTestCase https://review.openstack.org/507253 | 08:20 |
| *** lucas-afk is now known as lucasagomes | 08:21 | |
| *** tomhambleton_ has joined #openstack-nova | 08:22 | |
| *** larainema has joined #openstack-nova | 08:22 | |
| *** gmann has joined #openstack-nova | 08:22 | |
| *** vks1 has quit IRC | 08:22 | |
| *** zul has joined #openstack-nova | 08:22 | |
| *** hoonetorg has quit IRC | 08:25 | |
| *** belmoreira has quit IRC | 08:25 | |
| *** ying_zuo has joined #openstack-nova | 08:26 | |
| *** esberglu has joined #openstack-nova | 08:26 | |
| *** esberglu has quit IRC | 08:30 | |
| *** markvoelker has quit IRC | 08:31 | |
| priya_ | belmoreira - the default log level can be set in nova.conf | 08:34 |
| *** slaweq has quit IRC | 08:34 | |
| priya_ | hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/ | 08:34 |
| *** yamamoto has joined #openstack-nova | 08:36 | |
| *** slaweq has joined #openstack-nova | 08:36 | |
| *** vvargaszte has joined #openstack-nova | 08:39 | |
| *** vvargaszte has quit IRC | 08:40 | |
| *** yamamoto has quit IRC | 08:42 | |
| *** Tom_ has quit IRC | 08:44 | |
| *** vks1 has joined #openstack-nova | 08:44 | |
| *** markvoelker has joined #openstack-nova | 08:45 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add create inventories doc for placement https://review.openstack.org/512215 | 08:46 |
| *** diga has quit IRC | 08:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 08:48 |
| *** ralonsoh has joined #openstack-nova | 08:52 | |
| *** diga has joined #openstack-nova | 08:54 | |
| *** yamamoto has joined #openstack-nova | 09:04 | |
| *** belmoreira has joined #openstack-nova | 09:05 | |
| *** belmoreira has quit IRC | 09:06 | |
| *** slaweq has quit IRC | 09:08 | |
| *** yamamoto has quit IRC | 09:09 | |
| *** slaweq has joined #openstack-nova | 09:13 | |
| *** yamamoto has joined #openstack-nova | 09:13 | |
| *** sshwarts has joined #openstack-nova | 09:13 | |
| *** belmoreira has joined #openstack-nova | 09:13 | |
| *** esberglu has joined #openstack-nova | 09:20 | |
| *** esberglu has quit IRC | 09:20 | |
| *** esberglu has joined #openstack-nova | 09:20 | |
| *** esberglu has quit IRC | 09:21 | |
| openstackgerrit | Viktor Bass proposed openstack/nova master: doc: Fix command output in scheduler document https://review.openstack.org/511839 | 09:21 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Update "The nova Shell Utility" in the user guide https://review.openstack.org/512224 | 09:22 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Update "The nova Shell Utility" in the user guide https://review.openstack.org/512224 | 09:24 |
| *** sambetts|afk is now known as sambetts | 09:27 | |
| *** Shunli has quit IRC | 09:33 | |
| *** diga has quit IRC | 09:34 | |
| *** takashin has left #openstack-nova | 09:36 | |
| *** cdent has joined #openstack-nova | 09:37 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 09:38 |
| *** yassine has quit IRC | 09:40 | |
| *** yassine has joined #openstack-nova | 09:41 | |
| *** belmoreira has quit IRC | 09:43 | |
| *** belmoreira has joined #openstack-nova | 09:46 | |
| *** slaweq has quit IRC | 09:46 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 09:50 |
| *** AlexeyAbashkin has quit IRC | 09:53 | |
| *** AlexeyAbashkin has joined #openstack-nova | 09:53 | |
| *** slaweq has joined #openstack-nova | 09:53 | |
| *** mvk has quit IRC | 09:54 | |
| *** derekh has quit IRC | 09:58 | |
| *** hshiina has quit IRC | 09:58 | |
| *** derekh has joined #openstack-nova | 09:59 | |
| *** TuanLA_ has quit IRC | 10:01 | |
| *** ygl has joined #openstack-nova | 10:02 | |
| *** ygl has left #openstack-nova | 10:02 | |
| *** phuongnh has joined #openstack-nova | 10:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347 | 10:04 |
| openstackgerrit | jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510 | 10:05 |
| *** edand has quit IRC | 10:07 | |
| *** esberglu has joined #openstack-nova | 10:14 | |
| *** derekh has quit IRC | 10:15 | |
| *** sdague has joined #openstack-nova | 10:17 | |
| *** jichen has quit IRC | 10:18 | |
| *** esberglu has quit IRC | 10:18 | |
| *** derekh has joined #openstack-nova | 10:20 | |
| *** psachin has quit IRC | 10:31 | |
| *** openstackgerrit has quit IRC | 10:33 | |
| *** mvk has joined #openstack-nova | 10:33 | |
| *** trinaths has left #openstack-nova | 10:34 | |
| *** belmoreira has quit IRC | 10:37 | |
| *** lpetrut has joined #openstack-nova | 10:39 | |
| *** soum has joined #openstack-nova | 10:40 | |
| *** yangyapeng has quit IRC | 10:44 | |
| *** tbachman has quit IRC | 10:48 | |
| *** edand has joined #openstack-nova | 10:49 | |
| *** cshen has joined #openstack-nova | 10:51 | |
| *** udesale has quit IRC | 10:54 | |
| *** udesale has joined #openstack-nova | 10:54 | |
| *** brault has quit IRC | 10:58 | |
| efried | Good morning nova | 11:01 |
| *** AlexeyAbashkin has quit IRC | 11:04 | |
| *** dtantsur|afk is now known as dtantsur | 11:04 | |
| *** edmondsw has joined #openstack-nova | 11:05 | |
| *** udesale has quit IRC | 11:07 | |
| *** rodolof has joined #openstack-nova | 11:07 | |
| *** phuongnh has quit IRC | 11:08 | |
| *** dave-mccowan has joined #openstack-nova | 11:09 | |
| *** edmondsw has quit IRC | 11:09 | |
| *** strigazi has joined #openstack-nova | 11:09 | |
| *** markus_zz has joined #openstack-nova | 11:13 | |
| *** vks1 has quit IRC | 11:13 | |
| *** smatzek has joined #openstack-nova | 11:14 | |
| *** markus_z has quit IRC | 11:15 | |
| *** masber has quit IRC | 11:16 | |
| *** rodolof has quit IRC | 11:17 | |
| *** rodolof has joined #openstack-nova | 11:18 | |
| efried | alex_xu yt? | 11:19 |
| *** AlexeyAbashkin has joined #openstack-nova | 11:20 | |
| *** ociuhandu has quit IRC | 11:22 | |
| *** openstackgerrit has joined #openstack-nova | 11:26 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix cleaning up evacuated instances https://review.openstack.org/510938 | 11:26 |
| *** slaweq has quit IRC | 11:29 | |
| *** huanxie has quit IRC | 11:30 | |
| *** nicolasbock has joined #openstack-nova | 11:36 | |
| *** slaweq has joined #openstack-nova | 11:38 | |
| *** yamamoto has quit IRC | 11:38 | |
| *** lucasagomes is now known as lucas-hungry | 11:39 | |
| *** tasker has quit IRC | 11:39 | |
| *** brault has joined #openstack-nova | 11:39 | |
| *** tasker has joined #openstack-nova | 11:40 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 11:41 |
| *** diga has joined #openstack-nova | 11:45 | |
| *** udesale has joined #openstack-nova | 11:48 | |
| *** ociuhandu has joined #openstack-nova | 11:49 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 11:50 |
| *** yangyapeng has joined #openstack-nova | 11:50 | |
| *** udesale has quit IRC | 11:51 | |
| *** thorst has joined #openstack-nova | 11:54 | |
| *** thorst has quit IRC | 11:54 | |
| *** thorst has joined #openstack-nova | 11:56 | |
| *** yamamoto has joined #openstack-nova | 11:58 | |
| openstackgerrit | Merged openstack/nova master: TrivialFix: Fix the incorrect test case https://review.openstack.org/509758 | 11:58 |
| *** gcb has quit IRC | 12:01 | |
| *** jpena is now known as jpena|lunch | 12:04 | |
| *** huanxie has joined #openstack-nova | 12:08 | |
| *** edmondsw has joined #openstack-nova | 12:09 | |
| *** slaweq has quit IRC | 12:11 | |
| *** diga has quit IRC | 12:13 | |
| *** gszasz has joined #openstack-nova | 12:14 | |
| *** sree has quit IRC | 12:16 | |
| *** sree has joined #openstack-nova | 12:16 | |
| *** jmlowe has quit IRC | 12:17 | |
| *** slaweq has joined #openstack-nova | 12:18 | |
| *** sree has quit IRC | 12:20 | |
| *** soum has quit IRC | 12:21 | |
| *** yamamoto has quit IRC | 12:21 | |
| *** lucas-hungry is now known as lucasagomes | 12:23 | |
| *** dave-mccowan has quit IRC | 12:23 | |
| priya_ | hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/ | 12:24 |
| openstackgerrit | Merged openstack/nova master: Remove system_metadata loading in Instance._load_flavor https://review.openstack.org/508357 | 12:25 |
| openstackgerrit | Merged openstack/nova master: Don't use mock.patch.stopall https://review.openstack.org/507527 | 12:25 |
| openstackgerrit | Merged openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844 | 12:26 |
| openstackgerrit | Merged openstack/nova master: Filter out stale migrations in resource audit https://review.openstack.org/339715 | 12:26 |
| openstackgerrit | Merged openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582 | 12:26 |
| efried | stephenfin FYI, I'm spinning up another rev of that spec, in case you wanted to, I don't know, make fun of my hair or something. | 12:27 |
| alex_xu | efried: hi, i'm here | 12:30 |
| efried | alex_xu Hi, I was going to ask you something about aggregates wrt the granular RP syntax spec... | 12:30 |
| alex_xu | efried: yea | 12:30 |
| efried | alex_xu It was about dansmith's comment here: https://review.openstack.org/#/c/510244/4/specs/queens/approved/granular-resource-requests.rst@45 | 12:31 |
| efried | IIUC, the way you're implementing it, if I ask for a particular trait, you'll succeed even if the trait isn't on the compute node? | 12:32 |
| efried | alex_xu ...as long as it's on an aggregate? | 12:33 |
| alex_xu | yes, but the compute node and shared storage pool in the same aggregate | 12:33 |
| efried | alex_xu This actually brings up an interesting set of test cases to cover. If I ask for storage-related traits X and Y; and my compute node has storage with trait X but not Y; and my shared storage has trait Y but not X - what happens? | 12:34 |
| efried | I should *not* get that host/storage as a candidate. | 12:35 |
| alex_xu | efried: first, we don't have the case the comput enode have local storage and shared storage at same time | 12:35 |
| efried | But I suspect with the logic you're currently implementing, it might erroneously return that host. | 12:35 |
| efried | alex_xu What do you mean? | 12:35 |
| efried | You mean nobody has done it yet? Or we don't support it at all? | 12:36 |
| alex_xu | efried: yea, that's true, my implementation doesn't work for that | 12:36 |
| alex_xu | efried: we don't support it at all | 12:36 |
| efried | Okay. | 12:36 |
| efried | I mean, I get the impression that we're not trying to get shared RPs "working" in Queens, but I don't really know what that means in terms of what code we're actually going to merge around aggregates and such. | 12:36 |
| efried | Because ultimately it's not just about shared storage. There could be other kinds of shared RPs, and we would probably want that scenario to work properly. | 12:37 |
| alex_xu | efried: yea, but the current implementation of 'GET /allocation_candidates' support a buggy version shared RPs... | 12:37 |
| efried | right, I saw that too :( | 12:37 |
| *** esberglu has joined #openstack-nova | 12:38 | |
| alex_xu | efried: yea, I'm also think about, but I'm still trying to find a similar case in the real world for that | 12:38 |
| alex_xu | efried: thanks for point that out, I will keep that in mind | 12:38 |
| efried | Sure thing. | 12:39 |
| *** huanxie has quit IRC | 12:39 | |
| efried | alex_xu Thanks for the review, as well. I'm addressing your comments now. | 12:39 |
| alex_xu | efried: np | 12:39 |
| *** felipemonteiro has joined #openstack-nova | 12:39 | |
| *** felipemonteiro_ has joined #openstack-nova | 12:41 | |
| alex_xu | efried: for the case you pointed out, I think the code should generate two candidates, 1. node and local storage, 2. node and shared strorage. The current implementation without traits doesn't work for this case also. | 12:43 |
| alex_xu | I guess Jay may say we can't have duplicated resource class in the same aggregate | 12:44 |
| *** dave-mccowan has joined #openstack-nova | 12:45 | |
| *** felipemonteiro has quit IRC | 12:45 | |
| efried | alex_xu I would be very surprised if he said that | 12:45 |
| efried | mm, maybe not. | 12:45 |
| efried | I need to noodle that through. | 12:46 |
| efried | But for the case I pointed out, I would expect *no* candidates. Because neither the local nor the shared storage has *both* X and Y. | 12:46 |
| efried | Unless you're saying that the code as currently written would generate two candidates. I agree. And I think that's not correct. | 12:47 |
| alex_xu | efried: I mean the currently written with the same request but without required traits, there should be two candidates | 12:48 |
| efried | Oh, yeah, without traits involved, sure. | 12:48 |
| *** lyan has joined #openstack-nova | 12:51 | |
| *** slaweq has quit IRC | 12:51 | |
| *** yamamoto has joined #openstack-nova | 12:52 | |
| *** pchavva has joined #openstack-nova | 12:53 | |
| *** liverpooler has joined #openstack-nova | 12:54 | |
| *** lyan has quit IRC | 12:55 | |
| *** lyan has joined #openstack-nova | 12:56 | |
| *** lyan has quit IRC | 12:58 | |
| *** huanxie has joined #openstack-nova | 12:59 | |
| *** sree has joined #openstack-nova | 12:59 | |
| *** acormier has joined #openstack-nova | 13:00 | |
| *** tuanla_fujitsu has joined #openstack-nova | 13:02 | |
| *** slaweq has joined #openstack-nova | 13:02 | |
| *** tuanla__ has joined #openstack-nova | 13:02 | |
| *** mriedem has joined #openstack-nova | 13:02 | |
| *** ragiman has quit IRC | 13:03 | |
| *** jpena|lunch is now known as jpena | 13:03 | |
| *** acormier has quit IRC | 13:04 | |
| *** lyan has joined #openstack-nova | 13:04 | |
| *** dave-mccowan has quit IRC | 13:06 | |
| *** links has quit IRC | 13:12 | |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | 13:12 |
| *** bauwser is now known as bauzas | 13:12 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] manage cache headers for /resource_providers https://review.openstack.org/495380 | 13:13 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] manage cache headers for inventories https://review.openstack.org/510954 | 13:13 |
| *** dave-mccowan has joined #openstack-nova | 13:16 | |
| *** niraj_singh has joined #openstack-nova | 13:16 | |
| *** avolkov has joined #openstack-nova | 13:17 | |
| *** peter-hamilton has joined #openstack-nova | 13:18 | |
| *** peter-hamilton has quit IRC | 13:18 | |
| niraj_singh | i set send_service_user_token= true in nova.conf and tried to boot instance using image. I am getting error ERROR nova.api.openstack.extensions [None req-ff664a2e-3fdb-4cb9-bd02-78c662fdc0c2 admin admin] Unexpected exception in API method: AttributeError: 'NoneType' object has no attribute 'get_token'. is there any clue. | 13:18 |
| *** ragiman has joined #openstack-nova | 13:19 | |
| *** peter-hamilton has joined #openstack-nova | 13:19 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax https://review.openstack.org/510244 | 13:21 |
| efried | dansmith stephenfin alex_xu ^ | 13:21 |
| *** smatzek has quit IRC | 13:23 | |
| *** rmart04 has joined #openstack-nova | 13:23 | |
| *** gcb has joined #openstack-nova | 13:25 | |
| efried | niraj_singh Do you have a full stack trace? Can you paste.openstack.org it? | 13:25 |
| niraj_singh | efried: yes | 13:26 |
| efried | To me this sounds like your context doesn't have a good auth in it. | 13:28 |
| niraj_singh | efried: http://paste.openstack.org/show/623721/ | 13:28 |
| *** huanxie has quit IRC | 13:29 | |
| efried | niraj_singh What version are we at? | 13:29 |
| niraj_singh | 2 | 13:30 |
| *** erlon has joined #openstack-nova | 13:30 | |
| niraj_singh | efried: 2 | 13:31 |
| efried | niraj_singh 2? | 13:32 |
| efried | I mean like, ocata, pike, some devel branch... | 13:32 |
| niraj_singh | efried: ohh sorry... i am new to openstack. | 13:32 |
| *** slaweq has quit IRC | 13:34 | |
| efried | niraj_singh Okay, I think I see what's going on. Can you paste.o.o me your nova conf so I can confirm? | 13:35 |
| niraj_singh | yes | 13:36 |
| efried | niraj_singh In short: Do you have a [service_user] section set up with appropriate auth options? | 13:36 |
| *** baoli has joined #openstack-nova | 13:36 | |
| *** slaweq has joined #openstack-nova | 13:37 | |
| *** lbragstad has joined #openstack-nova | 13:37 | |
| *** ragiman has quit IRC | 13:38 | |
| niraj_singh | efried:http://paste.openstack.org/show/623723/ | 13:38 |
| efried | niraj_singh Okay, cool. So what you need to do is populate that [service_user] section with auth options. | 13:40 |
| efried | niraj_singh It would wind up looking something like what you've got in the [placement] and [neutron] sections above there, with an auth_type etc. | 13:41 |
| niraj_singh | efried:ok | 13:41 |
| *** ttsiouts has joined #openstack-nova | 13:41 | |
| *** hongbin has joined #openstack-nova | 13:41 | |
| efried | niraj_singh I think we could do better with our error message, though it wouldn't be easy to make it appear in the actual spot of the error. | 13:42 |
| efried | niraj_singh We actually may want to make the compute process blow up if you said "send_service_user_token" but didn't supply appropriate auth opts. | 13:43 |
| efried | niraj_singh Can you please open a bug at http://bugs.launchpad.net/nova/ for this? | 13:43 |
| niraj_singh | efried: yes sure. | 13:43 |
| *** mriedem1 has joined #openstack-nova | 13:43 | |
| *** crushil has quit IRC | 13:43 | |
| *** burt has joined #openstack-nova | 13:44 | |
| efried | niraj_singh Thanks. Hit me with the link once you've got it open. | 13:44 |
| niraj_singh | efried: yes thank you for help | 13:45 |
| edleafe | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | 13:45 |
| *** dtantsur_ has joined #openstack-nova | 13:46 | |
| *** mriedem has quit IRC | 13:48 | |
| *** liusheng has quit IRC | 13:48 | |
| *** slunkad has quit IRC | 13:48 | |
| *** dtantsur has quit IRC | 13:48 | |
| *** dtantsur_ is now known as dtantsur | 13:48 | |
| *** liusheng has joined #openstack-nova | 13:48 | |
| *** slunkad has joined #openstack-nova | 13:48 | |
| dtantsur | folks, do you have an early specs freeze this cycle? nothing on the official schedule, but johnthetubaguy says that you do :) | 13:49 |
| dtantsur | mriedem1: ^^^ | 13:49 |
| mriedem1 | yeah, this thursday | 13:49 |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | 13:49 |
| mriedem1 | 10/19 | 13:49 |
| *** ragiman has joined #openstack-nova | 13:49 | |
| *** mriedem1 is now known as mriedem | 13:49 | |
| mriedem | dtantsur: ^ | 13:49 |
| mriedem | dtantsur: not really early - we've done spec freeze on the first milestone the last 2 releases too | 13:50 |
| dtantsur | mriedem: ugh. I'd appreciate making it visible for outsider folks, like glance and manila did | 13:51 |
| dtantsur | (i.e. https://releases.openstack.org/queens/schedule.html ) | 13:51 |
| mriedem | i can update the main schedule | 13:51 |
| dtantsur | next time, I guess :) now it's too late | 13:51 |
| mriedem | is there something specific i can help you with regarding a nova spec? | 13:52 |
| mriedem | i'm assuming you're talking about john's ironic traits one? | 13:52 |
| dtantsur | mriedem: yep, that one | 13:52 |
| mriedem | the nova spec is already approved, it's just dependent on changes in ironic | 13:52 |
| dtantsur | I don't think we're ready to finish the ironic part this week | 13:52 |
| dtantsur | maybe we could, but that's not something I'd rush in | 13:52 |
| mriedem | that's fine, i told john to remove the depends-on for the spec, | 13:52 |
| mriedem | we can track the dependency via launchpad | 13:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | 13:53 |
| *** psachin has joined #openstack-nova | 13:53 | |
| dtantsur | mriedem: cool, so it was solved without me in the end :) thanks! | 13:53 |
| mriedem | dtantsur: ^ done | 13:53 |
| dtantsur | :) | 13:54 |
| *** gouthamr has joined #openstack-nova | 13:54 | |
| *** takashin has joined #openstack-nova | 13:55 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399 | 13:55 |
| *** armax has joined #openstack-nova | 13:56 | |
| *** tasker has quit IRC | 13:57 | |
| *** tasker has joined #openstack-nova | 13:59 | |
| *** zen369 has quit IRC | 14:01 | |
| *** crushil has joined #openstack-nova | 14:01 | |
| sahid | mriedem: when you have a moment i'm wondering if we can consider this for Queen http://lists.openstack.org/pipermail/openstack-dev/2017-October/123432.html ? | 14:01 |
| sahid | no worries it's not vGPUs or mdev things :) | 14:01 |
| *** kylek3h has quit IRC | 14:03 | |
| *** awaugama has joined #openstack-nova | 14:03 | |
| *** yamahata has joined #openstack-nova | 14:04 | |
| *** tuanla_fujitsu has quit IRC | 14:05 | |
| *** tuanla__ has quit IRC | 14:05 | |
| *** tuanla_fujitsu has joined #openstack-nova | 14:05 | |
| johnthetubaguy | dtantsur: mriedem: thanks folks | 14:05 |
| *** tuanla_fujitsu has quit IRC | 14:05 | |
| *** tuanla_fujitsu has joined #openstack-nova | 14:05 | |
| *** tuanla_fujitsu has quit IRC | 14:06 | |
| *** tuanla_ has joined #openstack-nova | 14:06 | |
| mriedem | sahid: just more config options for already complicated setup | 14:06 |
| mriedem | sahid: i don't have an opinion right now | 14:06 |
| mriedem | might be good to involve blair bethwaight (spelling?) and the HPC people | 14:06 |
| *** yamamoto has quit IRC | 14:07 | |
| mriedem | bethwaite | 14:08 |
| sahid | mriedem: yes this kind of setups are complicated, i don't think there is an ideal solution to avoid that | 14:08 |
| *** slaweq has quit IRC | 14:10 | |
| sahid | mriedem: siemens where also interested by this options for the realtime use cases | 14:11 |
| sahid | there is a thread on the ML | 14:11 |
| sean-k-mooney | sahid: mriedem it does not sean that unresonable to me | 14:11 |
| sean-k-mooney | but yes this are is already complcated | 14:11 |
| sean-k-mooney | sahid: yes realtime makes sense for this feature | 14:11 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: Spec for API extensions policy removal https://review.openstack.org/508101 | 14:12 |
| openstackgerrit | Merged openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | 14:12 |
| *** masber has joined #openstack-nova | 14:12 | |
| gmann | mriedem: separated the api extensions and policy removal BP. this is specless BP for api extensions work for queens - https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-queens | 14:13 |
| sahid | sean-k-mooney: yes there also some dpdk usages, they want to isolate the emulthreads to have the full capacity of the pCPUs pinned for the guests | 14:13 |
| sean-k-mooney | by the way on config option i never got around to bringin up at the ptg was the host reserved memory. i mentioned it to a few people but we should really have a numa aware version of that | 14:13 |
| *** david-lyle has joined #openstack-nova | 14:14 | |
| gmann | ll update the policy removal spec tomorrow | 14:15 |
| *** vks1 has joined #openstack-nova | 14:15 | |
| gibi | mriedem: hi! I fixed your comments in https://review.openstack.org/#/c/510938/ | 14:15 |
| sean-k-mooney | sahid: sure that makes sense for any io sensive application ( not just dpdk) if you have pinned cores you do not want the emulator thread to context switch out your application | 14:15 |
| *** tbachman has joined #openstack-nova | 14:15 | |
| sahid | sean-k-mooney: yes | 14:15 |
| sean-k-mooney | and since the emulator thread are largly ideal you do not want to spend the cost of allocatng extra cores to hanedl them per vm and would instead want a pool of cpus to use for them instead | 14:16 |
| *** slaweq has joined #openstack-nova | 14:17 | |
| *** masber has quit IRC | 14:17 | |
| sean-k-mooney | htop | 14:18 |
| *** tssurya has quit IRC | 14:18 | |
| sean-k-mooney | mriedem: if you get a chance can you take a look at https://review.openstack.org/#/c/375580/ later | 14:19 |
| *** vks1 has quit IRC | 14:19 | |
| sean-k-mooney | any other cores with opions on how to adress the use of multiple port binding for live migrate are also welcome to review :) | 14:20 |
| sean-k-mooney | or non cores but would like to get some nova feedback before sepc freeze | 14:21 |
| *** vks1 has joined #openstack-nova | 14:21 | |
| *** slaweq_ has quit IRC | 14:24 | |
| *** slaweq_ has joined #openstack-nova | 14:24 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410 | 14:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051 | 14:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Limit number of attempts to delete allocations https://review.openstack.org/507224 | 14:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625 | 14:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | 14:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 14:25 |
| *** vks1 has quit IRC | 14:29 | |
| *** slaweq_ has quit IRC | 14:29 | |
| *** david-lyle has quit IRC | 14:29 | |
| *** david-lyle has joined #openstack-nova | 14:30 | |
| *** markus_zz has quit IRC | 14:30 | |
| *** tbachman has quit IRC | 14:31 | |
| *** tssurya has joined #openstack-nova | 14:31 | |
| mriedem | sean-k-mooney: johnthetubaguy is the other person that should review those updates, since he started that spec :) | 14:34 |
| johnthetubaguy | mriedem: sean-k-mooney: I will try get to that today | 14:35 |
| sean-k-mooney | mriedem: johnthetubaguy that will make mlavalle happy to here | 14:36 |
| *** READ10 has joined #openstack-nova | 14:37 | |
| *** edand has quit IRC | 14:37 | |
| *** sree has quit IRC | 14:37 | |
| *** cleong has joined #openstack-nova | 14:39 | |
| openstackgerrit | Matthew Booth proposed openstack/nova-specs master: Add serial numbers for local disks https://review.openstack.org/511466 | 14:41 |
| dansmith | bauzas: you've seen this right? https://review.openstack.org/#/c/511879/3 | 14:41 |
| ildikov | mriedem: johnthetubaguy: hi. would that be possible to merge the live_migrate patch and fix that one issue with refresh_conn_info in the new attach patch? | 14:41 |
| dansmith | bauzas: once it's in, I think it probably makes sense to look at that data structure for "do I have an allocation for a vgpu" | 14:41 |
| mriedem | ildikov: not really.... | 14:43 |
| mriedem | they are separate issues | 14:43 |
| mdbooth | dansmith: I had an epiphany wrt local serial numbers in ironic, btw: we don't need to prescribe the method of assigning serial numbers as long as the interface to them is consistent. Ironic is already consistent, because it uses physical disks which don't have the problem we're trying to solve. | 14:43 |
| mriedem | ildikov: it might be best to actually split the refresh_connection_info handling out before the live migration change | 14:43 |
| mriedem | so it goes refresh_conn_info -> live migratoin new style attachments -> enable new style attachments in api | 14:44 |
| mdbooth | I updated the spec to define a driver contract for stable serial numbers, and a specific implementation for libvirt which is as previously discussed. | 14:44 |
| dansmith | mdbooth: it does if you expose the serial number out of the metadata api, and ironic is one of those places where tagging devices is pretty important | 14:44 |
| *** vks1 has joined #openstack-nova | 14:44 | |
| mdbooth | Right, I covered that. | 14:44 |
| dansmith | okay | 14:44 |
| mdbooth | The device metadata is always created by the driver anyway, because it's the only place which has that info. | 14:44 |
| mdbooth | So the ironic driver would expose the actual serial numbers it's using. | 14:45 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 14:45 |
| ildikov | mriedem: hmm, for that I will need to make a chain of these patches and I would really love to avoid that | 14:45 |
| mriedem | ildikov: i actually wish they were in a chain | 14:45 |
| mriedem | rather than depends-on | 14:45 |
| *** esberglu has quit IRC | 14:46 | |
| mriedem | but you don't need a topic branch if you don't want, | 14:46 |
| mriedem | you can do refresh_connection_info for new style attachments in a separate patch, | 14:46 |
| ildikov | mriedem: working together with someone on a chain is highly painful | 14:46 |
| mriedem | have the live migration patch depends-on that, and have the final api patch depends on the live migration patch, which it already does | 14:46 |
| ildikov | but if we do that changes in the refresh_conn_info I don't know how those will be picked up with only a depends-on | 14:47 |
| mdbooth | Incidentally, I'm also looking for re-approval of lyarwood 's previously approved stable rescue spec: https://review.openstack.org/#/c/510106/ | 14:47 |
| *** yamahata has quit IRC | 14:49 | |
| *** yamahata has joined #openstack-nova | 14:49 | |
| *** coreywright has quit IRC | 14:51 | |
| ildikov | mriedem: I get your point, I still think it would really be easier and quicker to merge the live_migrate patch; it works and functional and what we need to do is a refactor only | 14:51 |
| mriedem | ildikov: i don't see the problem with splitting them out, | 14:53 |
| mriedem | yo'uve managed things with depends-on up to this point anwyay | 14:53 |
| mriedem | how is this any different? | 14:53 |
| mriedem | ideally i like changes in a series, but as you said that gets hard with multiple owners in that series | 14:53 |
| *** gcb has quit IRC | 14:54 | |
| ildikov | just adds to the timeline which seems somewhat unnecessary | 14:54 |
| ildikov | and still extra work and coordination | 14:55 |
| ildikov | but as I usually don't win in these debates, I will look into splitting that out :) | 14:56 |
| *** slaweq_ has joined #openstack-nova | 14:56 | |
| *** sree has joined #openstack-nova | 14:56 | |
| dansmith | efried: I don't think it needs a spec | 14:58 |
| *** slaweq has quit IRC | 14:59 | |
| *** slaweq has joined #openstack-nova | 14:59 | |
| *** beekneemech is now known as bnemec | 15:00 | |
| *** slaweq_ has quit IRC | 15:00 | |
| *** slaweq has quit IRC | 15:00 | |
| *** mdnadeem has quit IRC | 15:01 | |
| sahid | dansmith: don't you think that could be interesting for the contributors/reviewers to understand the design here? | 15:01 |
| sahid | perhaps a related specs | 15:02 |
| dansmith | sahid: of passing request information into the virt driver? like a port or network? No, I think it's pretty clear | 15:02 |
| sahid | yes that si clear, but it's not the only method to update, and what is the aim, are you going to update all the drivers methods to add that new param | 15:03 |
| sahid | ? | 15:03 |
| efried | In this case I agree the intent can be expressed adequately in the commit message. | 15:03 |
| efried | We're not impacting APIs or objects or configs or anything. | 15:03 |
| sahid | well you are impacting the drivers | 15:03 |
| dansmith | sahid: he's already updating all the drivers' method signatures | 15:04 |
| efried | Yup. And updating them accordingly in the scope of the change. | 15:04 |
| efried | With a nice docstring on the new param. | 15:04 |
| *** coreywright has joined #openstack-nova | 15:04 | |
| sahid | ok fair, just asking... that change is happening like that without explaining, i was expected some | 15:04 |
| efried | Definitely a reasonable point sahid, thanks for bringing it up. | 15:05 |
| sahid | efried: if you can update the commit message that would be nice so we are all on the same page | 15:06 |
| efried | sahid Will do. | 15:06 |
| *** ijw has joined #openstack-nova | 15:07 | |
| *** sdague has quit IRC | 15:07 | |
| *** sdague has joined #openstack-nova | 15:07 | |
| *** yamamoto has joined #openstack-nova | 15:08 | |
| mriedem | ildikov: so to recap what i think the changes are, please correct me if i'm wrong: 1. split out the refresh_connection_info change to the bottom of the series. for old style attachments we still call os-initialize_connection, for new style we call attachment_get. 2. you or stvnoyes updates the live migration patch on top of that and do an explicit attachment_update call since refresh_connection_info won't do that for you anym | 15:08 |
| mriedem | 3. the new attach api flow goes on top of those other 2 (maybe not changes needed except splitting that code out) | 15:08 |
| mriedem | ildikov: stvnoyes: once those bottom 2 are done i think they are easy to get merged this week | 15:09 |
| openstackgerrit | Merged openstack/nova-specs master: Spec for limiting GET /allocation_candidates https://review.openstack.org/504540 | 15:09 |
| mriedem | as i think that's the only hold up on the live migration patch at this point | 15:09 |
| *** chyka has joined #openstack-nova | 15:10 | |
| *** tssurya has quit IRC | 15:11 | |
| efried | mriedem sdague This may have happened before you came on, but niraj_singh was testing out the service_user token thing and got a really cryptic exception. Turns out the issue was not having specified auth creds in the conf. I thought we could do a better job by blowing up here if _SERVICE_AUTH comes back None (which is what happens if you don't specify an auth_type): https://github.com/openstack/nova/blob/master/nova/serv | 15:11 |
| efried | ice_auth.py#L31 | 15:11 |
| efried | Nice. https://github.com/openstack/nova/blob/master/nova/service_auth.py#L31 | 15:12 |
| sdague | efried: yeh, that would be cool | 15:12 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix nova-manage commands that do not exist https://review.openstack.org/512324 | 15:13 |
| *** dave-mccowan has quit IRC | 15:13 | |
| efried | niraj_singh You open that bug yet? I don't see it. | 15:14 |
| *** yamamoto has quit IRC | 15:14 | |
| *** armax has quit IRC | 15:14 | |
| *** xyang1 has joined #openstack-nova | 15:14 | |
| *** acormier has joined #openstack-nova | 15:14 | |
| ildikov | mriedem: sorry, on a call | 15:15 |
| *** takashin has left #openstack-nova | 15:15 | |
| ildikov | mriedem: your summary looks good to me | 15:15 |
| *** rmart04 has quit IRC | 15:18 | |
| *** acormier has quit IRC | 15:22 | |
| mriedem | bauzas: you might find this ML thread about an interesting bug to be very interesting, interestingly enouh http://lists.openstack.org/pipermail/openstack-operators/2017-October/014393.html | 15:23 |
| dansmith | hmm, I'm not seeing things I submitted a few minutes ago in the zuul queue | 15:25 |
| *** gmann is now known as gmann_afk | 15:25 | |
| *** tssurya has joined #openstack-nova | 15:27 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Service user token requested with no auth https://review.openstack.org/512329 | 15:27 |
| efried | sdague ^ | 15:27 |
| *** vladikr has joined #openstack-nova | 15:27 | |
| *** lajoskatona has quit IRC | 15:29 | |
| openstackgerrit | garyk proposed openstack/nova master: Add debug information to metadata requests https://review.openstack.org/511895 | 15:30 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 15:30 |
| *** slaweq has joined #openstack-nova | 15:30 | |
| *** catintheroof has joined #openstack-nova | 15:30 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | 15:30 |
| *** sbezverk has quit IRC | 15:31 | |
| *** Oku_OS is now known as Oku_OS-away | 15:32 | |
| *** suresh12 has joined #openstack-nova | 15:33 | |
| sahid | mriedem or perhaps johnthetubaguy, we have an interesting fix here https://review.openstack.org/#/c/400384/18, perhaps you can have a look | 15:33 |
| *** slaweq has quit IRC | 15:34 | |
| *** jogo has quit IRC | 15:36 | |
| *** thorst has quit IRC | 15:37 | |
| *** catintheroof has quit IRC | 15:38 | |
| *** catintheroof has joined #openstack-nova | 15:38 | |
| *** sridharg has quit IRC | 15:39 | |
| *** slaweq has joined #openstack-nova | 15:44 | |
| *** jpena is now known as jpena|brb | 15:45 | |
| *** markvoelker has quit IRC | 15:47 | |
| *** slaweq has quit IRC | 15:48 | |
| *** tuanla_ has quit IRC | 15:48 | |
| *** tuanla_ has joined #openstack-nova | 15:48 | |
| *** tuanla__ has joined #openstack-nova | 15:49 | |
| *** tuanla__ has quit IRC | 15:51 | |
| *** tuanla_ has quit IRC | 15:51 | |
| johnthetubaguy | sean-k-mooney: mriedem: the upgrade bits seem super complicated in that port binding spec, I though we were going to just follow the Cinder attachments model there? | 15:53 |
| mriedem | i need to look at the latest revision, but i mentioned something about that in earlier review | 15:55 |
| *** tbachman has joined #openstack-nova | 15:56 | |
| mriedem | gibi: thanks re https://review.openstack.org/#/c/510938/ +2 | 15:56 |
| sean-k-mooney | johnthetubaguy: i am not that familire with how things are done for cinder but im happy to implement something similar if you can provide the wording | 15:56 |
| johnthetubaguy | sean-k-mooney: no worries, adding a comment | 15:57 |
| johnthetubaguy | sean-k-mooney: really I was thinking, we do nothing until everything is upgraded | 15:57 |
| johnthetubaguy | would that work for this case? | 15:57 |
| *** ttsiouts has quit IRC | 15:58 | |
| sean-k-mooney | johnthetubaguy: yes it would i think i have that in the alternitives | 15:58 |
| *** ttsiouts has joined #openstack-nova | 15:58 | |
| sean-k-mooney | basically we would fall back to todays behavior unless all partys support the new workflow else execut new workflow | 15:59 |
| johnthetubaguy | sean-k-mooney: that sounds safest and easiest to test and get right | 15:59 |
| johnthetubaguy | sean-k-mooney: am I missing something big we gain by supporting this across mixed compute nodes? | 16:00 |
| *** nikhil has joined #openstack-nova | 16:00 | |
| sean-k-mooney | johnthetubaguy: its slightly simpler to change neutron backend as part of a upgrade | 16:01 |
| johnthetubaguy | sean-k-mooney: yeah, that just needs to be the next upgrade I guess | 16:01 |
| sean-k-mooney | without the mixed mode basically you upgrade everything to queens then upgrade your neutron again to different network backend. | 16:02 |
| johnthetubaguy | right, only affects the first upgrade to queens though right? | 16:02 |
| sean-k-mooney | i have to join a call sorry but if you leave that comment ill swap the mixed migration and alternitves sections | 16:03 |
| *** dave-mccowan has joined #openstack-nova | 16:03 | |
| sean-k-mooney | yes | 16:03 |
| johnthetubaguy | sean-k-mooney: no worries | 16:03 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] manage cache headers for inventories https://review.openstack.org/510954 | 16:03 |
| sean-k-mooney | with mixed mode you can do a roleing upgrade and change the backend at the same time. with out you do one then the other | 16:03 |
| johnthetubaguy | sean-k-mooney: OK, so I am missing a bit, will re-read | 16:05 |
| *** slaweq has joined #openstack-nova | 16:05 | |
| *** ttsiouts has quit IRC | 16:05 | |
| *** rmart04 has joined #openstack-nova | 16:10 | |
| *** rmart04 has quit IRC | 16:10 | |
| *** yamamoto has joined #openstack-nova | 16:10 | |
| *** chyka has quit IRC | 16:12 | |
| *** chyka has joined #openstack-nova | 16:12 | |
| mriedem | melwitt: am i missing something or is nothing calling InstanceList.get_count_by_vm_state ? | 16:14 |
| *** mvk has quit IRC | 16:14 | |
| *** yamamoto has quit IRC | 16:15 | |
| mriedem | can someone remind me if there was supposed to be some quota advantage to shelving an instance? | 16:15 |
| mriedem | because i'm not seeing it | 16:16 |
| *** armax has joined #openstack-nova | 16:16 | |
| *** suresh12 has quit IRC | 16:17 | |
| *** suresh12 has joined #openstack-nova | 16:18 | |
| alaski | When I implemented shelving it did not reduce quota to shelve an instance | 16:18 |
| mriedem | https://blueprints.launchpad.net/nova/+spec/shelve-instance | 16:18 |
| mriedem | yeah was just reading | 16:18 |
| mriedem | alaski: also, why do you have a keyword match on "shelve"? :) | 16:18 |
| *** sshwarts has quit IRC | 16:19 | |
| alaski | hah. I just felt a twitch when it was mentioned. And happened to have this channel open | 16:19 |
| *** markvoelker has joined #openstack-nova | 16:20 | |
| *** jpena|brb is now known as jpena | 16:21 | |
| *** lucasagomes is now known as lucas-afk | 16:21 | |
| *** suresh12 has quit IRC | 16:22 | |
| mriedem | trying to see if a shelved offload instance affects the simple tenant usage result | 16:23 |
| mriedem | i don't think we make any distinction even though the instance isn't consuming local disk or ram or cpu | 16:23 |
| alaski | I can't recall what simple tenant usage is tracking. But shelve offload frees up the space for the usage tracker and scheduling, but doesn't return the quota to the user. | 16:25 |
| mriedem | sure, but as a user i don't care about freeing up space for others :) | 16:26 |
| mriedem | if i'm still going to get charged | 16:26 |
| alaski | yeah | 16:26 |
| alaski | I know that freeing up the quota has come up a few times, and there's no strong argument for not doing it | 16:26 |
| *** yamahata has quit IRC | 16:26 | |
| alaski | the thing to figure out is how to properly tell the user that they can't unshelve something if they've subsequently filled up their quota | 16:27 |
| *** slaweq has quit IRC | 16:27 | |
| *** slaweq has joined #openstack-nova | 16:27 | |
| mriedem | 409 overquota on unshelve | 16:27 |
| mriedem | we'd do a count during unshelve and fail if there is no room | 16:28 |
| mriedem | similar to resize up to a larger flavor i'd tihnk | 16:28 |
| mriedem | but, os-simple-tenant-usage doesn't check quota usage at all... :) | 16:28 |
| *** ociuhandu has quit IRC | 16:28 | |
| mriedem | sort of redundant in that way | 16:28 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 16:30 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove dead parameter from '_create_domain_and_network' https://review.openstack.org/512352 | 16:31 |
| *** sbezverk has joined #openstack-nova | 16:32 | |
| *** jogo has joined #openstack-nova | 16:33 | |
| alaski | I'll just apologize again for introducing shelve :) it's an interesting idea, but it really messes with the model of how things work | 16:34 |
| mriedem | heh | 16:35 |
| mriedem | oh laski | 16:35 |
| *** slaweq has quit IRC | 16:35 | |
| *** tesseract has quit IRC | 16:35 | |
| mriedem | alaski: this came up today b/c someone reported a bug where they created an instance in a specific AZ, shelve offloaded, then on unshelve the AZ was gone so they get a NoValidHost | 16:36 |
| mriedem | so that started a discussion on what we should do about fixing it, if anything | 16:36 |
| *** AlexeyAbashkin has quit IRC | 16:36 | |
| mriedem | which reminds me, if you created your server with personality files and shelved it, those files are gone on unshelve | 16:37 |
| mriedem | another reason to deprecate personality files https://review.openstack.org/#/c/509013/ | 16:37 |
| mriedem | any other specs cores around want to +W ^ ? | 16:37 |
| *** sambetts is now known as sambetts|afk | 16:37 | |
| alaski | My first inclination is to say that when that AZ was removed whoever was doing it probably didn't realize that there were shelved instances there, and maybe that needs to be exposed better. | 16:38 |
| *** penick has joined #openstack-nova | 16:38 | |
| mriedem | yeah that was one option - sylvain has a spec to not allow renaming or deleting AZs while there are instances tied to them | 16:39 |
| *** penick_ has joined #openstack-nova | 16:39 | |
| alaski | that would probably affect resize as well | 16:39 |
| alaski | I could also see an argument for being able to provide scheduling hints on any API requests that can cause an instance move. Though that's a fair amount of complexity to introeduce | 16:41 |
| mriedem | that was another option that came up | 16:41 |
| mriedem | and i'm on the fence about adding more functionality to the unshelve API if we kind of want to kill it long-term | 16:42 |
| *** ijw has quit IRC | 16:42 | |
| *** penick has quit IRC | 16:43 | |
| alaski | that's my favorite option, kill it, but I always felt like it was too entrenched for that | 16:43 |
| mriedem | we actually have no idea | 16:43 |
| mriedem | as to it's usage in the wild | 16:43 |
| mriedem | that's the frustrating part | 16:43 |
| alaski | yeah. I always wanted to think it wasn't really used. But at every summit a few people would mention it and want to make it better because they used it | 16:44 |
| mriedem | there were those one guys that wanted to make it super fast somehow | 16:45 |
| mriedem | like, enterprise fast | 16:45 |
| mriedem | shelve at the speed of business | 16:45 |
| alaski | haha | 16:45 |
| alaski | you need a (tm) there | 16:46 |
| mriedem | nova: shelving at the speed of business (tm) | 16:46 |
| mriedem | needs a t-shirt | 16:46 |
| *** penick_ has quit IRC | 16:46 | |
| alaski | I would wear that | 16:50 |
| *** links has joined #openstack-nova | 16:51 | |
| *** suresh12 has joined #openstack-nova | 16:51 | |
| mriedem | alaski: here you go https://www.customink.com/designs/novashelve/scj0-00b6-zdmw/share?pc=EMAIL-40778&utm_campaign=shared%20design&utm_source=share%20link&utm_medium=shared%20design&utm_content=shared%20desktop | 16:52 |
| mriedem | dansmith: ^ place orders now | 16:52 |
| mriedem | supplies are limited | 16:52 |
| alaski | minimum order of 6 | 16:53 |
| mriedem | i can change the color | 16:53 |
| mriedem | sec | 16:53 |
| *** mvk has joined #openstack-nova | 16:54 | |
| mriedem | https://www.customink.com/designs/novashelve/scj0-00b6-zdmw/share?pc=EMAIL-40778&utm_campaign=shared%20design&utm_source=share%20link&utm_medium=shared%20design&utm_content=shared%20desktop | 16:54 |
| *** penick has joined #openstack-nova | 16:55 | |
| *** markmcclain has quit IRC | 16:55 | |
| mriedem | $33?! | 16:55 |
| mriedem | i'm not sure the joke is worth that much | 16:55 |
| *** suresh12 has quit IRC | 16:55 | |
| *** markmcclain has joined #openstack-nova | 16:56 | |
| alaski | not quite | 16:56 |
| mriedem | i'm also pretty sure the foundation people would not allow the usage of the logo on a shirt that makes fun of nova | 16:56 |
| *** slaweq has joined #openstack-nova | 16:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/472933 | 16:58 |
| *** derekh has quit IRC | 16:58 | |
| *** lpetrut has quit IRC | 16:59 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Remove reboot argument to _create_domain_and_network https://review.openstack.org/512363 | 16:59 |
| dansmith | mriedem: lol | 17:03 |
| *** slaweq has quit IRC | 17:04 | |
| *** yamahata has joined #openstack-nova | 17:05 | |
| *** ragiman has quit IRC | 17:05 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Service user token requested with no auth https://review.openstack.org/512329 | 17:07 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 17:07 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 17:07 |
| *** sree has quit IRC | 17:07 | |
| dtantsur | folks, could you please put spec https://review.openstack.org/#/c/449155/ on your review radar? | 17:08 |
| dtantsur | this should be contained in the ironic driver, and may end up a big win for us | 17:08 |
| dtantsur | thanks! | 17:08 |
| *** sree has joined #openstack-nova | 17:08 | |
| dtantsur | mriedem: ^^ | 17:09 |
| *** sree has quit IRC | 17:11 | |
| *** sree has joined #openstack-nova | 17:12 | |
| *** yamamoto has joined #openstack-nova | 17:12 | |
| *** yamamoto has quit IRC | 17:18 | |
| *** karthiks has quit IRC | 17:19 | |
| *** jpena is now known as jpena|off | 17:20 | |
| *** Apoorva has joined #openstack-nova | 17:20 | |
| *** acormier has joined #openstack-nova | 17:22 | |
| *** sree has quit IRC | 17:23 | |
| *** sree has joined #openstack-nova | 17:25 | |
| *** cali_boxer has quit IRC | 17:26 | |
| *** acormier has quit IRC | 17:27 | |
| *** slaweq has joined #openstack-nova | 17:28 | |
| *** cali_boxer has joined #openstack-nova | 17:28 | |
| *** rodolof has quit IRC | 17:28 | |
| *** rodolof has joined #openstack-nova | 17:29 | |
| *** ijw has joined #openstack-nova | 17:29 | |
| ralonsoh | dansmith: hi, about https://review.openstack.org/#/c/449257/42/nova/objects/instance_pci_requests.py. | 17:29 |
| ralonsoh | dansmith: I modified the obj_make_compatible function to handle this. I don't know why I can't change the type of this field | 17:29 |
| *** sree has quit IRC | 17:29 | |
| dansmith | ralonsoh: because an older node sending one of these to a newer node won't trigger a backport operation (since the version says it should be compatible), but it won't deserialize it with the right field type | 17:30 |
| *** suresh12 has joined #openstack-nova | 17:31 | |
| dansmith | ralonsoh: let me pull your patch down and make a change to show you, just a sec | 17:31 |
| *** gjayavelu has joined #openstack-nova | 17:31 | |
| ralonsoh | dansmith: for sure, and thanks! | 17:31 |
| *** slaweq has quit IRC | 17:32 | |
| *** gszasz has quit IRC | 17:33 | |
| *** ijw has quit IRC | 17:33 | |
| *** vks1 has quit IRC | 17:37 | |
| abhi89 | dansmith, melwitt, sdague: please review https://review.openstack.org/#/c/505546/ .. its been pending since a long time now.. | 17:38 |
| dansmith | ralonsoh: here's a test: https://pastebin.com/LapK0Rnz | 17:40 |
| dansmith | ralonsoh: which gives you this: https://pastebin.com/cXTH5P2i | 17:41 |
| dansmith | ralonsoh: revert your field change and run that and you'll see it passes | 17:41 |
| *** links has quit IRC | 17:42 | |
| *** dave-mccowan has quit IRC | 17:42 | |
| ralonsoh | dansmith: but in the test, nova_object.version should be 1.1 if sent by an old node | 17:43 |
| ralonsoh | dansmith: shouldn't it? | 17:43 |
| dansmith | ralonsoh: it won't matter :) | 17:44 |
| *** esberglu has joined #openstack-nova | 17:45 | |
| *** smatzek has joined #openstack-nova | 17:45 | |
| *** slaweq has joined #openstack-nova | 17:48 | |
| *** suresh12 has quit IRC | 17:48 | |
| *** cali_boxer has quit IRC | 17:49 | |
| *** suresh12 has joined #openstack-nova | 17:50 | |
| *** cali_boxer has joined #openstack-nova | 17:51 | |
| *** slaweq has quit IRC | 17:53 | |
| *** rodolof has quit IRC | 17:54 | |
| *** rodolof has joined #openstack-nova | 17:55 | |
| *** soum has joined #openstack-nova | 17:58 | |
| *** AlexeyAbashkin has joined #openstack-nova | 17:59 | |
| *** soum has quit IRC | 17:59 | |
| *** dave-mccowan has joined #openstack-nova | 17:59 | |
| *** tonygunk has joined #openstack-nova | 18:00 | |
| *** psachin has quit IRC | 18:00 | |
| *** crushil has quit IRC | 18:00 | |
| mriedem | dansmith: want to send this ocata change in? https://review.openstack.org/#/c/482219/ - that's holding up a newton change | 18:01 |
| dansmith | "want" is a little strong | 18:02 |
| mriedem | if you could find it in your heart | 18:02 |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| *** AlexeyAbashkin has quit IRC | 18:03 | |
| *** fragatina has quit IRC | 18:07 | |
| *** catintheroof has quit IRC | 18:07 | |
| *** catintheroof has joined #openstack-nova | 18:08 | |
| mriedem | artom: hmm, tonyb raised a concern in https://review.openstack.org/#/c/471356/ about whether or not VIR_DOMAIN_BLOCK_REBASE_COPY_DEV would be in the libvirt version we support in newton | 18:09 |
| mriedem | it looks like VIR_DOMAIN_BLOCK_REBASE_COPY_DEV was added in 1.2.9 | 18:09 |
| mriedem | and min supported version of libvirt in newton (and ocata) is 1.2.1 | 18:09 |
| cdent | dansmith, mriedem, edleafe: in discussion on https://review.openstack.org/#/c/510625/ it was agreed that the GET /allocations/{consumer_uuid} rep should include project_id and user_id since PUT will require it. In your opinions should GET /resource_providers/{uuid}/allocations as well? I think not because the layout is much different (because the cardinality of the output) (see | 18:10 |
| cdent | https://developer.openstack.org/api-ref/placement/#resource-provider-allocations ) | 18:10 |
| *** baoli has quit IRC | 18:10 | |
| *** yamamoto has joined #openstack-nova | 18:14 | |
| dansmith | cdent: I dunno, if there's not a need for it currently I wouldn't personally spend time on it | 18:15 |
| *** catinthe_ has joined #openstack-nova | 18:16 | |
| dansmith | ideally I would think it would be best to have that information in each allocation, but.. | 18:16 |
| *** markvoelker has quit IRC | 18:16 | |
| *** markvoelker has joined #openstack-nova | 18:17 | |
| mriedem | cdent: on GET /resource_providers/{uuid}/allocations you'd have to put project_id and user_id within each consumer-uuid keyed dict entry right? | 18:17 |
| mriedem | i don't think we care about that, | 18:17 |
| mriedem | if you really needed to take the result of GET /resource_providers/{uuid}/allocations and figure out the project/user for one of those allocation cosumers, you could turn around and call GET /allocations/{consumer_uuid} yeah? | 18:17 |
| cdent | right, it would be weird | 18:17 |
| cdent | yes | 18:17 |
| cdent | cool, this is the answer I was hoping for | 18:18 |
| mriedem | artom: so i think we probably need to either revert https://review.openstack.org/#/c/471353/ or put a fix on top | 18:18 |
| mriedem | to not try and use VIR_DOMAIN_BLOCK_REBASE_COPY_DEV if it's not available | 18:18 |
| *** catintheroof has quit IRC | 18:18 | |
| dansmith | I can see uses for it it but like I say, I don't think there's anything begging for it right now | 18:18 |
| cdent | for the consumer side I had to take jay’s only one join query recent cleanup and add four more to get the project and user inf | 18:18 |
| *** yamamoto has quit IRC | 18:19 | |
| *** markvoelker has quit IRC | 18:21 | |
| melwitt | mriedem: yeah, nothing is calling InstanceList.get_count_by_vm_state. it was intended to be used for limiting the number of pending resizes but once we got to it, we decided not to change the existing quota behavior for resizes | 18:23 |
| *** catinthe_ has quit IRC | 18:23 | |
| *** shaner has quit IRC | 18:24 | |
| *** shaner_ has joined #openstack-nova | 18:24 | |
| *** READ10 has quit IRC | 18:24 | |
| *** lpetrut has joined #openstack-nova | 18:25 | |
| *** vvargaszte has joined #openstack-nova | 18:31 | |
| *** jmlowe has joined #openstack-nova | 18:32 | |
| efried | alex_xu I reread https://review.openstack.org/#/c/497713/ with fresh eyes and commented. | 18:33 |
| *** MasterOfBugs has joined #openstack-nova | 18:36 | |
| efried | alex_xu dansmith cdent edleafe mriedem Pursuant to this morning's discussion about traits-on-shared-with-nested, we should figure out whether there's a way to take aggregates out of the picture until ^ is figured out. | 18:37 |
| *** Aditya has joined #openstack-nova | 18:37 | |
| *** pramodrj07 has joined #openstack-nova | 18:39 | |
| *** ociuhandu has joined #openstack-nova | 18:40 | |
| cdent | efried: how far out of the picture do you want them to go? | 18:41 |
| efried | cdent Well, the logic in https://review.openstack.org/#/c/479766/ is trying to handle hitting shared RPs with the results. | 18:41 |
| efried | cdent And the logic it's using is at the very least implementing an arbitrary and undiscussed design point. | 18:42 |
| *** shaner_ has quit IRC | 18:43 | |
| efried | cdent Which is: When you ask for traits X, Y, and Z, I can give you back an allocation request (i.e. one hit in the list of GET /allocation_candidates) that has traits X and Y in the compute host RP but trait Z in a shared RP in the same aggregate as the compute host RP. | 18:43 |
| *** shaner has joined #openstack-nova | 18:44 | |
| * cdent nods | 18:44 | |
| cdent | presumably that shared RP is also satisfying some inventory requirement, not _just_ a trait? | 18:45 |
| efried | This is going to break down as soon as we have any scenario more complicated than traits X, Y, and Z being mutually exclusive on the compute host and shared RP. | 18:45 |
| artom | mriedem, ah crap | 18:45 |
| efried | cdent Yeah, there's that too. I'm not sure that's being accounted for. | 18:45 |
| mriedem | artom: i'm fixing | 18:45 |
| artom | I thought I checked for min libvirt version | 18:45 |
| artom | Clearly I didn't :( | 18:45 |
| artom | At least not far back enough | 18:46 |
| *** Aditya has quit IRC | 18:46 | |
| artom | mriedem, what's the fix? Check for libvirt version in code? | 18:46 |
| efried | cdent I think my main issue is that I don't think we have the semantic codified for the general case in any spec. | 18:47 |
| cdent | efried: my understanding, although I don’t know if this is explicitly expressed in spec or code anywhere, is that if something sets itself up to sharing provider (which requires two things: being in an aggregate with some other stuff, having the MISC_SHARES_VIA_AGGREGATE trait), then it is obliged to only aggregate with something that is a root provider | 18:47 |
| mriedem | artom: hasattr | 18:47 |
| cdent | efried: that is, it is the domain of aggregation that helps avoid some of the problems | 18:48 |
| *** READ10 has joined #openstack-nova | 18:48 | |
| artom | mriedem, ah. Would we have a way to test that in CI with an old libvirt version? | 18:48 |
| mriedem | artom: in unit tests | 18:49 |
| efried | cdent Is it possible for a compute host to have DISK_GB inventory *and* be in an aggregate with a shared storage RP? | 18:49 |
| mriedem | efried: yes | 18:49 |
| artom | mriedem, you mean with a fake python-libvirt? | 18:49 |
| *** tbachman has quit IRC | 18:50 | |
| mriedem | efried: well, maybe :) | 18:50 |
| mriedem | not sure actually | 18:50 |
| *** slaweq has joined #openstack-nova | 18:50 | |
| *** rodolof has quit IRC | 18:50 | |
| *** rodolof has joined #openstack-nova | 18:50 | |
| efried | mriedem cdent So this is the scenario alex_xu and I discussed earlier. If I ask for storagey traits X and Y; and my local disk has X but not Y; and my shared storage has Y but not X, I *should* not see this compute host in the allocation candidates. | 18:50 |
| * artom isn't feeling too confident and would feel better if the code ran against an actual python-libvirt to make sure it does what we think it does | 18:50 | |
| cdent | efried, mriedem: that’s something we initialy said “no” and then later changed to “yes” and then later “actually not sure" | 18:50 |
| efried | But I think as currently written, I will. | 18:51 |
| cdent | efried: you are correct, you should not get the compute host | 18:51 |
| efried | cdent mriedem Which probably means we should architect it in such a way that we don't lock ourselves into disallowing it. | 18:51 |
| cdent | because it can’t satisfy itself | 18:51 |
| cdent | nor can it satisfy with the help of an aggregate friend | 18:51 |
| efried | cdent Right; but I think as https://review.openstack.org/#/c/479766/10/nova/objects/resource_provider.py is written, that *will* hit. | 18:52 |
| efried | And once nested is in play, it gets even messier. Because we *are* allowing a certain amount of trait "propagation" within a tree. But only in some directions. | 18:53 |
| cdent | I’ve been unable to fully decode the SQL in that stack, so have mostly just been waiting for some tests along those lines | 18:53 |
| *** jmlowe has quit IRC | 18:53 | |
| cdent | so what I would think is the right thing here, is making sure, since we already have the concept of shared within the object code, is that there are functional tests in db/test_resource_provider.py for the scenarios you’ve described | 18:54 |
| cdent | and yeah, once nested is in play, it will get even more interesting | 18:54 |
| bauzas | dansmith: thanks for efried's change, will look | 18:54 |
| bauzas | mriedem: ack, will look | 18:54 |
| *** slaweq has quit IRC | 18:55 | |
| efried | bauzas Note that there's still work to be done there. The change is kinda blowing up. Hope to have another rev today or tomorrow. | 18:56 |
| *** sree has joined #openstack-nova | 18:57 | |
| openstackgerrit | Merged openstack/python-novaclient master: Update "The nova Shell Utility" in the user guide https://review.openstack.org/512224 | 19:00 |
| *** Guest28352 has joined #openstack-nova | 19:01 | |
| *** sree has quit IRC | 19:02 | |
| *** edmondsw has quit IRC | 19:04 | |
| *** edmondsw has joined #openstack-nova | 19:04 | |
| *** READ10 has quit IRC | 19:04 | |
| *** cdent has left #openstack-nova | 19:04 | |
| *** cdent has joined #openstack-nova | 19:05 | |
| openstackgerrit | Merged openstack/nova master: doc: Rework man pages https://review.openstack.org/502105 | 19:06 |
| *** edmondsw has quit IRC | 19:09 | |
| edleafe | efried: (reading back) so is the issue that the specified traits are not tied to a particular RP? | 19:10 |
| efried | edleafe Yes, that's one aspect of it. | 19:10 |
| efried | edleafe I'm comfortable with the semantics when we're just talking about nested RPs, but I think we have some holes (design points we haven't considered) when aggregates are put in the mix. | 19:11 |
| edleafe | efried: agreed | 19:11 |
| edleafe | efried: just wanted to make sure I followed what the problem was | 19:11 |
| efried | edleafe cdent One suggestion would be to force the aggregate inventory+traits to be specified in its own numbered group (per the 'granular' spec). | 19:12 |
| efried | Then we could greatly simplify the code. | 19:12 |
| efried | Because we could search aggregates separately. | 19:13 |
| efried | The thing I'm still unclear on is how much of a code commitment we've made to aggregates already. Are we reversing/breaking/removing "existing" function if we take aggregates completely out of the picture for Queens? | 19:14 |
| efried | Given that we've said we're not doing shared RPs in Queens. | 19:14 |
| cdent | efried: that kind of busts the point of aggregates, if you’re looking at things from the original shared disk need | 19:14 |
| cdent | efried: as far as I can tell, no, nothing breaks | 19:14 |
| cdent | the RT side keeps a map that it doesn’t use | 19:14 |
| efried | cdent Okay, talk me through the "original shared disk need". | 19:15 |
| cdent | “that kind of busts” -> where “that” is searching for aggregates separately | 19:15 |
| efried | I mean, I understand the scenario. | 19:15 |
| *** yamamoto has joined #openstack-nova | 19:15 | |
| efried | scenario / use case | 19:15 |
| cdent | the original shared disk need is: give me anything that can satisfying “I want some vcpu, disk and ram, doesn’t matter if some of it comes from a shared resource that is not the compute node and oh by the way, I’d like that disk to be SSD” | 19:16 |
| openstackgerrit | Merged openstack/nova stable/ocata: Test InstanceNotFound handling in 'nova usage' https://review.openstack.org/482219 | 19:16 |
| edleafe | efried: one idea had been to make traits on resources (e.g., per resource class). That was roundly rejected. | 19:16 |
| edleafe | efried: breaking up traits by numbers seems like a variation of that | 19:16 |
| efried | edleafe Ah, no, the traits are still on RPs. | 19:17 |
| cdent | I’m unclear on the exact problem that traits by numbers is solving? | 19:17 |
| edleafe | efried: I know, but the problem was the complexity, not the location | 19:17 |
| dansmith | mriedem: these are ready for a re-look I think: https://review.openstack.org/#/c/506420 | 19:19 |
| dansmith | fixed up based on the change I slid under both, and passing zuul | 19:19 |
| mriedem | ok, still working on this ocata snafu | 19:19 |
| efried | edleafe cdent Okay, expressing just this problem with aggregates... | 19:19 |
| *** yamamoto has quit IRC | 19:20 | |
| efried | Let's say hypothetically that I have a compute host with local disk that's capable of RAID5. And I have a shared storage provider whose storage is SSD. | 19:20 |
| *** hemna_ has quit IRC | 19:20 | |
| efried | Pre-nested, and pre-numbered-groups, but post-traits, you could only ask for inventory(VCPU, MEMORY_MB, DISK_GB) and traits(RAID5, SSD). | 19:21 |
| edleafe | If you requests traits of RAID5 and SSD, that host should not match | 19:21 |
| efried | Agreed. | 19:21 |
| efried | edleafe But how would that work? | 19:22 |
| *** READ10 has joined #openstack-nova | 19:22 | |
| efried | As currently architected, it *will* match that host. Cause it can't tell the difference between your CPU trait and your storage trait. | 19:22 |
| edleafe | well, those are both storage traits, no? | 19:23 |
| efried | So to make it clearer: | 19:23 |
| *** kylek3h has joined #openstack-nova | 19:23 | |
| efried | edleafe You ask for inventory(VCPU, MEMORY_MB, DISK_GB) and traits(RAID5, SSD, HW_CPU_X86_3DNOW) | 19:23 |
| efried | edleafe What part of the architecture knows that it's okay for HW_CPU_X86_3DNOW to be split up from SSD, but it's not okay for RAID5 to be split up from SSD? | 19:24 |
| edleafe | efried: for that you'll need jay to explain the big hairy sql | 19:25 |
| edleafe | efried: fwiw, I don't know that it can | 19:25 |
| efried | cdent So here's where numbered would come into play. I could ask for inventory1(VCPU, MEMORY_MB)&traits1(HW_CPU_X86_3DNOW) and inventory2(DISK_GB)&traits2(RAID5, SSD) | 19:25 |
| efried | edleafe I believe it can't. There's no possible way it could. | 19:25 |
| *** kylek3h has quit IRC | 19:26 | |
| cdent | efried: I still don’t see the problem in the example you’ve given (non-numbered): | 19:26 |
| efried | cdent Because it'll match that host+shared that it shouldn't. | 19:26 |
| cdent | [t NeX] | 19:26 |
| purplerbot | <efried> edleafe You ask for inventory(VCPU, MEMORY_MB, DISK_GB) and traits(RAID5, SSD, HW_CPU_X86_3DNOW) [2017-10-16 19:23:59.885269] [n NeX] | 19:26 |
| edleafe | efried: if that's the case, then shared storage can't work, because shared storage would never have CPU traits | 19:26 |
| cdent | right now that should not match, and you don’t want it to match | 19:27 |
| efried | cdent Agree you don't want it to. But it *will*. | 19:27 |
| cdent | efried: is that because of broken code or broken modelling, that’s what I’m trying to get at | 19:27 |
| *** suresh12 has quit IRC | 19:28 | |
| *** cleong has quit IRC | 19:28 | |
| cdent | you should only get a trait if it’s resource provider is also providing inventory | 19:28 |
| cdent | if that ^ statement is true | 19:28 |
| efried | cdent Well, if we put a stake in the ground and say you can't possibly ever have a compute host with local storage associated ("aggregated") with a shared storage provider, then the scenario doesn't arise. | 19:28 |
| cdent | and if you can’t split resource classes | 19:29 |
| cdent | then no problem | 19:29 |
| cdent | efried: let’s try to make this even more concrete, I’ll EOL when I’m done expressing the example | 19:29 |
| cdent | two resource providers: | 19:29 |
| cdent | compute_node1: inventory of VCPU 24, DISK_GB 1024, MEMORY_MB 1024, traits RAID5, HW_CPU_X86_3DNOW | 19:30 |
| *** mordred has quit IRC | 19:31 | |
| cdent | shared_diskA: inventory of DISK_GB 1024, traits RAID5, SSD, MISC_SHARES_VIA… | 19:31 |
| efried | no, take RAID5 off of shared_diskA | 19:31 |
| cdent | let me carry on as is | 19:32 |
| efried | otherwise you match and it's not a problem. | 19:32 |
| efried | okay. | 19:32 |
| cdent | and then we’ll do that scenario | 19:32 |
| *** burt has quit IRC | 19:32 | |
| cdent | those two are in an aggregate | 19:32 |
| efried | GET /allocation_candidates?resources=VCPU:2,MEMORY_MB:512,DISK_GB:2&required=HW_CPU_X86_3DNOW,RAID5,SSD | 19:33 |
| cdent | yes | 19:33 |
| *** hemna_ has joined #openstack-nova | 19:34 | |
| efried | You'll get { compute_node1: { VCPU: 2, MEMORY_MB: 512 }, shared_diskA: { DISK_GB: 2 } }. Cool. No problem. | 19:34 |
| cdent | okay | 19:34 |
| cdent | we agree that’s the design, right? | 19:34 |
| efried | yup | 19:34 |
| cdent | k, now | 19:34 |
| cdent | shared diskA loses the RAID5 trait, but compute_node1 keeps it | 19:35 |
| cdent | same request | 19:35 |
| efried | Per the existing design, you will get the same result. | 19:35 |
| *** suresh12 has joined #openstack-nova | 19:35 | |
| cdent | the expected result, as designed, is nothi8ng | 19:35 |
| efried | Correct. | 19:35 |
| efried | sorry, "as designed" - we don't have a design point for this. | 19:35 |
| *** slaweq has joined #openstack-nova | 19:36 | |
| efried | "as conceived", the expected result is nothing. | 19:36 |
| cdent | yes, better choice of word | 19:36 |
| efried | "as designed without having thought about this scenario" what you'll get is actually the same result as above. | 19:36 |
| cdent | you have a test that shows that? | 19:36 |
| efried | Because "as designed" we didn't think about this kind of split-trait scenario. | 19:36 |
| efried | Well, no, because we haven't actually implemented traits on RPs yet. That's what alex_xu's change set is doing. | 19:37 |
| efried | Which I suppose means there should be no harm in keeping that bit out of play for now? | 19:37 |
| efried | ( ref: alex_xu's change set: https://review.openstack.org/#/c/479766/10/nova/objects/resource_provider.py ) | 19:37 |
| cdent | a) there’s no harm keeping it out, but b) it’s the same thing I said before, we should write a test sooner than later that models the scenario you describe | 19:38 |
| efried | cdent Yes, good idea. | 19:38 |
| cdent | becuase as you talk about it, you make it sound like you think it expected to work. it is not | 19:38 |
| cdent | if the second scenario is returing a result there is a bug | 19:38 |
| *** burt has joined #openstack-nova | 19:39 | |
| efried | cdent Where should that test go? Cause without the above change, the framework isn't there. And if we keep aggregates out of the above change, same. | 19:39 |
| cdent | and even if there is no code for it yet, we’ve got a test case prepared (at least mentally) for when there is | 19:39 |
| cdent | isn’t there shared/aggregate code already pre-dating alex’s change? | 19:40 |
| cdent | in objects/resource_provider.py | 19:40 |
| efried | That's what I don't really know. | 19:40 |
| efried | His change was really my first exposure to shared/aggregate code. | 19:40 |
| *** crushil has joined #openstack-nova | 19:40 | |
| cdent | one sec | 19:40 |
| *** ralonsoh has quit IRC | 19:41 | |
| *** slaweq has quit IRC | 19:41 | |
| *** mordred has joined #openstack-nova | 19:42 | |
| cdent | efried: https://review.openstack.org/#/c/460798/ introduced getting rps that share via misc | 19:42 |
| efried | gaah, -1 for commit message width! | 19:43 |
| edleafe | the horror!!! | 19:43 |
| efried | cdent Aha, perfect, so yeah, this all makes total sense when traits are NOT in the picture at all. | 19:44 |
| efried | And breaks down pretty handily once traits are introduced. | 19:44 |
| *** READ10 has quit IRC | 19:45 | |
| cdent | so yeah, if a compute node is satisfy VCPU, and has a trait that is “about disk” | 19:45 |
| cdent | things do get very weird | 19:45 |
| efried | btw, I'm asserting that the scenario we walked through above is representative, not necessarily realistic. That is, even if we say "that'll never happen", something with the same characteristics is sure to pop up. | 19:45 |
| cdent | yeah, I get that | 19:46 |
| cdent | it’s better to be robust in the face of possibilities | 19:46 |
| efried | One could imagine similar scenarios where network resources are provided by both the compute node and some shared RP. | 19:46 |
| cdent | for a while the stake you mention above did exist, but then it went away, one option is for it to come back | 19:46 |
| efried | Okay, what about when I have my ephemeral boot disk via local storage and my volumes shared (as they're likely to be) | 19:48 |
| efried | Could solve it by modeling those storage types with different resource classes. But ew. | 19:48 |
| efried | And in any case, there we get into a situation where we need to request separate chunks of storage inventory from separate RPs. | 19:49 |
| *** vvargaszte has quit IRC | 19:49 | |
| cdent | yeah, this is not the first time that question comes up. I don’t know the answer right now. | 19:49 |
| efried | But it sounds like the change set you pointed out means we do actually need an answer for this right now. | 19:50 |
| *** armax has quit IRC | 19:50 | |
| *** markvoelker has joined #openstack-nova | 19:51 | |
| cdent | efried: except that I’m not sure that code is doing anything | 19:51 |
| *** markvoelker has quit IRC | 19:51 | |
| *** markvoelker has joined #openstack-nova | 19:51 | |
| cdent | it may very well be | 19:51 |
| efried | cdent Well, it looks to me like it's letting you use aggregates. | 19:52 |
| cdent | yes, but aggregates aren’t yet used in any particular way | 19:52 |
| efried | Meaning there's no code that's creating aggregations yet? | 19:53 |
| cdent | unless there are lots of clients of placement out there that we are not away of (which there very well may be, vmware has an entire feature set based around placement that has nothing to do with nova) | 19:53 |
| efried | whee | 19:53 |
| cdent | upstream, as far as I know, don’t think so, but I don’t know | 19:53 |
| *** edmondsw has joined #openstack-nova | 19:54 | |
| *** shaner has quit IRC | 19:55 | |
| mriedem | wtf, why would hasattr return True on something and then immediately after that if i try to use it, i get an AttributeError? | 19:55 |
| efried | mriedem Overrides? | 19:56 |
| cdent | is it an ovo with a coerce? | 19:56 |
| mriedem | no, fakelibvirtr | 19:56 |
| *** markvoelker_ has joined #openstack-nova | 19:57 | |
| *** shaner has joined #openstack-nova | 19:59 | |
| *** markvoelker has quit IRC | 20:01 | |
| mriedem | it's not even a Mock so getattr shouldn't be getting in the way | 20:01 |
| *** sahid has quit IRC | 20:04 | |
| *** abhi89 has quit IRC | 20:05 | |
| *** hongbin has quit IRC | 20:05 | |
| cfriesen | are there any gotchas around running two instances of nova-compute one one system, one for local kvm instances and one for managing separate baremetal instances (for use with ironic)? | 20:06 |
| *** bjolo has joined #openstack-nova | 20:11 | |
| *** penick has quit IRC | 20:13 | |
| cfriesen | I guess it might cause problems if the binary and hostname are the same for the two of them | 20:13 |
| *** salv-orlando has joined #openstack-nova | 20:16 | |
| *** yamamoto has joined #openstack-nova | 20:17 | |
| *** bjolo has quit IRC | 20:18 | |
| *** ijw has joined #openstack-nova | 20:19 | |
| *** yamamoto has quit IRC | 20:22 | |
| mriedem | my god, | 20:22 |
| mriedem | when you find out you've been bashing your brains for an hour b/c of a copy paste error | 20:22 |
| cdent | it’s full of stars | 20:22 |
| *** armax has joined #openstack-nova | 20:24 | |
| *** baoli has joined #openstack-nova | 20:25 | |
| *** bjolo has joined #openstack-nova | 20:25 | |
| *** smatzek has quit IRC | 20:26 | |
| *** smatzek has joined #openstack-nova | 20:26 | |
| *** baoli has quit IRC | 20:26 | |
| *** baoli has joined #openstack-nova | 20:27 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: libvirt: add check for VIR_DOMAIN_BLOCK_REBASE_COPY_DEV https://review.openstack.org/512406 | 20:27 |
| mriedem | artom: ^ | 20:27 |
| *** Swami has joined #openstack-nova | 20:29 | |
| *** smatzek has quit IRC | 20:30 | |
| *** pchavva has quit IRC | 20:37 | |
| *** tonygunk has quit IRC | 20:44 | |
| *** cdent has quit IRC | 20:48 | |
| *** suresh12 has quit IRC | 20:50 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625 | 20:54 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | 20:54 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 20:54 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id https://review.openstack.org/512420 | 20:54 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Include project_id and user_id in GET /allocations https://review.openstack.org/512421 | 20:54 |
| mriedem | dansmith: melwitt: need an opinion on a backport scenario for https://review.openstack.org/#/c/505771/ | 20:55 |
| mriedem | i need to do the same fix as in ^ but generically since we also need that fix for resize, unshelve and evacuate | 20:55 |
| mriedem | i don't want to rename the method and then backport | 20:55 |
| mriedem | so i can either copy the same fix outside the method in the 4 different places and backport it, or move the fix into the method with a todo to rename later | 20:56 |
| mriedem | the rename would be master only | 20:56 |
| *** edmondsw has quit IRC | 20:56 | |
| mriedem | i'm leaning toward the latter - fix in one place, rename later | 20:57 |
| *** cali_boxer has quit IRC | 20:58 | |
| *** cali_boxer has joined #openstack-nova | 20:58 | |
| *** smatzek has joined #openstack-nova | 21:00 | |
| *** sree has joined #openstack-nova | 21:02 | |
| *** burt has quit IRC | 21:02 | |
| *** smatzek has quit IRC | 21:04 | |
| *** chyka has quit IRC | 21:04 | |
| *** chyka has joined #openstack-nova | 21:05 | |
| mgagne | mriedem: what's the proper channel to get a blueprint approved/reviewed? https://blueprints.launchpad.net/nova/+spec/rebuild-ironic-config-drive | 21:05 |
| *** Guest28352 has quit IRC | 21:05 | |
| mriedem | mgagne: generally during the weekly nova meeting if it's a specless blueprint | 21:05 |
| mriedem | i think this would be ok as a specless feature parity bp | 21:07 |
| *** sree has quit IRC | 21:07 | |
| *** suresh12 has joined #openstack-nova | 21:08 | |
| *** kfarr has joined #openstack-nova | 21:08 | |
| mriedem | mgagne: so the issue is you create an ironic node instance with a config drive, and then when you rebuild the instance in nova, the new instance doesn't have a config drive, correct? | 21:08 |
| mgagne | will add to open discussion for next meeting | 21:08 |
| mgagne | mriedem: yes | 21:08 |
| mriedem | ok, really a bug fix | 21:08 |
| mgagne | mriedem: looks like someone from nova team had a different opinion in the bug report | 21:09 |
| mgagne | and suggested a blueprint | 21:09 |
| mgagne | https://bugs.launchpad.net/ironic/+bug/1575935/comments/10 | 21:10 |
| openstack | Launchpad bug 1575935 in Ironic "Rebuild should also accept a configdrive" [Wishlist,In progress] - Assigned to Mathieu Gagné (mgagne) | 21:10 |
| mriedem | "reno: nova rebuild of ironic instances will no longer trash your config drive. you're welcome." | 21:10 |
| mriedem | ok, well, i'm cool with fixing it either way | 21:10 |
| mgagne | mriedem: I'm not sure what you were referring to? any link? | 21:12 |
| mriedem | i was being facetious, | 21:12 |
| mriedem | i.e. how would one write a release note for this as a 'feature'? | 21:13 |
| mriedem | anyway, bp approved | 21:13 |
| mgagne | mriedem: Configdrive is now regenerated when rebuilding an Ironic instance (matching behaviour of other virt drivers) | 21:13 |
| mgagne | we can say it's done for feature parity | 21:14 |
| *** suresh12 has quit IRC | 21:14 | |
| *** yamamoto has joined #openstack-nova | 21:18 | |
| *** kfarr has quit IRC | 21:20 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't ignore original retried hosts when moving an instance https://review.openstack.org/505771 | 21:21 |
| mriedem | tasker: updated and simplified that change to also fix the issue for resize, unshelve and evacuate ^ | 21:21 |
| mriedem | zioproto: ^ | 21:21 |
| *** yamamoto has quit IRC | 21:24 | |
| dansmith | mriedem: not sure what you're asking: if we should unify all the places into one call and backport that change? | 21:27 |
| *** lpetrut has quit IRC | 21:27 | |
| mriedem | dansmith: yeah that's just what i did | 21:28 |
| *** crushil has quit IRC | 21:29 | |
| *** peter-hamilton has quit IRC | 21:29 | |
| *** suresh12 has joined #openstack-nova | 21:30 | |
| *** suresh12 has quit IRC | 21:32 | |
| *** suresh12 has joined #openstack-nova | 21:32 | |
| *** ijw has quit IRC | 21:32 | |
| mriedem | totally unrelated, but when querying stuff out of the db, if a sort key/direction is not specified, is it just left up to the rdbms? | 21:33 |
| mriedem | zzzeek: ^? | 21:34 |
| mriedem | or is there oslo.db magic happening somewhere by default? | 21:34 |
| dansmith | I think it's just undefined/insert order if you don't ask | 21:38 |
| dansmith | asking for it to be sorted when you don't care would just be overhead | 21:38 |
| mriedem | yeah, trying to reply to some questions on this spec for paging migrations | 21:38 |
| mriedem | and sort orders since we don't impose one today | 21:38 |
| mriedem | and if the changes-since filter should be on the created_at or updated_at fields | 21:39 |
| melwitt | mriedem: yeah, I think what you did makes sense. do the change in one place and rename the method in a separate change that won't be backported | 21:40 |
| *** claudiub has quit IRC | 21:40 | |
| *** mgariepy has quit IRC | 21:41 | |
| *** ijw has joined #openstack-nova | 21:43 | |
| *** MasterOfBugs has quit IRC | 21:49 | |
| *** pramodrj07 has quit IRC | 21:49 | |
| *** mgariepy has joined #openstack-nova | 21:54 | |
| *** takashin has joined #openstack-nova | 21:57 | |
| *** xyang1 has quit IRC | 21:58 | |
| edleafe | What is the ovo trick for testing if an object's field is populated, without getting the "Cannot load 'foo' in the base class" error? | 21:59 |
| *** bjolo has quit IRC | 22:00 | |
| *** jcook has quit IRC | 22:00 | |
| melwitt | edleafe: there's obj_attr_is_set() for checking if a field has been set | 22:01 |
| mriedem | 'in' | 22:01 |
| mriedem | if 'foo' in bar | 22:01 |
| mriedem | uses obj_attr_is_set() under the covers | 22:01 |
| melwitt | oh. I keep forgetting you added that | 22:01 |
| melwitt | yeah, use that | 22:01 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | 22:02 |
| edleafe | mriedem: melwitt: thanks | 22:03 |
| *** awaugama has quit IRC | 22:03 | |
| mriedem | edleafe: api wg question, | 22:04 |
| mriedem | does changes-since come up in guidelines? | 22:04 |
| mriedem | changes-since filtering i mean | 22:04 |
| mriedem | and if so, are there guidelines on which column to filter it on? we use updated_at for filtering instances by changes-since | 22:04 |
| *** crushil has joined #openstack-nova | 22:06 | |
| edleafe | mriedem: not sure off the top of my head | 22:07 |
| edleafe | mriedem: ok, a quick review doesn't turn up anything | 22:08 |
| edleafe | that means it's ripe for defining :) | 22:09 |
| edleafe | I can add it to the discussion for Thursday's meeting | 22:09 |
| mriedem | not a big deal, | 22:09 |
| mriedem | someone is just proposing to add changes-since filtering to the os-migrations api and i said they should filter on updated_at to be consistent with changes-since on filtering instances | 22:10 |
| edleafe | well, updated_at sounds like the most logical choice | 22:10 |
| *** lyan has quit IRC | 22:11 | |
| *** suresh12 has quit IRC | 22:12 | |
| *** suresh12 has joined #openstack-nova | 22:13 | |
| *** salv-orlando has quit IRC | 22:14 | |
| *** sapcc-bot has quit IRC | 22:16 | |
| *** sapcc-bot has joined #openstack-nova | 22:16 | |
| *** yamamoto has joined #openstack-nova | 22:21 | |
| *** edmondsw has joined #openstack-nova | 22:25 | |
| *** yamamoto has quit IRC | 22:26 | |
| *** masber has joined #openstack-nova | 22:26 | |
| *** edmondsw has quit IRC | 22:29 | |
| *** ijw has quit IRC | 22:33 | |
| *** ijw has joined #openstack-nova | 22:34 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Send Allocations to spawn https://review.openstack.org/511879 | 22:36 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 22:36 |
| *** esberglu has quit IRC | 22:36 | |
| *** felipemonteiro_ has quit IRC | 22:37 | |
| *** gouthamr has quit IRC | 22:37 | |
| *** cali_boxer has quit IRC | 22:43 | |
| *** rodolof has quit IRC | 22:46 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/510324 | 22:48 |
| *** sdague has quit IRC | 22:56 | |
| *** lbragstad has quit IRC | 22:57 | |
| *** liverpooler has quit IRC | 22:57 | |
| *** catintheroof has joined #openstack-nova | 22:58 | |
| *** gouthamr has joined #openstack-nova | 22:58 | |
| *** esberglu has joined #openstack-nova | 23:11 | |
| *** hieulq has quit IRC | 23:11 | |
| *** hieulq has joined #openstack-nova | 23:12 | |
| *** esberglu has quit IRC | 23:15 | |
| *** salv-orlando has joined #openstack-nova | 23:15 | |
| *** ijw has quit IRC | 23:17 | |
| *** salv-orlando has quit IRC | 23:21 | |
| *** yamamoto has joined #openstack-nova | 23:22 | |
| *** chyka has quit IRC | 23:22 | |
| *** gjayavelu has quit IRC | 23:22 | |
| *** yamamoto has quit IRC | 23:28 | |
| *** gmann_afk is now known as gmann | 23:30 | |
| *** ijw has joined #openstack-nova | 23:30 | |
| *** ijw has quit IRC | 23:33 | |
| *** edleafe- has joined #openstack-nova | 23:33 | |
| *** edleafe has quit IRC | 23:35 | |
| *** chyka has joined #openstack-nova | 23:47 | |
| *** ijw has joined #openstack-nova | 23:52 | |
| *** ijw has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!