*** erlon has quit IRC | 00:00 | |
*** achanda has joined #openstack-nova | 00:03 | |
*** VW__ has joined #openstack-nova | 00:06 | |
*** VW_ has quit IRC | 00:09 | |
*** moshele has joined #openstack-nova | 00:09 | |
*** VW__ has quit IRC | 00:11 | |
*** zzzeek has joined #openstack-nova | 00:12 | |
sajeesh | ping vishy | 00:13 |
---|---|---|
*** tsekiyam_ has joined #openstack-nova | 00:21 | |
*** moshele has quit IRC | 00:22 | |
*** ZZelle_ has quit IRC | 00:22 | |
*** mtanino has quit IRC | 00:23 | |
sajeesh | ping dims | 00:23 |
*** mdrabe has joined #openstack-nova | 00:23 | |
*** signed8b_ is now known as signed8bit_ZZZzz | 00:23 | |
*** tsekiyama has quit IRC | 00:24 | |
*** artom has quit IRC | 00:24 | |
*** tsekiyam_ has quit IRC | 00:25 | |
*** pixelbeat_ has quit IRC | 00:26 | |
sajeesh | ping melwitt | 00:26 |
*** tpeoples has joined #openstack-nova | 00:27 | |
openstackgerrit | sajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects https://review.openstack.org/200342 | 00:28 |
*** davideagnello has quit IRC | 00:28 | |
*** yamahata has quit IRC | 00:32 | |
*** annashen has joined #openstack-nova | 00:32 | |
*** ddaskal has joined #openstack-nova | 00:32 | |
*** annashen has quit IRC | 00:37 | |
*** ssurana has quit IRC | 00:38 | |
*** Jeffrey4l has joined #openstack-nova | 00:38 | |
*** ssurana has joined #openstack-nova | 00:38 | |
*** david-lyle has quit IRC | 00:40 | |
*** ssurana has quit IRC | 00:42 | |
*** promulo__ has joined #openstack-nova | 00:45 | |
openstackgerrit | sajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects https://review.openstack.org/200342 | 00:45 |
*** promulo_ has quit IRC | 00:49 | |
*** zhenguo has joined #openstack-nova | 00:52 | |
*** browne has joined #openstack-nova | 00:55 | |
*** signed8bit_ZZZzz is now known as signed8b_ | 00:57 | |
*** shashank_hegde has quit IRC | 01:02 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Finding parent_id and immediate child list https://review.openstack.org/200941 | 01:02 |
*** shashank_hegde has joined #openstack-nova | 01:02 | |
*** shashank_hegde has quit IRC | 01:04 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: Add total_vms field to ComputeNode https://review.openstack.org/202472 | 01:09 |
*** Kennan has quit IRC | 01:17 | |
*** dims has joined #openstack-nova | 01:20 | |
*** davideagnello has joined #openstack-nova | 01:20 | |
*** SourabhP has quit IRC | 01:22 | |
*** Kennan has joined #openstack-nova | 01:24 | |
*** angdraug has quit IRC | 01:24 | |
*** markvoelker has joined #openstack-nova | 01:25 | |
*** wanghua has joined #openstack-nova | 01:26 | |
*** haomaiwang has quit IRC | 01:26 | |
*** signed8b_ is now known as signed8bit_ZZZzz | 01:29 | |
*** ivasev has quit IRC | 01:33 | |
*** signed8bit_ZZZzz is now known as signed8b_ | 01:33 | |
*** annashen has joined #openstack-nova | 01:33 | |
*** jdandrea has quit IRC | 01:34 | |
*** shuquan has joined #openstack-nova | 01:36 | |
*** davideagnello has quit IRC | 01:37 | |
alex_xu | snikitin: hi, the patch LGTM :) | 01:40 |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Adding v2 and v3(v2.1) support https://review.openstack.org/149828 | 01:44 |
*** dougwig has quit IRC | 01:47 | |
*** dougwig has joined #openstack-nova | 01:47 | |
*** VW_ has joined #openstack-nova | 01:48 | |
*** liusheng has joined #openstack-nova | 01:49 | |
alex_xu | snikitin: fyi, after this patch https://review.openstack.org/124338 merged, you probably need a rebase | 01:50 |
*** thorst has joined #openstack-nova | 01:51 | |
*** baoli has joined #openstack-nova | 01:52 | |
*** lyanchih has joined #openstack-nova | 01:56 | |
*** signed8b_ is now known as signed8bit_ZZZzz | 01:56 | |
*** zzzeek has quit IRC | 01:57 | |
*** suresh12 has joined #openstack-nova | 01:59 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP][DNM] Not for review - get py34 test discovery to work https://review.openstack.org/204723 | 02:03 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 02:04 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image https://review.openstack.org/187857 | 02:09 |
*** yamahata has joined #openstack-nova | 02:09 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 02:09 |
*** VW_ has quit IRC | 02:10 | |
*** VW_ has joined #openstack-nova | 02:11 | |
*** achanda has quit IRC | 02:12 | |
*** jlearman has quit IRC | 02:13 | |
*** jlearman has joined #openstack-nova | 02:14 | |
*** VW_ has quit IRC | 02:15 | |
*** y_sawai has joined #openstack-nova | 02:16 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Set default values to sub-projects and users. https://review.openstack.org/151677 | 02:16 |
*** otter768 has joined #openstack-nova | 02:16 | |
*** gyee has quit IRC | 02:17 | |
*** jlearman has quit IRC | 02:17 | |
*** ijuwang has joined #openstack-nova | 02:18 | |
*** y_sawai has quit IRC | 02:19 | |
*** shuquan has quit IRC | 02:20 | |
*** shuquan has joined #openstack-nova | 02:22 | |
*** unicell has quit IRC | 02:22 | |
*** tpeoples has quit IRC | 02:22 | |
*** salv-orlando has joined #openstack-nova | 02:23 | |
*** suresh12 has quit IRC | 02:23 | |
*** suresh12 has joined #openstack-nova | 02:23 | |
*** shashank_hegde has joined #openstack-nova | 02:25 | |
*** dims has quit IRC | 02:27 | |
*** salv-orlando has quit IRC | 02:27 | |
*** suresh12 has quit IRC | 02:28 | |
*** ijuwang_ has joined #openstack-nova | 02:32 | |
*** ijuwang has quit IRC | 02:33 | |
openstackgerrit | sajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects https://review.openstack.org/200342 | 02:33 |
openstackgerrit | sajeesh cs proposed openstack/nova: NestedQuotaDriver:Modification of settable quotas of nested projects https://review.openstack.org/200342 | 02:35 |
*** yamahata has quit IRC | 02:36 | |
*** deepthi has joined #openstack-nova | 02:36 | |
*** VW_ has joined #openstack-nova | 02:39 | |
*** vilobhmm has quit IRC | 02:39 | |
*** ddaskal has quit IRC | 02:45 | |
*** kiran-r has joined #openstack-nova | 02:47 | |
*** hakimo_ has joined #openstack-nova | 02:52 | |
*** baoli has quit IRC | 02:53 | |
*** baoli has joined #openstack-nova | 02:53 | |
*** tpeoples has joined #openstack-nova | 02:53 | |
*** thorst has quit IRC | 02:54 | |
*** hakimo has quit IRC | 02:54 | |
*** sorrison_ has joined #openstack-nova | 02:57 | |
*** baoli has quit IRC | 02:57 | |
*** dave-mcc_ has joined #openstack-nova | 02:58 | |
*** signed8bit has joined #openstack-nova | 02:58 | |
*** bradjones has quit IRC | 02:59 | |
*** sorrison_ has quit IRC | 02:59 | |
*** ashuk has quit IRC | 02:59 | |
*** humble_ has joined #openstack-nova | 03:00 | |
*** ashuk has joined #openstack-nova | 03:00 | |
*** ByPasS has quit IRC | 03:01 | |
*** dave-mccowan has quit IRC | 03:01 | |
*** signed8bit_ZZZzz has quit IRC | 03:02 | |
*** bradjones has joined #openstack-nova | 03:02 | |
*** bradjones has quit IRC | 03:02 | |
*** bradjones has joined #openstack-nova | 03:02 | |
*** signed8bit is now known as signed8bit_ZZZzz | 03:02 | |
*** suresh12 has joined #openstack-nova | 03:02 | |
*** wanghua has quit IRC | 03:03 | |
*** suresh12 has quit IRC | 03:03 | |
*** kiran-r has quit IRC | 03:04 | |
*** suresh12 has joined #openstack-nova | 03:04 | |
*** baoli has joined #openstack-nova | 03:07 | |
*** piyanai has quit IRC | 03:08 | |
*** rushiagr_away is now known as rushiagr | 03:09 | |
*** jasondotstar has quit IRC | 03:10 | |
*** jwcroppe has joined #openstack-nova | 03:11 | |
*** baoli has quit IRC | 03:15 | |
*** kumar has joined #openstack-nova | 03:17 | |
*** david-lyle has joined #openstack-nova | 03:17 | |
*** shuquan has quit IRC | 03:19 | |
*** sajeesh has quit IRC | 03:20 | |
*** shuquan has joined #openstack-nova | 03:22 | |
*** signed8bit_ZZZzz is now known as signed8bit | 03:22 | |
*** otter768 has quit IRC | 03:25 | |
*** rushiagr is now known as rushiagr_away | 03:28 | |
openstackgerrit | Cale Rath proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 03:29 |
*** hightall has joined #openstack-nova | 03:36 | |
*** hightall has quit IRC | 03:37 | |
*** signed8bit has quit IRC | 03:37 | |
*** markvoelker has quit IRC | 03:38 | |
*** ashuk has quit IRC | 03:38 | |
*** liusheng has quit IRC | 03:39 | |
*** hightall has joined #openstack-nova | 03:39 | |
*** liusheng has joined #openstack-nova | 03:40 | |
*** coolsvap|away is now known as coolsvap | 03:40 | |
*** hightall has quit IRC | 03:40 | |
*** hightall has joined #openstack-nova | 03:41 | |
*** hightall has quit IRC | 03:44 | |
*** fawadkhaliq has joined #openstack-nova | 03:47 | |
*** salv-orlando has joined #openstack-nova | 03:49 | |
*** vilobhmm has joined #openstack-nova | 03:51 | |
*** shuquan has quit IRC | 03:51 | |
*** tpeoples has quit IRC | 03:54 | |
*** salv-orlando has quit IRC | 03:55 | |
*** unicell has joined #openstack-nova | 03:55 | |
*** deepthi has quit IRC | 03:58 | |
*** unicell has quit IRC | 04:00 | |
*** unicell has joined #openstack-nova | 04:01 | |
*** markvoelker has joined #openstack-nova | 04:03 | |
*** Mike_D_laptop has joined #openstack-nova | 04:03 | |
*** achanda has joined #openstack-nova | 04:07 | |
*** jwcroppe has quit IRC | 04:07 | |
*** dave-mcc_ has quit IRC | 04:07 | |
*** jwcroppe has joined #openstack-nova | 04:08 | |
*** markvoelker has quit IRC | 04:08 | |
*** rushiagr_away is now known as rushiagr | 04:09 | |
*** jwcroppe has quit IRC | 04:12 | |
*** Mike_D_laptop has quit IRC | 04:13 | |
*** ijuwang_ has quit IRC | 04:18 | |
*** mdrabe has quit IRC | 04:22 | |
*** kumar has quit IRC | 04:22 | |
*** rajesht has joined #openstack-nova | 04:23 | |
*** cfriesen has quit IRC | 04:24 | |
*** cfriesen_ has joined #openstack-nova | 04:24 | |
*** annashen has quit IRC | 04:24 | |
*** fawadk has joined #openstack-nova | 04:30 | |
*** shuquan has joined #openstack-nova | 04:32 | |
*** fawadkhaliq has quit IRC | 04:33 | |
*** lyanchih has quit IRC | 04:33 | |
*** annashen has joined #openstack-nova | 04:33 | |
*** cfriesen__ has joined #openstack-nova | 04:34 | |
*** cfriesen_ has quit IRC | 04:37 | |
*** hightall has joined #openstack-nova | 04:39 | |
*** armax has quit IRC | 04:40 | |
*** mutoulbj has joined #openstack-nova | 04:44 | |
*** y_sawai has joined #openstack-nova | 04:46 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add template_parameter_relation JSON-Home methods https://review.openstack.org/205374 | 04:54 |
*** Mike_D_laptop has joined #openstack-nova | 04:56 | |
*** lyanchih has joined #openstack-nova | 04:57 | |
*** coolsvap is now known as coolsvap|away | 05:00 | |
*** Mike_D_laptop has quit IRC | 05:03 | |
*** y_sawai_ has joined #openstack-nova | 05:04 | |
*** VW_ has quit IRC | 05:04 | |
*** markvoelker has joined #openstack-nova | 05:04 | |
*** VW_ has joined #openstack-nova | 05:04 | |
*** coolsvap|away is now known as coolsvap | 05:06 | |
*** y_sawai has quit IRC | 05:06 | |
*** y_sawai_ has quit IRC | 05:06 | |
*** deepthi has joined #openstack-nova | 05:08 | |
*** VW_ has quit IRC | 05:09 | |
*** markvoelker has quit IRC | 05:09 | |
*** lpetrut has joined #openstack-nova | 05:10 | |
*** coolsvap is now known as coolsvap|away | 05:12 | |
*** suresh12 has quit IRC | 05:15 | |
*** kmARC has joined #openstack-nova | 05:16 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Support for registering JSON-Home data- show/delete/update https://review.openstack.org/205001 | 05:16 |
*** kmARC_ has joined #openstack-nova | 05:17 | |
*** jwcroppe has joined #openstack-nova | 05:18 | |
*** belmoreira has joined #openstack-nova | 05:18 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Support for registering JSON-Home data- show/delete/update https://review.openstack.org/205001 | 05:20 |
*** lpetrut has quit IRC | 05:22 | |
*** xyang1 has quit IRC | 05:24 | |
*** otter768 has joined #openstack-nova | 05:26 | |
*** vilobhmm has quit IRC | 05:27 | |
*** otter768 has quit IRC | 05:31 | |
*** deepakcs has joined #openstack-nova | 05:32 | |
*** dannywilson has joined #openstack-nova | 05:33 | |
*** nkrinner has joined #openstack-nova | 05:33 | |
*** SourabhP has joined #openstack-nova | 05:35 | |
*** rajesht has quit IRC | 05:42 | |
*** Murali has joined #openstack-nova | 05:42 | |
*** ajayaa has joined #openstack-nova | 05:44 | |
*** salv-orlando has joined #openstack-nova | 05:48 | |
*** kiran-r has joined #openstack-nova | 05:51 | |
*** kiran-r has quit IRC | 05:52 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add register method for JSON-Home data- index/create https://review.openstack.org/204889 | 05:52 |
*** coolsvap|away is now known as coolsvap | 05:54 | |
*** dannywilson has quit IRC | 05:54 | |
*** dannywilson has joined #openstack-nova | 05:55 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add template_parameter_relation JSON-Home methods https://review.openstack.org/205374 | 05:55 |
*** salv-orlando has quit IRC | 05:55 | |
*** ajayaa has quit IRC | 06:00 | |
*** kmARC_ has quit IRC | 06:02 | |
*** kmARC has quit IRC | 06:02 | |
*** rex_lee has quit IRC | 06:03 | |
*** rex_lee has joined #openstack-nova | 06:03 | |
*** rotbeard has joined #openstack-nova | 06:04 | |
*** Nikolay_St has quit IRC | 06:04 | |
*** scheuran has joined #openstack-nova | 06:06 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Support for registering JSON-Home data- show/delete/update https://review.openstack.org/205001 | 06:07 |
*** belmoreira has quit IRC | 06:08 | |
*** ankit_ag has joined #openstack-nova | 06:10 | |
*** vishwanathj has quit IRC | 06:13 | |
*** itzikb_ has joined #openstack-nova | 06:13 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add resource_template_with_parent JSON-Home methods https://review.openstack.org/205392 | 06:17 |
*** jcoufal has joined #openstack-nova | 06:19 | |
*** ajayaa has joined #openstack-nova | 06:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/197213 | 06:24 |
*** apuimedo has joined #openstack-nova | 06:24 | |
*** exploreshaifali has joined #openstack-nova | 06:24 | |
openstackgerrit | Sergey Vilgelm proposed openstack/nova: Switch to the oslo_utils.fileutils https://review.openstack.org/199575 | 06:27 |
*** vilobhmm has joined #openstack-nova | 06:28 | |
*** itzikb_ has quit IRC | 06:31 | |
*** annashen has quit IRC | 06:31 | |
*** y_sawai has joined #openstack-nova | 06:34 | |
*** annashen has joined #openstack-nova | 06:36 | |
*** annashen has quit IRC | 06:36 | |
*** vilobhmm has quit IRC | 06:37 | |
*** VW_ has joined #openstack-nova | 06:43 | |
*** VW_ has quit IRC | 06:43 | |
*** VW_ has joined #openstack-nova | 06:43 | |
*** y_sawai has quit IRC | 06:45 | |
*** VW_ has quit IRC | 06:48 | |
*** salv-orlando has joined #openstack-nova | 06:53 | |
*** slong has quit IRC | 06:54 | |
*** mkoderer_cloud has joined #openstack-nova | 06:54 | |
*** davideagnello has joined #openstack-nova | 06:54 | |
*** belmoreira has joined #openstack-nova | 06:55 | |
*** sahid has joined #openstack-nova | 06:56 | |
openstackgerrit | Johnson koil raj proposed openstack/nova: Provide an option to set disk.EnableUUID in vmx https://review.openstack.org/205269 | 06:56 |
openstackgerrit | Li Yingjun proposed openstack/python-novaclient: Fixes table when there are multiline in result data https://review.openstack.org/204339 | 06:56 |
*** dannywilson has quit IRC | 06:57 | |
*** salv-orlando has quit IRC | 06:57 | |
*** mpavone has joined #openstack-nova | 06:58 | |
*** mpavone has quit IRC | 06:58 | |
*** Nikolay_St has joined #openstack-nova | 06:58 | |
*** SourabhP has quit IRC | 06:59 | |
*** markvoelker has joined #openstack-nova | 07:05 | |
*** moshele has joined #openstack-nova | 07:05 | |
*** rdopiera has joined #openstack-nova | 07:10 | |
*** pcaruana has joined #openstack-nova | 07:10 | |
*** markvoelker has quit IRC | 07:10 | |
*** belmoreira has quit IRC | 07:12 | |
*** pcaruana is now known as centos | 07:13 | |
*** centos is now known as Guest59730 | 07:13 | |
*** Guest59730 is now known as pcaruana | 07:13 | |
*** moshele has quit IRC | 07:15 | |
*** browne has quit IRC | 07:18 | |
*** annashen has joined #openstack-nova | 07:24 | |
*** otter768 has joined #openstack-nova | 07:27 | |
*** shuquan has quit IRC | 07:29 | |
*** achanda has quit IRC | 07:29 | |
*** otter768 has quit IRC | 07:32 | |
*** achanda has joined #openstack-nova | 07:34 | |
openstackgerrit | Abhijeet Malawade proposed openstack/nova: Validate quota class_name https://review.openstack.org/192115 | 07:35 |
*** davideagnello has quit IRC | 07:38 | |
openstackgerrit | Einst Crazy proposed openstack/nova: libvirtd: Bug fix for deleting snapshot of attached volume https://review.openstack.org/204617 | 07:41 |
*** jlanoux has joined #openstack-nova | 07:41 | |
*** shuquan has joined #openstack-nova | 07:42 | |
*** achanda has quit IRC | 07:44 | |
*** mkoderer has joined #openstack-nova | 07:50 | |
*** atuvenie has joined #openstack-nova | 07:51 | |
*** annashen has quit IRC | 07:52 | |
*** pixelbeat_ has joined #openstack-nova | 07:58 | |
*** mpaolino has joined #openstack-nova | 07:58 | |
*** Jeffrey4l has quit IRC | 07:59 | |
*** jwcroppe has quit IRC | 08:03 | |
*** SourabhP has joined #openstack-nova | 08:03 | |
*** jistr has joined #openstack-nova | 08:04 | |
*** ftersin has quit IRC | 08:04 | |
*** pixelbeat_ has quit IRC | 08:11 | |
*** romainh has joined #openstack-nova | 08:11 | |
*** derekh has joined #openstack-nova | 08:13 | |
*** kmARC_ has joined #openstack-nova | 08:14 | |
*** lpetrut has joined #openstack-nova | 08:14 | |
*** gszasz has joined #openstack-nova | 08:15 | |
*** belmoreira has joined #openstack-nova | 08:18 | |
*** moshele has joined #openstack-nova | 08:19 | |
*** yamahata has joined #openstack-nova | 08:19 | |
*** kmARC has joined #openstack-nova | 08:19 | |
*** salv-orlando has joined #openstack-nova | 08:20 | |
*** achanda has joined #openstack-nova | 08:21 | |
*** Nikolay_St has quit IRC | 08:24 | |
*** salv-orlando has quit IRC | 08:25 | |
*** SourabhP has quit IRC | 08:29 | |
*** garyk has quit IRC | 08:33 | |
*** aix has joined #openstack-nova | 08:38 | |
*** moshele has quit IRC | 08:38 | |
*** ZZelle has quit IRC | 08:40 | |
*** ZZelle has joined #openstack-nova | 08:40 | |
*** yamahata has quit IRC | 08:41 | |
*** karimb has joined #openstack-nova | 08:42 | |
*** y_sawai has joined #openstack-nova | 08:43 | |
*** belmoreira has quit IRC | 08:44 | |
*** moshele has joined #openstack-nova | 08:48 | |
*** shuquan_ has joined #openstack-nova | 08:49 | |
*** shuquan has quit IRC | 08:50 | |
*** shuquan_ is now known as shuquan | 08:50 | |
*** y_sawai has quit IRC | 08:50 | |
*** markvoelker has joined #openstack-nova | 08:51 | |
*** y_sawai has joined #openstack-nova | 08:52 | |
*** MarcF has quit IRC | 08:54 | |
*** markvoelker has quit IRC | 08:56 | |
*** danpb has joined #openstack-nova | 08:57 | |
*** shashank_hegde has quit IRC | 08:57 | |
*** pixelbeat_ has joined #openstack-nova | 09:00 | |
*** marzif has joined #openstack-nova | 09:02 | |
openstackgerrit | Marian Horban proposed openstack/nova: libvirt:Rsync remote FS driver was added https://review.openstack.org/146887 | 09:06 |
*** marzif has quit IRC | 09:08 | |
*** marzif has joined #openstack-nova | 09:09 | |
*** marzif has quit IRC | 09:11 | |
*** marzif has joined #openstack-nova | 09:12 | |
openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: hypervisor support matrix: fix snapshot for libvirt Xen https://review.openstack.org/197301 | 09:16 |
*** shuquan has quit IRC | 09:18 | |
*** shuquan has joined #openstack-nova | 09:18 | |
*** hightall has quit IRC | 09:20 | |
*** mpaolino has quit IRC | 09:20 | |
*** mpaolino has joined #openstack-nova | 09:21 | |
*** lsmola has quit IRC | 09:21 | |
*** salv-orlando has joined #openstack-nova | 09:22 | |
*** y_sawai has quit IRC | 09:26 | |
*** otter768 has joined #openstack-nova | 09:28 | |
*** otter768 has quit IRC | 09:33 | |
*** shuquan has quit IRC | 09:33 | |
*** achanda has quit IRC | 09:35 | |
*** coolsvap is now known as coolsvap|away | 09:39 | |
*** ftersin has joined #openstack-nova | 09:43 | |
*** Nikolay_St has joined #openstack-nova | 09:56 | |
*** jlanoux_ has joined #openstack-nova | 09:58 | |
*** y_sawai has joined #openstack-nova | 09:58 | |
*** y_sawai has quit IRC | 09:59 | |
*** jlanoux has quit IRC | 10:01 | |
*** y_sawai has joined #openstack-nova | 10:04 | |
*** romainh has quit IRC | 10:04 | |
*** mkoderer_cloud has quit IRC | 10:04 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw_cpu_threads_policy' image metadata prop https://review.openstack.org/202647 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'cpu_policy' and 'cpu_threads_policy' fields https://review.openstack.org/202648 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=avoid' filtering https://review.openstack.org/202649 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 10:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 10:04 |
*** y_sawai has quit IRC | 10:13 | |
*** hshiina has joined #openstack-nova | 10:16 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added removing of tags from instance after its destroying https://review.openstack.org/204053 | 10:16 |
*** humble__ has joined #openstack-nova | 10:23 | |
*** humble_ has quit IRC | 10:27 | |
*** jcoufal has quit IRC | 10:27 | |
*** lpetrut1 has joined #openstack-nova | 10:30 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: implement get_mks_console() https://review.openstack.org/204059 | 10:32 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: Add MKS console support https://review.openstack.org/203006 | 10:32 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: Add MKS protocol for remote consoles https://review.openstack.org/199129 | 10:32 |
*** lpetrut has quit IRC | 10:33 | |
*** y_sawai has joined #openstack-nova | 10:34 | |
*** jlanoux has joined #openstack-nova | 10:35 | |
*** dims has joined #openstack-nova | 10:36 | |
*** atuvenie_ has joined #openstack-nova | 10:38 | |
*** jlanoux_ has quit IRC | 10:38 | |
*** atuvenie has quit IRC | 10:41 | |
*** marzif has quit IRC | 10:45 | |
*** marzif has joined #openstack-nova | 10:46 | |
*** logan2 has quit IRC | 10:47 | |
*** mpaolino has quit IRC | 10:47 | |
*** alex_klimov has joined #openstack-nova | 10:49 | |
*** alex_klimov has quit IRC | 10:49 | |
*** alex_klimov has joined #openstack-nova | 10:49 | |
*** logan2 has joined #openstack-nova | 10:50 | |
*** piyanai has joined #openstack-nova | 10:50 | |
*** markvoelker has joined #openstack-nova | 10:52 | |
*** erlon has joined #openstack-nova | 10:52 | |
*** y_sawai has quit IRC | 10:53 | |
*** erlon has quit IRC | 10:53 | |
*** erlon has joined #openstack-nova | 10:53 | |
*** markvoelker has quit IRC | 10:57 | |
*** lyanchih has quit IRC | 10:58 | |
*** y_sawai has joined #openstack-nova | 11:00 | |
*** openstackgerrit has quit IRC | 11:01 | |
*** openstackgerrit has joined #openstack-nova | 11:02 | |
*** jlanoux has quit IRC | 11:07 | |
*** deepakcs has quit IRC | 11:07 | |
*** zhenguo has quit IRC | 11:09 | |
openstackgerrit | Sergey Vilgelm proposed openstack/nova: Switch to the oslo_utils.fileutils https://review.openstack.org/199575 | 11:11 |
*** abhishekk has joined #openstack-nova | 11:12 | |
*** jwcroppe has joined #openstack-nova | 11:17 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP][DNM] Not for review - get py34 test discovery to work https://review.openstack.org/204723 | 11:21 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 11:21 |
*** apuimedo|dinner has quit IRC | 11:27 | |
*** otter768 has joined #openstack-nova | 11:29 | |
*** romainh has joined #openstack-nova | 11:31 | |
*** otter768 has quit IRC | 11:33 | |
*** lyanchih has joined #openstack-nova | 11:36 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Changed backporting of embedded Network object when backporting a FixedIP https://review.openstack.org/173723 | 11:40 |
*** Marga_ has quit IRC | 11:42 | |
*** jwcroppe has quit IRC | 11:46 | |
*** jwcroppe has joined #openstack-nova | 11:47 | |
*** dave-mccowan has joined #openstack-nova | 11:49 | |
*** Marga_ has joined #openstack-nova | 11:49 | |
*** jlanoux has joined #openstack-nova | 11:51 | |
*** jwcroppe has quit IRC | 11:52 | |
*** apuimedo|dinner has joined #openstack-nova | 11:52 | |
*** gszasz has quit IRC | 11:53 | |
*** gszasz has joined #openstack-nova | 11:55 | |
*** gszasz has quit IRC | 11:55 | |
*** mjturek1 has joined #openstack-nova | 12:03 | |
*** Marga_ has quit IRC | 12:05 | |
*** thorst has joined #openstack-nova | 12:05 | |
*** Jeffrey4l has joined #openstack-nova | 12:06 | |
*** Marga_ has joined #openstack-nova | 12:06 | |
*** hshiina has quit IRC | 12:07 | |
*** moshele has quit IRC | 12:08 | |
*** ayoung has quit IRC | 12:10 | |
*** Jeffrey4l has quit IRC | 12:11 | |
*** Jeffrey4l has joined #openstack-nova | 12:12 | |
*** thorst has quit IRC | 12:13 | |
*** raildo has joined #openstack-nova | 12:14 | |
*** humble_ has joined #openstack-nova | 12:17 | |
*** y_sawai_ has joined #openstack-nova | 12:18 | |
*** deepthi has quit IRC | 12:19 | |
*** wasmum has quit IRC | 12:20 | |
*** humble__ has quit IRC | 12:20 | |
*** y_sawai has quit IRC | 12:21 | |
*** mgoddard has quit IRC | 12:22 | |
*** krtaylor has quit IRC | 12:23 | |
*** krtaylor has joined #openstack-nova | 12:25 | |
*** markvoelker has joined #openstack-nova | 12:25 | |
*** READ10 has joined #openstack-nova | 12:25 | |
*** apuimedo has quit IRC | 12:26 | |
*** baoli has joined #openstack-nova | 12:28 | |
*** aysyd has joined #openstack-nova | 12:30 | |
*** Jeffrey4l has quit IRC | 12:31 | |
*** baoli has quit IRC | 12:32 | |
*** scheuran has quit IRC | 12:32 | |
openstackgerrit | lyanchih proposed openstack/nova: Creating instance fail when inject ssh key in cells mode https://review.openstack.org/197514 | 12:34 |
openstackgerrit | lyanchih proposed openstack/nova: Remove fixed test in tempest-dsvm-cells-rc https://review.openstack.org/205512 | 12:34 |
*** piyanai has quit IRC | 12:34 | |
*** baoli has joined #openstack-nova | 12:35 | |
*** baoli has quit IRC | 12:35 | |
*** mgoddard has joined #openstack-nova | 12:35 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 12:35 |
*** baoli_ has joined #openstack-nova | 12:36 | |
*** vladikr has joined #openstack-nova | 12:37 | |
*** dprince has joined #openstack-nova | 12:39 | |
*** Murali has quit IRC | 12:40 | |
*** markus_z has joined #openstack-nova | 12:40 | |
*** baoli has joined #openstack-nova | 12:41 | |
*** mdrabe has joined #openstack-nova | 12:41 | |
*** zhenguo has joined #openstack-nova | 12:42 | |
*** baoli_ has quit IRC | 12:45 | |
*** mpaolino has joined #openstack-nova | 12:48 | |
*** mgoddard has quit IRC | 12:49 | |
*** chuckcarmack has joined #openstack-nova | 12:51 | |
*** dims has quit IRC | 12:52 | |
*** dims has joined #openstack-nova | 12:53 | |
*** humble_ has quit IRC | 12:53 | |
*** VW_ has joined #openstack-nova | 12:54 | |
*** annegentle has joined #openstack-nova | 12:55 | |
*** jdandrea has joined #openstack-nova | 12:59 | |
bauzas | OH "The change requires a local merge to resolve.Please merge (or rebase) the change locally and upload the resolution for review." | 12:59 |
bauzas | funny | 13:00 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Add two fields to ImageMetaProps object https://review.openstack.org/197911 | 13:00 |
*** MarcF has joined #openstack-nova | 13:00 | |
*** ferest has joined #openstack-nova | 13:02 | |
*** nkrinner has quit IRC | 13:02 | |
*** MentalRay has joined #openstack-nova | 13:03 | |
*** wasmum has joined #openstack-nova | 13:04 | |
*** ferest has quit IRC | 13:04 | |
*** abhishekk has quit IRC | 13:04 | |
*** lyanchih has quit IRC | 13:04 | |
*** VW_ has quit IRC | 13:05 | |
*** VW_ has joined #openstack-nova | 13:05 | |
*** yassine__ has joined #openstack-nova | 13:06 | |
*** rlrossit has joined #openstack-nova | 13:09 | |
*** lyanchih has joined #openstack-nova | 13:09 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw_cpu_threads_policy' to ImageMetaProps https://review.openstack.org/202647 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'cpu_policy' and 'cpu_threads_policy' fields https://review.openstack.org/202648 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=avoid' filtering https://review.openstack.org/202649 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 13:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 13:09 |
*** browne has joined #openstack-nova | 13:12 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 13:12 |
openstackgerrit | Daniel Berrange proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 13:12 |
*** ijuwang has joined #openstack-nova | 13:13 | |
*** jasondotstar has joined #openstack-nova | 13:13 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 13:14 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Check flavor type before add tenant access https://review.openstack.org/124338 | 13:14 |
*** annegentle has quit IRC | 13:15 | |
*** salv-orlando has quit IRC | 13:16 | |
*** annegentle has joined #openstack-nova | 13:17 | |
*** rushiagr is now known as rushiagr_away | 13:17 | |
*** snikitin has quit IRC | 13:17 | |
*** eharney has joined #openstack-nova | 13:18 | |
*** thorst1 has joined #openstack-nova | 13:18 | |
*** ankit_ag has quit IRC | 13:19 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 13:20 |
*** annegentle has quit IRC | 13:22 | |
*** fawadk has quit IRC | 13:23 | |
*** fawadkhaliq has joined #openstack-nova | 13:24 | |
*** lyanchih has quit IRC | 13:26 | |
*** ctrath has joined #openstack-nova | 13:26 | |
*** dane-fichter has joined #openstack-nova | 13:28 | |
*** jistr is now known as jistr|mtg | 13:29 | |
*** otter768 has joined #openstack-nova | 13:30 | |
*** mgoddard has joined #openstack-nova | 13:30 | |
*** exploreshaifali has quit IRC | 13:30 | |
*** tristanC has quit IRC | 13:31 | |
*** READ10 has quit IRC | 13:31 | |
*** annashen has joined #openstack-nova | 13:32 | |
*** tristanC has joined #openstack-nova | 13:32 | |
*** moshele has joined #openstack-nova | 13:32 | |
*** otter768 has quit IRC | 13:34 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: Change List objects to use obj_relationships https://review.openstack.org/198730 | 13:38 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 13:39 |
*** clopes has joined #openstack-nova | 13:40 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Fixed indentation https://review.openstack.org/205529 | 13:42 |
*** ayoung has joined #openstack-nova | 13:42 | |
*** mriedem has joined #openstack-nova | 13:44 | |
*** ctrath has quit IRC | 13:44 | |
*** annashen has quit IRC | 13:46 | |
*** ajayaa has quit IRC | 13:47 | |
*** armax has joined #openstack-nova | 13:49 | |
dansmith | mriedem: easy backport approval for your friday morning: https://review.openstack.org/#/c/203052/ | 13:51 |
*** dansmith is now known as superdan | 13:52 | |
mriedem | hrm, aren't we in stable freeze? | 13:52 |
superdan | for juno? | 13:52 |
mriedem | oh nvm | 13:52 |
superdan | and I thought kilo was done, fwiw | 13:52 |
mriedem | superdan: is this missing a dependent squash? the test diffs are quite different | 13:54 |
bauzas | oh man | 13:55 |
superdan | it's because of the unit move | 13:55 |
*** bauzas is now known as bauwser | 13:55 | |
mriedem | no that's not what i mean | 13:55 |
mriedem | LOC | 13:55 |
superdan | oh, right, | 13:55 |
mriedem | we're missing a thing that added the tests originally | 13:56 |
superdan | there was another couple of test things that these tests depended on | 13:56 |
mriedem | https://github.com/openstack/nova/commit/3ce8e8f90be67fb49efd0c60e2a4f5ae9da503c5 | 13:56 |
*** edmondsw has joined #openstack-nova | 13:56 | |
superdan | yeah | 13:56 |
*** markus_z has quit IRC | 13:56 | |
*** unicell1 has joined #openstack-nova | 13:58 | |
*** unicell has quit IRC | 13:58 | |
openstackgerrit | Atsushi SAKAI proposed openstack/nova: Fix seven typos on nova documentation https://review.openstack.org/205540 | 13:59 |
*** piyanai has joined #openstack-nova | 14:00 | |
*** rushiagr_away is now known as rushiagr | 14:01 | |
*** ctrath has joined #openstack-nova | 14:03 | |
*** thangp has joined #openstack-nova | 14:03 | |
*** browne has quit IRC | 14:05 | |
superdan | mriedem: oh, I was fixing the backport | 14:06 |
mriedem | superdan: you should have said so :) | 14:06 |
superdan | sorry | 14:06 |
superdan | but see if that looks right | 14:07 |
superdan | I'm glad you caught that.. my brain at 34000ft didn't realize/remember the context there | 14:07 |
*** thorst has joined #openstack-nova | 14:08 | |
*** thorst1 has quit IRC | 14:08 | |
mriedem | -1 bro | 14:09 |
mriedem | broseph | 14:09 |
superdan | oh it would help if I tried running tests | 14:09 |
mriedem | :) | 14:10 |
mriedem | feel free to gdi any time now | 14:10 |
superdan | GDI | 14:10 |
*** ramishra has quit IRC | 14:10 | |
*** ramishra has joined #openstack-nova | 14:10 | |
mriedem | fwiw you could squash those | 14:10 |
*** r-daneel has joined #openstack-nova | 14:11 | |
*** DuncanT has quit IRC | 14:11 | |
superdan | seems better to keep them separate if we can, no? | 14:11 |
*** DuncanT has joined #openstack-nova | 14:11 | |
superdan | what is the benefit of squashing? | 14:11 |
*** jistr|mtg is now known as jistr | 14:12 | |
mriedem | it doesn't matter too much either way, just we wouldn't take the test change w/o the bug fix after | 14:12 |
superdan | sure | 14:12 |
superdan | sigh | 14:12 |
mriedem | so kind of makes sense to have them together, but we'll merge together regardless, so doesn't matter | 14:12 |
superdan | tox or pbr or something hates me | 14:12 |
superdan | Error parsing /home/dan/nova/setup.cfg: ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.2.3), but target version is pbr.version.SemanticVersion(2014.2.2) | 14:13 |
mriedem | did you rebase juno? | 14:13 |
superdan | no | 14:13 |
mriedem | else they released juno but didn't bump the versions in setup.cfg | 14:13 |
mriedem | or there is a change outstanding - probably want to rebase first before you cherry pick | 14:13 |
mriedem | https://github.com/openstack/nova/commit/2fbc2a70cf21900c0bc12e4a02d54155f8162a81 | 14:14 |
mriedem | make sure you have that | 14:14 |
* bauwser is jealous to see network connectivity in places | 14:14 | |
bauwser | planes even | 14:14 |
bauwser | why is it something not compulsory for EU companies ? grrr | 14:14 |
superdan | mriedem: I have that | 14:14 |
mriedem | superdan: looks like you don't have the latest tags in your local repo | 14:15 |
mriedem | 2014.2.3 is the latest for juno | 14:15 |
superdan | oh maybe I need another thing | 14:15 |
mriedem | git pull origin? | 14:16 |
nagyz_ | mriedem, how hacky do you think it is to modify self.is_block_dev in either get_config or in connect_volume? unless I can get connection_info from a connection I cannot do it in the ctor. otherwise I have the patch working :) | 14:16 |
superdan | or fetch -t | 14:16 |
superdan | mriedem: still fails | 14:17 |
*** ctrath has quit IRC | 14:17 | |
mriedem | hrm, let me see if i'm having issues with stable/juno | 14:17 |
superdan | mriedem: and I have 2014.2.3 in my tags | 14:17 |
mriedem | what version of pbr / tox is running? | 14:17 |
superdan | I just pushed it up to see if it runs there | 14:17 |
superdan | pbr==1.3.0 | 14:18 |
superdan | tox==2.1.1 | 14:18 |
rgerganov | TIL "run_tests.sh -8" doesn't work for chain of patches | 14:18 |
mriedem | rgerganov: don't use run_tests.sh, use tox | 14:19 |
*** jecarey has joined #openstack-nova | 14:19 | |
mriedem | nagyz_: i'd probably have to see it in a change | 14:19 |
rgerganov | mriedem: tox takes forever | 14:19 |
mriedem | use latest tox which uses wheels | 14:19 |
mriedem | and probably latest pip | 14:19 |
*** ZZelle has quit IRC | 14:20 | |
rgerganov | run_tests.sh -8 is super fast because it does PEP8 check only for the modified files | 14:20 |
rgerganov | but in my case I had a chain of patches and it screw up | 14:20 |
danpb | rgerganov: you're supposed to do # git rebase -i master -x './run_tests.sh -8' | 14:20 |
danpb | if you want something to be invoked against each patch in a series | 14:20 |
rgerganov | danpb: ah, nice | 14:20 |
rgerganov | thanks | 14:21 |
mriedem | gross | 14:21 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 14:21 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object https://review.openstack.org/194538 | 14:21 |
danpb | even running 'tox -epep8' is not a subsitute because it only verifies the final state of your branch | 14:21 |
*** mpaolino has quit IRC | 14:22 | |
danpb | you need to have full bisectability of patches with each one passing pep8, so you must always use a git rebase -x "command to run against each patch" | 14:22 |
mriedem | or just run pep8 as you work on each commit | 14:22 |
mriedem | or go back to edit a commit | 14:22 |
mriedem | i wouldn't work on 8 commits in a series and run pep8 at the end | 14:22 |
danpb | you'll inevitably forgot todo that at some point - that's why the -x arg to rebase is so useful | 14:22 |
rgerganov | run_tests.sh -8 has saved hours for me | 14:22 |
mriedem | i'll never ever forget | 14:23 |
rgerganov | it's the best hack ever | 14:23 |
mriedem | ever | 14:23 |
mriedem | hours? | 14:23 |
mriedem | tox -e pep8 doesn't take that long | 14:23 |
danpb | mriedem: it does in comparison to run_tests.sh -8 | 14:24 |
danpb | (at least if your commit has only touched a couple of files) | 14:24 |
*** tsekiyama has joined #openstack-nova | 14:24 | |
mriedem | so just tell lifeless that run_tests.sh -8 is better than testr with tox -e pep8 and he'll go insane with jealously and make it better with testr | 14:25 |
rgerganov | everything is better than tox | 14:25 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: implement get_mks_console() https://review.openstack.org/204059 | 14:25 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: Add MKS console support https://review.openstack.org/203006 | 14:25 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: Add MKS protocol for remote consoles https://review.openstack.org/199129 | 14:25 |
danpb | as an example with my current git master, tox -epep8 is 1 minute 45, while run_tests.sh -8 is 1 second | 14:25 |
*** lyanchih has joined #openstack-nova | 14:26 | |
rgerganov | danpb: enough said | 14:26 |
danpb | so a merely x100 faster | 14:26 |
*** ildikov is now known as ildikov_afk | 14:26 | |
superdan | how is your tox -epep8 taking 1:45? | 14:26 |
rgerganov | danpb: I have to buy you a beer for creating the -8 option | 14:26 |
superdan | that's insanely long | 14:26 |
danpb | rotating rust disk on this particular machine i used to illustrate it | 14:27 |
danpb | even on machines whre its faster though, the relative difference between the two is the same | 14:27 |
superdan | I'm on a spinny disk | 14:27 |
clopes | Hello guys! I am trying to config cinder-volume (with BlockDeviceDriver) on a compute node. The volume creation works fine but I get 'iscsiadm: No session found.' when I try to attach the volume to a VM. Help please? | 14:27 |
*** rook has joined #openstack-nova | 14:27 | |
*** browne has joined #openstack-nova | 14:27 | |
danpb | its not hard to understand why - in one case you're running pep8 on ~12,000 python files and in the other case you're running pep8 on < 10 files typically | 14:28 |
kashyap | rgerganov: Dan doesn't drink beer (IIRC), you can buy him a nice expensive telescope, though :-) | 14:28 |
*** mpaolino has joined #openstack-nova | 14:28 | |
danpb | rgerganov: actually you should reward gilliard - i merely suggested the idea - he implemented it | 14:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 14:30 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 14:30 |
superdan | run_tests.sh -8 doesn't give the same results of tox -epep8 for me | 14:30 |
danpb | that's by design - its only checking source files which your current HEAD commit has modified | 14:31 |
superdan | no, I mean it complains about things that we have configured out of flake8 | 14:31 |
superdan | so tox -epep8 passes (which means the gate would pass it) but run_tests complains about a ton of things | 14:31 |
mriedem | yeah because run_tests.sh doesn't have https://github.com/openstack/nova/blob/master/tox.ini#L126 | 14:32 |
superdan | so given that tox is only 23 seconds slower on my system, it would take me a lot longer to "fix" the things run_tests thinks are wrong but aren't | 14:32 |
superdan | mriedem: right | 14:32 |
superdan | therefore, tox == faster :) | 14:32 |
mriedem | plus tox pep8 runs the max complexity check which is in the gate | 14:32 |
rgerganov | I don't have any false positives from run_tests.sh -8 | 14:32 |
superdan | rgerganov: touch a libvirt driver file | 14:33 |
mriedem | i believe this would be a false negative | 14:33 |
mriedem | anyway, i was wondering at what point the pissing contest officially begins because i'd like to mark it on my calendar | 14:33 |
danpb | superdan: i don't get any violations on libvirt driver files | 14:33 |
superdan | the real reason not to use run_tests.sh is that it's unmaintained and different from the requirements to pass the gate | 14:33 |
superdan | danpb: I get 37 across the tests and driver files | 14:34 |
* danpb wonders if we have different flake8 versions | 14:34 | |
superdan | danpb: http://pastebin.com/Zyp9Zser | 14:34 |
superdan | danpb: none of those are configured to complain in tox -epep8 so it passes | 14:34 |
danpb | odd, i don't see any of those when i run it | 14:35 |
mriedem | danpb: are you in a venv? | 14:35 |
danpb | i think it should be in the venv | 14:35 |
*** jwcroppe has joined #openstack-nova | 14:36 | |
*** diogogmt has joined #openstack-nova | 14:36 | |
nagyz_ | mriedem, if I have code that touches the LibvirtVolumeDrivers, is it ok to work on top of master or should I rebase it on your change re. the restructuring of those? | 14:37 |
bauwser | I never saw any discrepancy between the gate and my ./run_tests.sh -8 runs | 14:38 |
*** tpeoples has joined #openstack-nova | 14:38 | |
bauwser | it works like a charm and just saves me for storing multiple venvs | 14:38 |
mriedem | nagyz_: you probably want to just work on top of master | 14:38 |
rgerganov | bauwser: +1 | 14:38 |
bauwser | I aslo appreciate the ./with_venv.sh stuff for quickly debugging | 14:38 |
kashyap | mriedem: I recall once ndipanov said you can't manage virtualenvs `tox` out of the box, but there's an option to do so. . . | 14:39 |
* danpb wonders about how to port run_tests.sh -8 to tox too | 14:39 | |
kashyap | Now that I look up: "envdir + recreate=False" | 14:39 |
bauwser | but all of that convo is probably yakshaving - are we on Friday ? :D | 14:39 |
mriedem | kashyap: i don't know what that means, what you said | 14:39 |
* bauwser should just say to the world how much he loves Sublime Text | 14:40 | |
mriedem | source .tox/py27/bin/activate | 14:40 |
mriedem | then do things | 14:40 |
*** krtaylor has quit IRC | 14:40 | |
*** tonytan4ever has joined #openstack-nova | 14:40 | |
kashyap | mriedem: Tell tox to not build a venv, and manage on your own | 14:42 |
kashyap | By using the 'recreate' option mentioned here: http://tox.readthedocs.org/en/latest/config.html | 14:42 |
mriedem | recreate just blows away the current venv for the env target | 14:42 |
mriedem | it's not like you don't have a venv with tox | 14:43 |
mriedem | that's the whole point | 14:43 |
mriedem | gdi | 14:43 |
mriedem | why are we all talking about this stuff on a friday morning? | 14:43 |
kashyap | mriedem: I'm still wrapping my head around the myriad ways of tests you can run. At least from what I learn `tox` seems to the recommended way, as it runs in parallel using all available CPUs. | 14:43 |
mriedem | kashyap: and it's what we gate on | 14:43 |
mriedem | so sure you could use nosetests but don't | 14:43 |
*** annegentle has joined #openstack-nova | 14:43 | |
bauwser | I didn't heard about tox using wheels, I should give it a try | 14:44 |
*** marzif has quit IRC | 14:44 | |
mriedem | it could be latest pip | 14:44 |
bauwser | because indeed, tox -epep8 was just a pain to me | 14:44 |
mriedem | i have latest pip and latest tox | 14:44 |
mriedem | since that's what upstream has | 14:44 |
bauwser | meh | 14:45 |
*** marzif has joined #openstack-nova | 14:45 | |
bauwser | I still remember all the fun I had with tox>=1.7 | 14:45 |
bauwser | so I was sticking myself on tox==1.6.1 | 14:46 |
* bauwser is now benchmarking tox vs. ./run_tests.sh on his big series | 14:46 | |
*** artom has joined #openstack-nova | 14:47 | |
bauwser | using the git rebase -x flag | 14:47 |
rgerganov | tox -e pep8 173.85s | 14:48 |
rgerganov | ./run_tests.sh -8 3.13s | 14:48 |
rgerganov | :) | 14:48 |
*** jaypipes has joined #openstack-nova | 14:48 | |
rgerganov | this is with SSD and quad core CPU | 14:48 |
bauwser | rgerganov: pip freeze ? | 14:49 |
danpb | even if I use ./run_tests.sh -8, it is in fact still opening tox.ini to get the list of exclusions, according to my strace, so i wonder why that failed for superdan | 14:49 |
bauwser | danpb: yup, the hacking checks are taken in account by ./run_tests.sh | 14:50 |
bauwser | danpb: honestly, like I said, I never saw any behavioural change between those | 14:50 |
bauwser | my take is about seeing how tox can use the python wheels and if it's better | 14:51 |
*** ajayaa has joined #openstack-nova | 14:51 | |
*** rgerganov has quit IRC | 14:51 | |
superdan | danpb: not sure why it'd be opening tox.ini either | 14:52 |
*** xyang2 has joined #openstack-nova | 14:52 | |
danpb | superdan: flake8 itself opens tox.ini according to my strace | 14:52 |
danpb | tox doesn't pass any of that exclusion list under [flake8] down to the command it invokes, so flake8 must be reading it | 14:53 |
superdan | danpb: heh, okay | 14:53 |
danpb | the relationship is all a bit bizarre to me | 14:53 |
superdan | mriedem: so can you run tests on juno? | 14:54 |
mriedem | superdan: let me check | 14:54 |
*** yamahata has joined #openstack-nova | 14:56 | |
mriedem | superdan: btw, i'll trade you one https://review.openstack.org/#/c/178446/ | 14:57 |
*** zzzeek has joined #openstack-nova | 14:58 | |
*** ZZelle has joined #openstack-nova | 14:59 | |
*** dave-mccowan has quit IRC | 14:59 | |
*** liusheng has quit IRC | 15:00 | |
*** sc68cal has quit IRC | 15:00 | |
bauwser | mmm, interesting https://bugs.launchpad.net/nova/+bug/1414947 | 15:00 |
openstack | Launchpad bug 1414947 in OpenStack Compute (nova) "instance‘s root_gb is 0, but the actual root_gb size is not 0." [Wishlist,Won't fix] | 15:00 |
bauwser | so, while we accept to have a flavor with root_gb=0, it makes the RT wrong | 15:00 |
bauwser | at the end, the scheduler can just make bad decisions nope ? | 15:01 |
openstackgerrit | Shiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI https://review.openstack.org/202617 | 15:02 |
*** sc68cal has joined #openstack-nova | 15:03 | |
*** annegentle has quit IRC | 15:04 | |
mriedem | superdan: pep8 was fine on stable/juno | 15:04 |
*** fawadkhaliq has quit IRC | 15:04 | |
*** annegentle has joined #openstack-nova | 15:05 | |
superdan | okay, I can't make that target either | 15:05 |
superdan | sigh | 15:05 |
*** davideagnello has joined #openstack-nova | 15:05 | |
lifeless | bauwser: tox and wheels? | 15:05 |
lifeless | bauwser: it uses the pip version from the virtualenv package in the same python environment tox is running from | 15:06 |
superdan | lifeless: I get this on stable/juno: Error parsing /home/dan/nova/setup.cfg: ValueError: git history requires a target version of pbr.version.SemanticVersion(2014.2.3), but target version is pbr.version.SemanticVersion(2014.2.2) | 15:06 |
bauwser | superdan: oh I saw that before | 15:06 |
*** atuvenie_ has quit IRC | 15:06 | |
mriedem | jbernard: have you seen this? http://logs.openstack.org/36/193236/10/check/gate-tempest-dsvm-full-ceph/5549dfc/logs/screen-n-cpu.txt.gz?level=TRACE | 15:06 |
bauwser | thought we were removing explicit targets ? | 15:06 |
mriedem | jbernard: the error logging was added here https://review.openstack.org/#/c/180584/ | 15:06 |
mriedem | that seems nuts | 15:06 |
lifeless | superdan: usually means that setup.cfg had an exact version in it and you've tagged that version andn then committed another commit | 15:07 |
*** cfriesen has joined #openstack-nova | 15:07 | |
lifeless | superdan: which leads to an impossible situation, so pbr cries | 15:07 |
superdan | lifeless: I've tagged nothing | 15:07 |
lifeless | superdan: you==theproject | 15:07 |
*** dprince has quit IRC | 15:07 | |
superdan | heh, okay | 15:07 |
mriedem | superdan: py27 is fine for me too on juno | 15:07 |
lifeless | superdan: tl;dr open setup.cfg, bump the version, put that commit at the bottom of your stack | 15:07 |
mriedem | superdan: git tag | grep 2014.2. | 15:08 |
superdan | lifeless: well, looks like I can run on actual stable/juno, just not on my patches on top | 15:08 |
superdan | mriedem: yeah, I have those tags | 15:08 |
openstackgerrit | Daniel Berrange proposed openstack/nova: tox: make it possible to run pep8 on current patch only https://review.openstack.org/205575 | 15:08 |
mriedem | so 2014.2.3 is latest tag, | 15:08 |
lifeless | superdan: setup.cfg in my stable/juno says version = 2014.2.4 | 15:08 |
lifeless | superdan: what does yours? | 15:08 |
mriedem | 2014.2.4 is in setup.cfg | 15:09 |
mriedem | it all looks correct to me | 15:09 |
danpb | ^^^ since people don't like run_tests.sh, here's a hack to make 'tox -epep8 -- -HEAD' work as a way to only check the current git patch | 15:09 |
superdan | ah, I get it now | 15:09 |
*** derekh has quit IRC | 15:10 | |
superdan | I was rebasing on stable/juno not origin/stable/juno (like an idiot) which was a stale local branch | 15:10 |
lifeless | danpb: 12K files? wtf? | 15:10 |
superdan | sorry for the noise | 15:10 |
mriedem | ah | 15:10 |
mriedem | rookie mistake :) | 15:10 |
superdan | clearly should have taken vacation today to recoup | 15:11 |
*** cfriesen__ has quit IRC | 15:11 | |
*** nelsnels_ has joined #openstack-nova | 15:11 | |
mriedem | yes, i definitely don't feel like working today | 15:11 |
rlrossit | superdan: no you shouldn't've | 15:11 |
rlrossit | I need you | 15:11 |
*** piyanai has quit IRC | 15:11 | |
mriedem | but now http://logs.openstack.org/36/193236/10/check/gate-tempest-dsvm-full-ceph/5549dfc/logs/screen-n-cpu.txt.gz?level=TRACE will be my white whale | 15:11 |
*** piyanai has joined #openstack-nova | 15:11 | |
superdan | rlrossit: heh | 15:11 |
rlrossit | this one is my white whale: http://logs.openstack.org/30/198730/11/check/gate-grenade-dsvm-partial-ncpu/db6b045/logs/new/screen-n-cond.txt.gz?level=INFO | 15:12 |
*** dave-mccowan has joined #openstack-nova | 15:12 | |
bauwser | danpb: coolness | 15:12 |
rlrossit | superdan: I tried putting tons of logging in the IncompatibleObjectVersion except, and it turns out we never get there... https://review.openstack.org/#/c/198730/11/nova/objects/base.py,cm | 15:13 |
danpb | lifeless: doh. next time when I'm counting the number of .py files in a git tree, i'll remember to exclude those under .tox | 15:13 |
*** karimb has quit IRC | 15:13 | |
lifeless | danpb: so I think there is something fundamentally wrong with flake8 if its looking in the .tox | 15:13 |
lifeless | danpb: like I mean, sure, I like the fast pep8 hack | 15:14 |
superdan | rlrossit: okay let me finish this current thing and I can look | 15:14 |
lifeless | danpb: but shouldn't it be consulting only .py files in nova *anyway* ? | 15:14 |
rlrossit | superdan: coolness. thanks | 15:14 |
*** nelsnelson has quit IRC | 15:14 | |
danpb | lifeless: yeah, i'm puzzelled how/why it decides to read tox.ini, but it surely does | 15:15 |
danpb | $ strace -e trace=open flake8 2>&1 | grep tox.ini | 15:15 |
danpb | open("/home/berrange/src/cloud/nova/tox.ini", O_RDONLY) = 3 | 15:15 |
*** dannywilson has joined #openstack-nova | 15:15 | |
lifeless | danpb: it definitely does | 15:15 |
lifeless | wheres' sigmavirus | 15:15 |
*** angdraug has joined #openstack-nova | 15:16 | |
*** sigmavirus24 has joined #openstack-nova | 15:16 | |
sigmavirus24 | ping lifeless | 15:16 |
lifeless | sigmavirus24: hey so, danpb is finding flake8 awesomely slow | 15:16 |
lifeless | sigmavirus24: but would really like it to be super fast during rebase -x tox -epep8 | 15:16 |
lifeless | sigmavirus24: he's put up a patch to git diff and run flake8 --diff, but it set of a question... how slow/fast should flake8 be? | 15:17 |
openstackgerrit | Daniel Berrange proposed openstack/nova: tox: make it possible to run pep8 on current patch only https://review.openstack.org/205575 | 15:17 |
lifeless | sigmavirus24: and: is it looking at too many files at the moment | 15:17 |
danpb | sigmavirus24: i have this hack ^^^ which makes it only check files in the current git HEAD commit | 15:17 |
sigmavirus24 | Didn't jogo already add that to run_tests.sh? | 15:17 |
*** rdopiera has quit IRC | 15:17 | |
lifeless | sigmavirus24: yeah, there's a crazypants side story here :) | 15:18 |
danpb | sigmavirus24: yes, nova has run_tests.sh -8, but people generally want run_tests.sh to DIAF and everyone to use tox | 15:18 |
*** dannywilson has quit IRC | 15:18 | |
sigmavirus24 | ahah | 15:18 |
*** dannywilson has joined #openstack-nova | 15:18 | |
danpb | so my patch just does the equivalent hack via tox | 15:18 |
lifeless | *I* want the unit test suite to check flake8 from inside itself | 15:18 |
lifeless | so I never have to remember to run flake8 | 15:18 |
lifeless | but thats me. | 15:18 |
sigmavirus24 | lifeless: there's prior-art-ish things like that (setup.py command, mercurial integration, etc.) | 15:19 |
sigmavirus24 | We don't document our internals but they're pretty stable | 15:19 |
sigmavirus24 | Anyway | 15:19 |
lifeless | sigmavirus24: yeah - I've done such linting-as-test stuff before, but thats a separate discussion | 15:19 |
*** piyanai has quit IRC | 15:19 | |
sigmavirus24 | Jogo had talked about upstreaming the stuff in run_tests.sh to Flake8 | 15:19 |
bauwser | superdan: danpb: you both already +2/+W https://review.openstack.org/#/c/197911/8 Could one of you just +W again it ? (was for a trivial rebase) | 15:19 |
*** READ10 has joined #openstack-nova | 15:20 | |
*** piyanai has joined #openstack-nova | 15:20 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Revert "Add error message to failed block device transform" https://review.openstack.org/205583 | 15:20 |
mriedem | superdan: ^ | 15:20 |
mriedem | do'nt know what ndipanov is | 15:20 |
*** mtanino has joined #openstack-nova | 15:20 | |
*** piyanai has quit IRC | 15:20 | |
mriedem | danpb: &^ | 15:20 |
danpb | sigmavirus24: it'd be super neat if flake8 natively a mode where you could just give it a git commit range | 15:20 |
mriedem | s/what/where/ | 15:20 |
*** thorst has quit IRC | 15:20 | |
danpb | eg flake8 --commit master.. and it'd check each commit in that series individually :-) | 15:20 |
*** nelsnels_ has quit IRC | 15:21 | |
sigmavirus24 | That'd be an easy plugin to write too | 15:21 |
lifeless | real 1m22.711s | 15:21 |
lifeless | user 3m20.897s | 15:21 |
sigmavirus24 | or *should* be easy | 15:21 |
lifeless | its running against .update-venv | 15:21 |
sigmavirus24 | Are y'all taking advantage of the --jobs flag? | 15:21 |
lifeless | which is kinda weird | 15:22 |
* sigmavirus24 forgets what version of flake8 hacking currently uses | 15:22 | |
sigmavirus24 | lifeless: we have a list of default excludes (e.g., .tox, .git) that we ignore but .update-venv is not there because it's not common | 15:22 |
*** piyanai has joined #openstack-nova | 15:23 | |
lifeless | I don't even know how that venv appeared | 15:24 |
*** shihanzhang has joined #openstack-nova | 15:24 | |
superdan | mriedem: where is that being called from and skipped? | 15:24 |
*** davideagnello has quit IRC | 15:25 | |
mriedem | _convert_block_devices | 15:25 |
mriedem | it's a filter | 15:25 |
*** y_sawai_ has quit IRC | 15:25 | |
*** y_sawai has joined #openstack-nova | 15:25 | |
mriedem | commented in the code where it's passed | 15:25 |
*** y_sawai has quit IRC | 15:25 | |
*** piyanai has quit IRC | 15:26 | |
mriedem | there are about 3600 errors in the compute logs since that merged on 7/19 | 15:26 |
*** shz has quit IRC | 15:27 | |
*** krtaylor has joined #openstack-nova | 15:28 | |
*** ddaskal has joined #openstack-nova | 15:28 | |
superdan | mriedem: I see that it is, but it's not clear to me if that could ever be raised in another situation | 15:28 |
superdan | like outside that filtering loop | 15:28 |
superdan | but it's a private exception, so I guess if it is it's broken anyway | 15:28 |
*** clopes has quit IRC | 15:29 | |
*** piyanai has joined #openstack-nova | 15:29 | |
mriedem | i only see _InvalidType used in _transform | 15:29 |
mriedem | and _InvalidType is _NotTransformable, which is only handled in _convert_block_devices | 15:29 |
mriedem | for filtering | 15:29 |
mriedem | but yeah, if that shit gets out of that module it's busted anyway | 15:30 |
mriedem | http://goo.gl/6XJNJi if that helps | 15:30 |
*** otter768 has joined #openstack-nova | 15:31 | |
superdan | I was just grepping to see if it was caught anywhere else | 15:31 |
superdan | which doesn't really mean we're not just depending on it breaking out and not catching the thing, but yeah | 15:31 |
superdan | mriedem: was able to run tests on that backport of mine and verify that I could revert that instance object line | 15:32 |
superdan | so hopefully that is good now | 15:32 |
danpb | superdan: btw, was there any particular historical reason why ovo.fields.UUID doesn't do UUID validation in coerce() ? | 15:32 |
mriedem | superdan: yeah the actual module code is doing instance['uuid' | 15:32 |
* danpb was wondering if we'd risk problems with existing database content by adding uuid validation now | 15:33 | |
mriedem | tests | 15:33 |
sgordon | dumb question, who is responsible for moving the blueprints.launchpad.net entry to approved (specifically referring to https://review.openstack.org/#/c/202647/ and related spec https://review.openstack.org/#/c/194221/ ) | 15:33 |
mriedem | there are a gabillion tests with uuid='fake' | 15:33 |
superdan | danpb: yes, exactly what that review surmised... the number of unit tests that broke was large and we never went back to convert them | 15:33 |
mriedem | and stuff like that | 15:33 |
superdan | danpb: that's why my todo is in there | 15:33 |
*** nelsnelson has joined #openstack-nova | 15:33 | |
danpb | superdan: so, mostly about stupidly written nova unit tests, rather than bad end user data ? | 15:33 |
mriedem | sgordon: where is the spec? | 15:33 |
sgordon | actually nm i guess it's irrelevant now | 15:33 |
mriedem | oh heh | 15:33 |
superdan | danpb: s/mostly/completely/ | 15:34 |
danpb | ok, cool, that's something we can tackle then | 15:34 |
sgordon | mriedem, https://review.openstack.org/#/c/194221/ but i will agree that was non-obvious | 15:34 |
sgordon | mriedem, afaict it's not linked directly from the blueprint | 15:34 |
sgordon | had to go digging | 15:34 |
* danpb likes it when things aren't as bad as first feared | 15:34 | |
mriedem | sgordon: it is via the "Read the full specification" link in the blueprint | 15:34 |
*** lpetrut1 has quit IRC | 15:34 | |
sgordon | ahh | 15:34 |
*** otter768 has quit IRC | 15:35 | |
sigmavirus24 | lifeless: fwiw, there are py.test plugins to run pep8/flake8 as part of your test-suite :trollface.jpg: | 15:35 |
danpb | its annoying that the comments added to launchpad blueprints don't mention which source repo the commit is against | 15:35 |
danpb | as it makes it hard to distinguish spec commits from code commits | 15:36 |
mriedem | sgordon: "July 16: non-priority feature proposal freeze" | 15:36 |
mriedem | sgordon: that code all missed it by a day | 15:36 |
*** MarcF has quit IRC | 15:36 | |
sgordon | mriedem, yeah no i realized post asking the question was academic | 15:36 |
mriedem | ok | 15:36 |
lifeless | sigmavirus24: is that a challenge? | 15:36 |
sigmavirus24 | lifeless: more of a "we should just use py.test" joke | 15:36 |
lifeless | sigmavirus24: ah :). See yeah, no. | 15:37 |
sigmavirus24 | lol | 15:37 |
*** READ10 has quit IRC | 15:37 | |
*** thangp has quit IRC | 15:37 | |
*** artom has quit IRC | 15:38 | |
*** thangp has joined #openstack-nova | 15:38 | |
*** piyanai has quit IRC | 15:38 | |
*** piyanai has joined #openstack-nova | 15:41 | |
jcook | mriedem: I think nikhil_k can answer that question better than I. Unfortunately, I don't have enough cycles to spend a significant time with Glance or Nova ... but that's another story and I'm working on it | 15:41 |
mriedem | jcook: was there a question? | 15:42 |
jcook | mriedem: hmmm...I think I'm responding to a comment you posted yesterday | 15:42 |
mriedem | ha | 15:42 |
mriedem | yeah | 15:42 |
*** y_sawai has joined #openstack-nova | 15:42 | |
jcook | mriedem: lol | 15:42 |
jcook | mriedem: my bouncer had a highlight and my screen session was scrolled up.... | 15:43 |
jcook | nvm | 15:43 |
*** david-lyle has quit IRC | 15:44 | |
*** y_sawai has quit IRC | 15:47 | |
*** ayoung has quit IRC | 15:48 | |
*** earlephilhower has quit IRC | 15:49 | |
*** earlephilhower has joined #openstack-nova | 15:49 | |
*** alex_klimov has quit IRC | 15:52 | |
*** jlanoux has quit IRC | 15:53 | |
superdan | rlrossit: ah, so you're confused that you're not getting the logs that you're adding in the serializer, right? | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module https://review.openstack.org/197311 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir https://review.openstack.org/193223 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module https://review.openstack.org/197341 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module https://review.openstack.org/196859 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module https://review.openstack.org/193249 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move lvm/dmcrypt/rbd_utils modules under storage subdir https://review.openstack.org/193195 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module https://review.openstack.org/196276 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module https://review.openstack.org/193236 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module https://review.openstack.org/193278 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module https://review.openstack.org/193380 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 15:53 |
rlrossit | superdan: well, I'm getting some, but not the ones in the except, so I guess incompatible version isn't being raised | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move fibre channel specific utilities into fc module https://review.openstack.org/193317 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module https://review.openstack.org/193240 | 15:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202781 | 15:53 |
rlrossit | good grief mriedem | 15:53 |
*** sahid has quit IRC | 15:54 | |
mriedem | sorry charlie brown | 15:54 |
rlrossit | superdan: but it basically looks like the serializer is processing 1.16, then says obj_make_compatible for 1.19, then process 1.16, then obj_make_compatible of 1.16, and then we go forever and ever | 15:55 |
superdan | rlrossit: the reason is that the ones you're expecting to see are happening on the old code | 15:55 |
superdan | rlrossit: and you're not changing the old code | 15:55 |
* rlrossit is confused | 15:55 | |
superdan | rlrossit: so that partial job... | 15:55 |
superdan | it runs devstack at kilo | 15:55 |
superdan | then it shuts down everything except nova-compute, and upgrades to master | 15:56 |
*** vilobhmm has joined #openstack-nova | 15:56 | |
superdan | so you end up with code from kilo in nova-compute, but code from master everywhere else (like conductor) | 15:56 |
superdan | so the thing that would be logging then "uh, I got a new object I don't understand" is not running the code in your patch | 15:56 |
*** bitblt has joined #openstack-nova | 15:56 | |
mriedem | pixelbeat_: want to get these again? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/juno+topic:bug/1438331,n,z | 15:57 |
rlrossit | superdan: oh so that's raised on the old compute (where there is no log) so it isn't showing up, but the stuff I did add that's running on conductor is showing up? | 15:57 |
superdan | rlrossit: yep | 15:57 |
rlrossit | debugging is hard | 15:57 |
superdan | heh, and this is complicated for sure | 15:57 |
superdan | rlrossit: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L477-478 | 15:58 |
superdan | rlrossit: put some debug there | 15:58 |
superdan | dump the whole objinst and target_version | 15:59 |
superdan | which might give you some more detail | 15:59 |
*** annegentle has quit IRC | 15:59 | |
superdan | without having to get logs to run on the older code | 15:59 |
*** annegentle has joined #openstack-nova | 16:00 | |
pixelbeat_ | mriedem, I'd be inclined to merge those two. But they look good. | 16:00 |
*** suresh12 has joined #openstack-nova | 16:01 | |
rlrossit | superdan: any other suggestions for adding logs? it's also frustrating that I put it up and have to wait like 1.5 hours before I can get my logs | 16:01 |
superdan | rlrossit: you can run this locally | 16:01 |
superdan | rlrossit: either by running grenade itself, or just doing the devstack thing yourself | 16:02 |
superdan | rlrossit: I also think that sdague said that you could depends-on a stable patch (which I didn't think worked) | 16:02 |
superdan | so you could try adding the serializer debug to stable in front of your patch to master | 16:03 |
*** sajeesh has joined #openstack-nova | 16:03 | |
rlrossit | ugh this is not friday work... | 16:04 |
superdan | heh | 16:04 |
*** ctrath has joined #openstack-nova | 16:04 | |
*** marzif has quit IRC | 16:05 | |
lifeless | superdan: depends-on a stable patch will bring that patch in in the stable part of grenade, due to its ZUUL_BRANCH | 16:05 |
*** annegentle has quit IRC | 16:05 | |
lifeless | superdan: but I think we only added that for kilo, so if poking at stable/juno it won't | 16:05 |
superdan | lifeless: ah, okay, this is kilo (different thing from earlier) | 16:05 |
*** marzif has joined #openstack-nova | 16:06 | |
superdan | rlrossit: so I'd get that in front to make the next 1.5 hour wait more fruitful | 16:06 |
*** kiran-r has joined #openstack-nova | 16:06 | |
rlrossit | superdan: so yank out the incompatibleobject exception logging and put that in a patch that I propose to stable/kilo, and then add to the commit message for my real patch a depends on <stable/kilo change id>? | 16:07 |
superdan | rlrossit: aye | 16:07 |
superdan | rlrossit: make sure that's a different change-id from your current one | 16:07 |
superdan | i.e. not a normal backport | 16:07 |
superdan | else the dependency linkage won't work | 16:07 |
rlrossit | I was planning on just straight up proposing something to stable/kilo | 16:07 |
rlrossit | not even backport or anything | 16:07 |
superdan | yep | 16:10 |
*** jwcroppe has quit IRC | 16:11 | |
*** apuimedo|dinner is now known as apuimedo|away | 16:13 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling https://review.openstack.org/202652 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field https://review.openstack.org/202653 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values https://review.openstack.org/202654 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling https://review.openstack.org/202655 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter' https://review.openstack.org/202650 | 16:16 |
openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling https://review.openstack.org/202651 | 16:16 |
*** SourabhP has joined #openstack-nova | 16:17 | |
*** lpetrut has joined #openstack-nova | 16:17 | |
*** jecarey has quit IRC | 16:19 | |
*** tdurakov has quit IRC | 16:19 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: Change List objects to use obj_relationships https://review.openstack.org/198730 | 16:19 |
rlrossit | superdan: I hope I did ^ right | 16:19 |
rlrossit | because I'm going to spend my waiting time eating lunch | 16:20 |
*** unicell1 has quit IRC | 16:20 | |
*** david-lyle has joined #openstack-nova | 16:20 | |
jogo | sigmavirus24: I never got around to pushing the patch up, but I there is a bug open on it with a git command option that works well | 16:21 |
superdan | rlrossit: looks right to me | 16:21 |
sigmavirus24 | jogo: I think there is a bug open for it | 16:22 |
sigmavirus24 | *I think you're right that there is a bug open for it | 16:22 |
sigmavirus24 | (That's what I get for starting to write a response, switching windows, and finishing it) | 16:22 |
sigmavirus24 | If anyone cares to help out with that: https://gitlab.com/pycqa/flake8/issues/12 | 16:23 |
jbernard | mriedem: i have not | 16:23 |
jbernard | mriedem: no idea what's going on there | 16:23 |
jogo | https://gitlab.com/pycqa/flake8/issues/12 | 16:23 |
*** nic has joined #openstack-nova | 16:24 | |
*** nelsnelson has quit IRC | 16:24 | |
*** nelsnelson has joined #openstack-nova | 16:24 | |
jogo | sigmavirus24: while not as clean the git trick works nicely | 16:25 |
*** dprince has joined #openstack-nova | 16:27 | |
*** piyanai has quit IRC | 16:28 | |
*** lpetrut has quit IRC | 16:29 | |
*** yamahata has quit IRC | 16:31 | |
*** shashank_hegde has joined #openstack-nova | 16:32 | |
*** lpetrut has joined #openstack-nova | 16:32 | |
*** lpetrut has quit IRC | 16:35 | |
*** browne has quit IRC | 16:37 | |
*** mwagner_lap has quit IRC | 16:37 | |
*** piyanai has joined #openstack-nova | 16:37 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 16:39 |
*** tjones1 has joined #openstack-nova | 16:39 | |
*** sigmavirus24 has left #openstack-nova | 16:40 | |
*** vilobhmm has quit IRC | 16:41 | |
*** suresh12 has quit IRC | 16:44 | |
*** suresh12 has joined #openstack-nova | 16:45 | |
*** mjturek1 has quit IRC | 16:46 | |
*** mjturek1 has joined #openstack-nova | 16:47 | |
*** mjturek1 has quit IRC | 16:47 | |
*** unicell has joined #openstack-nova | 16:47 | |
*** mjturek1 has joined #openstack-nova | 16:47 | |
*** tjones1 has quit IRC | 16:48 | |
*** haomaiwang has joined #openstack-nova | 16:48 | |
*** kmARC_ has quit IRC | 16:49 | |
*** suresh12 has quit IRC | 16:49 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image https://review.openstack.org/187857 | 16:50 |
*** kmARC has quit IRC | 16:50 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 16:50 |
jaypipes | johnthetubaguy: good news. I made a lot of progress on the resource-objects work on the plane from minneapolis last night. should have a few patches to push up shortly. | 16:52 |
*** kmARC_ has joined #openstack-nova | 16:52 | |
*** kmARC has joined #openstack-nova | 16:53 | |
*** annegentle has joined #openstack-nova | 16:54 | |
*** ayoung has joined #openstack-nova | 16:55 | |
openstackgerrit | Ben Nemec proposed openstack/nova: Make ironic ephemeral bdm match default value https://review.openstack.org/205271 | 16:55 |
*** davideagnello has joined #openstack-nova | 16:55 | |
*** annegent_ has joined #openstack-nova | 16:55 | |
*** piyanai has quit IRC | 16:57 | |
*** kiran-r has quit IRC | 16:58 | |
*** annegentle has quit IRC | 16:59 | |
*** piyanai has joined #openstack-nova | 17:00 | |
*** tonytan4ever has quit IRC | 17:00 | |
*** lucas__ has joined #openstack-nova | 17:01 | |
danpb | it would be awesome if some nova core would slam in the first half of this series of mriedem https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/consolidate-libvirt-fs-volume-drivers,n,z | 17:01 |
danpb | as it is the kind of series that is total hell to rebase against other changes touching the same area | 17:02 |
danpb | first patch in the series is https://review.openstack.org/#/c/193195/10 | 17:02 |
*** lucas____ has joined #openstack-nova | 17:03 | |
*** lucas__ has quit IRC | 17:03 | |
*** ajayaa has quit IRC | 17:04 | |
*** dims is now known as dimsum__ | 17:04 | |
*** bnemec has joined #openstack-nova | 17:04 | |
*** annashen has joined #openstack-nova | 17:05 | |
dane-fichter | nagyz_ : any luck with your connection_info issue? | 17:05 |
*** diogogmt has quit IRC | 17:05 | |
*** diogogmt has joined #openstack-nova | 17:06 | |
*** vilobhmm has joined #openstack-nova | 17:07 | |
*** artom has joined #openstack-nova | 17:07 | |
*** artom has quit IRC | 17:07 | |
*** artom has joined #openstack-nova | 17:07 | |
danpb | jaypipes: perhaps you'd be able to look at that series of mriedem to get some of it flowing in the gate ^^ | 17:08 |
*** ddaskal has quit IRC | 17:10 | |
*** ftersin has quit IRC | 17:11 | |
*** artom has quit IRC | 17:12 | |
*** piyanai has quit IRC | 17:12 | |
*** whenry has joined #openstack-nova | 17:13 | |
*** artom has joined #openstack-nova | 17:15 | |
jaypipes | danpb: absolutely. will do that shortly | 17:15 |
*** suresh12 has joined #openstack-nova | 17:16 | |
*** romainh has quit IRC | 17:19 | |
*** piyanai has joined #openstack-nova | 17:19 | |
*** piyanai has quit IRC | 17:20 | |
*** subscope has quit IRC | 17:20 | |
*** ijuwang has quit IRC | 17:21 | |
*** ssurana has joined #openstack-nova | 17:21 | |
*** piyanai has joined #openstack-nova | 17:23 | |
*** kmARC_ has quit IRC | 17:27 | |
*** kmARC has quit IRC | 17:27 | |
*** achanda has joined #openstack-nova | 17:27 | |
*** armax has quit IRC | 17:29 | |
*** browne has joined #openstack-nova | 17:30 | |
mriedem | jbernard: it wasn't a ceph thing | 17:30 |
jbernard | mriedem: yeah, it looks like that's expected to fail | 17:30 |
mriedem | jbernard: yup https://review.openstack.org/#/c/205583/ | 17:31 |
jbernard | mriedem: we might want to change or remove it i suppose | 17:31 |
jbernard | mriedem: man you're fast | 17:31 |
*** otter768 has joined #openstack-nova | 17:31 | |
*** armax has joined #openstack-nova | 17:33 | |
*** piyanai has quit IRC | 17:35 | |
*** Sukhdev has joined #openstack-nova | 17:35 | |
*** SourabhP has quit IRC | 17:36 | |
*** otter768 has quit IRC | 17:36 | |
*** armax has quit IRC | 17:38 | |
*** lpetrut has joined #openstack-nova | 17:38 | |
*** vilobhmm has quit IRC | 17:39 | |
*** vilobhmm has joined #openstack-nova | 17:39 | |
*** armax has joined #openstack-nova | 17:39 | |
*** tjones1 has joined #openstack-nova | 17:40 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 17:42 |
*** davideagnello has quit IRC | 17:42 | |
*** piyanai has joined #openstack-nova | 17:43 | |
*** angdraug has quit IRC | 17:43 | |
*** mpaolino has quit IRC | 17:43 | |
*** mjturek1 has quit IRC | 17:46 | |
*** mriedem has left #openstack-nova | 17:46 | |
*** mriedem has joined #openstack-nova | 17:47 | |
*** lpetrut has quit IRC | 17:48 | |
*** danpb has quit IRC | 17:49 | |
*** mjturek1 has joined #openstack-nova | 17:52 | |
*** fawadkhaliq has joined #openstack-nova | 17:53 | |
*** garyk has joined #openstack-nova | 17:53 | |
*** mutoulbj has quit IRC | 17:53 | |
*** lpetrut has joined #openstack-nova | 17:55 | |
*** marzif has quit IRC | 17:58 | |
*** belmoreira has joined #openstack-nova | 17:58 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module https://review.openstack.org/202301 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module https://review.openstack.org/197311 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir https://review.openstack.org/193223 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module https://review.openstack.org/197341 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module https://review.openstack.org/196859 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module https://review.openstack.org/193249 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move lvm/dmcrypt/rbd_utils modules under storage subdir https://review.openstack.org/193195 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module https://review.openstack.org/196276 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module https://review.openstack.org/193236 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202779 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202778 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module https://review.openstack.org/193278 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module https://review.openstack.org/193380 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202777 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202783 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move fibre channel specific utilities into fc module https://review.openstack.org/193317 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver https://review.openstack.org/202782 | 17:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module https://review.openstack.org/193240 | 17:58 |
*** rotbeard has quit IRC | 18:00 | |
*** piyanai has quit IRC | 18:04 | |
*** belmoreira has quit IRC | 18:06 | |
*** lyanchih has quit IRC | 18:09 | |
*** tonytan4ever has joined #openstack-nova | 18:10 | |
*** ijw has joined #openstack-nova | 18:12 | |
*** kmARC has joined #openstack-nova | 18:12 | |
sajeesh | ping edleafe | 18:13 |
*** jistr has quit IRC | 18:14 | |
ctrath | mriedem: I'm getting an error in py34 from jenkins… It's because I'm using the unicode() function. so, I have a couple questions | 18:14 |
mriedem | ctrath: use six.text_type | 18:14 |
mriedem | https://pythonhosted.org/six/ | 18:15 |
ctrath | ok.. cool. how do I run just py34 in jenkins in a recheck? | 18:15 |
ctrath | next, should I be testing py34 in tox locally? | 18:15 |
mriedem | ctrath: you don't | 18:15 |
mriedem | ctrath: i don't | 18:15 |
mriedem | i have py34 locally but tox had problems building the venv with some local deps | 18:15 |
mikal | . | 18:15 |
*** davideagnello has joined #openstack-nova | 18:16 | |
mriedem | mikal: what?! | 18:16 |
*** yamahata has joined #openstack-nova | 18:16 | |
mriedem | mikal: get back ok? | 18:16 |
mikal | mriedem: nah, I'm hanging out in LA today | 18:16 |
ctrath | mriedem: thanks for the info | 18:16 |
mikal | mriedem: I don't do the long haul back until Sunday | 18:16 |
*** annashen has quit IRC | 18:19 | |
*** thorst1 has joined #openstack-nova | 18:20 | |
rlrossit | superdan: sdague: is there a bug/blueprint/something for that log trace thing? I'll look at that in my spare time and I wasn't sure if this was written down somewhere | 18:21 |
*** mgoddard has left #openstack-nova | 18:21 | |
clarkb | ctrath: you should run tox -epy34 locally and if tha tpasses push a new patchset up to gerrit which will run the tests again there | 18:21 |
superdan | rlrossit: I don't really know.. I think sdague is out until monday, but we should ask him then | 18:21 |
superdan | unless mriedem knows | 18:21 |
mriedem | rlrossit: ask dims | 18:22 |
*** annashen has joined #openstack-nova | 18:22 | |
ctrath | clarkb: thanks, I'll give it a shot | 18:22 |
clarkb | ctrath: we don't allow for running individual tests one at a time like that to avoid locking in tests and then merging code that doesn't actually work | 18:22 |
ctrath | ok | 18:22 |
clarkb | (the py34 fix could break py27 and if we locked in the py27 results we wouldn't know until the tests started failing for everyone) | 18:22 |
mriedem | http://www.dailymotion.com/video/xc2hjy_lucy-lawless_shortfilms | 18:24 |
mriedem | stevie nicks fajita roundup | 18:24 |
superdan | mriedem: lol | 18:25 |
mriedem | ever seen that? | 18:25 |
mriedem | i got some random fleetwood mac and it always reminds me of that skit | 18:26 |
*** lpetrut has quit IRC | 18:26 | |
rlrossit | superdan: here's the funness: http://logs.openstack.org/30/198730/12/check/gate-grenade-dsvm-partial-ncpu/4349a21/logs/old/screen-n-cpu.txt.gz?level=INFO#_2015-07-24_18_20_07_225 | 18:27 |
*** Sukhdev has quit IRC | 18:27 | |
superdan | rlrossit: nice | 18:28 |
*** Sukhdev has joined #openstack-nova | 18:28 | |
superdan | rlrossit: see that the instance object we're actually getting is Instance 1.21, but we think it should be 1.16? | 18:29 |
superdan | oh wait | 18:29 |
rlrossit | superdan: seeing the same thing... trying to analyze it right now | 18:29 |
rlrossit | 1.16 is instancelist | 18:29 |
rlrossit | I think it sees instance 1.21 in a 1.19 instancelist, then skips the instance part and tries to go to instancelist 1.16 | 18:29 |
*** ddaskal has joined #openstack-nova | 18:30 | |
ctrath | mriedem: yeah, tox -e py34 is blowing chunks on me whilst installing dependencies | 18:30 |
mriedem | ctrath: yeah, that's what happened to me | 18:30 |
mriedem | so i gave up | 18:30 |
superdan | rlrossit: yeah, something | 18:30 |
rlrossit | superdan: wait... the spots where it dumps the object it got makes no sense... | 18:30 |
mriedem | ctrath: anyway, you can't use unicode(), use six.text_type | 18:30 |
superdan | rlrossit: but the fact that it's asking for instancelist 1.19 is probably the bug in the serializer | 18:31 |
rlrossit | the incompatible object is instance list, but the object data it's dumping is an Instance | 18:31 |
ctrath | yeah, just using a plain string now to insert into tags table.. It gives me a warning, but works | 18:31 |
mriedem | warning? | 18:31 |
*** moshele has quit IRC | 18:31 | |
rlrossit | fffffffffffffuuuuuuuuuuuuuuuuuuuuuuuu the temptation to just close the laptop right now is overwhelming | 18:32 |
superdan | rlrossit: https://review.openstack.org/#/c/202641/ | 18:32 |
superdan | rlrossit: probably need that in master and kilo | 18:32 |
ctrath | mriedem: "SAWarning: Unicode type received non-unicode bind param value 'tag1'. (this warning may be suppressed after 10 occurrences)" | 18:32 |
*** davideagnello has quit IRC | 18:32 | |
mriedem | ctrath: yeah, so use six.text_type | 18:32 |
mriedem | we'd like to avoid adding new warnings | 18:32 |
superdan | rlrossit: because what is happening is we're trying to deserialize an instancelist, and we fail on instance 1.21, so we report instance 1.19 up as the supported version (of instance) | 18:32 |
superdan | rlrossit: and then the serializer incorrectly asks conductor for InstanceList 1.19 | 18:33 |
superdan | rlrossit: instead of something like 1.13 or something | 18:33 |
mriedem | that's fixed in ovo 0.6.0 | 18:33 |
mriedem | so should be fine on master, unless it's capped | 18:34 |
ctrath | the tag tests in test_db_api must be producing warnings, as they are just setting the tag as a plain string | 18:34 |
mriedem | aha | 18:34 |
mriedem | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L192 | 18:34 |
superdan | mriedem: the master code isn't using that from o.vo yet | 18:34 |
mriedem | superdan: yeah, because of ^ | 18:34 |
*** jaypipes is now known as funkypipes | 18:34 | |
*** jistr has joined #openstack-nova | 18:35 | |
superdan | mriedem: well, it's not using that whole method from o.vo yet I mean | 18:35 |
superdan | mriedem: it still has its own implementation I think | 18:35 |
mriedem | superdan: ooh | 18:35 |
mriedem | gotcha | 18:35 |
*** piyanai has joined #openstack-nova | 18:35 | |
*** jistr is now known as jistr|afk | 18:35 | |
superdan | rlrossit: I know it's frustrating, but this is crazy broken important shit, so it's awesome that you're fixing it | 18:35 |
superdan | even if it is costing you hair and cupcakes | 18:35 |
phschwartz | Hi all, we found an issue with the change to nova cells to send the instance object up to the global cell. Has anyone seen any issues where this makes the global cell very chatty with compute.instance.update events? | 18:35 |
mriedem | rlrossit could copy that into nova, unless how hard is it to make that part of nova use ovo? | 18:36 |
funkypipes | anybody seen Radoslav Gerganov today? | 18:36 |
superdan | mriedem: I would just copy it in for the moment, yeah | 18:36 |
mriedem | phschwartz: i'd ask alaski but i think he's traveling back from the midcycle today | 18:36 |
mriedem | funkypipes: he was here earlier causing problems for me | 18:36 |
mriedem | superdan: is that also needed in kilo? | 18:36 |
phschwartz | mriedem: he was my first choice, but yes, he is traveling back and I have not been able to get in touch with melwitt who made the change. | 18:37 |
rlrossit | mriedem: well if this is broken in Kilo we wouldn't want to have nova use the o.vo version of it right? | 18:37 |
mriedem | phschwartz: melwitt is the next person i'd ask | 18:37 |
superdan | mriedem: it's actually only needed in kilo for his bug, yeah, but since it's in master now, I'd just fix it there so it's a clean backport | 18:37 |
mriedem | phschwartz: btw, where were you this week? | 18:37 |
superdan | mriedem: she's also traveling back today I think | 18:37 |
mriedem | gdi | 18:37 |
* superdan increments the counter | 18:37 | |
*** Marga_ has quit IRC | 18:38 | |
phschwartz | mriedem: I couldn't get a flight that worked with getting back yesterday so I could make an MRI that was scheduled for this morning. Had to miss it because of it. But I will be in Tokyo which is good. | 18:38 |
mriedem | well, phschwartz is rax and cells == rax so there has to be one other person in that gd company that knows cells besides alaski | 18:38 |
phschwartz | Yeah, johnthetubaguy who is on a flight back to the UK. lol | 18:38 |
mriedem | superdan: gd counts as .5 | 18:38 |
superdan | mriedem: should be .6667 | 18:38 |
phschwartz | I am trying to get versed in it, but slow going as I dig through source | 18:38 |
mriedem | superdan: touche | 18:38 |
mriedem | how about _cerberus_? | 18:38 |
mriedem | belliott: ? | 18:39 |
mriedem | the whole gang | 18:39 |
mriedem | superdan: so backing up on the backport - to clarify, what should happen with that ovo change? copy to nova master? or backport to nova stable/kilo? | 18:39 |
*** Mike_D_laptop has joined #openstack-nova | 18:39 | |
phschwartz | mriedem: belliott is out on time off today. | 18:40 |
mriedem | heh, well at this point it's a joke | 18:40 |
rlrossit | both 2 and 3 mriedem? | 18:40 |
superdan | mriedem: copy that change to nova master, then backport that to kilo | 18:40 |
superdan | rlrossit: yeah | 18:40 |
mriedem | rlrossit: ah | 18:40 |
mriedem | ok, yeah | 18:40 |
mriedem | rlrossit: are you doing that? | 18:40 |
rlrossit | I can | 18:41 |
mriedem | i mean, i know it's almost 2pm for you | 18:41 |
*** markus_z has joined #openstack-nova | 18:41 | |
mriedem | once you hit that 3 hour window of quitting time.... | 18:41 |
rlrossit | well I was actually planning on just taking the entire afternoon off | 18:42 |
ctrath | lol | 18:42 |
rlrossit | gotta clean my car out | 18:42 |
*** iamjarvo has joined #openstack-nova | 18:42 | |
mriedem | for what? | 18:42 |
mriedem | impress that mystery gal? | 18:42 |
rlrossit | It's dirty after going back to the cruddy state of SD | 18:42 |
*** rfolco has quit IRC | 18:42 | |
mriedem | riiiight | 18:42 |
ctrath | for the 'friend' that stayed until 3:30 the other morning | 18:42 |
rlrossit | plus it's full of sand from volleyball | 18:42 |
* rlrossit has a life | 18:43 | |
rlrossit | who'dathunkit | 18:43 |
mriedem | might be able to just cherry pick this out of ovo | 18:44 |
rlrossit | .....how? | 18:44 |
* mriedem tries | 18:44 | |
mriedem | magic | 18:44 |
rlrossit | git cherry-pic --with-magic | 18:44 |
rlrossit | s/cherry-pic/cherry-pick | 18:44 |
*** ijw has quit IRC | 18:46 | |
mriedem | cherry pick gets a bit wonky, so nvm | 18:47 |
*** ijw has joined #openstack-nova | 18:47 | |
mriedem | rlrossit: i can do the sync and backport if you're taking off and rebase your change on top of it | 18:47 |
rlrossit | I'll do it and then head out. probably check it tonight to see if I can rest easy this weekend | 18:48 |
rlrossit | I want to do the backport so I know how that works | 18:48 |
rlrossit | mriedem: superdan: so should I say https://bugs.launchpad.net/oslo.versionedobjects/+bug/1475356 affects nova also and then push patches up against that bug? or how should we play this? | 18:50 |
openstack | Launchpad bug 1475356 in oslo.versionedobjects "Serializer reports wrong supported version" [High,Fix released] | 18:50 |
mriedem | rlrossit: yes | 18:50 |
mriedem | the backport is going to be a bit gross | 18:53 |
mriedem | since NovaObjectRegistry didn't exist in kilo | 18:53 |
*** dane-fichter has quit IRC | 18:55 | |
*** achanda has quit IRC | 18:59 | |
*** lucas____ has quit IRC | 19:01 | |
*** lucas__ has joined #openstack-nova | 19:03 | |
rlrossit | superdan: in order to match this in nova, instead of mocking out the indirection api, I have to mock out the conductor api correct? | 19:04 |
rlrossit | https://review.openstack.org/#/c/202641/2/oslo_versionedobjects/tests/test_objects.py,cm (for reference) | 19:04 |
mriedem | rlrossit: i'm running unit tests on the sync now | 19:05 |
mriedem | from ovo | 19:05 |
mriedem | rlrossit: i thought you were leaving | 19:05 |
*** mdrabe has quit IRC | 19:05 | |
mriedem | rlrossit: but yes | 19:05 |
rlrossit | dangit mriedem I said I was going to do it so I could practice the backport | 19:05 |
mriedem | the backport won't be clean | 19:05 |
mriedem | so if you plan on working for the next few hours, then go ahead | 19:05 |
rlrossit | well, I guess you do them then and add me to them when you have them done | 19:06 |
*** thorst1 has quit IRC | 19:06 | |
mriedem | that was the plan | 19:07 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 19:10 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists https://review.openstack.org/203037 | 19:10 |
rlrossit | mriedem: you going to rebase my patch on that yourself also? | 19:14 |
mriedem | rlrossit: yeah | 19:14 |
*** romainh has joined #openstack-nova | 19:14 | |
rlrossit | aight I spose there's nothing for me to do then | 19:14 |
mriedem | just clean out your car and get ready for the weekend | 19:15 |
rlrossit | you can't tell me what to do mriedem | 19:16 |
rlrossit | but yes, I will do that | 19:16 |
*** lucas__ has quit IRC | 19:16 | |
mriedem | rlrossit: how about you rebase once i have this thing up | 19:17 |
mriedem | then we don't have to both maintain your series locally | 19:17 |
mriedem | just finishing up tox | 19:17 |
rlrossit | tick tock mriedem | 19:17 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Fix serializer supported version reporting in object_backport https://review.openstack.org/205683 | 19:18 |
mriedem | there ^ | 19:18 |
rlrossit | mriedem: I'm going to need the backport too though aren't I? | 19:18 |
*** EmilienM is now known as EmilienM|brb | 19:18 | |
rlrossit | and then depend on that so grenade doesn't splode? | 19:18 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Fix serializer supported version reporting in object_backport https://review.openstack.org/205683 | 19:19 |
mriedem | rlrossit: how about review that copy to nova first | 19:19 |
mriedem | i'll start the backport | 19:19 |
rlrossit | woah no one said anything about a review... | 19:19 |
*** tjones1 has quit IRC | 19:21 | |
*** fawadkhaliq has quit IRC | 19:21 | |
*** fawadkhaliq has joined #openstack-nova | 19:22 | |
*** bitblt has quit IRC | 19:22 | |
mriedem | honestly i didn't know that depeneds-on worked between branches | 19:22 |
mriedem | like master depends-on stable/kilo | 19:22 |
mriedem | i knew it worked between projects | 19:22 |
rlrossit | that's pretty neat | 19:22 |
rlrossit | and clever | 19:22 |
rlrossit | Is there way to do a side-by-side of 2 side-by-side diffs? | 19:23 |
rlrossit | diffception | 19:23 |
mriedem | i just open a new tab | 19:23 |
mriedem | one for the nova change, one for the original | 19:23 |
*** jungleboyj has joined #openstack-nova | 19:24 | |
jungleboyj | garyk: You around ... mriedem pointed me your way with a question about a volume attachment problem I am seeing with VMWare VMDKs. | 19:25 |
rlrossit | mriedem: just put in a comment for updating the commit message to reference the ovo change | 19:25 |
rlrossit | other than that LGTM | 19:25 |
mriedem | rlrossit: the change ids are the same | 19:25 |
*** lucas__ has joined #openstack-nova | 19:25 | |
rlrossit | oh crap they are? | 19:25 |
rlrossit | dang | 19:25 |
rlrossit | so was that a magic cherry pick? | 19:26 |
mriedem | no | 19:26 |
mriedem | https://review.openstack.org/#/q/I7ac03717e17cd7229abfeef6d9c7d9d8ec9be27a,n,z | 19:26 |
mriedem | you can use the same changeid between projects and branches | 19:26 |
rlrossit | huh. TIL | 19:26 |
mriedem | the cherry pick had merge conflicts on all of the generic stuff in ovo | 19:26 |
mriedem | so i just did a copy and modify | 19:26 |
mriedem | since the things in nova are hard-coded, like nova_object.name | 19:27 |
mriedem | nova_object is the namespace and hard-coded in nova, but generic in ovo | 19:27 |
rlrossit | yup | 19:27 |
rlrossit | and VersionedObjectRegistry -> NovaObjectRegistry | 19:27 |
mriedem | yup | 19:27 |
rlrossit | I like how I copied the change id, but never clicked on it... if I would've I wouldn't have looked stupid :P | 19:28 |
rlrossit | I'll get rebased on that and prepare the backport for depends-on | 19:29 |
mriedem | rlrossit: yeah i enjoyed that also | 19:30 |
*** otter768 has joined #openstack-nova | 19:32 | |
*** lucas__ has quit IRC | 19:33 | |
*** Mike_D_laptop has quit IRC | 19:34 | |
*** krtaylor has quit IRC | 19:34 | |
*** piyanai has quit IRC | 19:34 | |
*** lpetrut has joined #openstack-nova | 19:34 | |
*** lucas__ has joined #openstack-nova | 19:35 | |
*** kmARC has quit IRC | 19:36 | |
*** lucas__ has quit IRC | 19:36 | |
*** lucas__ has joined #openstack-nova | 19:36 | |
*** Mike_D_laptop has joined #openstack-nova | 19:37 | |
*** otter768 has quit IRC | 19:37 | |
*** piyanai has joined #openstack-nova | 19:38 | |
*** lucas__ has joined #openstack-nova | 19:40 | |
mriedem | rlrossit: you'll also have to rebase your change on this probably https://review.openstack.org/#/c/202124/ | 19:41 |
mriedem | i assume there is a merge conflict now | 19:41 |
rlrossit | mriedem: you have that in your change right? so by me rebasing on you I got it | 19:42 |
mriedem | no | 19:42 |
rlrossit | nevermind | 19:42 |
mriedem | not how it works | 19:42 |
rlrossit | it just went in | 19:42 |
mriedem | yeah | 19:42 |
mriedem | here is the kilo backport btw https://review.openstack.org/#/c/205686/ | 19:43 |
mriedem | superdan: ^ | 19:43 |
*** lucas__ has quit IRC | 19:43 | |
*** VW__ has joined #openstack-nova | 19:45 | |
*** Piet has joined #openstack-nova | 19:46 | |
rlrossit | mriedem: how do I rebase on master and yours and push it up without re-reviewing yours? | 19:47 |
rlrossit | or have I earned myself a git-mess? | 19:47 |
*** shashank_hegde has quit IRC | 19:47 | |
*** shashank_hegde has joined #openstack-nova | 19:48 | |
*** achanda has joined #openstack-nova | 19:48 | |
*** ijw__ has joined #openstack-nova | 19:48 | |
mriedem | rlrossit: there are probably other ways to do this, but i would | 19:48 |
mriedem | git checkout -B master origin/master | 19:48 |
mriedem | git pull origin | 19:48 |
*** VW_ has quit IRC | 19:48 | |
mriedem | git review -d 205683 | 19:49 |
mriedem | to get my master change | 19:49 |
mriedem | then git cherry-pick f2d888919e63dccb2db98156b2440cd6d686542d | 19:49 |
mriedem | to get https://review.openstack.org/#/c/198730/ | 19:49 |
mriedem | and git cherry-pick 97a3a0608333e11ff997b8ecc79713bbf9b3b5de to get https://review.openstack.org/#/c/198904/ | 19:49 |
mriedem | if you want that all back on your bug/1471978 topic branch, you'd need to delete it and recreate it | 19:50 |
mriedem | git branch -D bug/1471978 | 19:50 |
mriedem | then git checkout -b bug/1471978 | 19:50 |
mriedem | like i said, there are probably fancier ways to do it, but that's how i do | 19:50 |
mriedem | oh you know what's goin to be f'ed up though | 19:51 |
mriedem | you want to use the depends-on which is based on change id | 19:51 |
mriedem | but I7ac03717e17cd7229abfeef6d9c7d9d8ec9be27a is in 3 different commits | 19:51 |
*** ijw has quit IRC | 19:51 | |
mriedem | i don't think zuul is going to be able to figure out that you mean the stable/kilo version | 19:52 |
rlrossit | mriedem: I hope it'll be smart enough to pull the stable/kilo one | 19:52 |
mriedem | i very much doubt it | 19:52 |
mriedem | we could ask in infra | 19:52 |
rlrossit | I'll try | 19:52 |
mriedem | clarkb: how does depends-on work when you have the same change id in 2 repos and multiple branches (master and stable)? | 19:52 |
mriedem | jeblair: ^ | 19:52 |
superdan | I bet not | 19:52 |
*** achanda_ has joined #openstack-nova | 19:52 | |
superdan | I think if there are any two patches with the same change-id, it either does nothing or depends on all/both | 19:53 |
clarkb | it depends on all | 19:53 |
superdan | I forget what the answer was last time I asked | 19:53 |
mriedem | ah | 19:53 |
mriedem | depends on all is what we'd want, so cool | 19:53 |
superdan | okay, so that might actually work since the o.vo one is already merged? | 19:53 |
rlrossit | yeah | 19:53 |
mriedem | nice | 19:53 |
mriedem | rlrossit: in that case, you don't even need to rebase on top of mine | 19:53 |
mriedem | just need to rebase with master | 19:53 |
mriedem | depends-on takes care of hte rest | 19:53 |
*** achanda has quit IRC | 19:53 | |
rlrossit | eh I did anyways | 19:53 |
mriedem | heh | 19:53 |
mriedem | we'll break zuul yet | 19:54 |
rlrossit | I did the rebase on master, then rebased on yours, and swapped the picks in the rebase -i which I think made it all right | 19:54 |
rlrossit | or maybe not | 19:54 |
rlrossit | lord, what did I get myself into | 19:55 |
*** piyanai has quit IRC | 19:56 | |
*** VW__ has quit IRC | 19:56 | |
*** jasondotstar has quit IRC | 19:57 | |
*** VW_ has joined #openstack-nova | 19:57 | |
*** achanda_ has quit IRC | 19:59 | |
*** kmARC has joined #openstack-nova | 19:59 | |
*** EmilienM|brb is now known as EmilienM | 19:59 | |
*** piyanai has joined #openstack-nova | 19:59 | |
*** achanda has joined #openstack-nova | 19:59 | |
*** lucas__ has joined #openstack-nova | 20:00 | |
*** jogo has quit IRC | 20:01 | |
*** dannywil_ has joined #openstack-nova | 20:01 | |
*** artom has quit IRC | 20:01 | |
*** chuckcarmack has left #openstack-nova | 20:02 | |
*** dannywilson has quit IRC | 20:03 | |
*** atuvenie has joined #openstack-nova | 20:04 | |
*** lucas__ has quit IRC | 20:04 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: Change List objects to use obj_relationships https://review.openstack.org/198730 | 20:05 |
rlrossit | superdan: mriedem: I'm going to let zuul have at that, and I'm going home! | 20:05 |
mriedem | sounds good | 20:05 |
*** rlrossit has left #openstack-nova | 20:06 | |
* superdan waves late-ly | 20:06 | |
mriedem | ha | 20:06 |
mriedem | and his thing just hit a merge conflict somehow | 20:06 |
mriedem | b/c of https://review.openstack.org/#/c/205611/1/nova/objects/base.py | 20:07 |
mriedem | derp | 20:07 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Add rootwrap daemon mode support https://review.openstack.org/180695 | 20:07 |
*** lpetrut has quit IRC | 20:10 | |
funkypipes | mriedem, superdan: https://review.openstack.org/#/c/188789/ if you have a chance... | 20:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 20:10 |
*** achanda has quit IRC | 20:10 | |
*** annashen has quit IRC | 20:10 | |
openstackgerrit | Jay Pipes proposed openstack/nova: VMware: add support for cores per socket https://review.openstack.org/187942 | 20:11 |
*** mjturek1 has quit IRC | 20:12 | |
funkypipes | garyk: ^ was just a rebase of one of your ptches. | 20:12 |
funkypipes | garyk: reviewing it now... | 20:12 |
*** rushiagr is now known as rushiagr_away | 20:12 | |
*** krtaylor has joined #openstack-nova | 20:13 | |
*** achanda has joined #openstack-nova | 20:15 | |
mriedem | funkypipes: -1 | 20:16 |
*** kmARC has quit IRC | 20:16 | |
mriedem | i haven't reviewed all of the hardware module changes, since it hurts my brain and eyes and general head area | 20:16 |
*** lucas__ has joined #openstack-nova | 20:17 | |
funkypipes | mriedem: https://review.openstack.org/#/c/188119/ makes ImageMeta used throughout, 2 patches later in that series. | 20:18 |
*** tjones1 has joined #openstack-nova | 20:18 | |
Piet | We're looking for a few folks that have either moved to Neutron from Nova Networks or planning to stay with Nova Networks for an a 45 minute interview. Please reach-out to me if you're interested in participating! | 20:19 |
mriedem | funkypipes: still | 20:19 |
Piet | As always, the results of the interviews will be shared with the community. | 20:20 |
funkypipes | mriedem: and for the resource tracker, https://review.openstack.org/#/c/189685/ converts compute manager to use image meta. | 20:20 |
mriedem | funkypipes: we shouldn't land a thing knowing it's not right, regardless of something that could land a month later | 20:20 |
funkypipes | mriedem: it's all in the patch series eventually. | 20:20 |
mriedem | funkypipes: yeah, but it doesn't matter IMO | 20:20 |
funkypipes | why? would you prefer one giant patch? :( | 20:20 |
mriedem | no | 20:20 |
mriedem | i'd prefer compat shims that can be removed when the things are actually converted to objects | 20:20 |
mriedem | otherwise we get shitty bugs in the meantime because people don't know what to send in and make assumptions | 20:21 |
*** lucas__ has quit IRC | 20:21 | |
mriedem | plus, i hate image_meta.properties.get fwiw | 20:21 |
mriedem | but not sure how easy that is to change | 20:21 |
mriedem | s/hate/dislike/ | 20:22 |
*** annashen has joined #openstack-nova | 20:23 | |
*** Marga_ has joined #openstack-nova | 20:24 | |
funkypipes | superdan: what is your opinion on this? | 20:24 |
* superdan reads | 20:25 | |
superdan | mriedem: the .get was my suggestion | 20:26 |
superdan | mriedem: because the alternative was insanely terrible | 20:26 |
mriedem | as noted in the review, i wasn't sure how possible it was to make that better | 20:26 |
mriedem | like checking image_meta first, then image_meta.properties | 20:27 |
mriedem | i'd even like this better: props = image_meta.properties | 20:27 |
mriedem | props.get('thing', None) | 20:27 |
superdan | funkypipes: you want my opinon on the .get? | 20:27 |
mriedem | superdan: on the compat shims i think | 20:27 |
*** Sukhdev has quit IRC | 20:28 | |
superdan | okay, I'll take a look | 20:28 |
superdan | I have to go deal with a thing first | 20:28 |
*** davideagnello has joined #openstack-nova | 20:29 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field https://review.openstack.org/200280 | 20:30 |
*** thangp has quit IRC | 20:30 | |
superdan | mriedem: are you interpreting the .get as a shim to handle dict conversions? | 20:31 |
funkypipes | mriedem: I'll push up a change to that patch series adding what you requested. | 20:32 |
mriedem | superdan: no | 20:32 |
mriedem | i have 2 separate complaints on that change | 20:32 |
mriedem | one is easy, the other might not be | 20:32 |
*** vladikr has quit IRC | 20:32 | |
superdan | okay, I'm not sure I understand | 20:33 |
superdan | ohh, | 20:34 |
superdan | you want to not do from_dict() if we're passed an object | 20:34 |
mriedem | right | 20:34 |
mriedem | that's my -1 | 20:34 |
mriedem | the .get is general grumpiness | 20:34 |
superdan | so, the only real place we've done that in the past is rpc methods where we literally can't change both sides at the same time for compat-ness | 20:34 |
superdan | and/or some library that gets called from a bunch of places, some of which might be old | 20:35 |
mriedem | there are or were type checks in RT flows at other times | 20:35 |
mriedem | or the scheduler | 20:35 |
*** achanda has quit IRC | 20:35 | |
*** davideag_ has joined #openstack-nova | 20:35 | |
mriedem | i remember seeing those when reviewing the flavor migrate changes | 20:35 |
superdan | yeah, because those things were bits received from rpc | 20:35 |
superdan | and a few things like RT where we converted manager before RT | 20:35 |
*** davideagnello has quit IRC | 20:35 | |
superdan | these things are called from much smaller scope I think, so ... I mean, I think I'd be fine with it either way | 20:36 |
superdan | i.e. it's not important to me, but if it is to you, then that's fine | 20:36 |
mriedem | heh | 20:36 |
mriedem | it's not unimportant to me :) | 20:36 |
superdan | well, there you go :) | 20:36 |
mriedem | i just feel like we hit too many bugs with type error type crap like this | 20:37 |
mriedem | anyway, i'm going to fix up rlrossit's things and then take off for the weekend | 20:38 |
*** Mike_D_laptop has quit IRC | 20:39 | |
*** fawadk has joined #openstack-nova | 20:46 | |
*** fawadkhaliq has quit IRC | 20:49 | |
*** achanda has joined #openstack-nova | 20:52 | |
*** vishwanathj has joined #openstack-nova | 20:53 | |
*** salv-orlando has joined #openstack-nova | 20:53 | |
*** mriedem has quit IRC | 20:53 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 20:54 |
*** VW_ has quit IRC | 20:55 | |
*** VW_ has joined #openstack-nova | 20:55 | |
*** VW_ has quit IRC | 20:56 | |
*** VW_ has joined #openstack-nova | 20:56 | |
*** iamjarvo has quit IRC | 20:57 | |
*** salv-orlando has quit IRC | 20:58 | |
*** raildo has quit IRC | 20:58 | |
cfriesen | hi all...I'm seeing something really odd. I'm running a "nova resize" test from novaclient that should result in exception.CannotResizeDisk being raise in nova-api, causing a 400 return value. I put in a pdb breakpoint in the nova-api code, but novaclient is reporting a 202 return code even before the nova code actually runs. | 20:58 |
cfriesen | any pointers on where novaclient is getting this invalid return code from? | 20:59 |
*** davideag_ has quit IRC | 21:02 | |
*** iamjarvo has joined #openstack-nova | 21:03 | |
*** edmondsw has quit IRC | 21:06 | |
*** annegent_ has quit IRC | 21:06 | |
*** VW_ has quit IRC | 21:09 | |
*** VW_ has joined #openstack-nova | 21:09 | |
*** piyanai has quit IRC | 21:12 | |
*** kmARC has joined #openstack-nova | 21:13 | |
*** Sukhdev has joined #openstack-nova | 21:23 | |
openstackgerrit | Vilobh Meshram proposed openstack/nova: Nested Quota Driver: Create column allocated in Quota table https://review.openstack.org/151327 | 21:24 |
*** moshele has joined #openstack-nova | 21:24 | |
*** iamjarvo has quit IRC | 21:25 | |
*** aysyd has quit IRC | 21:26 | |
*** piyanai has joined #openstack-nova | 21:27 | |
*** iamjarvo has joined #openstack-nova | 21:27 | |
*** xyang2 has quit IRC | 21:29 | |
*** romainh has left #openstack-nova | 21:32 | |
*** yassine__ has quit IRC | 21:33 | |
*** otter768 has joined #openstack-nova | 21:33 | |
*** iamjarvo has quit IRC | 21:36 | |
*** otter768 has quit IRC | 21:38 | |
*** jogo has joined #openstack-nova | 21:38 | |
*** atuvenie has quit IRC | 21:38 | |
*** tpeoples has quit IRC | 21:40 | |
*** annegentle has joined #openstack-nova | 21:42 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 21:43 |
*** mwagner_lap has joined #openstack-nova | 21:46 | |
*** vishwanathj has quit IRC | 21:47 | |
*** annashen has quit IRC | 21:49 | |
*** piyanai has quit IRC | 21:50 | |
*** r-daneel has quit IRC | 21:53 | |
*** erlon has quit IRC | 21:53 | |
*** ctrath has quit IRC | 21:53 | |
*** Marga_ has quit IRC | 21:55 | |
*** jungleboyj has quit IRC | 21:57 | |
*** SourabhP has joined #openstack-nova | 22:05 | |
*** achanda_ has joined #openstack-nova | 22:07 | |
*** dprince has quit IRC | 22:07 | |
*** annashen has joined #openstack-nova | 22:09 | |
openstackgerrit | Jay Pipes proposed openstack/nova: ironic: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189362 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: convert config drive to use ImageMeta object https://review.openstack.org/189323 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188119 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: make instance compulsory in blockinfo APIs https://review.openstack.org/188118 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: hyperv: convert driver to use nova.objects.ImageMeta https://review.openstack.org/189302 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: objects: add virtual 'image_meta' property to Instance object https://review.openstack.org/190145 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: virt: convert hardware module to use nova.objects.ImageMeta https://review.openstack.org/188789 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: libvirt: convert to use instance.image_meta property https://review.openstack.org/190146 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: compute: convert manager to use nova.objects.ImageMeta https://review.openstack.org/189685 | 22:11 |
openstackgerrit | Jay Pipes proposed openstack/nova: xenapi: convert driver to use nova.objects.ImageMeta https://review.openstack.org/188790 | 22:11 |
*** achanda has quit IRC | 22:11 | |
*** MentalRay has quit IRC | 22:13 | |
*** ddaskal has quit IRC | 22:22 | |
*** VW_ has quit IRC | 22:28 | |
*** diogogmt has quit IRC | 22:28 | |
*** achanda_ has quit IRC | 22:28 | |
*** tonytan4ever has quit IRC | 22:29 | |
*** dave-mccowan has quit IRC | 22:29 | |
*** ijw has joined #openstack-nova | 22:41 | |
*** ijw__ has quit IRC | 22:44 | |
*** kmARC has quit IRC | 22:45 | |
*** fawadk has quit IRC | 22:45 | |
*** dave-mccowan has joined #openstack-nova | 22:46 | |
*** krtaylor has quit IRC | 22:50 | |
*** annegentle has quit IRC | 22:59 | |
*** dimsum__ has quit IRC | 23:02 | |
*** krtaylor has joined #openstack-nova | 23:03 | |
*** dimsum__ has joined #openstack-nova | 23:05 | |
*** Piet has quit IRC | 23:08 | |
*** annashen has quit IRC | 23:11 | |
*** diogogmt has joined #openstack-nova | 23:12 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Get py34 subunit.run test discovery to work https://review.openstack.org/204723 | 23:12 |
*** Sukhdev has quit IRC | 23:14 | |
*** markvoelker has quit IRC | 23:15 | |
*** haomaiwang has quit IRC | 23:16 | |
*** moshele has quit IRC | 23:18 | |
*** achanda has joined #openstack-nova | 23:22 | |
*** Marga_ has joined #openstack-nova | 23:26 | |
*** baoli has quit IRC | 23:26 | |
*** achanda has quit IRC | 23:28 | |
*** jistr|afk has quit IRC | 23:29 | |
*** Marga_ has quit IRC | 23:30 | |
*** achanda has joined #openstack-nova | 23:31 | |
*** otter768 has joined #openstack-nova | 23:34 | |
*** otter768 has quit IRC | 23:39 | |
*** markus_z has quit IRC | 23:40 | |
*** zzzeek has quit IRC | 23:41 | |
openstackgerrit | Danny Wilson proposed openstack/nova: libvirt: add virtio-scsi support for attached volumes https://review.openstack.org/205726 | 23:41 |
*** achanda has quit IRC | 23:43 | |
*** achanda has joined #openstack-nova | 23:45 | |
*** achanda has quit IRC | 23:47 | |
*** funkypipes has quit IRC | 23:50 | |
*** doude_ has joined #openstack-nova | 23:52 | |
*** jmccrory has quit IRC | 23:54 | |
*** wasmum has quit IRC | 23:54 | |
*** doude has quit IRC | 23:54 | |
*** ildikov_afk has quit IRC | 23:54 | |
*** jmccrory has joined #openstack-nova | 23:54 | |
*** ildikov_afk has joined #openstack-nova | 23:54 | |
*** marzif has joined #openstack-nova | 23:55 | |
*** armax has quit IRC | 23:57 | |
*** dannywil_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!