| *** haomaiwang has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| *** ircuser-1 has joined #openstack-nova | 00:02 | |
| *** unicell has joined #openstack-nova | 00:05 | |
| *** Fdaisuke has joined #openstack-nova | 00:05 | |
| *** mylu has joined #openstack-nova | 00:07 | |
| *** thorst has joined #openstack-nova | 00:08 | |
| *** unicell has quit IRC | 00:10 | |
| *** Pistachio has quit IRC | 00:13 | |
| *** davideagle has quit IRC | 00:15 | |
| *** wmolicki has joined #openstack-nova | 00:16 | |
| *** unicell has joined #openstack-nova | 00:19 | |
| *** thorst has quit IRC | 00:27 | |
| *** wmolicki has quit IRC | 00:28 | |
| *** wmolicki has joined #openstack-nova | 00:28 | |
| *** beekneemech has quit IRC | 00:29 | |
| *** bnemec has joined #openstack-nova | 00:31 | |
| *** markvoelker has joined #openstack-nova | 00:43 | |
| *** wmolicki has quit IRC | 00:44 | |
| *** mylu has quit IRC | 00:46 | |
| *** markvoelker has quit IRC | 00:47 | |
| *** vladikr has quit IRC | 00:48 | |
| *** mylu has joined #openstack-nova | 00:50 | |
| *** hoangcx has joined #openstack-nova | 00:51 | |
| *** jinxing has joined #openstack-nova | 00:55 | |
| *** haomaiwang has quit IRC | 00:58 | |
| *** unicell has quit IRC | 00:59 | |
| *** achanda has quit IRC | 01:00 | |
| *** hayato_ has joined #openstack-nova | 01:00 | |
| *** Jeffrey4l has joined #openstack-nova | 01:01 | |
| *** vladikr has joined #openstack-nova | 01:01 | |
| *** Daisy has joined #openstack-nova | 01:03 | |
| *** EinstCrazy has joined #openstack-nova | 01:06 | |
| *** mylu has quit IRC | 01:07 | |
| *** mylu has joined #openstack-nova | 01:09 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds HostCapabilities object model https://review.openstack.org/285856 | 01:10 |
|---|---|---|
| *** vladikr has quit IRC | 01:12 | |
| *** jinxing has quit IRC | 01:12 | |
| *** salv-orlando has joined #openstack-nova | 01:12 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 01:14 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 01:16 |
| *** jinxing has joined #openstack-nova | 01:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add expected_task_state for instance.save() used by block_device_mapping https://review.openstack.org/245038 | 01:17 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Handle NotImplementedError for attach_interface https://review.openstack.org/272471 | 01:17 |
| *** aysyd has joined #openstack-nova | 01:20 | |
| *** salv-orlando has quit IRC | 01:20 | |
| *** Marga_ has quit IRC | 01:21 | |
| *** Marga_ has joined #openstack-nova | 01:22 | |
| *** Marga__ has joined #openstack-nova | 01:22 | |
| *** Marga__ has quit IRC | 01:23 | |
| *** Marga_ has quit IRC | 01:23 | |
| *** Marga_ has joined #openstack-nova | 01:23 | |
| *** Marga_ has quit IRC | 01:24 | |
| *** Marga_ has joined #openstack-nova | 01:25 | |
| eliqiao | andreykurilin: Hi, there? can I ask a question about https://review.openstack.org/#/c/260383/4/novaclient/v2/shell.py | 01:29 |
| *** aysyd has quit IRC | 01:31 | |
| *** Daisy has quit IRC | 01:32 | |
| *** Daisy has joined #openstack-nova | 01:32 | |
| *** claudiub has joined #openstack-nova | 01:35 | |
| *** Daisy has quit IRC | 01:36 | |
| *** haomaiwa_ has joined #openstack-nova | 01:42 | |
| *** Marga__ has joined #openstack-nova | 01:42 | |
| *** vilobhmm11 has joined #openstack-nova | 01:43 | |
| *** ljxiash has joined #openstack-nova | 01:43 | |
| *** Daisy has joined #openstack-nova | 01:43 | |
| *** mylu has quit IRC | 01:44 | |
| *** markvoelker has joined #openstack-nova | 01:44 | |
| *** ljxiash has quit IRC | 01:44 | |
| *** ljxiash has joined #openstack-nova | 01:44 | |
| *** ljxiash has quit IRC | 01:45 | |
| *** Marga_ has quit IRC | 01:46 | |
| *** Daisy has quit IRC | 01:46 | |
| *** mylu has joined #openstack-nova | 01:46 | |
| *** Daisy has joined #openstack-nova | 01:46 | |
| *** markvoelker has quit IRC | 01:48 | |
| *** Daisy_ has joined #openstack-nova | 01:50 | |
| *** Daisy has quit IRC | 01:51 | |
| *** ljxiash has joined #openstack-nova | 01:51 | |
| *** ducttape_ has joined #openstack-nova | 01:53 | |
| *** Sukhdev has joined #openstack-nova | 01:56 | |
| *** jinxing has quit IRC | 02:00 | |
| *** vilobhmm11 has quit IRC | 02:00 | |
| *** haomaiwa_ has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-nova | 02:01 | |
| *** jinxing has joined #openstack-nova | 02:04 | |
| *** jinxing has quit IRC | 02:04 | |
| *** Kevin_Zheng has joined #openstack-nova | 02:04 | |
| *** ducttape_ has quit IRC | 02:05 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: object: Adds host_capabilities in ComputeNode model https://review.openstack.org/228280 | 02:06 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds HostCapabilities object model https://review.openstack.org/285856 | 02:06 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: driver: Adds get_hypervisor_capabilities method https://review.openstack.org/254124 | 02:06 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds get_hypervisor_capabilities implementation https://review.openstack.org/254125 | 02:06 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: db: Adds host_capabilities column in compute_nodes https://review.openstack.org/228279 | 02:06 |
| *** Daisy_ has quit IRC | 02:06 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Fix _test_pci() to NOT actually claim device https://review.openstack.org/285865 | 02:09 |
| *** Daisy has joined #openstack-nova | 02:09 | |
| *** achanda has joined #openstack-nova | 02:10 | |
| *** ljxiash has quit IRC | 02:11 | |
| *** ljxiash has joined #openstack-nova | 02:11 | |
| *** ljxiash has quit IRC | 02:12 | |
| *** ljxiash has joined #openstack-nova | 02:12 | |
| *** Sukhdev has quit IRC | 02:13 | |
| *** Sukhdev has joined #openstack-nova | 02:13 | |
| *** achanda has quit IRC | 02:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check 'destination_type' instead of 'source_type' in _check_and_transform_bdm https://review.openstack.org/252836 | 02:15 |
| *** achanda has joined #openstack-nova | 02:15 | |
| *** mylu has quit IRC | 02:18 | |
| *** mylu has joined #openstack-nova | 02:20 | |
| *** achanda has quit IRC | 02:22 | |
| *** mylu has quit IRC | 02:28 | |
| *** thorst has joined #openstack-nova | 02:28 | |
| *** Yingxin has quit IRC | 02:29 | |
| *** mylu has joined #openstack-nova | 02:29 | |
| *** aginwala has joined #openstack-nova | 02:29 | |
| *** Yingxin has joined #openstack-nova | 02:29 | |
| *** Sukhdev has quit IRC | 02:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Add config option "scheduler_compute_node_update_interval" for scheduler https://review.openstack.org/260920 | 02:31 |
| *** Marga__ has quit IRC | 02:33 | |
| *** Marga_ has joined #openstack-nova | 02:33 | |
| *** mylu has quit IRC | 02:35 | |
| *** thorst has quit IRC | 02:36 | |
| *** salv-orlando has joined #openstack-nova | 02:37 | |
| *** mylu has joined #openstack-nova | 02:37 | |
| *** Daisy has quit IRC | 02:38 | |
| *** Daisy has joined #openstack-nova | 02:38 | |
| *** lixiaoy1 has joined #openstack-nova | 02:39 | |
| *** mylu has quit IRC | 02:40 | |
| *** Daisy has quit IRC | 02:43 | |
| *** Daisy has joined #openstack-nova | 02:43 | |
| *** markvoelker has joined #openstack-nova | 02:44 | |
| *** kashyap has quit IRC | 02:45 | |
| *** adreznec has quit IRC | 02:45 | |
| *** mdavidson has quit IRC | 02:46 | |
| *** mylu has joined #openstack-nova | 02:46 | |
| *** lennyb__ has quit IRC | 02:46 | |
| *** adreznec has joined #openstack-nova | 02:47 | |
| *** lennyb__ has joined #openstack-nova | 02:47 | |
| *** kfox1111 has quit IRC | 02:49 | |
| *** markvoelker has quit IRC | 02:49 | |
| *** kfox1111 has joined #openstack-nova | 02:49 | |
| *** kashyap has joined #openstack-nova | 02:49 | |
| *** mdavidson has joined #openstack-nova | 02:50 | |
| openstackgerrit | Wenzhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 02:50 |
| *** boris-42 has joined #openstack-nova | 02:50 | |
| *** ctrath has joined #openstack-nova | 02:51 | |
| *** Sukhdev has joined #openstack-nova | 02:52 | |
| *** salv-orlando has quit IRC | 02:53 | |
| *** vilobhmm11 has joined #openstack-nova | 02:58 | |
| *** mdnadeem has joined #openstack-nova | 02:59 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** tangchen has joined #openstack-nova | 03:01 | |
| *** mylu has quit IRC | 03:07 | |
| *** mylu has joined #openstack-nova | 03:07 | |
| *** nelsnels_ has joined #openstack-nova | 03:09 | |
| *** nelsnelson has quit IRC | 03:12 | |
| *** aginwala has quit IRC | 03:20 | |
| *** baoli has joined #openstack-nova | 03:21 | |
| *** tojuvone_ has quit IRC | 03:21 | |
| *** mylu has quit IRC | 03:23 | |
| *** aginwala has joined #openstack-nova | 03:23 | |
| *** Yingxin_ has joined #openstack-nova | 03:27 | |
| *** tojuvone has joined #openstack-nova | 03:28 | |
| *** ctrath has quit IRC | 03:28 | |
| *** ljxiash has quit IRC | 03:28 | |
| *** Yingxin has quit IRC | 03:29 | |
| *** Yingxin_ is now known as Yingxin | 03:29 | |
| *** ljxiash has joined #openstack-nova | 03:29 | |
| *** Daisy has quit IRC | 03:33 | |
| *** ljxiash has quit IRC | 03:33 | |
| *** Daisy has joined #openstack-nova | 03:34 | |
| *** Daisy has joined #openstack-nova | 03:34 | |
| *** thorst has joined #openstack-nova | 03:36 | |
| *** achanda has joined #openstack-nova | 03:37 | |
| *** hayato__ has joined #openstack-nova | 03:39 | |
| *** hayato_ has quit IRC | 03:41 | |
| *** haukebruno has joined #openstack-nova | 03:41 | |
| *** thorst has quit IRC | 03:41 | |
| *** achanda has quit IRC | 03:42 | |
| *** dave-mccowan has quit IRC | 03:42 | |
| *** Daisy has quit IRC | 03:43 | |
| *** links has joined #openstack-nova | 03:44 | |
| *** Daisy has joined #openstack-nova | 03:44 | |
| *** tyagiprince has joined #openstack-nova | 03:47 | |
| *** Daisy has quit IRC | 03:48 | |
| *** baoli has quit IRC | 03:49 | |
| *** achanda has joined #openstack-nova | 03:49 | |
| *** aginwala has quit IRC | 03:51 | |
| *** Daisy has joined #openstack-nova | 03:52 | |
| *** stevemar has joined #openstack-nova | 03:53 | |
| *** Daisy has quit IRC | 03:54 | |
| *** Daisy has joined #openstack-nova | 03:54 | |
| *** achanda has quit IRC | 03:55 | |
| *** salv-orlando has joined #openstack-nova | 03:57 | |
| *** Daisy has quit IRC | 03:59 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** mingyu has joined #openstack-nova | 04:01 | |
| *** Daisy has joined #openstack-nova | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** links has quit IRC | 04:01 | |
| *** salv-orlando has quit IRC | 04:03 | |
| *** Sukhdev has quit IRC | 04:04 | |
| *** hayato_ has joined #openstack-nova | 04:05 | |
| *** Daisy has quit IRC | 04:06 | |
| *** dimtruck is now known as zz_dimtruck | 04:07 | |
| *** hayato__ has quit IRC | 04:08 | |
| *** hayato__ has joined #openstack-nova | 04:09 | |
| *** hayato___ has joined #openstack-nova | 04:10 | |
| *** hayato_ has quit IRC | 04:10 | |
| *** hayato_ has joined #openstack-nova | 04:11 | |
| *** hayato__ has quit IRC | 04:11 | |
| *** hayato__ has joined #openstack-nova | 04:11 | |
| *** david-lyle has joined #openstack-nova | 04:13 | |
| *** mingyu has quit IRC | 04:14 | |
| *** hayato___ has quit IRC | 04:15 | |
| *** hayato_ has quit IRC | 04:15 | |
| *** claudiub has quit IRC | 04:17 | |
| *** links has joined #openstack-nova | 04:18 | |
| *** mingyu has joined #openstack-nova | 04:22 | |
| *** mingyu has quit IRC | 04:23 | |
| *** mingyu has joined #openstack-nova | 04:23 | |
| *** haomaiwang has quit IRC | 04:31 | |
| *** salv-orlando has joined #openstack-nova | 04:32 | |
| *** david-lyle has quit IRC | 04:36 | |
| *** salv-orlando has quit IRC | 04:38 | |
| *** markvoelker has joined #openstack-nova | 04:45 | |
| *** markvoelker has quit IRC | 04:50 | |
| *** david-lyle has joined #openstack-nova | 04:50 | |
| *** thorst has joined #openstack-nova | 04:54 | |
| *** achanda has joined #openstack-nova | 04:55 | |
| *** david-lyle has quit IRC | 04:56 | |
| *** achanda has quit IRC | 05:01 | |
| *** mingyu has quit IRC | 05:02 | |
| *** mingyu has joined #openstack-nova | 05:02 | |
| *** thorst has quit IRC | 05:03 | |
| *** chhavi has joined #openstack-nova | 05:06 | |
| *** mylu has joined #openstack-nova | 05:07 | |
| *** Daisy_ has joined #openstack-nova | 05:08 | |
| *** mingyu_ has joined #openstack-nova | 05:09 | |
| *** mingyu has quit IRC | 05:12 | |
| *** tyagiprince has quit IRC | 05:19 | |
| *** ishant has joined #openstack-nova | 05:19 | |
| *** mingyu_ has quit IRC | 05:25 | |
| *** ljxiash has joined #openstack-nova | 05:28 | |
| *** salv-orlando has joined #openstack-nova | 05:36 | |
| *** eliqiao_ has quit IRC | 05:38 | |
| *** aginwala has joined #openstack-nova | 05:38 | |
| *** zhangjn has quit IRC | 05:38 | |
| *** zhangjn has joined #openstack-nova | 05:42 | |
| *** chhavi has quit IRC | 05:45 | |
| *** salv-orlando has quit IRC | 05:47 | |
| *** mubirru has joined #openstack-nova | 05:50 | |
| *** ankit_ag has joined #openstack-nova | 05:50 | |
| *** jinxing has joined #openstack-nova | 05:52 | |
| *** mylu has quit IRC | 05:53 | |
| *** fawadkhaliq has joined #openstack-nova | 05:54 | |
| *** amotoki has joined #openstack-nova | 05:56 | |
| *** gongysh has joined #openstack-nova | 05:57 | |
| *** achanda has joined #openstack-nova | 05:58 | |
| *** gszasz has joined #openstack-nova | 05:59 | |
| *** thorst has joined #openstack-nova | 06:00 | |
| *** baoli has joined #openstack-nova | 06:01 | |
| *** achanda has quit IRC | 06:02 | |
| *** jinxing has quit IRC | 06:03 | |
| *** zenoway has joined #openstack-nova | 06:03 | |
| *** moshele has joined #openstack-nova | 06:03 | |
| *** Daisy has joined #openstack-nova | 06:04 | |
| *** baoli has quit IRC | 06:05 | |
| *** jinxing has joined #openstack-nova | 06:06 | |
| *** Daisy_ has quit IRC | 06:07 | |
| *** thorst has quit IRC | 06:07 | |
| *** ekuris_ has joined #openstack-nova | 06:08 | |
| *** omnipresent has joined #openstack-nova | 06:09 | |
| *** mingyu has joined #openstack-nova | 06:11 | |
| *** achanda has joined #openstack-nova | 06:11 | |
| *** zenoway has quit IRC | 06:12 | |
| openstackgerrit | Eli Qiao proposed openstack/python-novaclient: [microversion] Bump to 2.25 https://review.openstack.org/260383 | 06:13 |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for live-migration. https://review.openstack.org/241476 | 06:13 |
| *** Sukhdev has joined #openstack-nova | 06:13 | |
| *** zenoway has joined #openstack-nova | 06:15 | |
| *** omnipresent has quit IRC | 06:18 | |
| openstackgerrit | Abhishek Talwar proposed openstack/python-novaclient: nova quota-update shouldn't work with tenant-name https://review.openstack.org/284042 | 06:18 |
| *** aginwala has quit IRC | 06:19 | |
| *** mingyu_ has joined #openstack-nova | 06:20 | |
| *** omnipresent has joined #openstack-nova | 06:20 | |
| *** dguitarbite has joined #openstack-nova | 06:20 | |
| *** dguitarbite has quit IRC | 06:20 | |
| *** BobBall_AWOL has joined #openstack-nova | 06:21 | |
| *** Daisy has quit IRC | 06:22 | |
| *** Daisy has joined #openstack-nova | 06:22 | |
| *** mingyu has quit IRC | 06:23 | |
| *** dguitarbite has joined #openstack-nova | 06:23 | |
| *** dguitarbite has quit IRC | 06:24 | |
| *** zenoway has quit IRC | 06:24 | |
| *** dguitarbite has joined #openstack-nova | 06:24 | |
| *** BobBall_1WOL has quit IRC | 06:25 | |
| *** zenoway has joined #openstack-nova | 06:25 | |
| *** gcb has joined #openstack-nova | 06:26 | |
| *** Daisy_ has joined #openstack-nova | 06:26 | |
| *** Daisy has quit IRC | 06:26 | |
| *** BobBall_1WOL has joined #openstack-nova | 06:26 | |
| *** BobBall_AWOL has quit IRC | 06:26 | |
| *** esracelik has joined #openstack-nova | 06:28 | |
| *** omnipresent has quit IRC | 06:29 | |
| *** sridharg has joined #openstack-nova | 06:30 | |
| *** omnipresent has joined #openstack-nova | 06:31 | |
| *** Daisy has joined #openstack-nova | 06:31 | |
| *** nikhil has quit IRC | 06:32 | |
| *** zenoway has quit IRC | 06:34 | |
| *** Daisy_ has quit IRC | 06:35 | |
| *** omnipresent has quit IRC | 06:36 | |
| *** zenoway has joined #openstack-nova | 06:36 | |
| *** dedery has joined #openstack-nova | 06:36 | |
| *** lpetrut has joined #openstack-nova | 06:37 | |
| *** aginwala has joined #openstack-nova | 06:39 | |
| *** dedery_ has joined #openstack-nova | 06:40 | |
| *** moshele has quit IRC | 06:41 | |
| *** salv-orlando has joined #openstack-nova | 06:42 | |
| *** dedery has quit IRC | 06:42 | |
| *** liverpooler has joined #openstack-nova | 06:44 | |
| *** rcernin has joined #openstack-nova | 06:44 | |
| *** moshele has joined #openstack-nova | 06:45 | |
| *** ianw has quit IRC | 06:45 | |
| *** markvoelker has joined #openstack-nova | 06:46 | |
| *** zhangjn has quit IRC | 06:47 | |
| *** rcernin has quit IRC | 06:49 | |
| *** zhangjn has joined #openstack-nova | 06:49 | |
| *** Fdaisuke has quit IRC | 06:49 | |
| *** ianw has joined #openstack-nova | 06:50 | |
| *** markvoelker has quit IRC | 06:50 | |
| *** garyk has joined #openstack-nova | 06:54 | |
| *** zenoway has quit IRC | 06:54 | |
| *** zenoway has joined #openstack-nova | 06:54 | |
| *** nkrinner has joined #openstack-nova | 06:57 | |
| *** zenoway has quit IRC | 06:59 | |
| *** garyk has quit IRC | 06:59 | |
| *** jaosorior has joined #openstack-nova | 07:01 | |
| *** sheel has joined #openstack-nova | 07:02 | |
| *** hayato__ has quit IRC | 07:04 | |
| *** hoangcx has quit IRC | 07:04 | |
| *** mjura has joined #openstack-nova | 07:04 | |
| *** thorst has joined #openstack-nova | 07:05 | |
| *** dedery_ is now known as dedery | 07:11 | |
| *** thorst has quit IRC | 07:12 | |
| *** paul-carlton2 has joined #openstack-nova | 07:12 | |
| *** fawadk has joined #openstack-nova | 07:15 | |
| *** fawadkhaliq has quit IRC | 07:15 | |
| *** fawadkhaliq has joined #openstack-nova | 07:15 | |
| *** amotoki has quit IRC | 07:19 | |
| *** fawadk has quit IRC | 07:20 | |
| *** hoangcx has joined #openstack-nova | 07:21 | |
| *** rcernin has joined #openstack-nova | 07:24 | |
| *** belmoreira has joined #openstack-nova | 07:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Decode injected content when checking length limit https://review.openstack.org/285649 | 07:26 |
| *** mingyu_ has quit IRC | 07:30 | |
| *** scheuran has joined #openstack-nova | 07:30 | |
| *** zenoway has joined #openstack-nova | 07:30 | |
| *** amotoki has joined #openstack-nova | 07:31 | |
| *** nikhil has joined #openstack-nova | 07:32 | |
| *** paul-carlton2 has quit IRC | 07:32 | |
| *** zhangjn has quit IRC | 07:33 | |
| *** zenoway has quit IRC | 07:35 | |
| *** paul-carlton2 has joined #openstack-nova | 07:35 | |
| *** amotoki has quit IRC | 07:35 | |
| *** Daisy has quit IRC | 07:35 | |
| *** Daisy has joined #openstack-nova | 07:36 | |
| *** speller has joined #openstack-nova | 07:36 | |
| *** claudiub has joined #openstack-nova | 07:37 | |
| *** lpetrut has quit IRC | 07:38 | |
| *** zhangjn has joined #openstack-nova | 07:39 | |
| *** gongysh has quit IRC | 07:39 | |
| *** haomaiwang has joined #openstack-nova | 07:41 | |
| *** Daisy has quit IRC | 07:41 | |
| *** pcaruana has joined #openstack-nova | 07:43 | |
| *** fawadkhaliq has quit IRC | 07:43 | |
| eliqiao | andreykurilin: Hi, there? | 07:44 |
| *** dstepanenko__ has quit IRC | 07:45 | |
| *** lpetrut has joined #openstack-nova | 07:48 | |
| *** amotoki has joined #openstack-nova | 07:49 | |
| *** amotoki has quit IRC | 07:49 | |
| *** gus has quit IRC | 07:50 | |
| *** erno has joined #openstack-nova | 07:50 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: API: Mapping ConsoleTypeInvalid exception to HTTPBadRequest https://review.openstack.org/285922 | 07:51 |
| *** Daisy has joined #openstack-nova | 07:52 | |
| *** gus has joined #openstack-nova | 07:52 | |
| *** zhangjn has quit IRC | 07:54 | |
| *** zenoway has joined #openstack-nova | 07:54 | |
| *** gongysh has joined #openstack-nova | 07:55 | |
| *** gszasz has quit IRC | 07:57 | |
| *** zenoway has quit IRC | 07:58 | |
| *** zhangjn has joined #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** Daisy has quit IRC | 08:02 | |
| *** rk4n has joined #openstack-nova | 08:02 | |
| *** mattymo has quit IRC | 08:03 | |
| *** nik__ is now known as snikitin | 08:04 | |
| *** Daisy has joined #openstack-nova | 08:05 | |
| *** mattymo has joined #openstack-nova | 08:05 | |
| *** rk4n has quit IRC | 08:05 | |
| *** ihrachys has joined #openstack-nova | 08:06 | |
| *** rdopiera has joined #openstack-nova | 08:06 | |
| *** jichen has joined #openstack-nova | 08:07 | |
| jaosorior | gibi: Hey dude, are you around? | 08:07 |
| *** lpetrut has quit IRC | 08:08 | |
| *** wwwbukolaycom has quit IRC | 08:09 | |
| *** thorst has joined #openstack-nova | 08:11 | |
| *** paul-carlton2 has quit IRC | 08:13 | |
| *** hayato_ has joined #openstack-nova | 08:15 | |
| *** hayato_ has quit IRC | 08:15 | |
| *** hayato_ has joined #openstack-nova | 08:16 | |
| *** thorst has quit IRC | 08:17 | |
| *** zhangjn has quit IRC | 08:18 | |
| *** jed56 has joined #openstack-nova | 08:20 | |
| *** andrearosa_web has joined #openstack-nova | 08:22 | |
| *** zenoway has joined #openstack-nova | 08:22 | |
| *** jlanoux has joined #openstack-nova | 08:23 | |
| *** lpetrut has joined #openstack-nova | 08:23 | |
| *** yassine has joined #openstack-nova | 08:24 | |
| *** matrohon has joined #openstack-nova | 08:25 | |
| *** achanda has quit IRC | 08:26 | |
| *** paul-carlton2 has joined #openstack-nova | 08:26 | |
| *** fawadkhaliq has joined #openstack-nova | 08:27 | |
| *** gongysh has quit IRC | 08:29 | |
| *** zhangjn has joined #openstack-nova | 08:29 | |
| *** mingyu has joined #openstack-nova | 08:29 | |
| *** rcernin has quit IRC | 08:29 | |
| *** sudipto has joined #openstack-nova | 08:30 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: [WIP]Compute: Query procceeding status migration in confirm/revert_resize() https://review.openstack.org/285931 | 08:31 |
| *** mpavone has joined #openstack-nova | 08:35 | |
| *** amotoki has joined #openstack-nova | 08:35 | |
| *** amotoki has quit IRC | 08:36 | |
| *** achanda has joined #openstack-nova | 08:37 | |
| *** rcernin has joined #openstack-nova | 08:37 | |
| *** fawadkhaliq has quit IRC | 08:38 | |
| *** amotoki has joined #openstack-nova | 08:45 | |
| *** lpetrut has quit IRC | 08:45 | |
| *** markvoelker has joined #openstack-nova | 08:47 | |
| bauzas | good morning Nova | 08:47 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support https://review.openstack.org/248805 | 08:49 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags controller https://review.openstack.org/128940 | 08:49 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/268932 | 08:49 |
| *** gszasz has joined #openstack-nova | 08:49 | |
| *** markvoelker has quit IRC | 08:51 | |
| *** PaulMurray has joined #openstack-nova | 08:51 | |
| *** sahid has joined #openstack-nova | 08:54 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added missed '-' to the rest api history doc https://review.openstack.org/285941 | 08:54 |
| PaulMurray | johnthetubaguy, alex_xu it seems we still need to resolve shaohe_feng's novaclient patch for report migrations: https://review.openstack.org/#/c/281335/ | 08:56 |
| *** paul-carlton2 has quit IRC | 08:57 | |
| *** ndipanov has joined #openstack-nova | 08:57 | |
| *** omnipresent has joined #openstack-nova | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** zenoway has quit IRC | 09:03 | |
| *** achanda has quit IRC | 09:08 | |
| *** acabot has joined #openstack-nova | 09:09 | |
| *** huats__ has left #openstack-nova | 09:10 | |
| *** amotoki has quit IRC | 09:10 | |
| *** achanda has joined #openstack-nova | 09:12 | |
| *** amotoki has joined #openstack-nova | 09:13 | |
| *** Sukhdev has quit IRC | 09:13 | |
| *** thorst has joined #openstack-nova | 09:15 | |
| *** Daisy has quit IRC | 09:15 | |
| *** Daisy has joined #openstack-nova | 09:15 | |
| *** ljxiash has quit IRC | 09:17 | |
| *** ljxiash has joined #openstack-nova | 09:17 | |
| *** ljxiash has quit IRC | 09:17 | |
| *** ljxiash has joined #openstack-nova | 09:18 | |
| *** ljxiash has quit IRC | 09:18 | |
| *** rpodolyaka has left #openstack-nova | 09:19 | |
| *** jistr has joined #openstack-nova | 09:20 | |
| *** Daisy has quit IRC | 09:20 | |
| *** rk4n has joined #openstack-nova | 09:21 | |
| *** derekh has joined #openstack-nova | 09:21 | |
| *** huats is now known as Guest33875 | 09:22 | |
| *** thorst has quit IRC | 09:22 | |
| *** davideagle has joined #openstack-nova | 09:23 | |
| *** amotoki has quit IRC | 09:24 | |
| *** patrickeast has quit IRC | 09:24 | |
| *** patrickeast has joined #openstack-nova | 09:25 | |
| *** paul-carlton2 has joined #openstack-nova | 09:25 | |
| alex_xu | PaulMurray: I will take a look at after hour, have to go home now | 09:27 |
| *** vilobhmm11 has quit IRC | 09:31 | |
| *** danpb has joined #openstack-nova | 09:32 | |
| *** rpodolyaka has joined #openstack-nova | 09:34 | |
| PaulMurray | alex_xu, I'm going the other way, offline for a while to go to office | 09:34 |
| *** amotoki has joined #openstack-nova | 09:34 | |
| *** BobBall_1WOL is now known as BobBall | 09:35 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: functional: Grab the service version from the module https://review.openstack.org/285459 | 09:37 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 09:37 |
| *** davideagle has quit IRC | 09:37 | |
| *** davideagle has joined #openstack-nova | 09:38 | |
| *** PaulMurray has quit IRC | 09:38 | |
| andreykurilin | eliqiao: hi! yes | 09:40 |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add nova client server-migration-list and server-migration-show https://review.openstack.org/281335 | 09:42 |
| eliqiao | andreykurilin: hi, it's re-https://review.openstack.org/#/c/260383/ | 09:42 |
| *** e0ne has joined #openstack-nova | 09:43 | |
| eliqiao | andreykurilin: actually, I tried your suggestion (add start_version end_version) but I still get args conflict error. | 09:43 |
| andreykurilin | hm... | 09:43 |
| *** aginwala has quit IRC | 09:43 | |
| andreykurilin | it is strange | 09:43 |
| dedery | hello nova. I see an issue that I'm not sure if/how it should be addressed (a fix may be in order or not). I spin a VM with SRIOV port, I shutdown the PF in the host (the vNIC changes status to DOWN), I take a snapshot of this suboptimal VM. The snapshot fails due to the offline PF, the VM will get back up without the port | 09:44 |
| dedery | The fact that the VM doesn't have the port is understandable, but I'm not sure about the failed snapshot. Once i commented out two "raise" from libvirt.py everything worked | 09:45 |
| andreykurilin | eliqiao: it works for 2.21 microversion - https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/contrib/instance_action.py#L90-L100 | 09:45 |
| dedery | ndipanov: ping :) can you please address ^ ? | 09:46 |
| ndipanov | dedery, I don't understand the problem | 09:46 |
| eliqiao | andreykurilin: okay, thanks for point me out, I will do another try. | 09:46 |
| andreykurilin | eliqiao: I'll check your patch now. it should work!:) if not, there is a bug in novaclient's code, which should be fixed asap | 09:47 |
| ndipanov | dedery, it would be best to report a bug if you think it's a problem | 09:47 |
| dedery | ndipanov: I'm not sure that there's a problem here. Just wondering if this is the expected behavior or a missed spot. Should the snapshot creation fail on a suboptimal VM? | 09:47 |
| ndipanov | dedery, probably not | 09:48 |
| dedery | ndipanov: cool. plus, it fails due to the spin-up of the VM AFTER the snapshot, the snapshot by itself is successfull. | 09:48 |
| dedery | I'll issue a bug and try to fix it | 09:49 |
| dedery | (not by removing the exception raise of course) | 09:49 |
| *** roeyc has joined #openstack-nova | 09:49 | |
| *** Daisy_ has joined #openstack-nova | 09:52 | |
| *** lpetrut has joined #openstack-nova | 09:52 | |
| *** achanda has quit IRC | 09:53 | |
| *** boris-42 has quit IRC | 09:54 | |
| *** ihrachys has quit IRC | 09:54 | |
| *** e0ne has quit IRC | 09:55 | |
| eliqiao | andreykurilin: can you please check my testing step(diff is again to my patch) http://paste.openstack.org/show/488519/ | 09:55 |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add nova client server-migration-list and server-migration-show https://review.openstack.org/281335 | 09:56 |
| *** Daisy_ has quit IRC | 09:57 | |
| *** shaohe_feng has joined #openstack-nova | 09:58 | |
| *** salv-orl_ has joined #openstack-nova | 09:59 | |
| *** ihrachys has joined #openstack-nova | 09:59 | |
| *** thorst has joined #openstack-nova | 09:59 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** salv-orlando has quit IRC | 10:02 | |
| andreykurilin | eliqiao: hm. I can't find the difference in your code and mine( http://paste.openstack.org/show/488521/ ). But it works in my case:) | 10:02 |
| *** davideagle has quit IRC | 10:03 | |
| eliqiao | andreykurilin: oh, my bad. I forget to remove 2.25 version in http://paste.openstack.org/show/488521/@52 | 10:05 |
| *** ishant has quit IRC | 10:05 | |
| eliqiao | andreykurilin: it works perfect for me know, I am going to update my patch after I back to home. | 10:05 |
| eliqiao | andreykurilin: really thanks for you help :) | 10:06 |
| *** thorst has quit IRC | 10:06 | |
| andreykurilin | eliqiao: nice! | 10:06 |
| shaohe_feng | andreykurilin: hi | 10:06 |
| *** wznoinsk__ is now known as wznoinsk | 10:07 | |
| andreykurilin | shaohe_feng: hi! | 10:07 |
| shaohe_feng | andreykurilin: I have get into trouble with my novaclient | 10:07 |
| andreykurilin | shaohe_feng: what had happend? | 10:07 |
| *** PaulMurray has joined #openstack-nova | 10:07 | |
| shaohe_feng | andreykurilin: https://review.openstack.org/#/c/281335/ | 10:07 |
| shaohe_feng | andreykurilin: you have reviewed this patch. | 10:08 |
| andreykurilin | yeah, I remembered this patch | 10:08 |
| shaohe_feng | andreykurilin: see the Patch Set 6 comments "@ Andrey Kurilin" from me. | 10:09 |
| andreykurilin | shaohe_feng: one moment:) | 10:10 |
| PaulMurray | shaohe_feng, just looking at you patch | 10:11 |
| andreykurilin | shaohe_feng: it is strange, api_versions should be there | 10:11 |
| andreykurilin | will check | 10:11 |
| shaohe_feng | andreykurilin: I think so. | 10:11 |
| *** jinxing has quit IRC | 10:12 | |
| shaohe_feng | andreykurilin: I want to add a testcase for migration-list. | 10:14 |
| *** EinstCrazy has quit IRC | 10:14 | |
| shaohe_feng | andreykurilin: https://review.openstack.org/#/c/281335/8/novaclient/tests/unit/v2/fakes.py@2396 I hope the migration-list will call the get_os_migrations, and pass the api_version to it. | 10:15 |
| shaohe_feng | PaulMurray: Great. Thank you. | 10:15 |
| PaulMurray | shaohe_feng, just following really | 10:16 |
| shaohe_feng | PaulMurray: and thanks for update it last week. | 10:16 |
| PaulMurray | shaohe_feng, what are you planning to do about alex_xu comments? | 10:16 |
| shaohe_feng | PaulMurray: one concern. should we display the link column? | 10:17 |
| alex_xu | shaohe_feng: can we display it when it existed? | 10:17 |
| PaulMurray | shaohe_feng, I would say no - but I wanted to ask alex_xu and johnthetubaguy | 10:17 |
| shaohe_feng | alex_xu: yes. we cna | 10:17 |
| *** davideagle has joined #openstack-nova | 10:18 | |
| shaohe_feng | alex_xu: yes, we can. but the problem is that, ref is too long. | 10:18 |
| alex_xu | PaulMurray: ah, I think you are right, that isn't useful for cli user | 10:18 |
| PaulMurray | alex_xu, clients don't usually display links like this do they? | 10:18 |
| johnthetubaguy | I would focus on the new API, and not the os-migrations API | 10:18 |
| alex_xu | shaohe_feng: I think PaulMurray is right | 10:18 |
| johnthetubaguy | try server list, and look at the flavor, that has links in it | 10:19 |
| shaohe_feng | alex_xu: agree. I also do not like to display it. | 10:19 |
| johnthetubaguy | but regardless, I would focus on listing server migrations using the new API | 10:19 |
| shaohe_feng | johnthetubaguy: got it. | 10:21 |
| *** gszasz has quit IRC | 10:22 | |
| shaohe_feng | johnthetubaguy: I have get into trouble with the old api os-migrations in novaclient when add testcase for it. For I find I can not pass the api_version to FakeHTTPClient. | 10:23 |
| *** daemontool_ has quit IRC | 10:23 | |
| johnthetubaguy | shaohe_feng: hmm, there must be some existing similar tests for things like key pairs, I would have thought | 10:24 |
| *** omnipresent has quit IRC | 10:24 | |
| shaohe_feng | johnthetubaguy: I have check the existing code. not found. andreykurilin is helping to fix it. | 10:24 |
| *** zenoway has joined #openstack-nova | 10:26 | |
| *** roeyc has quit IRC | 10:27 | |
| andreykurilin | shaohe_feng: Existing TestCase for migrations inherits from utils.TestCase, can you try to use utils.FixturedTestCase ? ( https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/utils.py#L73 ) | 10:28 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: objects: add HyperVLiveMigrateData stub https://review.openstack.org/285370 | 10:29 |
| shaohe_feng | andreykurilin: let me try it. | 10:29 |
| *** daemontool has joined #openstack-nova | 10:29 | |
| *** Pistachio has joined #openstack-nova | 10:33 | |
| bauzas | johnthetubaguy: ready for being hassled by https://review.openstack.org/#/c/284959/ ? :D | 10:36 |
| *** sfinucan has joined #openstack-nova | 10:37 | |
| speller | Hi guys, I want to propose a blueprint for rebuild_server to be able to get not just an image, but a bdm, just like create_server. Do you have any insights on that before I submit a blueprint? | 10:37 |
| *** hayato_ has quit IRC | 10:39 | |
| *** daemontool_ has joined #openstack-nova | 10:39 | |
| johnthetubaguy | bauzas: oh, sure | 10:39 |
| johnthetubaguy | :) | 10:39 |
| bauzas | johnthetubaguy: so, you think we should update the RPC API for this, and not just commenting it ? | 10:40 |
| johnthetubaguy | so I am on the fence | 10:40 |
| bauzas | I agree I should respin with at least some docstrings | 10:40 |
| johnthetubaguy | a new version is free | 10:40 |
| johnthetubaguy | well, a few lines of code | 10:40 |
| bauzas | johnthetubaguy: well, my point is that we agreed to say that cells v1 are nice but not really having more features in there | 10:41 |
| *** daemontool has quit IRC | 10:41 | |
| johnthetubaguy | I am not saying pass the thing down, just bump the version number, so we have a hook to fix things | 10:41 |
| johnthetubaguy | agreed about no more cells v1 features | 10:41 |
| bauzas | johnthetubaguy: oh, just bumping the manager version and adding a new version string in the RPC API ? | 10:42 |
| johnthetubaguy | yeah, I think thats all we are both asking for | 10:42 |
| *** daemontool_ is now known as daemontool | 10:42 | |
| *** gszasz has joined #openstack-nova | 10:42 | |
| *** salv-orlando has joined #openstack-nova | 10:42 | |
| bauzas | johnthetubaguy: so a new version having the exact same payload than the older ? | 10:42 |
| johnthetubaguy | bauzas: yep | 10:42 |
| bauzas | mmm | 10:42 |
| johnthetubaguy | if we later have any issues, we know its a new caller | 10:42 |
| *** salv-orlando has quit IRC | 10:43 | |
| *** salv-orl_ has quit IRC | 10:43 | |
| johnthetubaguy | which might help us | 10:43 |
| bauzas | well, I see your point | 10:43 |
| *** salv-orlando has joined #openstack-nova | 10:43 | |
| bauzas | okay, lemme respin it quickly and see if that's okay | 10:43 |
| johnthetubaguy | its maybe pointless, I am not sure, just seemed like a cunning idea | 10:43 |
| bauzas | johnthetubaguy: well, honestly, I dunno if that's more helping that a docstring, but fine, I can do that | 10:44 |
| bauzas | helping than* | 10:44 |
| bauzas | johnthetubaguy: I would just say "okay, see folks, we're supporting the exact same stuff than the older, except that we fence those new bits to be passed down" | 10:45 |
| bauzas | that wouldn't technically change the RPC interface, just the Facade - but the facade isn't versioned | 10:45 |
| bauzas | anyway | 10:45 |
| bauzas | I can do that - fer sur | 10:46 |
| bauzas | I can now understand better matt's point | 10:46 |
| bauzas | johnthetubaguy: thnaks | 10:46 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Fix wrong watermark state during live migration https://review.openstack.org/279616 | 10:47 |
| alex_xu | bauzas: morning, quick question, i'm thinking we can add scheduler_hints flag for live-migration,evacuate,etc in n release, do you think it make sense or not or is there any people from scheduler sub-team have same plan? otherwise I plan to help on that. | 10:47 |
| johnthetubaguy | bauzas: no worries | 10:47 |
| *** markvoelker has joined #openstack-nova | 10:47 | |
| johnthetubaguy | bauzas: agreed, the facade is not versioned, as its only ever called by the "new" coe | 10:47 |
| johnthetubaguy | code | 10:47 |
| bauzas | johnthetubaguy: yup | 10:48 |
| bauzas | johnthetubaguy: but okay, I see why matt mentioned a "signal" | 10:48 |
| bauzas | alex_xu: you mean supporting to add hints when live-migrating ? | 10:48 |
| shaohe_feng | andreykurilin: https://paste.fedoraproject.org/331058/14567429/ | 10:48 |
| shaohe_feng | andreykurilin: it does not work. | 10:48 |
| bauzas | alex_xu: like nova live-migrate myinstance --hint foo=bar ? | 10:49 |
| alex_xu | bauzas: yes, not only live-migration, also can add for evacuate, rebuild..or other api we will rescheduling instance | 10:49 |
| bauzas | alex_xu: mmm, not sure I like that | 10:49 |
| alex_xu | bauzas: ok, what is your concern about that? | 10:49 |
| andreykurilin | shaohe_feng: ok, will check | 10:49 |
| bauzas | alex_xu: because you are defining your constraints when booting your instance | 10:49 |
| *** jinxing has joined #openstack-nova | 10:50 | |
| johnthetubaguy | alex_xu: need to look at why that is wanted, and if thats the best way to do it | 10:50 |
| bauzas | alex_xu: in case you want to break those constraints when migrating, you can force a destination | 10:50 |
| alex_xu | bauzas: that is a chance for people change the the constraints | 10:50 |
| alex_xu | johnthetubaguy: yea, the use-case :) | 10:50 |
| johnthetubaguy | right, the idea is maintain the user's suggestions, or force a particular host | 10:50 |
| bauzas | alex_xu: but that should be a clear claim that you're probably breaking something | 10:50 |
| johnthetubaguy | alex_xu: I don't like the admin changing the user's scheduler hints | 10:50 |
| bauzas | alex_xu: resize is probably a specific case, because you're going to ask a different constraint | 10:51 |
| bauzas | johnthetubaguy: ^ | 10:51 |
| alex_xu | johnthetubaguy: emm...good point, admin shouldn't change it, looks like I need think about it | 10:51 |
| johnthetubaguy | alex_xu: now if there were a live-resize, that allowed resize to the same flavor, that could have a user scheduler hint, for when an end user wants to move things | 10:51 |
| bauzas | johnthetubaguy: yeah that § | 10:52 |
| *** markvoelker has quit IRC | 10:52 | |
| johnthetubaguy | like, move server groups, for example | 10:52 |
| johnthetubaguy | that could be a live-resize | 10:52 |
| johnthetubaguy | maybe... | 10:52 |
| alex_xu | bauzas: yea, I missed the point about admin-api | 10:52 |
| johnthetubaguy | but its very anti-cloud | 10:52 |
| bauzas | ++ | 10:52 |
| bauzas | if you want to change the user constraints, it's a resize | 10:53 |
| bauzas | or a destructive rebuild, rather | 10:53 |
| bauzas | but that's furiously anti-cloud as johnthetubaguy said | 10:53 |
| *** mubirru has quit IRC | 10:53 | |
| *** andreykurilin__ has joined #openstack-nova | 10:54 | |
| bauzas | alex_xu: so, see, I think the main problem we have is the taxonomy we have on the API side about all our move operations | 10:54 |
| bauzas | ie. we're not clear at all | 10:54 |
| *** sdague has joined #openstack-nova | 10:55 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: object: Adds host_capabilities in ComputeNode model https://review.openstack.org/228280 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds host_capabilities to HostState https://review.openstack.org/254747 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: scheduler: Adds HostCapabilitiesFilter filter https://review.openstack.org/255073 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds HostCapabilities object model https://review.openstack.org/285856 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds ResourceTracker host capabilities update https://review.openstack.org/250166 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: driver: Adds get_hypervisor_capabilities method https://review.openstack.org/254124 | 10:55 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: db: Adds host_capabilities column in compute_nodes https://review.openstack.org/228279 | 10:55 |
| alex_xu | bauzas: yeah | 10:55 |
| bauzas | alex_xu: but TBC, changing or adding scheduler hints is destructive | 10:55 |
| *** daemontool__ has joined #openstack-nova | 10:56 | |
| bauzas | alex_xu: like if you're changing your flavor | 10:56 |
| *** flaper87 has quit IRC | 10:56 | |
| *** flaper87 has joined #openstack-nova | 10:56 | |
| alex_xu | bauzas: got it | 10:56 |
| alex_xu | johnthetubaguy: bauzas thanks a lot for helping me clear my mind, let me re-think of it. | 10:57 |
| bauzas | alex_xu: so, why not having something like "non-conservative rebuild" which would allow you to change something | 10:57 |
| *** daemontool has quit IRC | 10:57 | |
| bauzas | but still I sucks at naming, that "non-conservative rebuild" is certainly not a good name | 10:58 |
| alex_xu | bauzas: emm...what is mean non-conservative rebuild? | 10:58 |
| alex_xu | bauzas: ok, i got you, another name | 10:58 |
| bauzas | rebuild is just "replay", right | 10:58 |
| *** cdent has joined #openstack-nova | 10:58 | |
| bauzas | ? | 10:58 |
| johnthetubaguy | bauzas: so rebuild already allows the changing of the image | 10:58 |
| *** hoangcx has quit IRC | 10:59 | |
| alex_xu | may i ask what is the use-case for rebuild? | 10:59 |
| *** amotoki has quit IRC | 11:00 | |
| bauzas | johnthetubaguy: and resize does that for a flavor, I know :) | 11:00 |
| bauzas | johnthetubaguy: so, see my point is that all of that is just linked | 11:00 |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| *** rk4n_ has joined #openstack-nova | 11:01 | |
| bauzas | johnthetubaguy: from a user PoV, it's "I want to change something by *that | 11:02 |
| bauzas | for that specific instance | 11:02 |
| johnthetubaguy | bauzas: true | 11:03 |
| bauzas | or rather, "I want to replay the instance creation with that modified parameter" because either I want a new image with security fixes, or because I want this image but bigger in size | 11:03 |
| *** amotoki has joined #openstack-nova | 11:03 | |
| PaulMurray | alex_xu, bauzas if an admin wants to move a vm at the moment he can say "move it anywhere" or "move it here" | 11:03 |
| bauzas | PaulMurray: yup, and I want to keep that | 11:04 |
| PaulMurray | alex_xu, bauzas but he might want to say "move it to anywhere that uses the same shared storage" | 11:04 |
| *** rk4n has quit IRC | 11:04 | |
| PaulMurray | ? | 11:04 |
| alex_xu | PaulMurray: yes | 11:04 |
| bauzas | PaulMurray: you truly admit it can only be an admin thing, right ? | 11:04 |
| PaulMurray | bauzas, admins are the only ones that can chose to move a vm | 11:05 |
| alex_xu | bauzas: thinking of we have some hints is for admin-only | 11:05 |
| bauzas | PaulMurray: so, the problem is that shared storage is not seen as a constraint as of now | 11:05 |
| *** thorst has joined #openstack-nova | 11:06 | |
| bauzas | PaulMurray: but that's a constraint actually, so the admin shouldn't even specify that | 11:06 |
| PaulMurray | bauzas, why? | 11:06 |
| bauzas | PaulMurray: because that's a migration if the destination is not on a same storage, not a move | 11:07 |
| bauzas | migrating an instance is not moving | 11:07 |
| PaulMurray | bauzas, migration is a move - ok, lets say migration | 11:08 |
| PaulMurray | the admin wants to migrate a vm to a host that uses the same shared storage | 11:09 |
| PaulMurray | bauzas, why do you say migration is not a move? | 11:09 |
| bauzas | PaulMurray: and so, you think he should express that when moving ? | 11:09 |
| bauzas | or migrating | 11:09 |
| bauzas | PaulMurray: I just feel that we are not clear on the concepts | 11:10 |
| *** thorst has quit IRC | 11:11 | |
| *** amotoki has quit IRC | 11:11 | |
| PaulMurray | bauzas, users are concerned with definition of the server (and maybe server groups and aggregates etc) | 11:12 |
| *** xek has joined #openstack-nova | 11:12 | |
| PaulMurray | bauzas, admins are concerned (in some way) with operating an infrastructure | 11:12 |
| bauzas | PaulMurray: right, that's what I call user-defined constraints (that are necessarly explicit) vs. admin-defined constraints (that can be either explicit or implicit - thru flavor extra specs or image props) | 11:13 |
| bauzas | PaulMurray: right, so say that you have an instance on a shared storage, is that a constraint or just something that you ended up without asking for it ? | 11:14 |
| bauzas | PaulMurray: that can be both, agreed ? | 11:15 |
| PaulMurray | bauzas, yes | 11:15 |
| bauzas | PaulMurray: so, you want now to say that was probably not a constraint will become a constraint when you'll move that instance ? | 11:15 |
| bauzas | PaulMurray: if so, what if the user wants to resize ? | 11:16 |
| PaulMurray | bauzas, that's different | 11:16 |
| bauzas | PaulMurray: is that becoming a constraint for the subsequent resize ? | 11:16 |
| bauzas | PaulMurray: why ? | 11:16 |
| bauzas | why is that different ? | 11:16 |
| PaulMurray | bauzas, because the user doesn't get to select how things are done | 11:17 |
| PaulMurray | bauzas, so I agree with what you say from the user perspective | 11:17 |
| PaulMurray | bauzas, I'm exploring what the admin may need to be able to do | 11:17 |
| bauzas | PaulMurray: so you want to say "for that specific admin move, I'll get you some extra bits for scheduling that you should forget once that move is done" right? | 11:18 |
| PaulMurray | bauzas, pretty much - for example | 11:18 |
| openstackgerrit | baikai proposed openstack/nova: Sync up instance object with nova db before unshelve operation done. https://review.openstack.org/283418 | 11:18 |
| PaulMurray | bauzas, a migration could go to a different shared storage - which will require a block migration | 11:19 |
| PaulMurray | bauzas, but the admin might want to restrict it to the same shared storage to avoid that overhead | 11:19 |
| sdague | moring folks | 11:19 |
| *** _RuiChen has quit IRC | 11:19 | |
| bauzas | PaulMurray: well, I see that usecase as just a suboptimal placement | 11:20 |
| PaulMurray | bauzas, at another time he might specifically want to migrate vms off the shared storage so it can be decommisioned | 11:20 |
| bauzas | PaulMurray: ah | 11:20 |
| *** RuiChen has joined #openstack-nova | 11:20 | |
| PaulMurray | bauzas, that second one could be done like marking hosts as disabled I suppose | 11:20 |
| bauzas | PaulMurray: well, all of that sounds like "move my instance by changing this parameter" | 11:21 |
| bauzas | like a resize or a rebuild, but instead of changing a flavor or an image, you are asking for a 3rd item | 11:22 |
| bauzas | PaulMurray: sorry, misexplained, it's rather "move my instance by changing that constraint" (the constraint being the image, flavor, something else) | 11:23 |
| *** daemontool_ has joined #openstack-nova | 11:23 | |
| *** achanda has joined #openstack-nova | 11:24 | |
| PaulMurray | bauzas, might be useful to think about the things that affect scheduling at the moment and see if they do the job | 11:25 |
| openstackgerrit | Sean Dague proposed openstack/nova: complete the removal of api_version from rest client parameters https://review.openstack.org/284257 | 11:25 |
| PaulMurray | bauzas, e.g. aggregates can be changed, that will affect the scheduling without the vm filter properties changing | 11:25 |
| *** davideagle has quit IRC | 11:25 | |
| bauzas | PaulMurray: the real problem is that you know we're bad with shared storage | 11:26 |
| PaulMurray | :) | 11:26 |
| bauzas | PaulMurray: so take the aggregate example | 11:26 |
| bauzas | PaulMurray: and let's stop focusing on the shared storage problem | 11:26 |
| *** davideagle has joined #openstack-nova | 11:27 | |
| *** daemontool__ has quit IRC | 11:27 | |
| bauzas | PaulMurray: if you're changing an host from one aggregate to another, it will just check the instance AZs, but not if that's breaking an aggregate-related filter (eg. AggregateInstanceExtraSpecs) | 11:28 |
| bauzas | PaulMurray: but if you're going to move one instance from that host to another, it will verify again that | 11:28 |
| *** rk4n_ has quit IRC | 11:28 | |
| *** achanda has quit IRC | 11:29 | |
| bauzas | PaulMurray: and make sure that you're not violating the rules | 11:29 |
| *** dims has joined #openstack-nova | 11:29 | |
| bauzas | PaulMurray: so, say that for maintenance purpose, you know that you don't want to match the aggregate vs. extra specs, then you would like *as an admin* for that specific move to disable the extra specs | 11:30 |
| bauzas | PaulMurray: but IMHO, that's really hard from an UX PoV | 11:31 |
| bauzas | PaulMurray: my guts on that is that operators want to move instances to a list of hosts | 11:31 |
| *** omnipresent has joined #openstack-nova | 11:31 | |
| PaulMurray | bauzas, yes I agree | 11:32 |
| *** dims has quit IRC | 11:32 | |
| bauzas | PaulMurray: because they know those hosts are correct, they just want to make sure it's okay | 11:32 |
| PaulMurray | bauzas, aggregates are used (in some way or other) to maintain a pool of hosts that are used for a given purpose | 11:32 |
| *** dims has joined #openstack-nova | 11:32 | |
| PaulMurray | bauzas, and that pool can be updated dynamically | 11:32 |
| bauzas | PaulMurray: so it's rather live-migrate myinstance [--force] [dest1, dest2...] | 11:33 |
| openstackgerrit | pallavi proposed openstack/nova: Update time is not updated when metadata of aggregate is updated https://review.openstack.org/284023 | 11:33 |
| *** Guest22888 is now known as beagles | 11:33 | |
| *** rk4n has joined #openstack-nova | 11:33 | |
| PaulMurray | bauzas, that loses some expressiveness | 11:33 |
| eliqiao | andreykurilin: hi | 11:34 |
| PaulMurray | bauzas, I was rather wondering if changing an aggregate instead of specifying the "--force" might do the job | 11:34 |
| bauzas | PaulMurray: I agree, but adding more ways to refine your move would necessarly mean to match with how the scheduler is filtering | 11:34 |
| *** zenoway has quit IRC | 11:34 | |
| PaulMurray | bauzas, I need to do something else for a while - perhaps gathering some use cases and see how they can be done will be a good exercise | 11:35 |
| bauzas | PaulMurray: well, you'd rather like something like "nova live-migrate myinstance myagg" | 11:36 |
| *** omnipresent has quit IRC | 11:36 | |
| *** liverpooler has quit IRC | 11:36 | |
| *** Daisy has joined #openstack-nova | 11:36 | |
| bauzas | PaulMurray: and then you could express your needs in the agg metadata | 11:36 |
| *** liverpooler has joined #openstack-nova | 11:37 | |
| *** zenoway has joined #openstack-nova | 11:37 | |
| bauzas | PaulMurray: in that case, the scheduler will consider that "destination" exactly what I'd like for http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/check-destination-on-migrations.html | 11:37 |
| bauzas | PaulMurray: that sounds doable | 11:38 |
| PaulMurray | bauzas, hmm - so kind of "include this aggregate for this operation"? | 11:38 |
| bauzas | PaulMurray: no | 11:38 |
| bauzas | PaulMurray: it would be "scheduler, just run the filters against that subset of hosts" | 11:39 |
| bauzas | PaulMurray: since the aggregate-related filters would look up the aggregate metadata, you'd be sure to pass some extra bits in there | 11:39 |
| PaulMurray | bauzas, oh, so literally that set of hosts, not just adding some extra data to those hosts (which is what aggregates normally do) | 11:40 |
| bauzas | PaulMurray: yes | 11:41 |
| *** omnipresent has joined #openstack-nova | 11:42 | |
| *** ekuris_ has quit IRC | 11:42 | |
| alex_xu | bauzas: thinking of the thing about live-migrate instance to newer host for rolling-upgrae, how did you tell scheduler find newer host or not? | 11:42 |
| bauzas | PaulMurray: so back to your concern, you could create an aggregate grouping all your hosts sharing the same storage | 11:42 |
| eliqiao | hi johnthetubaguy seam GM, are you convenient to take a look at microver 2.25 patch start from https://review.openstack.org/#/c/254102/ | 11:42 |
| eliqiao | sorry, I meant sdague (instead of seam) | 11:43 |
| bauzas | alex_xu: so I have a draft BP that I should show up to johnthetubaguy https://blueprints.launchpad.net/nova/+spec/scheduler-filter-service-version | 11:43 |
| johnthetubaguy | eliqiao: oh, good point, I do need to take a look at that again, now its rebased | 11:44 |
| *** zenoway has quit IRC | 11:44 | |
| alex_xu | bauzas: so that need scheduler-hints flag for lm? | 11:44 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: os-vif: introduce a ComputeInfo object to represent compute info https://review.openstack.org/282436 | 11:45 |
| bauzas | alex_xu: why ? no | 11:45 |
| bauzas | alex_xu: ah I see your question | 11:45 |
| bauzas | alex_xu: well, no, I should remove that phrase, it's not good | 11:46 |
| alex_xu | bauzas: yes, that sounds like admin-only hint, and can be used in admin-only migration | 11:46 |
| *** Daisy has quit IRC | 11:46 | |
| bauzas | oh man | 11:46 |
| * bauzas is having a terrible headache | 11:46 | |
| *** Daisy has joined #openstack-nova | 11:46 | |
| *** zenoway has joined #openstack-nova | 11:48 | |
| *** raildo-afk is now known as raildo | 11:49 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix Cells RPC API by accepting a RequestSpec arg https://review.openstack.org/284959 | 11:50 |
| bauzas | johnthetubaguy: ^ | 11:51 |
| *** Daisy has quit IRC | 11:51 | |
| bauzas | johnthetubaguy: lemme know if that sounds good for you | 11:51 |
| *** haukebruno has quit IRC | 11:52 | |
| *** omnipresent has quit IRC | 11:52 | |
| *** omnipresent has joined #openstack-nova | 11:54 | |
| bauzas | alex_xu: I clarified the target of that bp in https://blueprints.launchpad.net/nova/+spec/scheduler-filter-service-version | 11:54 |
| *** roeyc has joined #openstack-nova | 11:54 | |
| gcb | alex_xu, bauzas , Do you have time to look at simple oslo library replacement commits, we'd better get them merged in Mitaka ? | 11:54 |
| gcb | https://review.openstack.org/#/c/257232/ Use imageutils from oslo.utils | 11:54 |
| bauzas | alex_xu: if check-destination-on-migrations is implemented, operator could ask to live-migrate to an host by proposing a destination that would cope with that filter | 11:55 |
| gcb | Use constant_time_compare from oslo.utils https://review.openstack.org/#/c/284635/ | 11:55 |
| *** dave-mccowan has joined #openstack-nova | 11:55 | |
| *** marcusvrn_ has joined #openstack-nova | 11:56 | |
| bauzas | gcb: we'll be cutting M-3 in 3 days, isn't that a bit risky? | 11:56 |
| gcb | bauzas: no risk , just simple replacement | 11:56 |
| *** sudipto has quit IRC | 11:56 | |
| gcb | same code with we have in nova | 11:56 |
| *** zenoway has quit IRC | 11:57 | |
| alex_xu | bauzas: you mean that is default behaviour, always move to newer host? | 11:58 |
| *** htruta` is now known as htruta | 11:58 | |
| openstackgerrit | Eli Qiao proposed openstack/python-novaclient: [microversion] Bump to 2.25 https://review.openstack.org/260383 | 11:58 |
| *** shh_away has joined #openstack-nova | 11:58 | |
| bauzas | alex_xu: if the new filter I'm going to write is enabled, yes | 11:58 |
| eliqiao | bauzas: but admin need to understand which host are newer host. | 11:59 |
| bauzas | alex_xu: but atm, (correct me if I'm wrong), you can still create some instances on an old compute | 11:59 |
| eliqiao | bauzas: if there are many many hosts, admin will be crazy. | 11:59 |
| bauzas | eliqiao: so, that's another problem that I discussed with PaulMurray 30 mins ago :) | 12:00 |
| eliqiao | bauzas: oh, yeah, thanks for point me out, I will read the log. | 12:00 |
| *** davideagle has quit IRC | 12:00 | |
| cdent | bauzas, dansmith, jaypipes, johnthetubaguy: There's a fair few questions laying about on this stack of patches, can people get in there for a look before the scheduler meeting so we're all aware of the state: https://review.openstack.org/#/c/281837/ | 12:00 |
| alex_xu | bauzas: yes, if the filter enabled, all new instance will go to the new host | 12:00 |
| *** haomaiwang has quit IRC | 12:01 | |
| bauzas | eliqiao: tl;dr it's hard to say to have hints for live-migrations, because that's destructive, I'd rather propose to pass an aggregate as a proposed destination | 12:01 |
| bauzas | cdent: oh man oh man | 12:01 |
| alex_xu | gcb: i will try to check that | 12:01 |
| gcb | alex_xu, bauzas: thanks for your time :-) | 12:01 |
| * bauzas moving out of keyboard | 12:01 | |
| *** haomaiwa_ has joined #openstack-nova | 12:02 | |
| eliqiao | bauzas: hmm... what do you mean bay destructive? Can you take an example? | 12:02 |
| cdent | bauzas i don't think it is too super "oh man oh man" just a bit of attention required so we can make sure we have the migrations and objects in for M. I'm going to change locations, but will check back in .5 hour | 12:03 |
| eliqiao | s/bay/by | 12:03 |
| alex_xu | eliqiao: you will get that from the log 30 mins ago :) | 12:03 |
| *** omnipresent has quit IRC | 12:03 | |
| eliqiao | alex_xu: oh man, thanks man. | 12:03 |
| *** cdent has quit IRC | 12:03 | |
| *** zenoway has joined #openstack-nova | 12:04 | |
| *** haukebruno has joined #openstack-nova | 12:06 | |
| *** lucasagomes is now known as lucas-hungry | 12:06 | |
| -openstackstatus- NOTICE: Infra currently has a long backlog. Please be patient and where possible avoid rechecks while it catches up. | 12:06 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: scheduler: removes check for supported_instances from filter https://review.openstack.org/285998 | 12:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: drivers: Removes supported_instances from get_available_resource https://review.openstack.org/285999 | 12:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: objects: Removes supported_instances from compute_node https://review.openstack.org/286000 | 12:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: db: Removes supported_instances from compute_node table https://review.openstack.org/286001 | 12:07 |
| *** smatzek has joined #openstack-nova | 12:08 | |
| *** rfolco has joined #openstack-nova | 12:08 | |
| *** omnipresent has joined #openstack-nova | 12:09 | |
| *** ndipanov has quit IRC | 12:10 | |
| PaulMurray | eliqiao, see https://review.openstack.org/#/c/259319/21/nova/tests/functional/api_sample_tests/test_migrate_server.py | 12:11 |
| eliqiao | PaulMurray: sure. | 12:11 |
| PaulMurray | eliqiao, do you mean to override that test method with one that doesn't do anything? | 12:12 |
| eliqiao | PaulMurray: yes, the thing is. v2.25 don't change anything of *cold* migration. | 12:13 |
| *** zenoway has quit IRC | 12:13 | |
| eliqiao | PaulMurray: and If I don't 'pass' that case, it will require another sample file in https://review.openstack.org/#/c/259319/21/nova/tests/functional/api_sample_tests/api_samples/os-migrate-server/v2.25/ | 12:13 |
| eliqiao | PaulMurray: but that sample file has nothing changes compared with v2.24 version. | 12:14 |
| PaulMurray | eliqiao, ok | 12:16 |
| *** EinstCrazy has joined #openstack-nova | 12:17 | |
| *** daemontool_ has quit IRC | 12:18 | |
| *** mingyu has quit IRC | 12:19 | |
| *** mingyu has joined #openstack-nova | 12:19 | |
| *** daemontool has joined #openstack-nova | 12:20 | |
| *** baoli has joined #openstack-nova | 12:22 | |
| *** baoli has quit IRC | 12:22 | |
| *** achanda has joined #openstack-nova | 12:26 | |
| *** cdent has joined #openstack-nova | 12:27 | |
| *** Mackie2k has joined #openstack-nova | 12:29 | |
| *** amotoki has joined #openstack-nova | 12:30 | |
| *** achanda has quit IRC | 12:30 | |
| openstackgerrit | esracelik proposed openstack/nova: Improve the help text for cells options (7) https://review.openstack.org/278927 | 12:31 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Remove unused variable and redundant code path https://review.openstack.org/286011 | 12:32 |
| *** thorst has joined #openstack-nova | 12:32 | |
| *** daemontool has quit IRC | 12:32 | |
| *** jaosorior has quit IRC | 12:33 | |
| *** jaosorior has joined #openstack-nova | 12:34 | |
| *** daemontool has joined #openstack-nova | 12:34 | |
| ankit_ag | sdauge: hi | 12:36 |
| ankit_ag | Could you please check Tushar's reply on https://review.openstack.org/#/c/221162/ | 12:36 |
| ankit_ag | I have also verified it and it does not raises key error on bdms | 12:36 |
| *** jschlueter has quit IRC | 12:38 | |
| *** mingyu has quit IRC | 12:38 | |
| *** mingyu has joined #openstack-nova | 12:38 | |
| *** rk4n_ has joined #openstack-nova | 12:39 | |
| *** mingyu has quit IRC | 12:39 | |
| *** mingyu has joined #openstack-nova | 12:39 | |
| *** davideagle has joined #openstack-nova | 12:40 | |
| *** wanghao has quit IRC | 12:41 | |
| sdague | ankit_ag: thanks | 12:42 |
| *** rk4n has quit IRC | 12:42 | |
| *** haomaiwa_ has quit IRC | 12:42 | |
| *** rtheis has joined #openstack-nova | 12:42 | |
| *** wanghao has joined #openstack-nova | 12:42 | |
| *** haomaiwang has joined #openstack-nova | 12:43 | |
| *** haomaiwang has quit IRC | 12:43 | |
| ankit_ag | sdauge: Thank you | 12:43 |
| *** haomaiwa_ has joined #openstack-nova | 12:43 | |
| *** haomaiwa_ has quit IRC | 12:43 | |
| *** chhavi has joined #openstack-nova | 12:44 | |
| *** amotoki has quit IRC | 12:45 | |
| *** Guest33875 has quit IRC | 12:45 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for migration. https://review.openstack.org/241479 | 12:45 |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for evacuation. https://review.openstack.org/241478 | 12:45 |
| openstackgerrit | Tang Chen proposed openstack/nova: Implement state machine for resize. https://review.openstack.org/241477 | 12:45 |
| *** huats_ has joined #openstack-nova | 12:47 | |
| *** ljxiash has joined #openstack-nova | 12:48 | |
| *** markvoelker has joined #openstack-nova | 12:48 | |
| openstackgerrit | Wenzhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 12:51 |
| *** markvoelker has quit IRC | 12:53 | |
| *** pcaruana has quit IRC | 12:53 | |
| *** josecastroleon has joined #openstack-nova | 12:54 | |
| *** baoli has joined #openstack-nova | 12:55 | |
| *** daemontool has quit IRC | 12:55 | |
| *** daemontool has joined #openstack-nova | 12:55 | |
| *** daemontool has quit IRC | 12:56 | |
| *** shaohe_feng_ has joined #openstack-nova | 12:56 | |
| *** tbachman has joined #openstack-nova | 12:57 | |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: prototype filter based scheduler weights https://review.openstack.org/286023 | 12:58 |
| *** daemontool has joined #openstack-nova | 12:59 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Improvements for BaseCoreFilter https://review.openstack.org/285427 | 13:00 |
| *** gcb has quit IRC | 13:01 | |
| *** amotoki has joined #openstack-nova | 13:02 | |
| *** jinxing has quit IRC | 13:02 | |
| *** andreykurilin__ has quit IRC | 13:03 | |
| *** jinxing has joined #openstack-nova | 13:04 | |
| *** Oku_OS has joined #openstack-nova | 13:05 | |
| BobBall | johnthetubaguy: You aware of the critical regression in XenAPI live migrate? https://review.openstack.org/#/c/285741/1 I've set the bug to High since it only affects XenAPI, but I think it's a must-fix for Mitaka | 13:05 |
| *** lucas-hungry is now known as lucasagomes | 13:06 | |
| openstackgerrit | Wenzhi Yu proposed openstack/nova: Make sure destination will rollback if live migration fails https://review.openstack.org/270093 | 13:06 |
| *** wanghao has quit IRC | 13:07 | |
| *** wanghao has joined #openstack-nova | 13:08 | |
| *** davideagle has quit IRC | 13:08 | |
| *** pcaruana has joined #openstack-nova | 13:08 | |
| *** jinxing has quit IRC | 13:09 | |
| *** huats_ has quit IRC | 13:10 | |
| *** huats_ has joined #openstack-nova | 13:11 | |
| *** huats_ has quit IRC | 13:11 | |
| *** huats_ has joined #openstack-nova | 13:11 | |
| *** ociuhandu has quit IRC | 13:11 | |
| *** ljxiash has quit IRC | 13:12 | |
| *** haomaiwang has joined #openstack-nova | 13:12 | |
| *** Mackie2k has quit IRC | 13:13 | |
| *** rfolco_ has joined #openstack-nova | 13:14 | |
| *** toabctl has quit IRC | 13:14 | |
| *** takedakn has joined #openstack-nova | 13:16 | |
| *** rfolco has quit IRC | 13:16 | |
| *** links has quit IRC | 13:18 | |
| *** toabctl has joined #openstack-nova | 13:18 | |
| *** Mackie2k has joined #openstack-nova | 13:20 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Generate better validation error message when using name regexes https://review.openstack.org/281143 | 13:24 |
| *** edmondsw has joined #openstack-nova | 13:25 | |
| *** davideagle has joined #openstack-nova | 13:26 | |
| *** ivase has joined #openstack-nova | 13:28 | |
| johnthetubaguy | BobBall: yeah, coreywright pinged me about that | 13:28 |
| *** dmellado is now known as dmellado|lunch | 13:28 | |
| *** dmellado|lunch is now known as dmellado | 13:28 | |
| *** huats_ has quit IRC | 13:30 | |
| *** jschlueter has joined #openstack-nova | 13:30 | |
| *** mingyu has quit IRC | 13:31 | |
| *** ociuhandu has joined #openstack-nova | 13:31 | |
| *** huats_ has joined #openstack-nova | 13:31 | |
| *** n0ano has joined #openstack-nova | 13:33 | |
| *** ankit_ag has quit IRC | 13:35 | |
| *** takedakn has quit IRC | 13:37 | |
| *** rook has joined #openstack-nova | 13:38 | |
| *** amotoki has quit IRC | 13:39 | |
| *** tongli has joined #openstack-nova | 13:40 | |
| *** scottda_is_outta is now known as scottda | 13:41 | |
| *** liverpooler has quit IRC | 13:42 | |
| *** dedery has quit IRC | 13:43 | |
| *** Pistachio33 has joined #openstack-nova | 13:43 | |
| *** ekuris_ has joined #openstack-nova | 13:43 | |
| *** fawadkhaliq has joined #openstack-nova | 13:43 | |
| *** aysyd has joined #openstack-nova | 13:45 | |
| shaohe_feng_ | andreykurilin: hi | 13:47 |
| *** markvoelker has joined #openstack-nova | 13:47 | |
| *** baoli has quit IRC | 13:47 | |
| andreykurilin | shaohe_feng_: hi! I'm not sure that understood what is not working in your patch. Can you explain more? | 13:48 |
| *** daemontool has quit IRC | 13:48 | |
| shaohe_feng_ | andreykurilin: OK. | 13:48 |
| *** baoli has joined #openstack-nova | 13:48 | |
| shaohe_feng_ | andreykurilin: my patch also need to do another things. | 13:49 |
| shaohe_feng_ | andreykurilin: to change another REST API "os-migration" GET with version bump | 13:50 |
| shaohe_feng_ | andreykurilin: this so I need to api_version.wraps to decorate the bumped api, migration-list, right? | 13:51 |
| andreykurilin | shaohe_feng_: right | 13:51 |
| shaohe_feng_ | andreykurilin: then I need to add a test case for it. | 13:51 |
| *** aysyd has quit IRC | 13:52 | |
| andreykurilin | shaohe_feng_: right | 13:52 |
| *** amotoki has joined #openstack-nova | 13:53 | |
| *** achanda has joined #openstack-nova | 13:53 | |
| shaohe_feng_ | andreykurilin: but I find, the test case use FakeHTTPClient as Client, I have set one of the api_version as run_command params | 13:53 |
| shaohe_feng_ | andreykurilin: but I find the test case does not work. | 13:53 |
| shaohe_feng_ | andreykurilin: for the api_version does not pass to the FakeHTTPClient _cs_request. | 13:54 |
| *** daemontool has joined #openstack-nova | 13:54 | |
| shaohe_feng_ | andreykurilin: and I do not find any example similar to my patch. | 13:54 |
| *** RuiChen has quit IRC | 13:54 | |
| *** aysyd has joined #openstack-nova | 13:55 | |
| *** RuiChen has joined #openstack-nova | 13:55 | |
| shaohe_feng_ | andreykurilin: I did not know why the api_version does not pass to the _cs_request. | 13:55 |
| *** daemontool_ has joined #openstack-nova | 13:55 | |
| *** davideagle has quit IRC | 13:55 | |
| andreykurilin | shaohe_feng_: it should work. key-pairs tests use this parameter. | 13:56 |
| shaohe_feng_ | andreykurilin: I remember, I have check the key pairs code. | 13:58 |
| shaohe_feng_ | andreykurilin: some different, the os-migrations is in v2/contrib dir | 13:58 |
| *** links has joined #openstack-nova | 13:58 | |
| shaohe_feng_ | andreykurilin: so what's wrong with http://paste.openstack.org/show/488521 ? | 13:59 |
| *** daemontool has quit IRC | 13:59 | |
| shaohe_feng_ | andreykurilin: sorry, not this link | 14:00 |
| *** eglynn has joined #openstack-nova | 14:00 | |
| *** rlrossit has joined #openstack-nova | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** bswartz has joined #openstack-nova | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| andreykurilin | shaohe_feng_: it shouldn't depend on contrib dir. So I got you problem and will check it now | 14:01 |
| shaohe_feng_ | andreykurilin: https://paste.fedoraproject.org/331058/14567429 thank you. | 14:02 |
| *** tbachman has quit IRC | 14:02 | |
| *** vladikr has joined #openstack-nova | 14:06 | |
| *** busterswt has joined #openstack-nova | 14:06 | |
| *** davideagle has joined #openstack-nova | 14:06 | |
| *** yvesbationo has joined #openstack-nova | 14:08 | |
| *** MVenesio has joined #openstack-nova | 14:09 | |
| *** yvesbationo has left #openstack-nova | 14:10 | |
| *** mingyu has joined #openstack-nova | 14:11 | |
| *** mriedem has joined #openstack-nova | 14:11 | |
| *** zz_dimtruck is now known as dimtruck | 14:12 | |
| andreykurilin | shaohe_feng_: there is no need to transmit api_version to FakeHTTPClient. In real HTTPClient api_version parameter is just used for setting headers. Headers are not checked in our shell testcases . | 14:12 |
| *** achanda has quit IRC | 14:13 | |
| shaohe_feng_ | andreykurilin: then, How do I add the test case for the bump? | 14:14 |
| *** mingyu has quit IRC | 14:14 | |
| *** erno has quit IRC | 14:16 | |
| *** links has quit IRC | 14:16 | |
| andreykurilin | shaohe_feng_: one moment, will finish a sample:) | 14:16 |
| shaohe_feng_ | andreykurilin: ok, thank you. | 14:17 |
| *** boris-42 has joined #openstack-nova | 14:17 | |
| *** mdrabe has joined #openstack-nova | 14:17 | |
| *** ljxiash has joined #openstack-nova | 14:20 | |
| *** cdelatte has joined #openstack-nova | 14:21 | |
| *** ekuris_ has quit IRC | 14:21 | |
| *** andrewbogott has quit IRC | 14:21 | |
| *** andrewbogott has joined #openstack-nova | 14:21 | |
| *** sudipto has joined #openstack-nova | 14:22 | |
| andreykurilin | shaohe_feng_: http://paste.openstack.org/show/488565/ In this example, I wrapped old list method with 2.0-2.22 versions, created new list method for 2.23(key for host is different) and added new testcase with another assert_called method | 14:23 |
| *** jichen has quit IRC | 14:23 | |
| *** sridharg has quit IRC | 14:23 | |
| *** ctrath has joined #openstack-nova | 14:24 | |
| openstackgerrit | Moshe Levi proposed openstack/nova-specs: Add scheduling with NIC capabilities https://review.openstack.org/286073 | 14:25 |
| *** andymaier has joined #openstack-nova | 14:26 | |
| shaohe_feng_ | andreykurilin: thank. how does the self.run_command can get my expected data? seems it is getting the old migrations data. | 14:26 |
| mriedem | tonyb: i'm happy with https://review.openstack.org/#/c/281341/ if you are, xenserver ci check was +1 | 14:27 |
| *** Pistachio33 has quit IRC | 14:30 | |
| *** Pistachio has quit IRC | 14:31 | |
| *** smatzek has quit IRC | 14:33 | |
| *** mylu has joined #openstack-nova | 14:34 | |
| claudiub | johnthetubaguy: hello. I've resubmitted previously approved specs to newton. Could you take a look at them pls? https://review.openstack.org/#/c/279045/ https://review.openstack.org/#/c/279044/ https://review.openstack.org/#/c/279043/ https://review.openstack.org/#/c/279042/ https://review.openstack.org/#/c/279041/ | 14:34 |
| *** eharney has joined #openstack-nova | 14:34 | |
| mriedem | claudiub: how do i recheck the hyper-v ci again? | 14:36 |
| mriedem | claudiub: i wanted to run it on https://review.openstack.org/#/c/285370/ | 14:36 |
| claudiub | mriedem: check hyper-v | 14:36 |
| mriedem | ok, do you know if it will run on ^ if i request it? or is it disabled? | 14:37 |
| *** dimtruck is now known as zz_dimtruck | 14:37 | |
| claudiub | mriedem: yeah, it'll run. | 14:37 |
| andreykurilin | shaohe_feng_: ok. In this case, you are right about transmitting api_version to FakeHTTPClient - http://paste.openstack.org/show/488567/ | 14:37 |
| mriedem | ok, cool | 14:37 |
| claudiub | mriedem: no reason why it won't run. | 14:37 |
| *** mylu has quit IRC | 14:38 | |
| mriedem | claudiub: also, does the hyper-v test live migration? | 14:38 |
| mriedem | *ci | 14:38 |
| *** pcaruana has quit IRC | 14:38 | |
| shaohe_feng_ | andreykurilin: yes, great. thank you very much. | 14:39 |
| *** zz_dimtruck is now known as dimtruck | 14:39 | |
| *** gszasz has quit IRC | 14:39 | |
| *** esracelik has quit IRC | 14:39 | |
| *** gszasz has joined #openstack-nova | 14:40 | |
| andreykurilin | shaohe_feng_: you are welcome | 14:40 |
| claudiub | it does. apparently, it wasn't for a while because there was a new config option introduced in tempest, specifying the number of compute nodes (default = 1). The test was being skipped because of this. | 14:40 |
| mriedem | ah and when you fixed that it failed b/c of the regression in nova | 14:40 |
| shaohe_feng_ | andreykurilin: that's really we need. :) | 14:41 |
| claudiub | mriedem: indeed. | 14:41 |
| shaohe_feng_ | andreykurilin: I think this can be an example for the following similar patches. | 14:41 |
| *** burt has joined #openstack-nova | 14:42 | |
| *** mylu has joined #openstack-nova | 14:42 | |
| *** knikolla has joined #openstack-nova | 14:42 | |
| shaohe_feng_ | andreykurilin: the following similar patch can ref your solution. really good. | 14:42 |
| andreykurilin | shaohe_feng_: I suppose your patch can be an example in future ;) | 14:45 |
| *** tbachman has joined #openstack-nova | 14:45 | |
| *** READ10 has joined #openstack-nova | 14:45 | |
| shaohe_feng_ | andreykurilin: yes. it should be. thanks. I will update the patch and re-submit. | 14:46 |
| *** dimtruck is now known as zz_dimtruck | 14:47 | |
| cdent | Yingxin: I've got to step away for about five minutes, but when I get back we can talk resource-* | 14:47 |
| Yingxin | cdent: np, thanks | 14:48 |
| openstackgerrit | Sergey Belous proposed openstack/nova: do not merge, trying to run tempest with enabled os-vif https://review.openstack.org/286086 | 14:48 |
| *** mrkz has joined #openstack-nova | 14:49 | |
| *** pradk has joined #openstack-nova | 14:50 | |
| *** wanghao has quit IRC | 14:51 | |
| johnthetubaguy | claudiub: afraid I will not get to many specs till we hit RC-1, snowed under with code reviews right now | 14:51 |
| *** gcb has joined #openstack-nova | 14:51 | |
| *** wanghao has joined #openstack-nova | 14:51 | |
| *** pcaruana has joined #openstack-nova | 14:52 | |
| *** ndipanov has joined #openstack-nova | 14:52 | |
| *** edtubill has joined #openstack-nova | 14:53 | |
| *** ducttape_ has joined #openstack-nova | 14:53 | |
| bauzas | cdent: so, now that we're off the meeting, we all agree on having that https://review.openstack.org/#/c/279313 implemented in Mitaka to prevent online migrations, right? | 14:54 |
| bauzas | dansmith: you agree ? | 14:54 |
| *** yamahata_ has quit IRC | 14:55 | |
| bauzas | if the compute node is not able to persist information on a fresh new style, then we would need to access that otherwise and populate it online, nope? | 14:55 |
| dansmith | bauzas: that's the point I wanted to get to yeah.. I was just looking for the deadline to see if we're still good or not.. I thought it was friday. | 14:55 |
| *** rk4n_ has quit IRC | 14:55 | |
| cdent | bauzas, dansmith: I defer to your guys' expertise, but my understanding is "yes we want that" | 14:56 |
| cdent | Yingxin: I'm back now. | 14:56 |
| *** rk4n has joined #openstack-nova | 14:56 | |
| *** doug-fish has joined #openstack-nova | 14:56 | |
| *** cfcp1 has joined #openstack-nova | 14:56 | |
| Yingxin | cdent: If I'm correct, the resource_provider.uuid is equal to compute_node.uuid if the provider is that compute node. | 14:56 |
| bauzas | dansmith: cdent: just to be sure, object remotable methods accessing both tables are in the previous commits of the branch, ie. https://review.openstack.org/#/c/284484/ and https://review.openstack.org/#/c/284833/ ? | 14:57 |
| cdent | Yingxin: that's correct | 14:57 |
| Yingxin | cdent: But what does the resource_provider.uuid mean when it refers to a shared resource pool? | 14:57 |
| bauzas | cdent: dansmith: I just want to make sure that we're not forgetting any object accessor | 14:57 |
| cdent | Yingxin: it is the uuid of the that resource, for example as shared disk server somewhere. The admin is expected to provide the uuid when created the resource provider | 14:58 |
| dansmith | bauzas: I'm not sure what you're asking.. those two commits are db_api methods to insulate the object from the fact that we're querying two tables | 14:58 |
| PaulMurray | shaohe_feng_, I'm not sure where you are with https://review.openstack.org/#/c/281335/8 - we can continue it for you ? | 14:58 |
| bauzas | Yingxin: then it would be the aggregate uuid AFAICU | 14:58 |
| PaulMurray | pkoniszewski, andreykurilin ^^ do you know what is happening with shaohe_feng_'s patch at the moment ? | 14:58 |
| Yingxin | bauzas: but there is a many-many relationship between provider and aggregates. | 14:59 |
| bauzas | dansmith: sorry I wasn't clear | 14:59 |
| andreykurilin | PaulMurray: yes:) | 14:59 |
| pkoniszewski | PaulMurray: let me try to ask him | 14:59 |
| bauzas | dansmith: so, object remotable methods are executed server-side (ie. possibly on old computes) | 14:59 |
| cdent | bauzas: no, it's not the aggregate uuid | 14:59 |
| cdent | the aggregate uuid is for the aggregate | 14:59 |
| dansmith | bauzas: server-side is conductor for remotables | 14:59 |
| bauzas | dansmith: I just want to make sure that we have those methods in the tree | 14:59 |
| andreykurilin | PaulMurray: we had a talk with him and it looks like shaohe_feng_ is working on new patch-set | 14:59 |
| Yingxin | cdent: so I don't know the meaning of aggregate uuid | 15:00 |
| cdent | a resource provider is associated with an aggregate accross the reosurceprovieraggregates table | 15:00 |
| bauzas | dansmith: oh, fsck, yeah, you're right, so we're good | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| shaohe_feng_ | PaulMurray: I'm here | 15:01 |
| dansmith | bauzas: :) | 15:01 |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| shaohe_feng_ | PaulMurray: WIP | 15:01 |
| bauzas | dansmith: that's what happens when you take a PTO | 15:01 |
| *** EinstCra_ has joined #openstack-nova | 15:01 | |
| cdent | Yingxin: the aggregate uuid is newly added to the aggregate table to allay concerns about aggregates being easy to migrate out of a individual cells into the some other db | 15:01 |
| PaulMurray | shaohe_feng_, ok - good, just trying to keep track | 15:02 |
| dansmith | bauzas: you forget everything after one day of PTO? Makes me wonder: what did you do on your PTO? :) | 15:02 |
| dansmith | bauzas: must've been a good time :D | 15:02 |
| cdent | Yingxin: In terms of understanding how they are involved in a resource pools this spec is the one: https://review.openstack.org/#/c/253187/ | 15:02 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Clean up networks with SR-IOV binding on reschedule https://review.openstack.org/283198 | 15:02 |
| bauzas | dansmith: playing with children, so you know, it's mostly braindraining | 15:02 |
| *** EinstCra_ has quit IRC | 15:02 | |
| dansmith | bauzas: +1000 | 15:02 |
| *** EinstCrazy has quit IRC | 15:02 | |
| *** smatzek has joined #openstack-nova | 15:02 | |
| *** lennyb__ is now known as lennyb | 15:03 | |
| shaohe_feng_ | PaulMurray: really thank you. | 15:03 |
| *** EinstCrazy has joined #openstack-nova | 15:03 | |
| Yingxin | cdent: so aggregate.uuid is not the resource provider things, and resource_provider.uuid cannot be aggregate.uuid? | 15:04 |
| cdent | Yingxin: correct | 15:05 |
| ZZelle | Hi everyone, | 15:06 |
| ZZelle | i have a trouble with vmware backend: VCenter returns shutdown status for running vms when under heavy load | 15:06 |
| mriedem | tjones_zzzz: is this a known issue with the vmware ci? http://208.91.1.172/logs/nova-vc6-dsvm-tempest/190627/7/2665/g-api.log.gz?level=TRACE | 15:07 |
| mriedem | getting a 503 while uploading an image to the vmware datastore | 15:07 |
| Yingxin | cdent: I was confused because this patch is in the bp/generic-resource-pool series https://review.openstack.org/#/c/282519/ | 15:07 |
| jaosorior | Does anyone have time to review this? https://review.openstack.org/#/c/248662/ Would be nice to fix that regression :D | 15:08 |
| cdent | Yingxin: dansmith is very bad at writing useful commit messages. I'm surprised he's allowed to get away with it. | 15:08 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 15:08 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 15:08 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 15:08 |
| mriedem | jaosorior: should probably have sdague look at that one | 15:08 |
| *** davideagle has quit IRC | 15:08 | |
| jaosorior | mriedem: Will add him to the review, thanks! | 15:09 |
| Yingxin | cdent: :-) no messages at all | 15:09 |
| mriedem | i already did | 15:09 |
| dansmith | cdent: dude, I usually write pretty long ones. That one got away from me, but ... seriously | 15:09 |
| * cdent passes dansmith some of his sense of humor | 15:10 | |
| mriedem | jaosorior: so by 'changed' that means, as in the v2.1 API is not the same as the v2 API | 15:10 |
| jaosorior | mriedem: That is the case, yes | 15:10 |
| cfriesen_ | dansmith: I agree, that one is too long. | 15:11 |
| johnthetubaguy | cdent: so can I sync on what we want in mitaka, and what we need to leave for newton, worried we are not all focusing on the same things | 15:11 |
| johnthetubaguy | cdent: so it seems like we need this patch, and all its dependencies: https://review.openstack.org/#/c/279313/ | 15:11 |
| cdent | yes | 15:11 |
| johnthetubaguy | cdent: thats because, when you do a live-upgrade, old and new computes, for the next release, we want to be able to assume all the data is in the correct position already | 15:12 |
| cdent | I think we also need this one and its dependencies: https://review.openstack.org/#/c/284963/ | 15:12 |
| johnthetubaguy | cdent: why do we need that one? | 15:12 |
| dansmith | I don't think we need that one | 15:12 |
| cdent | because, as I understood things, we need the object to exist before going into the next cycle | 15:12 |
| dansmith | no | 15:12 |
| cdent | how about we _want_? | 15:13 |
| dansmith | no | 15:13 |
| * cdent sighs | 15:13 | |
| cdent | this is entirely different from what we agreed last week | 15:13 |
| dansmith | I have been saying all along that the allocation stuff is not mitaka stuff at all | 15:13 |
| dansmith | there's no reason for it at all | 15:13 |
| *** haukebruno has quit IRC | 15:13 | |
| dansmith | we have to get the inventory stuff done in mitaka so that we can do the allocation stuff in newton without a multi-step dependency process | 15:14 |
| cdent | dansmith: so you're saying that tree should stop at: https://review.openstack.org/#/c/284895/ ? | 15:14 |
| johnthetubaguy | its a move existings data issue, that stuff is all new data I think? | 15:14 |
| cdent | s/tree/branch of the tree/ | 15:14 |
| dansmith | johnthetubaguy: well, the allocation stuff is kindof existing, because it's already in the instance. However, what I'm trying to get us to is all of one type of data (compute node) so that we can do allocations (instance stuff) in newton assuming the compute node stuff is there and done | 15:15 |
| sdague | jaosorior: just left a detailed review asking for some additional testing. Let me know if that's reasonable to do. | 15:16 |
| *** mtanino has joined #openstack-nova | 15:16 | |
| johnthetubaguy | dansmith: right, that seems like a good way to stack those | 15:16 |
| jaosorior | sdague: I actually left this patch alone since november cause it took me too long to do the functional testing part. Doing such a thing requires coming up with a framework and turned out way more complicated than the patch itself :/ | 15:17 |
| cdent | johnthetubaguy, dansmith: Should I still worry about the unique constraint? | 15:18 |
| cdent | I assume yes, given "get the data right, now" is a goal | 15:18 |
| johnthetubaguy | cdent: thats in the bottom patch I guess? it seems worth adding the constraint now, if we think its useful, as its easy to drop it (hard to add it later) | 15:18 |
| dansmith | the UC is part of the provider table which impacts the inventories, right? if so, then that's probably important to get in | 15:18 |
| * cdent nods | 15:19 | |
| dansmith | probably not critical until we have external resource providers, but better to do before we get lots of data in the table | 15:19 |
| *** permalac has joined #openstack-nova | 15:19 | |
| *** permalac has quit IRC | 15:19 | |
| cdent | johnthetubaguy, dansmith If you guys see jaypipes later in the day, can you make sure you sync up with him on this stuff. I've sent him a ping but I think he's on a plane. | 15:19 |
| bauzas | so I just left 10 mins the chan, and you folks litterally had that long convo :p | 15:19 |
| johnthetubaguy | OK, so turning this around, when can we get al the patches in this queue, ready to merge, based on the current comments? as in this one, and all its deps? https://review.openstack.org/#/c/284963 | 15:19 |
| dansmith | I feel like we've synced up on this maaaany times, but sure | 15:20 |
| bauzas | johnthetubaguy: yup | 15:20 |
| bauzas | getting the compute updating the stats is the most important big | 15:20 |
| *** permalac has joined #openstack-nova | 15:20 | |
| bauzas | bit | 15:20 |
| bauzas | cdent: ready to udate your -W by adding the UC ? | 15:20 |
| dansmith | johnthetubaguy: did you mean the other stack? | 15:20 |
| johnthetubaguy | crap... | 15:20 |
| cdent | bauzas: yes that's where I was headed when this conversation started | 15:20 |
| jaosorior | sdague: I could try again, but I'm actually not sure where those tests would go. Would that be in tempest? | 15:20 |
| bauzas | johnthetubaguy: so you said it was okay to merge https://review.openstack.org/#/c/281837/ without waiting the nova-specs side to be merged ? | 15:21 |
| johnthetubaguy | yeah, so its this one: https://review.openstack.org/#/c/279313/ | 15:21 |
| bauzas | (provided the UC is added) | 15:21 |
| dansmith | johnthetubaguy: I need to rewrite the resource tracker tests to make them work with this, which I'm going to do this morning, after which I'll remove my WIP and I think ready for review | 15:21 |
| dansmith | johnthetubaguy: I think the stack under that can all be reviewed now | 15:21 |
| * dansmith checks for comments | 15:21 | |
| jaosorior | mriedem ^^ | 15:21 |
| johnthetubaguy | dansmith: OK, that sounds good | 15:21 |
| bauzas | dansmith: we have a -W because of the UC at the bottom of the stack | 15:22 |
| bauzas | dansmith: that's just preventing us to call Jenkins | 15:22 |
| mriedem | jaosorior: the tests would live in nova/tests/functional | 15:22 |
| dansmith | bauzas: on which patch? | 15:22 |
| johnthetubaguy | cdent: dansmith: so I am thinking about dropping the -2 hammer on patches not in that high priority stack, at least until we have that full stack merged, so we are all on the same page? | 15:22 |
| dansmith | oh I see it now | 15:22 |
| bauzas | dansmith: https://review.openstack.org/#/c/281837/ | 15:22 |
| dansmith | johnthetubaguy: +1 | 15:22 |
| mriedem | jaosorior: i think sdague is looking for a fake hook to get registered that has similar logic to what you have in the bug report | 15:22 |
| bauzas | johnthetubaguy: ++ | 15:22 |
| dansmith | bauzas: yep got it | 15:22 |
| *** haukebruno has joined #openstack-nova | 15:23 | |
| sdague | mriedem: yeh, hooks are so weird | 15:23 |
| mriedem | so fake hook for this https://github.com/openstack/nova/blob/3da31e3262ee3b07e96efa4eca381953ff1de169/nova/compute/api.py#L1443 | 15:23 |
| bauzas | it would also be ideal if we could have the deps fully rebased in the stack, because https://review.openstack.org/#/c/279313/ is not current on top of it | 15:23 |
| jaosorior | mriedem: I could try, though I'm not very familiar with the tests... I was just trying to fix a bug :/ | 15:23 |
| bauzas | but that's something we can do on the fly | 15:23 |
| dansmith | bauzas: yeah, it's just because so many people are working on it.. it's been a struggle for me at the top :/ | 15:23 |
| sdague | jaosorior: right, I do get that you are trying to fix the bug. My concerns is this areas is so dark that we're just going to break it again by accident. | 15:24 |
| *** kfarr has joined #openstack-nova | 15:24 | |
| cdent | alright, I'll get that constraint, will ping bauzas when done | 15:25 |
| bauzas | dansmith: no worries, we can rebase that when needed | 15:25 |
| sdague | dhellmann: stevedore question, if you are around - we're trying to figure out a reasonable way to test nova.hooks at runtime without having to change setup.py to specify a stevedore entry point | 15:26 |
| bauzas | dansmith: ideally, I'd appreciate if cdent could take your latest SHA1s on his local branch, so it would set the branch correctly | 15:26 |
| *** irenab has quit IRC | 15:26 | |
| bauzas | cherry-picking the latest top patches I mean | 15:27 |
| bauzas | and not breaking the series | 15:27 |
| dhellmann | sdague : each stevedore manager class has a class method for building a test instance with explicitly specified plugin objects | 15:27 |
| dhellmann | sdague : they're meant for building fixtures that you can inject into your code to replace the "real" manager | 15:27 |
| *** ccarmack has joined #openstack-nova | 15:27 | |
| johnthetubaguy | dansmith: cdent: I have -2ed a bunch of stuff, just want to make sure the hammer hit the right places, see: https://review.openstack.org/#/c/279313/ and https://review.openstack.org/#/c/284963 | 15:27 |
| *** irenab has joined #openstack-nova | 15:28 | |
| sdague | dhellmann: do we have to replace the whole manager? | 15:28 |
| *** irenab has quit IRC | 15:28 | |
| *** mylu has quit IRC | 15:28 | |
| dhellmann | sdague : well, the idea is you'd mock.patch it, yeah | 15:28 |
| dhellmann | sdague : though that depends on exactly what you're trying to test, I guess | 15:28 |
| sdague | right, we're trying to test a large behavior | 15:28 |
| johnthetubaguy | cdent: dansmith: so its basically this one that we are doing right? https://blueprints.launchpad.net/nova/+spec/compute-node-inventory | 15:29 |
| sdague | https://github.com/openstack/nova/blob/8e8acd930fe9f9d6be99e4ea22c3e33176014ec4/nova/hooks.py#L132 - nova has this hooks facility, built on stevedore | 15:29 |
| dansmith | johnthetubaguy: -2 looks right.. don't ask me to make sense of the specs :) | 15:29 |
| sdague | it turns out some of the hooks fail now because of some behavior changes kind of far up as to what data is available | 15:29 |
| johnthetubaguy | dansmith: I was thinking we just spec-less approve that compute-node-inventory blueprint to track this effort | 15:29 |
| dansmith | johnthetubaguy: but yeah, inventory stuff | 15:29 |
| dansmith | okay | 15:30 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix Cells RPC API by accepting a RequestSpec arg https://review.openstack.org/284959 | 15:30 |
| *** mriedem is now known as mriedem_meeting | 15:30 | |
| cdent | johnthetubaguy: yes, I believe that's what you and dansmith have decided but I don't feel like it is what jaypipes thought was decided. I realize without him here and me being not entirely informed that's a bit complicated. | 15:30 |
| *** irenab has joined #openstack-nova | 15:30 | |
| *** zz_dimtruck is now known as dimtruck | 15:30 | |
| *** mgoddard__ has joined #openstack-nova | 15:30 | |
| sdague | so ideally I'd like to run some in tree full stack testing where when we setup our set of nova services, we can specify a hook class | 15:30 |
| dhellmann | sdague : I recommend a test fixture that replaces members of the _HOOKS registry with test instances of the manager, populated with -- right | 15:30 |
| sdague | https://github.com/openstack/nova/blob/8e8acd930fe9f9d6be99e4ea22c3e33176014ec4/doc/source/hooks.rst#L37-L57 | 15:30 |
| sdague | ok, that seems doable | 15:31 |
| dansmith | cdent: there is absolutely no way we can get the other stack into mitaka | 15:31 |
| dansmith | cdent: I will catch jaypipes when he shows up and confirm | 15:31 |
| sdague | dhellmann: thanks | 15:31 |
| cdent | dansmith: I'm not disagreeing with you. I just recall that there were _parts_ of it that we wanted, but I'm not certain of the details. Thanks. | 15:31 |
| dhellmann | sdague : fwiw, that line 142 is re-scanning the entry points every time it is invoked, and then throwing away the results sometimes. it would be much more efficient to do that as a separate "if not in" check | 15:32 |
| sdague | dhellmann: the decorator? | 15:32 |
| jaosorior | sdague: I'm still unsure if by full-stack you mean you want tests in tempest (with a full stack) or as mriedem_meeting said in nova/tests/functional | 15:33 |
| dhellmann | sdague : yeah, it's written with the implication that the same name might be used more than once, so if that's not true then nevermind | 15:33 |
| *** davideagle has joined #openstack-nova | 15:33 | |
| *** mgoddard_ has quit IRC | 15:33 | |
| dhellmann | sdague : but I would replace setdefault() with "if name not in _HOOKS" so the HookManager() is only constructed once | 15:33 |
| sdague | dhellmann: ok, good to know. This code is all circa 2012 and very poorly tested, so I would be very cautious in optimizing it | 15:33 |
| dhellmann | sdague : sure | 15:34 |
| sdague | dhellmann: but it's worth looking into for sure | 15:34 |
| sdague | jaosorior: nova/tests/funtional - let me poke at a fixture here that might be helpful | 15:34 |
| jaosorior | sdague: Alright, thanks | 15:34 |
| johnthetubaguy | cdent: just to double check, you are adding the UC on this right now I guess? https://review.openstack.org/#/c/281837/ | 15:35 |
| cdent | yes | 15:35 |
| *** mylu has joined #openstack-nova | 15:36 | |
| *** jaosorior is now known as jaosorior_away | 15:37 | |
| johnthetubaguy | cdent: cool, thanks | 15:37 |
| johnthetubaguy | sdague: do have any time to take a look at this final live-migrate API change? https://review.openstack.org/#/c/259319/ I would love a second opinion on that before we approved it, if you have the time | 15:42 |
| sdague | johnthetubaguy: yeh, I can look now | 15:43 |
| *** pushkaru has joined #openstack-nova | 15:45 | |
| *** moshele has quit IRC | 15:45 | |
| *** Sukhdev has joined #openstack-nova | 15:46 | |
| *** sneti has joined #openstack-nova | 15:47 | |
| *** jlanoux has quit IRC | 15:48 | |
| *** tbachman has quit IRC | 15:48 | |
| *** vladikr has quit IRC | 15:48 | |
| *** vladikr has joined #openstack-nova | 15:49 | |
| sdague | johnthetubaguy: I think the API change looks good, I just +Aed that. I haven't looked at the patch below it yet | 15:49 |
| *** armax has joined #openstack-nova | 15:50 | |
| sdague | I'm going to work on this hooks test fixture first unless you think you need me on the lower patch. You and alex_xu are both +2 on it. | 15:50 |
| johnthetubaguy | sdague: cool thats, I can just +W that one anyways, it was the API one that worried about | 15:50 |
| johnthetubaguy | sdague: thanks for that | 15:50 |
| johnthetubaguy | PaulMurray: now we have +W on https://review.openstack.org/#/c/259319/, is it mostly python-novaclient patches we need now? | 15:51 |
| *** mriedem_meeting is now known as mriedem | 15:51 | |
| sdague | I'll admit, I'm not thrilled about the way we're doing follow on json schema by patching the base one | 15:51 |
| sdague | but we do it all over the place | 15:51 |
| sdague | I kind of think we should be expanding those all out | 15:51 |
| johnthetubaguy | sdague: yeah, same, it follows the pattern, we can fix them all | 15:51 |
| *** davideagle has quit IRC | 15:52 | |
| johnthetubaguy | sdague: yeah, I don't think we can ever "fix" the old ones, 99% of the time, so obfuscating the schema is probably the wrong call | 15:52 |
| sdague | right | 15:53 |
| sdague | low hanging fruit newton tasks | 15:53 |
| *** wolsen has joined #openstack-nova | 15:53 | |
| doffm | alaski: I tried re-spining one of your patches last week. 'Add build request object'. Unfortunately still failing unit tests. | 15:53 |
| doffm | alaski: Sorry if that messed things up for you. | 15:53 |
| alaski | doffm: not at all, thanks for doing that. I'm looking at the failure now | 15:54 |
| PaulMurray | johnthetubaguy, shaohe_feng_ said he is doing a new patchset for https://review.openstack.org/#/c/281335/ | 15:54 |
| *** davideagle has joined #openstack-nova | 15:54 | |
| PaulMurray | johnthetubaguy, if he needs it we can take over from him later | 15:54 |
| *** haplo37 has joined #openstack-nova | 15:54 | |
| doffm | alaski: Ok. I was going to look at it if you were somehow out...or not available today. But i'll leave it to you. :) | 15:54 |
| PaulMurray | that will hopefully leave just eliqiao's last patch | 15:55 |
| *** EinstCrazy has quit IRC | 15:55 | |
| PaulMurray | to be completed | 15:55 |
| openstackgerrit | Chris Dent proposed openstack/nova: Adjust resource-providers models for resource-pools https://review.openstack.org/281837 | 15:56 |
| *** erno has joined #openstack-nova | 15:56 | |
| *** angdraug has joined #openstack-nova | 15:57 | |
| johnthetubaguy | PaulMurray: cools | 15:57 |
| cdent | bauzas, johnthetubaguy ^^ | 15:57 |
| openstackgerrit | Chris Dent proposed openstack/nova: Adjust resource-providers models for resource-pools https://review.openstack.org/281837 | 15:57 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 15:58 |
| *** penick has joined #openstack-nova | 15:59 | |
| bauzas | cdent: ack, going in a call for the next hour, but will review later tonight | 15:59 |
| cdent | thanks bauzas | 15:59 |
| *** salv-orl_ has joined #openstack-nova | 16:00 | |
| *** sc68cal has joined #openstack-nova | 16:01 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** wuhao has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** wuhao has joined #openstack-nova | 16:02 | |
| dansmith | ah, crap, I have to write some backport routine for this ComputeNode change | 16:02 |
| cfriesen_ | Any chance of getting someone to look at https://review.openstack.org/#/c/283861/ ? I got a +1 from the original author of the code being tweaked. It's a 1-liner plus unit test changes. | 16:03 |
| *** salv-orlando has quit IRC | 16:03 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: "nova list-extensions" not shwoing summary for all https://review.openstack.org/286130 | 16:03 |
| *** nkrinner has quit IRC | 16:03 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 16:05 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 16:05 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 16:05 |
| doffm | bauzas: I know you are busy, but if you get a chance could you take a look at the cell0 patches? ^^ | 16:06 |
| doffm | Anyone else welcome also. :) | 16:06 |
| *** mjura has quit IRC | 16:08 | |
| * PaulMurray AFK 30 mins | 16:11 | |
| *** browne has joined #openstack-nova | 16:11 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create BuildRequest object during boot process https://review.openstack.org/278125 | 16:11 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 16:11 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add BuildRequest object https://review.openstack.org/278124 | 16:11 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable https://review.openstack.org/278598 | 16:11 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 16:11 |
| *** mdrabe has quit IRC | 16:12 | |
| markusz | dansmith: Hi, could you maybe take a look at https://review.openstack.org/#/c/264157/ ? It fixes a networking bug for LXC (and afaik PowerVM too). | 16:15 |
| *** ptm_away has joined #openstack-nova | 16:15 | |
| *** markusz is now known as markus_z | 16:15 | |
| sheel | Dear All Nova members: need some eyes on one small patch https://review.openstack.org/#/c/285222/ | 16:15 |
| *** haukebruno has quit IRC | 16:15 | |
| sheel | Please find some spare time and have a review | 16:15 |
| sheel | Thanks | 16:15 |
| *** jlanoux has joined #openstack-nova | 16:16 | |
| thorst | markus_z: I think that patch fixes all operators using cloud-init to apply IP addresses (specifically IPv6 only) | 16:16 |
| *** kylek3h_ is now known as kylek3h | 16:16 | |
| *** rcernin has quit IRC | 16:16 | |
| markus_z | thorst: Ah, even better. | 16:16 |
| thorst | markus_z: Yeah, nothing PowerVM specific there. We just noticed it as well. | 16:16 |
| *** fawadkhaliq has quit IRC | 16:17 | |
| *** pcaruana has quit IRC | 16:17 | |
| markus_z | dansmith: Some time ago I asked for contacts in the networking area and your name was the only one coming up, that's why I pinged you. | 16:17 |
| *** PaulMurray has quit IRC | 16:18 | |
| *** ptm_away has quit IRC | 16:20 | |
| *** penick has quit IRC | 16:23 | |
| *** belmoreira has quit IRC | 16:24 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Cold migrate using the RequestSpec object https://review.openstack.org/284974 | 16:25 |
| *** mylu has quit IRC | 16:25 | |
| *** mdrabe has joined #openstack-nova | 16:26 | |
| *** ndipanov has quit IRC | 16:26 | |
| *** ndipanov has joined #openstack-nova | 16:26 | |
| sheel | markus_z: hi there | 16:26 |
| markus_z | sheel: hey ho | 16:27 |
| sheel | markus_z: any chances of reviewing https://review.openstack.org/#/c/285222/ | 16:27 |
| sheel | markus_z: actually its regarding https://bugs.launchpad.net/nova/+bug/1535224 where you had some interest.. | 16:27 |
| openstack | Launchpad bug 1535224 in OpenStack Compute (nova) "meta data of nova instance with case sensitive has problem when deleting" [Undecided,In progress] - Assigned to Sheel Rana (ranasheel2000) | 16:27 |
| *** abitha has joined #openstack-nova | 16:27 | |
| *** omnipresent has quit IRC | 16:28 | |
| sheel | markus_z: I found some comments from you in this bug, so though you might be interested in reviewing | 16:28 |
| *** mylu has joined #openstack-nova | 16:28 | |
| *** lxsli has joined #openstack-nova | 16:28 | |
| *** zenoway has joined #openstack-nova | 16:28 | |
| sheel | markus_z: so please review in spare time.. | 16:28 |
| markus_z | sheel: yep, I just read the bug entry to remember. I add it to my todo list of tomorrow | 16:29 |
| *** mgoddard__ has quit IRC | 16:29 | |
| *** mgoddard has joined #openstack-nova | 16:29 | |
| markus_z | sheel: thanks for the patch | 16:29 |
| *** rodrigods has quit IRC | 16:29 | |
| sheel | markus_z: ok thanks | 16:29 |
| *** rodrigods has joined #openstack-nova | 16:29 | |
| *** lxsli has quit IRC | 16:29 | |
| markus_z | sheel: Any change you can push test cases for that until tomorrow? | 16:30 |
| markus_z | s/change/chance | 16:30 |
| *** lxsli has joined #openstack-nova | 16:30 | |
| sheel | markus_z: yes, I will | 16:30 |
| *** hemna has joined #openstack-nova | 16:30 | |
| markus_z | sheel: cool. From a first glance it seems sane what you do. | 16:30 |
| sheel | markus_z: :) | 16:31 |
| *** jaugustine has joined #openstack-nova | 16:31 | |
| *** zenoway has quit IRC | 16:33 | |
| *** lxsli has quit IRC | 16:33 | |
| *** lxsli has joined #openstack-nova | 16:34 | |
| *** cdent has quit IRC | 16:35 | |
| *** tbachman has joined #openstack-nova | 16:40 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 16:40 |
| dansmith | markus_z: hmm, that's a little scary of a change to make because I think it will impact cloud-init and I'm not sure I know enough to confirm that it will work for all debian variants | 16:42 |
| dansmith | markus_z: not sure who to poke about that really | 16:42 |
| *** mylu has quit IRC | 16:42 | |
| dansmith | larsks: cloud-init parses the debian-style interfaces file that nova generates, is that right? | 16:42 |
| larsks | dansmith: yeah. as long as we are providing a deiban-style interfaces file. There is (was?) a bug open due to the fact that our nova packages had a broken template. | 16:43 |
| larsks | But that is the model, yes. | 16:43 |
| dansmith | larsks: can you check this then? https://review.openstack.org/#/c/264157/4 | 16:43 |
| dansmith | it's changing our template | 16:43 |
| *** thorst is now known as thorst_afk | 16:44 | |
| larsks | dansmith: that seems reasonable at first glance. I'm not familiar enough with debian to know whether it's *correct*, but it looks okay. | 16:45 |
| larsks | I can test that out, although probably not today... | 16:45 |
| dansmith | larsks: well, yeah, I'm kinda asking if you can test it :D | 16:46 |
| larsks | Yup. | 16:46 |
| *** pkarikh has quit IRC | 16:46 | |
| *** pkarikh has joined #openstack-nova | 16:47 | |
| *** ljxiash has quit IRC | 16:49 | |
| markus_z | thorst_afk: ^ | 16:50 |
| *** paul-carlton2 has quit IRC | 16:50 | |
| *** markstur has joined #openstack-nova | 16:50 | |
| *** scheuran has quit IRC | 16:51 | |
| *** cdent has joined #openstack-nova | 16:54 | |
| *** daemontool_ has quit IRC | 16:55 | |
| *** gcb has quit IRC | 16:56 | |
| *** davideagle has quit IRC | 16:56 | |
| *** PaulMurray has joined #openstack-nova | 16:57 | |
| *** matrohon has quit IRC | 16:57 | |
| *** apoorvad has joined #openstack-nova | 16:57 | |
| *** roeyc has quit IRC | 16:59 | |
| sdague | markus_z: hey, do you have scripts you use to process nova bugs? | 16:59 |
| markus_z | sdague: A few hacky things I use for querying stuff. I moved away from batch processing them. Need anything specific. | 17:00 |
| markus_z | ? | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| sdague | markus_z: just wondering, I was thinking about doing some collection per tag, and looking for things that don't have a tag | 17:01 |
| sdague | but didn't want to start from scratch | 17:01 |
| sdague | if you want to throw up a github repo or somethign with what you have, that might be handy | 17:01 |
| *** haomaiwa_ has joined #openstack-nova | 17:01 | |
| *** speller has quit IRC | 17:02 | |
| markus_z | sdague: let me take a look at the stuff I have, I'll come back | 17:02 |
| *** Jeffrey4l has quit IRC | 17:03 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks https://review.openstack.org/275073 | 17:03 |
| *** yamahata has joined #openstack-nova | 17:04 | |
| PaulMurray | eliqiao, merge conflict on https://review.openstack.org/#/c/259319/ | 17:04 |
| sdague | markus_z: cool, thanks | 17:05 |
| *** jaosorior_away is now known as jaosorior | 17:08 | |
| *** mpavone has quit IRC | 17:08 | |
| PaulMurray | eliqiao, sorry - my mistake that's not a merge conflict | 17:10 |
| *** gszasz has quit IRC | 17:13 | |
| *** amotoki has quit IRC | 17:13 | |
| *** paul-carlton2 has joined #openstack-nova | 17:13 | |
| *** boris-42 has quit IRC | 17:14 | |
| *** rhagarty has joined #openstack-nova | 17:14 | |
| *** mingyu has joined #openstack-nova | 17:15 | |
| markus_z | sdague: https://github.com/markuszoeller/openstack/blob/master/scripts/query_rc_blocker_reviews.py | 17:15 |
| markus_z | sdague: That's a simple one you can use as a template. | 17:15 |
| markus_z | sdague: replacing "liberty-rc-potential" with "-*" should do what you want. | 17:15 |
| sdague | markus_z: cool, thanks | 17:16 |
| *** andreykurilin__ has joined #openstack-nova | 17:18 | |
| *** openstackgerrit has quit IRC | 17:18 | |
| *** openstackgerrit has joined #openstack-nova | 17:18 | |
| *** mingyu has quit IRC | 17:19 | |
| *** busterswt has quit IRC | 17:19 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: scheduler: do not update HostState if compute_node has no valid data https://review.openstack.org/261145 | 17:20 |
| *** pkoniszewski_ has joined #openstack-nova | 17:20 | |
| *** sc68cal has quit IRC | 17:22 | |
| *** daemontool_ has joined #openstack-nova | 17:26 | |
| *** jistr has quit IRC | 17:26 | |
| *** nic has joined #openstack-nova | 17:27 | |
| johnthetubaguy | cdent: dansmith: does anyone know what the generation and can_host do in resource provider are doing? | 17:27 |
| johnthetubaguy | bauzas: including you, as you reviewed the previous patch ^ | 17:29 |
| *** david-lyle has joined #openstack-nova | 17:30 | |
| johnthetubaguy | I guess generation gets updated on every update, will dig in the rest of the cahin | 17:31 |
| johnthetubaguy | chain | 17:31 |
| *** wanghao has quit IRC | 17:32 | |
| alaski | johnthetubaguy: it's explained in the spec, if I can find that | 17:32 |
| johnthetubaguy | ah, I got lost in spec soup around that stuff | 17:33 |
| jaosorior | sdague: Thanks for starting the conversation on the nova hooks. One thing though... Those have been pretty useful, even if they lack documentation and are a bit hard to figure out. Wouldn't it be a possibility to refactor them instead and come up with a stable API for them? | 17:33 |
| *** wanghao has joined #openstack-nova | 17:34 | |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 17:34 |
| sdague | jaosorior: if it was something we wanted to support. It changes other expectations as well. | 17:34 |
| johnthetubaguy | alaski: you just reminded me, are there any critical cells v2 changes we need for mitaka, I am likely to start a new round of -2s tomorrow | 17:35 |
| jaosorior | can you ellaborate what you mean by " | 17:35 |
| jaosorior | it changes other expectations as well." | 17:35 |
| *** danpb has quit IRC | 17:35 | |
| jaosorior | sdague ^^ | 17:35 |
| *** abitha has quit IRC | 17:35 | |
| sdague | jaosorior: anyway, here is my take. If we decide to deprecate hooks, I'll +2 your bug fix as is, because it makes no sense to require testing the broader thing for your fix | 17:36 |
| alaski | johnthetubaguy: I'm not sure I'd call them critical from an upgrade PoV, but the series at https://review.openstack.org/#/c/278124/ would be good to have, as well as melwitt and doffms nova-manage changes | 17:36 |
| *** fawadkhaliq has joined #openstack-nova | 17:36 | |
| sdague | jaosorior: there are all kinds of implications in providing arbitrary code execution there, especially when things like cloud-init exist | 17:36 |
| alaski | johnthetubaguy: that should all be in the priority review etherpad | 17:36 |
| *** yassine has quit IRC | 17:36 | |
| sdague | honestly, I don't know the rdo usage enough to know why that's the plug point here | 17:37 |
| jaosorior | sdague: Well.. Actually what's being done there is processed by cloud-init | 17:38 |
| alaski | johnthetubaguy: here's the generation and can_host info https://review.openstack.org/#/c/283253/ | 17:38 |
| jaosorior | sdague: And notifications don't provide enough information | 17:39 |
| *** sfinucan has quit IRC | 17:39 | |
| sdague | jaosorior: well, then we'd have that discussion about what is needed | 17:39 |
| *** lpetrut has quit IRC | 17:40 | |
| *** davideagnello has joined #openstack-nova | 17:40 | |
| sdague | jaosorior: I can't ad-hoc all the implications of what you are doing and what's needed on the fly. But it's probably worth writing down your use cases in detailed regardless, to figure out what you might need in newton if this mechanism is removed | 17:40 |
| shaohe_feng_ | PaulMurray: andreykurilin: send the new patch. Thanks. | 17:40 |
| *** pece has joined #openstack-nova | 17:41 | |
| PaulMurray | shaohe_feng_, just looking : https://review.openstack.org/#/c/281335/8..9/novaclient/v2/contrib/migrations.py L69 should that be 2.23 | 17:41 |
| johnthetubaguy | jaosorior: we are re-working the notifications in newton, so getting the required information in there, could be a good way forward here | 17:41 |
| shaohe_feng_ | PaulMurray: yes. good catch. let me fix it. | 17:42 |
| johnthetubaguy | alaski: thanks for all those, looks good | 17:42 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Allow block_migration and disk_over_commit to be None https://review.openstack.org/254102 | 17:42 |
| johnthetubaguy | alaski: ah, needed for those atomic compare things, that sounds believable. | 17:43 |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 17:44 |
| *** penick has joined #openstack-nova | 17:44 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: API: Improve os-migrateLive input parameters https://review.openstack.org/259319 | 17:44 |
| *** sahid has quit IRC | 17:44 | |
| *** ihrachys has quit IRC | 17:45 | |
| pkoniszewski_ | hi, i just rebased +Wed patch that failed to merge, because of a change in doc/notification_samples/service-update.json | 17:45 |
| pkoniszewski_ | https://review.openstack.org/#/c/254102 | 17:45 |
| pkoniszewski_ | can it get re-approved? | 17:45 |
| PaulMurray | johnthetubaguy, simple rebase on patch you +W: https://review.openstack.org/#/c/254102/35 | 17:45 |
| *** wanghao has quit IRC | 17:45 | |
| *** wanghao has joined #openstack-nova | 17:46 | |
| johnthetubaguy | PaulMurray: ah, cools | 17:47 |
| openstackgerrit | ShaoHe Feng proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 17:47 |
| *** achanda has joined #openstack-nova | 17:47 | |
| PaulMurray | johnthetubaguy, and the follow-on : https://review.openstack.org/#/c/259319 | 17:47 |
| *** harlowja_at_home has joined #openstack-nova | 17:48 | |
| *** derekh has quit IRC | 17:53 | |
| *** Marga_ has quit IRC | 17:54 | |
| johnthetubaguy | PaulMurray: I hit both I thought? | 17:54 |
| johnthetubaguy | oh wait, timeline warp | 17:55 |
| PaulMurray | johnthetubaguy, yes, you did - yep, I'm slightly in the past | 17:55 |
| *** david-lyle has quit IRC | 17:55 | |
| *** david-lyle_ has joined #openstack-nova | 17:55 | |
| PaulMurray | (although being in bristol I am actually 5 mins ahead) | 17:56 |
| *** tongli has quit IRC | 17:56 | |
| * johnthetubaguy its funny that IRC conversions exhibit distributed system problems... | 17:56 | |
| *** jaugustine has quit IRC | 17:57 | |
| *** claudiub has quit IRC | 17:57 | |
| *** burt has quit IRC | 17:59 | |
| *** krtaylor has quit IRC | 17:59 | |
| *** haomaiwa_ has quit IRC | 18:01 | |
| *** claudiub has joined #openstack-nova | 18:01 | |
| *** jlanoux has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** Guest67703 is now known as me_ | 18:01 | |
| *** agireud has quit IRC | 18:01 | |
| *** me_ is now known as med_ | 18:01 | |
| *** med_ has quit IRC | 18:01 | |
| *** med_ has joined #openstack-nova | 18:01 | |
| *** baoli has quit IRC | 18:01 | |
| *** baoli has joined #openstack-nova | 18:03 | |
| *** baoli has quit IRC | 18:03 | |
| *** agireud has joined #openstack-nova | 18:03 | |
| *** aginwala has joined #openstack-nova | 18:04 | |
| *** achanda has quit IRC | 18:04 | |
| *** lucasagomes is now known as lucas-dinner | 18:05 | |
| *** sudipto has quit IRC | 18:08 | |
| *** aginwala has quit IRC | 18:09 | |
| *** zenoway has joined #openstack-nova | 18:10 | |
| *** matrohon has joined #openstack-nova | 18:10 | |
| *** baoli has joined #openstack-nova | 18:10 | |
| *** krtaylor has joined #openstack-nova | 18:11 | |
| *** vladikr_ has joined #openstack-nova | 18:11 | |
| pkoniszewski_ | andreykurilin: hi, there? | 18:12 |
| andreykurilin__ | pkoniszewski_: hi, yes | 18:13 |
| *** vladikr has quit IRC | 18:13 | |
| dansmith | bauzas: still around? | 18:13 |
| pkoniszewski_ | im fixing GET/SHOW migrations in client | 18:14 |
| pkoniszewski_ | wonder what you mean there https://review.openstack.org/#/c/281335/11/novaclient/v2/contrib/migrations.py | 18:14 |
| *** Swami has joined #openstack-nova | 18:14 | |
| pkoniszewski_ | is api_version part of 'cs' ? | 18:14 |
| pkoniszewski_ | andreykurilin__: ^^ | 18:14 |
| andreykurilin__ | pkoniszewski_: yes https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/client.py#L134 | 18:16 |
| *** vladikr_ has quit IRC | 18:16 | |
| andreykurilin__ | cs is just a initialized Client class, so it have api_version property | 18:16 |
| andreykurilin__ | *it has | 18:16 |
| pkoniszewski_ | got it! :) | 18:16 |
| *** sc68cal has joined #openstack-nova | 18:19 | |
| *** baoli_ has joined #openstack-nova | 18:19 | |
| *** baoli has quit IRC | 18:22 | |
| thorst_afk | dansmith markus_z: We had tested 264157 in our envs with IPv6 only and IPv4 only and it was working well. | 18:23 |
| *** thorst_afk is now known as thorst | 18:23 | |
| *** Guest63721 is now known as mfisch | 18:24 | |
| *** mfisch is now known as Guest2004 | 18:25 | |
| markus_z | thorst: maybe you can contact larsks to double-check with him? | 18:25 |
| larsks | markus_z: that sounds like suitable testing to me :). | 18:26 |
| thorst | definitely. larsks if it would help, I can also set up a devstack for KVM and verify in that env? Our testing of it had been with the PowerVM driver, but the code that touches is not PowerVM specific. Regardless, if we'd like to run it through a KVM env, I could run that quick as well | 18:27 |
| *** raildo is now known as raildo-afk | 18:27 | |
| *** krtaylor has quit IRC | 18:27 | |
| larsks | thorst: I think that makes sense, yes; that's what I was planning on doing either later today or early tomorrow, but if you can get to it now more power to you! | 18:27 |
| dansmith | thorst: I specifically want to make sure it works with ubuntu and rhel, since cloud-init has to translate | 18:28 |
| *** Marga_ has joined #openstack-nova | 18:28 | |
| dansmith | thorst: if you can do those, then that's cool | 18:28 |
| *** Guest2004 is now known as mfisch | 18:28 | |
| *** mfisch has quit IRC | 18:28 | |
| *** mfisch has joined #openstack-nova | 18:28 | |
| thorst | dansmith larsks: my testing had been with Ubuntu clients. I don't have RHEL images handy...I can probably build some though. But it might take me a bit longer? | 18:28 |
| dansmith | thorst: okay, well, I dunno... someone should test it :) | 18:29 |
| thorst | dansmith: agree. I can get the RHEL testing done. Just will take me a bit longer :-) | 18:29 |
| *** baoli_ has quit IRC | 18:29 | |
| thorst | I'll update the patch set when I finish the test with the results. | 18:29 |
| *** raildo-afk is now known as raildo | 18:29 | |
| dansmith | okay, thanks | 18:30 |
| thorst | dansmith larsks markus_z: thanks! | 18:30 |
| *** boris-42 has joined #openstack-nova | 18:30 | |
| *** baoli has joined #openstack-nova | 18:31 | |
| *** thorst is now known as thorst_afk | 18:31 | |
| *** abitha has joined #openstack-nova | 18:31 | |
| *** doug-fish has quit IRC | 18:32 | |
| *** vladikr_ has joined #openstack-nova | 18:32 | |
| *** whenry_ has joined #openstack-nova | 18:32 | |
| *** huats_ has quit IRC | 18:33 | |
| *** huats_ has joined #openstack-nova | 18:33 | |
| *** huats_ has quit IRC | 18:33 | |
| *** huats_ has joined #openstack-nova | 18:33 | |
| *** browne has quit IRC | 18:33 | |
| *** pece has quit IRC | 18:34 | |
| *** rdopiera has quit IRC | 18:35 | |
| *** PaulMurray has quit IRC | 18:35 | |
| *** paul-carlton2 has quit IRC | 18:36 | |
| *** chhavi has quit IRC | 18:39 | |
| *** krtaylor has joined #openstack-nova | 18:39 | |
| openstackgerrit | Kaitlin Farr proposed openstack/nova-specs: Integrate Castellan for Key Management https://review.openstack.org/247561 | 18:40 |
| *** baoli has quit IRC | 18:42 | |
| openstackgerrit | Sean Dague proposed openstack/nova: WIP: example code to try to test hooks https://review.openstack.org/286198 | 18:43 |
| *** baoli has joined #openstack-nova | 18:44 | |
| *** angdraug has quit IRC | 18:45 | |
| *** andrearosa_web has quit IRC | 18:45 | |
| *** rk4n has quit IRC | 18:45 | |
| *** Sukhdev has quit IRC | 18:45 | |
| *** doug-fish has joined #openstack-nova | 18:46 | |
| *** doug-fish has quit IRC | 18:46 | |
| *** doug-fish has joined #openstack-nova | 18:47 | |
| mriedem | man, check queue is 400 deep | 18:47 |
| *** vilobhmm11 has joined #openstack-nova | 18:49 | |
| dansmith | edleafe: so the problem with you using the model in the update method is that we don't get back the inv_* keys, which means we update the compute node with zeroes on save | 18:50 |
| edleafe | dansmith: hmmmm... | 18:51 |
| *** e0ne has joined #openstack-nova | 18:51 | |
| dansmith | I'm inclined to do something roundabout in the object to avoid this | 18:51 |
| dansmith | lemme see how terrible it is | 18:51 |
| edleafe | dansmith: so maybe update has to update two things for the time being? | 18:51 |
| edleafe | dansmith: i.e., both the compute node and the inv_* stuff? | 18:52 |
| dansmith | no | 18:52 |
| dansmith | we don't want it to do that | 18:52 |
| *** baoli has quit IRC | 18:52 | |
| dansmith | if anything, we just need to have it return the result of a get query | 18:52 |
| *** thorst_afk is now known as thorst | 18:52 | |
| dansmith | maybe I should just do that for now | 18:52 |
| *** vladikr_ has quit IRC | 18:54 | |
| *** baoli has joined #openstack-nova | 18:54 | |
| *** jaosorior has quit IRC | 18:54 | |
| *** baoli has quit IRC | 18:55 | |
| *** vladikr_ has joined #openstack-nova | 18:56 | |
| *** baoli has joined #openstack-nova | 18:58 | |
| *** jaosorior has joined #openstack-nova | 18:58 | |
| johnthetubaguy | mriedem: something tells me we have already merged all the patches for mitaka-3 | 18:59 |
| johnthetubaguy | an python-novaclient release | 18:59 |
| *** andymaier has quit IRC | 18:59 | |
| johnthetubaguy | anyways, time for dinner | 18:59 |
| * johnthetubaguy bravely runs away | 18:59 | |
| mriedem | m-3 for novaclient? | 18:59 |
| dansmith | gate blocker.. coming up? | 18:59 |
| mriedem | i think the gate is already pretty much at a standstill | 18:59 |
| mriedem | check queue is 410 and counting | 18:59 |
| ndipanov | how come it's so slow? | 19:00 |
| mriedem | there are ~650 nodes max in nodepool | 19:00 |
| mriedem | everyone pushing changes for FF week | 19:00 |
| *** delattec has joined #openstack-nova | 19:00 | |
| dansmith | and nova jobs take 25 nodes... | 19:00 |
| *** delattec has quit IRC | 19:00 | |
| dansmith | each one | 19:00 |
| mriedem | and tempest is like 30+ jobs | 19:01 |
| *** haomaiwang has quit IRC | 19:01 | |
| mriedem | so yeah, lots of patches, lots of jobs, not enough resources | 19:01 |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| ndipanov | ah right FF week | 19:01 |
| dansmith | mriedem: can you comment here? https://review.openstack.org/#/c/281837/22/nova/db/sqlalchemy/migrate_repo/versions/318_resource_provider_name_aggregates.py | 19:02 |
| mriedem | yeah | 19:03 |
| dansmith | cdent: after you get mriedem's read, please push up a new set so we can try to get everyone rebased on top of yours.. | 19:03 |
| cdent | dansmith: yup | 19:03 |
| dansmith | edleafe: you're working on that feedback from johnthetubaguy I assume? | 19:03 |
| *** cdelatte has quit IRC | 19:03 | |
| dansmith | I have a new patch to push but will wait until you push, rebased on cdent's latest to avoid confusing the issue further | 19:03 |
| *** doug-fis_ has joined #openstack-nova | 19:03 | |
| edleafe | dansmith: yeah, just started but taking awhile because... meetings. :( | 19:04 |
| dansmith | okay | 19:05 |
| mriedem | commented | 19:06 |
| *** doug-fi__ has joined #openstack-nova | 19:06 | |
| *** doug-fi__ has quit IRC | 19:06 | |
| *** doug-fi__ has joined #openstack-nova | 19:07 | |
| *** jaosorior has quit IRC | 19:07 | |
| mriedem | basically, if no fkey, then no effing problem | 19:07 |
| *** doug-fish has quit IRC | 19:07 | |
| *** doug-fis_ has quit IRC | 19:08 | |
| *** dims_ has joined #openstack-nova | 19:11 | |
| mriedem | alaski: doffm: several questions / comments in https://review.openstack.org/#/c/278124/ | 19:11 |
| *** dims has quit IRC | 19:11 | |
| *** angdraug has joined #openstack-nova | 19:13 | |
| mriedem | now i've been sucked into the black hole that is the resource providers series | 19:13 |
| openstackgerrit | Chris Dent proposed openstack/nova: Adjust resource-providers models for resource-pools https://review.openstack.org/281837 | 19:14 |
| cdent | there we go dansmith, edleafe, johnthetubaguy | 19:14 |
| *** mtanino has quit IRC | 19:14 | |
| cdent | mriedem: it is a spooky place | 19:14 |
| *** jichen has joined #openstack-nova | 19:15 | |
| *** doug-fi__ is now known as doug-fish | 19:16 | |
| *** ducttape_ has quit IRC | 19:16 | |
| *** baoli has quit IRC | 19:18 | |
| *** baoli has joined #openstack-nova | 19:18 | |
| *** ihrachys has joined #openstack-nova | 19:20 | |
| *** e0ne has quit IRC | 19:20 | |
| pkoniszewski_ | andreykurilin__: still there? | 19:21 |
| *** cdelatte has joined #openstack-nova | 19:22 | |
| *** baoli has quit IRC | 19:23 | |
| *** baoli has joined #openstack-nova | 19:23 | |
| *** ducttape_ has joined #openstack-nova | 19:23 | |
| mriedem | alaski: policy question, could one disable file injection by setting the personality policy to role:admin? https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L421 | 19:24 |
| andreykurilin__ | pkoniszewski_: yup | 19:24 |
| alaski | mriedem: not with that rule, the *:discoverable rules just indicate whether or not the extension is advertised as being available | 19:26 |
| *** baoli has quit IRC | 19:26 | |
| alaski | API extension | 19:26 |
| *** dedery has joined #openstack-nova | 19:26 | |
| *** jdillaman has quit IRC | 19:27 | |
| mriedem | so if i used ! for the discoverable policy, and tried to inject files at boot, i'm assuming that should fail | 19:27 |
| mriedem | or at least not pass them on | 19:27 |
| mriedem | same with scheduler hints | 19:27 |
| alaski | I'm pretty sure it doesn't work like that. It just controls whether or not it shows up in /extensions | 19:28 |
| mriedem | ok, i'll test it out | 19:29 |
| alaski | yeah, I don't know for sure. but my understanding is that extensions are always loaded, but may not always be shown in the /extensions list | 19:31 |
| mriedem | alaski: yeah, you're right. | 19:34 |
| *** knikolla has quit IRC | 19:34 | |
| mriedem | i listed extensions and scheduler hints was in there, i changed the os-scheduler-hints:discoverable policy to ! and now it doesn't show up, | 19:34 |
| mriedem | but nothing prevents me from booting with --hint foo=bar | 19:34 |
| alaski | that is some functionality we lost, as it used to be preventable by not enabling the extension | 19:35 |
| *** mtanino has joined #openstack-nova | 19:35 | |
| alaski | though I think we intentionally lost that | 19:35 |
| mriedem | do we persist scheduler hints in the request_specs table now? | 19:37 |
| alaski | yes. that's mostly what it's for | 19:38 |
| mriedem | ah yup | 19:38 |
| *** baoli has joined #openstack-nova | 19:39 | |
| mriedem | thar she blar "scheduler_hints": {"foo": ["bar"]}} | 19:40 |
| mriedem | in my api db | 19:40 |
| *** tbachman has quit IRC | 19:41 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 19:42 |
| *** tongli has joined #openstack-nova | 19:42 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 19:43 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 19:44 |
| *** ihrachys has quit IRC | 19:44 | |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Add ResourcePool object https://review.openstack.org/284963 | 19:44 |
| *** achanda has joined #openstack-nova | 19:47 | |
| *** cfcp1 has quit IRC | 19:48 | |
| pkoniszewski_ | andreykurilin__: i can't get one thing fixed in GET/SHOW migrations | 19:50 |
| pkoniszewski_ | andreykurilin__: Shaohe wanted to check api version in https://review.openstack.org/#/c/281335/11/novaclient/tests/unit/v2/fakes.py@2388 | 19:51 |
| *** knikolla has joined #openstack-nova | 19:51 | |
| pkoniszewski_ | andreykurilin__: point is that api_version is not set in FakeSessionMockClient | 19:52 |
| bauzas | dansmith: /me is back | 19:53 |
| bauzas | dansmith: 'supp ? | 19:53 |
| dansmith | bauzas: I figured out my problem, but thanks | 19:53 |
| bauzas | dansmith: heh, no worries, the callcenter is 24/7 ;) | 19:54 |
| *** vladikr__ has joined #openstack-nova | 19:55 | |
| *** vladikr_ has quit IRC | 19:55 | |
| *** jdillaman has joined #openstack-nova | 19:56 | |
| *** achanda has quit IRC | 19:57 | |
| *** tbachman has joined #openstack-nova | 19:57 | |
| *** dims_ has quit IRC | 19:58 | |
| *** armax has quit IRC | 19:59 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| *** baoli has quit IRC | 20:02 | |
| *** davideagle has joined #openstack-nova | 20:03 | |
| bauzas | cdent: comment in https://review.openstack.org/#/c/281837/ but I'm still fine | 20:04 |
| cdent | thanks bauzas, looking | 20:04 |
| bauzas | cdent: tbc, so creating a new resource provider would be done by an API post, right? | 20:04 |
| *** shaleh has joined #openstack-nova | 20:04 | |
| bauzas | cdent: using the new scheduler endpoint | 20:05 |
| bauzas | cdent: I just want to make sure that duplicates on name are provided to the user by a better UX | 20:05 |
| bauzas | if we have an UC | 20:05 |
| cdent | bauzas: that would be in the domain of either the api, or the ov.object, neither of which are planned to be done in this cycle. My expectation is that the ov.object would raise an exception that the api would properly transform into something useful | 20:07 |
| bauzas | cdent: sure, I'm fine with that hence my +2 | 20:07 |
| cdent | ✔ | 20:07 |
| bauzas | cdent: I just want to make sure you're keeping that in your mind | 20:07 |
| cdent | yes | 20:07 |
| *** Pistachio has joined #openstack-nova | 20:08 | |
| cdent | thinking of those things is why I discovered the need for the additional constraint | 20:08 |
| cdent | given the choices made earlier today on what gets a -2, the 'name' field will remain hidden for quite a while | 20:08 |
| bauzas | okay | 20:09 |
| shaleh | while investigating a couple of of nova questions I noticed that there is inconsistent logging from keystonemiddleware when a request is handled. Is that expected behavior or do I have an issue with lost log messages? | 20:09 |
| * bauzas winks to mriedem for https://review.openstack.org/#/c/284959/ | 20:10 | |
| * mriedem blushes | 20:11 | |
| bauzas | woah, the gate is just awesomeness tonigh | 20:11 |
| bauzas | tonight | 20:11 |
| *** achanda has joined #openstack-nova | 20:11 | |
| *** angdraug has quit IRC | 20:11 | |
| *** dims has joined #openstack-nova | 20:12 | |
| bauzas | mriedem: you're lucky, I'm not on your Facebook timeline so I won't poke you | 20:12 |
| mriedem | bauzas: i'm not on facebook | 20:12 |
| mriedem | bauzas: alaski is questioning the version bump in that change | 20:12 |
| bauzas | mriedem: you're not loosing that much | 20:12 |
| bauzas | orly? missed that | 20:12 |
| *** david-lyle_ is now known as david-lyle | 20:12 | |
| bauzas | hah | 20:13 |
| bauzas | alaski: mriedem: soooo, about the cells rpcapi change, it was just a discussion with johnthetubaguy | 20:13 |
| *** pushkaru has quit IRC | 20:13 | |
| *** pushkaru has joined #openstack-nova | 20:13 | |
| bauzas | alaski: I don't honestly want to modify the payload for that | 20:14 |
| bauzas | alaski: like we discussed earlier | 20:14 |
| alaski | yeah, I agree | 20:14 |
| alaski | I just don't understand the version bump then | 20:14 |
| bauzas | but john and matt had some concerns also about having a nice way to see that | 20:14 |
| *** eharney_ has joined #openstack-nova | 20:15 | |
| alaski | to see that the payload didn't change? | 20:15 |
| mriedem | i guess my concern was just tracking the version change in case we needed to come back to it later for some reason | 20:15 |
| bauzas | alaski: yup | 20:15 |
| mriedem | maybe dansmith has an opinion | 20:15 |
| alaski | mriedem: like if something broke because we did actually change the payload? | 20:16 |
| bauzas | so, I'm just opiniated on the non-payload change, I'm fine with any outcome on how to provide some clear doc | 20:16 |
| dansmith | I always have an opinon.. let me read the backscroll to see what I have an opinion about this time | 20:16 |
| alaski | dansmith: https://review.openstack.org/#/c/284959/ | 20:16 |
| bauzas | dansmith: tl;dr some ComputeTaskAPI changes required the cells API to modify as well, but since cells v1 is quite feature-freeze, I didn't want to pass thru the new object | 20:17 |
| dansmith | alaski: it's a new rpc param so it needs a version bump right? | 20:17 |
| bauzas | dansmith: so, the question is : "how to comment that it's normal to modify the facade but not the RPC interface" | 20:18 |
| dansmith | if not we get a dispatch failure instead of a nice "unsupported version" message | 20:18 |
| alaski | dansmith: nothing about the rpc actually changes. rpcapi is taking a new param but purposely not sending it over the wire | 20:18 |
| dansmith | oh | 20:18 |
| bauzas | yup | 20:18 |
| dansmith | then yeah, no version bump | 20:18 |
| *** yamahata has quit IRC | 20:18 | |
| *** eharney has quit IRC | 20:19 | |
| bauzas | so I'll just comment that | 20:19 |
| *** Sam-I-Am has joined #openstack-nova | 20:19 | |
| Sam-I-Am | mriedem: ping | 20:20 |
| bauzas | mriedem: agreed ? | 20:20 |
| mriedem | bauzas: sure, i think that's what i was asking for last week, not necessary the version bump, but something documented in the code | 20:20 |
| bauzas | mriedem: you asked for either, so I'm good :) | 20:20 |
| mriedem | "If we aren't going to do that, we should probably put a note in the methods themselves that we are intentionally not passing the request_spec down b/c $reason." | 20:20 |
| *** tbachman has quit IRC | 20:20 | |
| mriedem | Sam-I-Am: pong | 20:20 |
| *** ZZelle_ has joined #openstack-nova | 20:20 | |
| bauzas | typey typey | 20:20 |
| Sam-I-Am | mriedem: hey. trying to solve an error where nova-compute keeps throwing an error about 'missing cpu model name' in the libvirt xml | 20:21 |
| Sam-I-Am | mriedem: usually if i set virt_type = qemu, things Just Work | 20:21 |
| Sam-I-Am | but in this case, no value for cpu_mode works | 20:21 |
| *** fawadkhaliq has quit IRC | 20:22 | |
| Sam-I-Am | sort of bashing my head on the desk because just using virt_type=qemu works in devstack and on most of my labs, but not in this environment | 20:22 |
| mriedem | Sam-I-Am: related? https://ask.openstack.org/en/question/82918/nova-compute-is-not-running-using-devstack/ | 20:24 |
| *** doug-fish has quit IRC | 20:24 | |
| mriedem | are you doing nested virt or not? | 20:24 |
| Sam-I-Am | mriedem: this is a SL box | 20:24 |
| Sam-I-Am | mriedem: its a vm | 20:25 |
| mriedem | Sam-I-Am: so things are working now and you just want to know why it's different in this case? or it's not working and you're looking for help? | 20:28 |
| Sam-I-Am | mriedem: its not working | 20:28 |
| Sam-I-Am | using virt_type=qemu and defaulting cpu_mode should Just Work for nova-compute running on a cloud server | 20:28 |
| Sam-I-Am | just in this case it isnt | 20:28 |
| Sam-I-Am | so i'm trying to figure out if i'm missing something obvious, or there's a deeper problem... perhaps with libvirt itself | 20:29 |
| *** dims has quit IRC | 20:29 | |
| *** doug-fish has joined #openstack-nova | 20:30 | |
| mriedem | off the top of my head i don't know, there are several similar cases on the forum https://ask.openstack.org/en/questions/scope:all/sort:activity-desc/page:1/query:missing%20cpu%20model%20name/ | 20:30 |
| mriedem | danpb is the expert but he's not around | 20:30 |
| mriedem | my devstack is using virt_type=qemu and cpu_mode=none | 20:31 |
| Sam-I-Am | mriedem: yeah, thats what i'm using here. and it breaks. | 20:31 |
| Sam-I-Am | which is why i suspect something else is going on | 20:32 |
| mriedem | "If virt_type="kvm|qemu", it will default to "host-model", otherwise it will default to "none"" | 20:32 |
| mriedem | so...have you explicitly tried using cpu_mode=none? | 20:32 |
| *** doug-fis_ has joined #openstack-nova | 20:32 | |
| *** tongli has quit IRC | 20:34 | |
| Sam-I-Am | yes | 20:34 |
| Sam-I-Am | thats what i have it set to now | 20:34 |
| mriedem | have you tried setting cpu_mode=host-model? | 20:34 |
| Sam-I-Am | yep. also fails. | 20:34 |
| *** doug-fish has quit IRC | 20:34 | |
| *** salv-orlando has joined #openstack-nova | 20:36 | |
| *** yarkot_ has joined #openstack-nova | 20:36 | |
| Sam-I-Am | mriedem: its interesting that 'virsh capabilities' lists no <model> under <cpu> | 20:36 |
| Sam-I-Am | there should be at least a qemu32 there | 20:36 |
| *** mylu has joined #openstack-nova | 20:37 | |
| *** doug-fis_ has quit IRC | 20:37 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix Cells RPC API by accepting a RequestSpec arg https://review.openstack.org/284959 | 20:37 |
| *** xyang1 has joined #openstack-nova | 20:38 | |
| *** yarkot_ has quit IRC | 20:38 | |
| *** salv-orl_ has quit IRC | 20:39 | |
| *** Sukhdev has joined #openstack-nova | 20:40 | |
| *** eharney_ is now known as eharney | 20:41 | |
| mriedem | Sam-I-Am: can you do a paste of what's in /usr/share/libvirt/cpu_map.xml ? | 20:41 |
| *** READ10 has quit IRC | 20:42 | |
| mriedem | i mean, i see several forum threads and an older ML thread on the same issue, w/o resolution | 20:42 |
| mriedem | w/o danpb i don't really have an answer | 20:42 |
| *** zenoway has quit IRC | 20:42 | |
| Sam-I-Am | mriedem: http://paste.openstack.org/show/488629/ | 20:43 |
| *** zenoway has joined #openstack-nova | 20:43 | |
| mriedem | i know the SL hosts are running centos 7.1 with a much newer libvirt than we use in the gate | 20:43 |
| *** lpetrut has joined #openstack-nova | 20:43 | |
| Sam-I-Am | mriedem: http://paste.openstack.org/show/488630/ | 20:43 |
| Sam-I-Am | thats what virsh capabilities shows on this box | 20:43 |
| *** burt has joined #openstack-nova | 20:44 | |
| Sam-I-Am | on my other lab nodes in the cloud, i see this: http://paste.openstack.org/show/488631/ | 20:44 |
| Sam-I-Am | notice the <model> in there | 20:44 |
| *** knikolla has quit IRC | 20:45 | |
| mriedem | and what's /proc/cpuinfo in your guest showing for a model? | 20:45 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets https://review.openstack.org/284833 | 20:45 |
| *** baoli has joined #openstack-nova | 20:47 | |
| Sam-I-Am | mriedem: http://paste.openstack.org/show/488633/ | 20:47 |
| Sam-I-Am | thats on the SL box | 20:47 |
| Sam-I-Am | well, SL vm | 20:47 |
| ZZelle_ | Hi everyone, is it possible to run a liberty nova-conductor with juno nova-computes? | 20:47 |
| *** baoli has quit IRC | 20:48 | |
| *** baoli has joined #openstack-nova | 20:48 | |
| *** MVenesio has quit IRC | 20:49 | |
| *** ivase_ has joined #openstack-nova | 20:49 | |
| *** MVenesio has joined #openstack-nova | 20:49 | |
| *** ociuhandu has quit IRC | 20:49 | |
| *** MVenesio has quit IRC | 20:50 | |
| *** ducttape_ has quit IRC | 20:50 | |
| mriedem | Sam-I-Am: and the image you're using has worked before? | 20:50 |
| Sam-I-Am | mriedem: i havent been able to get an instance to boot on this node | 20:51 |
| Sam-I-Am | because of this libvirt thing | 20:51 |
| mriedem | ok, so i'd ping danpb in the morning | 20:51 |
| mriedem | he's in london | 20:51 |
| openstackgerrit | Sean Dague proposed openstack/nova: Deprecate nova.hooks https://review.openstack.org/286276 | 20:52 |
| edleafe | dansmith: pushed an update to 284833 | 20:52 |
| dansmith | edleafe: okay cool | 20:52 |
| edleafe | dansmith: no real changes to the code, though - just the tests | 20:53 |
| Sam-I-Am | mriedem: k, thx for the help. | 20:53 |
| Sam-I-Am | mriedem: i bet its something simple and wonky :/ | 20:53 |
| *** jpomeroy has joined #openstack-nova | 20:53 | |
| *** ivase has quit IRC | 20:53 | |
| ivase_ | ZZelle: best test it yourself, will be surprised if it works you can use vagrant with openstack ansible | 20:54 |
| mriedem | no idea, what version of libvirt? | 20:54 |
| mriedem | like 1.2.17 or something? | 20:54 |
| *** ivase_ is now known as ivase | 20:55 | |
| *** Sukhdev has quit IRC | 20:55 | |
| Sam-I-Am | mriedem: 1.2.2 | 20:55 |
| dansmith | edleafe: ah, meant to say.. I don't think we need this: https://review.openstack.org/#/c/284833/6/nova/db/api.py | 20:55 |
| sdague | Sam-I-Am: what's an SL box? | 20:55 |
| dansmith | edleafe: we don't need to do this from outside db/sqlalchemy right? | 20:56 |
| Sam-I-Am | sdague: softlayer | 20:56 |
| jpomeroy | mriedem, other nova people, I got this question i'm hoping you can answer: Are nova extensions even still real or are they replaced by microversions or will they be there for a few releases? | 20:56 |
| Sam-I-Am | mriedem: same ver that runs in the gate iirc | 20:56 |
| dansmith | jpomeroy: they're gone | 20:57 |
| mriedem | Sam-I-Am: wait, that's the guest or the host? | 20:57 |
| edleafe | dansmith: I wasn't sure if it would be needed for the compute node objects | 20:57 |
| dansmith | edleafe: not so far | 20:57 |
| jpomeroy | so checking that an extension exists is kind of meaningless then right? | 20:57 |
| dansmith | but I guess it's not too bad to leave it | 20:57 |
| Sam-I-Am | mriedem: this is on the "host" (which is a SL VM) | 20:57 |
| sdague | Sam-I-Am: what does cat /proc/cpuinfo produce on that box? I wonder if libvirt can't guess based on the odd flags passed to it | 20:57 |
| edleafe | dansmith: well, it is a temp hack... | 20:58 |
| mriedem | sdague: http://paste.openstack.org/show/488633/ | 20:58 |
| dansmith | edleafe: yeah, which is why I was going to say leave it out | 20:58 |
| dansmith | but | 20:58 |
| ZZelle_ | ivase, no migration matrix with nova-conductor/nova-compute supported versions? | 20:58 |
| Sam-I-Am | sdague: http://paste.openstack.org/show/488633/ | 20:58 |
| *** angdraug has joined #openstack-nova | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| dansmith | edleafe: re: john's comments.. the existing tests are kinda covering the didn't-break-existing-functionality case right? | 21:01 |
| *** sneti has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| bauzas | dansmith: cdent: so I'm a bit worried by +Wing https://review.openstack.org/#/c/284484/ since I'm not a DB expert | 21:02 |
| dansmith | bauzas: yeah, that's cool | 21:02 |
| openstackgerrit | Pawel Koniszewski proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 21:02 |
| dansmith | bauzas: I'm +2 because I've beat it up from above and so I'm pretty confident that it's right | 21:02 |
| edleafe | dansmith: yes, exactly | 21:02 |
| dansmith | bauzas: I don't know who else to get to review it, but I think it's fine for you to not +W it out of comfort | 21:02 |
| *** ducttape_ has joined #openstack-nova | 21:02 | |
| bauzas | dansmith: given that, a revert could be possible, right? | 21:02 |
| cdent | bauzas: I'm fairly confident that the sqlA generates the desired sql | 21:03 |
| dansmith | edleafe: okay, I complained about whitespace damage, but otherwise I think it's okay | 21:03 |
| bauzas | okay, let's wait for another reviewer and we'll see | 21:03 |
| sdague | Sam-I-Am: I wonder if this is too new a proc for the libvirt you have to play the guessing game. | 21:03 |
| dansmith | bauzas: until my patch that uses the inventories then a revert is fine yeah | 21:03 |
| dansmith | bauzas: but yeah, I think it's fine to punt to someone else | 21:03 |
| bauzas | dansmith: what worries me is the state of the gate | 21:03 |
| *** davideagnello has quit IRC | 21:03 | |
| dansmith | bauzas: sure | 21:04 |
| Sam-I-Am | sdague: could be. how does one play the guessing game? | 21:04 |
| *** davideagnello has joined #openstack-nova | 21:04 | |
| sdague | I don't know, I think at this point it's down to reading libvirt code | 21:04 |
| Sam-I-Am | awesome :/ | 21:04 |
| bauzas | dansmith: but it seems the gate is better now | 21:04 |
| edleafe | dansmith: fixing the unrelated whitespace beautification... :) | 21:04 |
| dansmith | edleafe: call it damage or beautification, it's still unrelated :) | 21:05 |
| bauzas | that's the check pipeline which is bad | 21:05 |
| edleafe | dansmith: :) | 21:06 |
| *** ndipanov has quit IRC | 21:06 | |
| *** knikolla has joined #openstack-nova | 21:07 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets https://review.openstack.org/284833 | 21:08 |
| *** salv-orlando has quit IRC | 21:08 | |
| * SpamapS pays attention to the CPU discussion | 21:08 | |
| edleafe | dansmith: ^^ gift-wrapped for you | 21:08 |
| dansmith | edleafe: thanks | 21:08 |
| Sam-I-Am | SpamapS: lol | 21:08 |
| *** smatzek has quit IRC | 21:09 | |
| *** salv-orlando has joined #openstack-nova | 21:09 | |
| *** pushkaru has quit IRC | 21:09 | |
| *** baoli_ has joined #openstack-nova | 21:10 | |
| *** baoli_ has quit IRC | 21:10 | |
| *** raildo is now known as raildo-afk | 21:12 | |
| *** baoli_ has joined #openstack-nova | 21:13 | |
| *** baoli has quit IRC | 21:14 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Deprecate nova.hooks https://review.openstack.org/286276 | 21:14 |
| sdague | dansmith: fixed | 21:14 |
| * SpamapS tries libvirt from wily | 21:15 | |
| dansmith | sdague: got it thanks | 21:15 |
| *** rfolco_ has quit IRC | 21:16 | |
| SpamapS | hrm, wily is still only 1.2.16 | 21:16 |
| SpamapS | might have to try xenial | 21:16 |
| Sam-I-Am | SpamapS: which is still a lot newer than 1.2.2 | 21:17 |
| *** sheel has quit IRC | 21:17 | |
| SpamapS | xenial has 1.3 | 21:17 |
| SpamapS | http://paste.openstack.org/show/488641/ | 21:18 |
| SpamapS | Sam-I-Am: ^ | 21:18 |
| SpamapS | sdague: ^ | 21:18 |
| SpamapS | trying xenial | 21:18 |
| Sam-I-Am | SpamapS: yep, still no <model> | 21:19 |
| bauzas | dansmith: cdent: so I just figured out http://docs.sqlalchemy.org/en/latest/core/tutorial.html#using-joins for testing to translate SQLA into SQL | 21:19 |
| bauzas | just testing to play with it | 21:19 |
| dansmith | bauzas: look at my patch above these too | 21:19 |
| dansmith | which relies on it behaving properly | 21:19 |
| cdent | bauzas: you can print the generated query at any stage of the process | 21:20 |
| *** gszasz has joined #openstack-nova | 21:20 | |
| bauzas | cdent: how ? | 21:20 |
| cdent | one sec | 21:20 |
| *** zenoway has quit IRC | 21:22 | |
| cdent | bauzas: If I recall correctly each of those variables (e.g ram_inv_join) will stringify to the corresponding SQL, so, for example, you could use a debugger or print to stderr or whatever at line 770 of 'select' to see the generated sql | 21:23 |
| SpamapS | Sam-I-Am: same | 21:23 |
| SpamapS | http://paste.openstack.org/show/488644/ | 21:23 |
| *** dims has joined #openstack-nova | 21:23 | |
| *** vishwanathj has joined #openstack-nova | 21:24 | |
| bauzas | cdent: yeah that's basically what says the above SQLA doc | 21:25 |
| * bauzas testing | 21:25 | |
| *** lucas-dinner has quit IRC | 21:28 | |
| *** yamahata has joined #openstack-nova | 21:29 | |
| *** davideagnello has quit IRC | 21:32 | |
| *** browne has joined #openstack-nova | 21:33 | |
| *** cdelatte has quit IRC | 21:33 | |
| *** davideagnello has joined #openstack-nova | 21:34 | |
| *** aysyd has quit IRC | 21:35 | |
| *** ociuhandu has joined #openstack-nova | 21:35 | |
| *** lucasagomes has joined #openstack-nova | 21:36 | |
| *** dims has quit IRC | 21:40 | |
| *** cdelatte has joined #openstack-nova | 21:42 | |
| *** jichen has quit IRC | 21:43 | |
| *** dims has joined #openstack-nova | 21:44 | |
| *** browne has quit IRC | 21:47 | |
| *** davideagle has quit IRC | 21:48 | |
| mriedem | dansmith: tonyb: for after the stable meeting https://review.openstack.org/#/c/285710/ | 21:48 |
| *** baoli_ has quit IRC | 21:48 | |
| tonyb | mriedem: Oh yeah I meat to +2 that yesterday | 21:48 |
| mriedem | mmm meat | 21:48 |
| *** davideagle has joined #openstack-nova | 21:49 | |
| mriedem | claudiub: dansmith: mikal: this needs a +W too https://review.openstack.org/#/c/284761/ | 21:49 |
| *** mingyu has joined #openstack-nova | 21:50 | |
| *** fawadkhaliq has joined #openstack-nova | 21:50 | |
| tonyb | mriedem: speakign of meat. I watch what we do in the shadows. | 21:50 |
| tonyb | mriedem: +1 would watch again :) | 21:50 |
| mriedem | it's nice to know at least some people around here have good taste | 21:51 |
| sdague | mriedem: were you ever going to merge this to kilo - https://review.openstack.org/#/c/278992/ ? | 21:51 |
| mriedem | bauzas: would like your review on this backport also https://review.openstack.org/#/c/276191/ | 21:51 |
| tonyb | mriedem: I grew up in wellington so there was extra stuff for me :) | 21:51 |
| mriedem | sdague: yeah maybe | 21:51 |
| tonyb | mriedem: well not just me .... you know what I mean | 21:52 |
| *** cdelatte has quit IRC | 21:52 | |
| *** cdelatte has joined #openstack-nova | 21:53 | |
| *** armax has joined #openstack-nova | 21:53 | |
| *** edmondsw has quit IRC | 21:53 | |
| bauzas | dansmith: cdent: question (while jay isn't there) https://review.openstack.org/#/c/284484/2/nova/db/sqlalchemy/api.py@756 | 21:53 |
| bauzas | it's a nit, but why not provding the reserved disk and cpu ones ? | 21:54 |
| *** cdelatte has quit IRC | 21:54 | |
| *** haplo37 has quit IRC | 21:54 | |
| *** cdelatte has joined #openstack-nova | 21:54 | |
| *** dedery has quit IRC | 21:54 | |
| claudiub | mriedem: the hyper-v ci was set in debug / silent mode until the live migration is fixed in nova. ociuhandu posted the job results on: https://review.openstack.org/#/c/285370/ | 21:55 |
| *** cdelatte has quit IRC | 21:55 | |
| tonyb | sdague: I think we're waitign to see the impact on master now that the liberty (for grenade) backport landed | 21:55 |
| cdent | bauzas: looking | 21:56 |
| bauzas | cdent: that's something we can modify later (so still a nit) | 21:57 |
| mriedem | claudiub: +2 | 21:58 |
| *** haplo37 has joined #openstack-nova | 21:58 | |
| mriedem | claudiub: although rather than disabling the CI, wouldn't the problem tests just have been skipped? | 21:58 |
| *** vilobhmm11 has quit IRC | 21:58 | |
| mriedem | or set the tempest config to FALSE for live migration testing until it was fixed? | 21:58 |
| mriedem | s/wouldn't/couldn't/ | 21:59 |
| *** vilobhmm11 has joined #openstack-nova | 21:59 | |
| *** davideagle has quit IRC | 21:59 | |
| *** salv-orl_ has joined #openstack-nova | 21:59 | |
| claudiub | mriedem: true, but you wanted to see the live migration tests run. :) | 22:00 |
| *** vilobhmm11 has quit IRC | 22:00 | |
| mriedem | claudiub: i don't remember asking for that before this change, but sure | 22:00 |
| *** vilobhmm11 has joined #openstack-nova | 22:00 | |
| mriedem | my point is, if there is a specific regression that only impacts a subset of configurable tests in tempest, the CI should skip/disable those tests until it's fixed, not completely take the CI down | 22:00 |
| claudiub | mriedem: yeah, you asked me last week if the hyper-v ci runs the live migration tests. :) | 22:00 |
| mriedem | since other bad things could be sneaking in | 22:01 |
| *** haomaiwang has quit IRC | 22:01 | |
| mriedem | yeah, i was surprised | 22:01 |
| mriedem | anyway, for future reference, i'd personally prefer to see the CI remain up but skip problematic tests if there is a regression | 22:01 |
| *** haomaiwa_ has joined #openstack-nova | 22:01 | |
| claudiub | mriedem: sure, that's what we typically do anyways. | 22:01 |
| *** salv-orlando has quit IRC | 22:02 | |
| bauzas | mriedem: so https://review.openstack.org/#/c/276191/2 is a bug yes, but IMHO the related bug report is wrong | 22:02 |
| claudiub | mriedem: we do exclude tests that are failing or can't pass on the current hyper-v ci (for example, the nic hot plug test cannot pass on 2012 r2, but it can on 2016). | 22:03 |
| cdent | bauzas: I can't figure it out. My first assumption was that in the pre-resource provider world ram reservation was thing where disk and cpu were not, but I don't know how to look to really figure out what's going on . Maybe dansmith will have some ideas when he surfaces? | 22:03 |
| bauzas | mriedem: that's only if operators use those filters that we would have some problems (but those are disabled by default) | 22:03 |
| bauzas | mriedem: TBH, it's an Ironic problem hence probably the bug report (because Ironic deployments use those in https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L58-L60 = | 22:05 |
| mriedem | bauzas: yeah i saw all the ironic stuff in the bug report | 22:05 |
| mriedem | but 'operators would only have problems if they used this code' is kind of true for anything in nova right> | 22:05 |
| mriedem | ? | 22:05 |
| dansmith | cdent: I'm not sure what you're asking | 22:05 |
| mriedem | bugs in the vmware driver are only issues if you're using the vmware driver | 22:06 |
| *** doug-fish has joined #openstack-nova | 22:06 | |
| mriedem | bauzas: i guess my point was i wanted to know if you were ok with that going back to stable/liberty | 22:06 |
| bauzas | mriedem: so, I'm fine with backporting that since it's an Ironic-related bug or for operators explicitly using those filters | 22:06 |
| cdent | dansmith at line 756 https://review.openstack.org/#/c/284484/2/nova/db/sqlalchemy/api.py@756 there's some handling of reserved ram (and a question from bauzas, which is the question) | 22:06 |
| bauzas | mriedem: yes, it's a valid bug, not a wishlist one | 22:06 |
| mriedem | ok | 22:06 |
| bauzas | mriedem: I should have commented that in the master review btw. | 22:07 |
| auggy | claudiub: what's the nosiest pepper? | 22:07 |
| bauzas | mriedem: because that big 'scheduler' bug is just a placeholder | 22:07 |
| mriedem | bauzas: want to adjust the bug severity? | 22:07 |
| bauzas | mriedem: no, wishlist is perfect | 22:08 |
| auggy | claudiub: a jalapeño cuz it's jalapeño (all up in yo') business!!! | 22:08 |
| dansmith | ah, I don't know | 22:08 |
| mriedem | bauzas: ahh! | 22:08 |
| mriedem | bauzas: so it's a bug, but it's not a bug? | 22:08 |
| cdent | good eyes there bauzas | 22:08 |
| bauzas | mriedem: I should have just said to mgoddard to create another bug report for that specific issue | 22:08 |
| openstackgerrit | Pawel Koniszewski proposed openstack/python-novaclient: Add two server-migration commands and bump migration-list command https://review.openstack.org/281335 | 22:08 |
| *** knikolla has quit IRC | 22:09 | |
| mriedem | ok, so original problem is wishlist, | 22:09 |
| mriedem | thing he fixed is a bug | 22:09 |
| claudiub | auggy: hmm... puns. :D i didn't know that one. you seem familiar. | 22:09 |
| mriedem | and part of the overall thingy | 22:09 |
| bauzas | mriedem: exactly, sorry for being unclear | 22:09 |
| mriedem | np | 22:09 |
| *** dims has quit IRC | 22:09 | |
| auggy | claudiub: we were making bad puns on the bus at the midcycle :) | 22:10 |
| *** baoli has joined #openstack-nova | 22:10 | |
| mriedem | bauzas: btw, | 22:10 |
| bauzas | mriedem: http://i.dailymail.co.uk/i/pix/2014/06/13/article-2657035-1EB9365300000578-176_634x505.jpg | 22:10 |
| mriedem | bauzas: you know about tasty cheese right? | 22:10 |
| claudiub | auggy: yeees, i thought so. :D | 22:10 |
| bauzas | mriedem: heh, probably | 22:10 |
| mriedem | bauzas: but do you know about tasty children? http://www.cbsnews.com/news/minnesota-woman-admits-to-threatening-to-taste-children/ | 22:10 |
| auggy | mriedem: you know i was just commenting to someone that i keep reading "incredible" as "inedible" | 22:11 |
| *** browne has joined #openstack-nova | 22:11 | |
| *** browne has quit IRC | 22:12 | |
| bauzas | mriedem: heh, I only know MN this way http://totalpackers.com/wp-content/uploads/2013/10/BViF0EfIcAABtZ0.jpg | 22:12 |
| *** ijw has joined #openstack-nova | 22:12 | |
| mriedem | that is the most inedible sunset i've ever seen... | 22:12 |
| mriedem | bauzas: omfg | 22:12 |
| dansmith | that's wisconsin | 22:12 |
| mriedem | bauzas: that's WI | 22:12 |
| dansmith | you just insulted him big time | 22:12 |
| mriedem | for shame! | 22:12 |
| bauzas | mriedem: I know :D | 22:12 |
| dansmith | haha | 22:12 |
| bauzas | that's an HIMYM episode | 22:12 |
| claudiub | oooh... himym. <3 | 22:12 |
| *** browne has joined #openstack-nova | 22:12 | |
| dansmith | all you need to know about american culture you can learn from HIMYM | 22:13 |
| mriedem | is nasal guy from MN in the show? | 22:13 |
| *** mylu_ has joined #openstack-nova | 22:13 | |
| claudiub | i actually have a copy of the playbook and the bro code. :D | 22:13 |
| *** cdelatte has joined #openstack-nova | 22:13 | |
| *** mylu has quit IRC | 22:13 | |
| dansmith | claudiub: I have the bro code | 22:13 |
| bauzas | +1 | 22:13 |
| dansmith | claudiub: courtesy of sdague | 22:13 |
| claudiub | :)) | 22:13 |
| claudiub | i actually prefer the playbook. some of the plays are actually practical. :D | 22:14 |
| dansmith | heh | 22:14 |
| *** phschwartz_ is now known as phschwartz | 22:14 | |
| mriedem | sdague: tonyb: i'm +2 on https://review.openstack.org/#/c/278992/ now | 22:15 |
| mriedem | i had to check logstash first | 22:15 |
| claudiub | i think i saw that show like 5 times. :D | 22:15 |
| *** browne has quit IRC | 22:17 | |
| *** sdague has quit IRC | 22:18 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova-specs: Re-propose spec for refresh-quotas-usage https://review.openstack.org/286302 | 22:18 |
| *** yamahata has quit IRC | 22:18 | |
| *** yamahata has joined #openstack-nova | 22:19 | |
| ccarmack | mriedem: I uploaded the re-proposed spec for "nova-manage usage refresh" | 22:19 |
| *** armax has quit IRC | 22:19 | |
| mriedem | ccarmack: ok | 22:21 |
| mriedem | dansmith: alaski: we aren't actually approving newton specs yet are we? even if they are just re-proposed from mitaka? | 22:22 |
| dansmith | I would expect not | 22:22 |
| alaski | I don't think there's a hard rule against it, but it's definitely not a focus right now | 22:22 |
| *** claudiub has quit IRC | 22:23 | |
| *** armax has joined #openstack-nova | 22:26 | |
| *** eharney has quit IRC | 22:26 | |
| *** browne has joined #openstack-nova | 22:27 | |
| *** shaleh has quit IRC | 22:35 | |
| *** doug-fis_ has joined #openstack-nova | 22:35 | |
| *** thorst has quit IRC | 22:35 | |
| *** gszasz has quit IRC | 22:37 | |
| *** doug-fish has quit IRC | 22:37 | |
| *** Scalefab has joined #openstack-nova | 22:38 | |
| mriedem | i found a way to work this image into a conversation http://www.todayifoundout.com/wp-content/uploads/2010/08/zz_top2crop1.jpg | 22:41 |
| mriedem | which i think means i've had a successful day | 22:41 |
| dansmith | woot | 22:41 |
| *** baoli has quit IRC | 22:41 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 22:41 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 22:41 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 22:41 |
| ccarmack | mriedem: which one are you? | 22:42 |
| mriedem | ccarmack: one of the main guys | 22:42 |
| mriedem | !drummer | 22:42 |
| openstack | mriedem: Error: "drummer" is not a valid command. | 22:42 |
| dansmith | mriedem: you wish | 22:42 |
| *** thorst has joined #openstack-nova | 22:42 | |
| ccarmack | you are not the sharp dressed man | 22:42 |
| mriedem | you guys don't have the context of the conversation though | 22:42 |
| *** lbeliveau has quit IRC | 22:43 | |
| *** rtheis has quit IRC | 22:43 | |
| *** claudiub has joined #openstack-nova | 22:43 | |
| *** thorst has quit IRC | 22:46 | |
| *** rlrossit has quit IRC | 22:48 | |
| *** haplo37 has quit IRC | 22:48 | |
| *** doug-fis_ has quit IRC | 22:48 | |
| *** doug-fish has joined #openstack-nova | 22:48 | |
| *** davideagle has joined #openstack-nova | 22:50 | |
| *** mingyu has quit IRC | 22:50 | |
| *** MVenesio has joined #openstack-nova | 22:50 | |
| *** edtubill has quit IRC | 22:51 | |
| *** Scalefab has quit IRC | 22:52 | |
| *** kfarr has quit IRC | 22:53 | |
| *** doug-fish has quit IRC | 22:53 | |
| *** doug-fish has joined #openstack-nova | 22:53 | |
| *** achanda has quit IRC | 22:54 | |
| *** MVenesio has quit IRC | 22:56 | |
| *** baoli has joined #openstack-nova | 22:57 | |
| *** ivase has quit IRC | 22:58 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create BuildRequest object during boot process https://review.openstack.org/278125 | 23:00 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 23:00 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add BuildRequest object https://review.openstack.org/278124 | 23:00 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable https://review.openstack.org/278598 | 23:00 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Populate instance_mappings during boot https://review.openstack.org/263925 | 23:00 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Include CellMapping in InstanceMapping object https://review.openstack.org/286309 | 23:00 |
| *** thorst has joined #openstack-nova | 23:00 | |
| *** tbachman has joined #openstack-nova | 23:00 | |
| *** haomaiwa_ has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| *** baoli has quit IRC | 23:04 | |
| *** jpomeroy has quit IRC | 23:04 | |
| *** thorst has quit IRC | 23:05 | |
| *** mdrabe has quit IRC | 23:07 | |
| *** davideagle has quit IRC | 23:07 | |
| *** ivase has joined #openstack-nova | 23:08 | |
| *** burt has quit IRC | 23:08 | |
| *** achanda has joined #openstack-nova | 23:08 | |
| *** browne has quit IRC | 23:09 | |
| *** cdent has quit IRC | 23:09 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add manage command for cell0. https://review.openstack.org/275266 | 23:12 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 23:12 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 23:12 |
| *** lpetrut has quit IRC | 23:12 | |
| *** ivase has quit IRC | 23:12 | |
| *** yamahata has quit IRC | 23:12 | |
| *** dimtruck is now known as zz_dimtruck | 23:12 | |
| *** browne has joined #openstack-nova | 23:14 | |
| *** ctrath has quit IRC | 23:14 | |
| *** browne has quit IRC | 23:14 | |
| *** dims has joined #openstack-nova | 23:14 | |
| *** Sukhdev has joined #openstack-nova | 23:15 | |
| *** salv-orl_ has quit IRC | 23:15 | |
| *** salv-orlando has joined #openstack-nova | 23:19 | |
| dansmith | eff | 23:21 |
| dansmith | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3669 | 23:21 |
| *** cdelatte has quit IRC | 23:22 | |
| *** pkoniszewski_ has quit IRC | 23:24 | |
| *** Daisy has joined #openstack-nova | 23:28 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 23:28 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries https://review.openstack.org/279312 | 23:28 |
| openstackgerrit | Dan Smith proposed openstack/nova: Adjust resource-providers models for resource-pools https://review.openstack.org/281837 | 23:28 |
| openstackgerrit | Dan Smith proposed openstack/nova: Use new inventory schema in all compute_node gets https://review.openstack.org/284833 | 23:28 |
| openstackgerrit | Dan Smith proposed openstack/nova: Use new inventory schema in compute_node_get_all() https://review.openstack.org/284484 | 23:28 |
| *** penick has quit IRC | 23:29 | |
| *** penick has joined #openstack-nova | 23:31 | |
| *** Daisy has quit IRC | 23:31 | |
| *** Daisy has joined #openstack-nova | 23:32 | |
| *** pradk has quit IRC | 23:32 | |
| *** sc68cal has quit IRC | 23:34 | |
| *** EinstCrazy has joined #openstack-nova | 23:34 | |
| *** baoli has joined #openstack-nova | 23:35 | |
| *** EinstCrazy has quit IRC | 23:35 | |
| *** Daisy has quit IRC | 23:36 | |
| *** andymaier has joined #openstack-nova | 23:36 | |
| *** cdelatte has joined #openstack-nova | 23:39 | |
| *** browne has joined #openstack-nova | 23:43 | |
| *** xyang1 has quit IRC | 23:46 | |
| *** matrohon has quit IRC | 23:50 | |
| *** mylu_ has quit IRC | 23:52 | |
| openstackgerrit | Scott DAngelo proposed openstack/nova: api_version_request.matches does not accept a string or None https://review.openstack.org/285409 | 23:52 |
| *** Sam-I-Am has left #openstack-nova | 23:53 | |
| *** mylu has joined #openstack-nova | 23:55 | |
| *** Swami has quit IRC | 23:55 | |
| *** thorst has joined #openstack-nova | 23:56 | |
| *** suresh12 has joined #openstack-nova | 23:56 | |
| *** baoli has quit IRC | 23:57 | |
| *** mrkz has quit IRC | 23:57 | |
| *** thorst_ has joined #openstack-nova | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
