| *** inc0 has joined #openstack-nova | 00:01 | |
| *** burgerk has joined #openstack-nova | 00:02 | |
| *** otter768 has joined #openstack-nova | 00:03 | |
| *** burgerk has quit IRC | 00:06 | |
| *** otter768 has quit IRC | 00:07 | |
| *** vilobhmm11 has quit IRC | 00:09 | |
| *** takedakn has quit IRC | 00:09 | |
| *** Sukhdev has quit IRC | 00:10 | |
| *** Sukhdev has joined #openstack-nova | 00:11 | |
| *** sneti has joined #openstack-nova | 00:13 | |
| *** inc0 has quit IRC | 00:14 | |
| *** jerrygb has quit IRC | 00:15 | |
| *** takedakn has joined #openstack-nova | 00:15 | |
| *** ljxiash has joined #openstack-nova | 00:17 | |
| *** mylu_ has quit IRC | 00:21 | |
| *** ljxiash has quit IRC | 00:21 | |
| *** mylu has joined #openstack-nova | 00:22 | |
| *** ducttape_ has quit IRC | 00:25 | |
| *** dims has quit IRC | 00:25 | |
| *** dims has joined #openstack-nova | 00:26 | |
| *** mylu has quit IRC | 00:27 | |
| *** mylu has joined #openstack-nova | 00:28 | |
| *** mgoddard has quit IRC | 00:31 | |
| *** sneti has quit IRC | 00:31 | |
| *** suro-patz has quit IRC | 00:32 | |
| *** Marga_ has quit IRC | 00:32 | |
| openstackgerrit | Michael Bayer proposed openstack/nova: Remove SQLite BigInteger/Integer translation logic https://review.openstack.org/253859 | 00:33 |
|---|---|---|
| *** fmccrthy has quit IRC | 00:34 | |
| *** fmccrthy has joined #openstack-nova | 00:34 | |
| *** ducttape_ has joined #openstack-nova | 00:35 | |
| *** dims has quit IRC | 00:35 | |
| *** salv-orlando has joined #openstack-nova | 00:37 | |
| *** ducttape_ has quit IRC | 00:41 | |
| *** salv-orlando has quit IRC | 00:44 | |
| *** takedakn has quit IRC | 00:47 | |
| *** gjayavelu has joined #openstack-nova | 00:57 | |
| *** Sukhdev has quit IRC | 00:58 | |
| *** gjayavelu has quit IRC | 00:58 | |
| *** thorst has joined #openstack-nova | 01:00 | |
| *** shaohe_feng has joined #openstack-nova | 01:00 | |
| *** thorst has quit IRC | 01:03 | |
| *** EinstCrazy has joined #openstack-nova | 01:04 | |
| *** zhangjn has joined #openstack-nova | 01:08 | |
| *** ducttape_ has joined #openstack-nova | 01:08 | |
| *** shaohe_feng has quit IRC | 01:10 | |
| *** gjayavelu has joined #openstack-nova | 01:12 | |
| *** ducttape_ has quit IRC | 01:17 | |
| *** claudiub has joined #openstack-nova | 01:18 | |
| *** tjones has joined #openstack-nova | 01:22 | |
| *** ssurana has joined #openstack-nova | 01:24 | |
| *** gjayavelu has quit IRC | 01:25 | |
| *** ljxiash has joined #openstack-nova | 01:26 | |
| *** ljxiash_ has joined #openstack-nova | 01:29 | |
| *** haomaiwang has joined #openstack-nova | 01:32 | |
| *** ljxiash has quit IRC | 01:33 | |
| *** ircuser-1 has joined #openstack-nova | 01:33 | |
| *** ljxiash has joined #openstack-nova | 01:33 | |
| *** ssurana has quit IRC | 01:35 | |
| *** ljxiash_ has quit IRC | 01:36 | |
| *** ljxiash__ has joined #openstack-nova | 01:36 | |
| *** haomaiwang has quit IRC | 01:37 | |
| *** ljxiash_ has joined #openstack-nova | 01:38 | |
| *** ljxias___ has joined #openstack-nova | 01:38 | |
| *** ljxiash has quit IRC | 01:38 | |
| *** ljxiash__ has quit IRC | 01:41 | |
| *** ljxiash_ has quit IRC | 01:42 | |
| *** ssurana has joined #openstack-nova | 01:42 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces izip_longest with six.moves.zip_longest https://review.openstack.org/238321 | 01:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes dict keys and items references for Python 3 https://review.openstack.org/232585 | 01:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 compatibility for filter results https://review.openstack.org/232620 | 01:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 unit tests for nova.compute https://review.openstack.org/247742 | 01:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces longs with ints https://review.openstack.org/238239 | 01:42 |
| *** ljxias___ has quit IRC | 01:43 | |
| *** ljxiash has joined #openstack-nova | 01:44 | |
| *** otter768 has joined #openstack-nova | 01:45 | |
| *** tjones has quit IRC | 01:46 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Transforms requests and responses into bytes https://review.openstack.org/238323 | 01:46 |
| *** jianghuaw has joined #openstack-nova | 01:46 | |
| *** jianghua_ has joined #openstack-nova | 01:48 | |
| *** jianghuaw has quit IRC | 01:51 | |
| *** ducttape_ has joined #openstack-nova | 01:52 | |
| *** ducttape_ has joined #openstack-nova | 01:56 | |
| *** jianghua_ has quit IRC | 02:02 | |
| *** otter768 has quit IRC | 02:03 | |
| *** haomaiwang has joined #openstack-nova | 02:03 | |
| *** thorst has joined #openstack-nova | 02:03 | |
| *** XuZhixing has joined #openstack-nova | 02:04 | |
| *** liusheng has joined #openstack-nova | 02:04 | |
| *** baoli has joined #openstack-nova | 02:06 | |
| *** thorst has quit IRC | 02:08 | |
| *** haomaiwang has quit IRC | 02:09 | |
| *** ssurana has quit IRC | 02:09 | |
| *** zhangjn has quit IRC | 02:10 | |
| *** jyuso1 has joined #openstack-nova | 02:11 | |
| *** ducttape_ has quit IRC | 02:11 | |
| *** Sukhdev has joined #openstack-nova | 02:18 | |
| *** 7JTABCRUQ has joined #openstack-nova | 02:22 | |
| *** ducttape_ has joined #openstack-nova | 02:23 | |
| *** baoli has quit IRC | 02:24 | |
| *** baoli has joined #openstack-nova | 02:25 | |
| *** diegows has joined #openstack-nova | 02:27 | |
| *** gouthamr has joined #openstack-nova | 02:28 | |
| *** baoli has quit IRC | 02:31 | |
| *** ducttape_ has quit IRC | 02:32 | |
| *** cfriesen has joined #openstack-nova | 02:32 | |
| *** baoli has joined #openstack-nova | 02:35 | |
| *** mgoddard has joined #openstack-nova | 02:41 | |
| *** zenoway has joined #openstack-nova | 02:41 | |
| *** ljxiash has quit IRC | 02:42 | |
| *** ljxiash has joined #openstack-nova | 02:43 | |
| *** gcb has joined #openstack-nova | 02:45 | |
| *** links has joined #openstack-nova | 02:45 | |
| *** ljxiash has quit IRC | 02:45 | |
| *** zenoway has quit IRC | 02:46 | |
| *** ljxiash has joined #openstack-nova | 02:46 | |
| *** XuZhixing has quit IRC | 02:47 | |
| *** mylu has quit IRC | 02:47 | |
| *** mgoddard has quit IRC | 02:48 | |
| *** yangyapeng has joined #openstack-nova | 02:49 | |
| *** diegows has quit IRC | 02:50 | |
| *** baoli has quit IRC | 02:50 | |
| *** baoli has joined #openstack-nova | 02:51 | |
| *** mgoddard has joined #openstack-nova | 02:52 | |
| *** klkumar has joined #openstack-nova | 02:53 | |
| *** houming has joined #openstack-nova | 02:53 | |
| *** ducttape_ has joined #openstack-nova | 02:53 | |
| *** Jianlee_ has joined #openstack-nova | 02:55 | |
| *** 7JTABCRUQ has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** baoli has quit IRC | 03:01 | |
| *** zhangjn has joined #openstack-nova | 03:03 | |
| *** zhangjn has quit IRC | 03:03 | |
| *** zhangjn has joined #openstack-nova | 03:05 | |
| *** thorst has joined #openstack-nova | 03:06 | |
| *** shaohe_feng has joined #openstack-nova | 03:06 | |
| *** cfriesen has quit IRC | 03:09 | |
| *** thorst has quit IRC | 03:10 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 compatibility for filter results https://review.openstack.org/232620 | 03:12 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 unit tests for nova.compute https://review.openstack.org/247742 | 03:12 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes conductor py3 unit tests https://review.openstack.org/253989 | 03:12 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes network py unit tests https://review.openstack.org/253990 | 03:12 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes network py3 unit tests https://review.openstack.org/253990 | 03:13 |
| *** zhangjn has quit IRC | 03:15 | |
| *** zenoway has joined #openstack-nova | 03:17 | |
| *** baoli has joined #openstack-nova | 03:19 | |
| *** ducttape_ has quit IRC | 03:20 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Scheduler: correct control flow when forcing host https://review.openstack.org/251221 | 03:21 |
| *** zenoway has quit IRC | 03:22 | |
| *** mgoddard has quit IRC | 03:23 | |
| *** baoli has quit IRC | 03:27 | |
| *** inc0 has joined #openstack-nova | 03:28 | |
| *** gongysh has quit IRC | 03:32 | |
| *** gongysh has joined #openstack-nova | 03:33 | |
| *** gjayavelu has joined #openstack-nova | 03:37 | |
| *** otter768 has joined #openstack-nova | 03:38 | |
| *** gcb has quit IRC | 03:38 | |
| *** sneti has joined #openstack-nova | 03:40 | |
| *** ljxiash has quit IRC | 03:40 | |
| *** ljxiash has joined #openstack-nova | 03:41 | |
| *** gjayavelu has quit IRC | 03:41 | |
| *** gcb has joined #openstack-nova | 03:42 | |
| *** sneti has quit IRC | 03:45 | |
| *** ljxiash has quit IRC | 03:45 | |
| *** inc0 has quit IRC | 03:46 | |
| *** gcb has quit IRC | 03:48 | |
| *** shaohe_feng has quit IRC | 03:50 | |
| *** gcb has joined #openstack-nova | 03:51 | |
| *** houming_ has joined #openstack-nova | 03:55 | |
| *** ducttape_ has joined #openstack-nova | 03:55 | |
| *** houming has quit IRC | 03:56 | |
| *** houming_ is now known as houming | 03:56 | |
| *** gcb has quit IRC | 03:56 | |
| *** mylu has joined #openstack-nova | 03:56 | |
| *** jinxing has joined #openstack-nova | 03:58 | |
| *** ianbrown has joined #openstack-nova | 04:00 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-nova | 04:01 | |
| *** dims has joined #openstack-nova | 04:03 | |
| *** gcb has joined #openstack-nova | 04:09 | |
| *** jinxing has quit IRC | 04:10 | |
| *** ducttape_ has quit IRC | 04:10 | |
| *** vilobhmm11 has joined #openstack-nova | 04:12 | |
| *** ianbrown has quit IRC | 04:15 | |
| *** ianbrown_ has joined #openstack-nova | 04:15 | |
| *** vilobhmm111 has joined #openstack-nova | 04:20 | |
| *** vilobhmm11 has quit IRC | 04:23 | |
| openstackgerrit | Stanislaw Pitucha proposed openstack/python-novaclient: Fix multiline string with missing space https://review.openstack.org/254003 | 04:25 |
| *** mdrabe has joined #openstack-nova | 04:26 | |
| *** ianbrown_ is now known as ianbrown | 04:28 | |
| *** zhangjn has joined #openstack-nova | 04:32 | |
| *** ljxiash has joined #openstack-nova | 04:32 | |
| *** ducttape_ has joined #openstack-nova | 04:33 | |
| *** dims has quit IRC | 04:34 | |
| *** Jianlee_ has quit IRC | 04:38 | |
| *** mdrabe has quit IRC | 04:42 | |
| *** ianbrown has quit IRC | 04:42 | |
| *** ianbrown has joined #openstack-nova | 04:42 | |
| *** gouthamr has quit IRC | 04:45 | |
| *** vilobhmm111 has quit IRC | 04:50 | |
| *** vilobhmm11 has joined #openstack-nova | 04:50 | |
| *** vilobhmm11 has quit IRC | 04:55 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Scheduler: correct control flow when forcing host https://review.openstack.org/251221 | 04:59 |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** shaohe_feng has joined #openstack-nova | 05:04 | |
| *** shaohe_feng1 has joined #openstack-nova | 05:06 | |
| *** shaohe_feng has quit IRC | 05:09 | |
| *** ljxiash has quit IRC | 05:09 | |
| *** ljxiash has joined #openstack-nova | 05:09 | |
| *** ducttape_ has quit IRC | 05:10 | |
| *** jinxing has joined #openstack-nova | 05:14 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient: Fix H404/405 violations in novaclient/tests/* https://review.openstack.org/252363 | 05:18 |
| *** shaohe_feng has joined #openstack-nova | 05:23 | |
| *** ducttape_ has joined #openstack-nova | 05:23 | |
| *** shaohe_feng1 has quit IRC | 05:24 | |
| *** dims has joined #openstack-nova | 05:24 | |
| *** ducttape_ has quit IRC | 05:28 | |
| *** ianbrown has quit IRC | 05:29 | |
| *** ianbrown has joined #openstack-nova | 05:29 | |
| *** mylu has quit IRC | 05:32 | |
| *** zhangjn has quit IRC | 05:33 | |
| *** mylu has joined #openstack-nova | 05:34 | |
| *** deepakcs has joined #openstack-nova | 05:36 | |
| *** takashin has joined #openstack-nova | 05:36 | |
| *** wyang has quit IRC | 05:37 | |
| *** shaohe_feng1 has joined #openstack-nova | 05:39 | |
| *** shaohe_feng has quit IRC | 05:42 | |
| *** lixiaoy1 has quit IRC | 05:43 | |
| *** snraju has joined #openstack-nova | 05:45 | |
| *** dims has quit IRC | 05:46 | |
| *** zhangjn has joined #openstack-nova | 05:49 | |
| *** wyang has joined #openstack-nova | 05:53 | |
| *** amotoki_ has joined #openstack-nova | 05:54 | |
| *** nihilifer has quit IRC | 05:56 | |
| *** nihilifer has joined #openstack-nova | 05:57 | |
| *** LiuNanke has joined #openstack-nova | 05:59 | |
| *** mylu has quit IRC | 05:59 | |
| *** mylu has joined #openstack-nova | 05:59 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-nova | 06:01 | |
| *** zhangjn has quit IRC | 06:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/252197 | 06:05 |
| *** zhangjn has joined #openstack-nova | 06:07 | |
| *** irenab has quit IRC | 06:08 | |
| *** Marga_ has joined #openstack-nova | 06:09 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Fix VIF plug and unplug problem https://review.openstack.org/254022 | 06:13 |
| *** jyuso1 has quit IRC | 06:13 | |
| *** Sukhdev has quit IRC | 06:14 | |
| *** Sukhdev has joined #openstack-nova | 06:15 | |
| *** Chaozhe has joined #openstack-nova | 06:17 | |
| *** ducttape_ has joined #openstack-nova | 06:24 | |
| *** gtt116_ has quit IRC | 06:25 | |
| *** lpetrut has joined #openstack-nova | 06:26 | |
| *** ianbrown has quit IRC | 06:26 | |
| *** ianbrown has joined #openstack-nova | 06:26 | |
| *** amotoki_ is now known as amotoki | 06:30 | |
| *** ianbrown has quit IRC | 06:30 | |
| *** ducttape_ has quit IRC | 06:30 | |
| *** ianbrown has joined #openstack-nova | 06:30 | |
| *** ianbrown has quit IRC | 06:31 | |
| *** dims has joined #openstack-nova | 06:32 | |
| *** ianbrown has joined #openstack-nova | 06:32 | |
| *** boris-42_ has quit IRC | 06:33 | |
| *** jwcroppe has quit IRC | 06:34 | |
| *** paul-carlton_ has joined #openstack-nova | 06:35 | |
| *** esracelik has joined #openstack-nova | 06:35 | |
| *** gtt116 has joined #openstack-nova | 06:37 | |
| *** jaosorior has joined #openstack-nova | 06:39 | |
| *** lpetrut has quit IRC | 06:44 | |
| *** dims has quit IRC | 06:45 | |
| *** suro-patz has joined #openstack-nova | 06:47 | |
| *** irenab has joined #openstack-nova | 06:48 | |
| *** markus_z has joined #openstack-nova | 06:49 | |
| *** openstackgerrit_ has joined #openstack-nova | 06:51 | |
| *** dims has joined #openstack-nova | 06:52 | |
| *** mjura has joined #openstack-nova | 06:54 | |
| markus_z | bauzas: johnthetubaguy. JFYI I'm calling in sick for today (in case anybody is asking). I should be back tomorrow. | 06:54 |
| *** markus_z has quit IRC | 06:54 | |
| *** chinmaya has joined #openstack-nova | 06:57 | |
| *** vilobhmm11 has joined #openstack-nova | 06:57 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** josecastroleon has joined #openstack-nova | 07:02 | |
| *** mylu has quit IRC | 07:03 | |
| *** jyuso1 has joined #openstack-nova | 07:05 | |
| *** doron_afk has joined #openstack-nova | 07:07 | |
| *** thorst has joined #openstack-nova | 07:07 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds spec for instance live resize https://review.openstack.org/141219 | 07:11 |
| *** thorst has quit IRC | 07:12 | |
| *** nkrinner has joined #openstack-nova | 07:13 | |
| *** suro-patz has quit IRC | 07:13 | |
| *** suro-patz has joined #openstack-nova | 07:15 | |
| *** sahid has joined #openstack-nova | 07:16 | |
| openstackgerrit | NanZhang proposed openstack/nova: live-migration: Update instance availability_zone https://review.openstack.org/254041 | 07:18 |
| *** jaypipes has joined #openstack-nova | 07:18 | |
| *** rotbeard has joined #openstack-nova | 07:22 | |
| *** alexschm has joined #openstack-nova | 07:26 | |
| *** ducttape_ has joined #openstack-nova | 07:27 | |
| *** shaohe_feng1 has quit IRC | 07:27 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Fix VIF plug and unplug problem https://review.openstack.org/254022 | 07:30 |
| *** sudipto has joined #openstack-nova | 07:31 | |
| *** ducttape_ has quit IRC | 07:31 | |
| *** inc0 has joined #openstack-nova | 07:34 | |
| *** suro-patz has quit IRC | 07:36 | |
| *** vilobhmm11 has quit IRC | 07:37 | |
| *** garyk has joined #openstack-nova | 07:40 | |
| *** Sukhdev has quit IRC | 07:41 | |
| openstackgerrit | Alexander Schmidt proposed openstack/nova: libvirt: use native AIO mode for image backends https://review.openstack.org/247396 | 07:41 |
| *** scheuran has joined #openstack-nova | 07:42 | |
| *** shaohe_feng has joined #openstack-nova | 07:42 | |
| *** jyuso1 has quit IRC | 07:42 | |
| *** jyuso1 has joined #openstack-nova | 07:43 | |
| *** zenoway has joined #openstack-nova | 07:43 | |
| *** inc0 has quit IRC | 07:46 | |
| *** bkopilov has quit IRC | 07:47 | |
| *** Sukhdev has joined #openstack-nova | 07:51 | |
| *** klkumar has quit IRC | 07:54 | |
| *** shaohe_feng1 has joined #openstack-nova | 07:55 | |
| *** klkumar has joined #openstack-nova | 07:55 | |
| *** zhangjn has quit IRC | 07:55 | |
| *** ianbrown_ has joined #openstack-nova | 07:55 | |
| *** shaohe_feng has quit IRC | 07:57 | |
| openstackgerrit | Alexander Schmidt proposed openstack/nova: libvirt: use native AIO mode for cinder volumes https://review.openstack.org/251829 | 07:58 |
| *** ianbrown has quit IRC | 07:58 | |
| *** Sukhdev has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** mpavone has joined #openstack-nova | 08:03 | |
| *** haomaiwang has quit IRC | 08:05 | |
| *** liverpooler has joined #openstack-nova | 08:05 | |
| *** haomaiwa_ has joined #openstack-nova | 08:05 | |
| *** paul-carlton1 has joined #openstack-nova | 08:06 | |
| *** jichen has joined #openstack-nova | 08:09 | |
| *** ttx has joined #openstack-nova | 08:09 | |
| *** salv-orlando has joined #openstack-nova | 08:09 | |
| *** lixiaoy1 has joined #openstack-nova | 08:10 | |
| *** rotbeard has quit IRC | 08:11 | |
| *** salv-orlando has quit IRC | 08:12 | |
| *** Sukhdev has joined #openstack-nova | 08:13 | |
| *** paul-carlton1 has quit IRC | 08:14 | |
| *** shakamunyi has quit IRC | 08:14 | |
| *** Sukhdev has quit IRC | 08:15 | |
| *** zhangjn has joined #openstack-nova | 08:16 | |
| *** ianbrown_ has quit IRC | 08:19 | |
| gmann | Vek: hi | 08:20 |
| *** rotbeard has joined #openstack-nova | 08:20 | |
| *** oomichi has quit IRC | 08:22 | |
| *** liverpoo1er has joined #openstack-nova | 08:22 | |
| *** ianbrown_ has joined #openstack-nova | 08:23 | |
| *** Marga_ has quit IRC | 08:26 | |
| *** matrohon has joined #openstack-nova | 08:26 | |
| *** rdopiera has joined #openstack-nova | 08:27 | |
| *** ducttape_ has joined #openstack-nova | 08:27 | |
| *** ljxiash has quit IRC | 08:28 | |
| *** ljxiash has joined #openstack-nova | 08:28 | |
| *** paul-carlton1 has joined #openstack-nova | 08:28 | |
| *** shz has quit IRC | 08:29 | |
| *** shz has joined #openstack-nova | 08:30 | |
| *** e0ne has joined #openstack-nova | 08:31 | |
| *** klkumar has quit IRC | 08:31 | |
| *** ducttape_ has quit IRC | 08:32 | |
| *** ljxiash has quit IRC | 08:33 | |
| *** zhangjn has quit IRC | 08:33 | |
| *** klkumar has joined #openstack-nova | 08:34 | |
| alexschm | garyk: Hi, I actually looked for places where the code I modified is unit tested but did not find anything | 08:35 |
| alexschm | garyk: it seemed to me that this glue code for calling os-brick is trivial and does not have unit tests so far | 08:36 |
| openstackgerrit | ShaoHe Feng proposed openstack/nova-specs: Attach/detach SR-IOV interface https://review.openstack.org/139910 | 08:36 |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient: Fix H404/405 violations in novaclient/v2/[a-f] https://review.openstack.org/254070 | 08:37 |
| bauzas | good morning Nova | 08:38 |
| alexschm | garyk: do you know of any place with existing unit tests? | 08:39 |
| *** rubasov has joined #openstack-nova | 08:40 | |
| *** yamahata has joined #openstack-nova | 08:40 | |
| *** ljxiash has joined #openstack-nova | 08:40 | |
| *** ildikov has quit IRC | 08:42 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 compatibility for filter results https://review.openstack.org/232620 | 08:44 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 unit tests for nova.compute https://review.openstack.org/247742 | 08:44 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes conductor py3 unit tests https://review.openstack.org/253989 | 08:44 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes network py3 unit tests https://review.openstack.org/253990 | 08:45 |
| *** zhangjn has joined #openstack-nova | 08:46 | |
| *** zhangjn has quit IRC | 08:46 | |
| *** abhishekk has joined #openstack-nova | 08:47 | |
| *** zenoway has quit IRC | 08:47 | |
| *** mhorban_ has quit IRC | 08:49 | |
| *** zenoway has joined #openstack-nova | 08:49 | |
| *** ianbrown_ has quit IRC | 08:51 | |
| *** e0ne has quit IRC | 08:55 | |
| *** ianbrown has joined #openstack-nova | 08:56 | |
| *** mkoderer has quit IRC | 08:59 | |
| *** andreykurilin__ has joined #openstack-nova | 08:59 | |
| *** takashin has left #openstack-nova | 08:59 | |
| *** zhangjn has joined #openstack-nova | 09:00 | |
| *** mkoderer has joined #openstack-nova | 09:00 | |
| *** e0ne has joined #openstack-nova | 09:00 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** jsuchome has joined #openstack-nova | 09:01 | |
| *** moshele has joined #openstack-nova | 09:01 | |
| *** paul-carlton1 has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:02 | |
| *** ljxiash has quit IRC | 09:02 | |
| *** jichen has quit IRC | 09:02 | |
| *** ljxiash has joined #openstack-nova | 09:03 | |
| *** e0ne has quit IRC | 09:03 | |
| openstackgerrit | gongysh proposed openstack/nova: Fix the duplicate nics for VMs https://review.openstack.org/252824 | 09:03 |
| *** zhangjn has quit IRC | 09:04 | |
| *** gjayavelu has joined #openstack-nova | 09:04 | |
| *** jsuchome has left #openstack-nova | 09:05 | |
| *** digvijay has joined #openstack-nova | 09:06 | |
| *** gjayavelu has quit IRC | 09:09 | |
| *** zhangjn has joined #openstack-nova | 09:09 | |
| *** zhangjn has quit IRC | 09:10 | |
| *** lpetrut has joined #openstack-nova | 09:11 | |
| *** zhangjn has joined #openstack-nova | 09:13 | |
| *** yassine__ has joined #openstack-nova | 09:13 | |
| *** zhangjn has quit IRC | 09:13 | |
| *** mnestratov has joined #openstack-nova | 09:16 | |
| *** MarcF has joined #openstack-nova | 09:17 | |
| *** vikram__ has joined #openstack-nova | 09:18 | |
| *** danpb has joined #openstack-nova | 09:21 | |
| *** paul-carlton_ has quit IRC | 09:22 | |
| *** ankit_ag has joined #openstack-nova | 09:22 | |
| *** salv-orlando has joined #openstack-nova | 09:22 | |
| openstackgerrit | celik esra proposed openstack/nova: Config options: centralize "configdrive" options https://review.openstack.org/254092 | 09:23 |
| *** eglynn has joined #openstack-nova | 09:24 | |
| digvijay | internal error: Cannot find suitable CPU model for given data | 09:24 |
| digvijay | can any one help me with this issue | 09:24 |
| *** ducttape_ has joined #openstack-nova | 09:28 | |
| *** shaohe_feng1 has quit IRC | 09:28 | |
| *** shaohe_feng has joined #openstack-nova | 09:29 | |
| *** sayali has joined #openstack-nova | 09:32 | |
| *** oomichi has joined #openstack-nova | 09:32 | |
| *** zhangjn has joined #openstack-nova | 09:32 | |
| *** ducttape_ has quit IRC | 09:32 | |
| *** gtt116 has quit IRC | 09:34 | |
| *** jistr has joined #openstack-nova | 09:36 | |
| *** zhangjn has quit IRC | 09:40 | |
| *** ianbrown has quit IRC | 09:40 | |
| *** MarcF has quit IRC | 09:42 | |
| *** ianbrown has joined #openstack-nova | 09:45 | |
| *** houming has quit IRC | 09:47 | |
| *** zhangjn has joined #openstack-nova | 09:47 | |
| *** zhangjn has quit IRC | 09:47 | |
| *** dims has quit IRC | 09:48 | |
| *** houming has joined #openstack-nova | 09:49 | |
| *** ildikov has joined #openstack-nova | 09:49 | |
| *** Chaozhe has quit IRC | 09:51 | |
| *** LiuNanke has quit IRC | 09:53 | |
| *** e0ne has joined #openstack-nova | 09:56 | |
| *** rk4n has joined #openstack-nova | 09:59 | |
| johnthetubaguy | markus_z: get well soon! | 09:59 |
| *** ljxiash has quit IRC | 09:59 | |
| *** ljxiash has joined #openstack-nova | 10:00 | |
| johnthetubaguy | digvijay: I would try #openstack for that kind of help, please see the channel topic | 10:00 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** dguitarbite has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| johnthetubaguy | bauzas: how are we looking for python-novaclient reno now? | 10:02 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Calculate block_migration in virt drivers if it's None https://review.openstack.org/254101 | 10:03 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 10:03 |
| *** zhenguo has quit IRC | 10:03 | |
| *** ljxiash has quit IRC | 10:04 | |
| *** dguitarbite has joined #openstack-nova | 10:05 | |
| *** danpb has quit IRC | 10:11 | |
| *** ianbrown has quit IRC | 10:13 | |
| *** danpb has joined #openstack-nova | 10:13 | |
| *** ianbrown has joined #openstack-nova | 10:13 | |
| *** ptm_away is now known as PaulMurray | 10:14 | |
| PaulMurray | johnthetubaguy, guess what - I changed a single like in the libvirt driver and the xenapi tests failed | 10:15 |
| johnthetubaguy | PaulMurray: sure, that happens | 10:16 |
| PaulMurray | oh, that's ok then | 10:16 |
| johnthetubaguy | PaulMurray: which tests? | 10:16 |
| *** ianw has quit IRC | 10:17 | |
| PaulMurray | I think they are all in test_xenapi - 354 of them - it doesn't happend if I run them alone and they all get: TypeError: this TransactionFactory is already started | 10:19 |
| *** aix has joined #openstack-nova | 10:19 | |
| PaulMurray | I assume something gets messed up and they then all hit the same thing | 10:19 |
| *** ildikov has quit IRC | 10:19 | |
| johnthetubaguy | PaulMurray: thats odd | 10:20 |
| johnthetubaguy | it sounds a bit like an ordering thing, where you have some resource left over, funky | 10:20 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes dict keys and items references for Python 3 https://review.openstack.org/232585 | 10:20 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces izip_longest with six.moves.zip_longest https://review.openstack.org/238321 | 10:21 |
| johnthetubaguy | PaulMurray: do you have the change up for review? | 10:21 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 compatibility for filter results https://review.openstack.org/232620 | 10:21 |
| *** digvijay has quit IRC | 10:21 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 unit tests for nova.compute https://review.openstack.org/247742 | 10:22 |
| *** cburgess has quit IRC | 10:22 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes conductor py3 unit tests https://review.openstack.org/253989 | 10:22 |
| openstackgerrit | Alexander Schmidt proposed openstack/nova: libvirt: use native AIO mode for cinder volumes https://review.openstack.org/251829 | 10:22 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes network py3 unit tests https://review.openstack.org/253990 | 10:23 |
| bauzas | johnthetubaguy: all the changes are merged, including the CI job | 10:23 |
| *** openstackgerrit_ has quit IRC | 10:23 | |
| *** salv-orlando has quit IRC | 10:23 | |
| PaulMurray | johnthetubaguy, sorry - got distracted for a moment | 10:24 |
| johnthetubaguy | bauzas: cool, did we do the nova style check for any un-renoed changes? | 10:25 |
| PaulMurray | no, but I can put one up - I don't want to waste your time | 10:25 |
| bauzas | johnthetubaguy: sorry I don't understand - you mean, do we have some merged changes without reno notes? | 10:26 |
| *** LiuNanke has joined #openstack-nova | 10:27 | |
| *** cburgess has joined #openstack-nova | 10:27 | |
| johnthetubaguy | bauzas: possibly, but I guess probably not since the last release, now you mention it | 10:27 |
| johnthetubaguy | bauzas: I forgot we release that so recently | 10:27 |
| bauzas | johnthetubaguy: or do you want to have that http://docs.openstack.org/developer/nova/code-review.html#when-a-release-note-is-needed implemented as an hacking check ? | 10:27 |
| johnthetubaguy | PaulMurray: OK, seems odd it would impact the XenAPI tests, it sounds like coincidence of something you haven't mocked doing something odd, but I could be wrong | 10:28 |
| *** cdent has joined #openstack-nova | 10:28 | |
| johnthetubaguy | bauzas: it would be good to get that automatically checked somehow | 10:29 |
| johnthetubaguy | bauzas: I think they are doing something for DocImpact now, that is similar | 10:29 |
| *** ducttape_ has joined #openstack-nova | 10:29 | |
| *** jlanoux has joined #openstack-nova | 10:29 | |
| *** jistr has quit IRC | 10:29 | |
| *** moshele has quit IRC | 10:29 | |
| *** moshele has joined #openstack-nova | 10:30 | |
| *** jistr has joined #openstack-nova | 10:30 | |
| *** ociuhandu has quit IRC | 10:30 | |
| *** cburgess has quit IRC | 10:31 | |
| openstackgerrit | Paul Murray proposed openstack/nova: libvirt: Drop JSON for supported_instances https://review.openstack.org/254115 | 10:31 |
| *** cburgess has joined #openstack-nova | 10:32 | |
| bauzas | johnthetubaguy: mmm, I was rather thinking on just having a document for that, but that's maybe something I could provide as a CI job to be discussed | 10:32 |
| bauzas | that still leaves some human review :) | 10:32 |
| PaulMurray | johnthetubaguy, ^^^ you'll see its a very simple change and should not affect xen tests - but your right, probably hits a mock or a fixture | 10:32 |
| PaulMurray | johnthetubaguy, I will chase it down, just curious in case you had any ideas | 10:33 |
| johnthetubaguy | bauzas: yeah, just the presence of a reno file would be good to check, I guess | 10:33 |
| *** ducttape_ has quit IRC | 10:33 | |
| *** klkumar has quit IRC | 10:34 | |
| johnthetubaguy | PaulMurray: oh, funny, I wonder if something used to mock jsonutils, seems odd though | 10:35 |
| *** ildikov has joined #openstack-nova | 10:35 | |
| *** wyang has quit IRC | 10:37 | |
| *** ljxiash has joined #openstack-nova | 10:38 | |
| smoriya_ | johnthetubaguy: hi, can I ask you a question? | 10:39 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Move Process and Mentoring pages to devref https://review.openstack.org/206070 | 10:39 |
| *** jaypipes has quit IRC | 10:39 | |
| *** cdent has quit IRC | 10:39 | |
| *** ljxiash has quit IRC | 10:40 | |
| *** jaypipes has joined #openstack-nova | 10:40 | |
| smoriya_ | johnthetubaguy: I have a question about your comment on https://review.openstack.org/#/c/211101/ | 10:40 |
| *** ljxiash has joined #openstack-nova | 10:40 | |
| *** gridinv1 has quit IRC | 10:41 | |
| *** cburgess has quit IRC | 10:41 | |
| kashyap | Just ask it, he'll notice it when he's paying attention here. | 10:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds host_capabilities conf options group https://review.openstack.org/250166 | 10:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: driver: Adds get_hypervisor_capabilities method https://review.openstack.org/254124 | 10:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: Hyper-V: Adds get_hypervisor_capabilities implementation https://review.openstack.org/254125 | 10:42 |
| smoriya_ | kashyap: ok, thanks | 10:43 |
| *** cburgess has joined #openstack-nova | 10:43 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Remove the executable bit from several python files https://review.openstack.org/253109 | 10:43 |
| smoriya_ | johnthetubaguy: you mentioned that if its just adding support for the ironic spec, we don't need spec | 10:43 |
| *** johnthetubaguy has quit IRC | 10:43 | |
| *** Guest81777 is now known as BobBall | 10:44 | |
| *** cdent has joined #openstack-nova | 10:45 | |
| *** klkumar has joined #openstack-nova | 10:46 | |
| smoriya_ | johnthetubaguy: OTOH, you mentioned that we need spec for it in the comment on its implementation patch https://review.openstack.org/#/c/215385/ | 10:46 |
| *** gszasz has joined #openstack-nova | 10:47 | |
| *** haomaiwang has quit IRC | 10:47 | |
| smoriya_ | johnthetubaguy: what should we do?? do I need spec or not? | 10:48 |
| *** johnthetubaguy has joined #openstack-nova | 10:49 | |
| *** haomaiwang has joined #openstack-nova | 10:50 | |
| BobBall | I forget... is mock or mox the preference for new unit tests? | 10:51 |
| *** cdent has quit IRC | 10:51 | |
| *** snraju has quit IRC | 10:54 | |
| *** ociuhandu has joined #openstack-nova | 10:54 | |
| *** cdent has joined #openstack-nova | 10:56 | |
| *** cburgess has quit IRC | 10:56 | |
| deepakcs | BobBall: its mock | 10:57 |
| BobBall | Ok, good. Thanks :) | 10:57 |
| *** inc0 has joined #openstack-nova | 10:58 | |
| *** cburgess has joined #openstack-nova | 10:59 | |
| *** yassine__ has quit IRC | 11:00 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class https://review.openstack.org/184040 | 11:03 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 11:03 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console handler class https://review.openstack.org/184039 | 11:03 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console proxy https://review.openstack.org/184038 | 11:03 |
| *** tbh1 has joined #openstack-nova | 11:03 | |
| johnthetubaguy | BobBall: see point 4 in here: http://docs.openstack.org/infra/manual/developers.html#peer-review | 11:04 |
| BobBall | yup - it was for a new file - I just couldn't remember what the preference was | 11:04 |
| johnthetubaguy | it just gives a good explanation for why thats the preference, makes it easier for me to remember that way | 11:05 |
| *** matrohon has quit IRC | 11:07 | |
| *** fmccrthy has quit IRC | 11:07 | |
| *** fmccrthy has joined #openstack-nova | 11:07 | |
| *** thorst has joined #openstack-nova | 11:09 | |
| *** aysyd has joined #openstack-nova | 11:10 | |
| *** daemontool has joined #openstack-nova | 11:10 | |
| daemontool | johnthetubaguy: ping | 11:10 |
| *** rfolco has joined #openstack-nova | 11:11 | |
| *** inc0 has quit IRC | 11:11 | |
| *** thorst has quit IRC | 11:13 | |
| *** aysyd has quit IRC | 11:15 | |
| *** enthurohini has joined #openstack-nova | 11:15 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient: Fix H404/405 violations in novaclient/v2/[h-s] https://review.openstack.org/254134 | 11:15 |
| *** aysyd has joined #openstack-nova | 11:16 | |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: This is 2nd part of changes according to pci-generate-stats blueprint. https://review.openstack.org/243131 | 11:16 |
| *** yangyapeng has quit IRC | 11:17 | |
| *** EinstCrazy has quit IRC | 11:17 | |
| *** shaohe_feng has quit IRC | 11:21 | |
| *** johnthetubaguy has quit IRC | 11:21 | |
| *** johnthetubaguy has joined #openstack-nova | 11:21 | |
| johnthetubaguy | daemontool: hi | 11:22 |
| bauzas | johnthetubaguy: now that we're past M-1, could you please quickly see https://review.openstack.org/#/c/202675/ and reapply +W ? | 11:27 |
| bauzas | (it was a rebase due to a merge conflict) | 11:28 |
| johnthetubaguy | bauzas: sure, let me check that series for you | 11:29 |
| *** lucas-afk is now known as lucasagomes | 11:29 | |
| *** ducttape_ has joined #openstack-nova | 11:30 | |
| *** ducttape_ has quit IRC | 11:34 | |
| *** claudiub|2 has joined #openstack-nova | 11:37 | |
| *** claudiub has quit IRC | 11:38 | |
| *** Chaozhe has joined #openstack-nova | 11:38 | |
| bauzas | johnthetubaguy: I need to rework on the RPC API change we discussed, but I'd love to land the filters changes before | 11:40 |
| johnthetubaguy | bauzas: sure thing, good to get them out the way | 11:40 |
| bauzas | lxsli: are you okay if I'm helping you on the check-dests BP ? | 11:41 |
| *** liverpoo1er has quit IRC | 11:42 | |
| *** liverpooler has quit IRC | 11:42 | |
| *** liverpooler has joined #openstack-nova | 11:42 | |
| *** liverpoo1er has joined #openstack-nova | 11:42 | |
| *** EinstCrazy has joined #openstack-nova | 11:43 | |
| smoriya_ | johnthetubaguy: Do I need a spec for supporting ironic boot from volume? https://review.openstack.org/#/c/211101/ | 11:43 |
| *** liverpoo1er has quit IRC | 11:43 | |
| *** liverpooler has quit IRC | 11:44 | |
| smoriya_ | johnthetubaguy: you mentioned that if its just adding support for the ironic spec, we don't need spec. | 11:44 |
| lxsli | bauzas: absolutely! | 11:44 |
| *** liverpooler has joined #openstack-nova | 11:44 | |
| smoriya_ | johnthetubaguy: OTOH, you also mentioned that we need a spec for this change in the comment on its implementation patch https://review.openstack.org/#/c/215385/ | 11:44 |
| smoriya_ | johnthetubaguy: I'm little bit confused... what should I do? do I need a spec or not? | 11:45 |
| johnthetubaguy | smoriya_: we need a blueprint for it, but not a spec, assuming the details are basically covered in the ironic spec | 11:45 |
| johnthetubaguy | smoriya_: we need the ironic spec to merge, before we can approve the blueprint | 11:45 |
| *** smatzek has joined #openstack-nova | 11:46 | |
| smoriya_ | johnthetubaguy: OK, I got it | 11:46 |
| johnthetubaguy | smoriya_: for more details, please see: https://wiki.openstack.org/wiki/Nova/Process#How_do_I_get_my_code_merged.3F | 11:46 |
| *** deepakcs has quit IRC | 11:46 | |
| johnthetubaguy | smoriya_: the problem is we are frozen for new specs and blueprints last thursday | 11:46 |
| johnthetubaguy | smoriya_: but there is an exception process | 11:47 |
| smoriya_ | johnthetubaguy: yes, now I'm going to propose it as an exception | 11:47 |
| alexus | johnthetubaguy: sdague just add more todo for doc sprint https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/complete-todo-in-api-concept-doc+owner:self,n,z would you like take a look at when you have time, thanks! | 11:47 |
| smoriya_ | johnthetubaguy: the BP is already there https://blueprints.launchpad.net/nova/+spec/ironic-boot-from-volume | 11:47 |
| *** noqa_v_q1ovnie is now known as noqa_v_qoovnie | 11:48 | |
| smoriya_ | johnthetubaguy: so what we need is to get approval of ironic spec | 11:48 |
| *** macsz has joined #openstack-nova | 11:48 | |
| johnthetubaguy | smoriya_: yes, thats the key bit, get the ironic work approved | 11:48 |
| smoriya_ | johnthetubaguy: if we don't need spec, how do I propose it as exception? | 11:50 |
| johnthetubaguy | smoriya_: same way, just add a link to your blueprint in the etherpad | 11:50 |
| *** klkumar has quit IRC | 11:51 | |
| smoriya_ | johnthetubaguy: Thanks, I understand. | 11:51 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Introduce utility module for pcp https://review.openstack.org/203163 | 11:53 |
| *** klkumar has joined #openstack-nova | 11:53 | |
| lxsli | bauzas: I haven't been doing a lot with it, what are you thinking? | 11:54 |
| *** tbh1 has quit IRC | 11:57 | |
| *** alexus is now known as alex_xu | 11:57 | |
| *** chinmaya has quit IRC | 11:57 | |
| *** sudipto has quit IRC | 11:58 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Add FK bw_usage_cache.uuid -> instances.uuid https://review.openstack.org/248780 | 11:59 |
| *** garyk has quit IRC | 12:01 | |
| *** vikram__ has quit IRC | 12:01 | |
| openstackgerrit | javeme proposed openstack/python-novaclient: encode the url parameters https://review.openstack.org/254144 | 12:01 |
| *** garyk has joined #openstack-nova | 12:02 | |
| *** andreykurilin__ has quit IRC | 12:02 | |
| *** salv-orlando has joined #openstack-nova | 12:03 | |
| *** salv-orlando has quit IRC | 12:03 | |
| *** salv-orlando has joined #openstack-nova | 12:04 | |
| *** Marga_ has joined #openstack-nova | 12:05 | |
| *** dims has joined #openstack-nova | 12:08 | |
| *** jaypipes has quit IRC | 12:12 | |
| *** mgoddard has joined #openstack-nova | 12:12 | |
| *** raildo-afk is now known as raildo | 12:14 | |
| *** lpetrut has quit IRC | 12:15 | |
| *** links has quit IRC | 12:15 | |
| *** lpetrut has joined #openstack-nova | 12:17 | |
| johnthetubaguy | lxsli: bauzas: as I understand it, we have the request spec persistence almost merged, so that implementation should be totally unblocked now? | 12:17 |
| *** flaper87 has quit IRC | 12:17 | |
| *** flaper87 has joined #openstack-nova | 12:17 | |
| lxsli | johnthetubaguy: there may also be some entanglement with ndipanov's https://review.openstack.org/#/c/244489, bauzas will know better than me | 12:18 |
| *** doug-fish has joined #openstack-nova | 12:19 | |
| *** xiexs has quit IRC | 12:20 | |
| *** raju has joined #openstack-nova | 12:20 | |
| *** houming has quit IRC | 12:21 | |
| *** ljxiash has quit IRC | 12:21 | |
| *** ljxiash has joined #openstack-nova | 12:21 | |
| *** cdelatte has joined #openstack-nova | 12:22 | |
| johnthetubaguy | lxsli: can we get the blueprint updated to state that, if possible? | 12:24 |
| lxsli | johnthetubaguy: I'd like to hear from bauzas first | 12:24 |
| *** ljxiash has quit IRC | 12:26 | |
| *** ducttape_ has joined #openstack-nova | 12:30 | |
| *** Chaozhe has quit IRC | 12:32 | |
| *** ducttape_ has quit IRC | 12:35 | |
| *** jaosorior has quit IRC | 12:36 | |
| *** jaosorior has joined #openstack-nova | 12:37 | |
| *** eglynn has quit IRC | 12:37 | |
| ndipanov | johnthetubaguy, I am not sure there is overlap tbh | 12:38 |
| *** jaosorior has quit IRC | 12:38 | |
| *** jaosorior has joined #openstack-nova | 12:38 | |
| ndipanov | that work was never going to be addressed by check destination iiuc <- bauzas | 12:38 |
| *** baoli has joined #openstack-nova | 12:41 | |
| *** eglynn has joined #openstack-nova | 12:41 | |
| *** andymaier has joined #openstack-nova | 12:42 | |
| *** baoli_ has joined #openstack-nova | 12:43 | |
| *** baoli_ has quit IRC | 12:45 | |
| *** ducttape_ has joined #openstack-nova | 12:45 | |
| *** baoli_ has joined #openstack-nova | 12:46 | |
| *** andymaier has quit IRC | 12:46 | |
| *** baoli has quit IRC | 12:46 | |
| *** andymaier has joined #openstack-nova | 12:47 | |
| *** thorst has joined #openstack-nova | 12:47 | |
| *** otter768 has quit IRC | 12:48 | |
| *** mylu has joined #openstack-nova | 12:48 | |
| *** Daisy has joined #openstack-nova | 12:50 | |
| *** nikhil_k has quit IRC | 12:53 | |
| garyk | danpb: bauzas: there is a problem with the filter ImagePropertiesFilter. The image mata from the galnce image loses the hypervisor_type. This emans that a hybrid cloud will not work. | 12:54 |
| garyk | danpb: why did we drop the hypervisor_type? | 12:54 |
| *** mylu has quit IRC | 12:55 | |
| *** haomaiwang has quit IRC | 12:56 | |
| danpb | we didn't drop anything intentionally - it was just a side effect of us having no record of what code uses what | 12:56 |
| garyk | yeah, i understand that nothing was done intentionally | 12:56 |
| *** ZZelle has joined #openstack-nova | 12:57 | |
| *** gcb has quit IRC | 12:58 | |
| garyk | danpb: question is do we want to add hypervisor_type or piggy back on something else. | 12:59 |
| garyk | there is already img_hv_type | 12:59 |
| garyk | it looks like we should - problem is that we need to add a configuration change between hypervisor_type and img_hv_type and img_hv_requested_version hypervisor_version_requires | 13:01 |
| garyk | unless i misunderstand. | 13:01 |
| *** smoriya_ has quit IRC | 13:01 | |
| danpb | hmm, nothing is using img_hv_type, so that must have been the reason we added it | 13:01 |
| garyk | what is your 2 cents | 13:01 |
| danpb | we're just missing the legacy prop name mapping from hypervisor_type -> img_hv_type | 13:01 |
| *** ducttape_ has quit IRC | 13:01 | |
| *** nikhil has joined #openstack-nova | 13:01 | |
| garyk | ok, i'll look into it | 13:02 |
| *** abhishekk has quit IRC | 13:02 | |
| *** dustins has joined #openstack-nova | 13:02 | |
| openstackgerrit | javeme proposed openstack/python-novaclient: remove the default arguments "[]" https://review.openstack.org/254171 | 13:03 |
| *** rpodolyaka has quit IRC | 13:08 | |
| *** tlbr has quit IRC | 13:09 | |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: This is 2nd part of changes according to pci-generate-stats blueprint. https://review.openstack.org/243131 | 13:10 |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: Implementation of the first part of pci-generate-stats blueprint https://review.openstack.org/242087 | 13:10 |
| *** w_verdugo has joined #openstack-nova | 13:11 | |
| *** busterswt has joined #openstack-nova | 13:11 | |
| *** LiuNanke has quit IRC | 13:11 | |
| *** sudipto has joined #openstack-nova | 13:15 | |
| *** gouthamr has joined #openstack-nova | 13:17 | |
| *** gouthamr_ has joined #openstack-nova | 13:22 | |
| *** sneti has joined #openstack-nova | 13:22 | |
| *** gouthamr has quit IRC | 13:23 | |
| *** gouthamr_ has left #openstack-nova | 13:25 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE Ceph for live-migration job https://review.openstack.org/252283 | 13:29 |
| *** jaypipes has joined #openstack-nova | 13:30 | |
| *** matrohon has joined #openstack-nova | 13:36 | |
| *** haomaiwang has joined #openstack-nova | 13:36 | |
| openstackgerrit | Paul Murray proposed openstack/nova: Drop JSON for stats in virt API https://review.openstack.org/253628 | 13:36 |
| openstackgerrit | Paul Murray proposed openstack/nova: libvirt: Drop JSON for supported_instances https://review.openstack.org/254115 | 13:36 |
| openstackgerrit | Paul Murray proposed openstack/nova: hyperv: Drop JSON for supported_instances https://review.openstack.org/254193 | 13:36 |
| openstackgerrit | Paul Murray proposed openstack/nova: ironic: Drop JSON for supported_instances https://review.openstack.org/254194 | 13:36 |
| openstackgerrit | Paul Murray proposed openstack/nova: vmware: Drop JSON for supported_instances https://review.openstack.org/254195 | 13:36 |
| openstackgerrit | Paul Murray proposed openstack/nova: xen: Drop JSON for supported_instances https://review.openstack.org/254196 | 13:36 |
| *** ljxiash has joined #openstack-nova | 13:36 | |
| *** ljxiash has quit IRC | 13:37 | |
| *** nikhil has quit IRC | 13:37 | |
| *** cfriesen has joined #openstack-nova | 13:37 | |
| *** ljxiash has joined #openstack-nova | 13:37 | |
| *** raju has quit IRC | 13:38 | |
| *** ljxiash has quit IRC | 13:38 | |
| *** ljxiash has joined #openstack-nova | 13:38 | |
| garyk | danpb: ok, i have done a test and it fixes it. i will send a patch soon | 13:38 |
| *** ljxiash_ has joined #openstack-nova | 13:39 | |
| *** edmondsw has joined #openstack-nova | 13:39 | |
| *** klkumar has quit IRC | 13:40 | |
| *** klkumar has joined #openstack-nova | 13:40 | |
| *** claudiub|2 has quit IRC | 13:41 | |
| *** ljxiash has quit IRC | 13:43 | |
| *** LiuNanke has joined #openstack-nova | 13:43 | |
| *** sudipto has quit IRC | 13:45 | |
| *** LiuNanke has quit IRC | 13:45 | |
| bauzas | lxsli: ndipanov: johnthetubaguy: I need to review https://review.openstack.org/#/c/244489 before saying if it's okay | 13:46 |
| ndipanov | bauzas, please do | 13:48 |
| bauzas | ndipanov: meh, https://review.openstack.org/#/c/244489/6/nova/conductor/tasks/live_migrate.py,cm is close to http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/check-destination-on-migrations.html | 13:50 |
| openstackgerrit | garyk proposed openstack/nova: Scheduler: honor the glance metadata for hypervisor details https://review.openstack.org/254206 | 13:50 |
| bauzas | ndipanov: but it's not persisting the spec object | 13:50 |
| *** tlbr has joined #openstack-nova | 13:50 | |
| bauzas | ndipanov: just call again the scheduler, without providing against the request | 13:50 |
| garyk | danpb: bauzas: https://review.openstack.org/254206 | 13:50 |
| *** rpodolyaka has joined #openstack-nova | 13:50 | |
| *** vladikr has joined #openstack-nova | 13:50 | |
| ndipanov | bauzas, absolutely no idea what you're talking about tbh - can you comment on the patch? | 13:52 |
| *** boris-42_ has joined #openstack-nova | 13:52 | |
| pkholkin | sdague, alaski, jaypipes, mdbooth: Good day guys! Could you please look and share your opinion about this enginefacade patch https://review.openstack.org/#/c/243496/. It is an example of current 'use_slave' concept. We can't continue working on several patches because we haven't come to some common decision. Please look at comments by JohnG and RomanP, too. Thanks in advance! | 13:52 |
| bauzas | ndipanov: just did | 13:55 |
| *** ducttape_ has joined #openstack-nova | 13:55 | |
| *** jaypipes_ has joined #openstack-nova | 13:55 | |
| *** mgoddard has quit IRC | 13:56 | |
| ndipanov | bauzas, you do realize I didn't change any of that | 13:57 |
| *** jaypipes_ has quit IRC | 13:58 | |
| ndipanov | no idea what you ant me to do? rebase on top of persisting request spec work | 13:58 |
| *** nikhil has joined #openstack-nova | 13:58 | |
| *** jyuso1 has quit IRC | 14:00 | |
| bauzas | ndipanov: okay, removing the -1 to be less harsh | 14:00 |
| ndipanov | bauzas, how about being coherent? | 14:00 |
| bauzas | ndipanov: I'm just trying to explain that it was something we discussed in the past with the community | 14:00 |
| bauzas | ndipanov: and that's something we agreed to wait until we had the persisted spec object | 14:01 |
| bauzas | ndipanov: because of that | 14:01 |
| ndipanov | are you serious? | 14:01 |
| *** rlrossit has joined #openstack-nova | 14:01 | |
| bauzas | ndipanov: coherent about what? | 14:02 |
| ndipanov | bauzas, your comments make no sense to me | 14:02 |
| PaulMurray | danpb, I made a quick list of libvirt/qemu work related out live migration specs on https://etherpad.openstack.org/p/mitaka-live-migration | 14:02 |
| ndipanov | the patch in no way attempts to solve the problem you are pointing out | 14:03 |
| ndipanov | so either recommend a change to rebase on top of | 14:03 |
| ndipanov | of review it on it's own merit | 14:03 |
| ndipanov | this makes no sense - | 14:03 |
| PaulMurray | danpb, is there anything else you can think of right now that should be on there (its near the bottom of the page) | 14:03 |
| bauzas | ndipanov: I'm sorry if you feel that I wasn't serious | 14:03 |
| danpb | PaulMurray: guess that's all for M | 14:03 |
| bauzas | ndipanov: okay, it seems we have a big misunderstanding and I'm just trying to explain my position | 14:03 |
| PaulMurray | danpb, thanks | 14:03 |
| PaulMurray | danpb, yes | 14:04 |
| ndipanov | bauzas, I have no idea what your position is | 14:04 |
| ndipanov | all I see is you -1ed a patch and commented on 2 lines that were not even touched by the patch | 14:04 |
| *** jerrygb has joined #openstack-nova | 14:04 | |
| bauzas | ndipanov: okay, then it's me who didn't understand the problem | 14:05 |
| bauzas | ndipanov: consider my position as invalid because of a misunderstanding | 14:05 |
| ndipanov | bauzas, how about you go back when you have a moment and clarify what you meant? | 14:06 |
| openstackgerrit | Sean Dague proposed openstack/nova: Clean up glance url handling https://review.openstack.org/254213 | 14:07 |
| openstackgerrit | Sean Dague proposed openstack/nova: Implement GlanceEndpoint object https://review.openstack.org/254214 | 14:07 |
| bauzas | ndipanov: I'm doing that | 14:07 |
| bauzas | ndipanov: the RPC API change probably confused me | 14:07 |
| *** dustins has quit IRC | 14:07 | |
| *** sfinucan has joined #openstack-nova | 14:09 | |
| bauzas | ndipanov: tbh, I need to review the whole series | 14:11 |
| daemontool | johnthetubaguy: I work on the freezer project, and with the other team member we were wondering if would be the case to have a conversation with the Nova | 14:12 |
| daemontool | in case you are interested on converging anything related to backup/restore | 14:12 |
| *** Daisy has quit IRC | 14:12 | |
| daemontool | or if you have any idea on these regards | 14:12 |
| *** ijuwang has joined #openstack-nova | 14:14 | |
| johnthetubaguy | daemontool: hello, not come across the freezer project, just googling | 14:14 |
| *** ducttape_ has quit IRC | 14:14 | |
| daemontool | johnthetubaguy: http://governance.openstack.org/reference/projects/freezer.html | 14:15 |
| *** shz has quit IRC | 14:16 | |
| *** dims has quit IRC | 14:16 | |
| *** ducttape_ has joined #openstack-nova | 14:16 | |
| *** ctrath has joined #openstack-nova | 14:16 | |
| *** shz has joined #openstack-nova | 14:17 | |
| johnthetubaguy | daemontool: yep, yep, are there any particular questions I can help with? | 14:17 |
| *** hogepodge has quit IRC | 14:17 | |
| daemontool | johnthetubaguy: I'm saying this because fundamentally we do backup and restore, I see there are good features in nova for that and I was wondering if some common effort/feature can be achieved | 14:17 |
| *** yamahata has quit IRC | 14:17 | |
| *** alaski has quit IRC | 14:17 | |
| daemontool | like incrementals | 14:17 |
| daemontool | encryption | 14:17 |
| ndipanov | bauzas, would be awesome - though everything before that is minor cleanups | 14:18 |
| *** ducttape_ has quit IRC | 14:18 | |
| daemontool | or store backups on media storage different than swift (i.e. for disaster recovery purpose) | 14:18 |
| *** breitz has joined #openstack-nova | 14:18 | |
| johnthetubaguy | daemontool: so Nova just stores our backups of disks in glance, it deals with the rest | 14:18 |
| daemontool | johnthetubaguy: my main question is, would be better that nova consume these features from freezer? | 14:19 |
| daemontool | or we can contribute to Nova to leverage more freezer? | 14:19 |
| daemontool | I'm just thinking loud | 14:19 |
| daemontool | no proposition, just a constructive conversation :) | 14:19 |
| *** mdrabe has joined #openstack-nova | 14:19 | |
| johnthetubaguy | daemontool: understood, I am not sure, how do your backups happen, inside or outside the VMs? | 14:20 |
| daemontool | both | 14:20 |
| daemontool | but on this case | 14:21 |
| *** lucasagomes is now known as lucas-hungry | 14:21 | |
| daemontool | I'd focud on outside the vm, for nova | 14:21 |
| *** klkumar has quit IRC | 14:21 | |
| *** klkumar has joined #openstack-nova | 14:22 | |
| *** mdbooth_ has joined #openstack-nova | 14:22 | |
| *** pratikmallya has joined #openstack-nova | 14:22 | |
| daemontool | i.e. nova vm snapshot -> block based incremental backup done by freezer on the snapshot -> upload to swift or other media storage | 14:22 |
| daemontool | something like that | 14:22 |
| johnthetubaguy | daemontool: so incremental snapshots has been discussed in the past, although I think getting support for that is more a glance discussion, at first | 14:22 |
| mordred | sdague: commented on your GlanceEndpoint patch but didn't vote because it's honestly just questions I have | 14:22 |
| sdague | mordred: sure | 14:22 |
| *** irenab has quit IRC | 14:22 | |
| johnthetubaguy | daemontool: Nova does not really upload to swift, currently, we upload to glance, (sometimes thats via ceph, for example) | 14:22 |
| sdague | mordred: mostly this is follow on from conversations with notmorgan on Friday, because this is a giant pile of spaghetti | 14:22 |
| daemontool | johnthetubaguy: ok, I'm wondering, why sould I pass through glance, to execute vm baskups and store them to a media storage? | 14:23 |
| daemontool | johnthetubaguy: yes, I think the approach is good | 14:23 |
| sdague | and I think we can throw out 50% of the code if we walk it back to passing around urls as a step 1 | 14:23 |
| daemontool | I'm just trying to figure out if there's anything we can do to improve anything | 14:23 |
| sdague | instead of that crazy tupple | 14:23 |
| *** mdbooth has quit IRC | 14:23 | |
| *** bauzas has quit IRC | 14:23 | |
| *** matrohon has quit IRC | 14:23 | |
| *** ttx has quit IRC | 14:23 | |
| *** yassine__ has joined #openstack-nova | 14:23 | |
| *** bauzas has joined #openstack-nova | 14:23 | |
| *** mdbooth_ is now known as mdbooth | 14:23 | |
| johnthetubaguy | daemontool: thats a long story, but glance is what manages the concept of our snapshots and backups, so making them incremental means adding the concept of incremental images into glance first | 14:23 |
| *** ttx has joined #openstack-nova | 14:24 | |
| *** matrohon has joined #openstack-nova | 14:24 | |
| johnthetubaguy | daemontool: on the volume side, we have incremental snapshots already, and some versions of that support encryption, so thats a bit closer, but none of that works with local storage | 14:24 |
| sdague | mordred: yeh, so in staring at all this code I think that transitioning it all at once is going to be an unreviewable patch | 14:24 |
| alex_xu | bauzas: phschwartz ndipanov PaulMurray danpb, I would like invited you guys join our api doc sprint if you have time :) it's two days sprint from tomorrow. you can get info from here http://lists.openstack.org/pipermail/openstack-dev/2015-November/079220.html, thanks in advance! | 14:24 |
| mordred | sdague: ++ | 14:24 |
| phschwartz | alex_xu: I will see if I can fit it into my schedule | 14:25 |
| daemontool | johnthetubaguy: ok, if it's ok for you, we can get back to you and the Team with a proposal, and you can provide your feedback, if that make sense to you or not, etc | 14:25 |
| alex_xu | phschwartz: thanks a lot :) | 14:25 |
| mordred | sdague: I think I just want to keep saying the phrase "we don't want to be passing urls around either" - but +100 to smaller chunks to make the transition sane | 14:25 |
| *** liusheng has quit IRC | 14:25 | |
| *** liusheng has joined #openstack-nova | 14:26 | |
| * alex_xu want to invited all the peoples | 14:26 | |
| sdague | yeh, though at least if we are passing around urls, we can get do https://review.openstack.org/#/c/253662/3/nova/image/glance.py,cm | 14:26 |
| sdague | _glance_endpoint_from_keystone | 14:26 |
| mordred | sdague: so... | 14:26 |
| mordred | yea | 14:26 |
| mordred | yes | 14:26 |
| *** artom has quit IRC | 14:27 | |
| sdague | the thing that is weird here, is we've got this adhoc load balancer in nova right now with the fact that api_servers is a list | 14:27 |
| sdague | which... is also in cinder | 14:27 |
| openstackgerrit | garyk proposed openstack/nova: Scheduler: honor the glance metadata for hypervisor details https://review.openstack.org/254206 | 14:27 |
| sdague | and is a thing that we're going to have to consider before we fully pull that out from under people | 14:27 |
| garyk | danpb: bauzas: ^ | 14:27 |
| johnthetubaguy | daemontool: OK, that works | 14:27 |
| mordred | yah - whereas it should really just be pulling the internal URL from the catalog and if you want an LB you need to put one there | 14:27 |
| bauzas | cdent: so I was saying during my network blip that we miss some functional testing for the filters | 14:27 |
| mordred | sdague: sure | 14:27 |
| *** BadCub has quit IRC | 14:27 | |
| daemontool | johnthetubaguy: ty | 14:28 |
| mordred | sdague: many things to consider for sure | 14:28 |
| johnthetubaguy | daemontool: but we have just pass the blueprint freeze for Nova for Mitaka, so this would be more talking about what to do in the N release I think | 14:28 |
| bauzas | cdent: the idea would be to figure out some pattern for doing functional tests in a common namespace | 14:28 |
| notmorgan | sdague: mordred and i am working through the endpoing patch because i've found a case that doesn't seem to have a context | 14:28 |
| notmorgan | among other things where it is broken. | 14:28 |
| bauzas | cdent: that would require to understand the placement logic etc. | 14:28 |
| notmorgan | but expected ways broken | 14:28 |
| daemontool | johnthetubaguy: yes, that's not an issue, better late then never :) | 14:28 |
| sdague | notmorgan: yeh, this should honestly be orthoginal to that | 14:28 |
| sdague | mostly make that patch easier | 14:28 |
| notmorgan | sdague: somewhat | 14:29 |
| cdent | bauzas: did you say you have a POC somewhere? | 14:29 |
| notmorgan | sdague: i don't know if this even registers as moving the needing on the glance thing. ;) | 14:29 |
| mordred | sdague: I _think_ I just realized from your patch that the problem space in that file overlaps pretty nicely with code we've already got solving it in OCC - I'm going to do a quick POC patch to see if leveraging that would make it better or worse (I honestly don't know) | 14:29 |
| *** sudipto has joined #openstack-nova | 14:30 | |
| sdague | mordred: sure | 14:30 |
| bauzas | cdent: it hasn't been uploaded yet | 14:30 |
| mordred | sdague: (I want a glanceclient object, and I may have a catalog or I may have specific override values please go get me one) | 14:30 |
| notmorgan | it's def. a pile of spaghetti code. | 14:31 |
| *** BadCub has joined #openstack-nova | 14:31 | |
| sdague | I'm going to admit the patch that notmorgan took over demonstrates how little I really understand keystone session, but I did figure if I walked back passing (host, port, use_ssl) everywhere, all the code would be better anyway | 14:31 |
| sdague | because that's part of the hitch with the glance xen plugins | 14:31 |
| *** yamahata has joined #openstack-nova | 14:32 | |
| *** superdan is now known as dansmith | 14:32 | |
| mordred | sdague: I think that's a great stepping stone - I think passing those three values is a thing that wants to eventually stop though | 14:32 |
| *** yasu has joined #openstack-nova | 14:32 | |
| mordred | because it's totally re-implementing guts of the session/catalog interactions - but encapsulating it now will make replacing it MUCH easier | 14:33 |
| *** inc0 has joined #openstack-nova | 14:33 | |
| sdague | mordred: it also prevents the suburl catalog entries | 14:33 |
| mordred | yup | 14:33 |
| sdague | because glance passes those 3 params over the wire to dom0 to do image download | 14:33 |
| sdague | today | 14:33 |
| mordred | wait, what? | 14:33 |
| notmorgan | yeah | 14:33 |
| sdague | in xen | 14:33 |
| cdent | bauzas: I'll do some digging in the existing tests and see what I can learn. At the moment I'm more than a little bit ignorant... | 14:34 |
| sdague | let me get the code reference | 14:34 |
| mordred | sdague: I'm really not going to be happy the more I dig in here am ? | 14:34 |
| phschwartz | mordred: probably now ;) | 14:34 |
| sdague | https://github.com/openstack/nova/blob/98fab44827502d34f4ad261120643fb80eed4bf1/plugins/xenserver/xenapi/etc/xapi.d/plugins/glance#L355 | 14:34 |
| notmorgan | mordred: no you wont | 14:34 |
| * cdent doesn't remember mordred ever being happy | 14:34 | |
| * BobBall quietly walks away from the conversation before mordred becomes unhappy | 14:34 | |
| *** dims has joined #openstack-nova | 14:34 | |
| sdague | so, remember that xenserver has a dom0 with python2.4 in it | 14:34 |
| bauzas | cdent: sure, take your time, just giving you the idea | 14:34 |
| BobBall | (for the record, I'm not here :P ) | 14:35 |
| johnthetubaguy | sdague: I should be able to help with tidying that up, but yeah, its certainly needed | 14:35 |
| bauzas | cdent: for example, I'd love to see the idea: | 14:35 |
| bauzas | cdent: say that you setup 2 fake nodes | 14:35 |
| bauzas | cdent: for a specific test | 14:35 |
| sdague | so there is supper basic python code in dom0 to do some work, and a messaging mechanism from the domU compute to it | 14:35 |
| notmorgan | sdague: ... really?! | 14:35 |
| sdague | notmorgan: oh yeh | 14:35 |
| notmorgan | hah | 14:35 |
| bauzas | cdent: with 2 distinct resources, | 14:35 |
| bauzas | cdent: then we could verify the ComputeFilter by making sure it selects the less heavy node | 14:36 |
| mordred | sdague: ok. that's cool - I thnk we can work with that | 14:36 |
| PaulMurray | dansmith, I've got a question about a change I want to make to compute_node object | 14:36 |
| dansmith | PaulMurray: okay | 14:36 |
| mordred | sdague: that's actually not NEARLY as bad as I was expecting :) | 14:36 |
| sdague | https://github.com/openstack/nova/blob/98fab44827502d34f4ad261120643fb80eed4bf1/plugins/xenserver/xenapi/README#L9 | 14:36 |
| PaulMurray | dansmith, if you look at https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L335-#L340 | 14:36 |
| sdague | mordred: right, that's why I wanted to get this intermediary bit which is passing urls | 14:36 |
| mordred | sdague: ++ | 14:36 |
| mordred | sdague, BobBall: that code runs in dom0 - does that mean it's running on 2.4? | 14:37 |
| BobBall | Yes :( | 14:37 |
| sdague | because we'll have to pass a url over that wire connection, we can't put keystoneclient down there | 14:37 |
| johnthetubaguy | yeah, you can't just download in the nova-compute node for XenAPI because it has no access to to any local storage | 14:37 |
| PaulMurray | I've done patches to stop virt drivers json encoding supported_instances | 14:37 |
| sdague | mordred: yep | 14:37 |
| mordred | ok | 14:37 |
| notmorgan | sdague: i think we can just rework to use the URL, since we are passing via urllib2 | 14:37 |
| PaulMurray | dansmith, is it ok to stop that method dealing with decoding them? | 14:37 |
| BobBall | For now. Next version of XenServer will make the massive leap to 2.7. Just in time for OpenStack to move to 3.0 only I bet :) | 14:37 |
| notmorgan | we just need to not "construct" the uri in the plugin | 14:37 |
| notmorgan | but pass the uri to the plugin | 14:37 |
| PaulMurray | by object change rules | 14:37 |
| notmorgan | should be straight forward(ish) as any of the rest of this cleanup | 14:38 |
| mordred | cool. I think I grok the steps to sanity here | 14:38 |
| mordred | which is great | 14:38 |
| *** wverdugo500 has joined #openstack-nova | 14:38 | |
| sdague | notmorgan: right, there are just a lot of places these tuples show up | 14:38 |
| sdague | which is why I started down this path - https://review.openstack.org/#/c/254214/ | 14:38 |
| *** yasu has quit IRC | 14:39 | |
| notmorgan | sdague: yah. | 14:39 |
| sdague | johnthetubaguy: does RAX use the multiple value api_servers today? | 14:39 |
| *** mpavone has quit IRC | 14:39 | |
| BobBall | Passing a single whole URI to the plugin will be nasty since it currently takes a few arguments... Might need a new method name to preserve some vague saneness rather than removing args? | 14:39 |
| johnthetubaguy | sdague: yes, we have probably about 5 or 6 URLs in every region | 14:39 |
| cdent | bauzas: make sense, gotta run off now but will do some digging and find you when I inevitably have a "wat" | 14:39 |
| mordred | BobBall: ++ new method name | 14:39 |
| *** smatzek has quit IRC | 14:39 | |
| notmorgan | BobBall: yeah will be new method | 14:39 |
| dansmith | PaulMurray: that's not remotable, so assuming it's not called by a remotable method, and the virt driver change keeps the same behavior from a remote perspective, then yes, you can change whatever you want | 14:39 |
| *** sneti has quit IRC | 14:40 | |
| notmorgan | sdague: i think we just deprecate the multi-api server opt and endcode the "select" logic in nova for the length that the opt lingers. | 14:40 |
| notmorgan | sdague: since the catalog is ultimately the better choice. | 14:40 |
| BobBall | So we could keep the old method construct the URI and call the new method - make it so the URI support is optional? | 14:40 |
| notmorgan | cc johnthetubaguy ^ | 14:40 |
| *** macsz has quit IRC | 14:40 | |
| *** chenli has joined #openstack-nova | 14:40 | |
| johnthetubaguy | sdague: having said that, we may have just stopped using that, for something slightly more confusing | 14:40 |
| PaulMurray | dansmith, ok - thanks - I'll get on with it then. | 14:40 |
| mordred | johnthetubaguy: how difficult/disruptive would it be to ask you to put those behind a load balancer and put that lb entry in the internalURL in keystone catalog? | 14:40 |
| BobBall | or do you really want to use URI only? | 14:40 |
| chenli | anyone can help review https://review.openstack.org/#/c/252237/ | 14:40 |
| johnthetubaguy | mordred: in summary, massively | 14:40 |
| notmorgan | BobBall: URI only | 14:40 |
| mordred | johnthetubaguy: awesome | 14:40 |
| sdague | notmorgan: so, the thing is, I think this is used more than we expected | 14:40 |
| *** macsz has joined #openstack-nova | 14:41 | |
| mordred | it's ok - I think we can still deal with that | 14:41 |
| sdague | which means this opens the can of worms of needing the service catalog to specify a set of urls per entry | 14:41 |
| mordred | not relaly | 14:41 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: api: add soft-affinity policies for server groups https://review.openstack.org/147520 | 14:41 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: scheduler: fill RequestSpec.instance_group.members https://review.openstack.org/148277 | 14:41 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers https://review.openstack.org/147516 | 14:41 |
| mordred | all of the catalog consuming code currently | 14:41 |
| mordred | understands the idea of an endpoint_override | 14:41 |
| notmorgan | sdague: we can do an override. | 14:41 |
| johnthetubaguy | mordred: the original approach is because we don't want to pay for the loadbalancer big enough, I think a few bits are trying a loadbalancer that sends back some 30x redirect to the appropriate server | 14:41 |
| notmorgan | sdague: ^ what mordred said | 14:41 |
| mordred | and if we make endpoint_override grok lists | 14:41 |
| mordred | we can encode the current behavior in a way that's still repeatble and non-special | 14:42 |
| sdague | hmmmm... which means that the glance catalog gets fully ignored again | 14:42 |
| notmorgan | BobBall: the construction bits will need to linger for compat i'm guessing? but really we don't want to ever "construct" the path at the plugin layer, we should already have that | 14:42 |
| mordred | sdague: kind of - but it means that you're doing a catalog url override, which is a thing | 14:42 |
| mordred | sdague: and is a normal part of all of the tooling out there | 14:43 |
| xek | dansmith, Hi, I added you to the keystone schema migration spec reviewers https://review.openstack.org/#/c/245186/ , this is just about adding the unit test for blocking incompatible schema changes | 14:43 |
| sdague | but it seems a little weird if this is a first class feature people want | 14:43 |
| xek | dansmith, we can discuss it on the keystone channel | 14:43 |
| sdague | not to make it first class in the catalog, no? | 14:43 |
| mordred | sdague: so the code flow in nova is still "hey, I wnat to talk to glance based on this cloud connection configuration" | 14:43 |
| *** burt has joined #openstack-nova | 14:43 | |
| BobBall | I would like to, ideally, yes notmorgan - in an upgrade case I'd like people to be able to use the new plugins independently of the new nova (and document a deprecation of the construction method); but if that's not possible, we can make a break | 14:43 |
| *** akshai has joined #openstack-nova | 14:43 | |
| *** wyang has joined #openstack-nova | 14:43 | |
| dansmith | xek: okay, I'm just now getting online and sifting through the pile of email, but I'll look in a bit | 14:44 |
| notmorgan | sdague: i think it is a little odd, but i don't think it's unreasonable... heck... maybe we get to deprecate this functionality down the line when deployments are all doing "hey this catalog thing is better" | 14:44 |
| *** raginbajin has quit IRC | 14:44 | |
| *** fawadkhaliq has joined #openstack-nova | 14:44 | |
| notmorgan | it's a small concession to get us where we need to be and we're still only doing URI passing vs construction | 14:44 |
| notmorgan | which is a huge win | 14:44 |
| *** cdent has quit IRC | 14:45 | |
| notmorgan | BobBall: it should be doable except that new nova will need the new xenplugin | 14:45 |
| BobBall | Absolutely, yes | 14:45 |
| sdague | yeh, that doesn't seem like a huge deal | 14:45 |
| BobBall | The plugin versioning we have will support that | 14:45 |
| notmorgan | the old nova could work with the new plugin | 14:45 |
| notmorgan | BobBall: yeah that is fine. i already know what needs to be done for that bit | 14:46 |
| BobBall | awesome | 14:46 |
| *** raginbajin has joined #openstack-nova | 14:46 | |
| notmorgan | BobBall: it's just changing the method name passed down to the plugin | 14:46 |
| notmorgan | BobBall: i already was mucking around in there for my first pass attempt at this. | 14:46 |
| sdague | notmorgan / mordred you folks have an etherpad with a list of the things you worked out here that we need to do to get to endgame? | 14:46 |
| BobBall | yup | 14:46 |
| notmorgan | sdague: i have a massive etherpad of all the sub-url things that are broken atm | 14:47 |
| notmorgan | this is just one item on the list. | 14:47 |
| notmorgan | so no not a specific etherpad for this task | 14:47 |
| garyk | edleafe: thanks for the review. what you suggested is better. i will update | 14:47 |
| sdague | notmorgan: ok, if we break this up into all the details we should be able to move through them quickly. | 14:47 |
| edleafe | garyk: cools | 14:47 |
| notmorgan | yeah. this one should be a real etherpad thing | 14:47 |
| notmorgan | not just a footnote in my otherone | 14:47 |
| bauzas | ndipanov: may I ask you a question ? | 14:48 |
| *** daemontool has left #openstack-nova | 14:48 | |
| *** otter768 has joined #openstack-nova | 14:48 | |
| bauzas | ndipanov: currently reviewing https://review.openstack.org/#/c/244485/2/nova/conductor/manager.py,cm | 14:48 |
| sdague | I'm going to keep whittling on this lower level code to get over to urls | 14:49 |
| *** jinxing has quit IRC | 14:49 | |
| *** inc0 has quit IRC | 14:49 | |
| bauzas | ndipanov: AFAIK, the Migration object is created in the Compute API for the evacuate, but live-migrate does it in the conductor | 14:49 |
| bauzas | ndipanov: do you know why such discrepancy ? | 14:50 |
| *** gcb has joined #openstack-nova | 14:50 | |
| notmorgan | sdague: want me to toss up the etherpad? | 14:50 |
| bauzas | ndipanov: and if you want to have the same consistency between resize/evacuate and live-migrate, wouldn't it be better to create the object in the API too ? | 14:50 |
| openstackgerrit | garyk proposed openstack/nova: Scheduler: honor the glance metadata for hypervisor details https://review.openstack.org/254206 | 14:51 |
| sdague | notmorgan: that would be great | 14:51 |
| notmorgan | sdague, mordred: https://etherpad.openstack.org/p/nova-to-glance-by-uri | 14:51 |
| openstackgerrit | Paul Murray proposed openstack/nova: Drop JSON decoding for supported_instances https://review.openstack.org/254231 | 14:52 |
| *** andrewbogott has quit IRC | 14:52 | |
| *** otter768 has quit IRC | 14:53 | |
| *** alaski has joined #openstack-nova | 14:55 | |
| *** mriedem has joined #openstack-nova | 14:55 | |
| *** lykinsbd has joined #openstack-nova | 14:56 | |
| ndipanov | bauzas, it would probably yes | 14:56 |
| bauzas | ndipanov: just thinking of the races | 14:57 |
| *** mwagner_lap has quit IRC | 14:57 | |
| bauzas | ndipanov: and what happens if we have some message queue blip | 14:57 |
| *** ducttape_ has joined #openstack-nova | 14:57 | |
| ndipanov | bauzas, what do you mean | 14:58 |
| *** nelsnelson has joined #openstack-nova | 14:58 | |
| bauzas | ndipanov: it's a conductor call, right? so AFAIK it's async | 14:58 |
| *** ccarmack has joined #openstack-nova | 14:58 | |
| ndipanov | so | 14:58 |
| bauzas | ndipanov: which means that we can have some corner cases when a call to migrate is done, without the persisted state, nope? | 14:59 |
| ndipanov | what do you mean by 'a call to migrate' exactly? | 14:59 |
| bauzas | here an user calls the API for live-migrate an instance | 15:00 |
| ndipanov | yes | 15:00 |
| johnthetubaguy | sdague: I think we have an issue with one of the live-migrate specs that got approved, this seems to be going the opposite direction to what we agreed with the API" http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/live-migration-progress-report.html | 15:01 |
| *** whenry has joined #openstack-nova | 15:01 | |
| johnthetubaguy | sdague: or am I miss-reading that? | 15:01 |
| sdague | johnthetubaguy: sorry, I've got all this other context in my head right now, can you narrow down the question | 15:01 |
| bauzas | ndipanov: so https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3255-L3257 can be wedged if the message queue has some problems, right? | 15:01 |
| *** dims has quit IRC | 15:02 | |
| johnthetubaguy | sdague: sure, we can leave this for the API meeting, if thats better, basically looks like its extending the old os-migrations API in an odd way | 15:02 |
| sdague | right, I do think we agreed not to do that | 15:02 |
| *** mc_nair has joined #openstack-nova | 15:03 | |
| ndipanov | bauzas, ok I don't know what you mean by 'wedged' here - can you use a different word? | 15:03 |
| bauzas | ndipanov: trampled? the message forgotten ? | 15:03 |
| johnthetubaguy | sdague: OK, yes, not just me thinking that then, spec got approved without me noticing | 15:03 |
| *** jinxing has joined #openstack-nova | 15:04 | |
| sdague | yeh, I thought all the details were going to be on the per server resource | 15:04 |
| ndipanov | bauzas, ok so the message is lost which is entirely possible in the way we use rabbit | 15:04 |
| ndipanov | bauzas, what's the problem with that? | 15:04 |
| johnthetubaguy | sdague: agreed | 15:04 |
| openstackgerrit | esracelik proposed openstack/nova: Config options: centralize "configdrive" options https://review.openstack.org/254092 | 15:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 15:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 15:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 15:05 |
| *** irenab has joined #openstack-nova | 15:05 | |
| bauzas | ndipanov: the problem is that we create an action and we set the state to ACTIVE | 15:05 |
| bauzas | ndipanov: I'd love to see the Migration object created at the same place than the two above | 15:05 |
| *** thangp has joined #openstack-nova | 15:06 | |
| *** mrkz has joined #openstack-nova | 15:07 | |
| *** esracelik has quit IRC | 15:07 | |
| *** smatzek has joined #openstack-nova | 15:09 | |
| *** rlrossit has quit IRC | 15:11 | |
| *** liusheng has quit IRC | 15:11 | |
| *** liusheng has joined #openstack-nova | 15:11 | |
| ndipanov | bauzas, I mean ok but not sure what we get by that exactly | 15:12 |
| ndipanov | or | 15:12 |
| ndipanov | I get that it would be more consistent | 15:13 |
| ndipanov | but it can be done in a separate patch if we decide we like it | 15:13 |
| ndipanov | we've only added the migration object in the live migration path last cycle | 15:14 |
| *** hogepodge has joined #openstack-nova | 15:14 | |
| ndipanov | and the point was to have better tracking of the internal stages of the migration task | 15:14 |
| ndipanov | so not sure moving this to the API would be a massive win over what we have now | 15:14 |
| ndipanov | bauzas, ^ | 15:15 |
| *** dustins has joined #openstack-nova | 15:15 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova-specs: ConsoleAuthTokens should work for all console types https://review.openstack.org/228778 | 15:16 |
| *** artom has joined #openstack-nova | 15:17 | |
| *** artom_ has joined #openstack-nova | 15:17 | |
| notmorgan | sdague: should i continue on your patch for the catalog-i-fication of glanc ethings or are you poking at it and i'd collide with your work? | 15:17 |
| *** artom_ has quit IRC | 15:17 | |
| *** artom has quit IRC | 15:17 | |
| notmorgan | sdague: don't want to end up with competing patchsets. | 15:17 |
| sdague | notmorgan: let me look at where you are with that patch | 15:18 |
| *** artom has joined #openstack-nova | 15:18 | |
| notmorgan | sdague: haven't touched it since you posted it, because i've been on otherthings | 15:18 |
| *** cdent has joined #openstack-nova | 15:18 | |
| notmorgan | like configuring a cloud / chasing down KSA things | 15:18 |
| sdague | ah, oh, just rebased? | 15:18 |
| notmorgan | sdague: yeah | 15:18 |
| sdague | did anyone figure out the keystonauth vs. requirements issue in those patches? | 15:18 |
| pkholkin | sdague, alaski, jaypipes, mdbooth, mriedem: Good day guys! Could you please look and share your opinion about this enginefacade patch https://review.openstack.org/#/c/243496/. It is an example of current 'use_slave' concept. We can't continue working on several patches because we haven't come to some common decision. Please look at comments by JohnG and RomanP, too. Thanks in advance! | 15:19 |
| notmorgan | sdague: the neutron deprecated opts? | 15:19 |
| *** tonytan4ever has joined #openstack-nova | 15:19 | |
| notmorgan | sdague: that is because the options mordred was using is ksa only for a couple things. | 15:19 |
| alaski | pkholkin: I'm mostly in meetings this morning, but I'll take a look later | 15:19 |
| pkholkin | alaski: ok, no problem, thanks! | 15:19 |
| sfinucan | bauzas: What's the process for moving a conf option that hasn't got a prefix? | 15:20 |
| notmorgan | sdague: where the keystoneclient ones are older/not maintained. so we need to either change out how we do that test or move to keystoneauth first | 15:20 |
| notmorgan | both on the table actually. | 15:20 |
| notmorgan | also there is an open question to mriedem on the patch re the test coverage concern | 15:20 |
| notmorgan | was waiting to get feedback on that before proceeding | 15:20 |
| mordred | yah. need the mriedem question answered first | 15:20 |
| mriedem | final solution for the mriedem? | 15:21 |
| *** mjura has quit IRC | 15:21 | |
| sdague | notmorgan: http://logs.openstack.org/62/253662/3/check/gate-tempest-dsvm-full/37fbe07/logs/devstacklog.txt.gz#_2015-12-06_00_58_21_390 | 15:21 |
| sdague | that issue | 15:21 |
| notmorgan | huh wait what? | 15:21 |
| notmorgan | that... shouldn't be an issue | 15:21 |
| mordred | that's from yesterday | 15:22 |
| sdague | yeh, well that patch is 100% fail and that's what's tripping the dsvm jobs | 15:22 |
| mordred | temporal something | 15:22 |
| notmorgan | ksa 2.1.0 is out now | 15:22 |
| sfinucan | bauzas: I presume this means it's currently included in the "default" group. If we're moving it to the 'nova.conf' module, should we just create a 'default' group there? | 15:22 |
| sdague | ok | 15:22 |
| mordred | mriedem: yes. we need a final solution for you | 15:22 |
| mordred | mriedem: SO | 15:22 |
| sdague | well, it would be nice to at least refresh the tests there | 15:22 |
| *** rlrossit has joined #openstack-nova | 15:23 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 15:23 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 15:23 |
| sdague | notmorgan: so, I think right now, unless you are massively chomping at the bit, I'd hold on updating that patch I hacked up | 15:23 |
| notmorgan | sdague: sure. let me just recheck it for the moment | 15:23 |
| mordred | mriedem: with those deprecated options | 15:23 |
| pkholkin | sdague: Sean, please take a look, too https://review.openstack.org/#/c/243496/ | 15:23 |
| mordred | mriedem: if we remove them as they're marked that we can | 15:23 |
| sdague | because getting a patch which is going to pass unit tests, is pretty involved, and part of my patch stack should make that a ton easier | 15:23 |
| notmorgan | sdague: i can work on some other things. but i've completed the other tasks i needed to do before staring at this stuff | 15:23 |
| *** delattec has joined #openstack-nova | 15:23 | |
| mordred | mriedem: there will no-longer be any mechanism in the config file to specify an override admin token | 15:24 |
| mordred | mriedem: so I thnk the right thing to do is just to remove the refernece I put in that patch to the admin_token plugin | 15:24 |
| mordred | since that really won't work anyway | 15:25 |
| bauzas | ndipanov: sure we can move that in another change | 15:25 |
| mordred | mriedem: but I wanted to sync with you on deprecation/removal intent | 15:25 |
| notmorgan | sdague: i'm mostly just doing the go around and find what i can help push through to keep momentum | 15:25 |
| mriedem | pkholkin: i don't know what's going on in that change, the commit message needs a lot more detail | 15:25 |
| notmorgan | sdague: so i can continue on that and working w/ mordred to land the neutron/ksa changover first | 15:25 |
| pkholkin | mriedem: thanks for review! the question is more general, to leave 'use_slave' or not and how to do it better | 15:25 |
| sdague | notmorgan: yeh, I think that might make sense | 15:26 |
| pkholkin | mriedem: we have several discussions with johnthetubaguy and he decides to wait to some other opinions | 15:26 |
| *** cdelatte has quit IRC | 15:26 | |
| ndipanov | bauzas keep in mind that for resize/cold migrate - migration record is created on the compute host when doing the claim | 15:26 |
| * mriedem is getting hammered with irc and internal pings after 3 days off | 15:26 | |
| mriedem | mordred: let me look at the change again | 15:27 |
| mordred | mriedem: mine is not urgent | 15:27 |
| mordred | mriedem: actually, let me know when you're not being hammered and can talk in depth-ish | 15:27 |
| *** mtanino has joined #openstack-nova | 15:27 | |
| sdague | notmorgan: is it possible to get a utility function to do the catalog uri to endpoint uri without copy / pasting all that session code as part of that? | 15:27 |
| *** PaulMurray has quit IRC | 15:27 | |
| notmorgan | sdague: that is a KSA thing | 15:28 |
| mordred | sdague: yeah - it'll be much easier once you have access to a ksa | 15:28 |
| notmorgan | sdague: moving to keystoneauth is really part of the deal that will help | 15:28 |
| edleafe | sfinucan: add a new .py file in nova/conf/ for that group. In the code, just register it as part of DEFAULT. | 15:28 |
| edleafe | sfinucan: see: https://review.openstack.org/#/c/245891 for an example | 15:28 |
| mordred | _way_ simpler | 15:28 |
| *** lykinsbd has quit IRC | 15:29 | |
| sdague | ok, cool, I'll accept that as an answer, let me know when there is a patch up to review | 15:29 |
| * notmorgan nods. | 15:29 | |
| sfinucan | edleafe: Ah - OK. Every similar review I'd seen so far had the 'groups' parameter specified. | 15:30 |
| sfinucan | edleafe: Thanks | 15:30 |
| openstackgerrit | Monty Taylor proposed openstack/nova: Remove deprecated neutron auth options https://review.openstack.org/253792 | 15:31 |
| openstackgerrit | Monty Taylor proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 15:31 |
| mordred | notmorgan, mriedem, sdague: change for my current understanding of the world ^^ will likely need at least 2 tests fixed, but notmorgan is looking at that | 15:31 |
| *** jichen has joined #openstack-nova | 15:32 | |
| *** mylu has joined #openstack-nova | 15:32 | |
| mriedem | mordred: replied to comments in ps2 | 15:32 |
| *** whenry has quit IRC | 15:32 | |
| mordred | cool | 15:32 |
| mriedem | mordred: if there are questions about the deprecation intent in kilo/liberty, we should rope in jamielennox and edmondw probably | 15:32 |
| *** sayali has quit IRC | 15:32 | |
| mriedem | my fear is we're dropping support for something that wasn't clearly logged/doc'ed with a deprecation cycle | 15:33 |
| edleafe | sfinucan: yeah, it's kind of a disorganized mess. Eventually we'd like to split DEFAULT, which is like 90% of the options, into more logically-related groups. | 15:33 |
| *** yonglihe has joined #openstack-nova | 15:33 | |
| *** kfarr has joined #openstack-nova | 15:33 | |
| sfinucan | edleafe: Yeah, I saw (reviewed) something like that for the vnc options. Maybe a future work item for the option I'm working on | 15:34 |
| *** sayali has joined #openstack-nova | 15:34 | |
| *** burgerk has joined #openstack-nova | 15:36 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Modify VM's updated_at field on volume actions https://review.openstack.org/247176 | 15:36 |
| mriedem | how much tar and feather would i get if i said i think this keystoneauth transition should be tracked with a bp? | 15:37 |
| *** nkrinner has quit IRC | 15:37 | |
| johnthetubaguy | mriedem: it does feel like we should have something for that | 15:37 |
| mriedem | b/c this is an upgrade impact right? | 15:37 |
| mriedem | if we're not tracking with a bp or release notes i feel like we're sneaking something in here that people are going to hit the hard way | 15:38 |
| johnthetubaguy | well, reno can cover that, just seems like an important thing to track, and help raise the profile of the reviews | 15:38 |
| *** haomaiwang has quit IRC | 15:39 | |
| notmorgan | mriedem: so the "admin_token" bit really was just a way to let the tests mock out | 15:39 |
| sdague | mordred / notmorgan - on https://review.openstack.org/#/c/253793/5 there is one inconsistency would would be good to fix when the unit tests get fixed | 15:39 |
| mordred | mriedem: in a current devstack, are we creating a proper admin user for nova to use to talk to neutron and putting that in the nova section of the conf file? | 15:39 |
| *** liverpooler has quit IRC | 15:39 | |
| sdague | mriedem: I think a blueprint is fine | 15:39 |
| notmorgan | mriedem: not really useful outside of unit (no devstack) tests | 15:39 |
| mriedem | mordred: i'd have to look | 15:40 |
| mriedem | or someone would have to look | 15:40 |
| *** smoriya_ has joined #openstack-nova | 15:40 | |
| mordred | ok. I can look | 15:40 |
| notmorgan | mriedem: sorry to clarify my comment was not regarding the removal of the test, but the other comments in that patch | 15:40 |
| mriedem | http://logs.openstack.org/96/243496/7/check/gate-tempest-dsvm-neutron-full/e3522de/logs/etc/nova/nova.conf.txt.gz | 15:40 |
| pkholkin | mdbooth: thanks a lot for your comments! the patch will be updated soon | 15:40 |
| *** haomaiwang has joined #openstack-nova | 15:40 | |
| mriedem | mordred: i don't see devstack setting anything in the [neutron] section in nova.conf http://logs.openstack.org/96/243496/7/check/gate-tempest-dsvm-neutron-full/e3522de/logs/etc/nova/nova.conf.txt.gz | 15:41 |
| *** jaypipes has quit IRC | 15:41 | |
| *** Sukhdev has joined #openstack-nova | 15:41 | |
| mordred | k. in the keystone_authtoken section, there is a nova user defined | 15:42 |
| *** changbl has quit IRC | 15:42 | |
| mordred | is that the uesr we're expecting to be nova's admin user in the cloud? | 15:42 |
| *** mdrabe has quit IRC | 15:42 | |
| *** Sukhdev has quit IRC | 15:43 | |
| mriedem | don't know, i guess i'd need to bring up a devstack to see what roles that user has | 15:43 |
| mordred | well | 15:43 |
| mordred | don't do that ... I currently don't see anything that actually consumes the keystone_authtoken section for crednetials | 15:43 |
| mordred | so we're a few steps back :) | 15:43 |
| *** mylu has quit IRC | 15:43 | |
| sdague | notmorgan / mordred - right that's the thing that gets a little masked when you have libraries have functions that register options per section | 15:44 |
| *** Sukhdev has joined #openstack-nova | 15:44 | |
| sdague | is understanding what the actual ops impact is | 15:44 |
| *** xyang1 has joined #openstack-nova | 15:44 | |
| *** ljxiash_ has quit IRC | 15:45 | |
| sdague | mordred: yes, username under keystone_authtoken is the nova service user for the cloud | 15:45 |
| *** chenli has quit IRC | 15:45 | |
| mordred | ok | 15:45 |
| *** ljxiash has joined #openstack-nova | 15:45 | |
| mordred | so currently we do not test anything related to the deprecated options in this removal patch | 15:45 |
| mordred | lemme try something | 15:46 |
| *** sneti has joined #openstack-nova | 15:46 | |
| *** changbl has joined #openstack-nova | 15:46 | |
| *** sudipto has quit IRC | 15:47 | |
| *** tjones has joined #openstack-nova | 15:47 | |
| mordred | sdague: any chance you can point me to what consumes the keystone_authtoken section? I'm not smart enough to find it | 15:47 |
| mriedem | mordred: well the _load_auth_plugin method would have been using the default auth_strategy and admin_auth_url for constructing the v2 password plugin | 15:47 |
| mriedem | i can't tell what the default auth plugin is...is that v2 password? | 15:48 |
| mordred | it's 'password' | 15:49 |
| mordred | which can figure out v2 or v3 as needed | 15:49 |
| mordred | based on parameters given | 15:49 |
| mriedem | https://github.com/openstack/keystoneauth/blob/master/setup.cfg#L38 | 15:49 |
| mriedem | ok | 15:49 |
| *** ljxiash has quit IRC | 15:49 | |
| mordred | mriedem: so, it seems that what people should alreayd be doing (and what devstack is doing) is specifying their admin connection credentials in that keystone_authtoken section | 15:50 |
| mordred | mriedem: rather than as [neutron]admin_username | 15:50 |
| *** moshele has quit IRC | 15:51 | |
| mordred | and that seems to be what the devstack config intends | 15:51 |
| sdague | mordred: I believe keystone_authtoken is directly consumed by keystoneclient | 15:51 |
| ansiwen | anyone can tell me how to disable SSL verification in nova, especially for the connection to the vsphere server? | 15:51 |
| mordred | the biggest weirdness now is that the functional tests are mocking out that code path effectively, so we may need to poke more to make sure that's all working | 15:51 |
| *** fawadkhaliq has quit IRC | 15:51 | |
| mriedem | are there configs in keystoneauth that nova needs to register, or are those all in keystonemiddleware? | 15:52 |
| mriedem | https://github.com/openstack/nova/blob/master/etc/nova/nova-config-generator.conf#L23 | 15:52 |
| *** rushiagr_away is now known as rushiagr | 15:52 | |
| mordred | sdague: so, how does nova get the creds to talk to neutron? | 15:53 |
| *** changbl has quit IRC | 15:53 | |
| *** lykinsbd has joined #openstack-nova | 15:53 | |
| mordred | sdague: because those are just consumed by keystoneclient, not by the neutornclient code paths | 15:53 |
| *** rushiagr is now known as rushiagr_away | 15:54 | |
| openstackgerrit | Monty Taylor proposed openstack/nova: Remove deprecated neutron auth options https://review.openstack.org/253792 | 15:55 |
| openstackgerrit | Monty Taylor proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 15:55 |
| notmorgan | mriedem: in cases where nova needs credentials it'll register the options, but that should be in no new cases. it'll be a couple lines of code change | 15:55 |
| sdague | mordred: because neutronclient inherits via keystoneclient? | 15:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L222 | 15:56 |
| mordred | mriedem, notmorgan, sdague: ^^ this is redone to _ONLY_ remove the deprecated options and their reference - we can track down with that what's going on neutron connections | 15:56 |
| mriedem | i'm assuming the session passed to create neutronclient has the creds from nova.conf | 15:56 |
| sdague | mriedem: yeh, that looks right | 15:56 |
| notmorgan | mriedem: right. i'm trying to figure out how that session is... oh wait is this just passing the user's token directly? | 15:56 |
| mordred | right - except that's looking for tem in NEUTRONGROUP | 15:57 |
| mriedem | right | 15:57 |
| mriedem | so idk wtf is going on | 15:57 |
| mriedem | would be nice to have jamie involved since he worked on all of this | 15:57 |
| notmorgan | so what does the admin context do? | 15:57 |
| mriedem | we also pass the auth_plugin to the neutronclient | 15:57 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L251 | 15:57 |
| notmorgan | that i think is the root of the confusion here, what is the admin context for neutron (that nova constructs) do? | 15:57 |
| mriedem | let's you list ports for all tenants, for example | 15:58 |
| notmorgan | mriedem: because it looks like the devstack neutron gate doesn't actually hit those code paths | 15:58 |
| mriedem | neutronclient assumes that if you pass admin context, all_tenants=1 | 15:58 |
| mriedem | although neutronclient doesn't have a concept of all_tenants | 15:58 |
| notmorgan | since the [neutron] section is not populated | 15:58 |
| notmorgan | in nova.conf | 15:58 |
| *** mdrabe has joined #openstack-nova | 15:58 | |
| mriedem | it has to | 15:58 |
| mriedem | allocate_for_instance creates an admin neutronclient | 15:59 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L405 | 15:59 |
| mriedem | we have to have admin context (client) to bind ports | 15:59 |
| mriedem | and unbind ports | 15:59 |
| mriedem | and we test attach/detach vif in tempest | 15:59 |
| notmorgan | possibly directly though neutron api | 15:59 |
| notmorgan | rather than proxied via nova? | 15:59 |
| sdague | hmmm... no it does - http://logs.openstack.org/44/252944/1/check/gate-tempest-dsvm-neutron-full/5b531cb/logs/etc/nova/nova.conf.txt.gz | 16:00 |
| notmorgan | sdague: oh the other one i was looking at was wrong | 16:00 |
| notmorgan | ok phew, i am less confused now | 16:00 |
| * mriedem heads to a meeting | 16:00 | |
| *** mriedem is now known as mriedem_meeting | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| notmorgan | ok - got it now. | 16:01 |
| sdague | well, I'm confused why it was wrong | 16:01 |
| notmorgan | sdague: old gate run? broken gate run? | 16:01 |
| notmorgan | this one looks much more sane/like i would expect | 16:02 |
| sdague | notmorgan: oh, it failed so early it never got set | 16:02 |
| notmorgan | yah | 16:02 |
| *** whenry has joined #openstack-nova | 16:02 | |
| notmorgan | ok phew | 16:02 |
| notmorgan | this looks sane | 16:02 |
| mordred | oh. good | 16:02 |
| notmorgan | alllllright. | 16:02 |
| mordred | that makes total sense | 16:02 |
| sdague | yeh, never trust anything that's a test failure for having valid config | 16:02 |
| notmorgan | now we know the [neutron] section is where we should be pulling from | 16:02 |
| sdague | the one you specified failed here - http://logs.openstack.org/96/243496/7/check/gate-tempest-dsvm-neutron-full/e3522de/logs/devstacklog.txt.gz#_2015-11-30_15_52_22_848 | 16:02 |
| mordred | yes. which is what the code communicates | 16:02 |
| notmorgan | yay i feel less confused. | 16:03 |
| mordred | yup | 16:03 |
| mordred | so - the main issue we're seeing, really | 16:03 |
| mordred | in this patch | 16:03 |
| mordred | is that the functional tests do not actually test using that config | 16:03 |
| mordred | but instead do *weirdthings* | 16:03 |
| mordred | that I'm pretty sure can luckily just go away and the test should work and do things correctly | 16:03 |
| *** ljxiash has joined #openstack-nova | 16:04 | |
| notmorgan | yeah they mock out the keystone client things and make some broad assumptions | 16:04 |
| mordred | yup | 16:04 |
| mordred | and the neat things is | 16:04 |
| mordred | we can STOP mocking that out | 16:04 |
| mordred | and it should work | 16:04 |
| notmorgan | yeah. | 16:04 |
| mordred | \o/ | 16:04 |
| mordred | thanks for bearing with us here sdague and mriedem_meeting | 16:04 |
| *** changbl has joined #openstack-nova | 16:06 | |
| *** mwagner_lap has joined #openstack-nova | 16:06 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 16:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add infra for versioned notifications https://review.openstack.org/247024 | 16:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 16:06 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Make emitting versioned notifications configurable https://review.openstack.org/247564 | 16:06 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Fix representation of VirtualInterface resource https://review.openstack.org/254272 | 16:08 |
| *** smurke has joined #openstack-nova | 16:11 | |
| *** xyang1 has quit IRC | 16:13 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 16:13 |
| notmorgan | mordred: ^ addressing sdague's comment on ks_loading | 16:13 |
| *** cdelatte has joined #openstack-nova | 16:14 | |
| *** delattec has quit IRC | 16:15 | |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: This is 3rd part of changes according to pci-generate-stats https://review.openstack.org/247692 | 16:15 |
| *** jaypipes has joined #openstack-nova | 16:15 | |
| *** devananda has quit IRC | 16:18 | |
| cdent | bauzas, jaypipes: If I were to go through the source of http://docs.openstack.org/developer/nova/filter_scheduler.html and fix up grammar and typos is that something the land of nova would like a bug to track it? I've been reading it and my previous identity as an editor is getting itchy. | 16:18 |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: This is 4th patch of changes according to pci-generate-stats blueprint. https://review.openstack.org/250757 | 16:18 |
| *** dustins_ has joined #openstack-nova | 16:18 | |
| bauzas | cdent: sorry otp atm | 16:18 |
| *** dustins has quit IRC | 16:19 | |
| *** zenoway has quit IRC | 16:20 | |
| *** zenoway has joined #openstack-nova | 16:20 | |
| *** devananda has joined #openstack-nova | 16:21 | |
| *** dustins_ is now known as dustins | 16:23 | |
| *** fawadkhaliq has joined #openstack-nova | 16:24 | |
| *** Daisy has joined #openstack-nova | 16:24 | |
| *** dustins has quit IRC | 16:25 | |
| *** zenoway has quit IRC | 16:25 | |
| jaypipes | cdent: no, those kinds of things just fire away patches. | 16:26 |
| *** EinstCrazy has quit IRC | 16:26 | |
| *** alexschm has quit IRC | 16:26 | |
| cdent | cool, thanks jaypipes | 16:26 |
| jaypipes | cdent: if you want to file a bug to track major problems with the devref, of course that is helpful too | 16:26 |
| cdent | if I find, will do | 16:27 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Add Hyper-V storage QoS support https://review.openstack.org/175479 | 16:27 |
| *** dustins has joined #openstack-nova | 16:27 | |
| *** aix has quit IRC | 16:28 | |
| *** Daisy has quit IRC | 16:29 | |
| *** PaulMurray has joined #openstack-nova | 16:30 | |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| sdague | notmorgan: ok, I think I finally got the next patch in this glanceendpoint stuff to pass our unit tests... sometimes mock just punches you in the face | 16:30 |
| *** mriedem_meeting is now known as mriedem | 16:30 | |
| notmorgan | sdague: yah, mock is like that.. just mean sometimes :( | 16:30 |
| *** armax has quit IRC | 16:31 | |
| *** kevinbenton has quit IRC | 16:31 | |
| *** kevinbenton has joined #openstack-nova | 16:31 | |
| *** dustins has quit IRC | 16:31 | |
| *** dustins has joined #openstack-nova | 16:32 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class https://review.openstack.org/184040 | 16:34 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 16:34 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console handler class https://review.openstack.org/184039 | 16:34 |
| *** doron_afk has quit IRC | 16:34 | |
| *** r-daneel has joined #openstack-nova | 16:34 | |
| *** neiljerram_bb has joined #openstack-nova | 16:34 | |
| *** alex_xu has quit IRC | 16:35 | |
| *** Yingxin has quit IRC | 16:38 | |
| *** Yingxin has joined #openstack-nova | 16:38 | |
| *** Jeffrey4l has joined #openstack-nova | 16:38 | |
| *** gzhai2 has quit IRC | 16:39 | |
| *** jinxing has quit IRC | 16:39 | |
| *** eliqiao has quit IRC | 16:39 | |
| *** eliqiao has joined #openstack-nova | 16:39 | |
| *** Jeffrey4l has quit IRC | 16:39 | |
| *** neiljerram_bb has quit IRC | 16:39 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [functional-tests] fix version of novaclient instance https://review.openstack.org/254294 | 16:39 |
| *** Jeffrey4l has joined #openstack-nova | 16:41 | |
| *** mylu has joined #openstack-nova | 16:41 | |
| *** Jeffrey4l has quit IRC | 16:42 | |
| *** mylu has quit IRC | 16:42 | |
| *** Jeffrey4l has joined #openstack-nova | 16:42 | |
| *** jogo has quit IRC | 16:43 | |
| *** alex_xu has joined #openstack-nova | 16:43 | |
| *** whenry has quit IRC | 16:44 | |
| *** gzhai2 has joined #openstack-nova | 16:45 | |
| *** paul-carlton2 has quit IRC | 16:45 | |
| *** jogo has joined #openstack-nova | 16:45 | |
| *** FL1SK has quit IRC | 16:46 | |
| *** apoorvad has joined #openstack-nova | 16:46 | |
| mriedem | can this be removed now? https://github.com/openstack/nova/blob/master/nova/api/openstack/auth.py#L78 | 16:48 |
| *** lucas-hungry is now known as lucasagomes | 16:48 | |
| mriedem | deprecated in https://review.openstack.org/#/c/214085/ | 16:48 |
| *** david-lyle has joined #openstack-nova | 16:49 | |
| *** otter768 has joined #openstack-nova | 16:49 | |
| *** jlanoux has quit IRC | 16:50 | |
| *** Jeffrey4l has quit IRC | 16:50 | |
| *** mnestratov has quit IRC | 16:50 | |
| *** haomaiwang has quit IRC | 16:51 | |
| notmorgan | mriedem: hopefullt | 16:52 |
| notmorgan | hopefully | 16:53 |
| *** links has joined #openstack-nova | 16:53 | |
| *** tjones has quit IRC | 16:53 | |
| mriedem | it's not in the default api-paste.ini anymore so i'm removing it | 16:53 |
| notmorgan | the other one (other fake one) can probably also be trashed if we use the new keystonemiddleware fixture that actually does keystonemiddleware things | 16:54 |
| *** otter768 has quit IRC | 16:54 | |
| notmorgan | mriedem: i have a patch up for Ceilometer to use it, and could probably convert nova over to it too | 16:54 |
| mriedem | notmorgan: you mean auth_strategy=noauth2? | 16:54 |
| notmorgan | NoAuthMiddlewareBase seems to be faking out keyustonemiddleware | 16:55 |
| notmorgan | right? | 16:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 16:55 |
| notmorgan | or is noauth something elsE? | 16:55 |
| * notmorgan is suddenly feeling brain fried | 16:55 | |
| alaski | cdent: I wouldn't expect a bug to track that | 16:56 |
| mriedem | notmorgan: i think that's correct | 16:56 |
| mriedem | noauth2 is set in the nova unit tests | 16:56 |
| notmorgan | mriedem: so we have https://review.openstack.org/#/c/253967/ [if you want to see the example of what it looks like] | 16:56 |
| notmorgan | so you can use the real keystonemiddleware and just fake out the token responses | 16:56 |
| alaski | cdent: ignore me, I was scrolled up in chat | 16:57 |
| notmorgan | mriedem: and that'll move with keystonemiddleware as changes do/don't occur so very unlikely to break your unit tests | 16:57 |
| mriedem | notmorgan: please agree to pay me $20 every time keystonemiddleware does a release and nova unit tests are then broken | 16:57 |
| notmorgan | mriedem: no deal | 16:58 |
| *** jdurgin1 has quit IRC | 16:58 | |
| notmorgan | mriedem: but to be fair, you don't have to guess about headers and other such changes. | 16:59 |
| notmorgan | mriedem: but anyway... just an offer | 16:59 |
| openstackgerrit | Sean Dague proposed openstack/nova: always create clients with GlanceEndpoint https://review.openstack.org/254306 | 17:01 |
| *** lpetrut has quit IRC | 17:02 | |
| sdague | notmorgan: the thing is, wheren noauthmiddleware exists, it's actually possible that people are using it out there for real | 17:02 |
| sdague | it's not guarunteed to only be a test fixture | 17:02 |
| notmorgan | sdague: that ... makes me sad | 17:02 |
| notmorgan | but sure. | 17:03 |
| sdague | yeh, well look at the user survey, nova is more deployed than keystone | 17:03 |
| sdague | so someone's doing something there | 17:03 |
| sdague | mriedem: yes, https://github.com/openstack/nova/blob/master/nova/api/openstack/auth.py#L78 can be deleted | 17:04 |
| *** scheuran has quit IRC | 17:04 | |
| sdague | mriedem: actually, no that can't be removed | 17:05 |
| sdague | we actually want that for the project_id optional stuff | 17:05 |
| sdague | it probably gets renamed | 17:05 |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: This is 4th patch of changes according to pci-generate-stats blueprint. https://review.openstack.org/250757 | 17:06 |
| openstackgerrit | garyk proposed openstack/nova: Scheduler: honor the glance metadata for hypervisor details https://review.openstack.org/254206 | 17:06 |
| *** moshele has joined #openstack-nova | 17:08 | |
| sc68cal | dougwig: ping - I need some liaison assistance - http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-12-07.log.html#t2015-12-07T15:53:21 - about the Linux Bridge CI job | 17:08 |
| *** alkhod has joined #openstack-nova | 17:08 | |
| *** moshele_1 has joined #openstack-nova | 17:09 | |
| *** moshele has quit IRC | 17:10 | |
| alkhod | Hi everyone. I have an issue when trying to attach NFS volume to an instance: libvirtError: operation failed: open disk image file failed. If someone has any tips on what could be the reason here, I'd really appreciate any help. | 17:10 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: Adds nova conductor build_instances logging https://review.openstack.org/235911 | 17:10 |
| *** jistr has quit IRC | 17:11 | |
| mriedem | sdague: ok, i'll kill my local branch | 17:11 |
| mriedem | sdague: it's not used anywhere in the api-paste.ini though | 17:12 |
| *** links has quit IRC | 17:14 | |
| *** Daisy has joined #openstack-nova | 17:15 | |
| *** Jeffrey4l has joined #openstack-nova | 17:15 | |
| *** gcb has quit IRC | 17:15 | |
| sdague | mriedem: right, not yet | 17:16 |
| sdague | auggy's patch is going to need that | 17:16 |
| mriedem | so when a new thing comes along, can't it just add the new thing? | 17:16 |
| sdague | I guess, except it means deleting it today, and adding it back on wed | 17:16 |
| sdague | which seems silly | 17:16 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Compute: Adds driver disk_gb instance overhead estimation https://review.openstack.org/234283 | 17:17 |
| *** claudiub has joined #openstack-nova | 17:18 | |
| *** matrohon has quit IRC | 17:19 | |
| mriedem | i will move on from said silliness | 17:19 |
| *** Daisy has quit IRC | 17:19 | |
| *** sudipto has joined #openstack-nova | 17:21 | |
| auggy | sdague thanks for speaking up on that! i would have missed that conversation | 17:22 |
| *** ssurana has joined #openstack-nova | 17:23 | |
| *** davideagnello has joined #openstack-nova | 17:24 | |
| *** zz_john5223 is now known as john5223 | 17:25 | |
| *** salv-orl_ has joined #openstack-nova | 17:25 | |
| mriedem | sdague: did you have any changes up for https://bugs.launchpad.net/nova/+bug/1522454 ? | 17:26 |
| openstack | Launchpad bug 1522454 in OpenStack Compute (nova) "Nova is leaking libvirt internal ids on some Instance Not Found errors" [Medium,Triaged] | 17:26 |
| mriedem | sdague: seems we at least need to get a logging patch to figure out where this is failing in n-cpu | 17:26 |
| *** yassine__ has quit IRC | 17:26 | |
| sdague | mriedem: I do not as of yet, I have a git stash with a starting point | 17:26 |
| *** e0ne has quit IRC | 17:27 | |
| mriedem | ok, i'll poke on it | 17:27 |
| sdague | my instinct is there are a couple of different couplings in there | 17:27 |
| sdague | and this is mostly some lower level fails which are exposing | 17:28 |
| *** salv-orlando has quit IRC | 17:28 | |
| *** amotoki has quit IRC | 17:32 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports https://review.openstack.org/242573 | 17:32 |
| *** Sukhdev has quit IRC | 17:34 | |
| *** danpb has quit IRC | 17:35 | |
| *** yamahata has quit IRC | 17:36 | |
| *** klkumar has quit IRC | 17:36 | |
| ndipanov | bauzas, thanks for the reviews! | 17:37 |
| *** pratikmallya has quit IRC | 17:38 | |
| *** alkhod has quit IRC | 17:38 | |
| *** whenry has joined #openstack-nova | 17:39 | |
| *** kfarr has quit IRC | 17:39 | |
| *** gzhai3 has joined #openstack-nova | 17:43 | |
| *** lbelivea has quit IRC | 17:43 | |
| *** gzhai2 has quit IRC | 17:44 | |
| *** unicell1 has quit IRC | 17:44 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 17:45 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 17:45 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 17:45 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Add better help text to scheduler options https://review.openstack.org/247181 | 17:46 |
| *** tonytan4ever has quit IRC | 17:46 | |
| *** whenry has quit IRC | 17:46 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use ContainerFormat instead of strings https://review.openstack.org/245835 | 17:48 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add ContainerFormat field https://review.openstack.org/245834 | 17:48 |
| sdague | BobBall / johnthetubaguy all the xen plugin stuff is completely untested? | 17:49 |
| *** armax has joined #openstack-nova | 17:49 | |
| dougwig | sc68cal: ack, here. | 17:49 |
| *** ildikov has quit IRC | 17:51 | |
| *** fawadkhaliq has quit IRC | 17:52 | |
| openstackgerrit | Michael Bayer proposed openstack/nova: Remove SQLite BigInteger/Integer translation logic https://review.openstack.org/253859 | 17:52 |
| *** rfolco has quit IRC | 17:52 | |
| BobBall | sdague: It's tested by the XenServer CI, but no unit tests :/ | 17:53 |
| sdague | BobBall: ok, well I've got a patch for you that hopefully has interesting results there | 17:53 |
| *** rfolco has joined #openstack-nova | 17:53 | |
| BobBall | which patch? | 17:53 |
| *** edmondsw has quit IRC | 17:53 | |
| openstackgerrit | Sean Dague proposed openstack/nova: retool xen glance plugin to work with urls https://review.openstack.org/254331 | 17:54 |
| sdague | that one | 17:54 |
| *** tjones has joined #openstack-nova | 17:55 | |
| *** lbelivea has joined #openstack-nova | 17:55 | |
| sc68cal | dougwig: oh, sorry. I put it in the wrong channel - mean to put it in neutron | 17:57 |
| *** jistr has joined #openstack-nova | 17:59 | |
| johnthetubaguy | sdague: unit test wise, I have WIP patches to add that, but its only tested by the XenServer CI right now | 18:01 |
| *** david_cu has joined #openstack-nova | 18:01 | |
| johnthetubaguy | ah, yes, what BobBall said there | 18:01 |
| *** fawadkhaliq has joined #openstack-nova | 18:02 | |
| *** moshele_1 has quit IRC | 18:02 | |
| *** inc0 has joined #openstack-nova | 18:03 | |
| BobBall | sdague: Fingers crossed that patch will work :) Since it doesn't actually change how glance is called... :) | 18:03 |
| *** rdopiera has quit IRC | 18:03 | |
| *** suro-patz has joined #openstack-nova | 18:03 | |
| sdague | yeh, that was my theory | 18:05 |
| *** cdent has quit IRC | 18:05 | |
| sdague | so I could ensure the contract was honored before adding a new contract | 18:05 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: WIP: Fix help message in case of microversions https://review.openstack.org/254335 | 18:06 |
| *** pratikmallya has joined #openstack-nova | 18:07 | |
| sdague | ok, taking a break for a bit, I'll plough through some more later | 18:08 |
| sdague | johnthetubaguy: also, these files don't get picked up with pep8 | 18:08 |
| sdague | which is weird | 18:08 |
| *** yamahata has joined #openstack-nova | 18:08 | |
| openstackgerrit | Ryan Hsu proposed openstack/nova: Test - DO NOT MERGE https://review.openstack.org/254339 | 18:09 |
| *** gszasz has quit IRC | 18:10 | |
| *** mgoddard has joined #openstack-nova | 18:12 | |
| *** smoriya_ has quit IRC | 18:12 | |
| johnthetubaguy | sdague: thats an odd one, I have had to fix the pep8 checks several times over the last few years | 18:13 |
| *** gzhai has joined #openstack-nova | 18:14 | |
| *** gzhai3 has quit IRC | 18:15 | |
| *** mgoddard has quit IRC | 18:16 | |
| *** dims has joined #openstack-nova | 18:16 | |
| *** mgoddard has joined #openstack-nova | 18:16 | |
| *** andymaier has quit IRC | 18:16 | |
| *** garyk has quit IRC | 18:17 | |
| *** RichardRaseley has joined #openstack-nova | 18:18 | |
| *** Marga_ has quit IRC | 18:18 | |
| *** lpetrut has joined #openstack-nova | 18:18 | |
| *** BobBall is now known as BobBall_AWOL | 18:19 | |
| *** unicell has joined #openstack-nova | 18:19 | |
| *** enthurohini has quit IRC | 18:20 | |
| *** enthurohini has joined #openstack-nova | 18:21 | |
| *** mgoddard has quit IRC | 18:22 | |
| enthurohini | hello everyone, I am newbee in openstack. I have installed devstack to lurk. When i try to launch instance i am getting error "Failed to perform requested operation on instance "test", the instance has an error status: Please try again later [Error: Build of instance 70520b78-436c-4b3b-b75f-33a2e9964b5d aborted: Failed to allocate the network(s), not rescheduling.]. please help me out" | 18:24 |
| *** e0ne has joined #openstack-nova | 18:24 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Remove deprecated neutron auth options https://review.openstack.org/253792 | 18:25 |
| *** mgoddard has joined #openstack-nova | 18:26 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Remove deprecated neutron auth options https://review.openstack.org/253792 | 18:27 |
| *** penick has joined #openstack-nova | 18:27 | |
| *** ctrath has quit IRC | 18:28 | |
| notmorgan | mriedem: ^ there we go on the opt deprecations that should work and maintain coverage | 18:28 |
| *** ljxiash has quit IRC | 18:28 | |
| *** EinstCrazy has joined #openstack-nova | 18:28 | |
| *** mgoddard_ has joined #openstack-nova | 18:28 | |
| *** browne has joined #openstack-nova | 18:28 | |
| *** ljxiash has joined #openstack-nova | 18:29 | |
| *** smurke has quit IRC | 18:30 | |
| notmorgan | mriedem: for what it's worth... i feel dirty about that code... but it works and we can improve it as we dive deeper into keystoneauth conversions | 18:30 |
| *** rotbeard has quit IRC | 18:31 | |
| *** zenoway has joined #openstack-nova | 18:32 | |
| *** mgoddard has quit IRC | 18:32 | |
| mriedem | notmorgan: can you creata nova bp for the keystoneauth conversion work? | 18:32 |
| mriedem | specless, but i'd like to track this as a work item | 18:32 |
| *** mgoddard_ has quit IRC | 18:33 | |
| *** ljxiash has quit IRC | 18:34 | |
| *** EinstCrazy has quit IRC | 18:34 | |
| enthurohini | Can anyone help me with the error "Build of instance 70520b78-436c-4b3b-b75f-33a2e9964b5d aborted: Failed to allocate the network(s), not rescheduling." | 18:35 |
| *** salv-orlando has joined #openstack-nova | 18:35 | |
| *** salv-orl_ has quit IRC | 18:35 | |
| notmorgan | mriedem: uhm. sure. | 18:36 |
| *** tonytan4ever has joined #openstack-nova | 18:36 | |
| *** PaulMurray is now known as ptm_away | 18:37 | |
| notmorgan | mriedem: https://blueprints.launchpad.net/nova/+spec/keystoneclient-to-keystoneauth | 18:38 |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Remove deprecated neutron auth options https://review.openstack.org/253792 | 18:38 |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 18:39 |
| *** edmondsw has joined #openstack-nova | 18:39 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 18:39 |
| *** sudipto has quit IRC | 18:39 | |
| notmorgan | mriedem: and added to the current in-flight patches | 18:39 |
| *** smurke has joined #openstack-nova | 18:42 | |
| mriedem | thanks, approved | 18:44 |
| *** ildikov has joined #openstack-nova | 18:45 | |
| *** Marga_ has joined #openstack-nova | 18:45 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: Add accessIPv4 and accessIPv6 when create server https://review.openstack.org/253435 | 18:48 |
| *** rfolco_ has joined #openstack-nova | 18:48 | |
| *** otter768 has joined #openstack-nova | 18:50 | |
| *** rfolco has quit IRC | 18:50 | |
| *** rlrossit has quit IRC | 18:51 | |
| *** jaypipes has quit IRC | 18:52 | |
| *** mnestratov has joined #openstack-nova | 18:55 | |
| *** otter768 has quit IRC | 18:55 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 18:56 |
| *** FL1SK has joined #openstack-nova | 18:57 | |
| mriedem | ndipanov: are you still around? | 18:59 |
| *** aginwala has joined #openstack-nova | 19:01 | |
| *** markmc has quit IRC | 19:03 | |
| *** lykinsbd has quit IRC | 19:04 | |
| *** lpetrut has quit IRC | 19:04 | |
| *** ociuhandu has quit IRC | 19:04 | |
| *** sahid has quit IRC | 19:05 | |
| ndipanov | mriedem, sup | 19:06 |
| sdague | johnthetubaguy: well, it's definitely not running on those files which I noticed when editing them and it not throwing up the standard errors | 19:07 |
| mriedem | ndipanov: any idea where we convert an image meta with bdms in it to actual bdm objects for attaching during boot? | 19:07 |
| mriedem | ndipanov: in the case of an image snapshot created from a volume backed instance | 19:07 |
| mriedem | when creating a 2nd server from that image snapshot, we attach the server to the backing volume for the vol snapshot | 19:08 |
| mriedem | trying to find that code | 19:08 |
| ndipanov | mriedem, let me see | 19:09 |
| ndipanov | mriedem, _get_image_defined_bdms | 19:10 |
| mriedem | yup, just found that in _check_and_transform_bdm | 19:10 |
| mriedem | cool | 19:10 |
| mriedem | trying to sort out this race failure with ceph in the gate | 19:11 |
| *** pratikma_ has joined #openstack-nova | 19:12 | |
| *** pratikmallya has quit IRC | 19:12 | |
| bauzas | mriedem: btw. not sure you saw but the reno job is not busted in stable/liberty | 19:14 |
| *** agireud has joined #openstack-nova | 19:14 | |
| mriedem | i didn't see | 19:15 |
| *** ctrath has joined #openstack-nova | 19:17 | |
| dansmith | if I don't show up here tomorrow, send the divers: http://imgur.com/MrQQp9U | 19:21 |
| *** rk4n has quit IRC | 19:22 | |
| mriedem | psh | 19:22 |
| sdague | johnthetubaguy / BobBall_AWOL because of the way flake8 collects up files, it means that none of the plugins are checked because they don't end in .py | 19:22 |
| sdague | which removes a pretty useful basic set of sanity checking | 19:22 |
| *** pratikma_ has quit IRC | 19:23 | |
| *** pratikmallya has joined #openstack-nova | 19:23 | |
| dansmith | sdague: wouldn't modern flake8 choke on a bunch of 2.4isms? | 19:23 |
| sdague | I'm not sure why | 19:24 |
| *** rk4n has joined #openstack-nova | 19:24 | |
| dansmith | well, 2.4.0 was around/before pep8 was even a thing, right? | 19:24 |
| sdague | the base language isn't dramatically different | 19:25 |
| dansmith | sure, but practices that flake8 complains about that might not be valid in 2.4? | 19:25 |
| sdague | the thing that's mostly a shame to lose is the undefined name bits | 19:25 |
| sdague | yeh, sure, you could noqa them | 19:25 |
| *** rk4n has quit IRC | 19:25 | |
| openstackgerrit | Ryan Hsu proposed openstack/nova: Test - DO NOT MERGE https://review.openstack.org/254339 | 19:26 |
| *** jistr has quit IRC | 19:26 | |
| sdague | if I copy the file to a .py version, I get this - http://paste.openstack.org/show/481073/ | 19:26 |
| dansmith | sdague: yeah, I guess I just don't care that much to have to maintain those things but that's probably the wrong thing to do | 19:26 |
| dansmith | yeah, see we have to noqa all those except things | 19:27 |
| sdague | sure | 19:27 |
| sdague | but it would have also caught 4 errors in the patch before pushing it to the xenci system | 19:27 |
| sdague | as there are *no* unit tests on any of this | 19:27 |
| sdague | so it's only validation is the xenci full stack testing | 19:27 |
| migi | dansmith, re: https://review.openstack.org/#/c/250493/4 | 19:28 |
| dansmith | sdague: does the xen ci stuff even use these really? | 19:28 |
| sdague | dansmith: yes | 19:28 |
| openstackgerrit | Brian Rosmaita proposed openstack/nova-specs: Consistent Success Reporting https://review.openstack.org/248248 | 19:28 |
| sdague | this is required to communicate with glance | 19:28 |
| dansmith | migi: right, so I think that changing all uuids will hit a lot more than are required, so I would stick to the ones that fail as a result of the field change | 19:28 |
| migi | dansmith, for the nova tests I'm changing all the uuids also the ones that aren't constrained by your proposal | 19:28 |
| dansmith | migi: at least as a first pass | 19:28 |
| dansmith | sdague: okay | 19:29 |
| migi | dansmith, it's easier to grep/replace then run & fix imo | 19:29 |
| dansmith | migi: I know, but you're changing a lot of things that don't need to be changed | 19:29 |
| dansmith | migi: you're going to have a much harder time getting 1000 changes reviewed than the 270 that are actually required | 19:29 |
| migi | the thing is that they are all over the place | 19:30 |
| dansmith | migi: I know they are | 19:30 |
| openstackgerrit | Sean Dague proposed openstack/nova: retool xen glance plugin to work with urls https://review.openstack.org/254331 | 19:30 |
| migi | so the approach I took was to list all of the: | 19:30 |
| migi | fake_uuid, noexist, fake-image-uuid, fake-instance, fake_uuid1, fake_uuid2 and so on and have mapping 1-1 with new "proper" uuids | 19:31 |
| migi | this way I don't need to re-run tests so many times | 19:31 |
| dansmith | migi: yeah, I get it, | 19:32 |
| migi | if you think it's better to stick with fail&fix I'll redo change and start fresh ;) | 19:32 |
| dansmith | but it's going to change a lot of places where 'fake-uuid' is perfectly fine, like mocking some small function calls | 19:32 |
| dansmith | migi: I most definitely do, yes | 19:32 |
| migi | ok | 19:32 |
| dansmith | migi: test smaller batches at a time, which will run faster | 19:32 |
| * migi thought will be smart to save some work | 19:32 | |
| tjones | edleafe: have you done any config changes that deal with deprecated options? looking for an example. if you have not i will figure it out and we will have an example | 19:32 |
| dansmith | mikal: like you can do the libvirt tests only like this: tox -epy27 tests.unit.virt.libvirt | 19:33 |
| dansmith | er, migi ^ | 19:33 |
| dansmith | migi: and then make that one patch, and move on to tests.unit.compute, etc | 19:33 |
| migi | I'm doing small tests already | 19:33 |
| *** cdent has joined #openstack-nova | 19:33 | |
| dansmith | okay cool | 19:33 |
| sdague | dansmith: if you care - http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/31/254331/1/18383/logs/screen-n-cpu.txt.gz - search for XENAPI_PLUGIN_FAILURE (there is no direct linking in xenci) | 19:33 |
| migi | Ran 1342 tests in 636.752s | 19:34 |
| migi | FAILED (failures=178) | 19:34 |
| migi | just for nova.tests.unit.compute.test_compute | 19:34 |
| dansmith | sdague: heh | 19:34 |
| migi | ok will do as you wish :) | 19:35 |
| dansmith | migi: I'm just trying to help you do this in a way that will ever get reviewed :) | 19:35 |
| * migi need coffee | 19:35 | |
| migi | dansmith, understood :) | 19:35 |
| dansmith | sdague: yeah, sadface | 19:35 |
| dansmith | migi: are you an emacs person or do you use one of those "other editors" ? | 19:39 |
| migi | I'm between vi and gedit | 19:39 |
| *** gzhai1 has joined #openstack-nova | 19:40 | |
| dansmith | heh, okay | 19:40 |
| dansmith | there are tools in emacs for running our tests one by one quickly, which makes this a little easier to iterate | 19:40 |
| openstackgerrit | Dave McCowan proposed openstack/nova: Check current context before returning cached key manager client https://review.openstack.org/254358 | 19:40 |
| *** gzhai has quit IRC | 19:40 | |
| migi | dansmith, currently I'm using as simple as run_tests.sh with test name | 19:41 |
| dansmith | migi: ah, you should be using tox, not run_tests.sh | 19:42 |
| dansmith | although it won't be faster, so if it's working for you right now, that's fine | 19:42 |
| dansmith | migi: it takes a regex though, in case you don't know, so no need to use the whole test path | 19:42 |
| *** dave-mccowan has joined #openstack-nova | 19:42 | |
| migi | with -N flag it's not setting virt env and running pretty ok | 19:42 |
| tjones | edleafe: nevermind. found some | 19:43 |
| *** aginwala has quit IRC | 19:44 | |
| *** e0ne has quit IRC | 19:45 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 19:47 |
| *** vilobhmm11 has joined #openstack-nova | 19:48 | |
| *** aginwala has joined #openstack-nova | 19:48 | |
| *** pumaranikar has joined #openstack-nova | 19:51 | |
| *** bklei has joined #openstack-nova | 19:52 | |
| *** dave-mccowan has quit IRC | 19:53 | |
| *** lykinsbd has joined #openstack-nova | 19:53 | |
| *** fawadkhaliq has quit IRC | 19:54 | |
| david_cu | hey everyone, i started working on this bug https://bugs.launchpad.net/horizon/+bug/1326124. i am wondering what the opinion is on this. i was thinking about how to do this and it seems i'd need to add api endpoints for nova security groups (http://developer.openstack.org/api-ref-compute-v2-ext.html#os-security-group-rules) probably two, a GET for fetching the data, and a POST for updating security group ru | 19:57 |
| openstack | Launchpad bug 1326124 in OpenStack Compute (nova) "Allow for editing of existing Security Rules" [Wishlist,Confirmed] - Assigned to jichenjc (jichenjc) | 19:57 |
| david_cu | les. i am thinking adding more api endpoints maybe out of the scope of fixing this bug, especially if compute api v3 is on the way. so yeah, im wondering if there are any plans for either of these features in v3 i can help with? or if it'll never be in the api and i should probably not work on this bug :P | 19:57 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: WIP: Fix help message in case of microversions https://review.openstack.org/254335 | 19:58 |
| *** lykinsbd_ has joined #openstack-nova | 19:58 | |
| *** rk4n has joined #openstack-nova | 19:59 | |
| sdague | man this xenserver simulator is weird | 19:59 |
| mriedem | david_cu: v3 api is not a thing anymore, it's actually the v2.1 api | 19:59 |
| *** e0ne has joined #openstack-nova | 19:59 | |
| mriedem | updating the update requires microversions | 20:00 |
| mriedem | *updating the api | 20:00 |
| mriedem | and api changes require a blueprint and spec | 20:00 |
| mriedem | david_cu: so if you're new and looking for something to do, that's probably not a good starter | 20:00 |
| *** lykinsbd has quit IRC | 20:00 | |
| *** kfarr has joined #openstack-nova | 20:00 | |
| bauzas | mriedem: was looking at https://bugs.launchpad.net/nova/+bug/1521823 | 20:01 |
| openstack | Launchpad bug 1521823 in OpenStack Compute (nova) "reboot test fails in gate-grenade-dsvm-multinode with missing disk path" [High,Confirmed] | 20:01 |
| bauzas | mriedem: IIRC, the live migration checks a file to make sure it's okay to migrate, right? | 20:02 |
| mriedem | i'd have to look | 20:02 |
| *** e0ne has quit IRC | 20:02 | |
| *** vilobhmm11 has quit IRC | 20:03 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce freeze filesystems https://review.openstack.org/249140 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce thaw filesystems https://review.openstack.org/249141 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: replace direct libvirt's call AbortJobBlock https://review.openstack.org/249138 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: move find_disk to Guest's object https://review.openstack.org/249139 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "snapshot" to Guest's object https://review.openstack.org/249353 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce block_stats to return device stats https://review.openstack.org/251288 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "pause" to Guest's object https://review.openstack.org/249834 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: wrapper list_guests to Host's object https://review.openstack.org/249835 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "shutdown" to Guest's object https://review.openstack.org/249833 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce get_job_info to Guest's object https://review.openstack.org/250740 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: configure migrate max downtime https://review.openstack.org/250739 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "abort_job" to Guest's object https://review.openstack.org/250738 | 20:03 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "migrate" to Guest's object https://review.openstack.org/250365 | 20:03 |
| *** vilobhmm11 has joined #openstack-nova | 20:03 | |
| *** artom has quit IRC | 20:03 | |
| *** lykinsbd_ has quit IRC | 20:05 | |
| *** dave-mccowan has joined #openstack-nova | 20:06 | |
| bauzas | mriedem: nvm | 20:06 |
| *** vilobhmm11 has quit IRC | 20:06 | |
| *** zenoway has quit IRC | 20:06 | |
| *** zenoway has joined #openstack-nova | 20:08 | |
| mriedem | bauzas: ok, i've been in -cinder all morning looking into the ceph ci failures | 20:09 |
| bauzas | mriedem: no worries | 20:09 |
| bauzas | mriedem: I've just been hit by the grenade multinode job failure a couple of times so I was looking at the bug report | 20:09 |
| bauzas | trying to see what could have been the root cause | 20:10 |
| *** vilobhmm11 has joined #openstack-nova | 20:11 | |
| *** lucasagomes is now known as lucas-dinner | 20:11 | |
| openstackgerrit | yalei wang proposed openstack/nova-specs: VM boot with unaddressed port https://review.openstack.org/239276 | 20:14 |
| *** ociuhandu has joined #openstack-nova | 20:14 | |
| vilobhmm11 | alaski : ping https://review.openstack.org/#/c/253306/ | 20:14 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 20:15 |
| alaski | vilobhmm11: I've been meaning to get to that. I'll take a look now while my tests are running | 20:16 |
| vilobhmm11 | alaski : thanks :) | 20:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Add support for 2.10 https://review.openstack.org/251486 | 20:17 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Increase max version to 2.11 https://review.openstack.org/253095 | 20:17 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] Increase max version to 2.12 https://review.openstack.org/254272 | 20:17 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [microversions] update test_versions with implemented versions https://review.openstack.org/254369 | 20:17 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 20:17 |
| *** e0ne has joined #openstack-nova | 20:19 | |
| *** e0ne has joined #openstack-nova | 20:19 | |
| *** e0ne has quit IRC | 20:20 | |
| *** ianw has joined #openstack-nova | 20:20 | |
| *** Sukhdev has joined #openstack-nova | 20:20 | |
| openstackgerrit | Ryan McNair proposed openstack/nova: Add retry logic for detaching device using LibVirt https://review.openstack.org/227851 | 20:21 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 20:21 |
| david_cu | ah gotcha, thanks mriedem! i'll look for something a bit less involved for my first contribution :) | 20:22 |
| *** EinstCrazy has joined #openstack-nova | 20:22 | |
| *** baoli_ has quit IRC | 20:24 | |
| inc0 | hey, is upgrade_level compute=auto working? | 20:24 |
| inc0 | or I'm doing something wrong? | 20:24 |
| *** migi is now known as migi_afk | 20:25 | |
| dansmith | inc0: upgrade_levels/compute, but yes | 20:25 |
| dansmith | inc0: it's running in gate that way | 20:25 |
| inc0 | ok, I'll look at gate jobs if I did misconfiguration somewhere | 20:26 |
| *** EinstCrazy has quit IRC | 20:27 | |
| dansmith | inc0: https://review.openstack.org/#/c/233289/ | 20:28 |
| *** boden has joined #openstack-nova | 20:28 | |
| inc0 | thanks a lot | 20:28 |
| inc0 | a yeah this review you linked in mailing thread | 20:28 |
| inc0 | I'll look what's wrong on my end | 20:29 |
| *** matrohon has joined #openstack-nova | 20:30 | |
| openstackgerrit | Ryan McNair proposed openstack/nova: Add retry logic for detaching device using LibVirt https://review.openstack.org/227851 | 20:30 |
| *** gzhai2 has joined #openstack-nova | 20:33 | |
| *** aginwala has quit IRC | 20:33 | |
| mordred | BobBall_AWOL: https://review.openstack.org/#/c/253792/7 has a xen CI failure, but the link is a 404 | 20:34 |
| *** gzhai1 has quit IRC | 20:35 | |
| *** changbl has quit IRC | 20:35 | |
| *** jaosorior has quit IRC | 20:36 | |
| *** jaosorior has joined #openstack-nova | 20:37 | |
| edleafe | BobBall_AWOL: yeah, I'm also getting a 404 on the Xen failure for https://review.openstack.org/#/c/247181/ | 20:40 |
| *** ljxiash has joined #openstack-nova | 20:41 | |
| *** aginwala has joined #openstack-nova | 20:41 | |
| *** jaosorior has quit IRC | 20:41 | |
| *** jaosorior has joined #openstack-nova | 20:43 | |
| *** ljxiash has quit IRC | 20:45 | |
| *** jaosorior has quit IRC | 20:45 | |
| *** nic has joined #openstack-nova | 20:46 | |
| *** baoli has joined #openstack-nova | 20:48 | |
| *** baoli has quit IRC | 20:48 | |
| *** pradk has joined #openstack-nova | 20:49 | |
| *** baoli has joined #openstack-nova | 20:49 | |
| *** xyang1 has joined #openstack-nova | 20:49 | |
| *** matrohon has quit IRC | 20:51 | |
| *** otter768 has joined #openstack-nova | 20:51 | |
| *** zhenq has joined #openstack-nova | 20:52 | |
| vilobhmm11 | alaski : the reason why i used ListOfStringsField is because same flavor can be part of multiple flavor classes ? We can have same flavor m1.small and flavor_class = ['vm', 'bm']. Also wanted it to be a listofObject rather than listofstrings because that way if we want to specialize particular objects we can fox example if flavor_class is object "VM" it can be specialized as vm.small, vm.large etc and quota can be set at | 20:53 |
| alaski | vilobhmm11: okay, I didn't recall that being in the spec. I have always been under the assumption that a flavor is only in one class | 20:55 |
| *** otter768 has quit IRC | 20:56 | |
| alaski | having the ability for multiple classes leads to a lot more complexity and that needs some careful thought before going down that path | 20:57 |
| *** aginwala has quit IRC | 20:57 | |
| mordred | mriedem: all of my blocking-deps for the novaclient patch are either relased or about to | 20:57 |
| vilobhmm11 | alaski : ok. I think you are right thats not in the spec but it can be a generalized version going ahead if you feel thats a good idea…will limit the scope to just one class per flavor for now | 20:57 |
| mordred | mriedem: I hope to have you an updated version by the time I land tomorro | 20:57 |
| vilobhmm11 | alaski : and hence will have a string field rather than having a listofstrings…can file a new spec for multiple classes per flavor…if that sounds ok to you | 20:58 |
| mriedem | mordred: ok | 20:59 |
| alaski | vilobhmm11: a spec would be the best way to get a discussion started about it | 20:59 |
| vilobhmm11 | alaski : having multiple classes per flavor and then having specilization of those classes can be a useful step in the direction of quota by az, quota by flavor IMHO | 20:59 |
| vilobhmm11 | alaski : sure…thanks for the feedback…as always helpful :) | 21:00 |
| *** ljxiash has joined #openstack-nova | 21:03 | |
| *** mwagner_lap has quit IRC | 21:04 | |
| *** ijuwang has quit IRC | 21:07 | |
| openstackgerrit | Morgan Fainberg proposed openstack/nova: Migrate from keystoneclient to keystoneauth https://review.openstack.org/253793 | 21:07 |
| *** ljxiash has quit IRC | 21:07 | |
| *** ijuwang has joined #openstack-nova | 21:08 | |
| *** harlowja has quit IRC | 21:08 | |
| *** harlowja has joined #openstack-nova | 21:08 | |
| *** rk4n has quit IRC | 21:08 | |
| *** baoli has quit IRC | 21:09 | |
| *** baoli has joined #openstack-nova | 21:10 | |
| *** ducttape_ has quit IRC | 21:10 | |
| *** migi_afk is now known as migi | 21:11 | |
| *** baoli has quit IRC | 21:11 | |
| *** baoli has joined #openstack-nova | 21:11 | |
| *** smatzek has quit IRC | 21:13 | |
| *** tpatil has joined #openstack-nova | 21:13 | |
| migi | dansmith, would you recommend splitting review per test class or per module ? | 21:15 |
| dansmith | migi: depends on how much is done in each | 21:15 |
| migi | ~25 class in module | 21:15 |
| dansmith | migi: submit a first patch with those changes and let's have a look | 21:15 |
| dansmith | don't wait until you're done to submit | 21:15 |
| migi | ok | 21:16 |
| dansmith | I have to run off in a few, but I can look when I get back | 21:16 |
| *** mrkz has quit IRC | 21:17 | |
| *** neiljerram_bb has joined #openstack-nova | 21:17 | |
| *** mrkz has joined #openstack-nova | 21:18 | |
| *** bradjones has quit IRC | 21:20 | |
| zhenq | hi, to fix a bug of openstack.compute.api (v2.1) in current trunk, should I also commit the change in legacy v2 api (api/openstack/compute/legacy_v2)? if yes, should they be included same commit? | 21:20 |
| *** baoli has quit IRC | 21:21 | |
| *** changbl has joined #openstack-nova | 21:21 | |
| *** bradjones has joined #openstack-nova | 21:22 | |
| *** bradjones has quit IRC | 21:22 | |
| *** bradjones has joined #openstack-nova | 21:22 | |
| *** pratikmallya has quit IRC | 21:24 | |
| *** aginwala has joined #openstack-nova | 21:26 | |
| *** vladikr has quit IRC | 21:26 | |
| *** thangp has quit IRC | 21:27 | |
| *** pratikmallya has joined #openstack-nova | 21:29 | |
| *** mariusv has joined #openstack-nova | 21:30 | |
| *** mariusv has quit IRC | 21:30 | |
| *** mariusv has joined #openstack-nova | 21:30 | |
| *** wyang has quit IRC | 21:34 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 21:34 |
| openstackgerrit | Chris Dent proposed openstack/nova: Improve Filter Scheduler doc clarity https://review.openstack.org/254394 | 21:35 |
| *** alejandrito has joined #openstack-nova | 21:36 | |
| aginwala | some one please review https://review.openstack.org/#/c/250767/ | 21:37 |
| *** xyang1 has quit IRC | 21:38 | |
| *** dustins is now known as dustins|gone | 21:38 | |
| *** ianbrown has quit IRC | 21:39 | |
| *** ianbrown has joined #openstack-nova | 21:39 | |
| openstackgerrit | Michal Pryc proposed openstack/nova: Implements proper UUID format for the ComputeAPITestCase https://review.openstack.org/254398 | 21:40 |
| *** aginwala has quit IRC | 21:40 | |
| *** xyang1 has joined #openstack-nova | 21:41 | |
| *** mwagner_lap has joined #openstack-nova | 21:43 | |
| migi | dansmith, ^^ | 21:44 |
| *** lykinsbd has joined #openstack-nova | 21:47 | |
| *** xyang1 has quit IRC | 21:47 | |
| openstackgerrit | jichenjc proposed openstack/python-novaclient: WIP:allow multiple confirm/revert resize https://review.openstack.org/254400 | 21:49 |
| *** busterswt has quit IRC | 21:49 | |
| *** gjayavelu has joined #openstack-nova | 21:50 | |
| *** mgoddard has joined #openstack-nova | 21:51 | |
| openstackgerrit | Sean Dague proposed openstack/nova: bifurcate for no project id tests https://review.openstack.org/254401 | 21:55 |
| sdague | auggy: ok, I think that is what I was imagining, though I think we need a conversation with gmann and alex_xu, because I feel like the general test_scenarios setup is more complicated here than I thought it should be. | 21:56 |
| RichardRaseley | Is there a strict requirement on memcache for nova-consoleauth? | 21:58 |
| *** changbl has quit IRC | 21:59 | |
| *** tongli has joined #openstack-nova | 21:59 | |
| *** ljxiash has joined #openstack-nova | 22:00 | |
| *** tongli has quit IRC | 22:01 | |
| *** boden has left #openstack-nova | 22:01 | |
| *** thorst has quit IRC | 22:01 | |
| auggy | sdauge: cool | 22:02 |
| auggy | sdague i mean | 22:02 |
| *** thorst has joined #openstack-nova | 22:03 | |
| *** ljxiash has quit IRC | 22:04 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 22:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 22:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 22:05 |
| *** pradk has quit IRC | 22:08 | |
| *** thorst has quit IRC | 22:08 | |
| *** aginwala has joined #openstack-nova | 22:08 | |
| *** mgoddard has quit IRC | 22:09 | |
| openstackgerrit | Eric Brown proposed openstack/nova: VMware: Enforce minimum vCenter version of 5.5 https://review.openstack.org/253666 | 22:09 |
| *** ljxiash has joined #openstack-nova | 22:12 | |
| *** yarkot has joined #openstack-nova | 22:13 | |
| RichardRaseley | What mechanism does nova-consoleauth use to validate a token passed to it? | 22:14 |
| *** aginwala has quit IRC | 22:15 | |
| *** e0ne has joined #openstack-nova | 22:15 | |
| *** lbelivea has quit IRC | 22:16 | |
| *** ljxiash has quit IRC | 22:16 | |
| *** lbelivea has joined #openstack-nova | 22:16 | |
| *** lykinsbd has quit IRC | 22:19 | |
| *** aginwala has joined #openstack-nova | 22:21 | |
| *** thorst has joined #openstack-nova | 22:22 | |
| mriedem | dansmith: looks like we'll still want that bdm uuid field i think | 22:26 |
| mriedem | b/c we really suck at handling bdms without a volume_id | 22:26 |
| alaski | are those patches still up somewhere? | 22:27 |
| *** thorst has quit IRC | 22:27 | |
| *** eliqiao_ has joined #openstack-nova | 22:27 | |
| mriedem | yeah | 22:27 |
| *** pratikma_ has joined #openstack-nova | 22:27 | |
| *** moshele_1 has joined #openstack-nova | 22:27 | |
| alaski | I may take a look at refreshing them | 22:28 |
| *** yarkot has quit IRC | 22:28 | |
| mriedem | comments 13-15 here are the suck https://bugs.launchpad.net/nova/+bug/1464259 | 22:28 |
| openstack | Launchpad bug 1464259 in OpenStack Compute (nova) "Volumes tests fails often with rbd backend" [High,Triaged] | 22:28 |
| mriedem | i think i have the race in my head at least | 22:28 |
| alaski | or at least sketching out what I was thinking for cells, because I don't think I'm explaining it well in my comments | 22:28 |
| alaski | ahh, that's good | 22:28 |
| mriedem | this is a non-cells race | 22:28 |
| mriedem | it's a thing with ceph | 22:28 |
| *** eliqiao has quit IRC | 22:28 | |
| mriedem | but the test is like almost the worst, it's that same ebs test | 22:29 |
| *** mariusv_ has joined #openstack-nova | 22:29 | |
| mriedem | create a volume snapshot and boot a server from it, nova creates a volume from the snapshot and while attaching, the test deletes the server | 22:29 |
| mriedem | before it's active | 22:29 |
| mriedem | so the nova db is, who konws | 22:29 |
| mriedem | the bdm.volume_id is not set, so we can't delete that volume that nova created | 22:29 |
| mriedem | and then tempest cna't delete the snapshot, and then everything falls over | 22:29 |
| mriedem | i think i might have a hack fix | 22:30 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L396 | 22:30 |
| mriedem | call save() right after that | 22:30 |
| *** pratikmallya has quit IRC | 22:30 | |
| mriedem | jbernard: ^ i'm going to try that | 22:30 |
| mriedem | have to leave soon though so might be tomorrow | 22:30 |
| *** pratikma_ has quit IRC | 22:31 | |
| mriedem | alaski: but i remember when we were looking at the cells failure that the bdm is keyed off the volume_id and device_name | 22:31 |
| mriedem | which in the case of a snapshot bdm, the volume_id isn't set | 22:31 |
| alaski | well, the bdm not having a stable thing we can reference with bugs me, cells or not. | 22:31 |
| mriedem | right | 22:32 |
| *** diogogmt has joined #openstack-nova | 22:32 | |
| mriedem | honestly (speaking as sdague), this ebs test is just plain racey | 22:32 |
| *** ZZelle_ has joined #openstack-nova | 22:32 | |
| *** mariusv_ has quit IRC | 22:32 | |
| mriedem | ceph or cells or not | 22:32 |
| mriedem | the lvm case is just fast enoguh | 22:32 |
| jdurgin | mriedem: seems like the test is making bad assumptions about deletion order, and should be fixed | 22:32 |
| mriedem | jdurgin: definitely | 22:32 |
| mriedem | jdurgin: well, it's rightly tickling a bad spot in nova | 22:33 |
| jdurgin | nice analysis in those comments though, several bugs hidden in there | 22:33 |
| alaski | is it doing something a user may reasonably do? if so then it may be good, as painful as it is | 22:33 |
| mriedem | chances are no.... | 22:33 |
| mriedem | if they wait for the instance to be active before deleting it, but who knows | 22:33 |
| mriedem | people might 'oh crap, i didn't want this, delete' | 22:33 |
| mriedem | and then you have an orphaned volume | 22:33 |
| alaski | I can assure you that instances are deleted before going active all the time | 22:34 |
| mriedem | this is also super confusing b/c it's not boot from volume | 22:34 |
| mriedem | it's boot from image with snapshot bdm info | 22:34 |
| *** aginwala has quit IRC | 22:34 | |
| mriedem | which does the sneaky nova orchestrated bfv | 22:34 |
| mriedem | throw a delete before active in the mix, and kapow! | 22:34 |
| alaski | another reason to remove it IMO | 22:34 |
| mriedem | never happen | 22:35 |
| mriedem | :) | 22:35 |
| alaski | I'm sure that's true, but I'll never give up on it :) | 22:35 |
| mriedem | this is probably another case for the force detach CLI thing in nova-manage | 22:35 |
| *** enthurohini has quit IRC | 22:37 | |
| *** neiljerram_bb has quit IRC | 22:38 | |
| openstackgerrit | Surojit Pathak proposed openstack/nova: New filter for metrics-threshold-scheduling https://review.openstack.org/254423 | 22:38 |
| *** kfarr has quit IRC | 22:39 | |
| *** smatzek has joined #openstack-nova | 22:39 | |
| *** akshai has quit IRC | 22:39 | |
| *** akshai has joined #openstack-nova | 22:41 | |
| *** changbl has joined #openstack-nova | 22:44 | |
| *** mariusv has quit IRC | 22:45 | |
| *** moshele_1 has quit IRC | 22:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Make sure bdm.volume_id is set after auto-creating volumes https://review.openstack.org/254428 | 22:49 |
| mriedem | jbernard: jdurgin: jgriffith: alaski: ndipanov: ^ still needs tests, but it's a start | 22:49 |
| openstackgerrit | Eric Brown proposed openstack/nova: VMware: Enforce minimum vCenter version of 5.5 https://review.openstack.org/253666 | 22:50 |
| *** thorst_ has joined #openstack-nova | 22:50 | |
| openstackgerrit | Eric Brown proposed openstack/nova: VMware: Enforce minimum vCenter version of 5.5 https://review.openstack.org/253666 | 22:50 |
| *** Sukhdev has quit IRC | 22:50 | |
| *** cdent has quit IRC | 22:51 | |
| *** sneti has quit IRC | 22:52 | |
| *** otter768 has joined #openstack-nova | 22:52 | |
| *** mriedem has quit IRC | 22:54 | |
| *** otter768 has quit IRC | 22:57 | |
| *** aginwala has joined #openstack-nova | 22:58 | |
| *** ducttape_ has joined #openstack-nova | 22:59 | |
| *** rk4n has joined #openstack-nova | 23:02 | |
| *** aysyd has quit IRC | 23:03 | |
| *** e0ne has quit IRC | 23:04 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Clean up filter_properties/availability_zone in boot https://review.openstack.org/254434 | 23:05 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-novaclient: Validation for arguments of list command passed by "--fields" https://review.openstack.org/253734 | 23:07 |
| *** jichen has quit IRC | 23:09 | |
| *** zhenq1 has joined #openstack-nova | 23:10 | |
| *** eglynn has quit IRC | 23:11 | |
| *** nic has quit IRC | 23:11 | |
| *** mgoddard has joined #openstack-nova | 23:11 | |
| *** zhenq has quit IRC | 23:11 | |
| *** ianbrown has quit IRC | 23:12 | |
| *** ianbrown has joined #openstack-nova | 23:12 | |
| *** ccarmack has left #openstack-nova | 23:17 | |
| *** apoorvad has quit IRC | 23:18 | |
| *** jamielennox is now known as jamielennox|away | 23:18 | |
| *** mgoddard has quit IRC | 23:19 | |
| *** mdrabe has quit IRC | 23:20 | |
| *** smurke has quit IRC | 23:20 | |
| *** gzhai3 has joined #openstack-nova | 23:22 | |
| *** boris-42_ has quit IRC | 23:23 | |
| *** pumaranikar has quit IRC | 23:23 | |
| *** gzhai2 has quit IRC | 23:25 | |
| *** salv-orl_ has joined #openstack-nova | 23:25 | |
| *** rk4n has quit IRC | 23:28 | |
| *** salv-orlando has quit IRC | 23:28 | |
| *** rk4n has joined #openstack-nova | 23:29 | |
| *** alex_xu has quit IRC | 23:29 | |
| *** ctrath has quit IRC | 23:30 | |
| *** alex_xu has joined #openstack-nova | 23:31 | |
| *** bklei has quit IRC | 23:32 | |
| *** rk4n has quit IRC | 23:33 | |
| *** apoorvad has joined #openstack-nova | 23:35 | |
| vilobhmm11 | alaski : ping qq | 23:38 |
| *** aginwala has quit IRC | 23:40 | |
| *** mrkz has quit IRC | 23:41 | |
| *** sfinucan has quit IRC | 23:41 | |
| *** ianbrown has quit IRC | 23:41 | |
| *** aginwala has joined #openstack-nova | 23:41 | |
| *** ianbrown has joined #openstack-nova | 23:42 | |
| openstackgerrit | Eric Brown proposed openstack/nova: VMware: Enforce minimum vCenter version of 5.5 https://review.openstack.org/253666 | 23:48 |
| *** thorst_ has quit IRC | 23:51 | |
| *** thorst has joined #openstack-nova | 23:51 | |
| *** tonytan_brb has joined #openstack-nova | 23:51 | |
| *** boris-42_ has joined #openstack-nova | 23:53 | |
| *** takedakn has joined #openstack-nova | 23:53 | |
| *** tonytan_brb has quit IRC | 23:54 | |
| *** tonytan4ever has quit IRC | 23:55 | |
| *** thorst has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!