| *** hemna is now known as hemnafk | 00:01 | |
| *** ijw_ has joined #openstack-nova | 00:02 | |
| *** tjones1 has quit IRC | 00:03 | |
| *** tjones2 has joined #openstack-nova | 00:03 | |
| *** browne has joined #openstack-nova | 00:03 | |
| *** Alexandra_ has quit IRC | 00:06 | |
| *** ijw has quit IRC | 00:06 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Fix some issues in devref for api_microversions https://review.openstack.org/187133 | 00:09 |
|---|---|---|
| *** BadCub is now known as BadCub_Hiding | 00:10 | |
| *** shakamunyi has joined #openstack-nova | 00:12 | |
| *** barra204 has joined #openstack-nova | 00:12 | |
| *** gholler has quit IRC | 00:13 | |
| *** oro has quit IRC | 00:19 | |
| *** patrickeast has quit IRC | 00:20 | |
| *** ian_brown has joined #openstack-nova | 00:21 | |
| *** nelsnelson has quit IRC | 00:24 | |
| *** nelsnelson has joined #openstack-nova | 00:25 | |
| *** ijw_ has quit IRC | 00:27 | |
| *** ijw has joined #openstack-nova | 00:27 | |
| *** VW has joined #openstack-nova | 00:29 | |
| *** zhenguo has joined #openstack-nova | 00:33 | |
| *** Sukhdev has quit IRC | 00:35 | |
| *** xyang1 has quit IRC | 00:35 | |
| *** shashankhegde has quit IRC | 00:47 | |
| *** fawadkhaliq has quit IRC | 00:48 | |
| *** salv-orlando has joined #openstack-nova | 00:48 | |
| *** promulo__ has joined #openstack-nova | 00:52 | |
| *** yingjun has joined #openstack-nova | 00:53 | |
| *** gokrokve has quit IRC | 00:54 | |
| *** gokrokve has joined #openstack-nova | 00:55 | |
| *** salv-orlando has quit IRC | 00:55 | |
| *** mtanino has quit IRC | 00:56 | |
| *** promulo has quit IRC | 00:56 | |
| *** irenab_ has joined #openstack-nova | 00:57 | |
| *** bkopilov has quit IRC | 00:58 | |
| *** irenab has quit IRC | 00:58 | |
| *** irenab_ is now known as irenab | 00:58 | |
| *** tsekiyama has quit IRC | 00:59 | |
| *** gokrokve has quit IRC | 00:59 | |
| *** Raildo_ has joined #openstack-nova | 01:02 | |
| *** Raildo has quit IRC | 01:04 | |
| *** tjones2 has quit IRC | 01:05 | |
| *** tjones1 has joined #openstack-nova | 01:06 | |
| *** bkopilov has joined #openstack-nova | 01:08 | |
| *** nelsnelson has quit IRC | 01:12 | |
| *** superfly_ has joined #openstack-nova | 01:12 | |
| *** asettle has joined #openstack-nova | 01:13 | |
| *** superfly_ is now known as superflyy | 01:14 | |
| *** gokrokve has joined #openstack-nova | 01:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 01:17 |
| *** Raildo_ has quit IRC | 01:18 | |
| *** mmedvede has quit IRC | 01:20 | |
| *** vilobhmm1 has joined #openstack-nova | 01:20 | |
| *** vilobhmm has quit IRC | 01:21 | |
| *** angdraug has quit IRC | 01:21 | |
| *** barra204 has quit IRC | 01:24 | |
| *** shakamunyi has quit IRC | 01:24 | |
| *** superflyy has quit IRC | 01:24 | |
| *** mmedvede has joined #openstack-nova | 01:25 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 01:29 |
| *** alanf-mc has quit IRC | 01:31 | |
| *** browne has quit IRC | 01:31 | |
| *** ssurana has quit IRC | 01:31 | |
| *** ssurana has joined #openstack-nova | 01:32 | |
| *** tobe has joined #openstack-nova | 01:33 | |
| *** ijw has quit IRC | 01:34 | |
| *** ijw has joined #openstack-nova | 01:35 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova-specs: Allow ip6 server search for non-admin https://review.openstack.org/187835 | 01:36 |
| *** ssurana has quit IRC | 01:36 | |
| *** ian_brown has quit IRC | 01:39 | |
| *** baoli has quit IRC | 01:41 | |
| *** vilobhmm1 has quit IRC | 01:42 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: add support for muti_statuses_query from migration https://review.openstack.org/169631 | 01:43 |
| *** nelsnelson has joined #openstack-nova | 01:51 | |
| *** pixelbeat has quit IRC | 01:52 | |
| *** pixelbeat_ has joined #openstack-nova | 01:52 | |
| *** jyuso has joined #openstack-nova | 01:53 | |
| *** otter768 has joined #openstack-nova | 01:55 | |
| *** davideagnello has quit IRC | 01:58 | |
| *** JHelf has joined #openstack-nova | 01:58 | |
| *** otter768 has quit IRC | 01:59 | |
| *** VW has quit IRC | 02:04 | |
| *** VW has joined #openstack-nova | 02:05 | |
| *** nelsnelson has quit IRC | 02:05 | |
| *** shashankhegde has joined #openstack-nova | 02:06 | |
| *** vivek_nandavanam has joined #openstack-nova | 02:06 | |
| *** shashankhegde has quit IRC | 02:06 | |
| *** ianbrown has joined #openstack-nova | 02:07 | |
| *** salv-orlando has joined #openstack-nova | 02:08 | |
| *** rushiagr_away is now known as rushiagr | 02:09 | |
| *** asettle has quit IRC | 02:11 | |
| *** sdake has quit IRC | 02:11 | |
| *** asettle has joined #openstack-nova | 02:12 | |
| *** JHelf has quit IRC | 02:12 | |
| *** asettle has quit IRC | 02:13 | |
| *** salv-orlando has quit IRC | 02:14 | |
| *** bzhao has joined #openstack-nova | 02:15 | |
| *** tjones1 has quit IRC | 02:15 | |
| *** rfolco has quit IRC | 02:17 | |
| *** yamahata has quit IRC | 02:17 | |
| *** zxen has joined #openstack-nova | 02:20 | |
| *** mohitsharma has joined #openstack-nova | 02:20 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 02:22 |
| *** unicell has quit IRC | 02:22 | |
| *** baoli has joined #openstack-nova | 02:23 | |
| *** IanGovett has joined #openstack-nova | 02:26 | |
| *** haigang has quit IRC | 02:26 | |
| *** hshiina has quit IRC | 02:26 | |
| *** yingjun has quit IRC | 02:26 | |
| *** haigang has joined #openstack-nova | 02:27 | |
| *** annegentle has joined #openstack-nova | 02:28 | |
| *** yingjun has joined #openstack-nova | 02:29 | |
| *** haigang has quit IRC | 02:30 | |
| *** haigang has joined #openstack-nova | 02:32 | |
| *** achanda has joined #openstack-nova | 02:32 | |
| *** mpaolino has joined #openstack-nova | 02:33 | |
| *** haigang has quit IRC | 02:33 | |
| *** haigang has joined #openstack-nova | 02:34 | |
| openstackgerrit | Auktavian Garrett proposed openstack/nova: Use unique values for hypervisor_hostnames https://review.openstack.org/187844 | 02:35 |
| *** Kennan2 has joined #openstack-nova | 02:37 | |
| *** Kennan has quit IRC | 02:38 | |
| *** haigang has quit IRC | 02:38 | |
| *** IanGovett has quit IRC | 02:39 | |
| *** rushiagr is now known as rushiagr_away | 02:40 | |
| *** mpaolino has quit IRC | 02:40 | |
| *** nelsnelson has joined #openstack-nova | 02:41 | |
| *** annegentle has quit IRC | 02:44 | |
| *** haigang has joined #openstack-nova | 02:46 | |
| *** Kennan2 is now known as Kennan | 02:49 | |
| *** baoli has quit IRC | 02:49 | |
| *** ijw has quit IRC | 02:50 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 02:51 |
| *** annegentle has joined #openstack-nova | 02:51 | |
| *** annegentle has quit IRC | 02:51 | |
| *** unicell has joined #openstack-nova | 02:52 | |
| *** tjones1 has joined #openstack-nova | 02:55 | |
| *** dims__ has quit IRC | 02:56 | |
| *** hshiina has joined #openstack-nova | 02:58 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref https://review.openstack.org/162913 | 02:59 |
| openstackgerrit | Alex Xu proposed openstack/nova: The devref for Nova stable API https://review.openstack.org/162912 | 02:59 |
| *** gyee is now known as operator99 | 03:00 | |
| *** ddaskal has joined #openstack-nova | 03:00 | |
| *** achanda has quit IRC | 03:02 | |
| *** achanda has joined #openstack-nova | 03:03 | |
| *** achanda has quit IRC | 03:08 | |
| *** unicell has quit IRC | 03:09 | |
| *** unicell has joined #openstack-nova | 03:10 | |
| *** mohitsharma has quit IRC | 03:13 | |
| *** asettle has joined #openstack-nova | 03:13 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Filter out inactive compute firstly https://review.openstack.org/180155 | 03:14 |
| *** asettle has quit IRC | 03:15 | |
| *** asettle has joined #openstack-nova | 03:15 | |
| *** tjones1 has left #openstack-nova | 03:16 | |
| *** mohitsharma has joined #openstack-nova | 03:17 | |
| *** gokrokve_ has joined #openstack-nova | 03:17 | |
| *** alanf-mc has joined #openstack-nova | 03:21 | |
| *** gokrokve has quit IRC | 03:21 | |
| *** gokrokve_ has quit IRC | 03:21 | |
| *** dboik has joined #openstack-nova | 03:23 | |
| *** alanf-mc has quit IRC | 03:25 | |
| *** dboik has quit IRC | 03:27 | |
| *** nelsnelson has quit IRC | 03:30 | |
| *** achanda has joined #openstack-nova | 03:34 | |
| *** hshiina has quit IRC | 03:34 | |
| *** yingjun has quit IRC | 03:34 | |
| *** kiran-r has joined #openstack-nova | 03:38 | |
| *** markvoelker has quit IRC | 03:44 | |
| *** iamjarvo has joined #openstack-nova | 03:49 | |
| *** Longgeek_ has quit IRC | 03:53 | |
| *** alexpilotti has joined #openstack-nova | 03:54 | |
| *** arnaud____ has joined #openstack-nova | 03:54 | |
| *** Longgeek has joined #openstack-nova | 03:54 | |
| *** otter768 has joined #openstack-nova | 03:55 | |
| *** browne has joined #openstack-nova | 03:55 | |
| *** Longgeek_ has joined #openstack-nova | 03:57 | |
| *** achanda has quit IRC | 03:58 | |
| *** achanda has joined #openstack-nova | 03:58 | |
| *** ddaskal has quit IRC | 03:59 | |
| *** changbl_ has joined #openstack-nova | 04:00 | |
| *** Longgeek has quit IRC | 04:00 | |
| *** otter768 has quit IRC | 04:01 | |
| *** hshiina has joined #openstack-nova | 04:01 | |
| *** Longgeek has joined #openstack-nova | 04:02 | |
| *** achanda has quit IRC | 04:03 | |
| *** Longgeek_ has quit IRC | 04:04 | |
| *** Longgeek_ has joined #openstack-nova | 04:08 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 04:08 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: imagebackend: Split out resize_image logic from create_image https://review.openstack.org/187857 | 04:08 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: imagebackend: Split out resize_image logic from create_image https://review.openstack.org/187857 | 04:09 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 04:10 |
| *** salv-orlando has joined #openstack-nova | 04:10 | |
| *** Longgeek has quit IRC | 04:10 | |
| *** nelsnelson has joined #openstack-nova | 04:11 | |
| *** gabor has joined #openstack-nova | 04:12 | |
| *** mdrabe has joined #openstack-nova | 04:12 | |
| *** gabor has quit IRC | 04:12 | |
| *** Longgeek_ has quit IRC | 04:12 | |
| *** artom has quit IRC | 04:13 | |
| *** gabor-t has joined #openstack-nova | 04:14 | |
| *** pixelbeat_ has quit IRC | 04:14 | |
| *** Longgeek has joined #openstack-nova | 04:18 | |
| *** Longgeek has quit IRC | 04:18 | |
| *** nic has quit IRC | 04:18 | |
| *** Longgeek has joined #openstack-nova | 04:19 | |
| *** Longgeek has quit IRC | 04:20 | |
| *** Longgeek has joined #openstack-nova | 04:20 | |
| *** Longgeek_ has joined #openstack-nova | 04:23 | |
| *** Longgee__ has joined #openstack-nova | 04:25 | |
| *** Longgeek has quit IRC | 04:25 | |
| *** jecarey has joined #openstack-nova | 04:25 | |
| *** VW has quit IRC | 04:26 | |
| *** Longgeek_ has quit IRC | 04:26 | |
| *** Longgeek has joined #openstack-nova | 04:26 | |
| *** Longgeek_ has joined #openstack-nova | 04:28 | |
| *** mdrabe has quit IRC | 04:28 | |
| *** sorrison_laptop has joined #openstack-nova | 04:29 | |
| *** Longge___ has joined #openstack-nova | 04:29 | |
| *** Longgee__ has quit IRC | 04:30 | |
| *** coolsvap|afk is now known as coolsvap | 04:30 | |
| *** coolsvap is now known as coolsvap|afk | 04:31 | |
| *** Longgeek has quit IRC | 04:32 | |
| *** coolsvap|afk is now known as coolsvap | 04:32 | |
| *** Longgeek_ has quit IRC | 04:32 | |
| *** bzhao has quit IRC | 04:34 | |
| *** changbl_ has quit IRC | 04:40 | |
| *** krtaylor has quit IRC | 04:40 | |
| *** hshiina has quit IRC | 04:44 | |
| *** markvoelker has joined #openstack-nova | 04:45 | |
| *** markvoelker has quit IRC | 04:50 | |
| *** garyk has joined #openstack-nova | 04:53 | |
| *** ildikov has quit IRC | 04:54 | |
| *** kiran-r has quit IRC | 04:59 | |
| *** nkrinner has joined #openstack-nova | 05:03 | |
| *** mohitsharma has quit IRC | 05:05 | |
| *** ildikov has joined #openstack-nova | 05:13 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref https://review.openstack.org/162913 | 05:15 |
| openstackgerrit | Alex Xu proposed openstack/nova: The devref for Nova stable API https://review.openstack.org/162912 | 05:15 |
| *** jecarey has quit IRC | 05:15 | |
| *** yamahata has joined #openstack-nova | 05:19 | |
| *** arnaud____ has quit IRC | 05:21 | |
| *** mpaolino has joined #openstack-nova | 05:23 | |
| *** mpaolino has quit IRC | 05:25 | |
| openstackgerrit | Ritesh proposed openstack/nova: Users will not be allowed to delete instance during snapshot https://review.openstack.org/185942 | 05:25 |
| openstackgerrit | Ritesh proposed openstack/nova: Users will not be allowed to delete instance during snapshot https://review.openstack.org/185942 | 05:26 |
| *** krtaylor has joined #openstack-nova | 05:27 | |
| *** exploreshaifali has joined #openstack-nova | 05:27 | |
| *** apuimedo has quit IRC | 05:27 | |
| *** hshiina has joined #openstack-nova | 05:36 | |
| *** nihilifer has joined #openstack-nova | 05:37 | |
| *** ildikov has quit IRC | 05:38 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova-specs: Allow ip6 server search for non-admin https://review.openstack.org/187835 | 05:38 |
| *** vilobhmm has joined #openstack-nova | 05:42 | |
| *** moshele has joined #openstack-nova | 05:42 | |
| *** exploreshaifali has quit IRC | 05:43 | |
| *** yingjun has joined #openstack-nova | 05:43 | |
| *** dulek has joined #openstack-nova | 05:46 | |
| *** jyuso has quit IRC | 05:49 | |
| frickler | Microsoft Hyper-V CI check takes 4 hours to fail on a doc only check, good job: https://review.openstack.org/187133 | 05:49 |
| frickler | I fail to find some error in their log, too | 05:49 |
| frickler | s/check/patch/ | 05:50 |
| *** veena has joined #openstack-nova | 05:50 | |
| *** sudipto has joined #openstack-nova | 05:50 | |
| *** ankit_ag has joined #openstack-nova | 05:51 | |
| *** Maike has joined #openstack-nova | 05:52 | |
| *** vipul has quit IRC | 05:53 | |
| *** gabor-t has quit IRC | 05:54 | |
| *** vipul has joined #openstack-nova | 05:54 | |
| *** mpaolino has joined #openstack-nova | 05:55 | |
| *** mohitsharma has joined #openstack-nova | 05:56 | |
| *** otter768 has joined #openstack-nova | 05:56 | |
| *** tcammann_ has quit IRC | 05:58 | |
| *** iamjarvo has quit IRC | 05:58 | |
| *** slong_ has joined #openstack-nova | 05:58 | |
| *** lpetrut has joined #openstack-nova | 05:58 | |
| *** tcammann_ has joined #openstack-nova | 05:58 | |
| *** vilobhmm has quit IRC | 05:58 | |
| *** slong has quit IRC | 05:59 | |
| *** josecastroleon has joined #openstack-nova | 05:59 | |
| *** armax has quit IRC | 06:01 | |
| *** otter768 has quit IRC | 06:01 | |
| *** mpaolino has quit IRC | 06:01 | |
| *** ildikov has joined #openstack-nova | 06:04 | |
| *** mpaolino has joined #openstack-nova | 06:04 | |
| *** radek_ has joined #openstack-nova | 06:07 | |
| *** sahid has joined #openstack-nova | 06:07 | |
| *** stefanb has quit IRC | 06:08 | |
| *** mpaolino has quit IRC | 06:09 | |
| *** kiran-r has joined #openstack-nova | 06:10 | |
| *** asettle has quit IRC | 06:11 | |
| *** romainh has joined #openstack-nova | 06:11 | |
| *** shakamunyi has joined #openstack-nova | 06:16 | |
| *** barra204 has joined #openstack-nova | 06:17 | |
| *** mnestratov has quit IRC | 06:17 | |
| *** mnestratov has joined #openstack-nova | 06:17 | |
| *** stefanb has joined #openstack-nova | 06:22 | |
| *** belmoreira has joined #openstack-nova | 06:26 | |
| *** zxen has quit IRC | 06:27 | |
| *** sorrison_laptop has quit IRC | 06:27 | |
| *** slong__ has joined #openstack-nova | 06:27 | |
| openstackgerrit | Ritesh proposed openstack/nova: Users will not be allowed to delete instance during snapshot https://review.openstack.org/185942 | 06:28 |
| *** moshele is now known as moshele_ | 06:28 | |
| *** moshele_ has quit IRC | 06:28 | |
| *** slong_ has quit IRC | 06:28 | |
| *** moshele has joined #openstack-nova | 06:29 | |
| *** mohitsharma has quit IRC | 06:29 | |
| *** mohitsharma has joined #openstack-nova | 06:29 | |
| *** nelsnelson has quit IRC | 06:30 | |
| *** markvoelker has joined #openstack-nova | 06:34 | |
| *** ianbrown has quit IRC | 06:35 | |
| *** Longgeek has joined #openstack-nova | 06:35 | |
| *** bzhao has joined #openstack-nova | 06:37 | |
| *** Longge___ has quit IRC | 06:37 | |
| *** markvoelker has quit IRC | 06:38 | |
| *** oro_ has joined #openstack-nova | 06:38 | |
| *** cfriesen has quit IRC | 06:39 | |
| *** slong__ has quit IRC | 06:40 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: libvirt: add missing iotune parse for LibvirtConfigGuestDisk https://review.openstack.org/187881 | 06:51 |
| *** pradipta has joined #openstack-nova | 06:55 | |
| sudipto | sahid, hello, regarding the review of https://review.openstack.org/185942 - i wanted to understand in the extensions.py - why should we not add a more generic base class of the exception which could be something like InstanceInvalidState ? | 06:58 |
| *** sahid has quit IRC | 07:01 | |
| *** radek_ has quit IRC | 07:02 | |
| *** browne has quit IRC | 07:02 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: libvirt:add instance disk IO control for rbd backend https://review.openstack.org/187888 | 07:04 |
| *** radek_ has joined #openstack-nova | 07:05 | |
| *** mpavone has joined #openstack-nova | 07:13 | |
| openstackgerrit | garyk proposed openstack/nova: DB: add support for volume_type by BDMS https://review.openstack.org/139992 | 07:13 |
| openstackgerrit | garyk proposed openstack/nova: Object: add volume_type_id to block_device objects https://review.openstack.org/139994 | 07:14 |
| *** Nakato has quit IRC | 07:16 | |
| *** exploreshaifali has joined #openstack-nova | 07:18 | |
| *** radek__ has joined #openstack-nova | 07:18 | |
| *** Nakato has joined #openstack-nova | 07:18 | |
| *** FL1SK has quit IRC | 07:18 | |
| *** gryf|afk is now known as _gryf | 07:19 | |
| *** haigang has quit IRC | 07:19 | |
| *** alex_klimov has joined #openstack-nova | 07:19 | |
| *** radek_ has quit IRC | 07:20 | |
| *** bradjones has quit IRC | 07:20 | |
| *** rotbeard has joined #openstack-nova | 07:22 | |
| *** bradjones has joined #openstack-nova | 07:22 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Refactor network API 'get_instance_nw_info' https://review.openstack.org/146036 | 07:24 |
| *** mohitsharma has quit IRC | 07:27 | |
| *** matrohon has joined #openstack-nova | 07:27 | |
| *** mohitsharma has joined #openstack-nova | 07:27 | |
| *** jyuso1 has joined #openstack-nova | 07:29 | |
| *** radek_ has joined #openstack-nova | 07:35 | |
| *** radek__ has quit IRC | 07:37 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix resource leaking when consume_from_instance raise exception https://review.openstack.org/182165 | 07:38 |
| *** pbandzi has joined #openstack-nova | 07:39 | |
| *** coolsvap is now known as coolsvap|afk | 07:41 | |
| *** belmoreira has quit IRC | 07:42 | |
| openstackgerrit | Zhang Ni proposed openstack/nova-specs: Add volume type to create server API https://review.openstack.org/187812 | 07:44 |
| *** jcoufal has joined #openstack-nova | 07:45 | |
| *** markus_z has joined #openstack-nova | 07:45 | |
| *** belmoreira has joined #openstack-nova | 07:46 | |
| *** coolsvap|afk is now known as coolsvap | 07:46 | |
| *** jistr has joined #openstack-nova | 07:51 | |
| *** tobe has quit IRC | 07:52 | |
| kaisers1 | Hi Nova! Small question: where exactly do i configure how much RAM can be used by nova-compute on a given nova-compute host? That is not set by the quota_RAM settings, isn't it? | 07:54 |
| gilliard | kaisers1: by nova-compute itself? | 07:55 |
| *** otter768 has joined #openstack-nova | 07:57 | |
| *** dims_ has joined #openstack-nova | 07:58 | |
| kaisers1 | gilliard: yep. I have a default RDO installation (Juno) that by default seems to provide 8GB RAM top be shared among VMs on a given compute host. As the machine has more available i wanted to reconfigure this. | 07:59 |
| kaisers1 | '...top ...' was '... to ...' | 07:59 |
| *** ndipanov has joined #openstack-nova | 08:00 | |
| kaisers1 | gilliard: but i only find options for quotas (default something like 52GB) and reserved_host_memory_mb. | 08:00 |
| kaisers1 | gilliard: The latter would make sense to me if nova would provide all available RAM to VMs... | 08:01 |
| *** otter768 has quit IRC | 08:02 | |
| *** rushiagr_away is now known as rushiagr | 08:02 | |
| gilliard | kaisers1: I only know of reserverd_host_memory_mb, but I think it does what you suggest... | 08:03 |
| *** dims_ has quit IRC | 08:03 | |
| gilliard | or, I *thought* it did ;) | 08:03 |
| kaisers1 | gilliard: Weird. Maybe there's a different layer in tenant management, etc. that filters available memory per host? I've just a very big gap between physical memory available and memory allowed by nova compute when instantiating VMs :-D | 08:04 |
| *** jlanoux has joined #openstack-nova | 08:05 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref https://review.openstack.org/162913 | 08:08 |
| openstackgerrit | Alex Xu proposed openstack/nova: The devref for Nova stable API https://review.openstack.org/162912 | 08:08 |
| *** exploreshaifali has quit IRC | 08:08 | |
| gilliard | yeah quotas will be per-tenant. Are you sure seeing boot requests fail for not-enough-memory reasons? | 08:08 |
| *** Kennan2 has joined #openstack-nova | 08:11 | |
| *** Kennan has quit IRC | 08:11 | |
| *** Nakato has quit IRC | 08:13 | |
| *** lucasagomes has joined #openstack-nova | 08:14 | |
| *** tobe has joined #openstack-nova | 08:14 | |
| *** Nakato has joined #openstack-nova | 08:14 | |
| *** yingjun has quit IRC | 08:14 | |
| *** oro_ has quit IRC | 08:15 | |
| *** yingjun has joined #openstack-nova | 08:19 | |
| *** derekh has joined #openstack-nova | 08:20 | |
| *** apuimedo has joined #openstack-nova | 08:21 | |
| *** sahid has joined #openstack-nova | 08:22 | |
| *** markvoelker has joined #openstack-nova | 08:22 | |
| bauzas | kaisers1: you can allocate more than what you want, since there are allocation ratios for RAM | 08:23 |
| bauzas | kaisers1: defaulting to 16 btw. | 08:23 |
| *** vivek_nandavanam has quit IRC | 08:23 | |
| *** lpetrut has quit IRC | 08:23 | |
| moshele | pczesno: hi | 08:25 |
| bauzas | gilliard: reserved_host_memory_mb is for making sure that you leave some space for else | 08:26 |
| bauzas | whatelse | 08:26 |
| *** danpb has joined #openstack-nova | 08:26 | |
| *** yassine_ has joined #openstack-nova | 08:27 | |
| bauzas | kaisers1 : so the RAM thay you dedicate to Nova per compute would be "(total RAM - reserved_host_memory_mb) * ram_allocation_ratio" | 08:27 |
| *** markvoelker has quit IRC | 08:27 | |
| bauzas | kaisers1: it will give you how much RAM you can cap | 08:27 |
| *** ZZelle has quit IRC | 08:28 | |
| *** ZZelle has joined #openstack-nova | 08:28 | |
| kaisers1 | bauzas: thanks for the info! | 08:30 |
| kaisers1 | bauzas: But there has to be another (maybe project related?) setting. I'm looking at a 64GB RAM machine, reserved_host_memory_mb is 16384. Nova shows 8GB RAM available for VMs. I'd expect around 48GB to be available for VMs, no? | 08:34 |
| *** heyongli has quit IRC | 08:34 | |
| kaisers1 | Maybe there are other (hypervisor? other?) related settings that influence this? | 08:34 |
| bauzas | kaisers1: your ram allocation ratio ? | 08:34 |
| kaisers1 | No, the total RAM available for VMs | 08:35 |
| bauzas | kaisers1: I mean, could you please give it to me ? | 08:35 |
| kaisers1 | bauzas: hmm, not set, what is the default value? | 08:36 |
| bauzas | kaisers1: 16 | 08:37 |
| kaisers1 | bauzas: setting this to 1.0 and restarting nova-compute.... | 08:37 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: docs: Add bug handling description to developer reference https://review.openstack.org/187571 | 08:37 |
| bauzas | kaisers1: technically, we're not following the $subject of this chan, but let's continue to see if there is bug | 08:37 |
| gilliard | BTW if you have the source checked out, run "tox -e genconfig" then see etc/nova/nova.conf.sample to find doc and defaults for all config options. | 08:38 |
| kaisers1 | bauzas: oh, sry. Wich channel would be appropriate? | 08:38 |
| bauzas | kaisers1: #openstack | 08:38 |
| bauzas | kaisers1: but let's continue | 08:38 |
| kaisers1 | bauzas: ok | 08:38 |
| bauzas | the room is quite quiet now | 08:38 |
| bauzas | kaisers1: so, no instances ? | 08:38 |
| bauzas | kaisers1: check the n-cpu log, it will tell you the physical capacity and the ram in use | 08:39 |
| kaisers1 | moment, checking on the changes i did | 08:39 |
| bauzas | kaisers1: here is how it calculates the usage https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L800-L826 | 08:39 |
| kaisers1 | hmm, now the available RAM is equal to the quota limit, yay! | 08:40 |
| kaisers1 | ~51GB, that's the default of the quota_RAM setting | 08:41 |
| *** ianbrown has joined #openstack-nova | 08:41 | |
| bauzas | cool | 08:42 |
| kaisers1 | Ok, from what i see now nova tells me 16GB are in use (the reserved_host_memory_mb? Is that show as 'in use'?). Up to a total of 52GB RAM is available for VMs. That's what i was aiming at | 08:42 |
| bauzas | kaisers1: yup for reserved_ram, see my link above | 08:43 |
| kaisers1 | That's with no running guests. | 08:43 |
| bauzas | kaisers1: yeah | 08:43 |
| kaisers1 | yep, will read now. Thanks a lot | 08:43 |
| bauzas | kaisers1: try to understand how the RT works | 08:43 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: docs: fixup libvirt NUMA testing docs to match reality https://review.openstack.org/178773 | 08:43 |
| kaisers1 | gilliard: bauzas: To both of you!! | 08:43 |
| bauzas | kaisers1: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L393 is the entry point, called every 60 secs | 08:44 |
| bauzas | kaisers1: follow the code, you'll see how Nova counts | 08:44 |
| kaisers1 | bauzas: ack | 08:44 |
| *** garyk has quit IRC | 08:44 | |
| *** garyk has joined #openstack-nova | 08:45 | |
| *** garyk has quit IRC | 08:46 | |
| *** garyk has joined #openstack-nova | 08:46 | |
| *** heyongli has joined #openstack-nova | 08:49 | |
| *** heyongli has quit IRC | 08:51 | |
| *** nkrinner has quit IRC | 08:52 | |
| openstackgerrit | Joe Gordon proposed openstack/nova: Prune development.environment doc https://review.openstack.org/182884 | 08:53 |
| openstackgerrit | Joe Gordon proposed openstack/nova: Remove jenkins, launchpad and gerrit docs https://review.openstack.org/183342 | 08:53 |
| openstackgerrit | Joe Gordon proposed openstack/nova: Make blueprints doc a reference for nova blueprints https://review.openstack.org/183343 | 08:53 |
| openstackgerrit | Joe Gordon proposed openstack/nova: Remove unit_test doc https://review.openstack.org/183697 | 08:53 |
| gilliard | kaisers1: :) | 08:55 |
| mdbooth | Is there any reason that we would have a requirement for oslo.db>=1.7.0 rather than the latest 1.8.0? | 08:56 |
| *** ianbrown has quit IRC | 08:56 | |
| mdbooth | I submitted a patch which uses 1.8.0. Works on some jenkins tests, not on others. | 08:56 |
| *** nkrinner has joined #openstack-nova | 08:57 | |
| mdbooth | On the ones where it doesn't it seems it hasn't pulled in the latest oslo.db | 08:57 |
| mdbooth | And the requirements is definitely bugged | 08:57 |
| johnthetubaguy | mdbooth: we have been pinning some releases due to compatibility issues, but I don't remember anything about oslo.db | 08:58 |
| mdbooth | johnthetubaguy: It doesn't currently have an upper bound in requirements.txt | 08:58 |
| johnthetubaguy | mdbooth: oh, you mean its just the old version thats installed in the test VM? | 08:59 |
| mdbooth | Yeah | 08:59 |
| mdbooth | However, the old version meets the constraint in requirements.txt | 08:59 |
| johnthetubaguy | mdbooth: hmm, is there anything in global requirements? | 08:59 |
| mdbooth | So that's obviously incorrect if I'm using new features | 08:59 |
| mdbooth | johnthetubaguy: No. Just wondering if there's any reason for that | 08:59 |
| mdbooth | There was an upper bound before L | 09:00 |
| mdbooth | So it was change to remove the upper bound, but the lower bound was not bumped | 09:00 |
| johnthetubaguy | yeah, we added a bound for the stable stuff | 09:00 |
| johnthetubaguy | unsure to be honest, dims and/or sdague would be almost certainly be able to tell you the answer | 09:00 |
| mdbooth | johnthetubaguy: Git says it was updated in a global removal of upper bounds, so likely no specific thought | 09:03 |
| * mdbooth submits a patch to global-requirements.txt, waits for review, waits for committee approval, waits for filter-down merge in nova, rebases patch on updated requirements.txt, waits for patch review | 09:04 | |
| kashyap | :-) | 09:05 |
| mdbooth | Ah, well. I submitted v1 of the patch over 6 months ago. I guess another 6 months won't hurt :) | 09:05 |
| kashyap | johnthetubaguy: Do you have a comment on this thread (especially, shortening the release cycle to 2 months aspect) - http://lists.openstack.org/pipermail/openstack-dev/2015-February/057614.html | 09:05 |
| *** lpetrut has joined #openstack-nova | 09:05 | |
| *** heyongli has joined #openstack-nova | 09:06 | |
| *** tellesnobrega has quit IRC | 09:07 | |
| *** heyongli has quit IRC | 09:07 | |
| mdbooth | Step 1 complete: https://review.openstack.org/187918 | 09:08 |
| *** dguitarbite has quit IRC | 09:09 | |
| *** dguitarbite has joined #openstack-nova | 09:09 | |
| *** dguitarbite has quit IRC | 09:10 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: Fix SMBFS volume attach race condition https://review.openstack.org/187673 | 09:11 |
| * danpb wonders if there's any cores around who can review a numa docs fix so we stop confusing our users https://review.openstack.org/#/c/178773/ | 09:15 | |
| *** FL1SK has joined #openstack-nova | 09:15 | |
| *** claudiub has joined #openstack-nova | 09:17 | |
| *** paul-carlton has joined #openstack-nova | 09:20 | |
| *** oro_ has joined #openstack-nova | 09:21 | |
| *** afazekas has joined #openstack-nova | 09:22 | |
| *** lpetrut has quit IRC | 09:22 | |
| *** paul-carlton has quit IRC | 09:25 | |
| *** gcb has joined #openstack-nova | 09:27 | |
| frickler | mdbooth: how about adding a reference to your nova patch into the commit message for the g-r update? and maybe also to the oslo.db update that introduced the function you need. | 09:33 |
| *** gszasz has joined #openstack-nova | 09:34 | |
| mdbooth | frickler: I could, and I considered that. However, I decided against it as that specific context seems irrelevant in the git history. | 09:35 |
| mdbooth | In fact, just the mention of update_match seems irrelevant. It's just an initial motivation. | 09:35 |
| mdbooth | There are presumably lots of potential users of new functionality. | 09:35 |
| mdbooth | More of a covering note thing. | 09:35 |
| *** yingjun has quit IRC | 09:36 | |
| mdbooth | However, I could be convinced otherwise if it would speed the process. | 09:36 |
| frickler | mdbooth: well, obviously I can't promise it will really help in the end, but with the commit message in its current form, for me it raises the question "Where does he need that?" and it might help if that question had a direct answer | 09:39 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: docs: Add bug handling description to developer reference https://review.openstack.org/187571 | 09:39 |
| *** pixelbeat_ has joined #openstack-nova | 09:39 | |
| *** abhishekk has joined #openstack-nova | 09:44 | |
| *** paul-carlton has joined #openstack-nova | 09:44 | |
| *** ianbrown has joined #openstack-nova | 09:47 | |
| *** pradipta has quit IRC | 09:48 | |
| johnthetubaguy | kashyap: danpb: looking at the thread, we consider every commit to be a release, but we do have lots of freezes, which are awkward | 09:51 |
| *** sudipto has quit IRC | 09:51 | |
| *** veena has quit IRC | 09:53 | |
| *** Kevin_Zheng has quit IRC | 09:55 | |
| *** Administrator_ has quit IRC | 09:55 | |
| markus_z | mdbooth: About review https://review.openstack.org/#/c/148273/ What can I do there? | 09:56 |
| *** ianbrown has quit IRC | 09:56 | |
| markus_z | mdbooth: I think a change of the commit message would be sufficient to +1 it. | 09:57 |
| *** lpetrut has joined #openstack-nova | 09:57 | |
| johnthetubaguy | danpb: your email on release schedule changes, have you seen what ironic are looking to do? its an interesting twist on what you are proposing here | 09:57 |
| *** otter768 has joined #openstack-nova | 09:58 | |
| *** dguitarbite has joined #openstack-nova | 09:59 | |
| markus_z | johnthetubaguy: Which email are you referring to? | 09:59 |
| johnthetubaguy | danpb: I have been doing some thinking about the release cycles, and honestly I think SemVer helps with a lot of these | 09:59 |
| johnthetubaguy | markus_z: http://lists.openstack.org/pipermail/openstack-dev/2015-February/057614.html | 09:59 |
| markus_z | johnthetubaguy: thanks | 09:59 |
| *** paul-carlton has quit IRC | 09:59 | |
| sdague | mdbooth: you are correct, if you are submitting patches that use new oslo.db features, you should get g-r bumped first with that explanation. The minimum should be real. | 10:00 |
| danpb | johnthetubaguy: yep, i saw their proposal, but I'm not a fan of the idea of doing feature based releases in general | 10:00 |
| johnthetubaguy | danpb: as in you prefer the time based releases? | 10:01 |
| danpb | johnthetubaguy: i've seen too many projects go wrong with that, as they continually delay the release "just one more week" to let the feature get ready | 10:01 |
| johnthetubaguy | yeah, that so happens | 10:01 |
| * kashyap notices SemVer = http://semver.org/ | 10:01 | |
| danpb | from what I've experianced strict time based releases are more effective - /if/ they are frequent enough | 10:01 |
| *** tellesnobrega has joined #openstack-nova | 10:02 | |
| danpb | which was why i suggested a 2 monthly release cycle, so we'd still match up with the 6 month cycle of other projects | 10:02 |
| danpb | but be able to deliver a couple of extra intermediate releases | 10:02 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add functional test for server group https://review.openstack.org/147514 | 10:02 |
| johnthetubaguy | danpb: so we do 6 weekly release right now, I am curious why they are not useful | 10:02 |
| *** otter768 has quit IRC | 10:03 | |
| johnthetubaguy | to be clear, they are not being consumed by people, so there must be something stopping that | 10:03 |
| danpb | well if we tagged those 6 weekly releases as formal production deployable releases that'd work fine too | 10:03 |
| mdbooth | markus_z: So you would like a link to the nova commit which uses it? | 10:03 |
| johnthetubaguy | danpb: so I am thinking we can do that now, as we move to semver | 10:03 |
| danpb | but the messaging around them is that they are development snapshots which i think discourages people from using them | 10:03 |
| johnthetubaguy | danpb: gotcha | 10:03 |
| johnthetubaguy | danpb: I think the plan is liberty is 12.x and M is probably 12.x | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: scheduler:refactor group info in filter_properties https://review.openstack.org/148275 | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: api: add soft-affinity policies for server groups https://review.openstack.org/147520 | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add functional tests to soft affinity capabilities https://review.openstack.org/147521 | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: scheduler: add num of instance per host in a group https://review.openstack.org/148277 | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: object:count members per hosts in InstanceGroup https://review.openstack.org/147517 | 10:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers https://review.openstack.org/147516 | 10:04 |
| johnthetubaguy | danpb: I am thinking we make liberty 1, something like 12.0.1 | 10:04 |
| * gilliard notes happily that semver.org is semantically versioned | 10:04 | |
| johnthetubaguy | danpb: and then liberty is 12.1.0 (the first stable release), then we start 13.0.0.dev1 | 10:05 |
| *** dims_ has joined #openstack-nova | 10:06 | |
| danpb | isn't that still giving the impression that liberty 1 is a lesser thing compared to liberty-final | 10:06 |
| johnthetubaguy | danpb: I am thinking if we actually give every commit an automatic version, but only tag on the milestones, so each commit gets, like if we were on the 42nd commit since master opened for liberty it would be: 12.0.0.dev42 | 10:06 |
| johnthetubaguy | danpb: so it kinda is, because the docs folks only really help with 12.1 | 10:07 |
| johnthetubaguy | danpb: I am thinking about upgrade here too | 10:07 |
| johnthetubaguy | 12.0 -> 13.0 means we can drop live upgrade code at those moments | 10:07 |
| johnthetubaguy | and thats the point where we force people to upgrade | 10:07 |
| johnthetubaguy | now that doesn't have to be every 6 months, but we could start that way | 10:07 |
| *** mohitsharma has quit IRC | 10:08 | |
| johnthetubaguy | danpb: a way of looking at the milestones is that we are doing a mini tick/tock style release train | 10:09 |
| *** markvoelker has joined #openstack-nova | 10:11 | |
| johnthetubaguy | danpb: 12.0.1 is open for all sorts of features (risky stuff goes here), 12.0.2 is also open to all features, 12.0.3 is only priority features, 12.1 only adds bug fixes on 12.0.3 | 10:11 |
| *** exploreshaifali has joined #openstack-nova | 10:12 | |
| danpb | so that's more or less what we do now today | 10:12 |
| dims_ | haypo: got a bunch of tests working python34 - https://review.openstack.org/#/c/187785/ | 10:13 |
| dims_ | oops he is not here | 10:13 |
| mdbooth | dims_: Thanks! | 10:13 |
| danpb | which leads to a very bursty development cycle which creates instability due to crunch times when lots of big features are all shoved in at roughly the same time | 10:13 |
| johnthetubaguy | danpb: agreed, but thats probably easier to understand, and we can evolve that once we work out why we are doing what we are doing today (plus understanding what we are doing) | 10:13 |
| dims_ | mdbooth: i went further to 0.10.0 :) | 10:13 |
| danpb | a big part of my idea was to try to smooth out the cycle to eliminate the highs & lows | 10:13 |
| mdbooth | dims_: tbh, I'm pretty sure I meant 0.10.0 | 10:13 |
| danpb | by making each time based release equivalent in the types of stuff accepted | 10:14 |
| mdbooth | However, I couldn't work out how version numbers are calculated anyway | 10:14 |
| dims_ | mdbooth: haha :) | 10:14 |
| danpb | so people don't feel them need to rush to shove in a big features when its not ready, just in order to avoid slipping another 6 months | 10:14 |
| danpb | iow, try and get rid of the idea that we have a specific time where we shove in big risky stuff | 10:15 |
| *** markvoelker has quit IRC | 10:15 | |
| danpb | as if we say we're allowed to shove in destablizing stuff at point X, it becomes a self-fullfilling prophecy | 10:16 |
| johnthetubaguy | danpb: agreed with the self-fullfilling prophecy, the "big risky stuff" is meant to be an exception, and generally not referring to a feature, but thats certainly not true any more | 10:17 |
| johnthetubaguy | danpb: so my take for this release, is that we should be better at communicating what we are doing and why, so we can have a better debate on what to do for M | 10:18 |
| *** garyk has quit IRC | 10:18 | |
| johnthetubaguy | danpb: its slower than I would want, but we get the feedback our users want stability over features, and thats what the process is optimising for, at some level | 10:18 |
| *** garyk has joined #openstack-nova | 10:18 | |
| *** garyk has quit IRC | 10:19 | |
| danpb | i guess if we can at least start improving the messaging around intermediate releases to encourage their use, that'd be a start in the right direction | 10:19 |
| *** garyk has joined #openstack-nova | 10:19 | |
| danpb | don't have to change everything at once | 10:19 |
| johnthetubaguy | danpb: yeah, thats true | 10:19 |
| johnthetubaguy | danpb: I think when we started calling them a1 and a2 we broke the original messaging there | 10:20 |
| johnthetubaguy | danpb: I am cool with doing that, plus advertising the fact that every commit is "released" in some sense | 10:20 |
| danpb | btw, another aspect of the proposal was that we stop marking specs against specific releases - allow specs to be submitted& reviewed & approved at any time and once approved allow their validity to remain in force for 12 months | 10:20 |
| mdbooth | johnthetubaguy: Ironically, though, stability can involve significant change without introducing new features. | 10:20 |
| danpb | so we avoid the tedium of having to re-submit a whole bunch of specs at the start of each cycle | 10:20 |
| johnthetubaguy | danp: then add that the 12.x -> 13.x is a forced upgrade point, to make sure we do live-upgrade with minimal technical debt | 10:21 |
| sahid | danpb: +1 | 10:21 |
| johnthetubaguy | danpb: the backlog concept helps there somewhat | 10:21 |
| johnthetubaguy | but yeah, the current version to too heavy weight | 10:22 |
| danpb | mostly just to cut down on work we've creating for ourselves | 10:22 |
| johnthetubaguy | mdbooth: you are totally right, the efforts to get more stability are quite de-stabilising, look at all the work to formalise the interface, rewrite cells so its more maintainable and a first class concept, plans to add task tracking and API, etc | 10:23 |
| johnthetubaguy | danpb: we have done a lot of that with abandoning milestone tracking, but yes, more of that is needed | 10:24 |
| danpb | yeah, that was an improvement | 10:24 |
| johnthetubaguy | danpb: just for the record, I think unlearning things is more important than adding process! | 10:24 |
| johnthetubaguy | (its also harder) | 10:24 |
| *** sahid has quit IRC | 10:25 | |
| *** mwagner_afk has quit IRC | 10:26 | |
| *** Gomeler has quit IRC | 10:34 | |
| johnthetubaguy | danpb: I am sending a proposal to the ML | 10:34 |
| *** samueldmq has joined #openstack-nova | 10:37 | |
| samueldmq | morning | 10:37 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/nova: Ironic: Parse and validate Node's properties https://review.openstack.org/176890 | 10:38 |
| *** qianli_ has quit IRC | 10:39 | |
| *** sudipto has joined #openstack-nova | 10:41 | |
| *** sudipto has quit IRC | 10:42 | |
| *** sudipto has joined #openstack-nova | 10:43 | |
| *** sudipto has quit IRC | 10:44 | |
| *** sudipto has joined #openstack-nova | 10:45 | |
| *** bzhao has quit IRC | 10:46 | |
| *** sudipto has quit IRC | 10:49 | |
| *** bzhao has joined #openstack-nova | 10:51 | |
| *** jyuso1 has quit IRC | 10:52 | |
| *** exploreshaifali has quit IRC | 10:54 | |
| *** garyk has quit IRC | 10:55 | |
| *** garyk has joined #openstack-nova | 10:55 | |
| * johnthetubaguy runs away for lunch | 10:56 | |
| *** garyk has quit IRC | 10:56 | |
| *** garyk has joined #openstack-nova | 10:56 | |
| *** vladikr has quit IRC | 10:56 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: add support for cores per socket https://review.openstack.org/187942 | 10:58 |
| *** sdake has joined #openstack-nova | 11:02 | |
| *** flwang1 has joined #openstack-nova | 11:05 | |
| *** sudipto has joined #openstack-nova | 11:07 | |
| samueldmq | sdague, hi, I'd like to talk about the dynamic policies + nova needs, you around ? | 11:07 |
| sdague | samueldmq: yep | 11:08 |
| *** lyanchih has joined #openstack-nova | 11:08 | |
| samueldmq | sdague, nice .. from what I understood, you main use case is to warn the user if he's changing the default and that may impact on the success of some operations | 11:09 |
| sdague | samueldmq: I also tried to put more details back on list posts | 11:09 |
| samueldmq | sdague, am I right? | 11:09 |
| sdague | samueldmq: that's one of the things that emerged as "we'd like to do that" | 11:09 |
| samueldmq | sdague, I saw a couple of messages in the ML | 11:09 |
| samueldmq | sdague, nice, what if we did that for all projects ? | 11:10 |
| sdague | samueldmq: I'm pro doing it for all projects | 11:10 |
| samueldmq | sdague, and the defaults (in code) would be managed by the policy management server | 11:10 |
| sdague | that's why I said "features we want in oslo.policy" | 11:10 |
| *** exploreshaifali has joined #openstack-nova | 11:10 | |
| sdague | samueldmq: ok, so where does the policy management server get it's seed of truth? | 11:10 |
| samueldmq | sdague, one of the ideas is to have a separate service for policy storage + management (changing the api's granularly, etc) | 11:11 |
| *** lyanchih has quit IRC | 11:11 | |
| samueldmq | sdague, I guess that would be in the code of the policy management service | 11:12 |
| samueldmq | sdague, that once started, will consider to load its database based on i) the defaults (probably in the code) + ii) the unified policy.json file | 11:12 |
| sdague | how does that work with Nova adding a feature, policy that goes with it, and it being CDed into production while all other components remain the same | 11:12 |
| sdague | I feel like the unified policy.json file makes an implicit assumption of lock step deploys of all components all at once | 11:13 |
| sdague | which ... isn't the real world | 11:13 |
| samueldmq | once we have the unified policy + dynamic changes is accepted, services will need to update i) local policies for backwards compt + ii) unified policy | 11:14 |
| samueldmq | but that's a different concern | 11:14 |
| garyk | danpb: thanks! will check if that works | 11:14 |
| sdague | samueldmq: I still don't understand how that anticipates version skew | 11:14 |
| samueldmq | sdague, so maybe I am not understanding your question | 11:15 |
| samueldmq | sdague, how do we keep the unified policy consistent with the inidividual ones from services ? | 11:15 |
| samueldmq | sdague, is that your question ? | 11:16 |
| sdague | especially when the components are upgrading on different schedules with new features being added all the time which add new policy bits | 11:16 |
| samueldmq | sdague, got it, I am not sure I have the complete answer for this question, but I will talk about this and define it better in the specs | 11:17 |
| sdague | right, the staggered upgrade seems intractable if you assume that all policy starts from a central registry, instead of federated up | 11:18 |
| *** lucasagomes has quit IRC | 11:18 | |
| sdague | it's kind of like saying "what if we remove all the db migrations from all the projects and have 1 db migration service instead" | 11:18 |
| sdague | which works only if everything is going to lock step all the time | 11:18 |
| samueldmq | sdague, looks like we will need to have a new version of this service each time any service using it does have a new version | 11:18 |
| sdague | samueldmq: projects are designed to be Continuously Deployed | 11:19 |
| sdague | so that's every git commit | 11:19 |
| sdague | for every project | 11:19 |
| sdague | also, there is no reason to assume it's linear. Rax has a 2 year old cinder, for instance. | 11:20 |
| samueldmq | sdague, hmm, good question then... I will bring this into discussion with other folks | 11:20 |
| samueldmq | sdague, yeah, and what if some api's have disappeared ? | 11:20 |
| sdague | this is why I wanted to get this into a wider discussion (and honestly, preferably on the mailing list, so we don't keep having point discussions and relays) | 11:21 |
| samueldmq | sdague, and you havve a 2 year old cinder, but running master on other components | 11:21 |
| *** flwang1 has quit IRC | 11:21 | |
| sdague | yep | 11:21 |
| samueldmq | sdague, nice, got it | 11:21 |
| sdague | which is why I think the only real answer is to have all projects that are going to be part of this implement a /policy end point to fetch their seed of truth | 11:21 |
| sdague | hopefully I explained it reasonably on the list | 11:22 |
| *** zul has quit IRC | 11:22 | |
| sdague | Nova makes for a good test case for this because we've got a huge API, people do CD nova, and we're doing API changes with microversions (and potentially policy adds) many times a cycle | 11:22 |
| sdague | I think that the original model of a unified file didn't account for much of that | 11:23 |
| garyk | danpb: with your suggestion of the topology do i need to change a scheduling filter? | 11:23 |
| danpb | not with kvm at least | 11:24 |
| openstackgerrit | Abhishek Kekane proposed openstack/nova: Extract cinder-client response to log request IDs https://review.openstack.org/187948 | 11:24 |
| danpb | any host can run vms with any cores/socket topology | 11:24 |
| garyk | the flavor has - "capabilities:cpu_info:topology:sockets": "2" - but the vanilla scheduling is failing | 11:24 |
| danpb | so there's no need to filter anything | 11:24 |
| samueldmq | sdague, yeah, at very least, if we go for that unified thing, changes in let's say nova which introduce new api's would be using 'depends-on' on the change in the policy management server | 11:24 |
| sdague | samueldmq: that's not good enough | 11:24 |
| sdague | because you assume a time linear relationship | 11:25 |
| sdague | now in order to update nova, you have to deploy that policy server as well | 11:25 |
| *** aix has quit IRC | 11:25 | |
| sdague | but what if there were also depends on from Trove | 11:25 |
| sdague | but you aren't upgrading trove for 6 months | 11:25 |
| samueldmq | sdague, deployments do not use 'depends-on' :/ | 11:25 |
| *** rushiagr is now known as rushiagr_away | 11:25 | |
| sdague | samueldmq: more importnatly, they shouldn't. A project needs to be able to roll forward independent of other projects | 11:26 |
| sdague | the policy server (which owns the policy source of truth) assumes a very specific deployment model that we've never required in OpenStack | 11:27 |
| samueldmq | sdague, if we go for implementing the /policy thing in each project | 11:28 |
| samueldmq | sdague, how do we handle old projects that don't know anything about the dynamic policies ? | 11:28 |
| *** zul has joined #openstack-nova | 11:28 | |
| sdague | they don't get dynamic policies until they support it | 11:28 |
| garyk | danpb: i get: The capabilities '' couldn't be loaded from JSON: No JSON object could be decoded. I am trying to investigate. something is fishy | 11:28 |
| samueldmq | sdague, oh that's easy, they keep enforcing as they were doing | 11:28 |
| sdague | but they weren't going to get it anyway, because they would need code changes to fetch and enforce policy from keystone anyway | 11:29 |
| samueldmq | sdague, only get, enforcement is kept at the service side | 11:29 |
| sdague | samueldmq: sure | 11:29 |
| danpb | garyk: i dunno what that flavor capabilities stuff is for - its not related to the cpu topology stuff i mentioned though | 11:29 |
| samueldmq | sdague, keystonemiddleware will be in charge of caching and updating the last policy for the endpoint it is service | 11:30 |
| sdague | but you still need code in a project to pull the dynamic computed policy back from keystone for checks | 11:30 |
| samueldmq | it is serving* | 11:30 |
| sdague | ok, sure | 11:30 |
| samueldmq | sdague, nice | 11:30 |
| sdague | I also think this kind of approach is more big tent friendly | 11:30 |
| *** ociuhandu has quit IRC | 11:30 | |
| garyk | danpb: ok, thanks. | 11:30 |
| samueldmq | sdague, ++ | 11:30 |
| sdague | because any project that does this gets to have dynamic policy, and keystone doesn't need to know about it semantically. It just needs to comply to some interfaces | 11:31 |
| samueldmq | sdague, those points on the unified policy need to be discussed on the ML | 11:31 |
| sdague | samueldmq: yeh... so check out my last posts | 11:31 |
| *** ferest has joined #openstack-nova | 11:31 | |
| samueldmq | sdague, I will talk to ayoung, we should have a good answer for that, and then we reply the ML | 11:31 |
| sdague | great | 11:31 |
| ankit_ag | Hi all, Jenkins is failing for os.utime() with Permission denied error on one of my patch https://review.openstack.org/#/c/185549/1, But it is not reproducible on local environment. So it looks like a jenkins issue for me. Is someone have any idea about this jenkins failure. | 11:32 |
| samueldmq | sdague, thanks | 11:32 |
| sdague | the trick of this being right is getting a very detailed specification for the /policy required interface, because the code implementing it will be different in different servers as they have different wsgi layers | 11:32 |
| samueldmq | sdague, ++ | 11:32 |
| ankit_ag | Please refer Jenkins logs at http://logs.openstack.org/49/185549/1/check/check-tempest-dsvm-nova-v21-full/17c7e67/logs/screen-n-cpu.txt.gz#_2015-05-28_06_51_49_809 | 11:32 |
| samueldmq | sdague, completely agree | 11:32 |
| danpb | garyk: unless you are doing strict cpu pinning with no over commit, it is pointless trying to match guest cores/sockets to host cores/sockets topologies | 11:32 |
| danpb | as you are not going to get any performance benefit that isn't wiped out by the over commit usage | 11:33 |
| garyk | danpb: the problem is that the scheduler expects the host cpu info in a json format. | 11:33 |
| samueldmq | sdague, policies change in different timings for different projects, unifying them may be a trap when services changes its interfaces | 11:33 |
| garyk | the vmware driver does not return this as the VC does the scheduling. so we have a little conflict here | 11:34 |
| garyk | i will see if we can report the sockets. | 11:34 |
| danpb | why are you trying to set schedular filters at all though | 11:34 |
| samueldmq | sdague, services versions would be very dependent on the new policy management service version | 11:34 |
| samueldmq | sdague, I think that summarizes well the issue we have been discussing | 11:34 |
| garyk | danpb: i have not changed the scheduling filters. they are the defautl ones | 11:34 |
| samueldmq | sdague, :) | 11:34 |
| danpb | "capabilities:cpu_info:topology:sockets": "2" | 11:34 |
| danpb | that's not a default setting is it | 11:35 |
| garyk | danpb: that is following your suggestion on review https://review.openstack.org/#/c/187942/ | 11:35 |
| garyk | we need to pass the number of sockets to the driver. | 11:35 |
| markus_z | mdbooth: sorry, I was AFK. I meant that the commit message could provide more details what is solved by that change. | 11:36 |
| *** ferest_ has joined #openstack-nova | 11:36 | |
| garyk | danpb: this setting does not work with the scheduling filter ComputeCapabilitiesFilter | 11:36 |
| markus_z | mdbooth: Does it solve bug 1412436 or should the test be active when bug 1412436 is solved? | 11:36 |
| openstack | bug 1412436 in OpenStack Compute (nova) "Race in instance_create with security_group_destroy" [Low,Confirmed] https://launchpad.net/bugs/1412436 | 11:36 |
| danpb | garyk: huh ? it shouldn't need anything more than the 2 lines of code i illustrated afaik | 11:37 |
| markus_z | uh, nice feature | 11:37 |
| danpb | the capabilities:cpu_info:topology:sockets thing is filtering host cpu topology - that's completely independent & unrelated to the question of setting guest CPU topology | 11:37 |
| frickler | ankit_ag: the same check passed before you did your recheck, so hopefully another recheck might solve it. there also seems to have been some issues with the nodepool earlier today | 11:38 |
| sdague | samueldmq: :) | 11:38 |
| danpb | a host with 4 sockets and 2 cores per socket is fine hosting a guest with virtual topology 2 sockets and 4 cores | 11:38 |
| sdague | samueldmq: yeh, I think that's the crux of it. And that we actually want to free people from having to do lock step upgrades of everything all at once, not make them have to do more of that. | 11:39 |
| *** ferest has quit IRC | 11:39 | |
| garyk | danpb: https://github.com/openstack/nova/blob/master/nova/scheduler/filters/compute_capabilities_filter.py#L39 and https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/driver.py#L461 | 11:39 |
| *** ferest_ has quit IRC | 11:39 | |
| sdague | so any future direction needs to assume components will be less likely to all upgrade at once, not more | 11:39 |
| jaypipes | good morning, all :) | 11:39 |
| *** bzhao has quit IRC | 11:39 | |
| gilliard | Hi jaypipes | 11:39 |
| garyk | danpb: so the scheduling filter is expecting us to return the 'sockets' which we do not | 11:40 |
| ankit_ag | frickler: Thanks for your reply ! I have added recheck for 2 times and each time a different test fails, but the failure reason on n-cpu screen logs is same . i.e. Permission denied error | 11:40 |
| danpb | garyk: so just don't set anything that triggers that filter | 11:40 |
| sdague | johnthetubaguy: when get back around for some interactive time, let me know. I need to talk through this API writeup to get past my current writers block on what's next. | 11:41 |
| ankit_ag | frickler: And that too with the code I have added os.utime(), So I think even jenkins passes on my patch, there may be other gate failures with change | 11:41 |
| garyk | the extra flavor spec '"capabilities:cpu_info:topology:sockets": "2" triggers that filter :( | 11:41 |
| danpb | so do not set that extra spec then | 11:42 |
| *** markvoelker has joined #openstack-nova | 11:42 | |
| garyk | danpb: that is why I passed vmware:cores_per_scoket - which cotradicts what you suggested inthe review | 11:42 |
| danpb | that change i suggested does not need / use that extra spec | 11:42 |
| garyk | let me look again. maybe i have missed something | 11:43 |
| garyk | danpb: so it should be "hw:cpu_sockets" ? | 11:44 |
| frickler | ankit_ag: ah, I didn't look into your patch deeply and assumed the error was completely unrelated | 11:45 |
| garyk | danpb: that works. thanks! | 11:45 |
| danpb | garyk: that sets the preferred number of sockets | 11:45 |
| frickler | ankit_ag: but it this case it is not unlikely that your code either has a new race condition or just triggers one that has existed already before | 11:46 |
| danpb | if you want to enforce it more strongly you can set cpu_max_sockets instead or as well | 11:46 |
| danpb | there's more docs here http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/virt-driver-vcpu-topology.html | 11:46 |
| garyk | danpb: yes, that is what i am looking for. cool. thanks | 11:46 |
| *** markvoelker has quit IRC | 11:46 | |
| *** baoli has joined #openstack-nova | 11:47 | |
| ankit_ag | frickler: Yes IMO it might be related to jenkins environment because I have tried to reproduce it for all the possible race conditions on my local environment | 11:47 |
| ankit_ag | frickler: looks like this is relaated to the user's permissions on jenkins | 11:48 |
| *** baoli has quit IRC | 11:49 | |
| *** sahid has joined #openstack-nova | 11:50 | |
| *** baoli has joined #openstack-nova | 11:50 | |
| *** dprince has joined #openstack-nova | 11:54 | |
| samueldmq | sdague, yes, I agree, thanks | 11:56 |
| *** tobe has quit IRC | 11:56 | |
| *** Raildo has joined #openstack-nova | 11:56 | |
| *** irenab has quit IRC | 11:56 | |
| *** irenab has joined #openstack-nova | 11:57 | |
| *** tobe has joined #openstack-nova | 11:57 | |
| *** mohitsharma has joined #openstack-nova | 11:57 | |
| sdague | samueldmq: awesome. thanks for engaging. And lets get as much of the conversation driven on the mailing list so we can include as many folks in different timezones as possible. Especially as I think this has pretty far reaching implications for all openstack projects. | 11:58 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 11:58 |
| *** sudipto has quit IRC | 11:58 | |
| *** lucasagomes has joined #openstack-nova | 11:59 | |
| *** otter768 has joined #openstack-nova | 11:59 | |
| samueldmq | sdague, ++ makes sense | 11:59 |
| samueldmq | sdague, I am glad you want to be involved and make it as good as we can | 11:59 |
| *** aix has joined #openstack-nova | 12:00 | |
| markus_z | kashyap: bauzas: We talked last week about the bug triaging process. I tried to clarify my newbie questions with the openstack wiki and packed my understanding in review 187571. Maybe you could have a look some day, no hurry. | 12:01 |
| *** markvoelker has joined #openstack-nova | 12:01 | |
| kashyap | Yep, will read through. | 12:02 |
| sdague | samueldmq: yeh, happy to help. It's a really cool future vision of openstack to have these dynamic policies, I really want that to succeed. Just want to make sure it works with all the other things we also need. | 12:02 |
| markus_z | kashyap: Thank you! | 12:02 |
| *** tobe has quit IRC | 12:02 | |
| samueldmq | sdague, ++ | 12:02 |
| kashyap | markus_z: Thanks for writing that up. | 12:03 |
| bauzas | markus_z: ack | 12:04 |
| *** otter768 has quit IRC | 12:04 | |
| bauzas | add me as CC please | 12:04 |
| *** lucasagomes is now known as lucas-hungry | 12:05 | |
| garyk | danpb: all i can say is 'NICE'. Thanks! | 12:05 |
| markus_z | sdague: Some months ago I "discovered" the "OpenStack Bootstrapping Hour" on Youtube. I liked the format and it helped me a lot. Unfortunately there are no new sessions. May I ask what the current state of play is? | 12:07 |
| markus_z | bauzas: Thanks! done | 12:07 |
| kashyap | markus_z: I think it's waiting on volunteers :-) | 12:08 |
| kashyap | I.e. someone willing to commit the time to do it in their area of expertise. | 12:08 |
| sdague | markus_z: the current state of play is .... I have a 8 month old daughter now, and had to prune some of my organizing activities | 12:08 |
| *** VW has joined #openstack-nova | 12:08 | |
| *** lennyb has joined #openstack-nova | 12:09 | |
| bauzas | sdague: count me in | 12:09 |
| sdague | given that I was organizing and hosting them all. I'd be happy to host again, but kind of need a producer to help line up content and help with logistics | 12:09 |
| markus_z | sdague: Oh, OK, that shifts some priorities :) | 12:09 |
| *** rfolco has joined #openstack-nova | 12:09 | |
| bauzas | sdague: I can discuss that with mriedem | 12:10 |
| *** derekh has quit IRC | 12:10 | |
| kashyap | markus_z: You might want to rebase that change, there's a merge conflict. | 12:14 |
| markus_z | sdague: I was just curious and wanted to let you know that there is an audience for that type of format. | 12:14 |
| sdague | markus_z: cool, thanks | 12:14 |
| *** lennyb_ has quit IRC | 12:14 | |
| markus_z | kashyap: Yepp, I'm rebasing. jogo is making changes faster than me :) | 12:15 |
| openstackgerrit | garyk proposed openstack/nova: VMware: add support for cores per socket https://review.openstack.org/187942 | 12:15 |
| *** IanGovett has joined #openstack-nova | 12:17 | |
| *** eharney has joined #openstack-nova | 12:18 | |
| *** amotoki has joined #openstack-nova | 12:18 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: docs: Add bug handling description to developer reference https://review.openstack.org/187571 | 12:18 |
| markus_z | jogo: I've seen you do a lot of work in the devref right now. Could you maybe have a look at review 187571 and tell me where this would make sense in your opinion? | 12:20 |
| markus_z | kashyap: From a previous patch set (same content): http://docs-draft.openstack.org/71/187571/4/check/gate-nova-docs/daa7215//doc/build/html/bugs.html | 12:21 |
| *** rotbeard has quit IRC | 12:22 | |
| *** ivasev has joined #openstack-nova | 12:22 | |
| kashyap | Thanks. From a cursory look, the state flow looks quite helpful :-) | 12:26 |
| *** artom has joined #openstack-nova | 12:26 | |
| markus_z | Yeah, the "blockdiag" package provides some cool features, I think. :) | 12:27 |
| *** aysyanne has joined #openstack-nova | 12:28 | |
| *** artom_ has joined #openstack-nova | 12:29 | |
| *** artom has quit IRC | 12:31 | |
| *** kiranr has joined #openstack-nova | 12:34 | |
| *** kiran-r has quit IRC | 12:34 | |
| *** baoli has quit IRC | 12:34 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 12:36 |
| alex_xu | jaypipes: sdague I send the microversion spec to api-wg https://review.openstack.org/187112, but there is a little depend on what decision make on python-novaclient. Do we follow the ironic client implementation? If yes, we need add two more headers for api. | 12:36 |
| jaypipes | alex_xu: which two more headers? | 12:37 |
| mdbooth | markus_z: Ah, I think we were talking about different bugs. The one you were referring to has dropped off my care-enough-to-push list :) I'm not going to devote further time to it, so rather than waste anybody else's time I might as well abandon it. | 12:38 |
| alex_xu | jaypipes: if the requested version won't supported by server, ironic return the min and max vesion by http headers. that is a little different with nova | 12:38 |
| *** kiranr has quit IRC | 12:39 | |
| jaypipes | alex_xu: I see... | 12:40 |
| jaypipes | alex_xu: I would not be opposed to including that behaviour in Nova. | 12:40 |
| markus_z | mdbooth: That's a pity. I feel a bit responsible because of my -1. | 12:41 |
| *** ociuhandu has joined #openstack-nova | 12:41 | |
| *** iamjarvo has joined #openstack-nova | 12:41 | |
| alex_xu | jaypipes: ok, got it. should I waiting for python-novaclient's nova-spec send out? | 12:41 |
| mdbooth | markus_z: Nah, don't worry about it. | 12:41 |
| markus_z | mdbooth: Despite of the effort you want to spent on that, do you still think that this is an issue which should be solved? | 12:42 |
| jaypipes | alex_xu: sorry, not sure what you are asking... are you asking if you should wait until the API-WG patch is merged before proceeding to add functionality to nova to return those two headers? | 12:42 |
| markus_z | I have to admit, I didn't quite understand the impact. | 12:43 |
| mdbooth | markus_z: It's just 1 of a zillion races. | 12:43 |
| mdbooth | I saw it, so I attempted to document it in code. | 12:44 |
| mdbooth | I couldn't think of a way to fix it. | 12:44 |
| sdague | alex_xu: I'm confused what other headers are needed | 12:44 |
| *** artom_ has quit IRC | 12:44 | |
| alex_xu | jaypipes: at summit we said we need nova-spec for python-novalcient microversion supported, so I think maybe we should waiting for what propose in that nova-spec. avoid if there are something different propose for version negotiate. | 12:44 |
| mdbooth | markus_z: The impact is probably pretty low, tbh. | 12:44 |
| alex_xu | sdague: the use-case 7A: https://github.com/openstack/ironic-specs/blob/master/specs/kilo/api-microversions.rst | 12:44 |
| sdague | alex_xu: yeh, I signed up for doing that, but it's going to be next week at the earliest | 12:44 |
| jaypipes | alex_xu: but this is a nova server thing, not a novaclient thing, right? | 12:45 |
| sdague | alex_xu: so... I think the return content should not be headers, it should be in the body | 12:45 |
| sdague | that seems really odd to return as headers in those cases | 12:45 |
| alex_xu | jaypipes: ^ ^ see... sdague have another thought :) | 12:46 |
| *** abhishekk has quit IRC | 12:46 | |
| markus_z | mdbooth: I see, thanks for clarifying that for me! | 12:46 |
| gilliard | yes, sdague, to make it analagous with content-negotiation, having the min/max as an entity makes sense. | 12:47 |
| gilliard | rather than in headers | 12:47 |
| sdague | yeh, agree with gilliard | 12:47 |
| *** artom has joined #openstack-nova | 12:48 | |
| gilliard | does ironic use headers in that case? | 12:48 |
| sdague | yes, apparently | 12:48 |
| alex_xu | sdague: emm...maybe we should refer to sub-error-code if the api-wg work out that. | 12:48 |
| jaypipes | gilliard: isn't content-negotiation done via HTTP request headers? | 12:48 |
| sdague | jaypipes: not errors | 12:49 |
| gilliard | yes, but if content negotiation fails, the reason is explained in the body of the response. | 12:49 |
| *** artom_ has joined #openstack-nova | 12:49 | |
| jaypipes | ah, gotcha, yes. | 12:49 |
| sdague | right, you can specify multiple accepts with weights and the server returns best thing it can, but if it can't fullfill the request the error is in the body | 12:49 |
| sdague | we specifically didn't do the range / weight thing because... its' really complicated | 12:50 |
| sdague | and requires way more conditional code on both sides | 12:50 |
| sdague | it was in an early version of the microversions spec for nova | 12:50 |
| johnthetubaguy | sdague: +1 baby steps forward, and see what clients actually need and why | 12:51 |
| sdague | so, I think the only thing to do here is recommend that this gets done in the body instead of headers, and probably ask the ironic team if there was a compelling reason they chose headers here | 12:52 |
| sdague | alex_xu: can you circle with them? | 12:52 |
| openstackgerrit | Ritesh proposed openstack/nova: Users will not be allowed to delete instance during snapshot https://review.openstack.org/185942 | 12:52 |
| *** jcoufal has quit IRC | 12:52 | |
| *** artom has quit IRC | 12:52 | |
| alex_xu | sdague: ok, no problem | 12:52 |
| sdague | alex_xu: it's probably worth defining in your recommendation a standard json payload with the mix / max that would be returned | 12:52 |
| *** ankit_ag has quit IRC | 12:53 | |
| sdague | I think that could be done without needing the whole sub error system sorted, which feels like it's probably another cycle at least | 12:53 |
| *** rwsu has joined #openstack-nova | 12:54 | |
| alex_xu | sdague: yes, agree we needn't whole sub error system now, just need a format good for add sub error system in the future | 12:54 |
| *** jistr is now known as jistr|mtg | 12:57 | |
| *** derekh has joined #openstack-nova | 12:58 | |
| *** VW has quit IRC | 12:59 | |
| *** tsekiyama has joined #openstack-nova | 13:01 | |
| *** Raildo_ has joined #openstack-nova | 13:04 | |
| *** tsekiyama has quit IRC | 13:04 | |
| *** mdrabe has joined #openstack-nova | 13:05 | |
| *** kylek3h has joined #openstack-nova | 13:06 | |
| *** Raildo has quit IRC | 13:07 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| *** unicell1 has joined #openstack-nova | 13:09 | |
| *** unicell has quit IRC | 13:10 | |
| *** Raildo__ has joined #openstack-nova | 13:12 | |
| *** nkrinner has quit IRC | 13:15 | |
| *** Raildo_ has quit IRC | 13:15 | |
| *** gholler has joined #openstack-nova | 13:17 | |
| *** dave-mcc_ has joined #openstack-nova | 13:19 | |
| garyk | danpb: did the patch with the image metadata object get approved? | 13:20 |
| *** Raildo__ has quit IRC | 13:21 | |
| *** dave-mccowan has quit IRC | 13:23 | |
| *** zzzeek has joined #openstack-nova | 13:23 | |
| dims_ | johnthetubaguy: i reworked the oslo-config-generator review a whole lot after the feedback from you and jogo - so please do take a quick look when you get a chance - https://review.openstack.org/#/c/180013/ | 13:27 |
| *** burt has joined #openstack-nova | 13:27 | |
| belmoreira | Hi, we are running Juno and now I noticed that on "vcpu placement" the "cpuset" is explicitly defined | 13:28 |
| johnthetubaguy | dims_: honestly, I still don't like maintaining those separate lists when really we just want to include all options | 13:28 |
| johnthetubaguy | dims_: do you know what the reason was to do it that way? | 13:28 |
| danpb | garyk: still waiting another core unfortunately | 13:28 |
| belmoreira | this is causing us some problems with "cpuset" overcommitment | 13:28 |
| dims_ | johnthetubaguy: yes, so we can generate multiple configuration files for different services with different set of options in each | 13:29 |
| dims_ | johnthetubaguy: once this base lands, i can try to do that | 13:29 |
| belmoreira | can't find any bug related. Do you have any ideia? Can the "cpuset" be disabled? | 13:30 |
| dhellmann | johnthetubaguy, dims_ : also because the code scanning was fragile | 13:30 |
| dims_ | ack dhellmann: ++ | 13:30 |
| johnthetubaguy | dims_: so thats a good point, but in theory can't we just want the code from the cmd point | 13:30 |
| danpb | perhaps dansmith will be able to do another review of it soon https://review.openstack.org/#/c/76234/ | 13:31 |
| johnthetubaguy | dhellmann: dims_: I just have the feeling that list will be just as fragile, to be honest, while it was nasty, I quite liked when we change the sample in tree when adding a config to prove it gets updated | 13:31 |
| johnthetubaguy | but I haven't tried to maintain the scanning code, so I am not really in a position to comment | 13:32 |
| *** jecarey has joined #openstack-nova | 13:32 | |
| dims_ | so johnthetubaguy please take our word for it? :) | 13:32 |
| dhellmann | are we talking about how the discovery works, or are we talking about gating on a generated file in git? | 13:32 |
| *** mriedem_away is now known as mriedem | 13:32 | |
| johnthetubaguy | dhellmann: the former really | 13:33 |
| dansmith | danpb: I was waiting for garyk to drop his -1 | 13:33 |
| *** jcoufal has joined #openstack-nova | 13:33 | |
| *** moshele has quit IRC | 13:33 | |
| dims_ | johnthetubaguy: we are not going to change back to the old discovery mechanism, so this is it or nova keeps doing what its doing | 13:34 |
| dhellmann | johnthetubaguy: ok. and the problem is that now we have to be explicit about what options are defined? | 13:34 |
| garyk | danpb: i am currently addressing your comments. i'll drop the -1 if it does not break the code with this landing. | 13:34 |
| garyk | danpb: dansmith: i will post a patch above this one addressing my concerns | 13:34 |
| johnthetubaguy | dhellmann: yeah, I worry about missing stuff when its added, tests might help there | 13:34 |
| dansmith | garyk: well, if it really needs addressing then the patch needs to come first, right? | 13:35 |
| dansmith | garyk: we don't want to break vmware knowingly even with a patch in the queue | 13:35 |
| dhellmann | johnthetubaguy: yeah, some of the other projects have unit tests to verify that plugins define the options they should, but I don't think that's necessarily going to resolve your concern here | 13:35 |
| garyk | dansmith: i will post the patch in a few minutes | 13:35 |
| dansmith | okay thanks | 13:35 |
| *** baoli has joined #openstack-nova | 13:35 | |
| danpb | garyk: this patch can't break anything, since it merely adds the object which nothing uses | 13:35 |
| *** moshele has joined #openstack-nova | 13:36 | |
| johnthetubaguy | dhellmann: yeah, a good test would help a lot I think | 13:36 |
| danpb | only once we convert drivers over to use the object is there a risk of stuff breaking | 13:36 |
| garyk | danpb: understood | 13:36 |
| dhellmann | johnthetubaguy: what would the test do? | 13:36 |
| garyk | danpb: my concern is that we do not want to break existing api usage… that will be challenging. | 13:36 |
| danpb | nothing about this image meta stuff is easy :-) | 13:37 |
| johnthetubaguy | dhellmann: sorry, in a meeting, let me get back to you, just something that proves it changes, and the variable is present might do the trick, not 100% sure | 13:37 |
| dansmith | danpb: we also don't want to merge something that we just have to change, which I guess is what I thought he was saying might be necessary | 13:37 |
| danpb | i've found and fixed soooooo many bugs in libvirt from doing this work | 13:37 |
| bauzas | garyk: honestly it has to be done in a separate change since it's not needed yet... | 13:37 |
| dhellmann | johnthetubaguy: ok -- doing this explicitly gives us a few features for other projects that we don't want to lose by going back to scanning code, so we need *some* sort explicit list | 13:38 |
| *** iamjarvo has quit IRC | 13:38 | |
| dhellmann | dims_: did you have a version of that patch using more than one entry point? | 13:38 |
| garyk | bauzas: i was not sure that we needed to be honest about it all. | 13:39 |
| danpb | garyk: so we just need to extend the constants in the network model module to cover the missing ones | 13:39 |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: Extract helper method to get image metadata from volume https://review.openstack.org/187461 | 13:39 |
| garyk | i have changed my score as this is blocking a number of thinsg and i am in favor of the change. i have some concrens which i think that we agree can be done in another patch which i am doing | 13:39 |
| garyk | danpb: agreed. that is what i am doing now | 13:39 |
| dims_ | dhellmann: the patch has 14 entrypoints - https://review.openstack.org/#/c/180013/12/setup.cfg,cm | 13:39 |
| danpb | ok, ping me when you have it ready for review | 13:39 |
| garyk | ok, will do. thanls | 13:40 |
| *** gokrokve has joined #openstack-nova | 13:41 | |
| dansmith | garyk: are they changes that will affect the object schema? meaning, would they be things that would technically require a version bump/ | 13:41 |
| dhellmann | dims_: oh, I should have looked, I thought this draft just had one | 13:41 |
| mdbooth | Does anybody here understand pbr? | 13:41 |
| dansmith | garyk: if not, then se should go ahead and merge this.. if so, then we should wait, IMHO | 13:41 |
| danpb | dansmith: they'd be adding some further keys to an enum field | 13:41 |
| *** sudipto has joined #openstack-nova | 13:41 | |
| dansmith | okay | 13:41 |
| danpb | so they'd invalidate the version hash, but don't think we need to bump the version number for that | 13:41 |
| *** VW has joined #openstack-nova | 13:42 | |
| garyk | dansmith: i will post in 5 min. | 13:42 |
| *** whenry has joined #openstack-nova | 13:42 | |
| dansmith | danpb: normally we would, but not in this case until it's used. However, unless it's laborious, it's still cleaner to do that than to have an unversioned change in the history we have to explain | 13:42 |
| * mdbooth is about to start some pbr rtfs | 13:42 | |
| danpb | ok | 13:43 |
| dhellmann | mdbooth: you should come chat in #openstack-oslo | 13:43 |
| dansmith | garyk: okay, thanks | 13:43 |
| *** dan_ has joined #openstack-nova | 13:43 | |
| *** dan_ is now known as Guest83679 | 13:43 | |
| kashyap | mdbooth: Maybe Alan Pevec from stable maint might know. | 13:43 |
| *** changbl_ has joined #openstack-nova | 13:43 | |
| dansmith | danpb: aside from that, I'd be +W, so lets just peek at his changes and either merge or squash and assume this will be merged today one way or the other :) | 13:44 |
| mriedem | mdbooth: if you have pbr questions i'd ask lifeless or mordred | 13:45 |
| *** IanGovett has quit IRC | 13:46 | |
| mdbooth | mriedem: dhellmann has directed me to #openstack-oslo | 13:46 |
| *** apuimedo has quit IRC | 13:47 | |
| dims_ | johnthetubaguy: do i wait for someone else's input on that oslo-config-generator patch? about ready to give up on it | 13:48 |
| *** vladikr has joined #openstack-nova | 13:50 | |
| openstackgerrit | garyk proposed openstack/nova: Network: add in additional networks types https://review.openstack.org/188008 | 13:50 |
| garyk | dansmith: danpb: ^ | 13:50 |
| mordred | what did I do? | 13:50 |
| dansmith | garyk: the point of this work is to get rid of things like VirtualE1000 in the external API, right? | 13:51 |
| *** cfriesen has joined #openstack-nova | 13:51 | |
| dansmith | garyk: so that and PCNet32 at least need to not be added | 13:52 |
| *** xyang1 has joined #openstack-nova | 13:52 | |
| *** mwagner_afk has joined #openstack-nova | 13:52 | |
| dansmith | er, wait | 13:52 |
| garyk | dansmith: we are unable to do that. there are people who have this stuff defined as image metadat data today in production. | 13:52 |
| danpb | dansmith: yes indeed, the names need to be canonicalized a bit | 13:52 |
| dansmith | maybe this is southbound? | 13:52 |
| danpb | garyk: the enumfield has the ability to canonicalize legacy names for back compat | 13:52 |
| dansmith | right | 13:52 |
| dansmith | garyk: one of the intents of this work is to stop that delta from widening and getting people to use standard names for standard things going forward | 13:53 |
| danpb | garyk: eg see SCSIModel at line 425 in this https://review.openstack.org/#/c/76234/36/nova/objects/fields.py | 13:53 |
| *** nelsnelson has joined #openstack-nova | 13:53 | |
| *** oro_ has quit IRC | 13:54 | |
| garyk | danpb: ok, understood. | 13:54 |
| *** BadCub_Hiding is now known as BadCub | 13:54 | |
| dansmith | danpb: I'm not sure what to do about the vmxnet, vmxnet3, and sriov ones though | 13:55 |
| garyk | we need to create new values for those | 13:55 |
| *** claudiub has quit IRC | 13:55 | |
| danpb | dansmith: garyk i've commented on the review showing how we can deal with it | 13:57 |
| *** gokrokve has quit IRC | 13:57 | |
| garyk | danpb: :) i was already doing what you suggested | 13:58 |
| dansmith | ugh | 13:58 |
| *** gokrokve has joined #openstack-nova | 13:58 | |
| dansmith | garyk: is vmxnet3 incompatible with vmxnet? or is it just newer such that an older driver will still work with it? | 13:58 |
| garyk | dansmith: they are different drivers. the vmxnet3 is newer | 13:59 |
| garyk | we have people using both | 13:59 |
| *** jistr|mtg is now known as jistr | 13:59 | |
| *** dulek has quit IRC | 14:00 | |
| *** coolsvap is now known as coolsvap|afk | 14:00 | |
| *** otter768 has joined #openstack-nova | 14:00 | |
| *** ildikov has quit IRC | 14:00 | |
| *** tonytan4ever has joined #openstack-nova | 14:00 | |
| danpb | dansmith: vmxnet is to vmxnet3 as e1000 is to e1000e, so it makes sense for them to be separate | 14:00 |
| dansmith | danpb: the e1000 driver will drive an e1000e though, won't it? | 14:01 |
| dansmith | http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=1001805 | 14:02 |
| dansmith | this says they're not compatible though, so yeah, I guess | 14:02 |
| danpb | well that's a guest OS question | 14:02 |
| *** dboik has joined #openstack-nova | 14:02 | |
| danpb | this is about host side hardware setup | 14:02 |
| *** Kennan2 has quit IRC | 14:02 | |
| dansmith | danpb: no, I meant I thought that e1000e just added some new transfer modes or something such that any e1000 driver should be able to drive the e1000e without noticing | 14:02 |
| *** moshele has quit IRC | 14:03 | |
| *** dprince has quit IRC | 14:03 | |
| danpb | according to that doc, we ought to have vmxnet2 as well | 14:03 |
| *** dprince has joined #openstack-nova | 14:03 | |
| *** apmelton_away is now known as apmelton | 14:03 | |
| *** Kennan has joined #openstack-nova | 14:03 | |
| *** ildikov has joined #openstack-nova | 14:03 | |
| *** moshele has joined #openstack-nova | 14:04 | |
| * danpb has a new found level of hatred for mox and the incredible difficulty in figuring out its failures | 14:04 | |
| danpb | someone needs to take a chainsaw to unit tests and rip out all trace of mox | 14:05 |
| *** otter768 has quit IRC | 14:05 | |
| mdbooth | danpb: I'll buy you a subscription to chainsaw monthly. | 14:05 |
| *** mpavone has quit IRC | 14:06 | |
| garyk | danpb: i am not aware of anyone using that | 14:06 |
| dansmith | danpb: so you're just going to squash this into your patch yes? | 14:06 |
| danpb | dansmith: i guess so | 14:07 |
| *** armax has joined #openstack-nova | 14:08 | |
| dansmith | I'm kinda conflicted on adding vmxnet2, etc, because I really wish we could be a little more opinionated and not support ancient things that nobody has any reason to actually use | 14:08 |
| *** thangp has joined #openstack-nova | 14:08 | |
| kashyap | What is recommended to invoke unit tests locally? Between: `tox -e py27 nova.tests.unit.virt` and ./run_tests.sh nova.tests.unit.virt | 14:08 |
| dansmith | at least enumerating the supported values is a major step, so I should be less annoyed, but.. | 14:08 |
| sdague | kashyap: tox -e py27 nova.tests.unit.virt | 14:09 |
| sdague | or use my fun emacs integration :) | 14:09 |
| kashyap | sdague: Thanks. Why `tox` as opposed to run_tests.sh (which uses virtualenv I guess)? | 14:09 |
| * kashyap goes to rtfm of `tox` | 14:10 | |
| sdague | virtual env isolation mostly | 14:10 |
| sdague | it's a handy common dispatch point given the amount of other setup that's done | 14:10 |
| sdague | and it's repeatability is high, as you don't need to setup your local system in advance, which means less support requests for people setting up unit tests | 14:11 |
| *** hshiina has quit IRC | 14:11 | |
| *** nickchase has joined #openstack-nova | 14:12 | |
| *** bkopilov has quit IRC | 14:12 | |
| *** sudipto has quit IRC | 14:12 | |
| danpb | sdague: is it possible to run a single test with tox ? | 14:12 |
| kashyap | Okay, /me recalls seeing a thread about "fastest way to run tests" or something. Goes to look for that. | 14:12 |
| sdague | yes, if you name it correctly | 14:12 |
| * danpb currently does "$ .venv/bin/python -m testtools.run nova.tests.unit.virt.libvirt.test_driver.LibvirtDriverTestCase.test_finish_revert_migration_preserves_disk_bus" | 14:12 | |
| sdague | it's just passing it down to testr | 14:13 |
| dansmith | danpb: usually just 'tox -epy27 name-of-test' is enough | 14:13 |
| dansmith | danpb: if the test is reasonably uniquely named | 14:13 |
| kashyap | I remember learning something like this to run individual tests: $ python -m testtools.run nova.tests.unit.virt.libvirt.test_driver | 14:13 |
| * danpb tries it | 14:13 | |
| sdague | dansmith: well, it's a regex, so it can often match more names | 14:13 |
| *** Gomeler has joined #openstack-nova | 14:13 | |
| kashyap | dansmith: Thanks for the new tip | 14:13 |
| dansmith | sdague: right, hence the uniqueness constraint | 14:13 |
| *** triggerz is now known as tobasco | 14:14 | |
| danpb | that's why people name test cases like test_this_is_the_feature_I_want_test_no_really_this_one_not_that_other_one | 14:14 |
| sdague | so, in all seriousness, if you are running emacs, I did build a tool that does inline testing - https://github.com/sdague/stacktest | 14:14 |
| tobasco | the fuck is up with this vif_type=binding_failed, upgraded from juno to kilo and shit breaks like hell | 14:14 |
| sdague | which is basically just doing what danpb is doing with calling testtools appropriately | 14:15 |
| *** FL1SK has quit IRC | 14:15 | |
| sdague | using tox on single tests is kind slow because it still does testr discovery | 14:15 |
| danpb | sdague: ah nice emacs magic ! | 14:16 |
| sdague | danpb: yep, based on the nosetests module that did similar things | 14:16 |
| kashyap | danpb: lol (About test case naming) | 14:16 |
| *** sahid has quit IRC | 14:17 | |
| bauzas | for a direct run of a single test, I'm using ./runtests.sh -d | 14:17 |
| bauzas | which prevents the testr discovery | 14:17 |
| garyk | tobasco: not sure that the explitives will help | 14:19 |
| beagles | tobasco: I'm assuming you are running neutron (afaik, that's the only place that comes from). It is a somewhat unfortunate mechanism to convey a potentially transient port binding failure in neutron. You might want to look over at neutron for reasons it might be happening. There is also a related patch up for review that attempts to fail early | 14:19 |
| beagles | tobasco, https://review.openstack.org/#/c/156112/ | 14:19 |
| beagles | tobasco, so that might give you some more context | 14:19 |
| *** mtanino has joined #openstack-nova | 14:20 | |
| openstackgerrit | garyk proposed openstack/nova: Network: add in additional networks types https://review.openstack.org/188008 | 14:20 |
| garyk | danpb: dansmith: i think i need to write tests for the conversions. | 14:21 |
| *** mdbooth has quit IRC | 14:21 | |
| tobasco | beagles: ty im just scratching my head over here trying to get it working, upgrades is a pain in the ass | 14:21 |
| * beagles nods | 14:22 | |
| dansmith | sdague: can haz +W? https://review.openstack.org/#/c/187690/ | 14:24 |
| danpb | garyk: yep, would look good with a test case in test_fields.py | 14:24 |
| garyk | danpb: sure, i am on it | 14:24 |
| *** zz_jgrimm is now known as jgrimm | 14:25 | |
| *** david-lyle has quit IRC | 14:25 | |
| *** tsekiyama has joined #openstack-nova | 14:27 | |
| *** derekh has quit IRC | 14:28 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Support --all-tenants in floatingips-list https://review.openstack.org/171389 | 14:29 |
| *** VW has quit IRC | 14:30 | |
| *** rwsu has quit IRC | 14:31 | |
| kashyap | tobasco: From about a year ago, a test-only, workaround (note: 'vif_plugging_is_fatal = False' is *not* useful if you want to setup credentials b/n Nova <-> Neutron) - https://www.redhat.com/archives/rdo-list/2014-May/msg00079.html | 14:31 |
| *** iamjarvo has joined #openstack-nova | 14:32 | |
| *** iamjarvo has quit IRC | 14:32 | |
| *** iamjarvo has joined #openstack-nova | 14:33 | |
| openstackgerrit | javeme proposed openstack/nova: VMware: Prefer ShutdownGuest to PowerOffVM_Task https://review.openstack.org/181634 | 14:33 |
| * bauzas writes too many emails today | 14:35 | |
| *** exploreshaifali has quit IRC | 14:36 | |
| openstackgerrit | garyk proposed openstack/nova: Network: add in additional networks types https://review.openstack.org/188008 | 14:36 |
| garyk | danpb: dansmith: with tests et al | 14:36 |
| *** barra204 has quit IRC | 14:36 | |
| *** shakamunyi has quit IRC | 14:36 | |
| * dansmith thinks this should be sqashed | 14:37 | |
| dansmith | or squashed even | 14:37 |
| openstackgerrit | Daniel Stelter-Gliese proposed openstack/nova: Extract device path after connecting volume when booting LXC instances from volumes https://review.openstack.org/188025 | 14:38 |
| *** raildo has joined #openstack-nova | 14:39 | |
| garyk | danpb: do you want to squash or should i? | 14:39 |
| *** VW has joined #openstack-nova | 14:39 | |
| *** artom_ is now known as artom | 14:40 | |
| *** lan_ has joined #openstack-nova | 14:40 | |
| *** Maike has quit IRC | 14:41 | |
| *** Jeffrey4l has joined #openstack-nova | 14:41 | |
| danpb | garyk: i'll take care of it | 14:42 |
| *** afazekas has quit IRC | 14:43 | |
| garyk | danpb: tx | 14:43 |
| openstackgerrit | Qin Zhao proposed openstack/nova: Ensure to store context in thread local after spawn/spawn_n https://review.openstack.org/183144 | 14:44 |
| *** VW has quit IRC | 14:44 | |
| *** lan__ has joined #openstack-nova | 14:45 | |
| *** lan_ has quit IRC | 14:48 | |
| *** nihilifer has quit IRC | 14:50 | |
| *** nikhil_k is now known as nikhil_k-brb | 14:52 | |
| *** VW has joined #openstack-nova | 14:53 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 14:54 |
| *** agireud has joined #openstack-nova | 14:58 | |
| *** dguitarbite has quit IRC | 14:59 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: objects: define the ImageMeta & ImageMetaProps objects https://review.openstack.org/76234 | 15:01 |
| *** gholler has quit IRC | 15:05 | |
| *** lan_ has joined #openstack-nova | 15:05 | |
| *** kiran-r has joined #openstack-nova | 15:05 | |
| *** Gomeler has quit IRC | 15:05 | |
| *** Gomeler has joined #openstack-nova | 15:07 | |
| *** lan__ has quit IRC | 15:08 | |
| *** mdrabe has quit IRC | 15:09 | |
| *** changbl_ has quit IRC | 15:10 | |
| *** jgrimm is now known as zz_jgrimm | 15:10 | |
| *** iamjarvo has quit IRC | 15:10 | |
| *** changbl_ has joined #openstack-nova | 15:11 | |
| *** kiran-r has quit IRC | 15:13 | |
| *** iamjarvo has joined #openstack-nova | 15:14 | |
| *** gholler has joined #openstack-nova | 15:16 | |
| *** marun has joined #openstack-nova | 15:16 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 15:17 |
| *** mestery_ has joined #openstack-nova | 15:17 | |
| *** ddaskal has joined #openstack-nova | 15:18 | |
| *** rwsu has joined #openstack-nova | 15:19 | |
| *** mestery has quit IRC | 15:20 | |
| sdague | danpb: looking | 15:20 |
| *** mdbooth has joined #openstack-nova | 15:20 | |
| sdague | oh, markmcclain got you already | 15:21 |
| dansmith | sdague: yep, thanks | 15:21 |
| *** claudiub has joined #openstack-nova | 15:22 | |
| *** artom has quit IRC | 15:23 | |
| *** alex_klimov has quit IRC | 15:23 | |
| *** alexpilotti has quit IRC | 15:24 | |
| *** jecarey has quit IRC | 15:24 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova-specs: Use os-brick library https://review.openstack.org/184360 | 15:25 |
| dansmith | jaypipes: danpb just added a couple more vmware-specific values, if you want to send this: https://review.openstack.org/#/c/76234/37 | 15:25 |
| bauzas | dansmith: jaypipes: yeah, thanks danpb | 15:26 |
| *** jecarey has joined #openstack-nova | 15:26 | |
| kashyap | Folks, is there a `dnf builddep` kind of a thing that I can run before I invoke `tox`, so that running `tox -e py27 [. . .]` won't fail because it can't find deps 'libpq-fe.h' (postgres-devel) | 15:26 |
| * kashyap just the deps installs manually. But, sometimes, I just invoke `tox` and go do something else, only to find 30 min later, it failed due to some system dep. | 15:27 | |
| *** jcoufal has quit IRC | 15:27 | |
| kashyap | s/just/just installs/ | 15:28 |
| bauzas | kashyap: there is a wiki page listing all system deps to install, sec checking | 15:28 |
| bauzas | (and this is not in devref AFAIK) | 15:28 |
| *** RawInfoSec has joined #openstack-nova | 15:28 | |
| bauzas | oh no | 15:29 |
| jaypipes | dansmith: done. | 15:29 |
| bauzas | kashyap: http://docs.openstack.org/developer/nova/devref/development.environment.html#using-devstack | 15:29 |
| dansmith | jaypipes: thanks | 15:29 |
| bauzas | jaypipes: thanks you help my rebasing hell | 15:29 |
| jaypipes | heh | 15:29 |
| kashyap | bauzas: It just contains regular DevStack info, and I already am looking at the unit tests page in devref | 15:29 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 15:29 |
| kashyap | bauzas: It doesn't enumerate any system deps to be installed. Or am I just blind? | 15:30 |
| bauzas | kashyap: it does :) | 15:30 |
| *** lan_ has quit IRC | 15:30 | |
| *** mestery_ is now known as mestery | 15:30 | |
| *** artom has joined #openstack-nova | 15:30 | |
| bauzas | jaypipes: thanks for that, but since I'm now playing with the object in the FilterScheduler, I feel the PITA so probably I should -W it | 15:31 |
| jaypipes | kk | 15:31 |
| kashyap | bauzas: You're right, I was blind. Thanks. | 15:31 |
| *** RawInfoSec has left #openstack-nova | 15:33 | |
| openstackgerrit | Andrew Laski proposed openstack/nova-specs: Proposal for cells v2 scheduling interaction https://review.openstack.org/141486 | 15:35 |
| *** kiran-r has joined #openstack-nova | 15:35 | |
| *** mohitsharma has quit IRC | 15:35 | |
| mriedem | melwitt: now that https://review.openstack.org/#/c/185141/ is merged and that pbr fix, i think we're ready for a python-novaclient release | 15:36 |
| mriedem | anything we really want in here before we release? https://review.openstack.org/#/q/status:open+project:openstack/python-novaclient+branch:master,n,z | 15:38 |
| *** david-lyle has joined #openstack-nova | 15:39 | |
| *** Sukhdev has joined #openstack-nova | 15:39 | |
| mriedem | melwitt: any second thoughts on this one that has a +2? https://review.openstack.org/#/c/175603/ | 15:40 |
| *** zz_jgrimm is now known as jgrimm | 15:40 | |
| *** romainh has left #openstack-nova | 15:40 | |
| mriedem | debate was over whether or not unit tests would be useful | 15:40 |
| *** hemnafk is now known as hemna | 15:41 | |
| *** claudiub|2 has joined #openstack-nova | 15:42 | |
| *** claudiub has quit IRC | 15:43 | |
| *** artom has quit IRC | 15:44 | |
| *** gszasz has quit IRC | 15:46 | |
| *** david-lyle has quit IRC | 15:47 | |
| *** gholler has quit IRC | 15:48 | |
| *** alexpilotti has joined #openstack-nova | 15:48 | |
| *** patrickeast has joined #openstack-nova | 15:48 | |
| *** mdbooth has quit IRC | 15:49 | |
| *** nikhil_k-brb is now known as nikhil_k | 15:50 | |
| *** mdbooth has joined #openstack-nova | 15:50 | |
| danpb | can someone remind me the equivalent of mox.IsA(objects.Instance) for using with "mock" and its assert_called_with method | 15:52 |
| *** mohitsharma has joined #openstack-nova | 15:52 | |
| *** david-lyle has joined #openstack-nova | 15:53 | |
| *** busterswt has joined #openstack-nova | 15:54 | |
| ndipanov | danpb, there is none really - you have to look into the call tuple and do assertEqual etc. | 15:55 |
| danpb | eww | 15:56 |
| ndipanov | there is a catch all value you can use but if you want to check just the type | 15:56 |
| ndipanov | you can't I think | 15:56 |
| * danpb goes to try to write an equiv | 15:58 | |
| bauzas | danpb: you mean self.assertIsInstance(obj, type) ? | 15:58 |
| bauzas | danpb: ie. here self.assertIsInstance(my_obj, objects.Instance) | 15:59 |
| dansmith | no | 15:59 |
| *** dulek has joined #openstack-nova | 15:59 | |
| dansmith | danpb: there might be some thing around "provide a callable for the argument" so that it calls it to check the value | 15:59 |
| dansmith | there are a few semi-hidden things for doing stuff like that that I never can remember | 16:00 |
| bauzas | aaaha | 16:01 |
| *** otter768 has joined #openstack-nova | 16:01 | |
| *** matrohon has quit IRC | 16:01 | |
| bauzas | you want to check the type of the arg ? | 16:01 |
| bauzas | gotcha | 16:01 |
| bauzas | danpb: there are the mock_calls that you can play with | 16:02 |
| *** gholler has joined #openstack-nova | 16:02 | |
| *** oro_ has joined #openstack-nova | 16:02 | |
| bauzas | danpb: http://www.voidspace.org.uk/python/mock/helpers.html#calls-as-tuples | 16:02 |
| bauzas | danpb: but that's hacky | 16:02 |
| bauzas | danpb: or use mock.ANY | 16:03 |
| danpb | this does the trick http://paste.fedoraproject.org/228511/34738514 | 16:03 |
| *** rushiagr_away is now known as rushiagr | 16:03 | |
| danpb | basically using the same approach as mock.ANY but considering only a specific type | 16:03 |
| claudiub|2 | hello folks. If you have some time available, could you take a look at the hyper-v nova rescue commit? It's been through quite a lot of review and it seems good to go. https://review.openstack.org/#/c/127159/ Thanks! | 16:03 |
| bauzas | danpb: yeah I sometimes have this PITA | 16:04 |
| *** FL1SK has joined #openstack-nova | 16:04 | |
| andreykurilin | sdague: Hi! Since we need a spec for implementation of microversion support on client side, should I write it? | 16:04 |
| mdbooth | Can anybody remind me what the timescale is for global-requirements to nova requirements? | 16:04 |
| bauzas | danpb: since 2 objects are different | 16:04 |
| ndipanov | danpb, nice - a bit dirty but nice | 16:04 |
| mdbooth | Do I recall it happens approximately weekly? | 16:04 |
| sdague | andreykurilin: sure, if you want to take a first pass. I have a bunch of what's needed in my head, but I'm not getting to it until next week | 16:05 |
| danpb | ndipanov: that's basically what mox.IsA is doing too | 16:05 |
| bauzas | danpb: wouldn't it be possible to create the mock object and stub the method to return it ? | 16:05 |
| *** artom has joined #openstack-nova | 16:05 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: use log file and serial console at the same time https://review.openstack.org/188058 | 16:05 |
| *** arnaud____ has joined #openstack-nova | 16:05 | |
| andreykurilin | sdague: nice, I can start working on it from tomorrow | 16:05 |
| mriedem | danpb: wonder if there is anything in mock that can do this with the spec kwarg stuff on mocks | 16:05 |
| bauzas | danpb: eg. mock.patch.objects(objects.MyObject, 'get_by_something', return_value=my_fake_obj) | 16:05 |
| mriedem | i know spec prevents calling things on the mock that aren't defined in the type | 16:06 |
| sdague | andreykurilin: sounds great, let me know when it's up, I'll try to throw in bits that emerged from the summit | 16:06 |
| bauzas | mriedem: http://www.voidspace.org.uk/python/mock/helpers.html#calls-as-tuples | 16:06 |
| *** otter768 has quit IRC | 16:06 | |
| danpb | bauzas: that's alot more cumbersome imho | 16:06 |
| bauzas | mriedem: and http://www.voidspace.org.uk/python/mock/mock.html#mock.Mock.mock_calls | 16:06 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: use log file and serial console at the same time https://review.openstack.org/188058 | 16:06 |
| mriedem | bauzas: yeah i know you can get the call args and check isinstance on those | 16:06 |
| *** jgrimm is now known as zz_jgrimm | 16:07 | |
| mriedem | but that's a bit heavy like danpb said | 16:07 |
| *** jecarey has quit IRC | 16:07 | |
| bauzas | totally agreed | 16:07 |
| danpb | this approach makes the test clearer to read | 16:07 |
| bauzas | hence my (18:05:59) bauzas: danpb: eg. mock.patch.objects(objects.MyObject, 'get_by_something', return_value=my_fake_obj) | 16:07 |
| mriedem | didn't mox have an IsA or something? | 16:07 |
| danpb | bauzas: that's rather obscure too imho | 16:07 |
| mriedem | wonder why mock doesn't have that | 16:07 |
| *** belmoreira has quit IRC | 16:07 | |
| danpb | mriedem: yes, that's what i'm attempting to replicate here | 16:07 |
| *** jlanoux has quit IRC | 16:07 | |
| bauzas | danpb: why? you stub the method you know you'll get the object, it makes the test more isolatedd | 16:08 |
| andreykurilin | sdague: It looks like spec from ironic can be used. It describes a lot of stuff and, btw, current status of patches mostly implements it. | 16:08 |
| bauzas | danpb: unless you're explicitely creating the object within the method you wanna test | 16:08 |
| danpb | bauzas: that's exactly what i'm doing | 16:09 |
| *** zz_jgrimm is now known as jgrimm | 16:09 | |
| bauzas | danpb: mmm | 16:09 |
| bauzas | danpb: nevermind, I'll review your change if I can, because apparently I'm missing some piece | 16:10 |
| *** jecarey has joined #openstack-nova | 16:10 | |
| *** moshele has quit IRC | 16:10 | |
| *** gholler_ has joined #openstack-nova | 16:11 | |
| dansmith | I think danpb's solution looks good and we should have it in one of our common test utils things | 16:11 |
| *** jistr has quit IRC | 16:11 | |
| * danpb wonders if mock has a receptive upstream | 16:12 | |
| *** gokrokve_ has joined #openstack-nova | 16:12 | |
| *** artom has quit IRC | 16:12 | |
| *** afazekas has joined #openstack-nova | 16:12 | |
| danpb | hmm, last upload to pypi 2012 ... guess that answers that question | 16:12 |
| *** dulek has quit IRC | 16:13 | |
| dansmith | well, I was going to say, I don't think it changes much | 16:13 |
| dansmith | not sure if that's by design or not | 16:13 |
| bauzas | dansmith: the AnyInstance() stuff would help a lot the contributors indeed | 16:14 |
| *** kiranr has joined #openstack-nova | 16:14 | |
| dansmith | eff that, it'd help *me* a lot :) | 16:14 |
| bauzas | dansmith: since lots of us are wondering why obj1 != obj2 if the fields are equal | 16:14 |
| *** kiran-r has quit IRC | 16:14 | |
| *** dulek has joined #openstack-nova | 16:14 | |
| *** gholler has quit IRC | 16:14 | |
| *** gokrokve has quit IRC | 16:14 | |
| dansmith | well, that's not really what he's trying to solve | 16:15 |
| ndipanov | dansmith, danpb it's part of stdlib | 16:15 |
| bauzas | is the problem that is_called_with complains that the callable is different ? | 16:15 |
| ndipanov | so your upstream is python | 16:15 |
| dansmith | oh? | 16:15 |
| danpb | ndipanov: oh, i see it merged into base | 16:15 |
| danpb | so what's on pypi is stale | 16:16 |
| bauzas | ndipanov: I've been heard that's called mock3, nope ? | 16:16 |
| dansmith | is it in stdlib in 2.7? | 16:16 |
| ndipanov | nope | 16:16 |
| ndipanov | only in python 3 | 16:16 |
| dansmith | right, okay | 16:16 |
| *** unicell1 has quit IRC | 16:16 | |
| *** vilobhmm has joined #openstack-nova | 16:16 | |
| ndipanov | bauzas, not sure I think that you're thinking of mox3 | 16:16 |
| * bauzas *facepalm* | 16:17 | |
| ndipanov | that iirc was the fork of the orriginal project that also got abandoned | 16:17 |
| ndipanov | well it took patches but did no releases | 16:17 |
| ndipanov | there was a very detailed thread about this on openstack-dev some time ago | 16:17 |
| openstackgerrit | Baodong (Robert) Li proposed openstack/nova-specs: Support live migration with macvtap SR-IOV https://review.openstack.org/136077 | 16:18 |
| *** dims_ has quit IRC | 16:18 | |
| *** dims_ has joined #openstack-nova | 16:18 | |
| *** artom has joined #openstack-nova | 16:20 | |
| *** dims__ has joined #openstack-nova | 16:20 | |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| *** barra204 has joined #openstack-nova | 16:21 | |
| *** david-lyle has quit IRC | 16:22 | |
| *** mlavalle has joined #openstack-nova | 16:22 | |
| mlavalle | johnthetubaguy: ping | 16:22 |
| johnthetubaguy | mlavalle: hello | 16:22 |
| *** sriatom has joined #openstack-nova | 16:23 | |
| mlavalle | johnthetubaguy: hi. during this session in Vancouver https://etherpad.openstack.org/p/YVR-nova-network I mentioned to you this spec https://review.openstack.org/#/c/90150/ | 16:23 |
| *** alanf-mc has joined #openstack-nova | 16:24 | |
| *** dims_ has quit IRC | 16:24 | |
| johnthetubaguy | mlavalle: ah, yes | 16:24 |
| mlavalle | johnthetubaguy: this spec addresses point 4 in the 'Priority' section of the etherpad | 16:24 |
| mlavalle | johnthetubaguy: when you have a chance, please help us to get it approved. I responded in gerrit to your concerns from previous review | 16:25 |
| johnthetubaguy | mlavalle: thats more a reference to the Nova priorities agreed here: http://specs.openstack.org/openstack/nova-specs/priorities/kilo-priorities.html | 16:25 |
| *** Mike_D_laptop has joined #openstack-nova | 16:26 | |
| mlavalle | johnthetubaguy: yeah, so the spec I am talking about fits under the nova-network / neutron migration title | 16:26 |
| johnthetubaguy | mlavalle: https://review.openstack.org/#/c/187272/ is the review for the liberty priorities | 16:27 |
| johnthetubaguy | mlavalle: yeah, thats not actually in the list of priorities for liberty at this point | 16:27 |
| johnthetubaguy | ...possibly incorrectly | 16:27 |
| bauzas | johnthetubaguy: PaulMurray: I was discussing with some HP Grenoble folks and we were okay to propose HP Grenoble as another offer for the M midcycle | 16:28 |
| johnthetubaguy | mlavalle: so I am about 4 weeks behind spec reviews at this point while I am writing up things from the summit, hopefully that will get fixed soon | 16:28 |
| *** barra204 has quit IRC | 16:28 | |
| johnthetubaguy | bauzas: for M, totally we can propose that, I am hoping someone steps up to get this agreed enough so we can talk about it at the midcycle this time | 16:29 |
| bauzas | johnthetubaguy: yeah, my point is not to say I don't want Bristol, just to say there is an alternative :) | 16:29 |
| mlavalle | johnthetubaguy: great, please keep this spec in your radar screen. Let me know if there is something I can do to help. I am willing to help implement also on the nova side. I will need guidance, but will be glad to help | 16:29 |
| johnthetubaguy | mlavalle: cool, thank you, its roughly on my radar, just swamped right now, sorry | 16:30 |
| mlavalle | johnthetubaguy: :-) | 16:30 |
| *** yassine_ has quit IRC | 16:31 | |
| *** arnaud____ has quit IRC | 16:31 | |
| *** dulek has quit IRC | 16:33 | |
| *** tjones1 has joined #openstack-nova | 16:33 | |
| *** marun has quit IRC | 16:33 | |
| *** marun has joined #openstack-nova | 16:35 | |
| *** dulek has joined #openstack-nova | 16:36 | |
| *** kiranr has quit IRC | 16:36 | |
| *** _kiran_ has joined #openstack-nova | 16:36 | |
| *** gabor-t has joined #openstack-nova | 16:36 | |
| *** nic has joined #openstack-nova | 16:37 | |
| *** marzif_ has joined #openstack-nova | 16:39 | |
| *** _kiran_ has quit IRC | 16:41 | |
| *** gokrokve has joined #openstack-nova | 16:46 | |
| *** hanzhang has quit IRC | 16:47 | |
| *** hanzhang has joined #openstack-nova | 16:48 | |
| *** thangp_ has joined #openstack-nova | 16:49 | |
| *** thangp has quit IRC | 16:49 | |
| *** gokrokve_ has quit IRC | 16:49 | |
| bauzas | alaski: while I'm writing ReqSpec and using it, I just rediscovered that HostManager expects task_state | 16:50 |
| bauzas | alaski: I remember that we discussed on that one saying it would be another way to get it, but do you remember how ? | 16:50 |
| *** gokrokve has quit IRC | 16:51 | |
| alaski | bauzas: I was saying that I don't think we need to get it. It's only ever set one way when passing requests to the scheduler | 16:52 |
| alaski | so you can just assume it's a certain thing | 16:52 |
| *** mohitsharma has quit IRC | 16:52 | |
| alaski | but it's likely that what HostManager is doing with it could be accomplished another way | 16:52 |
| *** sriatom is now known as _sri | 16:53 | |
| bauzas | alaski: mmm | 16:53 |
| *** ssurana has joined #openstack-nova | 16:53 | |
| bauzas | alaski: http://docs.openstack.org/developer/nova/devref/vmstates.html#create-instance-states I should always expect 'scheduling' but that's not something I like | 16:54 |
| bauzas | alaski: the problem is, I want to iterate over all the modules by providing an object and backport it to the dicts for the upper methods | 16:54 |
| *** garyk has quit IRC | 16:54 | |
| vilobhmm | jaypipes : hi | 16:54 |
| bauzas | alaski: I mean southbound | 16:55 |
| *** afazekas has quit IRC | 16:55 | |
| bauzas | alaski: so, I'm reconstructing the request_spec dict and the instance_props | 16:55 |
| jaypipes | vilobhmm: hi (though I don't really have time to chat right now... I can do async chat ;) | 16:55 |
| *** Longgeek has quit IRC | 16:55 | |
| bauzas | alaski: because if not, it will be an ugly fat change | 16:55 |
| vilobhmm | jaypipes : sure :) just a small request…can you please review https://review.openstack.org/#/c/138607/ | 16:55 |
| *** claudiub|2 has quit IRC | 16:55 | |
| vilobhmm | bauzas : you too : ^^ | 16:56 |
| bauzas | alaski: so I could just query the instance based on the id, and put it back | 16:56 |
| jaypipes | vilobhmm: yep, it's on my long list :) | 16:56 |
| bauzas | alaski: but due to the layer isolation, I would prefer to stick with what the object provides | 16:56 |
| bauzas | vilobhmm: oh yeah... | 16:56 |
| *** dulek has quit IRC | 16:58 | |
| *** belmoreira has joined #openstack-nova | 16:58 | |
| sdague | andreykurilin: links? | 16:58 |
| alaski | bauzas: I think I understand you, but HostState is using it for one very specific thing right? we should look at how that can be satisfied without needing task_state | 16:59 |
| bauzas | alaski: gotcha | 16:59 |
| bauzas | anyway, meeting time | 16:59 |
| bauzas | need a recreation with cells | 16:59 |
| *** markus_z has quit IRC | 17:01 | |
| *** unicell has joined #openstack-nova | 17:01 | |
| *** thangp_ has quit IRC | 17:03 | |
| *** patrickeast has quit IRC | 17:03 | |
| *** lucasagomes is now known as lucas-dinner | 17:04 | |
| *** thangp has joined #openstack-nova | 17:06 | |
| *** ildikov has quit IRC | 17:07 | |
| openstackgerrit | David Bingham proposed openstack/nova: Allow libvirt cleanup completion when serial ports already released https://review.openstack.org/179978 | 17:07 |
| *** amotoki has quit IRC | 17:09 | |
| *** afazekas has joined #openstack-nova | 17:09 | |
| *** david-lyle has joined #openstack-nova | 17:13 | |
| *** david-lyle has quit IRC | 17:13 | |
| *** josecastroleon has quit IRC | 17:14 | |
| *** marun has quit IRC | 17:14 | |
| *** david-lyle has joined #openstack-nova | 17:15 | |
| *** rushiagr is now known as rushiagr_away | 17:15 | |
| *** marun has joined #openstack-nova | 17:15 | |
| *** nic1 has joined #openstack-nova | 17:16 | |
| *** kaisers has quit IRC | 17:16 | |
| *** pbandzi has quit IRC | 17:17 | |
| *** nic has quit IRC | 17:17 | |
| *** kaisers1 has quit IRC | 17:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/188099 | 17:20 |
| *** yamahata has quit IRC | 17:21 | |
| *** kaisers has joined #openstack-nova | 17:22 | |
| *** kaisers1 has joined #openstack-nova | 17:22 | |
| *** iamjarvo has quit IRC | 17:24 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: restart serial console workers after instance power change https://review.openstack.org/163080 | 17:24 |
| *** patrickeast has joined #openstack-nova | 17:24 | |
| *** dboik has quit IRC | 17:26 | |
| *** adrian_otto has joined #openstack-nova | 17:26 | |
| *** iamjarvo has joined #openstack-nova | 17:26 | |
| *** lpetrut has quit IRC | 17:27 | |
| *** dboik has joined #openstack-nova | 17:27 | |
| *** adrian_otto has quit IRC | 17:27 | |
| *** david-lyle has quit IRC | 17:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/188099 | 17:29 |
| mriedem | johnthetubaguy: dansmith: you guys ok with a novaclient release? assuming yes | don't are | 17:31 |
| mriedem | *care | 17:31 |
| dansmith | go for gate blocker | 17:32 |
| mriedem | ha | 17:32 |
| mriedem | we are go for gate blocker | 17:32 |
| mriedem | count it down | 17:32 |
| * johnthetubaguy nodes with approval | 17:34 | |
| johnthetubaguy | lol | 17:34 |
| johnthetubaguy | nods | 17:34 |
| *** Sukhdev has quit IRC | 17:34 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Convert test objects to new field formats https://review.openstack.org/187719 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Begin the transition to an explicit object registry https://review.openstack.org/187718 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Use fields from oslo.versionedobjects https://review.openstack.org/187720 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove assertRemotes() from objects tests https://review.openstack.org/188109 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix an objects layering violation in compute/api https://review.openstack.org/188110 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Replace metaclass registry with explicit opt-in registry from oslo https://review.openstack.org/188111 | 17:36 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make readonly field tests use exception from o.vo https://review.openstack.org/188112 | 17:36 |
| *** danpb has quit IRC | 17:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/188099 | 17:38 |
| *** xgsa has quit IRC | 17:38 | |
| *** iamjarvo has quit IRC | 17:39 | |
| *** burt has quit IRC | 17:40 | |
| *** yamahata has joined #openstack-nova | 17:40 | |
| *** garyk has joined #openstack-nova | 17:41 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: only use non_inheritable_image_properties if snapshotting https://review.openstack.org/187250 | 17:41 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: compute: remove get_image_metadata method https://review.openstack.org/187251 | 17:41 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 17:41 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: WIP: libvirt: convert to use nova.objects.ImageMeta https://review.openstack.org/188119 | 17:41 |
| *** kiran-r has joined #openstack-nova | 17:41 | |
| johnthetubaguy | ndipanov: sorry about your email on testing, I totally miss read what you put, makes more sense to me now! | 17:44 |
| ndipanov | johnthetubaguy, yeah when I read it back I totally could see how someone could read that like I argue against unit tests | 17:44 |
| ndipanov | :) | 17:44 |
| ndipanov | johnthetubaguy, maybe we should move your email to a separate thread in [Nova] | 17:45 |
| ndipanov | I am particularly interested in documenting "tribal knowledge" | 17:46 |
| johnthetubaguy | ndipanov: oh, totally, I went off on one | 17:46 |
| johnthetubaguy | ndipanov: I mean its a hard nut to crack, but I think we need to make some progress with the "big things" | 17:47 |
| *** adrian_otto has joined #openstack-nova | 17:47 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: tidy up legacy treatment for vif types https://review.openstack.org/188122 | 17:48 |
| mriedem | wait we aren't doing unit tests anymore? | 17:48 |
| mriedem | sweet | 17:48 |
| * johnthetubaguy makes gun shooting noises like the crazy rich texan in the Simpsons | 17:50 | |
| mriedem | yeeehaaawww! | 17:51 |
| *** iamjarvo has joined #openstack-nova | 17:52 | |
| *** agireud has quit IRC | 17:53 | |
| *** busterswt has quit IRC | 17:57 | |
| mriedem | it is done https://pypi.python.org/pypi/python-novaclient/2.26.0 | 17:59 |
| openstackgerrit | melanie witt proposed openstack/nova: Handle cells race condition deleting unscheduled instance https://review.openstack.org/188126 | 18:00 |
| *** kiran-r has quit IRC | 18:02 | |
| *** otter768 has joined #openstack-nova | 18:02 | |
| *** belmoreira has quit IRC | 18:02 | |
| *** VW_ has joined #openstack-nova | 18:03 | |
| *** VW has quit IRC | 18:05 | |
| *** tjones2 has joined #openstack-nova | 18:06 | |
| *** tjones2 has left #openstack-nova | 18:06 | |
| melwitt | mriedem: awesome, thanks. is there another thing we need to run to make the milestone things go to Fix Released? | 18:06 |
| mriedem | melwitt: i have to do it manually since the script doesn't work apparently | 18:06 |
| *** otter768 has quit IRC | 18:07 | |
| *** dboik has quit IRC | 18:07 | |
| *** dboik has joined #openstack-nova | 18:07 | |
| *** afazekas has quit IRC | 18:07 | |
| *** tjones1 has quit IRC | 18:07 | |
| mriedem | melwitt: johnthetubaguy: dansmith: sdague: now that novaclient 2.26.0 is out with the deprecation warnings for using the volume APIs/CLIs, i'm thinking we should bump the minimum required version global-requirements so anyone using it pulls that in and gets aware of the deprecation as soon as possible | 18:09 |
| mriedem | anyone against that? | 18:09 |
| sdague | mriedem: fine by me | 18:09 |
| johnthetubaguy | mriedem: cunning, worth a try | 18:09 |
| *** apuimedo has joined #openstack-nova | 18:09 | |
| mriedem | cool, i didn't want to have to release Mujina on anyone | 18:10 |
| * mriedem looked that up in the last 20 minutes | 18:10 | |
| *** adrian_otto has quit IRC | 18:10 | |
| *** ildikov has joined #openstack-nova | 18:11 | |
| kragniz | >The Nova team is beside themselves with glee | 18:11 |
| kragniz | that script never ceases to entertain | 18:11 |
| mriedem | the nova team is mildly apathetic about releasing novaclient 2.27.0 | 18:11 |
| mriedem | that can be the next one | 18:11 |
| lifeless | mriedem: so thats actually against the g-r policy | 18:11 |
| * mriedem goes to read the readme in the requirements repo | 18:12 | |
| lifeless | mriedem: specifically http://git.openstack.org/cgit/openstack/requirements/tree/README.rst#n181 | 18:13 |
| mriedem | yes i just read that | 18:13 |
| lifeless | mriedem: I'm not saying this is right | 18:13 |
| lifeless | just that I read it yesterday and so had it paged in :) | 18:14 |
| mriedem | damn your brain | 18:14 |
| lifeless | the gate will see your warnings immediately as we don't upper cap most things | 18:14 |
| mriedem | fwiw, we're updating packages weekly if not daily | 18:14 |
| mriedem | and have just gotten used to it | 18:14 |
| mriedem | since oslo releases weekly | 18:14 |
| mriedem | anyway, i'll leave it out and check the gate logs to see if there is any obvious usage that needs to be cleaned up | 18:15 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 18:15 |
| *** agireud has joined #openstack-nova | 18:16 | |
| *** marzif_ has quit IRC | 18:19 | |
| *** VW_ has quit IRC | 18:19 | |
| *** marzif_ has joined #openstack-nova | 18:19 | |
| *** VW has joined #openstack-nova | 18:20 | |
| *** annegentle has joined #openstack-nova | 18:23 | |
| *** yamahata has quit IRC | 18:25 | |
| *** yamahata has joined #openstack-nova | 18:25 | |
| *** lpetrut has joined #openstack-nova | 18:27 | |
| *** xgsa has joined #openstack-nova | 18:27 | |
| *** adrian_otto has joined #openstack-nova | 18:27 | |
| *** artom_ has joined #openstack-nova | 18:30 | |
| openstackgerrit | Qiu Yu proposed openstack/nova: Force iSCSI disconnect after timeout https://review.openstack.org/167815 | 18:31 |
| *** artom has quit IRC | 18:31 | |
| *** david-lyle has joined #openstack-nova | 18:33 | |
| *** artom_ has quit IRC | 18:33 | |
| *** ijw has joined #openstack-nova | 18:35 | |
| *** adrian_otto has quit IRC | 18:38 | |
| *** gokrokve has joined #openstack-nova | 18:39 | |
| *** aix has quit IRC | 18:39 | |
| *** flaper87 has quit IRC | 18:44 | |
| *** mlavalle has quit IRC | 18:46 | |
| *** jecarey has quit IRC | 18:49 | |
| *** baoli has quit IRC | 18:52 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 18:53 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: imagebackend: Split out resize_image logic from create_image https://review.openstack.org/187857 | 18:53 |
| *** pradipta has joined #openstack-nova | 18:53 | |
| *** mordred has quit IRC | 18:55 | |
| *** mordred has joined #openstack-nova | 18:55 | |
| *** adrian_otto has joined #openstack-nova | 18:56 | |
| *** marzif_ has quit IRC | 18:57 | |
| *** barra204 has joined #openstack-nova | 18:57 | |
| *** ijw has quit IRC | 19:00 | |
| dims__ | mriedem: dansmith: how are we able to compare a variable of type nova.db.sqlalchemy.models.Quota to a integer? (https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3431). where's the magic that converts user_quotas[res] into an integer so the if condition works...does not work in python 3, hence asking :) | 19:02 |
| *** barra204 has quit IRC | 19:02 | |
| *** alanf-mc has quit IRC | 19:03 | |
| *** ssurana has quit IRC | 19:06 | |
| mriedem | dims__: i thought user_quotas was updated before _calculate_overquota was called | 19:07 |
| *** alanf-mc has joined #openstack-nova | 19:07 | |
| mriedem | dims__: i think i've wondered about that myself though fwiw, it's not easy to trace how those dicts are populated throughout that path | 19:07 |
| *** agireud_ has joined #openstack-nova | 19:08 | |
| *** sjcherry has joined #openstack-nova | 19:08 | |
| *** flaper87 has joined #openstack-nova | 19:08 | |
| *** ssurana has joined #openstack-nova | 19:09 | |
| *** ssurana has quit IRC | 19:09 | |
| *** flaper87 has quit IRC | 19:09 | |
| *** flaper87 has joined #openstack-nova | 19:09 | |
| *** agireud has quit IRC | 19:09 | |
| *** agireud_ is now known as agireud | 19:09 | |
| mriedem | dims__: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3063 | 19:09 |
| dims__ | mriedem: i printed out user_quotas[res] : <nova.db.sqlalchemy.models.Quota object at 0x10d219d90> | 19:10 |
| mriedem | user_quotas[res] should be mapping to the hard_limit int here https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L443 | 19:10 |
| dims__ | mriedem: both in py3 and py2 | 19:10 |
| mriedem | it's mutated somewhere then | 19:11 |
| dansmith | dims__: I'm touched that you think that's a thing I know about :) | 19:11 |
| dims__ | mriedem: right don't know ehre | 19:11 |
| dims__ | :( | 19:11 |
| mriedem | dims__: basically you have to find out what in http://git.openstack.org/cgit/openstack/nova/tree/nova/quota.py is eventually passing in user_quotas to db.quota_reserve | 19:11 |
| *** adrian_otto has quit IRC | 19:11 | |
| dims__ | dansmith: :) | 19:11 |
| mriedem | since http://git.openstack.org/cgit/openstack/nova/tree/nova/quota.py gets a bit goofy with changing things | 19:11 |
| *** ssurana has joined #openstack-nova | 19:11 | |
| mriedem | dims__: it's probably this http://git.openstack.org/cgit/openstack/nova/tree/nova/quota.py#n248 | 19:12 |
| mriedem | it fills in user_quotas with Quota objects from project_quotas | 19:12 |
| mriedem | but those should also be ints https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3078 | 19:13 |
| dims__ | mriedem: right, that's not the problem the python statement i linked to above someone works in py2 but not py3 | 19:13 |
| mriedem | idk, does something in the sqlalchemy model coerce the value? | 19:13 |
| dims__ | mriedem: if i print "user_quotas[res] >= 0" it fails with TypeError: unorderable types: Quota() >= int() | 19:13 |
| dims__ | right mriedem | 19:14 |
| mriedem | dims__: can you get a traceback when it fails? | 19:14 |
| dims__ | yep | 19:14 |
| dims__ | right on that line :) | 19:14 |
| mriedem | but the full trace going back to nova.quota | 19:14 |
| dims__ | http://paste.openstack.org/show/260221/ | 19:15 |
| dims__ | zzzeek: around? | 19:16 |
| zzzeek | dims__: hey | 19:16 |
| mriedem | dims__: are you sure that test case isn't just borked? | 19:16 |
| dims__ | zzzeek: in https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L3431 - user_quotas[res] is of type nova.db.sqlalchemy.models.Quota | 19:17 |
| dims__ | zzzeek: and somehow in python2 the >= against integer works fine | 19:17 |
| dims__ | zzzeek: and in python3 it doesn't | 19:17 |
| dims__ | zzzeek: fails with http://paste.openstack.org/show/260221/ | 19:17 |
| zzzeek | dims__: in py3k it’s not valid to compare disparate types | 19:17 |
| mriedem | dims__: the tests are wrong | 19:17 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L112 | 19:17 |
| mriedem | user_quotas[resource] = db.quota_create(context, project_id, | 19:18 |
| mriedem | the test is setting a Quota model dict like thing in the user_quotas dict that's passed to db.api.quota_reserve | 19:18 |
| zzzeek | dims__: in that code you’d want to have an integer on the other side, if that’s an attribute of the Quota object perhaps, or the user_quotas dict is mis-popualted as it seems mriedem is saying | 19:18 |
| mriedem | which is wrong since user_quotas should be mapping of resources to limit integers | 19:18 |
| zzzeek | dims__: see? this is py3k finding a bug :) | 19:18 |
| mriedem | yay! | 19:19 |
| mriedem | so...when can we just move to java? | 19:19 |
| mriedem | with typing? | 19:19 |
| mriedem | an serializable objects | 19:19 |
| dims__ | haha | 19:19 |
| *** apuimedo has quit IRC | 19:19 | |
| dims__ | mriedem: i have a whole bunch of these | 19:19 |
| mriedem | so i guess python 3 is just becoming more java like huh | 19:19 |
| mriedem | dims__: is this a recent regression? i thought the nova py34 job was passing last week? | 19:20 |
| dims__ | hmmm. now i have to fix the test! | 19:20 |
| mriedem | shouldn't be hard | 19:20 |
| *** pradipta has quit IRC | 19:20 | |
| dims__ | mriedem: py34 had just 2 tests :) i have 2 reviews now that brings it to like 1500+ and now working on another one | 19:21 |
| mriedem | user_quotas[resource] = db.quota_create(context, project_id,...).hard_limit | 19:21 |
| mriedem | oh i didn't realize it was filtering stuff out | 19:21 |
| dims__ | mriedem: look at the tox.ini | 19:21 |
| dims__ | the idea was we got all the dependencies lined up so we could enable the test job last weeek | 19:22 |
| mriedem | ha | 19:22 |
| mriedem | https://github.com/openstack/nova/blob/master/tox.ini#L44 | 19:22 |
| mriedem | well as long as nova.tests.unit.test_versions works i think we can ship it | 19:22 |
| dims__ | mriedem: what the hell was the tests testing? :) | 19:23 |
| mriedem | the db api one? idk | 19:23 |
| mriedem | obviously not enough | 19:23 |
| mriedem | when i refactored quota_reserve in kilo i added a slew of unit tests for that path but they were using mock | 19:23 |
| dims__ | zzzeek: thanks! | 19:24 |
| zzzeek | dims__: youre welcome | 19:24 |
| dims__ | mriedem: dansmith: so i take a stab at adding the .hard_limit in a separate patch? | 19:24 |
| mriedem | dims__: is there a patch you're looking at? | 19:25 |
| mriedem | otherwise yes this is a bug in the test code | 19:25 |
| mriedem | you could even open a bug for tracking it | 19:25 |
| dims__ | mriedem: only on my hard disk | 19:25 |
| dims__ | mriedem: let me figure out a bit and throw in a bug and review | 19:26 |
| *** jgrimm is now known as zz_jgrimm | 19:26 | |
| dims__ | who do we know from bluebox? | 19:36 |
| *** VW has quit IRC | 19:38 | |
| *** iamjarvo has quit IRC | 19:39 | |
| *** adrian_otto has joined #openstack-nova | 19:39 | |
| dansmith | dims__: jlk | 19:39 |
| dims__ | mriedem: here's the bug and a way to repro it in python2 - https://bugs.launchpad.net/nova/+bug/1461665 | 19:40 |
| openstack | Launchpad bug 1461665 in OpenStack Compute (nova) "nova.tests.unit.db.test_db_api sets up quotas incorrectly" [Undecided,New] | 19:40 |
| *** igordcard_ has quit IRC | 19:40 | |
| *** claudiub has joined #openstack-nova | 19:40 | |
| mriedem | dims__: want me to push the change? | 19:41 |
| dims__ | mriedem: sure thanks | 19:41 |
| dims__ | dansmith: ah. thanks | 19:42 |
| mriedem | i hail my new blue box overlords | 19:43 |
| *** artom has joined #openstack-nova | 19:45 | |
| *** artom has quit IRC | 19:47 | |
| *** artom has joined #openstack-nova | 19:47 | |
| *** ociuhandu has quit IRC | 19:48 | |
| *** Mike_D_laptop has quit IRC | 19:52 | |
| *** _sri has quit IRC | 19:53 | |
| *** armax has quit IRC | 19:53 | |
| *** alanf-mc has quit IRC | 19:55 | |
| *** burt has joined #openstack-nova | 19:55 | |
| bauzas | alaski: around ? | 19:56 |
| alaski | bauzas: yes | 19:57 |
| *** gabor-t has quit IRC | 19:57 | |
| *** adrian_otto has quit IRC | 19:57 | |
| bauzas | alaski: I'm looking at https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L296-L301 | 19:57 |
| *** flaper87 has quit IRC | 19:57 | |
| *** VW has joined #openstack-nova | 19:58 | |
| bauzas | alaski: honestly, I don't figure out how to check for that without the task_state :( do you have another idea ? | 19:58 |
| *** barra204 has joined #openstack-nova | 19:58 | |
| bauzas | alaski: I can suppose that vm_state is always BUILDING but I need to figure out if all calls to the scheduler are done this way | 19:58 |
| *** flaper87 has joined #openstack-nova | 19:59 | |
| *** VW_ has joined #openstack-nova | 19:59 | |
| *** thangp has quit IRC | 20:00 | |
| *** iamjarvo has joined #openstack-nova | 20:00 | |
| alaski | bauzas: right. my thinking was that since you have to synthesize an instance from the request spec when this is called in the scheduler you can just set it to building. But really I would probably change the api to have an inc_ip_ops=False parameter that can be set, and let the caller decide rather than having HostManager care about vm/task_states | 20:01 |
| *** xyang1 has quit IRC | 20:01 | |
| *** patrickeast_ has joined #openstack-nova | 20:01 | |
| *** dave-mcc_ has quit IRC | 20:02 | |
| bauzas | alaski: I see | 20:02 |
| bauzas | alaski: there are a few number of callers for select_dests() | 20:02 |
| *** patrickeast has quit IRC | 20:02 | |
| *** patrickeast_ is now known as patrickeast | 20:02 | |
| bauzas | alaski: but we would then have to check the instance.task_state each time before calling select_dest() | 20:03 |
| *** otter768 has joined #openstack-nova | 20:03 | |
| bauzas | alaski: which is a bit hard IMHO | 20:03 |
| *** VW has quit IRC | 20:03 | |
| bauzas | alaski: just wonder if it would be acceptable to port this in the scheduler client | 20:03 |
| *** alanf-mc has joined #openstack-nova | 20:03 | |
| *** barra204 has quit IRC | 20:03 | |
| *** flaper87 has quit IRC | 20:03 | |
| alaski | bauzas: we can assume that if the scheduler is calling this that the instance is in motion, and therefore would be an iop, right? | 20:03 |
| *** marun_ has joined #openstack-nova | 20:03 | |
| *** marun has quit IRC | 20:03 | |
| bauzas | alaski: ie. the sched client would check vm_state and task_state and provide the parameter right | 20:03 |
| bauzas | alaski: probably, but I can't be sure about that | 20:04 |
| bauzas | alaski: I would have just said that the conditional would be True each time, but I just wonder if that's actually right | 20:04 |
| alaski | I don't know, but I would expect that when it's called from the scheduler it is | 20:05 |
| bauzas | alaski: I mean, there are so many corner cases when scheduling and you know that we don't have a clear state | 20:05 |
| *** dave-mccowan has joined #openstack-nova | 20:05 | |
| *** mtanino has quit IRC | 20:06 | |
| bauzas | alaski: mmm | 20:06 |
| *** jecarey has joined #openstack-nova | 20:06 | |
| alaski | another option would be to add vm_state/task_state to requestspec, but not persist them. though that's not my preference | 20:07 |
| bauzas | alaski: hardly checking http://docs.openstack.org/developer/nova/devref/vmstates.html | 20:07 |
| bauzas | alaski: yeah, I will have to do this for the context too | 20:07 |
| *** otter768 has quit IRC | 20:07 | |
| bauzas | alaski: and an ugly thing | 20:07 |
| bauzas | alaski: that => https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L126 | 20:08 |
| *** salv-orlando has quit IRC | 20:09 | |
| *** salv-orlando has joined #openstack-nova | 20:09 | |
| *** baoli has joined #openstack-nova | 20:10 | |
| alaski | ugh | 20:10 |
| alaski | that looks like it just needs to move out into a new parameter to host_manager methods | 20:11 |
| alaski | or just set host_manager.config_options | 20:11 |
| alaski | really shouldn't be in filter_properties | 20:12 |
| *** baoli has quit IRC | 20:13 | |
| *** alanf-mc has quit IRC | 20:13 | |
| bauzas | alaski: I'll just provide it as a non-field | 20:13 |
| bauzas | alaski: discussed that with dansmith | 20:13 |
| *** baoli has joined #openstack-nova | 20:13 | |
| bauzas | alaski: like a getter/setter | 20:14 |
| *** alanf-mc has joined #openstack-nova | 20:14 | |
| alaski | okay. since that doesn't appear to ever go over the wire, or need to at least | 20:14 |
| bauzas | alaski: yeah | 20:14 |
| bauzas | alaski: here io_ops can't be done that way, since all the non-fields are not provided over the wire | 20:15 |
| *** adrian_otto has joined #openstack-nova | 20:15 | |
| *** lucas-dinner has quit IRC | 20:15 | |
| bauzas | alaski: I mean the task_state | 20:15 |
| *** alanf-mc has quit IRC | 20:16 | |
| bauzas | alaski: so we honestly have 2 possibilities : | 20:16 |
| bauzas | 1/ add a parameter to select_dest() considering the need for io_ops | 20:16 |
| *** flaper87 has joined #openstack-nova | 20:16 | |
| openstackgerrit | melanie witt proposed openstack/nova: Handle cells race condition deleting unscheduled instance https://review.openstack.org/188126 | 20:16 |
| bauzas | 2/ add some bad thing in the spec object | 20:17 |
| bauzas | mmm | 20:17 |
| bauzas | 3/ figure out which cases are not io_ops | 20:17 |
| *** adrian_otto has quit IRC | 20:17 | |
| alaski | I think we can safely assume that if the scheduler calls consume_from_instance that iops should be incremented | 20:17 |
| alaski | but that may be proved wrong later | 20:17 |
| *** alexpilotti has quit IRC | 20:17 | |
| bauzas | alaski: exactly :( | 20:18 |
| *** gokrokve has quit IRC | 20:18 | |
| bauzas | alaski: I don't want to bet on it | 20:18 |
| dims__ | dansmith: https://review.openstack.org/#/c/187720/ reminds me the hash calculation may end up being python version specific, i have not been able to get the same hashes with python3 | 20:18 |
| alaski | bauzas: I think it can be audited easily enough, there are few calls to the scheduler | 20:18 |
| alaski | bauzas: should only be boot and resize/migrate, and both of those would increment io_ops | 20:19 |
| openstackgerrit | melanie witt proposed openstack/nova: Handle cells race condition deleting unscheduled instance https://review.openstack.org/188126 | 20:19 |
| bauzas | alaski: mmm, I should probably add a DNM change to add something so Tempest could just check that | 20:19 |
| bauzas | alaski: a log saying 'eh, here is the state of the instance before calling out the scheduler' | 20:20 |
| alaski | bauzas: sure. I just can't think of a case where the scheduler would be called and not result in an io_ops. it is a precursor to moving an instance | 20:20 |
| bauzas | alaski: agreed | 20:21 |
| bauzas | alaski: if we consume from an instance, that should be an IO op | 20:21 |
| bauzas | alaski: the opposite I'm thinking is a check_dest() API method, which would return True or False by checking all filters | 20:21 |
| bauzas | in that case, the hostmanager wouldn't be consuming usage | 20:22 |
| *** tjones1 has joined #openstack-nova | 20:22 | |
| alaski | right | 20:22 |
| cfriesen | who knows stuff about the nova interface to glance? I think we've found a problem that can lead to glance leaving a file open. | 20:23 |
| bauzas | alaski: okay, will write a DNM and see but I think it should be true that way - or leave a big comment saying to throw me tomatoes if I'm wrong | 20:23 |
| *** ddaskal has quit IRC | 20:25 | |
| *** busterswt has joined #openstack-nova | 20:25 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Make the guest console configurable https://review.openstack.org/122523 | 20:27 |
| *** ociuhandu has joined #openstack-nova | 20:29 | |
| *** ivasev has quit IRC | 20:29 | |
| *** Sukhdev has joined #openstack-nova | 20:29 | |
| *** matrohon has joined #openstack-nova | 20:29 | |
| mriedem | cfriesen: probably jaypipes | 20:29 |
| cfriesen | mriedem: thx | 20:30 |
| dansmith | dims__: okay, that calculation is done in o.vo now and it passes both versions of python, FWIW | 20:30 |
| dansmith | dims__: I mean, it's done in o.vo and nova will soon use that implementation | 20:30 |
| mriedem | dansmith: have you ever thought about logging how long we waited for an external event to come back from neutron? | 20:32 |
| *** adrian_otto has joined #openstack-nova | 20:33 | |
| *** alanf-mc has joined #openstack-nova | 20:33 | |
| *** Nastooh_ has joined #openstack-nova | 20:33 | |
| dansmith | mriedem: I think we did in a test version before landing it whilst making sure it was working | 20:33 |
| mriedem | we have some some logging around preparing to wait for events and getting them back, but you have to diff the timestamps in the logs | 20:33 |
| dansmith | mriedem: but it seems like a reasonable thing to log all the time, yeah | 20:33 |
| mriedem | i'll take a look at adding something | 20:33 |
| dims__ | dansmith: i see, will check my work again then | 20:33 |
| dansmith | depends on whether you want to just time the wait (easy) or the prepare...wait end | 20:33 |
| Nastooh_ | No dice in openstack-containers, maybe here: Can't ssh, externally or via netns, to a nova-docker instance. I can ping it, netns and externally; however. Any suggestions? | 20:35 |
| mriedem | dansmith: not sure yet, i'm looking into the code to see why i have a 10 second black box here after getting the vif plug event back from neutron and then timeout http://paste.openstack.org/show/260394/ | 20:35 |
| dansmith | mriedem: I don't see any timeout in that paste | 20:36 |
| mriedem | dansmith: it's right after the error | 20:37 |
| mriedem | it's an rpc_response_timeout | 20:37 |
| mriedem | i didn't put it in the paste but that's what it was | 20:37 |
| mriedem | copy_qcow2_image takes 28 seconds, vif plug event takes about 6 seconds, processing that takes at least 10 seconds and we timeout | 20:38 |
| mriedem | with random stuff in between taking time in a 60 second rpc timeout | 20:38 |
| dansmith | ah, okay | 20:38 |
| *** vladikr has quit IRC | 20:41 | |
| *** adrian_otto has quit IRC | 20:43 | |
| *** radek_ has quit IRC | 20:44 | |
| *** matrohon has quit IRC | 20:46 | |
| *** baoli has quit IRC | 20:46 | |
| claudiub | hello folks. If you have a bit of time, could you take a look at the hyper-v nova rescue patch? https://review.openstack.org/#/c/127159/ It has been reviewed quite a lot for some time, Jenkins is happy, CI is happy, we are happy, but we would be happier to get some extra reviews on it. :D | 20:49 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: DO NOT MERGE: Ajouter un log moche pour le scheduler https://review.openstack.org/188178 | 20:54 |
| *** apuimedo has joined #openstack-nova | 20:54 | |
| openstackgerrit | Daniel Stelter-Gliese proposed openstack/nova: Extract device path after connecting volume when booting LXC instances from volumes https://review.openstack.org/188025 | 20:56 |
| mriedem | dansmith: i guess network vif plug wasn't my issue, it was this turd http://paste.openstack.org/show/260548/ | 20:57 |
| mriedem | which we also see upstream | 20:57 |
| *** baoli has joined #openstack-nova | 20:57 | |
| *** adrian_otto has joined #openstack-nova | 20:58 | |
| *** marun_ has quit IRC | 20:58 | |
| dansmith | mriedem: oh lovely | 21:00 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/188099 | 21:00 |
| *** adrian_otto has quit IRC | 21:01 | |
| *** angdraug has joined #openstack-nova | 21:01 | |
| openstackgerrit | sean mooney proposed openstack/nova: fix AggregateTypeAffinityFilter multi values support https://review.openstack.org/141883 | 21:02 |
| *** burt has quit IRC | 21:02 | |
| flwang | mriedem: dansmith: could you pls revisit https://review.openstack.org/#/c/164494/ ? the nova backup fix, thanks | 21:03 |
| *** sabari has quit IRC | 21:03 | |
| *** swamireddy has quit IRC | 21:03 | |
| *** nickchase has quit IRC | 21:04 | |
| *** tsekiyama has quit IRC | 21:05 | |
| *** annegentle has quit IRC | 21:06 | |
| *** samueldmq has quit IRC | 21:06 | |
| *** adrian_otto has joined #openstack-nova | 21:08 | |
| *** artom has quit IRC | 21:10 | |
| openstackgerrit | Chris Friesen proposed openstack/nova: don't leave glance with open file descriptor on error https://review.openstack.org/188179 | 21:13 |
| cfriesen | jaypipes: (and anyone else interested) ^ | 21:13 |
| *** Gomeler has quit IRC | 21:13 | |
| cfriesen | if anyone has a more elegant way to handle this, please let me know | 21:13 |
| *** iamjarvo has quit IRC | 21:14 | |
| *** dboik_ has joined #openstack-nova | 21:17 | |
| *** sabari has joined #openstack-nova | 21:18 | |
| *** sjcherry has quit IRC | 21:20 | |
| *** dboik has quit IRC | 21:20 | |
| *** dboik_ has quit IRC | 21:21 | |
| *** alex_klimov has joined #openstack-nova | 21:21 | |
| mriedem | cfriesen: could you write a simple unit test such that data is a mock and on like the 1st call to write() you get the IOError but then assert that it keeps going (len(image_chunks) == 2 or something in the test) | 21:22 |
| mriedem | and finally assert that the IOError is raised | 21:23 |
| *** swamireddy has joined #openstack-nova | 21:24 | |
| *** iamjarvo has joined #openstack-nova | 21:25 | |
| *** baoli has quit IRC | 21:28 | |
| *** baoli has joined #openstack-nova | 21:29 | |
| *** lpetrut has quit IRC | 21:34 | |
| *** dontalton has joined #openstack-nova | 21:39 | |
| *** busterswt has quit IRC | 21:40 | |
| openstackgerrit | Baodong (Robert) Li proposed openstack/nova: Handle port delete event https://review.openstack.org/179390 | 21:42 |
| *** alexpilotti has joined #openstack-nova | 21:45 | |
| *** iamjarvo has quit IRC | 21:49 | |
| *** iamjarvo has joined #openstack-nova | 21:49 | |
| *** mwagner_afk has quit IRC | 21:49 | |
| *** iamjarvo has quit IRC | 21:50 | |
| *** ianbrown has joined #openstack-nova | 21:52 | |
| *** dprince has quit IRC | 21:53 | |
| *** mwagner_afk has joined #openstack-nova | 21:53 | |
| *** lpetrut has joined #openstack-nova | 21:55 | |
| *** lpetrut has quit IRC | 21:56 | |
| *** mriedem has quit IRC | 21:57 | |
| *** ianbrown has quit IRC | 22:00 | |
| *** agireud has quit IRC | 22:01 | |
| *** armax has joined #openstack-nova | 22:03 | |
| *** otter768 has joined #openstack-nova | 22:04 | |
| *** tsekiyama has joined #openstack-nova | 22:04 | |
| *** ianbrown has joined #openstack-nova | 22:05 | |
| *** otter768 has quit IRC | 22:09 | |
| *** tsekiyama has quit IRC | 22:09 | |
| *** gholler__ has joined #openstack-nova | 22:14 | |
| *** mwagner_afk has quit IRC | 22:16 | |
| *** dave-mccowan has quit IRC | 22:18 | |
| *** gholler_ has quit IRC | 22:18 | |
| *** dave-mccowan has joined #openstack-nova | 22:20 | |
| *** jamielennox|away is now known as jamielennox | 22:20 | |
| *** dave-mccowan has quit IRC | 22:24 | |
| *** oro_ has quit IRC | 22:26 | |
| *** VW_ has quit IRC | 22:27 | |
| *** Sukhdev has quit IRC | 22:29 | |
| *** adrian_otto has quit IRC | 22:34 | |
| *** mriedem has joined #openstack-nova | 22:34 | |
| *** gholler__ has quit IRC | 22:35 | |
| *** mriedem has quit IRC | 22:37 | |
| *** mriedem has joined #openstack-nova | 22:38 | |
| *** kylek3h has quit IRC | 22:39 | |
| openstackgerrit | eeway hsu proposed openstack/nova: Fixes referenced path in nova/doc/README.rst https://review.openstack.org/188197 | 22:39 |
| *** adrian_otto has joined #openstack-nova | 22:40 | |
| *** vilobhmm has quit IRC | 22:41 | |
| *** vilobhmm has joined #openstack-nova | 22:41 | |
| *** VW has joined #openstack-nova | 22:41 | |
| *** tonytan4ever has quit IRC | 22:41 | |
| *** tonytan4ever has joined #openstack-nova | 22:42 | |
| *** VW_ has joined #openstack-nova | 22:43 | |
| *** VW_ has quit IRC | 22:43 | |
| *** adrian_otto has quit IRC | 22:43 | |
| *** VW_ has joined #openstack-nova | 22:43 | |
| *** vilobhmm1 has joined #openstack-nova | 22:44 | |
| *** VW has quit IRC | 22:45 | |
| *** vilobhmm has quit IRC | 22:46 | |
| *** tonytan4ever has quit IRC | 22:46 | |
| *** barra204 has joined #openstack-nova | 22:48 | |
| *** VW_ has quit IRC | 22:48 | |
| *** tpatil has joined #openstack-nova | 22:50 | |
| *** whenry has quit IRC | 22:50 | |
| *** barra204 has quit IRC | 22:53 | |
| *** apuimedo has quit IRC | 22:53 | |
| *** nic1 has quit IRC | 22:58 | |
| *** Sukhdev has joined #openstack-nova | 22:59 | |
| *** baoli has quit IRC | 23:01 | |
| *** Raildo_ has joined #openstack-nova | 23:03 | |
| *** destiny has quit IRC | 23:04 | |
| *** tjones1 has quit IRC | 23:06 | |
| *** jecarey has quit IRC | 23:06 | |
| *** tjones1 has joined #openstack-nova | 23:07 | |
| *** aysyanne has quit IRC | 23:08 | |
| *** pixelbeat_ has quit IRC | 23:11 | |
| *** vladikr has joined #openstack-nova | 23:11 | |
| flwang | jaypipes: ping | 23:14 |
| flwang | re glance v2 in nova | 23:15 |
| *** angdraug has quit IRC | 23:18 | |
| *** tsekiyama has joined #openstack-nova | 23:25 | |
| mikal | dims__: you around? | 23:25 |
| *** tjones1 has quit IRC | 23:33 | |
| *** tjones1 has joined #openstack-nova | 23:34 | |
| *** sdake has quit IRC | 23:35 | |
| *** VW has joined #openstack-nova | 23:35 | |
| *** dontalton has quit IRC | 23:37 | |
| *** Raildo_ has quit IRC | 23:39 | |
| *** tellesnobrega has quit IRC | 23:40 | |
| *** rwsu has quit IRC | 23:41 | |
| *** VW has quit IRC | 23:44 | |
| dims__ | mikal: pong | 23:48 |
| *** dims_ has joined #openstack-nova | 23:51 | |
| *** bzhao has joined #openstack-nova | 23:52 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/nova: libvirt: Make behavior of os_require_quiesce consistent https://review.openstack.org/167819 | 23:53 |
| mikal | dims__: I am trying to remember what we agreed to with that oslo.conconcurrenty patch we talked about at the summit. I think we agreed I should send the minimum viable patch against master in public and then we'd back port it? Is that your recollection? | 23:55 |
| *** dims__ has quit IRC | 23:55 | |
| dims_ | mikal: yes. that's right. | 23:56 |
| mikal | dims_: ok, cool. I've been off sick so I wasn't 100% sure I'd made that up. | 23:56 |
| mikal | dims_: I'm work on progressing that today then. | 23:56 |
| *** claudiub has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!