| *** sacharya_ has quit IRC | 00:01 | |
| *** mc_nair has joined #openstack-nova | 00:02 | |
| *** aginwal__ has quit IRC | 00:02 | |
| openstackgerrit | Zhongcheng Lao proposed openstack/nova: VMware: Omit exceptions during snapshot cleanup https://review.openstack.org/216086 | 00:04 |
|---|---|---|
| *** jaypipes has quit IRC | 00:04 | |
| *** akshai_ has quit IRC | 00:05 | |
| *** pushkaru has quit IRC | 00:07 | |
| *** pixelbeat has quit IRC | 00:09 | |
| *** pixelbeat has joined #openstack-nova | 00:10 | |
| *** aginwala has joined #openstack-nova | 00:10 | |
| *** derekh has joined #openstack-nova | 00:13 | |
| *** josecastroleon has quit IRC | 00:14 | |
| *** aginwala has quit IRC | 00:14 | |
| *** aginwala_ has joined #openstack-nova | 00:15 | |
| *** mc_nair has quit IRC | 00:15 | |
| *** aginwala_ has quit IRC | 00:15 | |
| *** aginwala has joined #openstack-nova | 00:18 | |
| *** stackdump has quit IRC | 00:18 | |
| *** rushil has quit IRC | 00:19 | |
| *** penick has quit IRC | 00:20 | |
| *** otter768 has joined #openstack-nova | 00:20 | |
| *** derekh has quit IRC | 00:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/245248 | 00:21 |
| *** otter768 has quit IRC | 00:25 | |
| *** penick has joined #openstack-nova | 00:27 | |
| *** aginwala has quit IRC | 00:29 | |
| *** shaohe_feng has joined #openstack-nova | 00:39 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 00:43 |
| *** shaohe_feng1 has joined #openstack-nova | 00:43 | |
| *** chenli has joined #openstack-nova | 00:44 | |
| *** mhorban__ has quit IRC | 00:44 | |
| *** mhorban__ has joined #openstack-nova | 00:44 | |
| *** thorst has joined #openstack-nova | 00:44 | |
| *** shaohe_feng has quit IRC | 00:45 | |
| *** leseb has quit IRC | 00:47 | |
| *** pratikmallya has quit IRC | 00:48 | |
| *** rushil has joined #openstack-nova | 00:48 | |
| *** thorst has quit IRC | 00:51 | |
| *** thorst has joined #openstack-nova | 00:51 | |
| *** leseb has joined #openstack-nova | 00:52 | |
| *** RichardRaseley has quit IRC | 00:53 | |
| *** jerrygb has quit IRC | 00:55 | |
| *** thorst has quit IRC | 00:56 | |
| *** aginwala has joined #openstack-nova | 00:56 | |
| *** electrocucaracha has quit IRC | 00:56 | |
| *** armax has joined #openstack-nova | 00:57 | |
| openstackgerrit | li,chen proposed openstack/nova: Add unit tests for nova.virt.firewall.IpTablesFirewallDriver https://review.openstack.org/246274 | 00:58 |
| *** electrocucaracha has joined #openstack-nova | 00:59 | |
| *** Kennan has quit IRC | 00:59 | |
| *** rushil has quit IRC | 01:01 | |
| *** Kennan has joined #openstack-nova | 01:03 | |
| *** liusheng has joined #openstack-nova | 01:03 | |
| *** hemna is now known as hemnafk | 01:04 | |
| *** mriedem has joined #openstack-nova | 01:05 | |
| *** thumpba has joined #openstack-nova | 01:06 | |
| *** zhangjn has joined #openstack-nova | 01:06 | |
| *** EinstCrazy has joined #openstack-nova | 01:07 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 01:07 |
| *** ljxiash has joined #openstack-nova | 01:07 | |
| *** jamielennox is now known as jamielennox|away | 01:10 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 01:11 |
| *** thumpba has quit IRC | 01:12 | |
| *** electrocucaracha has quit IRC | 01:12 | |
| *** ildikov has quit IRC | 01:15 | |
| *** ildikov has joined #openstack-nova | 01:16 | |
| *** shaohe_feng1 has quit IRC | 01:16 | |
| *** penick has quit IRC | 01:19 | |
| *** thorst has joined #openstack-nova | 01:23 | |
| *** thorst has quit IRC | 01:23 | |
| *** aginwala has quit IRC | 01:23 | |
| *** stevemar_ has quit IRC | 01:23 | |
| *** thorst has joined #openstack-nova | 01:23 | |
| *** shaohe_feng has joined #openstack-nova | 01:24 | |
| *** thorst has quit IRC | 01:24 | |
| *** Sree has joined #openstack-nova | 01:24 | |
| *** Sree has quit IRC | 01:24 | |
| *** thorst has joined #openstack-nova | 01:25 | |
| *** Sree has joined #openstack-nova | 01:25 | |
| *** oomichi has joined #openstack-nova | 01:25 | |
| *** aginwala has joined #openstack-nova | 01:27 | |
| *** aginwala has quit IRC | 01:27 | |
| *** thorst has quit IRC | 01:29 | |
| *** Sree has quit IRC | 01:29 | |
| *** oomichi has quit IRC | 01:31 | |
| mriedem | garyk: fyi, looks like intermittent vmware ci failure on image create http://208.91.1.172/logs/246635/1/1447801224/stack.sh.log.gz#_2015-11-17_23_00_08_639 | 01:33 |
| *** jaypipes has joined #openstack-nova | 01:33 | |
| mriedem | ooo glance_store bug http://208.91.1.172/logs/246635/1/1447801224/g-api.log.gz?level=TRACE | 01:33 |
| *** shaohe_feng has quit IRC | 01:34 | |
| mriedem | garyk: fyi https://bugs.launchpad.net/glance-store/+bug/1517275 | 01:36 |
| openstack | Launchpad bug 1517275 in glance_store "VMware NSX CI failing on image create with "AttributeError: HTTPResponse instance has no attribute 'body'"" [Undecided,New] | 01:36 |
| *** eharney has quit IRC | 01:38 | |
| *** nic has quit IRC | 01:38 | |
| *** tpeoples has quit IRC | 01:40 | |
| *** tpeoples_ is now known as tpeoples | 01:40 | |
| *** ssurana has quit IRC | 01:40 | |
| *** mhorban__ has quit IRC | 01:41 | |
| *** zhenguo has joined #openstack-nova | 01:42 | |
| *** mhorban__ has joined #openstack-nova | 01:42 | |
| *** annegentle has quit IRC | 01:47 | |
| *** pratikmallya has joined #openstack-nova | 01:48 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Use stevedore for scheduler host manager https://review.openstack.org/246476 | 01:48 |
| *** boris-42 has joined #openstack-nova | 01:51 | |
| *** pratikmallya has quit IRC | 01:53 | |
| *** otter768 has joined #openstack-nova | 01:53 | |
| *** penick has joined #openstack-nova | 01:58 | |
| *** rushil has joined #openstack-nova | 02:03 | |
| *** baoli has joined #openstack-nova | 02:03 | |
| *** sacharya has joined #openstack-nova | 02:05 | |
| *** apoorvad has quit IRC | 02:07 | |
| *** rushil has quit IRC | 02:10 | |
| *** Marga_ has quit IRC | 02:11 | |
| *** vilobhmm has quit IRC | 02:12 | |
| *** stevemar_ has joined #openstack-nova | 02:14 | |
| *** apoorvad has joined #openstack-nova | 02:15 | |
| *** jwcroppe has quit IRC | 02:21 | |
| mriedem | BobBall: seems we could exclude boto s3 tests for xen project CI? http://logs.openstack.xenproject.org/35/246635/1/check/dsvm-tempest-xen/7bfd07c/console.html | 02:21 |
| *** shaohe_feng has joined #openstack-nova | 02:22 | |
| *** unicell1 has quit IRC | 02:22 | |
| mriedem | actually that's not an s3 test failure, looks like snapshot fails | 02:24 |
| *** baoli_ has joined #openstack-nova | 02:25 | |
| mriedem | BobBall: https://bugs.launchpad.net/nova/+bug/1517282 | 02:27 |
| openstack | Launchpad bug 1517282 in OpenStack Compute (nova) "ImagesOneServerTestJSON.test_create_delete_image fails in dsvm-tempest-xen due to snapshot failure" [Undecided,New] | 02:27 |
| *** penick has quit IRC | 02:27 | |
| *** baoli has quit IRC | 02:28 | |
| *** aginwala has joined #openstack-nova | 02:28 | |
| *** apoorvad has quit IRC | 02:28 | |
| *** davideagnello has quit IRC | 02:28 | |
| *** davideagnello has joined #openstack-nova | 02:29 | |
| *** jerrygb has joined #openstack-nova | 02:29 | |
| *** 7JTAA216T has joined #openstack-nova | 02:30 | |
| *** dims_ has quit IRC | 02:30 | |
| *** 7JTAA216T has quit IRC | 02:30 | |
| *** 21WAACTNP has joined #openstack-nova | 02:31 | |
| *** rubasov has quit IRC | 02:31 | |
| *** edleafe_ has quit IRC | 02:31 | |
| *** jyuso1 has joined #openstack-nova | 02:32 | |
| *** aginwala has quit IRC | 02:32 | |
| *** ccarmack has joined #openstack-nova | 02:33 | |
| *** rk4n has quit IRC | 02:34 | |
| mriedem | mnestratov: your virtuozzo ci doesn't need to be running heat services, fyi | 02:34 |
| mriedem | http://openstack-3rd-party-ci-logs.odin.net/35/246635/1/check/check-dsvm-vz7-tempest-exe-minimal/f2e289b/logs/ | 02:34 |
| *** rk4n has joined #openstack-nova | 02:34 | |
| *** annegentle has joined #openstack-nova | 02:38 | |
| *** rk4n has quit IRC | 02:39 | |
| *** mhorban__ has quit IRC | 02:39 | |
| *** mhorban__ has joined #openstack-nova | 02:40 | |
| *** stevemar_ has quit IRC | 02:42 | |
| *** fawadkhaliq has joined #openstack-nova | 02:42 | |
| *** yonglihe has quit IRC | 02:42 | |
| *** stevemar_ has joined #openstack-nova | 02:42 | |
| *** annegentle has quit IRC | 02:46 | |
| *** klkumar has joined #openstack-nova | 02:53 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 02:56 |
| *** thorst has joined #openstack-nova | 02:57 | |
| *** thorst has quit IRC | 02:57 | |
| *** 21WAACTNP has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** pixelbeat has quit IRC | 03:01 | |
| *** pixelbeat has joined #openstack-nova | 03:02 | |
| *** aginwala has joined #openstack-nova | 03:04 | |
| *** venkat_p has joined #openstack-nova | 03:04 | |
| *** venkat_p has left #openstack-nova | 03:06 | |
| *** rfolco has joined #openstack-nova | 03:08 | |
| *** aginwala_ has joined #openstack-nova | 03:09 | |
| *** browne has joined #openstack-nova | 03:09 | |
| *** aginwala_ has quit IRC | 03:10 | |
| *** ijw has quit IRC | 03:10 | |
| *** aginwala has quit IRC | 03:12 | |
| *** aginwala has joined #openstack-nova | 03:12 | |
| *** aginwala has quit IRC | 03:16 | |
| *** Sree has joined #openstack-nova | 03:20 | |
| *** shaohe_robot has quit IRC | 03:20 | |
| *** Sree has quit IRC | 03:21 | |
| *** rfolco has quit IRC | 03:21 | |
| *** Sree has joined #openstack-nova | 03:21 | |
| *** Sree has quit IRC | 03:22 | |
| *** guilh has quit IRC | 03:22 | |
| *** klkumar has quit IRC | 03:22 | |
| *** Sree has joined #openstack-nova | 03:24 | |
| *** achanda has joined #openstack-nova | 03:24 | |
| *** tpeoples is now known as tpeoples_away | 03:27 | |
| *** tjones has joined #openstack-nova | 03:28 | |
| *** tjones has left #openstack-nova | 03:28 | |
| *** liusheng has quit IRC | 03:35 | |
| *** klkumar has joined #openstack-nova | 03:35 | |
| *** gjayavelu has joined #openstack-nova | 03:35 | |
| *** mhorban__ has quit IRC | 03:37 | |
| *** chenli_ has joined #openstack-nova | 03:38 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron https://review.openstack.org/242846 | 03:39 |
| *** mhorban__ has joined #openstack-nova | 03:40 | |
| *** pixelbeat has quit IRC | 03:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: xen: mask passwords in volume connection_data dict https://review.openstack.org/245987 | 03:41 |
| *** chenli has quit IRC | 03:42 | |
| *** akshai has joined #openstack-nova | 03:44 | |
| *** sacharya has quit IRC | 03:46 | |
| *** liusheng has joined #openstack-nova | 03:48 | |
| *** achanda has quit IRC | 03:48 | |
| *** Sree has quit IRC | 03:49 | |
| *** Sree has joined #openstack-nova | 03:49 | |
| mriedem | jaypipes: you don't want to keep instance actions around forever, like in sandlot forever | 03:52 |
| *** jhesketh has quit IRC | 03:52 | |
| *** jhesketh has joined #openstack-nova | 03:53 | |
| *** mriedem has quit IRC | 03:55 | |
| *** achanda has joined #openstack-nova | 03:55 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** Sree has quit IRC | 04:03 | |
| *** Sree has joined #openstack-nova | 04:04 | |
| *** fawadkhaliq has quit IRC | 04:06 | |
| *** leseb has quit IRC | 04:16 | |
| *** leseb has joined #openstack-nova | 04:18 | |
| *** hparekh2 has quit IRC | 04:20 | |
| *** leseb has quit IRC | 04:22 | |
| *** leseb has joined #openstack-nova | 04:25 | |
| *** gjayavelu has quit IRC | 04:27 | |
| *** edtubill has joined #openstack-nova | 04:27 | |
| *** fawadkhaliq has joined #openstack-nova | 04:27 | |
| *** hparekh has joined #openstack-nova | 04:27 | |
| *** abhishekk has joined #openstack-nova | 04:28 | |
| *** amotoki has joined #openstack-nova | 04:34 | |
| *** Sree_ has joined #openstack-nova | 04:34 | |
| *** mhorban__ has quit IRC | 04:34 | |
| *** Sree_ is now known as Guest53303 | 04:35 | |
| *** mhorban__ has joined #openstack-nova | 04:35 | |
| *** chenli_ has quit IRC | 04:36 | |
| *** Guest53303 has quit IRC | 04:37 | |
| *** zhangjn has quit IRC | 04:37 | |
| *** Sree has quit IRC | 04:37 | |
| *** Sree has joined #openstack-nova | 04:38 | |
| *** zhangjn has joined #openstack-nova | 04:38 | |
| *** edtubill has quit IRC | 04:38 | |
| *** leseb has quit IRC | 04:38 | |
| *** markvoelker_ has quit IRC | 04:40 | |
| *** leseb has joined #openstack-nova | 04:41 | |
| *** annegentle has joined #openstack-nova | 04:47 | |
| *** otter768 has quit IRC | 04:47 | |
| *** amotoki has quit IRC | 04:49 | |
| *** chhavi has joined #openstack-nova | 04:51 | |
| *** leseb has quit IRC | 04:52 | |
| *** vilobhmm has joined #openstack-nova | 04:52 | |
| *** annegentle has quit IRC | 04:52 | |
| *** leseb has joined #openstack-nova | 04:53 | |
| *** jamielennox|away is now known as jamielennox | 04:54 | |
| *** mdrabe has joined #openstack-nova | 04:55 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** amotoki has joined #openstack-nova | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** gcb has joined #openstack-nova | 05:01 | |
| *** Sree has quit IRC | 05:02 | |
| *** jwcroppe has joined #openstack-nova | 05:03 | |
| *** Sree has joined #openstack-nova | 05:04 | |
| *** akshai has quit IRC | 05:05 | |
| *** vilobhmm has quit IRC | 05:07 | |
| *** mdrabe has quit IRC | 05:09 | |
| *** mtanino has quit IRC | 05:09 | |
| *** amotoki has quit IRC | 05:10 | |
| *** achanda has quit IRC | 05:12 | |
| *** amotoki has joined #openstack-nova | 05:14 | |
| *** Marga_ has joined #openstack-nova | 05:20 | |
| *** jaypipes has quit IRC | 05:20 | |
| *** Marga_ has quit IRC | 05:20 | |
| *** Marga_ has joined #openstack-nova | 05:21 | |
| *** vilobhmm has joined #openstack-nova | 05:21 | |
| *** jerrygb has quit IRC | 05:25 | |
| *** achanda has joined #openstack-nova | 05:25 | |
| *** zhangjn_ has joined #openstack-nova | 05:26 | |
| *** zhangjn has quit IRC | 05:27 | |
| *** stevemar_ has quit IRC | 05:27 | |
| *** stevemar_ has joined #openstack-nova | 05:28 | |
| *** krtaylor has quit IRC | 05:29 | |
| *** achanda has quit IRC | 05:29 | |
| *** whenry has quit IRC | 05:30 | |
| *** stevemar_ has quit IRC | 05:30 | |
| *** mhorban__ has quit IRC | 05:32 | |
| *** mhorban__ has joined #openstack-nova | 05:32 | |
| *** rpodolyaka1 has joined #openstack-nova | 05:33 | |
| *** achanda has joined #openstack-nova | 05:35 | |
| *** zhangjn_ has quit IRC | 05:37 | |
| *** rpodolyaka1 has quit IRC | 05:37 | |
| *** zhangjn has joined #openstack-nova | 05:38 | |
| *** rpodolyaka1 has joined #openstack-nova | 05:38 | |
| *** amotoki has quit IRC | 05:40 | |
| *** rpodolyaka1 has quit IRC | 05:46 | |
| *** jwcroppe has quit IRC | 05:50 | |
| *** jwcroppe has joined #openstack-nova | 05:50 | |
| *** salv-orl_ has joined #openstack-nova | 05:50 | |
| *** rpodolyaka1 has joined #openstack-nova | 05:51 | |
| *** deepakcs has joined #openstack-nova | 05:52 | |
| *** achanda has quit IRC | 05:53 | |
| *** salv-orlando has quit IRC | 05:53 | |
| *** jwcroppe has quit IRC | 05:54 | |
| *** salv-orl_ has quit IRC | 05:55 | |
| *** achanda has joined #openstack-nova | 05:58 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/242727 | 06:04 |
| *** rpodolyaka1 has quit IRC | 06:06 | |
| *** suro-patz has joined #openstack-nova | 06:18 | |
| *** suro-patz has quit IRC | 06:21 | |
| *** changbl has quit IRC | 06:22 | |
| *** changbl has joined #openstack-nova | 06:23 | |
| *** Sree_ has joined #openstack-nova | 06:24 | |
| *** Sree_ is now known as Guest40667 | 06:24 | |
| *** rpodolyaka1 has joined #openstack-nova | 06:25 | |
| *** tpeoples has joined #openstack-nova | 06:26 | |
| *** unicell has joined #openstack-nova | 06:26 | |
| *** vilobhmm1 has joined #openstack-nova | 06:26 | |
| *** Sree has quit IRC | 06:27 | |
| *** rpodolyaka1 has quit IRC | 06:28 | |
| *** tyagiprince has joined #openstack-nova | 06:28 | |
| *** vilobhmm has quit IRC | 06:29 | |
| *** mhorban__ has quit IRC | 06:30 | |
| *** rpodolyaka1 has joined #openstack-nova | 06:30 | |
| *** mhorban__ has joined #openstack-nova | 06:30 | |
| *** angdraug has joined #openstack-nova | 06:30 | |
| *** klkumar has quit IRC | 06:31 | |
| *** lpetrut has joined #openstack-nova | 06:33 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron https://review.openstack.org/242846 | 06:33 |
| *** ryandahp has joined #openstack-nova | 06:34 | |
| *** jinxing has joined #openstack-nova | 06:35 | |
| *** liusheng has quit IRC | 06:36 | |
| *** rcernin has joined #openstack-nova | 06:37 | |
| *** liusheng has joined #openstack-nova | 06:37 | |
| *** irenab has joined #openstack-nova | 06:38 | |
| *** tpeoples has quit IRC | 06:39 | |
| *** markvoelker has joined #openstack-nova | 06:40 | |
| *** ankit_ag has joined #openstack-nova | 06:40 | |
| *** rajesht has joined #openstack-nova | 06:41 | |
| *** haomaiwang has quit IRC | 06:42 | |
| *** markvoelker has quit IRC | 06:45 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova-specs: Improve performance of UnShelve API https://review.openstack.org/135387 | 06:45 |
| *** korzen has joined #openstack-nova | 06:48 | |
| *** annegentle has joined #openstack-nova | 06:48 | |
| *** klkumar has joined #openstack-nova | 06:48 | |
| *** otter768 has joined #openstack-nova | 06:48 | |
| *** haomaiwang has joined #openstack-nova | 06:49 | |
| *** salv-orlando has joined #openstack-nova | 06:51 | |
| *** otter768 has quit IRC | 06:53 | |
| *** annegentle has quit IRC | 06:53 | |
| openstackgerrit | Abhishek Kekane proposed openstack/nova-specs: Improve performance of UnShelve API https://review.openstack.org/135387 | 06:54 |
| *** markmc has quit IRC | 06:59 | |
| *** markmc has joined #openstack-nova | 06:59 | |
| *** rpodolyaka1 has quit IRC | 06:59 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** alex_klimov has joined #openstack-nova | 07:01 | |
| *** rpodolyaka1 has joined #openstack-nova | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** Marga_ has quit IRC | 07:03 | |
| *** angdraug has quit IRC | 07:03 | |
| *** Marga_ has joined #openstack-nova | 07:05 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: libvirt: mlnx_direct vif type removal https://review.openstack.org/181671 | 07:06 |
| *** cfriesen__ has quit IRC | 07:08 | |
| *** tdurakov has joined #openstack-nova | 07:08 | |
| *** unicell1 has joined #openstack-nova | 07:08 | |
| *** unicell has quit IRC | 07:09 | |
| *** josecastroleon has joined #openstack-nova | 07:10 | |
| *** mleroy has quit IRC | 07:11 | |
| *** salv-orlando has quit IRC | 07:12 | |
| *** nkrinner has joined #openstack-nova | 07:14 | |
| *** armax has quit IRC | 07:16 | |
| *** garyk1 has joined #openstack-nova | 07:16 | |
| *** tdurakov has quit IRC | 07:18 | |
| *** mjura has joined #openstack-nova | 07:18 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 07:20 |
| *** tdurakov has joined #openstack-nova | 07:22 | |
| ankit_ag | Hi All, A new nova subteam "Operators Bug Triage subteam" was announced to keep a curated list of critical OPS bugs that have reviews that need attention. https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking | 07:24 |
| *** subscope has joined #openstack-nova | 07:24 | |
| ankit_ag | Is there any weekly meeting scheduled for this subteam "Operators Bug Triage subteam" | 07:24 |
| *** Guest40667 has quit IRC | 07:24 | |
| *** Sree has joined #openstack-nova | 07:25 | |
| *** vilobhmm1 has quit IRC | 07:25 | |
| *** jerrygb has joined #openstack-nova | 07:25 | |
| *** mhorban__ has quit IRC | 07:27 | |
| *** mhorban__ has joined #openstack-nova | 07:28 | |
| ankit_ag | Is there any weekly meeting scheduled today for Nova Bugs ? | 07:29 |
| *** Sree has quit IRC | 07:29 | |
| *** jerrygb has quit IRC | 07:30 | |
| *** tdurakov has quit IRC | 07:31 | |
| *** eglynn has quit IRC | 07:32 | |
| *** asselin__ has joined #openstack-nova | 07:32 | |
| ankit_ag | Can someone please point me the right place where I should ask to review pending Nova bugs. e.g https://review.openstack.org/185549 | 07:32 |
| *** achanda has quit IRC | 07:32 | |
| *** asselin_ has quit IRC | 07:33 | |
| *** hparekh has quit IRC | 07:33 | |
| garyk1 | johnthetubaguy: mikal: the subteam has looked at https://review.openstack.org/#/c/218776/ | 07:34 |
| *** alexschm has joined #openstack-nova | 07:34 | |
| *** korzen has quit IRC | 07:34 | |
| *** mleroy has joined #openstack-nova | 07:38 | |
| *** mhorban__ has quit IRC | 07:39 | |
| *** hparekh has joined #openstack-nova | 07:40 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Replaced private field in get_session/engine with public method https://review.openstack.org/246337 | 07:41 |
| *** markvoelker has joined #openstack-nova | 07:41 | |
| *** mhorban__ has joined #openstack-nova | 07:42 | |
| *** sahid has joined #openstack-nova | 07:44 | |
| *** markvoelker has quit IRC | 07:46 | |
| *** angdraug has joined #openstack-nova | 07:46 | |
| *** jmccrory has quit IRC | 07:49 | |
| *** scheuran has joined #openstack-nova | 07:49 | |
| *** abhishekk has quit IRC | 07:49 | |
| *** ankit_ag has quit IRC | 07:50 | |
| *** abhishekk has joined #openstack-nova | 07:50 | |
| *** ankit_ag has joined #openstack-nova | 07:50 | |
| openstackgerrit | lyanchih proposed openstack/nova: NUMATopologyFilter raise exception and not continue filter next node https://review.openstack.org/200630 | 07:51 |
| *** browne has quit IRC | 07:52 | |
| *** rotbeard has joined #openstack-nova | 07:53 | |
| *** jmccrory has joined #openstack-nova | 07:54 | |
| *** abhishekk has quit IRC | 07:54 | |
| *** belmoreira has joined #openstack-nova | 07:57 | |
| *** breitz has quit IRC | 07:58 | |
| *** breitz1 has joined #openstack-nova | 07:58 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 07:59 |
| *** liusheng has quit IRC | 07:59 | |
| *** liusheng has joined #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** meh_di has joined #openstack-nova | 08:02 | |
| meh_di | hi all dears. in nova.compute.api _create_instance how determine the host that run instance(s) | 08:02 |
| *** matrohon has joined #openstack-nova | 08:05 | |
| *** lpetrut has quit IRC | 08:07 | |
| *** lpetrut has joined #openstack-nova | 08:07 | |
| *** liverpooler has joined #openstack-nova | 08:07 | |
| ndipanov | hi meh_di - it's actually not determined there | 08:08 |
| ndipanov | it's done by the scheduler service | 08:08 |
| *** Sree has joined #openstack-nova | 08:09 | |
| ndipanov | this is where we call the conductor service https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1202 | 08:09 |
| ndipanov | which then calls the scheduler here:https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L358 | 08:11 |
| *** rmart04 has joined #openstack-nova | 08:11 | |
| ndipanov | that should allow you to figure it out if you read the code meh_di :) hope it helps! | 08:11 |
| *** eglynn has joined #openstack-nova | 08:12 | |
| *** mhorban__ has quit IRC | 08:12 | |
| *** stevemar_ has joined #openstack-nova | 08:12 | |
| *** mkoderer has quit IRC | 08:15 | |
| meh_di | ndipanov: ok, I trace the _create_instance in compute.api and I reached to https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1001 and follow _create_db_for_new_instance , in this method mentioned "This is called by the scheduler after a location for the instance has been determined." but where the scheduler determine the location of instance ?? | 08:16 |
| meh_di | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1451 | 08:16 |
| *** stevemar_ has quit IRC | 08:16 | |
| *** mkoderer has joined #openstack-nova | 08:17 | |
| ndipanov | meh_di, so _create_db_entry_for_new_instance would just add the record to the Nova DB | 08:18 |
| *** salv-orlando has joined #openstack-nova | 08:19 | |
| ndipanov | then it goes on to call the conductor service which in turn valls the scheduler service which executes this method: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L62 | 08:19 |
| openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 08:20 |
| ndipanov | meh_di, than based on the return value of that - the conductor would call the chosen compute node to finish actually provisioning the instance | 08:20 |
| *** alex_klimov has quit IRC | 08:21 | |
| bauzas | good morning Nova | 08:25 |
| *** salv-orlando has quit IRC | 08:27 | |
| *** mhorban__ has joined #openstack-nova | 08:29 | |
| meh_di | ndipanov: could you tell me on which part of this process scheduler service intervene ? | 08:29 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron https://review.openstack.org/242846 | 08:29 |
| meh_di | ndipanov : my mean is in the create_instance | 08:30 |
| ndipanov | meh_di, so the last thing create_instance does is call the conductor - that's the first link I sent you | 08:30 |
| alexschm | dansmith: Hi, are you around? | 08:31 |
| ndipanov | (if you trace the calls in compute/api.py you will see that that link is the last line to get executed) | 08:31 |
| *** tyagiprince has quit IRC | 08:31 | |
| ndipanov | alexschm, unlikely he's on the west coast meaning it's past midnight there | 08:32 |
| *** alex_klimov has joined #openstack-nova | 08:32 | |
| alexschm | ndipanov: okay thank you! | 08:32 |
| *** tyagiprince has joined #openstack-nova | 08:33 | |
| *** jichen has joined #openstack-nova | 08:33 | |
| *** mpavone has joined #openstack-nova | 08:33 | |
| meh_di | ndipanov: your mean is https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L62 | 08:35 |
| *** mhorban__ has quit IRC | 08:35 | |
| *** sudipto has joined #openstack-nova | 08:36 | |
| *** mhorban__ has joined #openstack-nova | 08:36 | |
| ndipanov | no no | 08:36 |
| ndipanov | that's what the scheduler does | 08:36 |
| ndipanov | but that does not get called immediately | 08:36 |
| bauzas | meh_di: nova boot is an async action | 08:37 |
| ndipanov | first the nova API service calls the conductor service | 08:37 |
| bauzas | meh_di: the API just returns you an instance UUID directly, and then the boot itself is async'd by calling the conductor like ndipanov explained | 08:38 |
| ndipanov | yeah that is a good point meh_di, what bauzas is saying | 08:38 |
| bauzas | meh_di: we never do API long-running actions synchronisously | 08:39 |
| ndipanov | async'd means - the API call returns, but it has posted a message that the conductor service will receive and then send relevant messages to the scheduler (to decide on the destination) and then to the destination compute host to do the work | 08:39 |
| bauzas | meh_di: because you wouldn't want to wait for 10 or more secs the answer to your HTTP request, right? | 08:39 |
| *** fawadkhaliq has quit IRC | 08:41 | |
| *** fawadkhaliq has joined #openstack-nova | 08:42 | |
| *** abhishekk has joined #openstack-nova | 08:45 | |
| meh_di | ndipanov and bauzas : that really helped me to understand how it works, thank you both | 08:45 |
| *** lpetrut has quit IRC | 08:45 | |
| *** jlanoux has joined #openstack-nova | 08:45 | |
| *** ferest has joined #openstack-nova | 08:48 | |
| *** annegentle has joined #openstack-nova | 08:49 | |
| *** sacharya has joined #openstack-nova | 08:49 | |
| *** otter768 has joined #openstack-nova | 08:49 | |
| ndipanov | meh_di, you're welcome :) | 08:52 |
| *** ferest has quit IRC | 08:53 | |
| *** sacharya has quit IRC | 08:53 | |
| *** annegentle has quit IRC | 08:53 | |
| *** otter768 has quit IRC | 08:54 | |
| *** fawadkhaliq has quit IRC | 08:57 | |
| sudipto | ndipanov, have a very small review out here: https://review.openstack.org/#/c/245765/ jenkins failure is not dependent on the patch, wanted to get your thoughts on the same, when you have some time... | 08:59 |
| *** ihrachys has joined #openstack-nova | 09:00 | |
| ndipanov | sudipto, cool - will look at it | 09:00 |
| sudipto | ndipanov, thanks! | 09:00 |
| *** haomaiwang has quit IRC | 09:01 | |
| aloga | johnthetubaguy: could you please raise your -2 from these patch series? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/cache-aware-weigher,n,z | 09:01 |
| aloga | johnthetubaguy: thank you in advance :) | 09:01 |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| rajesht | ndipanov: could you please review https://review.openstack.org/215483 | 09:03 |
| *** rubasov has joined #openstack-nova | 09:04 | |
| *** tyagiprince has quit IRC | 09:04 | |
| ndipanov | rajesht, interesting - will look at it | 09:04 |
| rajesht | ndipanov: thanks :) | 09:05 |
| *** achanda has joined #openstack-nova | 09:05 | |
| *** rpodolyaka1 has quit IRC | 09:08 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove version hardcoding for 'show' method https://review.openstack.org/228578 | 09:10 |
| ankit_ag | ndipanov: Could you please help me to understand your comment on https://review.openstack.org/#/c/221162/ how migration will cause issues in fetching bdms | 09:10 |
| ndipanov | ankit_ag, ah yes | 09:10 |
| ndipanov | well it's quite simple though it's not up to that patch - I should probably remove the -1 | 09:11 |
| *** romainh has joined #openstack-nova | 09:11 | |
| ndipanov | basically if we have a migration in progress | 09:11 |
| ndipanov | we need to account for disk usage on both hosts | 09:11 |
| *** lucas-afk is now known as lucasagomes | 09:12 | |
| *** yassine__ has joined #openstack-nova | 09:12 | |
| *** danpb has joined #openstack-nova | 09:12 | |
| ndipanov | because in case the migration fails we need to be able to revert to the old host, and also we want to make sure that any other boot requests that happen while the instance is being migrated know that the resources on the destination are (soon to be) gone | 09:13 |
| ndipanov | ankit_ag, ^ does that make sense? | 09:13 |
| *** stevemar_ has joined #openstack-nova | 09:13 | |
| ndipanov | the reason this is broken currently is that we only ask libvirt for instances it can see, which does not account for instances that are currently being migrated | 09:14 |
| ndipanov | (both live and cold) | 09:14 |
| ankit_ag | ndipanov: I am planning to add following comment | 09:15 |
| ankit_ag | It should be based on instances and migrations to report more accurate info in edge cases with migrations. | 09:15 |
| ankit_ag | If it helps to understand the issue with migration | 09:15 |
| ankit_ag | ndipanov: Should I also add this in note "we only ask libvirt for instances it can see, which does not account for instances that are currently being migrated" | 09:16 |
| ndipanov | ankit_ag, that would be awesome - I will raise a bug for this and we can fix it later then | 09:17 |
| ndipanov | and get you patch merged asap | 09:17 |
| *** stevemar_ has quit IRC | 09:17 | |
| ankit_ag | ndipanov: Sure, Thank you ! | 09:18 |
| *** mhorban__ has quit IRC | 09:20 | |
| *** mhorban__ has joined #openstack-nova | 09:20 | |
| *** Piet has quit IRC | 09:21 | |
| *** achanda has quit IRC | 09:22 | |
| johnthetubaguy | aloga: I can't remove the -2 until the blueprint is approved for mitaka | 09:23 |
| aloga | johnthetubaguy: arg | 09:23 |
| aloga | johnthetubaguy: I missed that | 09:23 |
| aloga | johnthetubaguy: thanks anyway | 09:23 |
| johnthetubaguy | aloga: so I can help you get that approved | 09:23 |
| * aloga sometimes is dazed with the processes | 09:24 | |
| rajesht | johnthetubaguy: could you please give your feedback on https://review.openstack.org/163798 | 09:24 |
| johnthetubaguy | aloga: agreed, tried to demystify it here: https://wiki.openstack.org/wiki/Nova/Process | 09:24 |
| *** tdurakov_ has joined #openstack-nova | 09:24 | |
| johnthetubaguy | aloga: I will put it on the etherpad of requests for spec-less blueprints here: https://etherpad.openstack.org/p/mitaka-nova-spec-review-tracking | 09:25 |
| *** salv-orlando has joined #openstack-nova | 09:26 | |
| johnthetubaguy | aloga: the next step is it gets raised in tomorrows nova meeting, if no objects, it is usually approved by the end of Friday | 09:26 |
| aloga | johnthetubaguy: I was going trough the doc | 09:26 |
| aloga | johnthetubaguy: thanks, is it needed that I am at the meeting? | 09:27 |
| johnthetubaguy | aloga: cools, let me know any questions, I can add claifications | 09:27 |
| johnthetubaguy | aloga: no, you don't have to, I think your blueprint describes its-self well, so they shouldn't be too much of a problem | 09:27 |
| johnthetubaguy | aloga: PS, I love your blueprints, its been on my TODO list for about a year, but never got around to it, so great to see that happening | 09:28 |
| *** rk4n has joined #openstack-nova | 09:28 | |
| johnthetubaguy | s/blueprints/blueprint/ | 09:28 |
| aloga | johnthetubaguy: thanks :-) | 09:29 |
| aloga | johnthetubaguy: I leave them somehow abandoned from time to time, as some other urget matters are raised here :( | 09:30 |
| johnthetubaguy | aloga: understood | 09:30 |
| aloga | johnthetubaguy: but I try to not leave them die | 09:30 |
| *** rk4n has quit IRC | 09:31 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration https://review.openstack.org/229040 | 09:31 |
| johnthetubaguy | aloga: appreciate you pushing on them, if you get them all passing jenkins and rebased now, it should be all good when I remove the -2 | 09:31 |
| *** e0ne has joined #openstack-nova | 09:32 | |
| aloga | johnthetubaguy: I'm on it :) | 09:32 |
| *** jinxing has quit IRC | 09:32 | |
| johnthetubaguy | aloga: thank you! | 09:33 |
| aloga | johnthetubaguy: you're welcme, thank you as well | 09:33 |
| *** jinxing has joined #openstack-nova | 09:34 | |
| *** jinxing has quit IRC | 09:35 | |
| *** mhorban__ has quit IRC | 09:35 | |
| *** salv-orlando has quit IRC | 09:36 | |
| *** salv-orlando has joined #openstack-nova | 09:37 | |
| *** xek has joined #openstack-nova | 09:37 | |
| *** mpavone has quit IRC | 09:37 | |
| *** tdurakov_ has quit IRC | 09:37 | |
| *** Sree__ has joined #openstack-nova | 09:38 | |
| *** tdurakov_ has joined #openstack-nova | 09:38 | |
| *** pixelbeat has joined #openstack-nova | 09:38 | |
| *** rdopiera has joined #openstack-nova | 09:38 | |
| *** mhorban__ has joined #openstack-nova | 09:39 | |
| *** tdurakov_ has quit IRC | 09:39 | |
| *** Sree has quit IRC | 09:41 | |
| *** rpodolyaka1 has joined #openstack-nova | 09:41 | |
| *** tdurakov_ has joined #openstack-nova | 09:42 | |
| *** markvoelker has joined #openstack-nova | 09:42 | |
| *** jinxing has joined #openstack-nova | 09:42 | |
| *** moshele has joined #openstack-nova | 09:42 | |
| *** jistr has joined #openstack-nova | 09:42 | |
| *** ZZelle has quit IRC | 09:44 | |
| *** ZZelle has joined #openstack-nova | 09:44 | |
| *** markvoelker has quit IRC | 09:46 | |
| *** rk4n has joined #openstack-nova | 09:49 | |
| *** ljxiash has quit IRC | 09:50 | |
| *** lpetrut has joined #openstack-nova | 09:52 | |
| *** tyagiprince has joined #openstack-nova | 09:57 | |
| *** ociuhandu has quit IRC | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** aix has joined #openstack-nova | 10:08 | |
| *** rk4n has quit IRC | 10:09 | |
| *** Sree__ has quit IRC | 10:10 | |
| *** Sree has joined #openstack-nova | 10:10 | |
| *** alex_klimov has quit IRC | 10:11 | |
| *** rk4n has joined #openstack-nova | 10:15 | |
| *** eglynn has quit IRC | 10:15 | |
| *** jinxing has quit IRC | 10:16 | |
| *** prometheanfire has joined #openstack-nova | 10:19 | |
| *** rk4n has quit IRC | 10:19 | |
| prometheanfire | so, can't boot images in liberty because of glanceclient-1.0 defaults to the --os-image-api-version 2 | 10:21 |
| prometheanfire | any workaround? | 10:21 |
| *** Sree has quit IRC | 10:21 | |
| *** Sree has joined #openstack-nova | 10:22 | |
| *** rk4n has joined #openstack-nova | 10:22 | |
| sudipto | prometheanfire nova is still using glance 1.0 - can you explain how you are trying this? | 10:24 |
| *** jerrygb has joined #openstack-nova | 10:26 | |
| *** Sree has quit IRC | 10:26 | |
| prometheanfire | well, when doing a nova boot I get the 'AttributeError: id' backtrace, so it seems to be calling with verions 2 | 10:26 |
| *** tdurakov_ has quit IRC | 10:27 | |
| *** zhangjn has quit IRC | 10:28 | |
| *** daemontool has joined #openstack-nova | 10:28 | |
| *** jwcroppe has joined #openstack-nova | 10:30 | |
| *** daemontool has quit IRC | 10:30 | |
| *** daemontool has joined #openstack-nova | 10:30 | |
| *** jerrygb has quit IRC | 10:31 | |
| *** daemontool has quit IRC | 10:31 | |
| *** ZZelle has quit IRC | 10:31 | |
| *** ZZelle has joined #openstack-nova | 10:32 | |
| moshele | ndipanov: hi | 10:33 |
| *** jwcroppe has quit IRC | 10:34 | |
| mdbooth | How do I type an i with dots on? | 10:34 |
| *** fawadkhaliq has joined #openstack-nova | 10:36 | |
| danpb | mdbooth: ï | 10:38 |
| *** ihrachys has quit IRC | 10:38 | |
| mdbooth | danpb: How did you get that? | 10:38 |
| danpb | cut+paste ;-P | 10:38 |
| mdbooth | I got one by cut/pasting it from a website :) | 10:38 |
| danpb | mdbooth: Ctrl+Alt+u00ef+Enter should do it | 10:40 |
| danpb | mdbooth: doh, i mean Ctrl+Shift+u00ef+Enter | 10:42 |
| mdbooth | I feel like I'm too stupid to operate a keyboard | 10:43 |
| *** e0ne has quit IRC | 10:43 | |
| danpb | mdbooth: or if you still have a arsdigita kinesis.... https://plus.google.com/u/0/+DanielBerrange/posts/P6KyYL78Hpm :-) | 10:44 |
| mdbooth | I never got on with those things. | 10:44 |
| *** naggappan has joined #openstack-nova | 10:45 | |
| *** prometheanfire has left #openstack-nova | 10:48 | |
| *** annegentle has joined #openstack-nova | 10:49 | |
| *** prometheanfire has joined #openstack-nova | 10:49 | |
| prometheanfire | looks like it was fixed in glanceclient 1.1.0 | 10:50 |
| *** otter768 has joined #openstack-nova | 10:50 | |
| ndipanov | moshele, hello | 10:50 |
| *** deepakcs has quit IRC | 10:50 | |
| naggappan | HI i am trying to extract official ubuntu image for openstack using command "sudo kpartx -va trusty-server-cloudimg-amd64-disk1.img " but it doesn't list me any /dev | 10:50 |
| *** zhangjn has joined #openstack-nova | 10:51 | |
| *** rpodolyaka1 has quit IRC | 10:52 | |
| *** rpodolyaka1 has joined #openstack-nova | 10:53 | |
| mhorban_ | Hi guys, there is really old review https://review.openstack.org/#/c/223496/5 Could you please look at that one please? | 10:54 |
| *** sacharya has joined #openstack-nova | 10:54 | |
| *** annegentle has quit IRC | 10:54 | |
| *** otter768 has quit IRC | 10:54 | |
| moshele | ndipanov: regarding you comment in https://review.openstack.org/#/c/227160/ when VF =0 I see that dev_type is type-PCI and not type-PF ? | 10:56 |
| mhorban | @johnthetubaguy Could you please review trivial fix https://review.openstack.org/#/c/223496/5 ? | 10:56 |
| *** prometheanfire has left #openstack-nova | 10:57 | |
| moshele | ndipanov: what should be the value in such case ? | 10:58 |
| *** sacharya has quit IRC | 10:58 | |
| ndipanov | moshele, hmmm that is very interesting | 11:00 |
| moshele | johnthetubaguy: can you +W this commit https://review.openstack.org/#/c/181671/ again I had to rebase it | 11:00 |
| sdague | moshele: +A | 11:00 |
| *** haomaiwang has quit IRC | 11:01 | |
| moshele | sdague: thanks | 11:01 |
| ndipanov | moshele, let me think about it a bit... | 11:01 |
| sdague | danpb: you good with cburgess_'s ebtables patch - https://review.openstack.org/#/c/246580/ ? | 11:01 |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| moshele | ndipanov: sure :) | 11:02 |
| ndipanov | moshele, let me try and see what happens with intel cards in that case | 11:02 |
| *** aysyd has joined #openstack-nova | 11:03 | |
| *** ptm_away is now known as PaulMurray | 11:03 | |
| moshele | ndipanov: ok anyway we can add in https://github.com/openstack/nova/blob/c18799945d6e36092e2cfe93485a8178a0ffa2fb/nova/pci/manager.py#L115 to skip type-PCI as well | 11:04 |
| *** gszasz has joined #openstack-nova | 11:04 | |
| *** e0ne has joined #openstack-nova | 11:04 | |
| *** jeblair has quit IRC | 11:04 | |
| *** atuvenie has joined #openstack-nova | 11:05 | |
| *** ildikov has quit IRC | 11:05 | |
| *** exploreshaifali has joined #openstack-nova | 11:05 | |
| ndipanov | moshele, well not sure that's desireable as there might be genuine cases when you want to pass it thorough | 11:05 |
| *** jeblair has joined #openstack-nova | 11:05 | |
| ndipanov | moshele, anyway - will comment soon | 11:05 |
| *** moshele has quit IRC | 11:06 | |
| *** dims has joined #openstack-nova | 11:07 | |
| *** moshele has joined #openstack-nova | 11:07 | |
| PaulMurray | alex_xu, looking at your comment on https://review.openstack.org/#/c/215483/ about distinction between 'failed' and 'error' | 11:08 |
| *** alex_klimov has joined #openstack-nova | 11:08 | |
| PaulMurray | alex_xu, is that used consistently at all - makes sense but I didn't know | 11:08 |
| *** moshele has quit IRC | 11:09 | |
| ndipanov | hey PaulMurray - have you had a chance to look at https://review.openstack.org/#/c/244489/6 | 11:09 |
| *** rk4n has quit IRC | 11:10 | |
| *** moshele has joined #openstack-nova | 11:10 | |
| *** zhangjn has quit IRC | 11:11 | |
| PaulMurray | ndipanov, actually I haven't - I will take a look | 11:11 |
| *** dims has quit IRC | 11:12 | |
| ndipanov | I was thinking that we could add the whole series to the subteam reviews list, but would be good to get more reviews frrom the subteam folks first | 11:12 |
| PaulMurray | ndipanov, did you see alex_xu's comment about error vs failed - is that a distinction that is used in general as far as you know? | 11:12 |
| ndipanov | PaulMurray, I have that patch marked to look at this morning but still on something else | 11:13 |
| *** EinstCrazy has quit IRC | 11:13 | |
| ndipanov | will read in a bit and comment | 11:13 |
| PaulMurray | ndipanov, ok - won't distract you | 11:13 |
| *** naggappan has quit IRC | 11:13 | |
| *** naggappan has joined #openstack-nova | 11:13 | |
| ndipanov | fwiw - tdurakov looked at a bunch of the patches in that branch already | 11:14 |
| PaulMurray | ndipanov, for your patch - that's part of the improve resource tracking work item right? | 11:14 |
| *** stevemar_ has joined #openstack-nova | 11:14 | |
| PaulMurray | ndipanov, as listed here: https://etherpad.openstack.org/p/mitaka-live-migration | 11:15 |
| *** daemontool has joined #openstack-nova | 11:17 | |
| ndipanov | PaulMurray, it is yes | 11:17 |
| ndipanov | PaulMurray, should we list it there? | 11:17 |
| *** stevemar_ has quit IRC | 11:18 | |
| PaulMurray | ndipanov, yes we should | 11:18 |
| *** klkumar has quit IRC | 11:18 | |
| *** dims has joined #openstack-nova | 11:18 | |
| *** daemontool has quit IRC | 11:18 | |
| *** klkumar has joined #openstack-nova | 11:18 | |
| PaulMurray | ndipanov, the list has "for subteam" and a "ready for cores" sections, so put them in as you see fit | 11:19 |
| *** daemontool has joined #openstack-nova | 11:19 | |
| *** romainh has quit IRC | 11:19 | |
| ndipanov | PaulMurray, will do - thanks!! | 11:19 |
| *** tdurakov_ has joined #openstack-nova | 11:20 | |
| *** rk4n has joined #openstack-nova | 11:20 | |
| openstackgerrit | Madhu Mohan proposed openstack/nova: Corrected repeated text in doc https://review.openstack.org/246872 | 11:20 |
| ndipanov | PaulMurray, erm - where is that exactly ? | 11:21 |
| *** daemontool has quit IRC | 11:21 | |
| *** tdurakov_ has quit IRC | 11:21 | |
| lxsli | PaulMurray: live migration subteam should list reviews on https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking but I don't see a section | 11:22 |
| lxsli | oh wait, it is there, just not under subteams heading | 11:22 |
| openstackgerrit | Madhu Mohan proposed openstack/nova: Corrected repeated text and another minor correction https://review.openstack.org/246872 | 11:22 |
| *** daemontool has joined #openstack-nova | 11:24 | |
| *** tdurakov_ has joined #openstack-nova | 11:24 | |
| *** Okihita has joined #openstack-nova | 11:24 | |
| *** Okihita has left #openstack-nova | 11:25 | |
| *** zhenguo has quit IRC | 11:27 | |
| *** klkumar has quit IRC | 11:29 | |
| *** klkumar has joined #openstack-nova | 11:30 | |
| johnthetubaguy | ndipanov: have you looked at the instance actions stuff at all? does feel like we need to somehow merge migration objects and instance actions at some point | 11:31 |
| *** aix has quit IRC | 11:31 | |
| *** aswadr has joined #openstack-nova | 11:32 | |
| *** tdurakov_ has quit IRC | 11:32 | |
| *** aix has joined #openstack-nova | 11:32 | |
| *** mwagner has quit IRC | 11:34 | |
| *** tdurakov_ has joined #openstack-nova | 11:35 | |
| *** rfolco has joined #openstack-nova | 11:35 | |
| openstackgerrit | Zhihai Song proposed openstack/python-novaclient: Disable the client to retrive instance by ID in the DB https://review.openstack.org/246881 | 11:35 |
| *** smatzek has joined #openstack-nova | 11:39 | |
| *** mhorban__ has quit IRC | 11:39 | |
| garyk1 | johnthetubaguy: mikal: can you guys please look at https://review.openstack.org/218776 - it has been reveiwed by the subteam | 11:39 |
| *** ihrachys has joined #openstack-nova | 11:40 | |
| ndipanov | johnthetubaguy, well yes I thought the same some time ago | 11:40 |
| ndipanov | but tbh not sure what we get by that other than "purity" | 11:40 |
| ndipanov | as long as we use the migration record properly we should be good | 11:41 |
| *** mhorban__ has joined #openstack-nova | 11:41 | |
| *** subscope has quit IRC | 11:41 | |
| ndipanov | that is to say - I am not convinced that the major refactor is worth the gain | 11:41 |
| ndipanov | same thing with request_spec, vs instance, vs ... | 11:42 |
| ndipanov | and which bits go where.. | 11:42 |
| *** subscope has joined #openstack-nova | 11:42 | |
| johnthetubaguy | ndipanov: it might just be some API "sugar" we can add, honestly | 11:42 |
| ndipanov | as long as we keep the data and use it consistently and in a non-racy manner | 11:42 |
| johnthetubaguy | ndipanov: I am thinking about us ensuring you can look up any action request-id in a single place, at some point | 11:42 |
| ndipanov | it's not critical whether we keep the flavor on the instance or request_spec etc | 11:42 |
| *** markvoelker has joined #openstack-nova | 11:43 | |
| johnthetubaguy | its tempting to add an instance action record in the DB, thats updating on the migration state changes, but just reading both lists in the API might be a smarter move. | 11:44 |
| PaulMurray | lxsli, thanks for posting the link - ndipanov sorry - I turned away for a while | 11:44 |
| *** claudiub has joined #openstack-nova | 11:44 | |
| johnthetubaguy | garyk1: cools, thanks for the ping, looks good now | 11:44 |
| garyk1 | johnthetubaguy: thanks | 11:46 |
| *** daemontool has quit IRC | 11:47 | |
| PaulMurray | lxsli, ndipanov it should go in the live migration section under Priority Features - not subteams section | 11:47 |
| *** markvoelker has quit IRC | 11:47 | |
| *** daemontool has joined #openstack-nova | 11:47 | |
| *** rdopiera has quit IRC | 11:48 | |
| *** exploreshaifali has quit IRC | 11:49 | |
| *** mhorban__ has quit IRC | 11:51 | |
| *** haomaiwang has quit IRC | 11:51 | |
| *** mhorban__ has joined #openstack-nova | 11:51 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Add more 'actions' for server concepts doc https://review.openstack.org/236947 | 11:52 |
| lxsli | PaulMurray: yep figured that out, maybe "subteams" section should be "Other features" but w/e | 11:52 |
| *** rook has quit IRC | 11:52 | |
| *** exploreshaifali has joined #openstack-nova | 11:53 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 11:53 |
| ndipanov | johnthetubaguy, so that's one of the reasons I'd like for us to have a programming API (objects? not sure it's the right place) | 11:53 |
| *** ildikov has joined #openstack-nova | 11:54 | |
| bauzas | johnthetubaguy: so, I'm about republishing a new Tasks spec | 11:55 |
| *** jhesketh has quit IRC | 11:55 | |
| *** sudipto has quit IRC | 11:55 | |
| bauzas | johnthetubaguy: and I'll also upload a new manifesto | 11:56 |
| bauzas | in case people wonder about instance actions | 11:56 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 11:56 |
| *** romainh has joined #openstack-nova | 11:57 | |
| johnthetubaguy | ndipanov: we do have the compute/api.py, but I guess thats not what you mean for that? | 11:58 |
| johnthetubaguy | bauzas: OK, sounds promising, I am thinking we should upload a very small spec around making instance actions consistent everywhere? is that the spec? | 11:58 |
| bauzas | johnthetubaguy: yup, agreed | 11:59 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 11:59 |
| ndipanov | johnthetubaguy, no I mean a place where we insulate how we evolve this stuff from the business logic | 11:59 |
| bauzas | johnthetubaguy: since the manifesto is there for saying which specs we should provide, I'm +1 | 11:59 |
| bauzas | that way, we can also have one blueprint for every change related to that | 12:00 |
| *** jinxing has joined #openstack-nova | 12:00 | |
| bauzas | and providing to the low-hanging-fruit etherpad | 12:00 |
| *** Sree has joined #openstack-nova | 12:00 | |
| ndipanov | johnthetubaguy, currently we either add a decorator (for example actions/faults) and miss a ton of cases, or we have to do a messy data migration which has it's consequences | 12:00 |
| *** jhesketh has joined #openstack-nova | 12:00 | |
| ndipanov | johnthetubaguy, not sure if you get what I mean :) | 12:01 |
| johnthetubaguy | ndipanov: oh sorry, I get what you mean now | 12:01 |
| ndipanov | so to me it feels that having something like | 12:02 |
| johnthetubaguy | ndipanov: yeah, I would be happy with a consistent pattern that makes sense, its very messy right now | 12:02 |
| ndipanov | instance.initialize_migration() | 12:02 |
| *** doug-fish has joined #openstack-nova | 12:02 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: adds os-win library https://review.openstack.org/212102 | 12:02 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: removes *Utils modules and unit tests https://review.openstack.org/215917 | 12:02 |
| ndipanov | or even relocation = Relocation.intialize_for(instance) | 12:02 |
| ndipanov | or something | 12:02 |
| * bauzas goes for an appointment | 12:02 | |
| *** exploreshaifali has quit IRC | 12:03 | |
| ndipanov | where there is an API as opposed to just listing stuff that needs to happen in rpc callbacks | 12:03 |
| johnthetubaguy | yeah, I quite like the attached to instance idea | 12:03 |
| johnthetubaguy | for instance actions and the migration stuff | 12:03 |
| johnthetubaguy | (and instance faults, I guess) | 12:03 |
| ndipanov | johnthetubaguy, I worry that it will make the instance object that now does also sever additional things unweildy | 12:03 |
| ndipanov | just because it's the central model of the business logic | 12:04 |
| ndipanov | it "looks" like the natural place to put things | 12:04 |
| johnthetubaguy | yeah, agreed, its tempting and concerning | 12:04 |
| ndipanov | but we might end up with a "manager" class :) | 12:04 |
| johnthetubaguy | true | 12:04 |
| *** rk4n has quit IRC | 12:04 | |
| ndipanov | though that is just a guess | 12:04 |
| ndipanov | it may be fine | 12:05 |
| johnthetubaguy | yeah, feels like a "suck it and see kind" of thing | 12:05 |
| ndipanov | john5223, so that's what I tried to do with "apply_migration_context" stuff | 12:05 |
| ndipanov | but that might be too low level | 12:05 |
| johnthetubaguy | we have those always lazyloaded things already, it feels like one of those | 12:05 |
| *** EinstCrazy has joined #openstack-nova | 12:06 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Converting nova.virt.hyperv to py3 https://review.openstack.org/232554 | 12:07 |
| *** atuvenie has quit IRC | 12:07 | |
| *** zhangjn has joined #openstack-nova | 12:09 | |
| *** zhangjn has quit IRC | 12:09 | |
| *** zhangjn has joined #openstack-nova | 12:10 | |
| *** zhangjn has quit IRC | 12:10 | |
| *** zhangjn has joined #openstack-nova | 12:10 | |
| *** rpodolyaka1 has quit IRC | 12:12 | |
| *** tojuvone has quit IRC | 12:13 | |
| *** tdurakov_ has quit IRC | 12:14 | |
| *** aswadr has quit IRC | 12:15 | |
| *** tdurakov_ has joined #openstack-nova | 12:16 | |
| *** rk4n has joined #openstack-nova | 12:19 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Compute: Adds driver disk_gb instance overhead estimation https://review.openstack.org/234283 | 12:19 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Fixes disk overhead claim issue https://review.openstack.org/246307 | 12:19 |
| *** ociuhandu has joined #openstack-nova | 12:19 | |
| *** shaohe_feng has quit IRC | 12:20 | |
| *** rk4n has quit IRC | 12:20 | |
| *** subscope has quit IRC | 12:20 | |
| *** wolsen has quit IRC | 12:20 | |
| *** subscope has joined #openstack-nova | 12:21 | |
| *** subscope has quit IRC | 12:21 | |
| ndipanov | moshele, https://github.com/openstack/nova/blob/5eaf4fe5236fcd1356205bf10c8d477aa9fc2892/nova/virt/libvirt/driver.py#L4802-L4803 | 12:21 |
| ndipanov | so what you are seeing is to be expected actually | 12:22 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces longs with ints https://review.openstack.org/238239 | 12:22 |
| *** ociuhandu_ has joined #openstack-nova | 12:22 | |
| ndipanov | the problem is the "PF address whitelists all of VFs" is just a bad decision that opens us up to edge cases and ambiguities like this | 12:23 |
| *** ociuhandu has quit IRC | 12:23 | |
| *** ociuhandu_ is now known as ociuhandu | 12:23 | |
| *** alejandrito has joined #openstack-nova | 12:23 | |
| *** aarefiev has quit IRC | 12:25 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes dict keys and items references for Python 3 https://review.openstack.org/232585 | 12:25 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Pass bdm info to _get_instance_disk_info method https://review.openstack.org/221162 | 12:26 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Replaces izip_longest with six.moves.zip_longest https://review.openstack.org/238321 | 12:26 |
| *** aarefiev has joined #openstack-nova | 12:27 | |
| alex_xu | PaulMurray: at least from this patch https://review.openstack.org/#/c/219299/1/nova/compute/manager.py, _cleanup_incomplete_migrations clean up all the error migration, and set them to failed. | 12:27 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: fixed warn logging in sqlalchemy/api.py https://review.openstack.org/246901 | 12:28 |
| *** jkraj has joined #openstack-nova | 12:28 | |
| *** thorst has joined #openstack-nova | 12:28 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 compatibility for filter results https://review.openstack.org/232620 | 12:28 |
| *** CORTEX\krfa has joined #openstack-nova | 12:30 | |
| CORTEX\krfa | Hello. I've configured cinder to use our NFS NetApp storage solution for the shared storage. How should I proceed to use the NFS shared storage for the compute node using nova? I didn't find any exemple about it. | 12:32 |
| openstackgerrit | bharaththiruveedula proposed openstack/nova-specs: Returning Compute Node IP address in hypervisor list https://review.openstack.org/215207 | 12:33 |
| *** rook has joined #openstack-nova | 12:33 | |
| *** ihrachys_ has joined #openstack-nova | 12:34 | |
| *** ihrachys has quit IRC | 12:35 | |
| *** tojuvone has joined #openstack-nova | 12:37 | |
| *** lucasagomes is now known as lucas-hungry | 12:37 | |
| openstackgerrit | Madhu Mohan proposed openstack/nova: Corrected few grammatical nitpics https://review.openstack.org/246872 | 12:41 |
| *** fawadkhaliq has quit IRC | 12:42 | |
| *** abhishekk has quit IRC | 12:43 | |
| *** markvoelker has joined #openstack-nova | 12:43 | |
| alex_xu | sdague: looks like there still can have some improvement for https://review.openstack.org/#/c/236947/11/api-guide/source/server_concepts.rst, or you want to merge first version, continue improve later? | 12:43 |
| *** ziggypup has joined #openstack-nova | 12:43 | |
| *** ziggypup has left #openstack-nova | 12:44 | |
| sdague | honestly, I'd rather merge and add more patches later | 12:44 |
| sdague | because a merged document goes straight to the web, so is better for folks immediately | 12:44 |
| alex_xu | sdague: ok, cool, let me +w | 12:45 |
| sdague | I think with docs we're never going to be perfect, so the important criteria is "does this make things better", I find we can go around and around on docs patches and not land anything some times | 12:46 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state https://review.openstack.org/168916 | 12:46 |
| *** romainh has quit IRC | 12:47 | |
| *** markvoelker has quit IRC | 12:48 | |
| alex_xu | sdague: ok, got it | 12:48 |
| *** annegentle has joined #openstack-nova | 12:50 | |
| *** otter768 has joined #openstack-nova | 12:51 | |
| *** atuvenie has joined #openstack-nova | 12:52 | |
| alex_xu | sdague: I'm thinking the different between concept doc and api ref, for example, the concept already explain what is evacuate, then the api ref is more focus what effect based on each parameter? | 12:52 |
| *** naggappan has quit IRC | 12:53 | |
| *** atuvenie_ has joined #openstack-nova | 12:53 | |
| *** klkumar has quit IRC | 12:53 | |
| *** otter768 has quit IRC | 12:55 | |
| *** annegentle has quit IRC | 12:55 | |
| PaulMurray | alex_xu, I also noticed migration.status being set to failed a lot when I was going through patches ndipanov is doing at the moment | 12:56 |
| PaulMurray | https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1417667,n,z | 12:56 |
| *** atuvenie has quit IRC | 12:57 | |
| *** tdurakov_ has quit IRC | 12:57 | |
| openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | 12:57 |
| *** e0ne has quit IRC | 12:57 | |
| *** mpavone has joined #openstack-nova | 12:57 | |
| jichen | alex_xu: I think api ref should tell user what's the input and what's the expect output... that's why I think http://developer.openstack.org/api-ref-compute-v2.1.html#lock is fine , we can't have too much detail on the api-ref | 12:58 |
| *** tdurakov_ has joined #openstack-nova | 12:58 | |
| *** moshele has quit IRC | 12:58 | |
| alex_xu | PaulMurray: cool, will try to take a look at those patches also. It will be good we have consistent on those status, also a good chance doc them | 12:59 |
| *** rotbeard has quit IRC | 13:00 | |
| *** moshele has joined #openstack-nova | 13:02 | |
| sdague | alex_xu: yeh, I honestly also think the concept doc should explain why you'd use a particular action | 13:03 |
| sdague | because snapshot vs. backup | 13:03 |
| sdague | or why anyone would want to use pause | 13:03 |
| sdague | but, honestly, almost all the time more explanation is better than less | 13:04 |
| *** obondarev_afk is now known as obondarev | 13:04 | |
| alex_xu | sdague: yea | 13:04 |
| *** rk4n has joined #openstack-nova | 13:04 | |
| *** jyuso1 has quit IRC | 13:04 | |
| *** angdraug has quit IRC | 13:05 | |
| *** jerrygb has joined #openstack-nova | 13:06 | |
| johnthetubaguy | sdague: alex_xu: +1 those comments about the concept guide. | 13:06 |
| alex_xu | sdague: I'm think we should have some patch to show what the api ref is we expected, then can guide people to improve it | 13:06 |
| alex_xu | we already have example for concept doc | 13:06 |
| sdague | yeh, probably good point | 13:06 |
| johnthetubaguy | alex_xu: +1 for getting good worked example folks can copy, I would do the API complete ref and concept doc, linking to each other maybe? | 13:07 |
| sdague | ok, I have to drop for a bit and will be intermitent over the next couple of hours | 13:07 |
| alex_xu | johnthetubaguy: yea, I think we can linking each other | 13:07 |
| alex_xu | sdague: no problem | 13:07 |
| sdague | johnthetubaguy: https://review.openstack.org/#/c/246580/ - completes our parity with libvirt on ebtables. Would like to make sure that lands | 13:07 |
| PaulMurray | alex_xu, at the summit we talked about better doc for live-migration - I see that in that concept guide patch | 13:08 |
| PaulMurray | but we also talked about giving an example of where and how it might be used | 13:08 |
| PaulMurray | e.g. rolling through a site to implement kernel patch upgrades that require reboot | 13:08 |
| PaulMurray | does that fit in there somehow? | 13:09 |
| alex_xu | PaulMurray: yea, I think that is good in concept doc | 13:09 |
| *** markvoelker has joined #openstack-nova | 13:09 | |
| johnthetubaguy | PaulMurray: totally should be in the concept guide I think, explains the why you use it that way, and the patterns that work well. | 13:09 |
| * johnthetubaguy needs to go for some food | 13:09 | |
| PaulMurray | alex_xu, I can add something like that in another patch if you like | 13:09 |
| alex_xu | PaulMurray: cool, appreciate the doc patch :) | 13:10 |
| PaulMurray | It was in the back of my mind alread | 13:10 |
| *** jwcroppe has joined #openstack-nova | 13:10 | |
| *** jerrygb has quit IRC | 13:10 | |
| *** e0ne has joined #openstack-nova | 13:10 | |
| *** klkumar has joined #openstack-nova | 13:13 | |
| *** vladikr has joined #openstack-nova | 13:14 | |
| *** jwcroppe has quit IRC | 13:15 | |
| *** stevemar_ has joined #openstack-nova | 13:15 | |
| *** jwcroppe has joined #openstack-nova | 13:15 | |
| *** jwcroppe_ has joined #openstack-nova | 13:17 | |
| *** stevemar_ has quit IRC | 13:18 | |
| *** ihrachys_ has quit IRC | 13:18 | |
| *** atuvenie_ has quit IRC | 13:18 | |
| *** ihrachys has joined #openstack-nova | 13:19 | |
| *** sacharya has joined #openstack-nova | 13:19 | |
| *** jwcroppe has quit IRC | 13:20 | |
| *** rotbeard has joined #openstack-nova | 13:21 | |
| *** angdraug has joined #openstack-nova | 13:21 | |
| *** sacharya has quit IRC | 13:24 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE https://review.openstack.org/245050 | 13:25 |
| *** rpodolyaka1 has joined #openstack-nova | 13:26 | |
| *** irenab_ has joined #openstack-nova | 13:26 | |
| *** annegentle has joined #openstack-nova | 13:26 | |
| *** irenab has quit IRC | 13:27 | |
| *** irenab_ is now known as irenab | 13:27 | |
| ndipanov | alex_xu, PaulMurray - I am -1 on this whole approach tbh https://review.openstack.org/#/c/215483/3 | 13:30 |
| PaulMurray | ndipanov, ack | 13:32 |
| *** alex_klimov has quit IRC | 13:33 | |
| *** alex_klimov has joined #openstack-nova | 13:33 | |
| PaulMurray | ndipanov, what is the security issue? | 13:33 |
| *** rubasov has quit IRC | 13:34 | |
| PaulMurray | some kind of DOS? | 13:34 |
| ndipanov | PaulMurray, well check the commit message and the linked bug | 13:34 |
| ndipanov | yeah | 13:34 |
| *** atuvenie_ has joined #openstack-nova | 13:34 | |
| *** atuvenie has joined #openstack-nova | 13:34 | |
| ndipanov | allegedly since all migrations are admin actions | 13:35 |
| *** kbyrne has quit IRC | 13:36 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Transforms requests and responses into bytes https://review.openstack.org/238323 | 13:37 |
| *** ndipanov has quit IRC | 13:38 | |
| *** ndipanov has joined #openstack-nova | 13:39 | |
| *** atuvenie has quit IRC | 13:39 | |
| *** atuvenie has joined #openstack-nova | 13:40 | |
| *** meh_di has left #openstack-nova | 13:42 | |
| *** mwagner has joined #openstack-nova | 13:42 | |
| alex_xu | ndipanov: miss the rollback set status to error also. really mess on status error and failed | 13:43 |
| *** kbyrne has joined #openstack-nova | 13:43 | |
| *** achanda has joined #openstack-nova | 13:44 | |
| ndipanov | alex_xu, consider that before the migration object was introduced - there was no way to tell what was happening | 13:44 |
| ndipanov | unless you trace the code and logs | 13:44 |
| alex_xu | ndipanov: yea, indeed :) | 13:45 |
| alex_xu | ndipanov: is it worth there is propose to clear the usage of failed and error? | 13:45 |
| *** krtaylor has joined #openstack-nova | 13:45 | |
| *** achanda has quit IRC | 13:46 | |
| *** lucas-hungry is now known as lucasagomes | 13:48 | |
| ndipanov | alex_xu, well sure I'd like to see it cleaned up but on their own, not tied to that half-assed security fix | 13:49 |
| *** achanda has joined #openstack-nova | 13:49 | |
| ndipanov | <airquote> security fix </airquote> | 13:49 |
| alex_xu | ndipanov: yea | 13:50 |
| *** tpeoples has joined #openstack-nova | 13:50 | |
| ndipanov | alex_xu, so that's why I am -1 on that change really (as I tried to explain on the commit message comment) | 13:50 |
| *** rubasov has joined #openstack-nova | 13:51 | |
| alex_xu | ndipanov: yea, agree | 13:51 |
| *** sudipto has joined #openstack-nova | 13:53 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Support configuration of multiple pci_alias within an array https://review.openstack.org/242555 | 13:53 |
| *** achanda has quit IRC | 13:53 | |
| *** eharney has joined #openstack-nova | 13:53 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Move Process and Mentoring pages to devref https://review.openstack.org/206070 | 13:55 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Move Process and Mentoring pages to devref https://review.openstack.org/206070 | 13:56 |
| *** gjayavelu has joined #openstack-nova | 13:57 | |
| *** doug-fish has quit IRC | 13:58 | |
| *** carolynvs has joined #openstack-nova | 13:59 | |
| *** rook has quit IRC | 13:59 | |
| *** tdurakov_ has quit IRC | 13:59 | |
| *** rlrossit has joined #openstack-nova | 14:00 | |
| *** carolynvs has quit IRC | 14:00 | |
| *** tpeoples has quit IRC | 14:00 | |
| *** tpeoples_away is now known as tpeoples | 14:01 | |
| *** changbl has quit IRC | 14:01 | |
| *** rook has joined #openstack-nova | 14:01 | |
| *** burgerk has joined #openstack-nova | 14:02 | |
| *** akshai has joined #openstack-nova | 14:04 | |
| *** exploreshaifali has joined #openstack-nova | 14:04 | |
| *** mdrabe has joined #openstack-nova | 14:05 | |
| *** lykinsbd has joined #openstack-nova | 14:05 | |
| *** ijuwang has joined #openstack-nova | 14:06 | |
| *** LukeHinds has joined #openstack-nova | 14:06 | |
| *** tdurakov_ has joined #openstack-nova | 14:06 | |
| *** bapalm has joined #openstack-nova | 14:08 | |
| *** boris-42 has quit IRC | 14:08 | |
| *** vlaza has joined #openstack-nova | 14:09 | |
| *** ctrath has joined #openstack-nova | 14:09 | |
| *** burgerk has quit IRC | 14:11 | |
| *** angdraug has quit IRC | 14:12 | |
| *** ankit_ag has quit IRC | 14:12 | |
| *** achanda has joined #openstack-nova | 14:12 | |
| garyk1 | sdague: you want me to update the commit message here? https://review.openstack.org/#/c/218644/ | 14:14 |
| sdague | garyk: yeh | 14:15 |
| sdague | then we can flush it back through | 14:15 |
| sdague | it just needed to get pulled because ever nova change was exploding | 14:15 |
| *** dane-fichter has joined #openstack-nova | 14:15 | |
| openstackgerrit | garyk proposed openstack/nova: Use flavor instead of flavour https://review.openstack.org/218644 | 14:16 |
| garyk1 | sdague: ^ | 14:16 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Multinode job for live-migration https://review.openstack.org/245050 | 14:16 |
| *** mgoddard__ has joined #openstack-nova | 14:18 | |
| garyk1 | sdague: gracias | 14:19 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo FlexibleBooleanField https://review.openstack.org/245336 | 14:19 |
| *** romainh has joined #openstack-nova | 14:20 | |
| tdurakov_ | sdague, johnthetubaguy hi, could you assist with merging this one: https://review.openstack.org/#/c/245050/ | 14:20 |
| *** mgoddard_ has quit IRC | 14:20 | |
| *** doug-fish has joined #openstack-nova | 14:21 | |
| *** jichen has quit IRC | 14:22 | |
| *** jaypipes has joined #openstack-nova | 14:23 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo VersionPredicateField https://review.openstack.org/245327 | 14:24 |
| *** daemontool_ has joined #openstack-nova | 14:25 | |
| *** daemontool_ has quit IRC | 14:26 | |
| *** lsmola has quit IRC | 14:26 | |
| *** daemontool_ has joined #openstack-nova | 14:27 | |
| *** Sree has quit IRC | 14:27 | |
| *** daemontool has quit IRC | 14:27 | |
| *** Sree has joined #openstack-nova | 14:27 | |
| *** angdraug has joined #openstack-nova | 14:27 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Use o.vo DictOfListOfStringsField https://review.openstack.org/245344 | 14:27 |
| *** daemontool_ has quit IRC | 14:28 | |
| *** mgoddard__ has quit IRC | 14:28 | |
| *** shaohe_feng has joined #openstack-nova | 14:29 | |
| *** mgoddard has joined #openstack-nova | 14:29 | |
| *** daemontool_ has joined #openstack-nova | 14:29 | |
| *** lsmola has joined #openstack-nova | 14:29 | |
| *** dustins has joined #openstack-nova | 14:30 | |
| *** lsmola has quit IRC | 14:31 | |
| *** smatzek has quit IRC | 14:31 | |
| *** openstackgerrit has quit IRC | 14:31 | |
| *** Sree has quit IRC | 14:32 | |
| *** openstackgerrit has joined #openstack-nova | 14:32 | |
| *** aspiers has quit IRC | 14:34 | |
| *** tojuvone has quit IRC | 14:35 | |
| *** jkraj has quit IRC | 14:37 | |
| tdurakov_ | jaypipes, morning | 14:38 |
| lxsli | anyone to look at https://review.openstack.org/#/c/215568/ please? | 14:39 |
| *** bapalm has quit IRC | 14:43 | |
| *** angdraug has quit IRC | 14:43 | |
| *** angdraug has joined #openstack-nova | 14:43 | |
| *** bapalm has joined #openstack-nova | 14:43 | |
| *** burt has joined #openstack-nova | 14:44 | |
| *** fawadkhaliq has joined #openstack-nova | 14:44 | |
| *** mriedem has joined #openstack-nova | 14:45 | |
| lxsli | johnthetubaguy: FYI https://review.openstack.org/#/c/206070 | 14:45 |
| *** raildo is now known as raildo-afk | 14:46 | |
| *** cfriesen__ has joined #openstack-nova | 14:48 | |
| tdurakov_ | bauzas, alex_xu, folks, are you around?) | 14:49 |
| bauzas | tdurakov_: yep | 14:49 |
| bauzas | tdurakov_: wassup ? | 14:50 |
| *** vlaza has left #openstack-nova | 14:50 | |
| tdurakov_ | bauzas, https://review.openstack.org/#/c/245050/ - could you review this? | 14:50 |
| bauzas | tdurakov_: I'm honestly not that seasoned | 14:51 |
| bauzas | tdurakov_: so I could +1, but I'm a bit torn with providing a +W if I agree | 14:51 |
| *** daemontool_ has quit IRC | 14:51 | |
| tdurakov_ | bauzas, heh, it's actually hook for job that is in experimental pipeline, this one blocks me from implementing nfs | 14:52 |
| *** otter768 has joined #openstack-nova | 14:52 | |
| tdurakov_ | bauzas, trust me:) | 14:52 |
| *** garyk1 has quit IRC | 14:52 | |
| mriedem | tdurakov_: come on | 14:52 |
| bauzas | sdague: ^ | 14:52 |
| bauzas | tdurakov_: that's not that I don't trust you | 14:53 |
| bauzas | tdurakov_: but you see, I don't have a context for that | 14:53 |
| mriedem | i don't trust anyone that says trust me | 14:53 |
| mriedem | tdurakov_: more details in the commit message would be helpful, or as a reply in the comments | 14:53 |
| sdague | yeh, probably another check experimental | 14:54 |
| sdague | so we can be confident it's working | 14:54 |
| jaypipes | tdurakov_: Утро добрым не бывает | 14:54 |
| tdurakov_ | sdague, it won't work | 14:54 |
| tdurakov_ | there is no changes in code, so experimental will ignore this | 14:55 |
| mriedem | so i guess it's just a hook called from the gate-tempest-dsvm-multinode-live-migration job in project-config | 14:55 |
| mriedem | http://logs.openstack.org/50/245050/16/experimental/gate-tempest-dsvm-multinode-live-migration/6e12318/console.html#_2015-11-17_23_22_53_963 | 14:55 |
| *** pedroalvarez has left #openstack-nova | 14:55 | |
| bauzas | tdurakov_: so, see, like mriedem explained, if you want to have cores reviewing, you should maybe explain more what's the problem and what can be fixed | 14:55 |
| mriedem | https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L2014 | 14:55 |
| mriedem | and the hook is here https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L2039 | 14:56 |
| tdurakov_ | bauzas, mriedem added comments | 14:56 |
| *** smatzek has joined #openstack-nova | 14:56 | |
| bauzas | tdurakov_: okay, I need to ramp up, see | 14:56 |
| bauzas | ? | 14:56 |
| *** otter768 has quit IRC | 14:56 | |
| mriedem | and i see it's an experimental queue job here https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L6570 | 14:56 |
| bauzas | tdurakov_: that's why I think I could +1 but not +W | 14:57 |
| tdurakov_ | jaypipes, heh, why so? | 14:57 |
| mriedem | i also see that we're running this on stable/kilo and stable/liberty https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L6570 | 14:57 |
| mriedem | which won't work | 14:57 |
| bauzas | je ne comprend pas le russe :p | 14:57 |
| *** dustins has quit IRC | 14:57 | |
| bauzas | mriedem: oh that would break the experimental pipeline then ? | 14:58 |
| mriedem | bauzas: experimental isn't voting | 14:58 |
| *** ctrath has quit IRC | 14:58 | |
| bauzas | mriedem: that I know | 14:58 |
| mriedem | so that doesn't really matter, but it's a waste of resources to run a job we know won't work | 14:58 |
| bauzas | mriedem: that's my point | 14:58 |
| mriedem | https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1037 | 14:58 |
| mriedem | so i can update that and filter it from kilo/liberty | 14:58 |
| bauzas | mriedem: even if your dog doesn't vote, will you accept that some bad guy hurts your dog ? | 14:59 |
| mriedem | since it uses nova/tests/live_migration/hooks/run_tests.sh which isn't in kilo and liberty | 14:59 |
| mriedem | i would, personally, but i don't like our dog | 14:59 |
| *** moshele has quit IRC | 14:59 | |
| mriedem | sdague: so what does this buy us, having this hook in tree? since it's just a pass through to running tempest | 15:00 |
| mriedem | all of the config still has to happen elsewhere | 15:00 |
| *** tyagiprince has quit IRC | 15:00 | |
| *** dustins has joined #openstack-nova | 15:00 | |
| mriedem | i guess so we don't have another DEVSTACK_GATE_TEMPEST_LIVE_MIGRATION flag in d-g | 15:00 |
| mriedem | s/another/yet a new/ | 15:00 |
| tdurakov_ | mriedem, well, it fair enough to drop it from kilo and liberty until it will be back ported, but what about my hook? | 15:00 |
| mriedem | tdurakov_: i don't know what it's fixing, your commit message doesn't explain that | 15:01 |
| mriedem | i'm assuming it's just fixing the way the args are passed | 15:01 |
| mriedem | and that you're actually running tempest (hence the cd) | 15:01 |
| tdurakov_ | not only, ok will update commit message | 15:01 |
| *** daemontool has joined #openstack-nova | 15:01 | |
| mriedem | but you can't just be like, here is this change i have, new core people, please +W it | 15:01 |
| mriedem | "trust me" | 15:02 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Multinode job for live-migration https://review.openstack.org/245050 | 15:03 |
| *** angdraug has quit IRC | 15:03 | |
| tdurakov_ | mriedem, updated commit message | 15:04 |
| jaypipes | tdurakov_: because it's way too early for me :) | 15:04 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Remove SoftDeleteMixin from BandwidthUsage https://review.openstack.org/240361 | 15:04 |
| *** signed8bit has joined #openstack-nova | 15:04 | |
| tdurakov_ | mriedem, it wasn't about hey, new cores, please merge my patch, btw it was a joke | 15:05 |
| johnthetubaguy | lxsli: thanks for getting that review up there, will try look at that | 15:05 |
| dansmith | tdurakov_: not funny :/ | 15:05 |
| *** klkumar has quit IRC | 15:05 | |
| lxsli | johnthetubaguy: cheers :) | 15:06 |
| tdurakov_ | dansmith, sorry then, there was a smile. don't expect such reaction:( | 15:06 |
| *** subscope has joined #openstack-nova | 15:08 | |
| *** thangp has joined #openstack-nova | 15:08 | |
| openstackgerrit | Paul Murray proposed openstack/nova: docs: update the concept guide shelve actions https://review.openstack.org/246976 | 15:08 |
| lxsli | Cores: I have a bunch of patches (in merge conflict) which convert mox tests to mock. If I fix them, might you look at them, or should I just abandon? | 15:09 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Multinode job for live-migration https://review.openstack.org/245050 | 15:09 |
| mriedem | sdague: you can +W https://review.openstack.org/#/c/245050/ | 15:09 |
| bauzas | tdurakov_: tbh, you should identify who in the core team is more seasoned re: your change, and ping him directly | 15:10 |
| bauzas | tdurakov_: because while I'm able to review that change, it will also take more time for me to ramp up and get context | 15:10 |
| bauzas | in particular since the commit message was pretty brief | 15:11 |
| mriedem | tdurakov_: this filters out the stable branches https://review.openstack.org/#/c/246977/ | 15:11 |
| bauzas | ^that I can +1 because I understand it | 15:12 |
| dansmith | bauzas: alaski: This release note is passing jenkins now: https://review.openstack.org/#/c/245828/ | 15:12 |
| mriedem | +W | 15:12 |
| *** sahid has quit IRC | 15:12 | |
| bauzas | mriedem: https://review.openstack.org/#/c/245050/ maybe a Depend-On https://review.openstack.org/#/c/246977/1 ? | 15:12 |
| * mriedem feels left out | 15:12 | |
| mriedem | bauzas: not necessary | 15:12 |
| mriedem | the hook script is already in nova and the job is already defined | 15:12 |
| mriedem | i.e. it's already busted | 15:12 |
| tdurakov_ | well, this job already here, so no need for deps | 15:12 |
| *** exploreshaifali has quit IRC | 15:12 | |
| bauzas | oh snap | 15:13 |
| bauzas | okay, I see | 15:13 |
| sdague | right, so lets get check experimental run | 15:13 |
| bauzas | +1 | 15:13 |
| bauzas | mriedem: dansmith: so the reno CI job is still blocked because of stable in the weeds | 15:14 |
| dansmith | wait, what? | 15:14 |
| tdurakov_ | sdague, check experimental will ignore my job | 15:14 |
| sdague | tdurakov_: ? | 15:14 |
| dansmith | bauzas: I thought that landed, sorry | 15:14 |
| tdurakov_ | sdague, no change in code | 15:14 |
| lxsli | ok going to abandon those mox patches | 15:14 |
| tdurakov_ | sdague, only in tests folder | 15:14 |
| sdague | oh, because it's the same as in PS 16? | 15:14 |
| dansmith | bauzas: wanna give me the change-id and I'll make this depends-on? | 15:15 |
| bauzas | dansmith: https://review.openstack.org/243320 is still pending because of stable a bit unstable | 15:15 |
| dansmith | okay | 15:15 |
| bauzas | dansmith: and https://review.openstack.org/242027 is depend-on the above | 15:15 |
| tdurakov_ | sdague, i've provided tests results explicitly | 15:15 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 15:15 |
| bauzas | dansmith: I don't feel it's necessary | 15:15 |
| bauzas | dansmith: so my take is | 15:15 |
| sdague | tdurakov_: oh, we're going to need to fix that eventually | 15:15 |
| bauzas | either we hold any +W for reno patches | 15:15 |
| *** mrsoul has quit IRC | 15:15 | |
| bauzas | until https://review.openstack.org/242027 lands | 15:15 |
| bauzas | or | 15:15 |
| *** stevemar_ has joined #openstack-nova | 15:16 | |
| bauzas | we agree to test locally a reno run before +W'ing | 15:16 |
| *** liverpooler has quit IRC | 15:16 | |
| tdurakov_ | sdague, should we? it's functional job, or it will prevent from breaking it? | 15:16 |
| *** andrewbogott has quit IRC | 15:16 | |
| dansmith | bauzas: yeah, I was just going to document the linkage so I don't forget to see that the other lands | 15:16 |
| *** sneti has joined #openstack-nova | 15:16 | |
| sdague | tdurakov_: we need to run live migration tests when we change stuff here | 15:16 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add release note for API DB migration requirements https://review.openstack.org/245828 | 15:16 |
| bauzas | dansmith: I'm fine with the latter, ie. testing on my local venv before +W'ing | 15:16 |
| *** shaohe_feng has quit IRC | 15:16 | |
| sdague | we don't want it getting skipped | 15:16 |
| sdague | so can you figure out what we need in project config to make that happen | 15:17 |
| *** sballe has quit IRC | 15:17 | |
| tdurakov_ | sdague, ok, will add this to project-config | 15:17 |
| bauzas | dansmith: the bonus point being that https://review.openstack.org/242027 will skip a full Tempest pass for reno patches | 15:17 |
| dansmith | bauzas: well, I just added the link for my own records :) | 15:17 |
| *** d34dh0r53 has quit IRC | 15:17 | |
| *** angdraug has joined #openstack-nova | 15:17 | |
| *** Nakato has quit IRC | 15:17 | |
| bauzas | dansmith: that could save a bit of resources | 15:17 |
| *** mc_nair has joined #openstack-nova | 15:17 | |
| *** d34dh0r53 has joined #openstack-nova | 15:17 | |
| bauzas | dansmith: hence the +W being okay but should be very very with caution | 15:17 |
| *** andrewbogott has joined #openstack-nova | 15:18 | |
| *** sballe has joined #openstack-nova | 15:18 | |
| bauzas | dansmith: like accepting a reno-only patch wasting all CI resources is a bit bad IMHO | 15:18 |
| *** ndipanov has quit IRC | 15:18 | |
| *** Nakato has joined #openstack-nova | 15:18 | |
| *** pumaranikar has joined #openstack-nova | 15:18 | |
| *** stevemar_ has quit IRC | 15:18 | |
| *** ndipanov has joined #openstack-nova | 15:19 | |
| dansmith | yep | 15:19 |
| *** markmc has quit IRC | 15:19 | |
| *** mrsoul has joined #openstack-nova | 15:19 | |
| *** markmc has joined #openstack-nova | 15:19 | |
| *** jinxing has quit IRC | 15:19 | |
| *** rubasov has quit IRC | 15:20 | |
| *** sudipto has quit IRC | 15:20 | |
| *** rpodolyaka1 has quit IRC | 15:20 | |
| *** tonytan4ever has joined #openstack-nova | 15:21 | |
| *** rushil has joined #openstack-nova | 15:22 | |
| *** rpodolyaka1 has joined #openstack-nova | 15:22 | |
| *** sahid has joined #openstack-nova | 15:23 | |
| *** sacharya has joined #openstack-nova | 15:23 | |
| *** andrewbogott has quit IRC | 15:23 | |
| *** andrewbogott has joined #openstack-nova | 15:23 | |
| lxsli | thanks for +2 jaypipes! Anyone else for https://review.openstack.org/#/c/215568 please? | 15:25 |
| jaypipes | bauzas: all the python job failures in the base patch in your requestspec series.. are those due to a known issue? | 15:25 |
| *** pratikmallya has joined #openstack-nova | 15:25 | |
| bauzas | jaypipes: sec | 15:25 |
| bauzas | jaypipes: checking | 15:25 |
| *** raildo-afk is now known as raildo | 15:25 | |
| bauzas | jaypipes: the gate was pretty unhappy when I uploaded my series, so I would bet yes, but I'll doublecheck | 15:25 |
| *** pratikma_ has joined #openstack-nova | 15:25 | |
| *** rushil has quit IRC | 15:26 | |
| *** mrkz has joined #openstack-nova | 15:26 | |
| bauzas | jaypipes: oh yeah, the failing unittests, yup it's a known issue | 15:27 |
| *** pradk has joined #openstack-nova | 15:28 | |
| *** sacharya has quit IRC | 15:28 | |
| bauzas | jaypipes: but sec, I'm going to -2 https://review.openstack.org/#/c/197912/ because of the reno file | 15:28 |
| *** ildikov has quit IRC | 15:28 | |
| *** yonglihe has joined #openstack-nova | 15:29 | |
| *** pratikmallya has quit IRC | 15:30 | |
| *** salv-orlando has quit IRC | 15:30 | |
| *** salv-orlando has joined #openstack-nova | 15:31 | |
| bauzas | mriedem: so I litteraly had zero time to work further on https://review.openstack.org/#/c/246635/ | 15:34 |
| mriedem | bauzas: i know what hte issue is, it's in the -dev list | 15:34 |
| mriedem | with options to resolve | 15:34 |
| bauzas | oh ack | 15:34 |
| mriedem | jaypipes wants to hard delete instance_actions when we archive instnaces | 15:34 |
| mriedem | there might be other tables after we get past those though | 15:35 |
| *** tojuvone has joined #openstack-nova | 15:35 | |
| *** edtubill has joined #openstack-nova | 15:35 | |
| mriedem | i'm interested in getting alaski's thoughts on it | 15:35 |
| alaski | mriedem: I responded, though I didn't address hard vs soft delete | 15:36 |
| mriedem | you neatly sidestepped that issue | 15:36 |
| mriedem | which is why i'm calling you out :) | 15:36 |
| alaski | heh | 15:37 |
| *** haomaiwang has joined #openstack-nova | 15:37 | |
| mriedem | "However there is a use case for holding on to the deleted action so that someone could query when or by whom their instance was deleted" | 15:37 |
| mriedem | so we don't have a query API for this | 15:37 |
| mriedem | so if they really wanted those, they could query them out of shadow tables after archive | 15:37 |
| alaski | right | 15:37 |
| mriedem | which sucks, but... | 15:37 |
| alaski | that was a bit of a tangent, I don't think it should affect anything here | 15:38 |
| mriedem | so are you leaning toward soft delete when we delete an instance? | 15:38 |
| *** atuvenie has quit IRC | 15:38 | |
| alaski | well, I'm not sure I see a reason for it really. What's important is what we do when we hard delete or archive an instance | 15:38 |
| bauzas | mriedem: alaski: honestly, I tend to agree with jaypipes | 15:39 |
| bauzas | mriedem: alaski: because I wonder why we should keep the actions if the instance is deleted | 15:39 |
| mriedem | bauzas: well the instance is soft deleted, and then archive puts it in shadow tables, all of which you can still query if needed, | 15:40 |
| bauzas | if that's for tracking purposes, notifications are far better IMHO | 15:40 |
| alaski | when we soft delete an instance I'm not sure I see why it matters if we soft delete the instance-actions or not | 15:40 |
| mriedem | although it's just more pack rat stuff | 15:40 |
| *** stevemar_ has joined #openstack-nova | 15:40 | |
| alaski | though I don't see a reason not to do it | 15:40 |
| mriedem | alaski: heh | 15:40 |
| mriedem | pick a side! | 15:41 |
| bauzas | alaski: mriedem: so I'm okay with soft-deleting an instance, not soft-deleting the actions related to it | 15:41 |
| mriedem | team edward or team jacob! | 15:41 |
| alaski | I would soft-delete | 15:42 |
| bauzas | I'm actually even wondering why we're soft-deleting instances, but that's another question :p | 15:42 |
| *** electrocucaracha has joined #openstack-nova | 15:42 | |
| bauzas | if that's for tracking purpose | 15:42 |
| alaski | the advantage I see for soft delete is being able to easily query for what would be cleaned up on an archive/purge | 15:43 |
| bauzas | alaski: if we soft-delete an instance but hard-delete all the actions associated to it, it's still the same stuff for getting what could be archived | 15:45 |
| bauzas | alaski: because when an instance is deleted, you cannot resurrect it, right? | 15:46 |
| bauzas | so getting all the actions you had for that specific instance when it was alive sounds a bit weird to me | 15:46 |
| *** rmart04 has quit IRC | 15:46 | |
| alaski | bauzas: what about soft-delete? though I'm not sure how that's implemented | 15:47 |
| bauzas | alaski: that would dramatically increase the archive size, nope ? | 15:47 |
| *** xyang1 has joined #openstack-nova | 15:48 | |
| mriedem | alaski: "the advantage I see for soft delete is being able to easily query for what would be cleaned up on an archive/purge" - yeah, it makes archive/purge easier, *but* i'd also have to add some logic in there for handling non-soft deleted instance_actions that are tied to soft deleted instances | 15:49 |
| mriedem | from before the time that we start soft deleting instance_actions - if that's what we do | 15:49 |
| *** mhorban__ has quit IRC | 15:49 | |
| *** mhorban__ has joined #openstack-nova | 15:50 | |
| alaski | bauzas: okay, API soft delete doesn't set the deleted flag on an instance. so yeah, you can't resurrect a db deleted instance. | 15:51 |
| *** ccarmack has left #openstack-nova | 15:51 | |
| alaski | bauzas: you're in favor of hard deleting actions when an instance is set to deleted in the db? | 15:51 |
| bauzas | yup | 15:51 |
| bauzas | alaski: wait, what? | 15:52 |
| bauzas | alaski: you said 'API soft-delete doesn't do deleted=instance_id in the DB ? | 15:52 |
| alaski | bauzas: not that I saw | 15:52 |
| alaski | instance.destroy is called in terminate_instance on compute, which soft-delete doesn't call | 15:53 |
| bauzas | alaski: so it keeps the virt files, but still puts the deleted flag in the DB right? | 15:53 |
| bauzas | alaski: so soft-delete is just a sync call to modify the state, right? | 15:54 |
| *** annasort_ has quit IRC | 15:54 | |
| alaski | mriedem: this is why I don't have a strong opinion on soft-deleting on instance delete. I don't see a strong advantage either way. | 15:54 |
| mriedem | alaski: ok, want to reply with those details to the ML? | 15:56 |
| mriedem | i have a meeting then can get back into this | 15:56 |
| *** mriedem is now known as mriedem_meeting | 15:56 | |
| mriedem_meeting | the purge stuff that ctrath is working on is going to have to deal with this also https://review.openstack.org/#/c/203751/ | 15:56 |
| *** jerrygb has joined #openstack-nova | 15:57 | |
| alaski | mriedem_meeting: sure | 15:57 |
| alaski | bauzas: soft-delete seems to just set the vm_state to soft-delete and shut down the instance. But it's not something we offer so I haven't spent much time looking at it before so I could be missing something | 15:57 |
| *** daemontool has quit IRC | 15:58 | |
| *** dustins has quit IRC | 15:58 | |
| *** dustins has joined #openstack-nova | 15:58 | |
| *** daemontool has joined #openstack-nova | 15:59 | |
| bauzas | alaski: oh fun https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L751-L760 | 15:59 |
| *** yamahata has joined #openstack-nova | 15:59 | |
| *** mgoddard has quit IRC | 15:59 | |
| *** moshele has joined #openstack-nova | 16:00 | |
| *** gjayavelu has quit IRC | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** tdurakov_ has quit IRC | 16:01 | |
| *** tdurakov_ has joined #openstack-nova | 16:01 | |
| *** tdurakov_ has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** pratikma_ is now known as pratikmallya | 16:02 | |
| *** alex_klimov has quit IRC | 16:02 | |
| *** mdrabe has quit IRC | 16:03 | |
| *** mhorban__ has quit IRC | 16:04 | |
| *** mjura has quit IRC | 16:05 | |
| *** paul-carlton1 has quit IRC | 16:06 | |
| *** hemnafk is now known as hemna | 16:06 | |
| *** rushil has joined #openstack-nova | 16:08 | |
| bauzas | mmm, http://developer.openstack.org/api-ref-compute-v2.1.html#restore doesn't seem to match https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/deferred_delete.py right? | 16:08 |
| *** erlon has quit IRC | 16:08 | |
| *** READ10 has joined #openstack-nova | 16:09 | |
| *** rcernin has quit IRC | 16:09 | |
| mdbooth | bauzas: Are you aware of a doc describing the purpose of soft delete, btw? Don't worry if not. I've been wondering about this lately. Seems like it's mostly just a source of pain. | 16:09 |
| bauzas | mdbooth: like I understand, it's just a way to mark an instance deleted in the Nova DB without notifying the driver - but I'm just checking that | 16:10 |
| mdbooth | Ah... I'm getting confused about my soft deletes | 16:10 |
| mdbooth | I was thinking of db records. | 16:11 |
| mdbooth | I get the purpose of the other type. | 16:11 |
| *** wuhg has quit IRC | 16:11 | |
| sdague | mdbooth: mostly it's because the database was very normalized | 16:12 |
| sdague | and there are fk between object types | 16:12 |
| sdague | so if you delete a flavor, for instance, how do you display the flavor info in an instance | 16:13 |
| bauzas | mdbooth: oh, sec, that's more complex than what I said | 16:13 |
| sdague | which, granted is now fixed | 16:13 |
| *** jaypipes has quit IRC | 16:13 | |
| *** sayali has quit IRC | 16:13 | |
| cfriesen__ | question for people...why does the os-migrateLive API have the caller specifying "block_migration"? Shouldn't nova be able to figure that out on its own? | 16:13 |
| *** sahid has quit IRC | 16:13 | |
| sdague | but the entire object model has some implicit keys that aren't enforced | 16:13 |
| mdbooth | 'some' :/ | 16:14 |
| mdbooth | sdague: Yeah, I guessed it would be something like that, thanks. | 16:14 |
| ndipanov | cfriesen__, it should and we should remove it imho | 16:14 |
| *** shaohe_feng has joined #openstack-nova | 16:15 | |
| bauzas | mdbooth: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2565-L2570 | 16:16 |
| bauzas | mdbooth: so that's a full nova delete | 16:16 |
| dansmith | bauzas: that's not the soft delete that alaski is talking about right? | 16:17 |
| dansmith | oh, it is | 16:17 |
| dansmith | nevermind me | 16:17 |
| *** moshele has quit IRC | 16:17 | |
| bauzas | dansmith: well, there is no soft-delete API method AFAICT | 16:17 |
| dansmith | correct | 16:17 |
| bauzas | dansmith: there is only a delete that can be a soft delete if the operator asks for a delayed delete | 16:17 |
| mdbooth | dansmith: Glad it's not just me :) | 16:18 |
| dansmith | you guys went from db purged to soft deleted | 16:18 |
| dansmith | and I didn't make the transition with you :) | 16:18 |
| *** burgerk has joined #openstack-nova | 16:18 | |
| bauzas | well, I'm caring of soft-delete because of the restore action | 16:18 |
| bauzas | because it *is* possible to restore a soft-deleted instance | 16:18 |
| bauzas | which means that it should be idempotent in terms of instance actions | 16:19 |
| bauzas | that leads me to : | 16:19 |
| *** wolsen has joined #openstack-nova | 16:19 | |
| bauzas | mriedem_meeting: alaski: nevermind my position, we need to soft-delete instance actions too | 16:19 |
| *** edmondsw has quit IRC | 16:19 | |
| *** mpavone has quit IRC | 16:20 | |
| *** mdrabe has joined #openstack-nova | 16:20 | |
| *** fawadkhaliq has quit IRC | 16:20 | |
| *** dims_ has joined #openstack-nova | 16:22 | |
| *** flip214 is now known as flip211 | 16:23 | |
| *** jwcroppe_ has quit IRC | 16:24 | |
| *** dims has quit IRC | 16:24 | |
| *** belmoreira has quit IRC | 16:24 | |
| *** rpodolyaka1 has quit IRC | 16:25 | |
| *** exploreshaifali has joined #openstack-nova | 16:25 | |
| *** alexschm has quit IRC | 16:25 | |
| *** sayali has joined #openstack-nova | 16:26 | |
| *** rpodolyaka1 has joined #openstack-nova | 16:26 | |
| openstackgerrit | Paul Murray proposed openstack/nova: docs: describe migration and other movement concepts https://review.openstack.org/247016 | 16:26 |
| bauzas | mriedem_meeting: alaski: oh snap, nevermind, do_soft_delete() never calls instance.destroy() which means that when it's called, there is no way to revert its state => now I understand your point, alaski | 16:26 |
| mdbooth | Hmm, that's a wart. Was just looking to see why a resize operation required converting the disk image to raw and back. Turns out it's because we run resize2fs directly on the image file. We could do that with libguestfs and save the double copy. | 16:26 |
| alaski | bauzas: yeah. I'm in a meeting as well now so I'm not following here much | 16:27 |
| bauzas | alaski: np, I went thru code to understand the soft delete action | 16:28 |
| *** mhorban__ has joined #openstack-nova | 16:28 | |
| *** edmondsw has joined #openstack-nova | 16:28 | |
| PaulMurray | johnthetubaguy, alex_xu a couple of concept doc patches for shelve and migrations: https://review.openstack.org/246976 https://review.openstack.org/247016 | 16:29 |
| bauzas | and at the end, that doesn't change my position of 'if an instance is API deleted, it's soft-deleted in DB but with no way to resurrect it, so we can drop the actions related to it' | 16:29 |
| *** sneti has quit IRC | 16:29 | |
| PaulMurray | alaski, you might be interested in the second one ^^^ | 16:29 |
| mdbooth | Do we have a policy on libguestfs, btw? We use it elsewhere in Nova, but iirc it has a wrapper round it for if it's not available. Is libguestfs ever not available? | 16:29 |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| *** ssurana has joined #openstack-nova | 16:31 | |
| *** mgoddard has joined #openstack-nova | 16:31 | |
| danpb | mdbooth: non-KVM hypervisors for example | 16:32 |
| danpb | lxc, xen, vz | 16:32 |
| *** annegentle has quit IRC | 16:33 | |
| *** sneti has joined #openstack-nova | 16:33 | |
| mdbooth | All of those could use libguestfs, surely? | 16:33 |
| mdbooth | The VMware driver might have trouble, though... | 16:33 |
| *** annegentle has joined #openstack-nova | 16:33 | |
| mdbooth | But in that case, what would calling resize2fs on it do? | 16:33 |
| mdbooth | This is some weird code. | 16:34 |
| ndipanov | hey johnthetubaguy can you look at this https://review.openstack.org/#/c/207385/ | 16:34 |
| mdbooth | I mean, the VMware driver could still run libguestfs, but the compute node doesn't have access to the disks. | 16:34 |
| ndipanov | it's been stuck in the queue for very long for no reason :/ | 16:34 |
| *** sahid has joined #openstack-nova | 16:35 | |
| danpb | mdbooth: if any of those tried to use libguestfs it'd at best be falling back to running qemu w/ tcg | 16:35 |
| mdbooth | danpb: Right. Is that a problem? | 16:35 |
| danpb | apart from the general slowness | 16:35 |
| mdbooth | It's not generally that awful, iirc, especially if you just doing a bunch of io. | 16:36 |
| *** sahid has quit IRC | 16:36 | |
| mdbooth | Like resizing a disk partition. | 16:36 |
| *** suro-patz has joined #openstack-nova | 16:37 | |
| mdbooth | It's going to be less awful than converting to raw and back, though. Although, its weird that this code is calling qemu-img. | 16:38 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 16:38 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add infra for versioned notifications https://review.openstack.org/247024 | 16:38 |
| mdbooth | As it's not in the libvirt driver | 16:38 |
| *** Piet has joined #openstack-nova | 16:38 | |
| *** ijuwang has quit IRC | 16:38 | |
| *** kfarr has joined #openstack-nova | 16:39 | |
| *** Marga_ has quit IRC | 16:40 | |
| *** Marga_ has joined #openstack-nova | 16:42 | |
| *** sahid has joined #openstack-nova | 16:44 | |
| *** mgoddard_ has joined #openstack-nova | 16:45 | |
| *** sacharya has joined #openstack-nova | 16:45 | |
| *** ijuwang has joined #openstack-nova | 16:45 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Replaced deprecated timeutils methods https://review.openstack.org/241179 | 16:45 |
| *** rpodolyaka1 has quit IRC | 16:45 | |
| *** EinstCrazy has quit IRC | 16:46 | |
| *** jwcroppe has joined #openstack-nova | 16:46 | |
| *** mgoddard has quit IRC | 16:48 | |
| *** scheuran has quit IRC | 16:49 | |
| *** mhorban__ has quit IRC | 16:49 | |
| *** pumaranikar has quit IRC | 16:50 | |
| *** pumaranikar has joined #openstack-nova | 16:50 | |
| *** haomaiwang has quit IRC | 16:51 | |
| *** sudipto has joined #openstack-nova | 16:52 | |
| *** rpodolyaka1 has joined #openstack-nova | 16:52 | |
| *** otter768 has joined #openstack-nova | 16:53 | |
| *** jaypipes has joined #openstack-nova | 16:53 | |
| *** rpodolyaka1 has quit IRC | 16:53 | |
| *** irenab has quit IRC | 16:54 | |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Allow saving empty pci_device_pools in ComputeNode object https://review.openstack.org/216049 | 16:54 |
| *** fawadkhaliq has joined #openstack-nova | 16:54 | |
| *** fawadkhaliq has quit IRC | 16:55 | |
| *** fawadkhaliq has joined #openstack-nova | 16:55 | |
| *** rpodolyaka1 has joined #openstack-nova | 16:56 | |
| *** otter768 has quit IRC | 16:57 | |
| *** belmoreira has joined #openstack-nova | 16:58 | |
| *** stevemar_ has quit IRC | 16:59 | |
| *** tongli has joined #openstack-nova | 17:00 | |
| *** tdurakov_ has joined #openstack-nova | 17:00 | |
| *** tdurakov_ has quit IRC | 17:00 | |
| *** e0ne has quit IRC | 17:01 | |
| *** penick has joined #openstack-nova | 17:02 | |
| *** Sree has joined #openstack-nova | 17:04 | |
| *** diana_clarke has quit IRC | 17:05 | |
| *** jlanoux has quit IRC | 17:06 | |
| *** unicell has joined #openstack-nova | 17:07 | |
| *** matrohon has quit IRC | 17:07 | |
| *** unicell1 has quit IRC | 17:07 | |
| *** moshele has joined #openstack-nova | 17:07 | |
| *** mriedem_meeting is now known as mriedem | 17:08 | |
| *** sahid has quit IRC | 17:10 | |
| mriedem | cfriesen__: there is already a spec to remove block_migration from the internal live migration stuff | 17:10 |
| *** sileht has quit IRC | 17:10 | |
| *** ctrath has joined #openstack-nova | 17:10 | |
| *** electrocucaracha has quit IRC | 17:11 | |
| *** sileht has joined #openstack-nova | 17:11 | |
| *** _diana_ has joined #openstack-nova | 17:11 | |
| *** jistr has quit IRC | 17:11 | |
| *** _diana_ is now known as diana_clarke | 17:11 | |
| *** suro-patz has quit IRC | 17:12 | |
| *** romainh has quit IRC | 17:15 | |
| *** ociuhandu has quit IRC | 17:17 | |
| *** daemontool has quit IRC | 17:20 | |
| mriedem | cfriesen__: https://blueprints.launchpad.net/nova/+spec/remove-shared-storage-flag-in-evacuate-api | 17:20 |
| *** browne has joined #openstack-nova | 17:20 | |
| mriedem | gibi_: btw, someone in the ops list did ask about deprecating the nova-manage service commands http://lists.openstack.org/pipermail/openstack-operators/2015-November/008860.html | 17:21 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Add log warning hints in sqlalchemy/api.py https://review.openstack.org/246901 | 17:21 |
| *** exploreshaifali has quit IRC | 17:22 | |
| mriedem | bauzas: so did you make up your mind on soft deleting instance_actions? | 17:22 |
| mriedem | btw, by soft delete i'm not talking about the api | 17:22 |
| bauzas | mriedem: yup | 17:22 |
| mriedem | i'm talking about instance_actions.deleted != 0 | 17:22 |
| bauzas | mriedem: yeah, but I wanted to see what was the stuff with restore | 17:22 |
| bauzas | mriedem: hence me soliloquing in the channel | 17:23 |
| *** dguitarbite has quit IRC | 17:23 | |
| bauzas | hah, bad translation | 17:23 |
| mriedem | i get it | 17:23 |
| mriedem | talking to yourself? | 17:23 |
| *** moshele has quit IRC | 17:23 | |
| *** nkrinner has quit IRC | 17:23 | |
| *** tyagiprince has joined #openstack-nova | 17:23 | |
| bauzas | yup | 17:23 |
| *** exploreshaifali has joined #openstack-nova | 17:23 | |
| bauzas | soliloquizing | 17:23 |
| mriedem | i took high school english class... | 17:23 |
| *** dguitarbite has joined #openstack-nova | 17:24 | |
| *** nkrinner has joined #openstack-nova | 17:24 | |
| melwitt | me too, I remember that, soliloquy | 17:24 |
| bauzas | mriedem: so the outcome is, there is no way to resurrect a soft-deleted instance in DB | 17:24 |
| bauzas | mriedem: which means that when it's deleted, it's deleted | 17:25 |
| *** angdraug has quit IRC | 17:25 | |
| *** lpetrut has quit IRC | 17:25 | |
| bauzas | mriedem: in that case, I don't see the point of getting the list of actions | 17:25 |
| *** belmoreira has quit IRC | 17:25 | |
| bauzas | because you deleted that instance, it's not like if it was in error state and you needed to investigate | 17:25 |
| melwitt | bauzas mriedem: on that I was thinking the instance_actions could be some historical data on a deleted instance however IIRC if you actually try to show instance_actions for a deleted instance it will show you nothing, ironically | 17:25 |
| melwitt | because it tries to do an instance lookup first in the api | 17:25 |
| bauzas | the contract is "go f*ck, I'm gonna recreate a fresh new instance, I don't care about that one particularly" | 17:26 |
| bauzas | melwitt: that's the usecase I don't understand | 17:26 |
| mriedem | you care if it was your pet | 17:26 |
| bauzas | shhhhht | 17:26 |
| mriedem | right, but you don't care about people that care about pets :) | 17:27 |
| *** gszasz has quit IRC | 17:27 | |
| bauzas | mriedem: if it's a pet, you have something like notifications and AODH ? | 17:27 |
| alaski | melwitt: that could be useful. I'm not sure if it's worth adding the semantics to the API to do that since there's talk of deprecating instance-actions with tasks | 17:27 |
| *** Sree has quit IRC | 17:27 | |
| mriedem | bauzas: if you're using those, yeah | 17:27 |
| melwitt | I'm thinking of things like, someone deleted an instance in a tenant and someone else in the tenant asks ops, who deleted that instance? instance_actions would have that, I *think* | 17:27 |
| *** tyagiprince has quit IRC | 17:27 | |
| mriedem | melwitt: it owuld | 17:27 |
| mriedem | *would | 17:27 |
| mriedem | stores the action and user that performed the action | 17:28 |
| bauzas | melwitt: again, that's a notification query I feel | 17:28 |
| melwitt | yeah, I realize that notifications is supposed to cover all of the instance_actions usecases for forensic data. I am not sure if it's in a great state by now that we can sort of get rid of instance_actions use in that way? | 17:28 |
| *** changbl has joined #openstack-nova | 17:29 | |
| bauzas | melwitt: well, that's the point | 17:29 |
| mriedem | also, where are people storing notifications and for how long | 17:29 |
| bauzas | mriedem: that's certainly something we can explain to operatoes | 17:30 |
| *** ildikov has joined #openstack-nova | 17:30 | |
| bauzas | mriedem: on the other hand, archiving the full list of actions for a deleted instance increases dramatically the size of the archive, nope ? | 17:30 |
| bauzas | if we consider that *all* instance actions are covered - which is not the case atm | 17:31 |
| *** stevemar_ has joined #openstack-nova | 17:31 | |
| *** tyagiprince has joined #openstack-nova | 17:32 | |
| bauzas | but I feel you're considering me opiniated while honestly I defer that point to operatoes | 17:32 |
| mriedem | bauzas: yes it increases the size of the archive, but it's not a change in the actual DB since the rows are moved to shadow tables | 17:32 |
| mriedem | yeah, so i was going to bring this up in ops in a bit | 17:32 |
| mriedem | was looking at stacktach first, which i've never used | 17:32 |
| bauzas | mriedem: well, the telemetry team delivered a new backend for storing timestamp-related info, called aodh | 17:33 |
| *** unicell has quit IRC | 17:33 | |
| bauzas | I heard some ops feedback that it was filling up the hole | 17:33 |
| alaski | mriedem: we rely on stacktach for storage, but I don't know our retention policy. I don't think it goes beyond 90 days though. | 17:33 |
| bauzas | nah | 17:33 |
| *** tonytan4ever has quit IRC | 17:33 | |
| bauzas | not aodh, gnocchi - snap | 17:34 |
| bauzas | meh, the retention is a fair point | 17:34 |
| bauzas | that said, when I was ops, I was usually doing forensics with logs not more than 90 days ago | 17:35 |
| bauzas | I mean, apache logs and other stuff | 17:35 |
| mriedem | i thought gnocchi was metrics | 17:36 |
| mriedem | honestly, it'd be nice if these projects could have a name that remotely resembes wtf they do | 17:36 |
| bauzas | mriedem: like nova ? :D | 17:36 |
| melwitt | heh | 17:36 |
| mriedem | i know | 17:36 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Config options: centralize section "scheduler" https://review.openstack.org/245891 | 17:37 |
| openstackgerrit | Ed Leafe proposed openstack/nova: config options: centralize section "serial_console" https://review.openstack.org/244177 | 17:37 |
| bauzas | meh, neither gordc nor jd are around | 17:37 |
| mriedem | ah http://docs.openstack.org/developer/gnocchi/rest.html#resources | 17:37 |
| mriedem | so you store an 'instance' resource in gnocchi | 17:38 |
| *** mwagner has quit IRC | 17:38 | |
| *** yassine__ has quit IRC | 17:38 | |
| *** mgoddard has joined #openstack-nova | 17:38 | |
| *** mgoddard_ has quit IRC | 17:38 | |
| bauzas | okay, I don't want to overthink about notifications | 17:39 |
| dansmith | alaski: question for you on https://review.openstack.org/#/c/211753/33 | 17:39 |
| mriedem | ok, so i guess we punt and say we hard delete instance_actions during archive/purge of soft deleted instances, if you want instance_actions stored elsewhere, hook up stacktach or gnocchi or $custom_notification_listener_storage_thingy | 17:39 |
| bauzas | if operators think it's a good idea to keep the actions as longer as the instance in DB, then I'm fine | 17:39 |
| PaulMurray | ndipanov, I saw this spec on migration with PCI device and wondered if it overlaps with anything you have been doing? | 17:40 |
| PaulMurray | ndipanov, https://review.openstack.org/#/c/239123 | 17:40 |
| bauzas | mriedem: do a x-post with -ops@ I guess | 17:40 |
| *** tyagiprince1 has joined #openstack-nova | 17:40 | |
| mriedem | bauzas: yup, doing that now | 17:40 |
| ndipanov | PaulMurray, that came out of a patch discussion recently I think | 17:40 |
| ndipanov | PaulMurray, it's complementary | 17:40 |
| *** tyagiprince has quit IRC | 17:41 | |
| *** tyagiprince1 is now known as tyagiprince | 17:41 | |
| *** armax has joined #openstack-nova | 17:41 | |
| * bauzas puts his parent hat on | 17:41 | |
| bauzas | ttyl | 17:41 |
| PaulMurray | ndipanov, I was going to review it - is there anything related I should look at to understand how its related? | 17:42 |
| * PaulMurray not sur eI make sense in the evening | 17:42 | |
| *** garyk1 has joined #openstack-nova | 17:45 | |
| *** garyk has quit IRC | 17:45 | |
| *** claudiub has quit IRC | 17:46 | |
| *** tdurakov_ has joined #openstack-nova | 17:47 | |
| *** exploreshaifali has quit IRC | 17:47 | |
| *** exploreshaifali has joined #openstack-nova | 17:48 | |
| mriedem | what was that other M project that competed with ceilometer? | 17:48 |
| mriedem | monasca? | 17:48 |
| melwitt | yeah | 17:48 |
| dims_ | y that's the one | 17:48 |
| *** electrocucaracha has joined #openstack-nova | 17:50 | |
| *** electrocucaracha has quit IRC | 17:50 | |
| mriedem | m_not_ceilometer | 17:50 |
| mriedem | let's rename it | 17:50 |
| *** aix has quit IRC | 17:51 | |
| *** pratikmallya has quit IRC | 17:52 | |
| *** shaohe_feng1 has joined #openstack-nova | 17:52 | |
| mriedem | alaski: heh, we posted at the same time | 17:52 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Config options: centralize section "scheduler" https://review.openstack.org/245891 | 17:52 |
| openstackgerrit | Ed Leafe proposed openstack/nova: config options: centralize section "serial_console" https://review.openstack.org/244177 | 17:52 |
| *** electrocucaracha has joined #openstack-nova | 17:52 | |
| *** electrocucaracha has quit IRC | 17:53 | |
| *** shaohe_feng1 has quit IRC | 17:54 | |
| *** electrocucaracha has joined #openstack-nova | 17:54 | |
| *** shaohe_feng has quit IRC | 17:54 | |
| *** jerrygb_ has joined #openstack-nova | 17:54 | |
| *** stackdump has joined #openstack-nova | 17:54 | |
| *** jerrygb_ has quit IRC | 17:54 | |
| *** jerrygb_ has joined #openstack-nova | 17:55 | |
| *** apoorvad has joined #openstack-nova | 17:56 | |
| *** tyagiprince1 has joined #openstack-nova | 17:56 | |
| *** tdurakov_ has quit IRC | 17:57 | |
| *** jerrygb has quit IRC | 17:57 | |
| *** EinstCrazy has joined #openstack-nova | 17:58 | |
| *** tdurakov_ has joined #openstack-nova | 17:59 | |
| *** jerrygb_ has quit IRC | 17:59 | |
| *** tyagiprince has quit IRC | 17:59 | |
| *** tyagiprince1 is now known as tyagiprince | 17:59 | |
| *** achanda has quit IRC | 18:00 | |
| alaski | dansmith: commenting on the db index, but testing showed that it does work. I messed it up the first time and migration tests yelled at me and don't any longer | 18:00 |
| alaski | dansmith: on https://review.openstack.org/#/c/211753/33/nova/objects/request_spec.py,cm I think I'm doing what you're saying on line 327 | 18:01 |
| *** jerrygb_ has joined #openstack-nova | 18:01 | |
| *** jerrygb_ has quit IRC | 18:01 | |
| *** jerrygb_ has joined #openstack-nova | 18:01 | |
| alaski | mriedem: nice. you at least cross-posted though | 18:02 |
| *** achanda has joined #openstack-nova | 18:02 | |
| *** achanda has quit IRC | 18:03 | |
| raildo | Hey guys, I'm with a doubt here about default quotas... There is a API call for default quotas for tenant http://developer.openstack.org/api-ref-compute-v2.1.html#showdefaultquotas | 18:04 |
| dansmith | alaski: hmm, okay | 18:04 |
| raildo | Is there a default quota for the service, and a default quota for every project? | 18:04 |
| dansmith | alaski: oh, right, so why are you iterating the fields again? | 18:04 |
| *** belmoreira has joined #openstack-nova | 18:04 | |
| *** EinstCrazy has quit IRC | 18:05 | |
| *** daemontool has joined #openstack-nova | 18:05 | |
| *** belmoreira has quit IRC | 18:05 | |
| *** tyagiprince has quit IRC | 18:06 | |
| *** tyagiprince1 has joined #openstack-nova | 18:06 | |
| *** PaulMurray is now known as ptm_away | 18:07 | |
| *** tyagiprince1 is now known as tyagiprince | 18:08 | |
| *** READ10 has quit IRC | 18:10 | |
| *** tdurakov_ has quit IRC | 18:11 | |
| *** tdurakov_ has joined #openstack-nova | 18:11 | |
| *** gjayavelu has joined #openstack-nova | 18:12 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: set base_dir as a property https://review.openstack.org/209912 | 18:13 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: imagecache: implement listing of cached images https://review.openstack.org/209913 | 18:13 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: virt: add method for obtaining the list of cached images https://review.openstack.org/209914 | 18:13 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: implement get_cached_images method https://review.openstack.org/209915 | 18:13 |
| *** Marga_ has quit IRC | 18:13 | |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Cancel ongoing live migrations https://review.openstack.org/228828 | 18:13 |
| alaski | dansmith: because I used an intermediate object. I see now, you're recommending not doing that | 18:13 |
| dansmith | yeah, just load-n-go (tm) | 18:14 |
| alaski | heh. just need to pull id and instance_uuid | 18:14 |
| dansmith | why? | 18:14 |
| dansmith | they're on the object already no? | 18:14 |
| dansmith | I mean, id I guess | 18:14 |
| dansmith | although it will be in the blob already if you've saved it more than once | 18:15 |
| *** e0ne has joined #openstack-nova | 18:15 | |
| *** tonytan4ever has joined #openstack-nova | 18:15 | |
| alaski | yeah, that's right. I should stop looking at this while distracted | 18:16 |
| *** daemontool has quit IRC | 18:16 | |
| *** gjayavelu has quit IRC | 18:16 | |
| dansmith | alaski: I thought his name was more conventional, like "luke" or something | 18:17 |
| *** signed8b_ has joined #openstack-nova | 18:17 | |
| *** bkopilov has joined #openstack-nova | 18:17 | |
| *** suro-patz has joined #openstack-nova | 18:17 | |
| *** jerrygb_ has quit IRC | 18:17 | |
| bkopilov | Hi Experts, need help | 18:17 |
| alaski | dansmith: lol | 18:18 |
| dansmith | :) | 18:18 |
| *** pratikmallya has joined #openstack-nova | 18:18 | |
| *** danpb has quit IRC | 18:18 | |
| bkopilov | i have test image deactivate / activate and boot instance , this test fails only on citrix xen only | 18:18 |
| *** unicell has joined #openstack-nova | 18:18 | |
| *** signed8bit has quit IRC | 18:19 | |
| bkopilov | could you please check and let me know if its a bug - https://review.openstack.org/#/c/245519/ | 18:20 |
| *** njirap has joined #openstack-nova | 18:21 | |
| bkopilov | here is the bug https://bugs.launchpad.net/nova/+bug/1517367 | 18:21 |
| openstack | Launchpad bug 1517367 in OpenStack Compute (nova) "unable to boot instance after image reactivate with xenapi citrix" [Undecided,New] | 18:21 |
| *** fawadkhaliq has quit IRC | 18:21 | |
| *** atuvenie has joined #openstack-nova | 18:21 | |
| *** tyagiprince has quit IRC | 18:22 | |
| *** tyagiprince has joined #openstack-nova | 18:23 | |
| *** ociuhandu has joined #openstack-nova | 18:24 | |
| *** ihrachys has quit IRC | 18:24 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 18:27 |
| *** mwagner has joined #openstack-nova | 18:27 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE NFS setup for multinode job https://review.openstack.org/247081 | 18:27 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 18:28 |
| *** vilobhmm has joined #openstack-nova | 18:28 | |
| *** rpodolyaka1 has quit IRC | 18:28 | |
| *** annegentle has quit IRC | 18:30 | |
| *** tyagiprince1 has joined #openstack-nova | 18:31 | |
| *** rpodolyaka1 has joined #openstack-nova | 18:31 | |
| *** rpodolyaka1 has quit IRC | 18:31 | |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Cancel ongoing live migrations https://review.openstack.org/228828 | 18:32 |
| tdurakov_ | sdague, ping, need for project-config tests enablement for live-migration job | 18:32 |
| *** rpodolyaka1 has joined #openstack-nova | 18:32 | |
| tdurakov_ | *need advice | 18:32 |
| *** tyagiprince has quit IRC | 18:33 | |
| *** tyagiprince1 is now known as tyagiprince | 18:33 | |
| *** jaypipes has quit IRC | 18:34 | |
| *** tonytan4ever has quit IRC | 18:37 | |
| andrewbogott | bauzas: I’d appreciate another look at https://review.openstack.org/#/c/242251/ when you have the time. (Also I could use guidance through the CI maze… currently can’t tell if my patch is buggy or the tests are buggy.) | 18:38 |
| *** sudipto has quit IRC | 18:38 | |
| mriedem | raildo: default quota per project | 18:38 |
| mriedem | raildo: not sure what you mean by service | 18:38 |
| mriedem | the quotas are on projects/users and resources | 18:38 |
| mriedem | andrewbogott: recheck, that unit test thing was fixed late yesterday | 18:40 |
| andrewbogott | mriedem: ok! | 18:41 |
| *** lucasagomes is now known as lucas-dinner | 18:41 | |
| *** thangp has quit IRC | 18:42 | |
| mriedem | dansmith: sounds like cern is moving to kilo next week, | 18:44 |
| *** signed8b_ is now known as signed8bit_ZZZzz | 18:44 | |
| mriedem | dansmith: and don't have a ton of conductors either :) | 18:44 |
| *** jerrygb has joined #openstack-nova | 18:44 | |
| dansmith | where? | 18:45 |
| mriedem | http://lists.openstack.org/pipermail/openstack-operators/2015-November/008858.html | 18:45 |
| mriedem | 16-48 conductor workers with a max of 200 computes | 18:46 |
| *** obondarev has quit IRC | 18:46 | |
| *** signed8bit_ZZZzz is now known as signed8b_ | 18:47 | |
| dansmith | if they're not already concerned with cpu usage, then I expect they'll be okay | 18:47 |
| openstackgerrit | Corey Wright proposed openstack/nova: Prevent redundant instance.update notifications https://review.openstack.org/245376 | 18:47 |
| dansmith | although they may want a few extra nodes for the initial bump | 18:47 |
| raildo | mriedem: when I create a project, nova will update the project quota using the default quota, right? | 18:48 |
| dansmith | sounds like it was about 4% normal overhead to have that code enabled for the makeup of the other one, so really not that big | 18:48 |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| mriedem | raildo: create a project in keystone? | 18:48 |
| raildo | mriedem: yes, and later nova put the quota information about this project in the quota table | 18:49 |
| raildo | maybe, I'm creating a confusion here... | 18:49 |
| mriedem | default quotas are created on first access yes | 18:50 |
| *** erlon has joined #openstack-nova | 18:50 | |
| *** pratikma_ has joined #openstack-nova | 18:50 | |
| mriedem | the update API actually does the create https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/quota_sets.py#L140 | 18:50 |
| mriedem | and falls back to update if create fails due to the quota record already existing | 18:51 |
| mriedem | somewhere in that mess the default quota class is used | 18:51 |
| raildo | haha | 18:51 |
| raildo | mriedem: ok, got it... thanks! | 18:51 |
| melwitt | I thought default quotas were in absence of a set quota. like if you haven't yet set quotas for a project, the defaults are returned | 18:52 |
| *** thangp has joined #openstack-nova | 18:52 | |
| *** jwcroppe_ has joined #openstack-nova | 18:52 | |
| *** pratikmallya has quit IRC | 18:52 | |
| mriedem | melwitt: yeah i think that's right | 18:52 |
| mriedem | v | 18:52 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/quota.py#L182 | 18:52 |
| melwitt | I didn't think it actually sets the defaults | 18:52 |
| melwitt | your project will not have any quota entry until you set quotas | 18:53 |
| mriedem | this gets down in there https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/quota_sets.py#L115 | 18:53 |
| mriedem | get_settable_quotas gets the default quotas | 18:53 |
| mriedem | if you don't pass anything in, it just uses the defaults | 18:53 |
| raildo | melwitt: but if didn't update the quota before create a intance, for example, the default quota will be used as the quota limit for this project | 18:53 |
| *** otter768 has joined #openstack-nova | 18:53 | |
| melwitt | raildo: right | 18:54 |
| *** lykinsbd_ has joined #openstack-nova | 18:54 | |
| *** lykinsbd_ has quit IRC | 18:54 | |
| melwitt | I'm just saying I don't think your project will have a quota record in the table unless you call the quota update api first | 18:54 |
| *** rpodolyaka1 has quit IRC | 18:54 | |
| mriedem | or the nova-manage quota update command | 18:54 |
| *** salv-orl_ has joined #openstack-nova | 18:54 | |
| mriedem | or whatever it is | 18:54 |
| *** salv-orl_ has quit IRC | 18:54 | |
| melwitt | okay | 18:54 |
| mriedem | which we shoujd probably deprecate... | 18:54 |
| raildo | melwitt: right | 18:54 |
| *** tyagiprince1 has joined #openstack-nova | 18:54 | |
| *** lykinsbd_ has joined #openstack-nova | 18:54 | |
| melwitt | cool, sorry if I needlessly said stuff :) | 18:55 |
| mriedem | no it's good, | 18:55 |
| *** tonytan4ever has joined #openstack-nova | 18:55 | |
| mriedem | because like i said, somewhere in the mess, | 18:55 |
| raildo | can we have more than one "default quota" in nova? | 18:55 |
| *** jwcroppe has quit IRC | 18:55 | |
| mriedem | that is a good question | 18:56 |
| raildo | like default quota flavors? :P | 18:56 |
| mriedem | i think there is only one default quota class | 18:56 |
| *** tyagiprince has quit IRC | 18:57 | |
| *** tyagiprince1 is now known as tyagiprince | 18:57 | |
| *** lykinsbd has quit IRC | 18:57 | |
| mriedem | quota class is global | 18:57 |
| *** salv-orlando has quit IRC | 18:57 | |
| mriedem | and there is only one, 'default' | 18:57 |
| mriedem | at least upstream | 18:57 |
| mriedem | rax might have some custom quota class mumbo jumbo out of tree | 18:57 |
| melwitt | yeah, that's what I'm seeing in the db api | 18:58 |
| raildo | right :) | 18:58 |
| mriedem | everything else for quotas is per project | 18:58 |
| *** otter768 has quit IRC | 18:58 | |
| *** lykinsbd has joined #openstack-nova | 18:59 | |
| mriedem | so if you do nova quota-class-update --instance 20 default | 18:59 |
| mriedem | that changes the default quota for instances to 20 globally for any new project that's created | 18:59 |
| *** rushil has quit IRC | 18:59 | |
| *** obondarev has joined #openstack-nova | 19:00 | |
| *** lykinsbd_ has quit IRC | 19:00 | |
| *** doug-fis_ has joined #openstack-nova | 19:00 | |
| raildo | mriedem: makes sense | 19:00 |
| *** suro-patz has quit IRC | 19:00 | |
| *** rotbeard has quit IRC | 19:01 | |
| *** rook has quit IRC | 19:01 | |
| *** ctrath has quit IRC | 19:02 | |
| alaski | mriedem: we don't use quota classes. we have a hack for quotas per flavor class that's done via a different method | 19:02 |
| *** suro-patz has joined #openstack-nova | 19:02 | |
| *** suro-patz has quit IRC | 19:03 | |
| *** suro-patz has joined #openstack-nova | 19:03 | |
| *** signed8bit has joined #openstack-nova | 19:03 | |
| *** doug-fi__ has joined #openstack-nova | 19:03 | |
| *** signed8b_ has quit IRC | 19:04 | |
| *** doug-fish has quit IRC | 19:04 | |
| *** doug-fi__ has quit IRC | 19:04 | |
| *** rk4n has quit IRC | 19:04 | |
| *** tyagiprince has quit IRC | 19:04 | |
| *** Guest12059 is now known as med_ | 19:04 | |
| *** med_ has quit IRC | 19:04 | |
| *** med_ has joined #openstack-nova | 19:04 | |
| *** tyagiprince1 has joined #openstack-nova | 19:04 | |
| *** doug-fish has joined #openstack-nova | 19:04 | |
| *** doug-fish has quit IRC | 19:04 | |
| *** doug-fis_ has quit IRC | 19:05 | |
| *** doug-fish has joined #openstack-nova | 19:05 | |
| *** harlowja has quit IRC | 19:05 | |
| mriedem | i wonder how much cruft we could remove from nova and the quotas flow if we can assume there is only one quota class and it's name is 'default' | 19:05 |
| *** jwcroppe_ is now known as jwcroppe | 19:06 | |
| *** rlrossit has quit IRC | 19:06 | |
| *** doug-fish has quit IRC | 19:06 | |
| *** doug-fish has joined #openstack-nova | 19:07 | |
| *** tyagiprince1 is now known as tyagiprince | 19:07 | |
| *** moshele has joined #openstack-nova | 19:07 | |
| *** harlowja has joined #openstack-nova | 19:08 | |
| *** ctrath has joined #openstack-nova | 19:09 | |
| *** rlrossit has joined #openstack-nova | 19:09 | |
| *** moshele has quit IRC | 19:10 | |
| *** rushil has joined #openstack-nova | 19:15 | |
| *** tyagiprince1 has joined #openstack-nova | 19:17 | |
| *** chhavi has quit IRC | 19:19 | |
| *** Marga_ has joined #openstack-nova | 19:19 | |
| *** tyagiprince has quit IRC | 19:19 | |
| *** tyagiprince1 is now known as tyagiprince | 19:19 | |
| *** mriedem has quit IRC | 19:20 | |
| *** ndipanov has quit IRC | 19:20 | |
| *** penick has quit IRC | 19:20 | |
| *** penick has joined #openstack-nova | 19:20 | |
| *** doug-fish has quit IRC | 19:22 | |
| *** browne has quit IRC | 19:22 | |
| *** doug-fish has joined #openstack-nova | 19:23 | |
| rlrossit | gibi_: do you mind if I pull down the NotificationBase patch and add unit tests to it? | 19:23 |
| *** bapalm has quit IRC | 19:23 | |
| *** tdurakov_ has quit IRC | 19:24 | |
| *** tdurakov_ has joined #openstack-nova | 19:25 | |
| *** tdurakov_ has quit IRC | 19:25 | |
| *** bapalm has joined #openstack-nova | 19:26 | |
| cfriesen__ | what's the expected behaviour if we do a cold migration on a boot-from-volume instance with ephemeral storage backed by qcow2 using libvirt/kvm? Would we properly copy the ephemeral storage over to the destination and not copy the cinder volume? | 19:28 |
| *** tyagiprince has quit IRC | 19:29 | |
| cfriesen__ | oh, and the compute nodes are not on shared storage | 19:29 |
| *** vishwanathj has quit IRC | 19:29 | |
| cburgess_ | sdague *ping* | 19:29 |
| *** mriedem has joined #openstack-nova | 19:30 | |
| *** jichen has joined #openstack-nova | 19:31 | |
| sdague | cburgess_: pong | 19:33 |
| cburgess_ | sdague Want to chat https://review.openstack.org/#/c/246581/ the workaround script | 19:34 |
| cburgess_ | ? | 19:35 |
| cburgess_ | cfriesen__ What exactly do you mean by ephemeral storage in that BFV scenario? | 19:36 |
| cburgess_ | sdague Or you know.. run away screaming from it is also a valid strategy. | 19:36 |
| *** LukeHinds has quit IRC | 19:36 | |
| *** lykinsbd has quit IRC | 19:36 | |
| sdague | cburgess_: sure | 19:40 |
| cfriesen__ | cburgess: I mean booting with "nova --ephemeral=X --block-device-mapping...." where the rootfs is on a cinder volume, but there's a local scratch disk on the compute node. | 19:40 |
| cburgess_ | cfriesen__ I didn't think we migrated any of the ephemeral disk. I thought those were always re-created on the destination. Maybe I'm wrong. | 19:41 |
| cburgess_ | sdague You mentioned we should have a conversation in the channel. Do you want to do that now or table this for the nova meeting? | 19:41 |
| cfriesen__ | cburgess: I thought ephemeral disappeared only when the instance was terminated, in which case it would make sense to migrate them. | 19:42 |
| *** rushil has quit IRC | 19:42 | |
| *** dane-fichter has quit IRC | 19:43 | |
| *** rushil has joined #openstack-nova | 19:43 | |
| *** burt has quit IRC | 19:44 | |
| cfriesen__ | cburgess: a superficial scan of the libvirt code make it look like we'll migrate the ephemeral/swap disks. I should probably actually try it though. | 19:46 |
| *** Marga_ has quit IRC | 19:47 | |
| *** alex_klimov has joined #openstack-nova | 19:48 | |
| *** lykinsbd has joined #openstack-nova | 19:49 | |
| *** onder has quit IRC | 19:50 | |
| *** lykinsbd has quit IRC | 19:53 | |
| *** onder has joined #openstack-nova | 19:53 | |
| sdague | cburgess_: so, honestly, because I've got a few patches in flight that I'm trying to land today, lets postpone that till tomorrow if that's ok. mordred pointed out there is a dpkg trick we could use so it would survive package upgrade, which we should probably mention to people | 19:53 |
| sdague | the biggest issue is that you don't need to do all the stop / start things you put in the commit message | 19:53 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: update functional tests to support optional project id https://review.openstack.org/246728 | 19:55 |
| *** nic has joined #openstack-nova | 19:55 | |
| *** rpodolyaka1 has joined #openstack-nova | 19:55 | |
| *** lpetrut has quit IRC | 19:57 | |
| *** nic1 has joined #openstack-nova | 19:57 | |
| cburgess_ | sdague OK fair enough. | 19:57 |
| *** stackdump has quit IRC | 19:57 | |
| cburgess_ | sdague Lets discuss tomorrow, I'll ping mordred for the dpkg trick, doesn't help RHEL customers but whatever. | 19:57 |
| sdague | I think the more important part is landing the first patch | 19:57 |
| cburgess_ | sdague Right | 19:58 |
| sdague | well rhel 7 customers will have new enough libvirt | 19:58 |
| sdague | so that's at least something | 19:58 |
| *** suro-patz has quit IRC | 19:58 | |
| sdague | honestly, I expect on low rates for spawns it won't be seen super often | 19:59 |
| sdague | especially if you don't use floating ips | 19:59 |
| *** nic has quit IRC | 19:59 | |
| *** lpetrut has joined #openstack-nova | 20:00 | |
| *** suro-patz has joined #openstack-nova | 20:00 | |
| *** lykinsbd has joined #openstack-nova | 20:02 | |
| sdague | auggy: on https://review.openstack.org/246728, we also need a change to the scenarios - https://github.com/openstack/nova/blob/a363e1c82eeeded0a2d109b7cc9b085f492b3d27/nova/tests/functional/api_sample_tests/api_sample_base.py#L37 | 20:03 |
| sdague | to run a version of the tests without project_id | 20:04 |
| auggy | right yeah that's in the todo list | 20:04 |
| auggy | oh wait | 20:04 |
| auggy | so my plan was to try to get it working by forcing it to run without project id and get that working | 20:04 |
| *** ZZelle_ has joined #openstack-nova | 20:04 | |
| *** atuvenie has quit IRC | 20:05 | |
| auggy | then muck around with the scenarios | 20:05 |
| *** subscope has quit IRC | 20:06 | |
| auggy | i'll add a todo item to the patch for that though | 20:06 |
| *** subscope has joined #openstack-nova | 20:06 | |
| auggy | right now i'm just running test_simple_tennant_usage.py to try to isolate what it's failing on and it looks like it's an issue with the template data not matching the response data | 20:09 |
| sdague | ok, let me take a look | 20:09 |
| *** subscope has quit IRC | 20:09 | |
| auggy | i think i need to create another template for the no project id case? | 20:09 |
| *** subscope has joined #openstack-nova | 20:10 | |
| auggy | or make the project id a variable | 20:10 |
| *** salv-orlando has joined #openstack-nova | 20:11 | |
| auggy | sdague: so here's what i saw for the response vs template - http://paste.openstack.org/show/479336/ | 20:11 |
| auggy | i will pull up a link to the chunk of code | 20:11 |
| auggy | sdague: https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_simple_tenant_usage.py#L47 | 20:13 |
| auggy | that's where it's starting the post command | 20:13 |
| *** e0ne has quit IRC | 20:13 | |
| auggy | what i'm seeing is the links generated in the response don't have the project name "openstack" in them, but the template data expects them to have it | 20:14 |
| *** Marga_ has joined #openstack-nova | 20:14 | |
| *** njirap has quit IRC | 20:16 | |
| sdague | oh, you mean the bookmarks? | 20:16 |
| sdague | https://github.com/openstack/nova/blob/fa53e0aa7cdc9fd3e9cab29a55296100b460a93f/nova/tests/functional/api_sample_tests/api_samples/servers/server-post-resp.json.tpl#L11 ? | 20:16 |
| auggy | yeah whatever those links are for | 20:16 |
| sdague | right | 20:17 |
| *** subscope has quit IRC | 20:17 | |
| sdague | yeh, hmmm... I think we're going to need to make the infrastructure support inflating those | 20:17 |
| auggy | yeah i noticed some variables in the templating | 20:18 |
| auggy | eg %(uuid)s | 20:18 |
| auggy | so it would make sense if possible to do something like that for project id | 20:18 |
| auggy | if that's the way to go then that's what i was contemplating pursuing | 20:19 |
| sdague | yeh, that seems sensible | 20:19 |
| auggy | i didn't -1 WIP my patch so the full functional tests would run on it | 20:19 |
| auggy | in case anything else pops up unrelated to this issue | 20:19 |
| *** ccard has quit IRC | 20:19 | |
| sdague | they'll run even on a WIP | 20:19 |
| auggy | oh they will! i did not know! | 20:20 |
| auggy | ok i'll WIP it then | 20:20 |
| auggy | WIP it good | 20:20 |
| * dansmith slow claps | 20:21 | |
| *** yamahata has quit IRC | 20:21 | |
| sdague | but yeh, I think a way to make the project_id optional in those templates is going to be the right thing to do | 20:21 |
| sdague | it will also make them a bit less magical with the openstack keyword | 20:22 |
| dansmith | alaski: so we decided to do all the sqlalchemy bits in the object for the API db going forward, right? | 20:22 |
| *** lpetrut has quit IRC | 20:22 | |
| dansmith | alaski: I think I was in favor of that, but just caught myself barfing over that as done in the flavor-api patch | 20:23 |
| alaski | dansmith: we did, and I think you were in favor as well | 20:23 |
| alaski | I need to look at that patch again because I think they did it differently than in other patches | 20:24 |
| openstackgerrit | jichenjc proposed openstack/nova: Change Invalid exception to a specified exception https://review.openstack.org/232506 | 20:24 |
| dansmith | alaski: it's going to make every piece of nova dependent on sqlalchemy, even though almost none of it will use it | 20:24 |
| dansmith | alaski: and thus, we'll have to install SA on compute nodes | 20:24 |
| * auggy is trying to say sqlalchemy five times fast | 20:25 | |
| *** ccard has joined #openstack-nova | 20:25 | |
| alaski | dansmith: hmm | 20:25 |
| alaski | I'm not sure if that is actually a problem | 20:26 |
| alaski | but I'm heading out for the day. I'll be back to discuss though | 20:26 |
| dansmith | slacker! | 20:29 |
| dansmith | I'll comment | 20:29 |
| *** penick has quit IRC | 20:34 | |
| *** harlowja has quit IRC | 20:36 | |
| *** tongli has quit IRC | 20:38 | |
| *** harlowja has joined #openstack-nova | 20:41 | |
| *** tonytan4ever has quit IRC | 20:45 | |
| *** Marga_ has quit IRC | 20:45 | |
| *** nic has joined #openstack-nova | 20:45 | |
| bauzas | yay, reno change merged for liberty \o/ | 20:47 |
| *** ctrath has quit IRC | 20:47 | |
| *** thorst has quit IRC | 20:47 | |
| *** nic1 has quit IRC | 20:48 | |
| *** rk4n has joined #openstack-nova | 20:48 | |
| *** thorst has joined #openstack-nova | 20:48 | |
| bauzas | since I'm not a stable core, could someone +W again https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/liberty+topic:add-reno,n,z | 20:49 |
| bauzas | ? | 20:49 |
| bauzas | (I would like to not recheck, just go the gate pipeline) | 20:49 |
| bauzas | sdague: dansmith: ^ | 20:49 |
| *** thorst has quit IRC | 20:49 | |
| dansmith | did it | 20:49 |
| bauzas | cool, ty | 20:49 |
| *** thorst has joined #openstack-nova | 20:49 | |
| *** thorst has quit IRC | 20:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/247118 | 20:50 |
| *** thorst has joined #openstack-nova | 20:50 | |
| *** ctrath has joined #openstack-nova | 20:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/247159 | 20:51 |
| *** thorst_ has joined #openstack-nova | 20:53 | |
| *** rpodolyaka1 has quit IRC | 20:53 | |
| *** thorst has quit IRC | 20:54 | |
| *** FL1SK has quit IRC | 20:54 | |
| *** otter768 has joined #openstack-nova | 20:54 | |
| *** jogo has quit IRC | 20:55 | |
| *** jogo has joined #openstack-nova | 20:55 | |
| *** jogo has quit IRC | 20:55 | |
| *** jogo has joined #openstack-nova | 20:55 | |
| *** thorst_ has quit IRC | 20:57 | |
| *** EinstCrazy has joined #openstack-nova | 20:58 | |
| *** otter768 has quit IRC | 20:59 | |
| *** rk4n has quit IRC | 20:59 | |
| *** thangp has quit IRC | 21:01 | |
| *** openstack has joined #openstack-nova | 21:04 | |
| *** alejandrito has quit IRC | 21:04 | |
| *** e0ne has joined #openstack-nova | 21:09 | |
| *** pumaranikar has quit IRC | 21:09 | |
| *** pumaranikar has joined #openstack-nova | 21:10 | |
| *** penick has joined #openstack-nova | 21:10 | |
| *** rfolco has quit IRC | 21:10 | |
| *** jistr has joined #openstack-nova | 21:10 | |
| *** thorst has joined #openstack-nova | 21:11 | |
| openstackgerrit | melanie witt proposed openstack/nova: xenapi: Add helper function and unit tests for client session https://review.openstack.org/247175 | 21:11 |
| *** jerrygb_ has joined #openstack-nova | 21:12 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Build timeouts task filters deleting instances https://review.openstack.org/240017 | 21:13 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Modify VM's updated_at field on volume actions https://review.openstack.org/247176 | 21:13 |
| mriedem | our db archive unit tests are pretty hokey, they test archive on a table by table basis, or very limited set | 21:13 |
| mriedem | really need a functional test for that...boot an instance, delete, then archive it | 21:13 |
| * mriedem gets to work | 21:14 | |
| mriedem | also interesting, there are no-db functional tests in the nova.tests.functional.db | 21:15 |
| mriedem | class InstanceMappingTestCase(test.NoDBTestCase): | 21:15 |
| mriedem | seems counter productive | 21:15 |
| *** thorst has quit IRC | 21:15 | |
| *** penick has quit IRC | 21:15 | |
| *** jerrygb has quit IRC | 21:15 | |
| *** raildo is now known as raildo-afk | 21:15 | |
| *** rpodolyaka1 has joined #openstack-nova | 21:16 | |
| dansmith | bauzas: sdague: do we need to recheck this? https://review.openstack.org/#/c/242027/ | 21:18 |
| bauzas | dansmith: I was waiting for https://review.openstack.org/#/c/244761/ | 21:18 |
| dansmith | cripes | 21:19 |
| *** signed8bit_ZZZzz is now known as signed8bit | 21:19 | |
| *** rpodolyaka1 has quit IRC | 21:19 | |
| *** changbl has joined #openstack-nova | 21:19 | |
| bauzas | the Depends-On is a bit wrong | 21:19 |
| mriedem | dansmith: yes on the recheck | 21:19 |
| mriedem | the dependent changes are merged | 21:19 |
| bauzas | mriedem: sec | 21:19 |
| *** penick has joined #openstack-nova | 21:19 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Config options: centralize section "scheduler" https://review.openstack.org/245891 | 21:19 |
| openstackgerrit | Ed Leafe proposed openstack/nova: WIP: Add better help text to scheduler options https://review.openstack.org/247181 | 21:19 |
| bauzas | mriedem: dansmith: http://lists.openstack.org/pipermail/openstack-dev/2015-November/078940.html | 21:20 |
| mriedem | bauzas: you only need the first change in that series though right? | 21:20 |
| mriedem | for the tox bits | 21:20 |
| mriedem | ok, well, should be long for 244761 to land anyway | 21:21 |
| *** jerrygb_ has quit IRC | 21:22 | |
| bauzas | mriedem: well, I feel it should work, but I wonder if we could break the CI if not landing the unrelease patch first | 21:22 |
| bauzas | lemme think about that | 21:22 |
| *** ijw has joined #openstack-nova | 21:22 | |
| bauzas | no, it's okay | 21:22 |
| *** ijw has quit IRC | 21:22 | |
| bauzas | because that's only if we add some reno changes for liberty | 21:23 |
| bauzas | mriedem: ^ | 21:23 |
| *** ijw has joined #openstack-nova | 21:23 | |
| *** jerrygb_ has joined #openstack-nova | 21:23 | |
| bauzas | so yeah, I'm okay +W'ing again https://review.openstack.org/#/c/242027/ | 21:23 |
| bauzas | dansmith: mriedem: sdague: ^ | 21:23 |
| *** thorst has joined #openstack-nova | 21:24 | |
| sdague | bauzas: tickled the +A bit | 21:24 |
| sdague | it's now in the gate | 21:24 |
| *** breitz1 is now known as breitz | 21:25 | |
| bauzas | sdague: cool, thanks ! | 21:25 |
| *** thorst_ has joined #openstack-nova | 21:25 | |
| *** tonytan4ever has quit IRC | 21:25 | |
| *** smatzek has quit IRC | 21:26 | |
| *** jerrygb__ has joined #openstack-nova | 21:26 | |
| *** jerrygb__ has quit IRC | 21:27 | |
| *** tonytan4ever has joined #openstack-nova | 21:28 | |
| *** jerrygb has joined #openstack-nova | 21:28 | |
| *** jerrygb_ has quit IRC | 21:28 | |
| *** thorst has quit IRC | 21:28 | |
| mriedem | sdague: looking at some of the in tree functional tests that create a server, they dont have addCleanup for deleting the server after the create, is that because we're just testing against a sqlite db that gets created for each test run? | 21:31 |
| mriedem | so we don't care about leaking resources | 21:31 |
| sdague | mriedem: yep | 21:31 |
| *** e0ne has quit IRC | 21:31 | |
| mriedem | cool | 21:31 |
| sdague | sqlite and fakevirt | 21:31 |
| sdague | so everything is gone at the end of the test | 21:31 |
| dansmith | heh | 21:31 |
| dansmith | ephemeral cloud | 21:31 |
| sdague | it's the best ephemeral cloud | 21:31 |
| dansmith | yeah | 21:32 |
| sdague | we even have multinode | 21:32 |
| bauzas | alaski: around ? | 21:32 |
| mriedem | as long as i have resources in the db when i run the archive i'm happy | 21:32 |
| sdague | mriedem: https://github.com/openstack/nova/blob/a363e1c82eeeded0a2d109b7cc9b085f492b3d27/nova/tests/functional/test_server_group.py#L133-L136 | 21:32 |
| mriedem | sdague: yeah i was going to basically copy https://github.com/openstack/nova/blob/a363e1c82eeeded0a2d109b7cc9b085f492b3d27/nova/tests/functional/test_servers.py#L414 except for the rename stuff | 21:33 |
| bauzas | alaski: nvm, will look into fixing https://review.openstack.org/#/c/211753/ | 21:33 |
| mriedem | so create server, wait for active, delete it, wait for deleted, then archive it | 21:33 |
| sdague | sure, though I don't know there is much wait for active | 21:34 |
| mriedem | creating the server should give us at least some interesting foreign keys, like instance_actions | 21:34 |
| mriedem | it just waits for !BUILD | 21:34 |
| sdague | yep | 21:34 |
| sdague | right, sure | 21:34 |
| *** Marga_ has joined #openstack-nova | 21:34 | |
| *** pratikma_ has quit IRC | 21:35 | |
| *** pratikmallya has joined #openstack-nova | 21:35 | |
| dansmith | sdague: did you see my query during the cells meeting earlier/ | 21:35 |
| sdague | just with fakevirt I don't think we even bother with a build step | 21:35 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Modify VM's updated_at field on volume actions https://review.openstack.org/247176 | 21:36 |
| sdague | dansmith: I guess I missed it | 21:39 |
| *** rwmjones has quit IRC | 21:39 | |
| *** Marga_ has quit IRC | 21:39 | |
| *** Marga_ has joined #openstack-nova | 21:40 | |
| dansmith | sdague: I was wondering if you'd be okay with merging that grenade cells api change before all this reno crap actually lands | 21:40 |
| *** kfarr has quit IRC | 21:40 | |
| dansmith | sdague: given it's approved and just waiting, and the PTL has acked | 21:41 |
| *** jistr has quit IRC | 21:41 | |
| *** rwmjones has joined #openstack-nova | 21:41 | |
| sdague | ok, how far away are we from getting reno to land? | 21:41 |
| bauzas | not that far | 21:41 |
| dansmith | it's been soonish for a while | 21:42 |
| dansmith | sdague: https://review.openstack.org/#/c/245828/ | 21:42 |
| bauzas | but there is still a chain of 2 patches to land before the grenade change can land | 21:42 |
| bauzas | ie. merging the CI job + the reno patch itself | 21:43 |
| sdague | dansmith: sure, pull the depends on | 21:43 |
| dansmith | sdague: okay | 21:43 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add release note for API DB migration requirements https://review.openstack.org/245828 | 21:43 |
| dansmith | dammit | 21:43 |
| dansmith | that is not what I wanted | 21:43 |
| bauzas | hah :) | 21:44 |
| bauzas | I can fast-verify that one^ | 21:44 |
| dansmith | sdague: https://review.openstack.org/#/c/190399/ | 21:44 |
| dansmith | bauzas: yea, you wanted that change anyway, so.. :D | 21:44 |
| sdague | but if the reno doesn't land by M1 then you'll owe us all beer :) | 21:44 |
| sdague | or maybe bauzas will | 21:44 |
| bauzas | heh | 21:44 |
| *** dustins has quit IRC | 21:45 | |
| bauzas | dansmith: I actually already verified that one because of the prelude section | 21:45 |
| bauzas | dansmith: so I'm +2 with it | 21:45 |
| dansmith | sdague: if it's the kind of beers that are oft-promised and ne'r-collected, then sure :) | 21:46 |
| sdague | I'm collecting from someone :) | 21:46 |
| dansmith | "Currency Ale" I believe it is | 21:46 |
| dansmith | https://www.youtube.com/watch?v=uT120JBL4yo | 21:47 |
| *** e0ne has joined #openstack-nova | 21:47 | |
| sdague | mriedem: anyway, if you build a test, build one like either test_server_groups.py | 21:47 |
| sdague | or there is some stuff in wsgi/ dir there that also pokes directly into the db | 21:47 |
| sdague | in case you want grey box testing | 21:48 |
| *** rook has joined #openstack-nova | 21:48 | |
| mriedem | sdague: this first rev is quick and dirty but re-uses parts of the other tests, like test_server_groups | 21:49 |
| mriedem | so minimal copy/paste | 21:49 |
| mriedem | just want to see if i can make this hit the archive db api | 21:49 |
| sdague | yeh, right, test_server_groups is a good starting point | 21:49 |
| *** rlrossit has left #openstack-nova | 21:49 | |
| sdague | because it doesn't go through a huge indirection like the samples tests | 21:50 |
| *** e0ne has quit IRC | 21:50 | |
| sdague | this is how you hit the db api - https://github.com/openstack/nova/blob/a363e1c82eeeded0a2d109b7cc9b085f492b3d27/nova/tests/functional/wsgi/test_flavor_manage.py#L205 | 21:51 |
| sdague | in a test like that | 21:51 |
| *** yamahata has joined #openstack-nova | 21:51 | |
| sdague | bauzas: the release notes project-config bit merged | 21:52 |
| bauzas | \o/ | 21:52 |
| mriedem | sdague: yeah that's basically what i'm writing | 21:53 |
| sdague | mriedem: cool | 21:53 |
| sdague | and ++ on more tests like that | 21:53 |
| bauzas | let's make it a success, nope? | 21:53 |
| sdague | bauzas: go for it | 21:53 |
| sdague | it was your patch :) | 21:53 |
| bauzas | #success reno in place for the nova release notes | 21:53 |
| openstackstatus | bauzas: Added success to Success page | 21:53 |
| yonglihe | anyone running into this error:AttributeError: 'ConductorAPI' object has no attribute 'object_class_action_versions' ? | 21:54 |
| bauzas | okay, I can now check for all the merged UpgradeImpact patches | 21:54 |
| dansmith | yonglihe: where? | 21:54 |
| yonglihe | dansmith: some time PCI CI will trigger this one, and seems not like a configration error, i'm not sure if it's a bug or not | 21:55 |
| dansmith | yonglihe: seems like a mismatch-of-versions sort of problem to me | 21:55 |
| bauzas | right | 21:55 |
| bauzas | an old conductor | 21:55 |
| yonglihe | today, one time hit, jenkins not finish it's testing, i will check that to confirm , thanks dansmith | 21:56 |
| dansmith | well, | 21:56 |
| dansmith | yeah, could be old conductor, depending on where it's coming in | 21:56 |
| *** salv-orl_ has joined #openstack-nova | 21:57 | |
| *** rwmjones has quit IRC | 21:57 | |
| yonglihe | dansmith: n-cpu is die while boot up time | 21:57 |
| *** salv-orlando has quit IRC | 21:57 | |
| yonglihe | http://52.27.155.124/pci/247175/1/screen-logs/n-cpu.log.gz | 21:58 |
| dansmith | yonglihe: too new of o.vo for the nova code you're running? | 21:58 |
| yonglihe | it's pull the oslo based on requirements, let me check the pip freeze | 21:59 |
| dansmith | yonglihe: is this on master? | 22:00 |
| *** jerrygb has quit IRC | 22:00 | |
| klindgren | dansmith, I ran a cProfiler on conductor with a single worker. Call graph is here: fhttp://tempsend.com/5340867576/F283/conductor1worker.svg - unsure if this helps spans any additional ideas? | 22:00 |
| klindgren | spawns* | 22:00 |
| yonglihe | dansmith: it's a new patch, but nothing related to that change. i'm trying to figure what's wrong | 22:00 |
| *** penick has quit IRC | 22:01 | |
| klindgren | http://tempsend.com/5340867576/F283/conductor1worker.svg | 22:01 |
| *** pratikmallya has quit IRC | 22:01 | |
| dansmith | klindgren: I dunno what the percentages are, so, not really | 22:02 |
| *** changbl has quit IRC | 22:02 | |
| dansmith | klindgren: you have lots of build/destroy traffic going on on your cloud right? | 22:03 |
| klindgren | very very very little | 22:03 |
| dansmith | oh I thought you said you did | 22:03 |
| klindgren | maybe 80 or so a day | 22:03 |
| klindgren | actually I would say we might have ~500 creates a day | 22:03 |
| klindgren | similar number of deletes | 22:04 |
| yonglihe | required oslo.versionedobjects>=0.9.0, in the test it's oslo.versionedobjects==0.13.0, dansmith | 22:04 |
| dansmith | yonglihe: I'm not sure what you're saying | 22:04 |
| dansmith | yonglihe: 0.13 is the current release | 22:05 |
| dansmith | yonglihe: which should be fine for master, and clearly is because it works in our gate | 22:05 |
| klindgren | dansmith, the call graph was generated via: gprof2dot - do you have something that you would rather see used? | 22:05 |
| *** gjayavelu has joined #openstack-nova | 22:05 | |
| dansmith | klindgren: no, I just don't know what the numbers mean | 22:07 |
| mriedem | dansmith: klindgren: i might not be reading this correctly, but it looks like the deep stuff at the bottom is sqlalchemy ORM | 22:07 |
| dansmith | klindgren: but it looks to me like there are a bunch of timer things that are running a ton | 22:07 |
| mriedem | doing a query, fetching results, iterating | 22:08 |
| dansmith | a lot more than anything else | 22:08 |
| dansmith | mriedem: yes, and those are called a ton as well | 22:08 |
| mriedem | are the timers/pollers rabbit heartbeat things? | 22:08 |
| dansmith | mriedem: I dunno | 22:09 |
| yonglihe | dansmith, anyway, thanks , i'm trying to figure out if it's a mis-configuration or not. | 22:09 |
| dansmith | looks to me like langhelpers.__get__ gets called like 40,000 times as a result of one service call | 22:09 |
| mriedem | erlang | 22:09 |
| mriedem | oh that's sqlalchemy | 22:10 |
| *** burt has joined #openstack-nova | 22:10 | |
| dansmith | yeah | 22:10 |
| mriedem | or mysql-python, no idea really | 22:10 |
| *** doug-fish has quit IRC | 22:10 | |
| dansmith | sqlalchemy I think | 22:10 |
| dansmith | klindgren: I assume you've done slow and large query analysis already? | 22:11 |
| dansmith | because if something is pulling a ton of items, it'll cause a lot of churn here | 22:11 |
| klindgren | I haven't checked the slow_querry log in a while | 22:12 |
| klindgren | but last time I did anything that was logged was just over 1 second | 22:12 |
| *** rwmjones has joined #openstack-nova | 22:12 | |
| dansmith | that's slow, right? | 22:12 |
| klindgren | We also pruned out database for soft_deleted items about a month ago | 22:13 |
| klindgren | iirc | 22:13 |
| dansmith | klindgren: are you running ssl talking to rabbit? | 22:15 |
| dansmith | the first percentage in the box is total time | 22:15 |
| dansmith | ssl:read is 65% | 22:15 |
| dansmith | dispatch and reply, which is where all the real work is done is 9% | 22:15 |
| *** edtubill has quit IRC | 22:17 | |
| klindgren | we are running ssl talking to rabbit | 22:17 |
| dansmith | can you turn that off as a test? | 22:17 |
| dansmith | maybe that's why others aren't seeing the same pressure | 22:17 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE NFS setup for multinode job https://review.openstack.org/247081 | 22:17 |
| *** edtubill has joined #openstack-nova | 22:18 | |
| *** jichen has quit IRC | 22:18 | |
| *** claudiub has joined #openstack-nova | 22:19 | |
| yonglihe | dansmith: i'm going to open a bug tracking this problem, include figure out if it's a bug or not. | 22:19 |
| dansmith | yonglihe: okay | 22:19 |
| yonglihe | 2 times hit this today, it's worth to track. | 22:20 |
| dansmith | yonglihe: it should be pretty easy to look at whatever this is running on and see if conductor/rpcapi has that newer method | 22:20 |
| dansmith | yonglihe: are you guys properly rebasing every patch on top of master when you run it? | 22:20 |
| dansmith | (zuul does this I think, if you're running zuul) | 22:21 |
| dansmith | but this would be easily hit if you're not and someone rechecked an old patch | 22:21 |
| *** edtubill has quit IRC | 22:21 | |
| *** edtubill has joined #openstack-nova | 22:21 | |
| yonglihe | dansmith: yes we pull new nova, and pull the testing patch. | 22:22 |
| yonglihe | dansmith: i got your point | 22:22 |
| bauzas | does someone know how to pdb a functional test? | 22:22 |
| *** edtubill has quit IRC | 22:23 | |
| klindgren | dansmith, soooo | 22:23 |
| klindgren | CPU usage dropped like a mofo | 22:24 |
| dansmith | klindgren: ORLY | 22:24 |
| mriedem | ssl! | 22:24 |
| klindgren | 7% -> 0.7% | 22:24 |
| klindgren | when not processing a message | 22:24 |
| yonglihe | dansmith: this is does happen if author not re-base his patch, got. i will track this in CI. | 22:24 |
| dansmith | yonglihe: sounds like you're not rebasing then | 22:24 |
| *** penick has joined #openstack-nova | 22:24 | |
| *** tdurakov has joined #openstack-nova | 22:25 | |
| dhellmann | bauzas : \o/ | 22:25 |
| yonglihe | dansmith: i remember not re-basing, i better do that, thanks. | 22:25 |
| klindgren | so yea - conductor is basically sleeping on cpu now | 22:27 |
| mriedem | klindgren: cool, time to update that etherpad :) | 22:27 |
| mriedem | edmondsw: ^ re: the ssl thing, i know you love you some ssl | 22:30 |
| yonglihe | dansmith: tracking this in this link: https://etherpad.openstack.org/p/third-party-ci-status-tracking-intel-hardware | 22:30 |
| edmondsw | mriedem, reading back... | 22:30 |
| *** rushil has quit IRC | 22:31 | |
| *** rwmjones has quit IRC | 22:31 | |
| claudiub | mriedem: hello. If you have a teeny tiny amount of time available, could you take a look at the os-win in nova patch? Hyper-V CI has +1, subteam has +1, your comments have been addressed. Thanks! https://review.openstack.org/#/c/212102/ | 22:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Reverse sort tables before archiving https://review.openstack.org/246635 | 22:32 |
| mriedem | sdague: ^ i'm pretty proud of the test added there | 22:32 |
| edmondsw | we've been using SSL/TLS (really TLS) with rabbit for several releases now in the product I work on | 22:32 |
| mriedem | claudiub: ok, probably tomorrow | 22:32 |
| edmondsw | (PowerVC) | 22:33 |
| claudiub | mriedem: cool. :) | 22:33 |
| mriedem | edmondsw: scaled out to 250 compute nodes though? | 22:33 |
| mriedem | and 3 control nodes | 22:33 |
| edmondsw | no... we only have one control node, 30 hosts | 22:34 |
| *** FL1SK has joined #openstack-nova | 22:35 | |
| *** hemna has quit IRC | 22:35 | |
| *** penick has quit IRC | 22:36 | |
| *** hemna has joined #openstack-nova | 22:36 | |
| *** pratikmallya has joined #openstack-nova | 22:37 | |
| *** lykinsbd has quit IRC | 22:37 | |
| *** sacharya has quit IRC | 22:38 | |
| *** browne has joined #openstack-nova | 22:39 | |
| *** tonytan4ever has quit IRC | 22:40 | |
| *** rwmjones has joined #openstack-nova | 22:41 | |
| *** browne has quit IRC | 22:41 | |
| edmondsw | mriedem, actually, I think we've gone up to 200 hosts with NovaLink (the 30 was KVM) | 22:44 |
| dansmith | oooh, what's novalink? | 22:44 |
| mriedem | powervm v2 | 22:44 |
| dansmith | ooooooh | 22:44 |
| *** jaypipes has joined #openstack-nova | 22:47 | |
| *** burgerk_ has joined #openstack-nova | 22:48 | |
| *** burgerk has quit IRC | 22:49 | |
| *** edmondsw has quit IRC | 22:49 | |
| mriedem | awesome, so i now have a functional test showing why nova-manage db archive_deleted_rows is busted | 22:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/247118 | 22:51 |
| mriedem | forgot we had to flip a switch to tell sqlite to care about fkeys | 22:51 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE NFS setup for multinode job https://review.openstack.org/247081 | 22:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Reverse sort tables before archiving https://review.openstack.org/246635 | 22:51 |
| *** gjayavelu has quit IRC | 22:52 | |
| *** burgerk_ has quit IRC | 22:54 | |
| *** mriedem has quit IRC | 22:54 | |
| *** otter768 has joined #openstack-nova | 22:55 | |
| *** unicell has quit IRC | 22:55 | |
| *** otter768 has quit IRC | 23:01 | |
| *** mdrabe has quit IRC | 23:02 | |
| *** ijw has quit IRC | 23:03 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: DO NOT MERGE NFS setup for multinode job https://review.openstack.org/247081 | 23:03 |
| *** ijuwang has quit IRC | 23:03 | |
| klindgren | so turns I got way ahead of myself | 23:05 |
| *** jerrygb has joined #openstack-nova | 23:05 | |
| klindgren | 2015-11-18 16:02:13.374 16155 ERROR oslo_messaging._drivers.impl_rabbit [req-dfe0c216-9f36-4a57-970d-0475dc80a01c - - - - -] AMQP server p3plcldcap001-03.prod.phx3.secureserver.net:5671 closed the connection. Check login credentials: Socket closed | 23:05 |
| *** Piet has quit IRC | 23:05 | |
| klindgren | it wasn't connecting to rabbitmq hence why hte cpu dropped so much | 23:05 |
| *** penick has joined #openstack-nova | 23:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/247118 | 23:06 |
| *** burt has quit IRC | 23:07 | |
| *** ijw has joined #openstack-nova | 23:07 | |
| *** jerrygb has quit IRC | 23:09 | |
| *** ctrath has quit IRC | 23:10 | |
| *** jerrygb has joined #openstack-nova | 23:14 | |
| *** pumaranikar has quit IRC | 23:16 | |
| *** ctrath has joined #openstack-nova | 23:16 | |
| *** sneti has quit IRC | 23:16 | |
| *** pradk has quit IRC | 23:19 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 23:20 | |
| auggy | bauzas: did anyone ever answer your how to pdb functional tests question? | 23:21 |
| bauzas | auggy: nope, but I'm okay | 23:21 |
| auggy | idk if this is useful or relevant to what you were asking, but i just throw a unittest.main loop into the test and run it through the debugger | 23:21 |
| bauzas | auggy: I just used -m testtools.run | 23:22 |
| bauzas | in my functional venv | 23:22 |
| auggy | ah cool, and you were able to interact with pdb through that? | 23:22 |
| bauzas | yup | 23:22 |
| * auggy makes a note | 23:22 | |
| bauzas | like for unittests | 23:22 |
| auggy | yeah, i use a visual debugger in PyCharm for that | 23:22 |
| auggy | but it's good to know how to do it on the shell too, so thanks for that :) | 23:23 |
| bauzas | I don't like visual debuggers :) | 23:23 |
| auggy | ha | 23:23 |
| auggy | i got hooked early | 23:23 |
| bauzas | and I also don't like IDEs :p | 23:24 |
| bauzas | but it's not Friday yet... | 23:24 |
| auggy | my favorite is still gdb in emacs | 23:24 |
| bauzas | well, it's post-midnight and I have to leave so... | 23:25 |
| bauzas | Atom + pdb ! | 23:25 |
| * auggy is still getting used to not thinking of pdb as Perl Debugger ;) | 23:26 | |
| *** alex_klimov has quit IRC | 23:26 | |
| *** xyang1 has quit IRC | 23:28 | |
| *** suro-patz has quit IRC | 23:30 | |
| *** aysyd has quit IRC | 23:30 | |
| *** rmart04 has joined #openstack-nova | 23:31 | |
| *** suro-patz has joined #openstack-nova | 23:32 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 23:34 | |
| *** penick has quit IRC | 23:35 | |
| *** ZZelle_ has quit IRC | 23:35 | |
| *** tdurakov has quit IRC | 23:38 | |
| dansmith | dims_: you're not in -operators? | 23:45 |
| *** suro-patz has quit IRC | 23:46 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 23:50 | |
| *** ctrath has quit IRC | 23:52 | |
| *** thorst_ has quit IRC | 23:55 | |
| *** rmart04 has quit IRC | 23:56 | |
| *** mc_nair has quit IRC | 23:56 | |
| *** stackdump has joined #openstack-nova | 23:57 | |
| *** suro-patz has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
