| *** gmann_ has joined #openstack-nova | 00:03 | |
| *** Sukhdev has quit IRC | 00:07 | |
| *** Sukhdev has joined #openstack-nova | 00:07 | |
| *** gmann_ has quit IRC | 00:07 | |
| *** gmann has joined #openstack-nova | 00:09 | |
| *** gmann has quit IRC | 00:10 | |
| *** belmoreira has joined #openstack-nova | 00:11 | |
| *** thorst has joined #openstack-nova | 00:12 | |
| *** gmann has joined #openstack-nova | 00:12 | |
| *** annashen has quit IRC | 00:16 | |
| *** n0ano has joined #openstack-nova | 00:16 | |
| *** thorst has quit IRC | 00:20 | |
| *** yingjun has joined #openstack-nova | 00:21 | |
| *** rfolco has quit IRC | 00:24 | |
| *** diogogmt has quit IRC | 00:31 | |
| *** mingy has quit IRC | 00:31 | |
| *** amotoki has joined #openstack-nova | 00:33 | |
| *** yingjun has quit IRC | 00:33 | |
| *** yingjun has joined #openstack-nova | 00:33 | |
| *** alexpilotti has quit IRC | 00:36 | |
| *** tsekiyama has quit IRC | 00:36 | |
| *** mnestratov has quit IRC | 00:37 | |
| *** amotoki has quit IRC | 00:38 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: CONF.allow_resize_to_same_host should check only once in controller https://review.openstack.org/186236 | 00:43 |
|---|---|---|
| *** mwagner_afk has joined #openstack-nova | 00:43 | |
| *** mtanino has quit IRC | 00:44 | |
| *** kiran-r has joined #openstack-nova | 00:44 | |
| *** pece has quit IRC | 00:46 | |
| *** IanGovett has joined #openstack-nova | 00:47 | |
| *** annashen has joined #openstack-nova | 00:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/190362 | 00:47 |
| *** ijw has quit IRC | 00:50 | |
| *** ijw has joined #openstack-nova | 00:50 | |
| *** annashen has quit IRC | 00:54 | |
| *** browne has joined #openstack-nova | 01:02 | |
| *** thorst has joined #openstack-nova | 01:05 | |
| *** rdo has quit IRC | 01:08 | |
| *** diogogmt has joined #openstack-nova | 01:09 | |
| *** rdo has joined #openstack-nova | 01:10 | |
| *** angdraug has quit IRC | 01:11 | |
| *** dims_ has joined #openstack-nova | 01:18 | |
| *** otter768 has joined #openstack-nova | 01:20 | |
| *** dims has quit IRC | 01:21 | |
| *** isd has joined #openstack-nova | 01:22 | |
| *** tobe has joined #openstack-nova | 01:24 | |
| *** otter768 has quit IRC | 01:25 | |
| *** pixelbeat_ has quit IRC | 01:27 | |
| *** arnaud____ has joined #openstack-nova | 01:27 | |
| *** iamjarvo has joined #openstack-nova | 01:30 | |
| *** arnaud____ has quit IRC | 01:32 | |
| *** RuiChen has quit IRC | 01:36 | |
| *** RuiChen has joined #openstack-nova | 01:37 | |
| *** dsanders has quit IRC | 01:37 | |
| *** IanGovett has quit IRC | 01:39 | |
| *** vilobhmm has quit IRC | 01:41 | |
| *** yingjun has quit IRC | 01:45 | |
| *** thorst has quit IRC | 01:45 | |
| *** yingjun has joined #openstack-nova | 01:45 | |
| *** belmoreira has quit IRC | 01:50 | |
| *** marcusvrn has joined #openstack-nova | 02:01 | |
| *** marun has joined #openstack-nova | 02:01 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] cleanup openstack-common.conf and sync updated files https://review.openstack.org/189118 | 02:02 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 02:02 |
| *** VW has joined #openstack-nova | 02:06 | |
| *** ian_brown has joined #openstack-nova | 02:07 | |
| *** Sukhdev has quit IRC | 02:07 | |
| *** rmstar_ is now known as rmstar | 02:11 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Use oslo-config-generator instead of generate_sample.sh https://review.openstack.org/180013 | 02:15 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Add rootwrap daemon mode support https://review.openstack.org/180695 | 02:15 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Eventlet supports python logging directly https://review.openstack.org/179801 | 02:15 |
| *** unicell has quit IRC | 02:18 | |
| *** amotoki has joined #openstack-nova | 02:21 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 02:24 |
| *** claudiub has quit IRC | 02:30 | |
| *** baoli has joined #openstack-nova | 02:30 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 tests https://review.openstack.org/188243 | 02:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 tests - take 2 https://review.openstack.org/188785 | 02:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/virt/vmwareapi/test*.py https://review.openstack.org/188653 | 02:30 |
| *** dims_ has quit IRC | 02:33 | |
| *** yingjun_ has joined #openstack-nova | 02:35 | |
| *** yingjun has quit IRC | 02:35 | |
| *** amotoki has quit IRC | 02:40 | |
| *** varya has joined #openstack-nova | 02:43 | |
| *** tobe has quit IRC | 02:46 | |
| *** tobe has joined #openstack-nova | 02:47 | |
| *** vilobhmm has joined #openstack-nova | 02:47 | |
| *** ijw has quit IRC | 02:50 | |
| *** tobe has quit IRC | 02:50 | |
| *** kiran-r has quit IRC | 02:51 | |
| *** ajayaa has joined #openstack-nova | 02:51 | |
| *** tobe has joined #openstack-nova | 02:51 | |
| *** baoli has quit IRC | 02:52 | |
| *** vilobhmm has quit IRC | 02:59 | |
| *** yamahata has quit IRC | 03:00 | |
| *** Administrator__ has joined #openstack-nova | 03:02 | |
| *** david-lyle has joined #openstack-nova | 03:04 | |
| *** varya_ has joined #openstack-nova | 03:04 | |
| *** Administrator_ has quit IRC | 03:05 | |
| *** iamjarvo has quit IRC | 03:05 | |
| *** dsanders has joined #openstack-nova | 03:06 | |
| *** varya has quit IRC | 03:07 | |
| *** garyk has quit IRC | 03:14 | |
| *** arnaud____ has joined #openstack-nova | 03:16 | |
| *** deepthi has joined #openstack-nova | 03:19 | |
| *** otter768 has joined #openstack-nova | 03:21 | |
| *** arnaud____ has quit IRC | 03:21 | |
| *** setuid has quit IRC | 03:24 | |
| *** setuid has joined #openstack-nova | 03:24 | |
| *** setuid has joined #openstack-nova | 03:24 | |
| *** otter768 has quit IRC | 03:26 | |
| *** ijw has joined #openstack-nova | 03:28 | |
| *** marun has quit IRC | 03:28 | |
| *** dims has joined #openstack-nova | 03:34 | |
| *** varya_ has quit IRC | 03:35 | |
| *** ijw_ has joined #openstack-nova | 03:35 | |
| *** wanghua has joined #openstack-nova | 03:38 | |
| *** ijw has quit IRC | 03:39 | |
| *** artom has quit IRC | 03:40 | |
| *** kiran-r has joined #openstack-nova | 03:42 | |
| *** yingjun_ has quit IRC | 03:46 | |
| *** wanghua has quit IRC | 03:46 | |
| *** wanghua has joined #openstack-nova | 03:47 | |
| *** varya_ has joined #openstack-nova | 03:51 | |
| *** kiran-r has quit IRC | 03:57 | |
| *** ijw_ has quit IRC | 03:57 | |
| *** fifieldt has joined #openstack-nova | 03:58 | |
| *** ajayaa has quit IRC | 03:58 | |
| *** iamjarvo has joined #openstack-nova | 04:01 | |
| *** vitikkan has joined #openstack-nova | 04:01 | |
| *** garyk has joined #openstack-nova | 04:01 | |
| *** tobe has quit IRC | 04:04 | |
| *** Erming_ has quit IRC | 04:06 | |
| *** garyk1 has joined #openstack-nova | 04:07 | |
| *** garyk has quit IRC | 04:07 | |
| *** tobe has joined #openstack-nova | 04:13 | |
| *** ajayaa has joined #openstack-nova | 04:15 | |
| openstackgerrit | Rui Chen proposed openstack/nova: CreateVM supports subnet specified https://review.openstack.org/182969 | 04:21 |
| openstackgerrit | Rui Chen proposed openstack/nova: Dump microversion for bp selecting-subnet-when-creating-vm https://review.openstack.org/190474 | 04:21 |
| openstackgerrit | Rui Chen proposed openstack/nova: Add subnet_id in NetworkRequest object https://review.openstack.org/190475 | 04:21 |
| openstackgerrit | Rui Chen proposed openstack/nova: Support creating port with subnet_id in neutron api https://review.openstack.org/190476 | 04:21 |
| *** p_deore has joined #openstack-nova | 04:21 | |
| *** annashen has joined #openstack-nova | 04:22 | |
| *** irenab has quit IRC | 04:23 | |
| *** marcusvrn has quit IRC | 04:24 | |
| *** markvoelker has quit IRC | 04:26 | |
| *** annashen has quit IRC | 04:27 | |
| *** tjones1 has joined #openstack-nova | 04:28 | |
| *** RuiChen has quit IRC | 04:33 | |
| *** RuiChen has joined #openstack-nova | 04:33 | |
| *** Kennan2 has joined #openstack-nova | 04:36 | |
| *** Kennan has quit IRC | 04:37 | |
| *** diogogmt has quit IRC | 04:38 | |
| *** changbl has joined #openstack-nova | 04:42 | |
| *** shaohe_robot1 has quit IRC | 04:42 | |
| *** tjones1 has quit IRC | 04:47 | |
| *** ajayaa has quit IRC | 04:50 | |
| *** tjones1 has joined #openstack-nova | 04:54 | |
| *** haomaiw__ has joined #openstack-nova | 04:55 | |
| *** haomaiwang has quit IRC | 04:56 | |
| *** tobe has quit IRC | 04:56 | |
| *** yingjun has joined #openstack-nova | 04:56 | |
| *** yingjun has quit IRC | 04:56 | |
| *** yingjun has joined #openstack-nova | 04:56 | |
| *** sdake has joined #openstack-nova | 04:57 | |
| *** achanda has joined #openstack-nova | 04:59 | |
| *** tjones1 has quit IRC | 05:00 | |
| *** tobe has joined #openstack-nova | 05:01 | |
| *** ijw has joined #openstack-nova | 05:03 | |
| *** rajesht has joined #openstack-nova | 05:04 | |
| *** VW has quit IRC | 05:07 | |
| *** patrickeast has quit IRC | 05:11 | |
| *** varya_ has quit IRC | 05:15 | |
| *** kiran-r has joined #openstack-nova | 05:15 | |
| *** varya has joined #openstack-nova | 05:16 | |
| *** prashant__ has joined #openstack-nova | 05:17 | |
| *** irenab has joined #openstack-nova | 05:20 | |
| *** otter768 has joined #openstack-nova | 05:22 | |
| *** belmoreira has joined #openstack-nova | 05:22 | |
| *** achanda has quit IRC | 05:22 | |
| *** Nikolay_St has quit IRC | 05:23 | |
| *** unicell has joined #openstack-nova | 05:24 | |
| *** isd has quit IRC | 05:25 | |
| *** otter768 has quit IRC | 05:27 | |
| *** markvoelker has joined #openstack-nova | 05:27 | |
| *** markvoelker has quit IRC | 05:32 | |
| *** nkrinner has joined #openstack-nova | 05:34 | |
| *** aswadr has joined #openstack-nova | 05:34 | |
| *** wxy has joined #openstack-nova | 05:34 | |
| *** yingjun has quit IRC | 05:34 | |
| *** ildikov_ has quit IRC | 05:35 | |
| *** unicell1 has joined #openstack-nova | 05:35 | |
| *** unicell has quit IRC | 05:36 | |
| *** zxen has quit IRC | 05:37 | |
| *** belmoreira has quit IRC | 05:38 | |
| *** nihilifer has joined #openstack-nova | 05:38 | |
| *** rushiagr_away is now known as rushiagr | 05:41 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to add 'locked_by' in extended server status https://review.openstack.org/190487 | 05:42 |
| *** ajayaa has joined #openstack-nova | 05:44 | |
| *** varya has quit IRC | 05:45 | |
| *** varya has joined #openstack-nova | 05:45 | |
| *** tjones1 has joined #openstack-nova | 05:48 | |
| *** ildikov_ has joined #openstack-nova | 05:48 | |
| *** lpetrut has joined #openstack-nova | 05:49 | |
| *** coolsvap|afk is now known as coolsvap | 05:50 | |
| *** haomaiw__ has quit IRC | 05:51 | |
| *** lsmola has joined #openstack-nova | 05:57 | |
| *** vilobhmm has joined #openstack-nova | 05:57 | |
| *** haomaiwang has joined #openstack-nova | 05:57 | |
| *** armax has quit IRC | 05:59 | |
| *** achanda has joined #openstack-nova | 05:59 | |
| *** johnthetubaguy has quit IRC | 06:00 | |
| *** mdbooth has quit IRC | 06:00 | |
| *** mdbooth has joined #openstack-nova | 06:02 | |
| *** Administrator__ has quit IRC | 06:02 | |
| *** johnthetubaguy has joined #openstack-nova | 06:02 | |
| *** Administrator__ has joined #openstack-nova | 06:03 | |
| *** Murali has joined #openstack-nova | 06:04 | |
| *** tjones1 has quit IRC | 06:05 | |
| *** heyongli has quit IRC | 06:06 | |
| *** heyongli- is now known as heyongli | 06:06 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/python-novaclient: Client exposes requests library exception. https://review.openstack.org/189540 | 06:06 |
| *** yonglihe has joined #openstack-nova | 06:06 | |
| *** sdake has quit IRC | 06:08 | |
| *** asettle has quit IRC | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/190034 | 06:10 |
| cfriesen | mriedem_away: Is this an issue? http://goo.gl/YvTVqW I hit one of these in gate-tempest-dsvm-full-ceph. | 06:10 |
| openstackgerrit | Kuo-tung Kao proposed openstack/python-novaclient: Client exposes requests library exception. https://review.openstack.org/189540 | 06:10 |
| *** ildikov_ has quit IRC | 06:12 | |
| *** yingjun has joined #openstack-nova | 06:14 | |
| *** Nikolay_St has joined #openstack-nova | 06:16 | |
| *** haomaiwang has quit IRC | 06:18 | |
| *** haomaiwa_ has joined #openstack-nova | 06:19 | |
| openstackgerrit | ZhaoBo proposed openstack/nova: Fixed raise SecurityGroupCannotBeApplied as network port_security off https://review.openstack.org/188260 | 06:20 |
| openstackgerrit | Chris Friesen proposed openstack/nova: don't leave glance with open file descriptor on error https://review.openstack.org/188179 | 06:23 |
| openstackgerrit | Jimmy McCrory proposed openstack/python-novaclient: Cache a new token when the existing token expires https://review.openstack.org/189151 | 06:24 |
| *** annashen has joined #openstack-nova | 06:24 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to add 'locked_by' in extended server status https://review.openstack.org/190487 | 06:24 |
| *** iamjarvo has quit IRC | 06:26 | |
| *** markvoelker has joined #openstack-nova | 06:28 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: CONF.allow_resize_to_same_host should check only once in controller https://review.openstack.org/186236 | 06:28 |
| *** annashen has quit IRC | 06:29 | |
| *** sahid has joined #openstack-nova | 06:30 | |
| *** achanda has quit IRC | 06:32 | |
| *** markvoelker has quit IRC | 06:32 | |
| *** ijw has quit IRC | 06:33 | |
| *** ian_brown has quit IRC | 06:33 | |
| *** aswadr has quit IRC | 06:35 | |
| *** p_deore has quit IRC | 06:35 | |
| *** mpaolino has joined #openstack-nova | 06:37 | |
| openstackgerrit | Divakar Padiyar Nandavar proposed openstack/nova: VMware: use vCenter instead of VC https://review.openstack.org/190506 | 06:37 |
| *** apuimedo has joined #openstack-nova | 06:39 | |
| *** achanda has joined #openstack-nova | 06:41 | |
| *** mpaolino has quit IRC | 06:42 | |
| *** ZZelle_ has joined #openstack-nova | 06:45 | |
| *** moshele has joined #openstack-nova | 06:46 | |
| openstackgerrit | Chris Friesen proposed openstack/nova: don't leave glance with open file descriptor on error https://review.openstack.org/188179 | 06:47 |
| *** jcoufal has joined #openstack-nova | 06:49 | |
| *** rushiagr is now known as rushiagr_away | 06:50 | |
| openstackgerrit | Anand Shanmugam proposed openstack/nova: Changing default value of description from None to empty string https://review.openstack.org/187741 | 06:50 |
| openstackgerrit | Jimmy McCrory proposed openstack/python-novaclient: Cache a new token when the existing token expires https://review.openstack.org/189151 | 06:55 |
| *** VW has joined #openstack-nova | 06:55 | |
| *** alex_klimov has joined #openstack-nova | 06:55 | |
| *** abhishekk has joined #openstack-nova | 06:59 | |
| *** VW has quit IRC | 06:59 | |
| *** arnaud____ has joined #openstack-nova | 07:08 | |
| *** exploreshaifali has joined #openstack-nova | 07:08 | |
| *** markus_z has joined #openstack-nova | 07:12 | |
| *** arnaud____ has quit IRC | 07:12 | |
| *** subscope has joined #openstack-nova | 07:15 | |
| *** sahid has quit IRC | 07:15 | |
| *** atuvenie has joined #openstack-nova | 07:17 | |
| *** fawadkhaliq has joined #openstack-nova | 07:19 | |
| *** matrohon has joined #openstack-nova | 07:20 | |
| *** vilobhmm has quit IRC | 07:21 | |
| *** otter768 has joined #openstack-nova | 07:23 | |
| *** ZZelle_ has quit IRC | 07:23 | |
| *** vilobhmm has joined #openstack-nova | 07:23 | |
| *** ndipanov has quit IRC | 07:26 | |
| *** cfriesen has quit IRC | 07:26 | |
| *** ajayaa has quit IRC | 07:27 | |
| *** bradjones has quit IRC | 07:27 | |
| *** otter768 has quit IRC | 07:27 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Remove unnecessary ValueError exception https://review.openstack.org/190525 | 07:28 |
| *** bradjones has joined #openstack-nova | 07:29 | |
| *** bradjones has quit IRC | 07:29 | |
| *** bradjones has joined #openstack-nova | 07:29 | |
| *** ajayaa has joined #openstack-nova | 07:30 | |
| *** rotbeard has joined #openstack-nova | 07:30 | |
| *** pbandzi has joined #openstack-nova | 07:31 | |
| *** achanda has quit IRC | 07:33 | |
| *** jistr has joined #openstack-nova | 07:33 | |
| *** jlanoux has joined #openstack-nova | 07:37 | |
| *** yassine has joined #openstack-nova | 07:38 | |
| *** eglynn_ has quit IRC | 07:39 | |
| *** claudiub has joined #openstack-nova | 07:39 | |
| *** dsanders has quit IRC | 07:41 | |
| *** browne has quit IRC | 07:45 | |
| *** takedakn has joined #openstack-nova | 07:46 | |
| *** vilobhmm has quit IRC | 07:46 | |
| *** ildikov_ has joined #openstack-nova | 07:47 | |
| *** pilgrimstack has joined #openstack-nova | 07:47 | |
| *** lpetrut has quit IRC | 07:52 | |
| *** takedakn has quit IRC | 07:52 | |
| *** achanda has joined #openstack-nova | 07:57 | |
| *** ZZelle has quit IRC | 08:01 | |
| *** ZZelle has joined #openstack-nova | 08:02 | |
| *** eglynn_ has joined #openstack-nova | 08:02 | |
| *** ajayaa has quit IRC | 08:03 | |
| *** haypo has joined #openstack-nova | 08:04 | |
| *** achanda_ has joined #openstack-nova | 08:08 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 08:10 |
| *** achanda has quit IRC | 08:11 | |
| *** achanda_ has quit IRC | 08:13 | |
| *** atuvenie_ has joined #openstack-nova | 08:14 | |
| *** yingjun has quit IRC | 08:15 | |
| *** atuvenie has quit IRC | 08:15 | |
| *** markvoelker has joined #openstack-nova | 08:16 | |
| *** romainh has joined #openstack-nova | 08:18 | |
| *** yingjun has joined #openstack-nova | 08:19 | |
| *** lucasagomes has joined #openstack-nova | 08:20 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** sahid has joined #openstack-nova | 08:22 | |
| *** dulek has joined #openstack-nova | 08:24 | |
| openstackgerrit | garyk proposed openstack/nova: DB: add support for volume_type by BDMS https://review.openstack.org/139992 | 08:24 |
| openstackgerrit | garyk proposed openstack/nova: Object: add volume_type_id to block_device objects https://review.openstack.org/139994 | 08:24 |
| *** annashen has joined #openstack-nova | 08:25 | |
| *** jistr has quit IRC | 08:27 | |
| *** annashen has quit IRC | 08:30 | |
| *** oomichi has joined #openstack-nova | 08:33 | |
| bauzas | morning | 08:33 |
| *** oomichi has left #openstack-nova | 08:34 | |
| *** fawadkhaliq has quit IRC | 08:35 | |
| *** fawadkhaliq has joined #openstack-nova | 08:35 | |
| gilliard | allo | 08:38 |
| lxsli | good moaning | 08:38 |
| *** dhvanan_ has joined #openstack-nova | 08:38 | |
| bauzas | moaning ? meh | 08:38 |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 08:40 |
| *** afazekas has joined #openstack-nova | 08:41 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Add microversion to allow server search option ip6 for non-admin https://review.openstack.org/179569 | 08:41 |
| *** claudiub has quit IRC | 08:41 | |
| *** ajayaa has joined #openstack-nova | 08:42 | |
| *** claudiub has joined #openstack-nova | 08:42 | |
| *** jistr has joined #openstack-nova | 08:43 | |
| frickler | ^^ rebasing microversions is no fun, when git-rebase just discards half of the updates because they are already in the code | 08:44 |
| *** ndipanov has joined #openstack-nova | 08:46 | |
| * frickler really hopes https://review.openstack.org/187835 will pass on Friday | 08:46 | |
| openstackgerrit | Joe Gordon proposed openstack/nova: Update formatting of microversion 2.4 documentation https://review.openstack.org/190549 | 08:47 |
| *** exploreshaifali has quit IRC | 08:48 | |
| openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 08:49 |
| lxsli | bauzas: reference to 80s TV show, Allo Allo | 08:49 |
| *** prashant__ has quit IRC | 08:49 | |
| bauzas | lxsli: obviously, my childness is different from yours :p | 08:49 |
| openstackgerrit | Matt Thompson proposed openstack/python-novaclient: Pass full path to pkgutil.iter_modules() https://review.openstack.org/190141 | 08:50 |
| *** danpb has joined #openstack-nova | 08:51 | |
| frickler | jogo: You didn't miss 2.4 before, it was merged in parallel with your other patch. And /me is seeing yet another rebase coming along :-S | 08:52 |
| jogo | frickler: :/ | 08:52 |
| *** dulek has quit IRC | 08:53 | |
| *** oro has joined #openstack-nova | 08:54 | |
| lxsli | bauzas: http://en.wikipedia.org/wiki/Officer_Crabtree | 08:54 |
| *** yingjun has quit IRC | 08:54 | |
| * kashyap wonders if it's just him, with the new Gerrit change view, the commit message one line summary is not in bold anymore, making me squint to read it. | 08:56 | |
| bauzas | kashyap: correct | 08:57 |
| bauzas | kashyap: you mean the first line ? | 08:57 |
| kashyap | bauzas: Yes. | 08:58 |
| bauzas | kashyap: so yup | 08:58 |
| *** pixelb has joined #openstack-nova | 09:01 | |
| *** yamahata has joined #openstack-nova | 09:06 | |
| *** RuiChen has quit IRC | 09:06 | |
| *** RuiChen has joined #openstack-nova | 09:06 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Fix overloading of block device on boot by device name https://review.openstack.org/190324 | 09:06 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: API: make sure a blank volume with no size is rejected https://review.openstack.org/188913 | 09:06 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Make Nova better at keeping track of volume sizes in BDM https://review.openstack.org/188914 | 09:06 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: compute: Make swap_volume with resize updates BDM size https://review.openstack.org/188915 | 09:06 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: snapshot: Add device_name to the snapshot bdms https://review.openstack.org/190323 | 09:06 |
| *** pilgrimstack has quit IRC | 09:06 | |
| *** pilgrimstack has joined #openstack-nova | 09:07 | |
| johnthetubaguy | jogo: ah, we got the docs patches merged then? | 09:07 |
| *** rook has quit IRC | 09:07 | |
| *** rook has joined #openstack-nova | 09:08 | |
| *** zxen has joined #openstack-nova | 09:09 | |
| *** wwriverrat has quit IRC | 09:10 | |
| *** wwriverrat has joined #openstack-nova | 09:11 | |
| *** dulek has joined #openstack-nova | 09:12 | |
| *** wxy has quit IRC | 09:13 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to add 'locked_by' in extended server status https://review.openstack.org/190487 | 09:14 |
| ndipanov | johnthetubaguy, hey so https://review.openstack.org/#/c/138607/ is really relevant to my interests | 09:15 |
| *** Administrator_ has joined #openstack-nova | 09:16 | |
| ndipanov | iiuc - it's blocked now on Jay's complaints that we need to fix the servicegropup API first? | 09:16 |
| ndipanov | is that the only thing or you had some more concerns | 09:16 |
| johnthetubaguy | ndipanov: so I wrote the current version of the spec, to try get this merged | 09:17 |
| *** Administrator__ has quit IRC | 09:17 | |
| ndipanov | ok cool - so does it somehow punt on jaypipes concerns? | 09:18 |
| *** otter768 has joined #openstack-nova | 09:23 | |
| johnthetubaguy | ndipanov: it doesn't but I think a new version should | 09:24 |
| openstackgerrit | Divakar Padiyar Nandavar proposed openstack/nova: VMware: use vCenter instead of VC https://review.openstack.org/190506 | 09:24 |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Add microversion to allow server search option ip6 for non-admin https://review.openstack.org/179569 | 09:25 |
| hanzhang | ndipanov: can you help to review this patch #https://review.openstack.org/#/c/175742 again | 09:26 |
| ndipanov | johnthetubaguy, kk let me look at the code to understand better what jay meant | 09:27 |
| johnthetubaguy | ndipanov: basically the APIs read from the DB | 09:27 |
| johnthetubaguy | ndipanov: so if you don't store stuff in the DB, then the APIs don't work, so the tooz stuff would have to sync into the DB, or fix that in the code | 09:27 |
| ndipanov | so even the zk driver would do a DB reasd? | 09:28 |
| johnthetubaguy | ndipanov: its simpler than that, the API talks to the DB later, not the service group API | 09:28 |
| johnthetubaguy | layer^ | 09:28 |
| *** otter768 has quit IRC | 09:28 | |
| ndipanov | hmmm | 09:28 |
| johnthetubaguy | basically, the abstraction is busted | 09:28 |
| *** dguitarbite has joined #openstack-nova | 09:28 | |
| ndipanov | right | 09:28 |
| ndipanov | so basically any Service.blah should be replaced by servicegroup call | 09:29 |
| ndipanov | since in case of ZK we don't even want to have a db record | 09:29 |
| johnthetubaguy | now I just want someone to give it some love, and since they stepped up to do that, I want to let them do that | 09:29 |
| johnthetubaguy | yeah | 09:29 |
| ndipanov | yes awesome | 09:29 |
| ndipanov | I will track this and maybe land a hand if needed | 09:29 |
| ndipanov | I have one more idea for liberty - will put up a spec today or tomorrow - it's around some problems in the scheduler | 09:30 |
| ndipanov | would love to hear your thoughts once I write it up | 09:30 |
| ndipanov | (from a tech not ptl perspective) | 09:31 |
| johnthetubaguy | ndipanov: so I need to write up my new scheduler design I shared at the summit too | 09:31 |
| johnthetubaguy | ndipanov: but certainly interested | 09:31 |
| johnthetubaguy | ndipanov: is this all the NUMA races you are thinking about? | 09:31 |
| ndipanov | basically | 09:32 |
| ndipanov | and pci | 09:32 |
| johnthetubaguy | right | 09:33 |
| ndipanov | well and really all other stuff that is racy too but we don't care so much | 09:33 |
| ndipanov | (rightfully so) | 09:33 |
| johnthetubaguy | so most of the new stuff is making sure the races are resolved with a commit between the resource tracker and the scheduler, before the scheduler returns the chosen host | 09:33 |
| ndipanov | well that was not what my idea is :) | 09:33 |
| johnthetubaguy | ndipanov: my view here is that long term there are likely going to be many schedulers, depending on your use case | 09:34 |
| ndipanov | right | 09:34 |
| johnthetubaguy | ndipanov: anyways, sounds interesting | 09:34 |
| ndipanov | so my idea here would be to introduce granular locking on hoststate level just to make sure internal state is more correct | 09:34 |
| johnthetubaguy | ndipanov: have you read the google omega paper, lots of folks do rave about that, its kinda interesting | 09:34 |
| ndipanov | yeah | 09:34 |
| ndipanov | well | 09:34 |
| johnthetubaguy | ndipanov: oh so thats mostly what the above is talking about as well, just not explaining it properly | 09:35 |
| *** mnestratov has joined #openstack-nova | 09:35 | |
| ndipanov | there is a BP for this already? | 09:35 |
| ndipanov | ?? | 09:35 |
| johnthetubaguy | ndipanov: so its likely to be next week before I write up this stuff, not sure if thats useful | 09:35 |
| *** yamahata has quit IRC | 09:35 | |
| ndipanov | well | 09:35 |
| johnthetubaguy | ndipanov: I have an action post summit to get these ideas written up, well alongside the scheduler folk | 09:36 |
| johnthetubaguy | ndipanov: I hope next week to do a "first pass" to get the big strokes written down for lots of this stuff | 09:36 |
| johnthetubaguy | then go back an fill in details, etc | 09:36 |
| ndipanov | the way I see the whole problem is this - unless you care about numa/pci - our scheduler is probably fine except for the read the world thing which could be improved | 09:37 |
| johnthetubaguy | ndipanov: turns out its not fine, because you can't run multiple schedulers without massive races | 09:37 |
| ndipanov | right | 09:37 |
| johnthetubaguy | cells works around that a little bit, but it breaks in other interesting ways | 09:37 |
| ndipanov | but in case of counting mem and cpu | 09:37 |
| ndipanov | races hit you only at peak capacity | 09:38 |
| johnthetubaguy | I think thinking purely counting mem, it still breaks a lot | 09:38 |
| ndipanov | and even then they manifest as retries | 09:38 |
| ndipanov | it does of course | 09:38 |
| ndipanov | I keep thinking that's by design | 09:38 |
| ndipanov | :) | 09:38 |
| johnthetubaguy | you are right though, the problems are much smaller until you add NUMA and PCI | 09:39 |
| ndipanov | johnthetubaguy, and ironic | 09:39 |
| ndipanov | lifeless, has a bug open for this | 09:39 |
| johnthetubaguy | well, yes, sort of | 09:39 |
| johnthetubaguy | ndipanov: you got a link for that? | 09:39 |
| ndipanov | lemme look | 09:39 |
| ndipanov | https://bugs.launchpad.net/nova/+bug/1341420 | 09:40 |
| openstack | Launchpad bug 1341420 in OpenStack Compute (nova) "gap between scheduler selection and claim causes spurious failures when the instance is the last one to fit" [High,Confirmed] | 09:40 |
| ndipanov | basically complaining about the same thing | 09:40 |
| johnthetubaguy | so we run the ironic stuff in production, I know we have some issues with it | 09:40 |
| johnthetubaguy | ah, right, thats the same issue | 09:40 |
| ndipanov | that scheduler is racy even internally with multiple threads | 09:40 |
| ndipanov | but yes if you want to run multiple workers | 09:40 |
| ndipanov | you need consensus, | 09:40 |
| openstackgerrit | Zhang Ni proposed openstack/nova-specs: Add local-to-instance to create server api https://review.openstack.org/190561 | 09:41 |
| ndipanov | so my proposal will be | 09:41 |
| johnthetubaguy | (given how our db threading works, turns out you want the evenlet thread count to be less than 4 for max performance...) | 09:42 |
| ndipanov | johnthetubaguy, less threads less races :D | 09:42 |
| ndipanov | it's almost a good thing | 09:42 |
| johnthetubaguy | well, its more odd than that, but yes | 09:42 |
| * ndipanov jokes | 09:42 | |
| openstackgerrit | Zhang Ni proposed openstack/nova-specs: Add local-to-instance to create server api https://review.openstack.org/190561 | 09:42 |
| johnthetubaguy | the db data gets stale by the time you get back to process it, when there are lots of requests | 09:42 |
| *** yingjun has joined #openstack-nova | 09:42 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova-specs: Support force detach volume to nova https://review.openstack.org/84048 | 09:42 |
| johnthetubaguy | due to how the eventlet scheduling works, sigh | 09:43 |
| ndipanov | johnthetubaguy, ah that | 09:43 |
| johnthetubaguy | (like service group reports get old) | 09:43 |
| ndipanov | so there is another way to look at that problem | 09:43 |
| johnthetubaguy | thats while I did the caching_scheduler that we run | 09:43 |
| ndipanov | in our current design | 09:43 |
| ndipanov | that does no locking ever | 09:43 |
| johnthetubaguy | but lets not get side tracked... continue | 09:43 |
| ndipanov | and is racy | 09:43 |
| ndipanov | why tf do we switch trheads | 09:43 |
| ndipanov | (I know why because filters do db access) | 09:43 |
| *** achanda has joined #openstack-nova | 09:44 | |
| johnthetubaguy | it might have been the logging, I don't remember now | 09:44 |
| ndipanov | ah yes logging | 09:44 |
| lifeless | ndipanov: or partitioning | 09:44 |
| lifeless | ndipanov: [to needing consensus]. | 09:44 |
| *** dims_ has joined #openstack-nova | 09:44 | |
| ndipanov | lifeless, yes | 09:45 |
| ndipanov | that is another approach | 09:45 |
| johnthetubaguy | lifeless: we actually have some cool looking scheduler plans, just need to write them up and get them ripped to shreds | 09:45 |
| johnthetubaguy | so my proposal kinda talks about using both, I so need to write that all up | 09:45 |
| openstackgerrit | Zhang Ni proposed openstack/nova-specs: Add local-to-instance to create server api https://review.openstack.org/190561 | 09:46 |
| *** kiran-r is now known as kiran-r|afk | 09:46 | |
| *** kiran-r|afk is now known as kiran-r | 09:46 | |
| ndipanov | so one of the reasons why I was really hoping to get the interface solidified | 09:46 |
| ndipanov | which seems to be stalled a bit | 09:46 |
| johnthetubaguy | so I think we have quite a bit of agreement on what that should be, but its all the ground work thats taking ages | 09:46 |
| johnthetubaguy | I am hoping to get the big strokes down on paper to speed that up | 09:47 |
| *** dims has quit IRC | 09:47 | |
| ndipanov | it would be much easier to do swapable implementations like paritioning/consensus | 09:47 |
| johnthetubaguy | (getting stuck in debates about service group apis being one example) | 09:47 |
| johnthetubaguy | ndipanov: +1 | 09:47 |
| ndipanov | a big ball of mud | 09:47 |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 09:47 |
| johnthetubaguy | its totally blocking progress right now, thats why its a priority | 09:47 |
| johnthetubaguy | similar to the API work before | 09:48 |
| ndipanov | johnthetubaguy, which exact spec is that? | 09:48 |
| johnthetubaguy | ndipanov: its a theme, we don't have the list of specs yet, the scheduler group are yet to agree that, as I understand | 09:48 |
| bauzas | correct | 09:49 |
| * bauzas just takes the convo now | 09:49 | |
| bauzas | ndipanov: https://etherpad.openstack.org/p/YVR-nova-scalling-out-scheduler-for-cells was part of the discussion | 09:49 |
| ndipanov | hanzhang, hey so back to you | 09:50 |
| ndipanov | I though I +2ed that patch | 09:50 |
| ndipanov | what changed? | 09:50 |
| * bauzas also just discovers jaypipes's disagreement on https://review.openstack.org/#/c/138607/ | 09:50 | |
| *** yingjun has quit IRC | 09:53 | |
| *** yingjun has joined #openstack-nova | 09:53 | |
| johnthetubaguy | bauzas: I am just updating that, jaypipes has a very valid point, as he usually does | 09:53 |
| ndipanov | bauzas, so the append only log approach for updates was proposed by boris-42 in Atlanta | 09:53 |
| bauzas | johnthetubaguy: eh :) | 09:53 |
| bauzas | johnthetubaguy: yeah I saw that, MHO is maybe that we need to call the SG API only when needed | 09:54 |
| ndipanov | throwing kafka into the mix is probably using the right tool for the job (they had something with memcached working) | 09:54 |
| bauzas | johnthetubaguy: ie. for example, knowing if the service is disabled doesn't need to call the SG API | 09:54 |
| boris-42 | ndipanov: hm? | 09:54 |
| bauzas | ndipanov: yeah, there are 2 items we would like to work on | 09:55 |
| *** mpaolino has joined #openstack-nova | 09:55 | |
| johnthetubaguy | ndipanov: its an implementation detail, but yeah | 09:55 |
| boris-42 | ndipanov: scheduler? | 09:55 |
| bauzas | ndipanov: 1/ use some ways to increment the updates | 09:55 |
| *** Daviey has quit IRC | 09:55 | |
| *** Daviey has joined #openstack-nova | 09:55 | |
| johnthetubaguy | bauzas: so I need to write up my proposal, I am hoping to do that next week | 09:55 |
| bauzas | 2/ see how we can scale out by having an optimistic scheduler | 09:55 |
| *** fawadkhaliq has quit IRC | 09:55 | |
| bauzas | johnthetubaguy: np, I'm really busy these days with ReqSpec | 09:55 |
| ndipanov | but as I said above - all of that is much harder until we have a clear interface for talking to the scheduler | 09:56 |
| johnthetubaguy | bauzas: thats cool, thats important stuff | 09:56 |
| bauzas | johnthetubaguy: my review ratio dropped dramatically | 09:56 |
| ndipanov | boris-42, yea | 09:56 |
| bauzas | ndipanov: that's exactly why we're working this way | 09:56 |
| bauzas | ndipanov: johnthetubaguy is trying to draft some proposal | 09:56 |
| bauzas | ndipanov: so we can see what's needed or not | 09:57 |
| sdague | morning folks | 09:57 |
| bauzas | ndipanov: and then, we'll see how to do thru this direction | 09:57 |
| ndipanov | johnthetubaguy, so my spec will be of a much smaller scope than that | 09:57 |
| boris-42 | ndipanov: heh never ending story | 09:57 |
| *** yingjun has quit IRC | 09:58 | |
| ndipanov | yea I think we probably should have bit the bullet and moved away from the db for the scheduler earlier | 09:58 |
| sdague | johnthetubaguy: so... given that we're now defaulting to the python native mysql driver, have you run that in your pre-prod cluster to see how it moves around the db performance issues? | 09:58 |
| ndipanov | but I can see how that was scary :) | 09:58 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Replace Zookeeper, Memcache servicegroup driver with respective Tooz drivers https://review.openstack.org/138607 | 10:00 |
| johnthetubaguy | sdague: in a word... no | 10:01 |
| ndipanov | heh | 10:01 |
| johnthetubaguy | sdague: I want to test that for nova-cells in a production region soon, and ideally test out the conductor using it | 10:01 |
| *** varya has quit IRC | 10:01 | |
| johnthetubaguy | sdague: (we don't use conductor yet) | 10:01 |
| johnthetubaguy | sdague: honestly, I think alaski is going to be more upto date with our plans there at this point | 10:02 |
| *** pask81 has joined #openstack-nova | 10:02 | |
| sdague | gotcha | 10:02 |
| johnthetubaguy | sdague: taking a step back from the internal stuff, except for when it all breaks and alaski is on holiday or asleep | 10:02 |
| johnthetubaguy | at least, thats my current plan! | 10:02 |
| johnthetubaguy | sdague: we should totally get that data for you soon though | 10:03 |
| sdague | it would be interesting to see how that impacts decisions based on "foo is the bottleneck in the db layer" | 10:03 |
| sdague | because I think those are all going to move around a bunch | 10:03 |
| johnthetubaguy | so historically nova-cells is where we hit DB being a bottleneck | 10:03 |
| *** achanda has quit IRC | 10:04 | |
| johnthetubaguy | we actually have a hand coded mysqldb based solution right now, which we are trying to drop, and to try pymsql thingy again, just last time we tested that, it performed much worse | 10:04 |
| johnthetubaguy | although we just turned off the threadpooling we used to use with that (required a hacked up eventlet, and that didn't survive the eventlet bump), so I suspect it will win this time | 10:05 |
| *** markvoelker has joined #openstack-nova | 10:05 | |
| *** dulek_ has joined #openstack-nova | 10:05 | |
| johnthetubaguy | once we got the ability to run two nova-cells workers going (those fixes are totally upstream, as I understand it), we stopped looking at that too much at that stuff | 10:06 |
| johnthetubaguy | anyways, there is some context | 10:06 |
| *** dulek has quit IRC | 10:07 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 tests - take 2 https://review.openstack.org/188785 | 10:08 |
| johnthetubaguy | bauzas: jaypipes: I just updated that tooz spec to see if that sounds better: https://review.openstack.org/#/c/138607/29 | 10:08 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: WIP: adding a volume detach_force call https://review.openstack.org/184537 | 10:08 |
| bauzas | johnthetubaguy: will see how I can take a look today | 10:08 |
| *** dulek_ has quit IRC | 10:09 | |
| bauzas | johnthetubaguy: I remember the spec review day being tomorrow so I would like it to be merged soon | 10:09 |
| *** zhenguo has quit IRC | 10:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] cleanup openstack-common.conf and sync updated files https://review.openstack.org/189118 | 10:09 |
| bauzas | I should probably discuss that with jaypipes too | 10:09 |
| *** markvoelker has quit IRC | 10:10 | |
| *** achanda has joined #openstack-nova | 10:10 | |
| *** fawadkhaliq has joined #openstack-nova | 10:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 tests https://review.openstack.org/188243 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 tests - take 2 https://review.openstack.org/188785 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/virt/vmwareapi/test*.py https://review.openstack.org/188653 | 10:10 |
| johnthetubaguy | bauzas: +1 | 10:12 |
| *** ajayaa has quit IRC | 10:12 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Remove openstack/common/versionutils module https://review.openstack.org/189118 | 10:15 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Sync with latest oslo-incubator https://review.openstack.org/190566 | 10:15 |
| *** dhvanan_ has quit IRC | 10:15 | |
| *** achanda has quit IRC | 10:15 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Fix formatting issue of rest_api_version_history https://review.openstack.org/190567 | 10:16 |
| *** ajayaa has joined #openstack-nova | 10:16 | |
| *** asselin has quit IRC | 10:20 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Fix formatting issue of rest_api_version_history https://review.openstack.org/190567 | 10:20 |
| *** kmartin has quit IRC | 10:21 | |
| *** achanda has joined #openstack-nova | 10:21 | |
| *** achanda has quit IRC | 10:22 | |
| *** exploreshaifali has joined #openstack-nova | 10:25 | |
| *** fawadkhaliq has quit IRC | 10:27 | |
| *** annashen has joined #openstack-nova | 10:28 | |
| *** ociuhandu has joined #openstack-nova | 10:30 | |
| *** VW has joined #openstack-nova | 10:32 | |
| *** annashen has quit IRC | 10:32 | |
| openstackgerrit | sahid proposed openstack/nova-specs: console: convert consoles code to use objects framework https://review.openstack.org/165838 | 10:34 |
| *** ociuhandu has quit IRC | 10:35 | |
| *** gszasz has joined #openstack-nova | 10:35 | |
| *** VW has quit IRC | 10:36 | |
| *** romainh has quit IRC | 10:36 | |
| *** fawadkhaliq has joined #openstack-nova | 10:40 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Remove openstack/common/versionutils module https://review.openstack.org/189118 | 10:42 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Sync with latest oslo-incubator https://review.openstack.org/190566 | 10:42 |
| *** ildikov_ has quit IRC | 10:43 | |
| *** tojuvone has joined #openstack-nova | 10:44 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 10:45 |
| *** lpetrut has joined #openstack-nova | 10:45 | |
| *** atuvenie_ has quit IRC | 10:47 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova-specs: Support force detach volume to nova https://review.openstack.org/84048 | 10:50 |
| frickler | I'm getting errors on running tox locally because of this: ERROR [nova.wsgi] Could not bind to 0.0.0.0:8774 | 10:53 |
| *** ildikov_ has joined #openstack-nova | 10:53 | |
| frickler | shouldn't any real socket operations be stubbed out for unit tests? | 10:53 |
| *** romainh has joined #openstack-nova | 10:56 | |
| openstackgerrit | garyk proposed openstack/nova: DB: add support for volume_type by BDMS https://review.openstack.org/139992 | 10:58 |
| *** tojuvone has quit IRC | 10:59 | |
| bauzas | frickler: which errors ? | 10:59 |
| bauzas | frickler: I mean which tests are failign ? | 10:59 |
| frickler | nova.tests.unit.test_service.TestWSGIService.test_workers_set_default and others from nova/tests/unit/test_service.py | 11:01 |
| *** kbyrne has joined #openstack-nova | 11:03 | |
| *** atuvenie_ has joined #openstack-nova | 11:04 | |
| ndipanov | johnthetubaguy, fwiw - I like almost all of the ideas here: https://etherpad.openstack.org/p/liberty-nova-scalable-scheduler even added some comments | 11:05 |
| bauzas | frickler: what shows lsof -i :8774 ? | 11:06 |
| *** oro has quit IRC | 11:06 | |
| openstackgerrit | Abhijeet Malawade proposed openstack/nova: Correct disk usages for boot-from-volume instances https://review.openstack.org/189581 | 11:06 |
| *** pask81 has quit IRC | 11:06 | |
| bauzas | frickler: it was if like you were running tests while running nova-api at the same time | 11:06 |
| frickler | bauzas: it shows some other process listening on that port, but my point is that this really shouldn't matter for unit tests | 11:06 |
| bauzas | frickler: so I guess you're testing on your devstack VM with n-api already started | 11:06 |
| *** achanda has joined #openstack-nova | 11:07 | |
| *** aix has quit IRC | 11:07 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova-specs: Hyper-V: Add storage QoS support https://review.openstack.org/175422 | 11:07 |
| frickler | bauzas: I'm testing patches on my workstation before I submit them (at least I try to most of the time) | 11:07 |
| bauzas | frickler: well, that's a debate on if a port testing needs to be isolated or not | 11:07 |
| bauzas | frickler: but since all tests are explicitely trying to start the osapi_compute service, you have to assume your local port to be freed | 11:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/nova: Preserve usage and reservations on quota deletion https://review.openstack.org/163423 | 11:11 |
| frickler | bauzas: hmm, o.k., do you know whether this was discussed somewhere before? | 11:11 |
| bauzas | frickler: that said, there is a OSAPIFixture that provides what you need | 11:11 |
| bauzas | frickler: I guess you could propose a change using that fixture to have a full isolation | 11:12 |
| gilliard | Nova team meeting in about 4h, right? | 11:12 |
| *** subscope has quit IRC | 11:12 | |
| gilliard | or 3h. anyway, it's an early week isn't it? | 11:13 |
| *** openstackgerrit has quit IRC | 11:13 | |
| *** openstackgerrit has joined #openstack-nova | 11:14 | |
| bauzas | gilliard: 1400UTC today yes | 11:14 |
| *** subscope has joined #openstack-nova | 11:14 | |
| bauzas | frickler: tbp, that fixture is not usable as it is, but the idea is to say that since you want to test the service, allocating a dynamic port for testing other than that is acceptable | 11:15 |
| sdague | bauzas: what changes would be needed to the fixture to work in this other case? | 11:15 |
| frickler | bauzas: seems a bit strange to me that this is used for functional tests but not in the unit tests | 11:15 |
| bauzas | sdague: I mean, the service testcase explicitely checks the WSGI service | 11:16 |
| bauzas | sdague: while the fixture just creates it but doesn't pass the reference | 11:17 |
| sdague | bauzas: gotcha, is that test relevant any more now that a bunch of this stuff is shuffled off to oslo? | 11:17 |
| bauzas | sdague: fair point | 11:18 |
| *** baoli has joined #openstack-nova | 11:18 | |
| *** varya has joined #openstack-nova | 11:19 | |
| sdague | frickler: so... when looking at oddities in the tests, realize we're 5 years in on this code base. The OSAPIFixture was added last cycle to consolidate (and clean up) a bunch of code in the api samples tests. There are lots of tests that remain crufty and wonky. | 11:19 |
| *** setuid has quit IRC | 11:20 | |
| *** setuid has joined #openstack-nova | 11:20 | |
| sdague | but, I'd agree, make OSAPIFixture also keep a service reference, then move these tests over | 11:20 |
| bauzas | yup | 11:20 |
| *** markvoelker has joined #openstack-nova | 11:21 | |
| sdague | it's actually kind of amazing, given testr, we didn't hit this more often in the past actually | 11:22 |
| bauzas | eh | 11:24 |
| sdague | all of these tests could be running at the same time | 11:24 |
| frickler | sdague: I'm fine with that, I just stumbled upon this when testing a completely unrelated patch, and that got me wondering whether this is something that could be improved or whether there is some reasoning hidden behind it which I fail to see | 11:24 |
| *** otter768 has joined #openstack-nova | 11:24 | |
| *** achanda has quit IRC | 11:25 | |
| *** markvoelker has quit IRC | 11:25 | |
| openstackgerrit | garyk proposed openstack/nova: Object: add volume_type_id to block_device objects https://review.openstack.org/139994 | 11:25 |
| frickler | sdague: and my expectation was that running "tox -epy27" on my workstation should do its nice little tests within its venv and not even look at anything outside. and implicitly opening listening sockets on 0.0.0.0 instead of at least using 127.0.0.1 could be even considered a security issue | 11:28 |
| sdague | frickler: you are running arbitrary code from the internet that could dynamically rewrite itself. So binding to 0.0.0.0 is the least of your concerns :) | 11:29 |
| *** otter768 has quit IRC | 11:29 | |
| sdague | frickler: long term, we should probably move tests like this over to the functional side probably, but there are 15k tests, and the split this far took much of the last cycle. | 11:31 |
| frickler | sdague: right, so if we agree on "if someone ever has time, this should be improved", then that's a completely sufficient answer to me | 11:32 |
| sdague | frickler: oh, yeh, we agree on that | 11:33 |
| johnthetubaguy | frickler: totally the right question on this | 11:36 |
| johnthetubaguy | sdague: I guess we could note this when we get our test stragegy written down in devref stuff | 11:36 |
| sdague | johnthetubaguy: sure | 11:38 |
| *** deepthi has quit IRC | 11:41 | |
| *** thorst has joined #openstack-nova | 11:42 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 11:43 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: docs: Add bug handling description to developer reference https://review.openstack.org/187571 | 11:45 |
| *** baoli has quit IRC | 11:45 | |
| *** baoli has joined #openstack-nova | 11:46 | |
| *** oro has joined #openstack-nova | 11:48 | |
| bauzas | ndipanov: have a sec for a discussion about pci_requests ? | 11:49 |
| bauzas | ndipanov: since you worked on the numa stuff, you had to look at it | 11:49 |
| *** derekh has joined #openstack-nova | 11:49 | |
| bauzas | ndipanov: so my question is, there is an instance field called pci_requests, but also a filter property called pci_requests | 11:50 |
| bauzas | ndipanov: do you know the difference between both ? | 11:50 |
| *** thorst has left #openstack-nova | 11:51 | |
| * bauzas really likes to work on unversioned and undocumented dicts as blobs | 11:52 | |
| *** eglynn_ has quit IRC | 11:52 | |
| *** eglynn__ has joined #openstack-nova | 11:52 | |
| *** VW has joined #openstack-nova | 11:52 | |
| *** markvoelker has joined #openstack-nova | 11:56 | |
| *** marcusvrn has joined #openstack-nova | 11:57 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Add config drive support for Virtuozzo containers https://review.openstack.org/149253 | 11:58 |
| ndipanov | bauzas, lemme look | 11:58 |
| bauzas | ndipanov: IIUC, it looks the same, but they are used in 2 separate codepaths | 11:59 |
| bauzas | ndipanov: at least, it's the same object type | 11:59 |
| bauzas | ie. InstancePCIRequests (with an ending 's') | 12:00 |
| *** Murali_ has joined #openstack-nova | 12:00 | |
| *** alexpilotti has joined #openstack-nova | 12:01 | |
| *** iamjarvo has joined #openstack-nova | 12:03 | |
| *** Murali has quit IRC | 12:04 | |
| *** ferest has joined #openstack-nova | 12:04 | |
| ndipanov | bauzas, it's the same thing | 12:04 |
| *** setuid has quit IRC | 12:05 | |
| *** claudiub has quit IRC | 12:05 | |
| ndipanov | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1122 <- bauzas | 12:05 |
| *** Murali_ has quit IRC | 12:05 | |
| *** setuid has joined #openstack-nova | 12:05 | |
| bauzas | ndipanov: yeah found that, but it doesn't mean that the instance field is based on the same reference, right ? | 12:06 |
| *** Murali has joined #openstack-nova | 12:06 | |
| ndipanov | pretty much does since base options is what we create instance records with | 12:06 |
| bauzas | ndipanov: because here, it takes the flavor metadata and creates an instance pic objects | 12:06 |
| ndipanov | look it up earlier in that functions | 12:07 |
| bauzas | ndipanov: oh ok, so that's 2 distinct objects but having the same fields | 12:07 |
| ndipanov | um no it's a dict | 12:07 |
| bauzas | ndipanov: just to make sure, the instance fields are set before that call ? | 12:07 |
| bauzas | ndipanov: yeah that's my point | 12:07 |
| bauzas | ndipanov: see https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L265-L267 | 12:08 |
| bauzas | ndipanov: okay, so my point is : I only need to provide one pci_requests and make sure it's an object | 12:08 |
| *** aix has joined #openstack-nova | 12:09 | |
| *** alexpilotti has quit IRC | 12:09 | |
| bauzas | ndipanov: because it's based on the same flavor, but since the instance and the filter_properties were different, we had to have like this ^ above ugly if | 12:09 |
| ndipanov | bauzas, not sure I understand | 12:09 |
| bauzas | ndipanov: the current proposal for ReqSpec only cares about one pci_reqs field | 12:10 |
| *** rfolco has joined #openstack-nova | 12:10 | |
| bauzas | ndipanov: but I just discovered there were actually 2 distinct pci_reqs, hence my question | 12:10 |
| ndipanov | so imho they should be dropped from filter properties and all places that use them should be moved to use it on the instance | 12:10 |
| *** ociuhandu has joined #openstack-nova | 12:10 | |
| bauzas | ndipanov: yeah, that's my point, we only need 1 | 12:10 |
| bauzas | ndipanov: so I'm taking the instance one as a reference | 12:11 |
| *** dulek has joined #openstack-nova | 12:13 | |
| *** ociuhandu has quit IRC | 12:18 | |
| *** sahid has quit IRC | 12:26 | |
| danpb | ndipanov: could you do a review of this for me https://review.openstack.org/#/c/132023/ - you already gave +2 to every other patch before & after it in that series | 12:26 |
| openstackgerrit | Chris Gacsal proposed openstack/nova: Allow multiple volume attachments https://review.openstack.org/153033 | 12:27 |
| ndipanov | hmm must have missed that one | 12:31 |
| ndipanov | ah I remember why | 12:31 |
| ndipanov | danpb, done | 12:32 |
| *** aysyanne has joined #openstack-nova | 12:33 | |
| *** ociuhandu has joined #openstack-nova | 12:33 | |
| *** sdake has joined #openstack-nova | 12:33 | |
| danpb | ndipanov: thankyou | 12:34 |
| lpetrut | danpb: Hi. Could you please take a look over the following bug fix on Libvirt, targeting the SMB volume driver? https://review.openstack.org/#/c/187619/ This affects our Cinder CI | 12:35 |
| lpetrut | danpb: thanks in advance | 12:35 |
| *** ivasev has joined #openstack-nova | 12:37 | |
| danpb | lpetrut: ok i'll look | 12:38 |
| lpetrut | danpb: appreciate it | 12:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/190362 | 12:39 |
| *** dprince has joined #openstack-nova | 12:39 | |
| *** marun has joined #openstack-nova | 12:40 | |
| *** derekh has quit IRC | 12:40 | |
| jaypipes | danpb: hey, quick question for you... what is the difference between LocalImage and LocalFileImage? | 12:40 |
| *** derekh has joined #openstack-nova | 12:42 | |
| *** yassine has quit IRC | 12:42 | |
| *** ferest has quit IRC | 12:43 | |
| *** atuvenie__ has joined #openstack-nova | 12:43 | |
| *** nihilifer has quit IRC | 12:43 | |
| *** marun has quit IRC | 12:45 | |
| *** atuvenie_ has quit IRC | 12:45 | |
| jogo | danpb: re: https://review.openstack.org/#/c/181781/3 did you look at dstat http://logs.openstack.org/89/179689/34/check/check-tempest-dsvm-centos7/c7fb267/logs/screen-dstat.txt.gz ? | 12:46 |
| danpb | jogo: no, i didn't see the dstat | 12:47 |
| danpb | jaypipes: localimage is an abstract base for any image that is exposed in the filesystem | 12:48 |
| *** wanghua has quit IRC | 12:48 | |
| danpb | LocalFileImage is for images backed by plain files, while LocalBlockImage is for images backed by block devices | 12:48 |
| *** kudryashova has joined #openstack-nova | 12:48 | |
| *** kudryashova has quit IRC | 12:48 | |
| jaypipes | danpb: ah, got it. thx. | 12:48 |
| danpb | this distinction is because the VFS code needs a different approach for each - in plain file case it must use a loop or qemu-nbd device | 12:49 |
| danpb | whereas in the block device case it can mount it directly | 12:49 |
| jogo | danpb: it may be useful to figure out if we are hitting any hardware (virtual machine) bottlenecks | 12:49 |
| jaypipes | danpb: thx for the explanation, very useful. | 12:50 |
| *** jcoufal has quit IRC | 12:51 | |
| openstackgerrit | Dan Prince proposed openstack/nova: Revert "Add the swap and ephemeral BDMs if needed" https://review.openstack.org/190622 | 12:51 |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova-specs: Adds Hyper-V imagecache cleanup https://review.openstack.org/190625 | 12:53 |
| danpb | jogo: i guess if the underlying cloud we run gate on over-commits cpu / ram to any significant level and we end up with multiple gate jobs on the same host we could easily suffer significant delays | 12:53 |
| *** fifieldt has quit IRC | 12:53 | |
| * danpb has no idea how much rax/hp overcomit their hardware though | 12:54 | |
| gilliard | no overcommit in hp | 12:54 |
| *** vladikr has joined #openstack-nova | 12:55 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/nova-specs: Adds Hyper-V imagecache cleanup https://review.openstack.org/190625 | 12:55 |
| *** yassine_ has joined #openstack-nova | 12:55 | |
| *** Murali has quit IRC | 12:55 | |
| *** tobe has quit IRC | 12:55 | |
| jogo | danpb: can us using qemu and not kvm in the gate be part of the issue? | 12:55 |
| danpb | jogo: i don't think it should matter - shutdown doesn't have any dependancy on the performance of the 2nd level guest | 12:56 |
| *** Murali has joined #openstack-nova | 12:56 | |
| danpb | so when we send sigkill it should be killed just as fast by kerenl whether its qemu or kvm | 12:57 |
| kashyap | jogo: But, would be nice if _one_ of the cloud providers switched to KVM in the gate (for nested). | 12:57 |
| * kashyap is randomly chiming in | 12:57 | |
| *** tobe has joined #openstack-nova | 12:57 | |
| danpb | kashyap: the cloud providers are all using kvm | 12:58 |
| *** ajayaa has quit IRC | 12:58 | |
| jogo | kashyap: well rax uses Xen | 12:58 |
| danpb | they just don't enable nested kvm afaik | 12:58 |
| kashyap | jogo: Monty Taylor mentioned in one of the Paris sessions about ther _is_ a plan to switch to nested KVM. | 12:58 |
| danpb | oh, well yeah. that's true | 12:58 |
| kashyap | danpb: Yeah, I was referring to the latter. | 12:58 |
| danpb | forgot rax still used xen | 12:58 |
| kashyap | jogo: For the HP cloud, I mean. But not sure where that went. | 12:59 |
| openstackgerrit | garyk proposed openstack/nova: VMware: Use actual VM state instead of using the instance vm_state https://review.openstack.org/190627 | 12:59 |
| kashyap | Probably should ask on the -infra mailing list | 12:59 |
| sdague | kashyap: honestly, I don't think that's something that should be banked on. | 12:59 |
| *** samueldmq has joined #openstack-nova | 12:59 | |
| sdague | it's the cloud, you get what you get | 12:59 |
| *** ajayaa has joined #openstack-nova | 12:59 | |
| kashyap | sdague: Sure, not "banking" on it. Just that is that explored, if there are any bottlenecks or other technical issues | 13:00 |
| kashyap | That can be talked with the KVM community - as they're very responsive on bug reports in this area | 13:00 |
| danpb | for the purpose of tempest testing, qemu vs kvm shouldn't really matter for the majority of cases | 13:00 |
| kashyap | (And in general too.) | 13:00 |
| *** exploreshaifali has quit IRC | 13:00 | |
| danpb | there's only a handful of areas where it has a fnuctional impact on stuff we test | 13:00 |
| sdague | right, agreed, especially when using cirros, which is so minimal in what's in there | 13:00 |
| danpb | off the top of my head, there is some hyper timer source that requires kvm | 13:01 |
| danpb | and per-vcpu numa cpu affinity | 13:01 |
| jogo | danpb: anyway dstat is good at answering most questions about hardware bottlenecks and if something is missing from it we can add some more | 13:01 |
| sdague | I mean, we peg our core during cpu boot, but it's only about 20s | 13:01 |
| *** ajayaa has quit IRC | 13:01 | |
| *** tobe has quit IRC | 13:02 | |
| *** xyang1 has joined #openstack-nova | 13:02 | |
| jogo | sdague: right | 13:02 |
| jogo | sdague: btw your review on https://review.openstack.org/#/c/188410/4 would be very helpful | 13:03 |
| sdague | ok, added to my queue | 13:03 |
| jogo | sdague: it is about making novaclient support microversions | 13:04 |
| sdague | yep | 13:04 |
| *** josecastroleon has quit IRC | 13:05 | |
| *** Longgeek_ has quit IRC | 13:06 | |
| jogo | johnthetubaguy: do you think we need a spec for this: https://review.openstack.org/#/c/190625/ ? | 13:07 |
| *** garyk1 has quit IRC | 13:09 | |
| beagles | danpb: unless you have an objection, to address johnthetubaguy's comments on the vif-plug-script I'm going to include your comments pretty much verbatim and add you as co-author, Okay with you? | 13:10 |
| danpb | fine | 13:11 |
| *** atuvenie__ has quit IRC | 13:12 | |
| *** garyk has joined #openstack-nova | 13:12 | |
| johnthetubaguy | jogo: looking | 13:13 |
| *** mdrabe has joined #openstack-nova | 13:13 | |
| jaypipes | johnthetubaguy: reading your updates to the tooz spec now. | 13:13 |
| jaypipes | johnthetubaguy: FYI, the other spec you were asking about is this one: https://review.openstack.org/#/c/190322/ | 13:13 |
| sdague | jogo: ok, commented | 13:14 |
| johnthetubaguy | jaypipes: ah, sweet, so we should probably add that as a dependency then | 13:14 |
| sdague | jogo: I'm a little concerned the spec missed the point of "what do we expect novaclient to look like" | 13:15 |
| jogo | sdague: :/ well best to catch these issues now | 13:15 |
| sdague | yep, I have a longish comment about what I'd expect | 13:15 |
| sdague | see if you agree with it | 13:15 |
| danpb | jaypipes: haha, thanks for not -1'ing me for spelling Cthulhu wrong ;-P | 13:16 |
| jaypipes | hehe :) | 13:17 |
| *** salv-orlando has quit IRC | 13:18 | |
| * danpb wonders if anyone has noticed the other special mathematical numbers he's slipped into nova source | 13:18 | |
| *** atuvenie has joined #openstack-nova | 13:19 | |
| *** salv-orlando has joined #openstack-nova | 13:19 | |
| johnthetubaguy | jogo: added it the meeting agenda, seems like we don't need a spec | 13:20 |
| *** zzzeek has joined #openstack-nova | 13:21 | |
| *** irenab has quit IRC | 13:22 | |
| jogo | sdague: agreed, the spec doesn't really mention a lot about the UX | 13:24 |
| *** salv-orl_ has joined #openstack-nova | 13:24 | |
| *** salv-orlando has quit IRC | 13:25 | |
| *** marun has joined #openstack-nova | 13:25 | |
| johnthetubaguy | curious, which spec is this you are looking at jogo sdague ? | 13:25 |
| *** otter768 has joined #openstack-nova | 13:25 | |
| jogo | johnthetubaguy: https://review.openstack.org/#/c/188410/4 | 13:25 |
| johnthetubaguy | I have been looking at all the really old specs and working forward, so don't seem to be cross paths with too many folks right now | 13:25 |
| jogo | novaclient microversions | 13:26 |
| johnthetubaguy | ah, cools | 13:26 |
| *** nelsnelson has joined #openstack-nova | 13:28 | |
| *** unicell has joined #openstack-nova | 13:30 | |
| *** unicell1 has quit IRC | 13:30 | |
| *** annashen has joined #openstack-nova | 13:30 | |
| *** otter768 has quit IRC | 13:30 | |
| *** unicell1 has joined #openstack-nova | 13:31 | |
| *** abhishekk has quit IRC | 13:32 | |
| *** ociuhandu has quit IRC | 13:32 | |
| *** setuid has quit IRC | 13:33 | |
| *** unicell has quit IRC | 13:34 | |
| *** annashen has quit IRC | 13:35 | |
| *** setuid has joined #openstack-nova | 13:36 | |
| *** eharney has quit IRC | 13:38 | |
| *** mriedem_away is now known as mriedem | 13:38 | |
| nagyz | is there currently any way to have two different ephemeral types supported (let's say local qcow based and ceph) and decide on a per-flavor basis which one to use? | 13:40 |
| nagyz | if I'm right currently the only way is to run different host aggregates with different nova.confs and bind the flavor to the host aggregate? | 13:40 |
| *** yassine__ has joined #openstack-nova | 13:41 | |
| *** nelsnelson has quit IRC | 13:41 | |
| *** VW has quit IRC | 13:43 | |
| *** yassine_ has quit IRC | 13:44 | |
| *** VW has joined #openstack-nova | 13:44 | |
| *** fawadkhaliq has quit IRC | 13:45 | |
| *** alexpilotti has joined #openstack-nova | 13:46 | |
| *** claudiub has joined #openstack-nova | 13:47 | |
| bauzas | jaypipes: around for discussing https://review.openstack.org/#/c/138607/ . | 13:47 |
| bauzas | . | 13:47 |
| bauzas | ? | 13:47 |
| bauzas | (dammit) | 13:47 |
| *** garyk has quit IRC | 13:48 | |
| *** dsanders has joined #openstack-nova | 13:49 | |
| *** ociuhandu has joined #openstack-nova | 13:49 | |
| jaypipes | bauzas: sure, but gimme a few to read johnthetubaguy's changes. | 13:49 |
| bauzas | jaypipes: k, we have the nova meeting right in 10 mins, so let's discuss that around 1530UTC (need to take my daughter at 1500) | 13:50 |
| bauzas | jaypipes: I just wanted to discuss how the Service object can still be useful | 13:51 |
| jaypipes | bauzas: need to be after 16:00 UTC... have another meeting | 13:51 |
| bauzas | jaypipes: ack, ping me when you're done then | 13:51 |
| jaypipes | will do. | 13:51 |
| *** dpaterson has joined #openstack-nova | 13:52 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 13:52 |
| *** pbandzi has quit IRC | 13:53 | |
| *** artom has joined #openstack-nova | 13:53 | |
| *** asselin_ has quit IRC | 13:54 | |
| *** ociuhandu has quit IRC | 13:56 | |
| *** afaranha has joined #openstack-nova | 13:56 | |
| *** afaranha has left #openstack-nova | 13:57 | |
| *** ociuhandu has joined #openstack-nova | 13:57 | |
| *** afaranha has joined #openstack-nova | 13:57 | |
| *** oro has quit IRC | 13:58 | |
| *** asselin has joined #openstack-nova | 13:59 | |
| *** thangp has joined #openstack-nova | 14:00 | |
| johnthetubaguy | abhishekk mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes gilliard garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov nova meeting times | 14:01 |
| *** achanda has joined #openstack-nova | 14:03 | |
| *** exploreshaifali has joined #openstack-nova | 14:04 | |
| *** achanda has quit IRC | 14:07 | |
| *** tjones1 has joined #openstack-nova | 14:08 | |
| dansmith | alaski: I wonder if I found a real bug? http://logs.openstack.org/99/190399/1/check/check-grenade-dsvm/38ef38b/logs/grenade.sh.txt.gz | 14:09 |
| *** dboik has joined #openstack-nova | 14:11 | |
| *** eharney has joined #openstack-nova | 14:11 | |
| *** achanda has joined #openstack-nova | 14:11 | |
| alaski | dansmith: seems like it, though it's odd since that passed on the devstack change | 14:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref https://review.openstack.org/162913 | 14:12 |
| dansmith | alaski: oh, you know, I wonder if that is because we're coming from kilo and it's not set up there, | 14:12 |
| *** Nikolay_St has quit IRC | 14:13 | |
| dansmith | and we're just trying to sync it | 14:13 |
| openstackgerrit | Brent Eagles proposed openstack/nova-specs: VIF plug script support for Nova https://review.openstack.org/162468 | 14:13 |
| alaski | dansmith: the sync should create it though | 14:14 |
| dansmith | alaski: not if the database itself isn't created right? | 14:15 |
| alaski | dansmith: but I'm not totally clear yet on what the environment is that this is getting called in | 14:15 |
| dansmith | alaski: I think it's actually got a url to a db that isn't created (and thus we have no perms for anything) and it's trying to do a sync | 14:15 |
| alaski | oh, I see | 14:15 |
| *** Nikolay_St has joined #openstack-nova | 14:15 | |
| alaski | yeah | 14:15 |
| *** browne has joined #openstack-nova | 14:16 | |
| *** jecarey has joined #openstack-nova | 14:17 | |
| *** achanda has quit IRC | 14:17 | |
| alaski | there's no recreate_database nova_api call | 14:17 |
| *** achanda has joined #openstack-nova | 14:17 | |
| *** diogogmt has joined #openstack-nova | 14:18 | |
| *** takedakn has joined #openstack-nova | 14:19 | |
| dansmith | yeah | 14:20 |
| *** exploreshaifali has quit IRC | 14:20 | |
| dansmith | after the meeting I'll see whether sdague wants that in the from-kilo grenade bit or backport the devstack change to kilo | 14:20 |
| *** achanda has quit IRC | 14:22 | |
| openstackgerrit | Brent Eagles proposed openstack/nova-specs: VIF plug script support for Nova https://review.openstack.org/162468 | 14:23 |
| openstackgerrit | Andrey Kurilin proposed openstack/nova-specs: Nova API Microversions support in NovaClient https://review.openstack.org/188410 | 14:26 |
| *** jaypipes has quit IRC | 14:26 | |
| *** cfriesen has joined #openstack-nova | 14:27 | |
| *** zz_jgrimm is now known as jgrimm | 14:28 | |
| *** electrocucaracha has joined #openstack-nova | 14:28 | |
| *** sahid has joined #openstack-nova | 14:30 | |
| *** eharney_ has joined #openstack-nova | 14:30 | |
| *** bnemec has joined #openstack-nova | 14:30 | |
| *** eharney has quit IRC | 14:32 | |
| *** Murali has quit IRC | 14:33 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova-specs: Add support for HGST Solutions volumes to libvirt https://review.openstack.org/190663 | 14:33 |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova-specs: Add support for HGST Solutions volumes to libvirt https://review.openstack.org/190663 | 14:36 |
| *** eharney_ is now known as eharney | 14:38 | |
| *** mdrabe has quit IRC | 14:38 | |
| *** garyk has joined #openstack-nova | 14:39 | |
| *** achanda has joined #openstack-nova | 14:40 | |
| *** takedakn has quit IRC | 14:41 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova-specs: Add support for HGST Solutions volumes to libvirt https://review.openstack.org/190663 | 14:42 |
| *** nkrinner has quit IRC | 14:43 | |
| *** mpaolino has quit IRC | 14:43 | |
| *** kiran-r has quit IRC | 14:44 | |
| *** tsekiyama has joined #openstack-nova | 14:45 | |
| *** achanda has quit IRC | 14:45 | |
| *** tjones1 has quit IRC | 14:47 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova-specs: Add support for HGST Solutions volumes to libvirt https://review.openstack.org/190663 | 14:48 |
| *** jcoufal has joined #openstack-nova | 14:50 | |
| *** mtanino has joined #openstack-nova | 14:52 | |
| *** subscope has quit IRC | 14:53 | |
| openstackgerrit | Andre Aranha proposed openstack/nova: Pass Down the Instance Name to Ironic Driver https://review.openstack.org/190675 | 14:55 |
| baoli | dansmith: ping. RE: port delete: https://review.openstack.org/#/c/179390/, https://review.openstack.org/#/c/188783/. There are discussions with sahid. When you have a moment, can you check it out and see what's your take on the approach to move forward? Tow things: modify the get_instance_nw_info to include a callback for obsolete ports; and handling of network-vif-deleted event. thanks | 14:55 |
| *** mpaolino has joined #openstack-nova | 14:55 | |
| *** mdrabe has joined #openstack-nova | 14:56 | |
| mriedem | cfriesen: thanks for the link, https://bugs.launchpad.net/cinder/+bug/1464259 | 14:57 |
| openstack | Launchpad bug 1464259 in Cinder "EC2VolumesTest fails with rbd backend" [Low,Triaged] | 14:57 |
| dansmith | baoli: okay | 14:58 |
| mriedem | cfriesen: i think the delete_snapshot method in the rbd volume driver in cinder just needs a retry loop like we have in nova, should be fairly easy to add | 14:58 |
| *** mmedvede has quit IRC | 14:58 | |
| *** browne has quit IRC | 14:58 | |
| baoli | dansmith: also I'd like to bother you with this review: https://review.openstack.org/#/c/136077/ whenever you have a chance. I'd like to get it done for Liberty. | 14:59 |
| baoli | dansmith: thanks! | 14:59 |
| dansmith | baoli: get danpb and ndipanov to review that spec first | 14:59 |
| *** ildikov_ has quit IRC | 15:00 | |
| *** ildikov_ has joined #openstack-nova | 15:00 | |
| *** romainh has left #openstack-nova | 15:01 | |
| *** apuimedo has quit IRC | 15:01 | |
| sahid | baoli: i will reply to you | 15:01 |
| openstackgerrit | Chuck Short proposed openstack/nova: Add "LXD" to the lxd of known hypervisors https://review.openstack.org/190678 | 15:01 |
| baoli | sahid: thanks | 15:01 |
| baoli | dansmith, will ping them. | 15:01 |
| *** dims_ has quit IRC | 15:02 | |
| *** fawadkhaliq has joined #openstack-nova | 15:02 | |
| *** yamahata has joined #openstack-nova | 15:02 | |
| *** dims has joined #openstack-nova | 15:02 | |
| *** exploreshaifali has joined #openstack-nova | 15:03 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adjust os_command_empty test to ImageMetaProps object https://review.openstack.org/190017 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: raise a meaningful exception when validating VIF properties https://review.openstack.org/190018 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy https://review.openstack.org/149625 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding libosinfo configuration https://review.openstack.org/149626 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 15:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adjust test_get_disk_bus test to ImageMetaProps object https://review.openstack.org/190682 | 15:03 |
| *** vladikr_ has joined #openstack-nova | 15:03 | |
| *** thedodd has joined #openstack-nova | 15:04 | |
| *** zsais has joined #openstack-nova | 15:04 | |
| ndipanov | baoli, I will review it but non of this will work until live migration uses claims | 15:05 |
| *** vladikr has quit IRC | 15:05 | |
| ndipanov | so you will really need to fix that first if you want this to work | 15:05 |
| alex_xu | sdague: btw, there is devref about stable api includes the explain about removing extensions in v2.1 also https://review.openstack.org/#/c/162913/ | 15:05 |
| *** fawadkhaliq has quit IRC | 15:05 | |
| baoli | ndipanov, can you point to me that work, is it in progress? | 15:05 |
| ndipanov | I think what I want to do is disable live migration for PCI/NUMA instances until we fix it alltoghether | 15:06 |
| ndipanov | baoli, no it's not :D | 15:06 |
| cfriesen | mriedem: glad to be of assistance | 15:06 |
| ndipanov | baoli, but you assume in a lot of places that in your spec (from just skimming it that devices are reserved - they are not | 15:06 |
| *** romainh has joined #openstack-nova | 15:06 | |
| ndipanov | so while your migration is running | 15:07 |
| *** subscope has joined #openstack-nova | 15:07 | |
| *** bnemec has quit IRC | 15:07 | |
| *** romainh has left #openstack-nova | 15:07 | |
| ndipanov | you could schedule another instance to the same device | 15:07 |
| ndipanov | boom | 15:07 |
| ndipanov | especially for something as long running as live mig | 15:07 |
| *** atuvenie has quit IRC | 15:07 | |
| baoli | ndipanov, are you saying that during live migration, device shouldn't be reserved on the target? | 15:07 |
| sdague | alex_xu: ok, great, I'll take a look | 15:08 |
| alex_xu | sdague: thanks | 15:08 |
| ndipanov | it should but it's not ATM | 15:10 |
| mriedem | zzzeek: want to rebase this? https://review.openstack.org/#/c/154243/ | 15:10 |
| mriedem | you need to use 295 now | 15:10 |
| zzzeek | mriedem: see, this is why im jsut not bothering :) | 15:10 |
| mriedem | zzzeek: i'll do it if you're not going to | 15:10 |
| zzzeek | mriedem: i have to run out now, i can get to it later | 15:10 |
| mriedem | i'll do it | 15:10 |
| mriedem | np | 15:10 |
| zzzeek | mriedem: can you? that woudl help | 15:10 |
| zzzeek | hx | 15:10 |
| zzzeek | thx | 15:10 |
| *** vladikr_ is now known as vladikr | 15:11 | |
| baoli | ndipanov: the spec explains how to reserve devices on the target during migration | 15:11 |
| ndipanov | baoli, but let me review that spec in detail | 15:12 |
| baoli | ndipanov: sure | 15:12 |
| *** agireud has joined #openstack-nova | 15:12 | |
| baoli | ndipanov: also there is a POC patch that used to be working, but not updated pending for spec review. | 15:13 |
| *** zzzeek has quit IRC | 15:13 | |
| baoli | ndipanov: https://review.openstack.org/#/c/141799/ | 15:13 |
| ndipanov | baoli, yeah the problem I have with this is that we are going to do this for reserving PCI devices only | 15:13 |
| ndipanov | while this problem should be solved by doing claims with live migration | 15:14 |
| *** yassine__ has quit IRC | 15:14 | |
| ndipanov | so it is likely going to be a special cased mess | 15:14 |
| ndipanov | and also it is likely to stil be racy | 15:15 |
| ndipanov | so while I am all for getting the features in | 15:15 |
| *** kmartin has joined #openstack-nova | 15:15 | |
| ndipanov | this is actually not that much of a feature that I am happy to sacrifice further code cleanups | 15:15 |
| ndipanov | plus - we know what needs to be done - why not do it | 15:15 |
| ndipanov | so while I am going to read the spec in detail | 15:16 |
| ndipanov | my answer is likely going to be - nope, we need to fix this for realz | 15:16 |
| ndipanov | baoli, ^ | 15:16 |
| ndipanov | dansmith, ^ | 15:17 |
| *** jistr is now known as jistr|mtg | 15:17 | |
| ndipanov | (will review the patch too) | 15:17 |
| *** Marga_ has joined #openstack-nova | 15:17 | |
| baoli | ndipanov: I think that I need a little bit more detail about the "doing claims with live migration" to understand what you are saying. So I must have lost the context. | 15:18 |
| *** Marga_ has quit IRC | 15:18 | |
| dansmith | ndipanov: that's always the right answer, IMHO :) | 15:18 |
| *** Marga_ has joined #openstack-nova | 15:18 | |
| edleafe | dansmith: heh | 15:19 |
| ndipanov | dansmith, I disagree | 15:20 |
| ndipanov | we need to weigh the benefits of having a feature in tree against the mess it causes | 15:20 |
| ndipanov | in this case I think the benefit is meh | 15:21 |
| mriedem | ndipanov: i think dansmith was agreeing with not introducing gorp for a feature | 15:21 |
| ndipanov | well in this case | 15:21 |
| ndipanov | he is | 15:21 |
| ndipanov | but in the general case | 15:21 |
| *** markus_z has quit IRC | 15:21 | |
| ndipanov | I dissagree with myself :D | 15:21 |
| *** sdake has quit IRC | 15:21 | |
| mriedem | heh | 15:21 |
| mriedem | simple spec for spec peeps https://review.openstack.org/#/c/187835/ | 15:22 |
| mriedem | mechanical microversion change | 15:22 |
| *** vivek_nandavanam has joined #openstack-nova | 15:26 | |
| bauzas | baoli: ndipanov: oh, given how the live migration is currently buggy, I would -1 https://review.openstack.org/#/c/136077/ | 15:26 |
| *** otter768 has joined #openstack-nova | 15:26 | |
| bauzas | baoli: like ndipanov said, there is no resource claiming for live-migrate, plus there is another problem | 15:26 |
| mriedem | flwang: were you working on getting a bp and spec up for this? https://review.openstack.org/#/c/164494/ | 15:28 |
| * bauzas tries to remember the second problem he discovered yesterday | 15:28 | |
| garyk | anyone know who is responsible for XenProject CI | 15:28 |
| bauzas | garyk: BobBall | 15:29 |
| * BobBall wonders if he should raise his hand | 15:29 | |
| BobBall | I could always hide for a bit... | 15:29 |
| baoli | bauzas, ok, I understand that's an existing issue with live migration, so scheduler finds a target but not reservering resource (such as memory, cpu, etc) on it before live migrating the vm. | 15:29 |
| bauzas | baoli: oh, yeah, I remember, we don't recheck the retries | 15:29 |
| garyk | BobBall: i guess that may be a give away :) | 15:29 |
| BobBall | Which change? :) | 15:29 |
| garyk | i did a recheck this at 11am and nudda | 15:29 |
| garyk | https://review.openstack.org/#/c/190047/ | 15:29 |
| BobBall | Ah yes | 15:29 |
| *** pbrooko has joined #openstack-nova | 15:29 | |
| BobBall | sorry about that | 15:29 |
| BobBall | host had some memory issues which killed everything | 15:30 |
| garyk | heh np. just seeing red again | 15:30 |
| BobBall | so the gerrit watcher was broken | 15:30 |
| garyk | np | 15:30 |
| BobBall | Could you recheck again and we'll just confirm it got queued? | 15:30 |
| bauzas | baoli: while I'm discussing with you, could you please help me understand why we need to have 2 distinct items for pci_requests ? one in filter_props, one as an instance field ? | 15:30 |
| garyk | keeping ones eye on the stuff is a full time job. jogo you found a replacement yet :) | 15:30 |
| bauzas | that's just... weird, since you can get the instance field using the filt_props too | 15:31 |
| *** Marga_ has quit IRC | 15:31 | |
| *** mmedvede has joined #openstack-nova | 15:31 | |
| *** coolsvap is now known as coolsvap|afk | 15:31 | |
| *** otter768 has quit IRC | 15:31 | |
| *** annashen has joined #openstack-nova | 15:32 | |
| baoli | bauzas, I vaguely remember some discussion on that when sr-iov was introduced. But it has to do with the availability of PCI requests in the scheduler. | 15:32 |
| BobBall | garyk: FYI looks like you hit https://bugs.launchpad.net/nova/+bug/1461642 | 15:32 |
| openstack | Launchpad bug 1461642 in OpenStack Compute (nova) "libvirt-xen: Race between nova and a Xen script for updating the iptables" [Medium,Confirmed] | 15:32 |
| bauzas | baoli: yup, the filter just calls filt_props | 15:33 |
| BobBall | It's times like this I wish the xenproject CI had elasticsearch enabled :) | 15:33 |
| *** salv-orlando has joined #openstack-nova | 15:34 | |
| *** salv-orl_ has quit IRC | 15:34 | |
| *** unicell has joined #openstack-nova | 15:34 | |
| *** jistr|mtg is now known as jistr | 15:35 | |
| *** unicell1 has quit IRC | 15:36 | |
| garyk | BobBall: ok, thanks | 15:36 |
| *** annashen has quit IRC | 15:36 | |
| *** vilobhmm has joined #openstack-nova | 15:37 | |
| dprince | mriedem: hi, working w/ jbernard to track down some ceph CI issues | 15:37 |
| garyk | BobBall: nova-network is so 80's | 15:37 |
| dprince | mriedem: any idea why ceph job didn't run here? https://review.openstack.org/#/c/190240/ | 15:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add an index to virtual_interfaces.uuid https://review.openstack.org/154243 | 15:38 |
| mriedem | dprince: we don't run functional test jobs on docs changes now | 15:38 |
| BobBall | What are you saying?! The 80's were fantastic! | 15:39 |
| kashyap | mriedem: Wow, some serious sleuthing going on here - https://review.openstack.org/#/c/181781/ | 15:39 |
| garyk | :) | 15:39 |
| dprince | mriedem: oh, right. Sorry. that one just surprised me | 15:39 |
| mriedem | kashyap: yes danpb has gum all over his shoes | 15:39 |
| *** hemnafk is now known as hemna | 15:40 | |
| openstackgerrit | jichenjc proposed openstack/nova: Remove policy check security_group_default_rule https://review.openstack.org/181467 | 15:40 |
| *** vilobhmm1 has joined #openstack-nova | 15:41 | |
| dprince | Need a quick Nova revert to fix the Ironic ephemeral partition. This breaks all the TripleO CI jobs... https://review.openstack.org/#/c/190622/1 | 15:41 |
| bauzas | baoli: ahem, https://github.com/openstack/nova/commit/a8a5d44c8aca218f00649232c2b8a46aee59b77e | 15:41 |
| dprince | https://bugs.launchpad.net/tripleo/+bug/1464239 | 15:41 |
| openstack | Launchpad bug 1464239 in tripleo "mount: special device /dev/sdb does not exist" [Critical,In progress] - Assigned to Dan Prince (dan-prince) | 15:41 |
| *** vilobhmm has quit IRC | 15:42 | |
| bauzas | baoli: it's you who introduce filter_properties['pci_reqs'] nope ? | 15:42 |
| *** coolsvap|afk is now known as coolsvap | 15:42 | |
| *** yamahata has quit IRC | 15:43 | |
| baoli | bauzas, yes | 15:43 |
| kashyap | mriedem: Totally. danpb - Thanks, reading that debugging session was educating! | 15:43 |
| dansmith | ndipanov: ^ | 15:44 |
| bauzas | baoli: I also find a reference in https://github.com/openstack/nova/commit/ff693722639387a7a3ea62e291a8306569392ab9 | 15:44 |
| dansmith | dprince: that fixed a critical bug in nova that causes data loss, IIRC | 15:44 |
| bauzas | baoli: which means it was first set in filter_scheduler.py and then you provided it directly in compute.api right? | 15:45 |
| *** afazekas has quit IRC | 15:45 | |
| *** browne has joined #openstack-nova | 15:45 | |
| * bauzas tries to jump on the time machine | 15:45 | |
| *** rajesht has quit IRC | 15:45 | |
| dprince | dansmith: well, its got a side effect that nukes the ephemeral partition... | 15:45 |
| baoli | bauzas, it was last year. I have to check the code again to recall. There were comments in those area while the patches were being worked on. you might want to check the review instead. | 15:46 |
| dprince | dansmith: We had 0 jobs passing since that fix landed. http://goodsquishy.com/downloads/tripleo-jobs.html (look at the undercloud-precise-nonha column for example) | 15:46 |
| mriedem | is anyone on the hook for monitoring the non-voting docker job in nova? http://logs.openstack.org/62/190362/4/check/check-tempest-dsvm-docker/2aebdfc/logs/screen-n-cpu.txt.gz?level=TRACE | 15:46 |
| mriedem | dims: ^? | 15:46 |
| bauzas | baoli: okay, my point is that I guess I can just modify the filter to use the instance field instead nope ? | 15:47 |
| dprince | dansmith: we just landed a temprevert to fix our CI though that does this revert for us so shortly our CI will be fixed again... | 15:47 |
| *** armax has joined #openstack-nova | 15:47 | |
| bauzas | baoli: because I want to use a consistent pci_requests info since I'm working on objectifying request_spec and filt_props | 15:47 |
| dansmith | dprince: cool, then the pressure is lower.. ndipanov and mikal are the two people that need to see that revert and decide what to do, IMHO | 15:47 |
| ndipanov | I loves me a revert | 15:48 |
| dprince | dansmith: reverts like this shoudl almost always land IMO | 15:48 |
| ndipanov | where's the revert? | 15:48 |
| openstackgerrit | jichenjc proposed openstack/nova: Remove cell policy check https://review.openstack.org/184052 | 15:48 |
| *** exploreshaifali has quit IRC | 15:48 | |
| dansmith | dprince: well, given the bug it was supposedly fixing, I don't agree | 15:48 |
| dprince | ndipanov: https://review.openstack.org/#/c/190622/ | 15:49 |
| openstackgerrit | Chris Gacsal proposed openstack/nova: Allow multiple volume attachments https://review.openstack.org/153033 | 15:49 |
| openstackgerrit | Chris Gacsal proposed openstack/nova: REST API changes to support multiple volume attach https://review.openstack.org/153038 | 15:49 |
| dansmith | dprince: blocking tripleo CI is the lesser evil | 15:49 |
| dprince | dansmith: yeah, well take this up w/ Ironic too I guess | 15:49 |
| ndipanov | hmm | 15:49 |
| dprince | dansmith: anyone using an Ironic ephemeral partion would get hosed by this I think | 15:49 |
| ndipanov | yeah - I'd like to know a bit more | 15:49 |
| *** atuvenie has joined #openstack-nova | 15:49 | |
| dansmith | too bad we don't actually have ironic CI | 15:50 |
| ndipanov | not that I'm against the revert | 15:50 |
| dansmith | else we might've caught this | 15:50 |
| ndipanov | but I'd imagine it's not much more difficult to fix on the ironic side | 15:50 |
| jbernard | mriedem: yeah, i think you're right about a retry loop | 15:51 |
| jbernard | mriedem: this one looks to fail on every run | 15:51 |
| ndipanov | dprince, is there any other info other than - this looks suspicious? | 15:51 |
| jbernard | mriedem: im not sure how we reached that state | 15:51 |
| jbernard | mriedem: ill try to get something useful posted | 15:51 |
| *** tjones1 has joined #openstack-nova | 15:51 | |
| mriedem | jbernard: it's not every run, it's like 8 hits in 7 days | 15:52 |
| mriedem | check and gate | 15:52 |
| dprince | ndipanov: I haven't spent time on it. But I'm 100% sure reverting just this patch fixes the issue. And it only started happening since yesterday evening... | 15:52 |
| baoli | bauzas, I see. If I remember correctly, the issue was that the instance used in the scheduler was not objectified, and the pci_requests couldn't be obtained from the instance in the scheduler, so we have to pass it in by means of filter property. | 15:52 |
| mriedem | jbernard: so quite rare actually | 15:52 |
| jbernard | mriedem: hmm, well that's good i suppose | 15:52 |
| dprince | ndipanov: revert fast, take the pressure off, and fix it right | 15:52 |
| dprince | ndipanov: everyone always seems to want to debate this however :) | 15:52 |
| ndipanov | dprince, I gues | 15:52 |
| dansmith | dprince: you have a temporary fix, right? | 15:53 |
| dansmith | so no pressure, IMHO | 15:53 |
| dansmith | especially if reverting it gets CDers back to losing data | 15:53 |
| *** mmedvede has quit IRC | 15:53 | |
| *** Mike_D_laptop has joined #openstack-nova | 15:53 | |
| bauzas | baoli: I found the SHA1 | 15:53 |
| *** tjones1 has left #openstack-nova | 15:53 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 15:53 |
| dprince | dansmith: we have a fix for CI. Delorean packages however taht use trunk for Nova are still hosed | 15:53 |
| dprince | dansmith: this is still painful... | 15:53 |
| mriedem | dprince: this probably helps, i added to the bug http://goo.gl/3P4THW | 15:54 |
| dansmith | this broke code that isn't CI tested so reverting it and breaking code that is would be the wrong move, IMHO | 15:54 |
| mriedem | dprince: shows the spike in fails better than the matrix table link above | 15:54 |
| bauzas | baoli: https://review.openstack.org/#/c/98828/ | 15:54 |
| *** atuvenie has quit IRC | 15:54 | |
| bauzas | baoli: but since the HostManager was still using the instance field, then dansmith had to write a workaround | 15:54 |
| bauzas | :) | 15:55 |
| bauzas | baoli: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L265-L273 | 15:55 |
| dprince | dansmith: it *is* CI tested by TripleO | 15:55 |
| dprince | dansmith: which you can easily run w/ 'check experimental' if you'd like BTW | 15:56 |
| bauzas | baoli: okay, I see how things happened now | 15:56 |
| *** ijw has joined #openstack-nova | 15:56 | |
| bauzas | baoli: that's all a pure mess IMHO, so I'm gonna clean that up | 15:56 |
| dansmith | dprince: so then s/CI// :) | 15:56 |
| baoli | bauzas, go for it. That's the area that had gone a lot of churns during the review. | 15:57 |
| bauzas | baoli: and consistently use the instance field which would be objectified using dansmith's workaround for *all* the scheduler filters using it | 15:57 |
| mriedem | dprince: i don't see the tripleo job in the experimental run here https://review.openstack.org/#/c/190622/1 | 15:57 |
| dprince | mriedem: thanks for the kibana link | 15:57 |
| *** patrickeast has joined #openstack-nova | 15:57 | |
| mriedem | dprince: oh nvm i see it now | 15:57 |
| bauzas | baoli: and just stop using that filt_props stuff | 15:57 |
| mriedem | check-tripleo-ironic-undercloud-precise-nonhaSUCCESS in 1h 00m 04s | 15:57 |
| dprince | mriedem: yeah, they passed on the revert... | 15:58 |
| bauzas | baoli: do you know how can I test all of this ? Is the PCI CI able to catch any problems like that ? | 15:58 |
| *** coolsvap is now known as coolsvap|afk | 15:58 | |
| bauzas | baoli: at least for the PCI filter | 15:58 |
| *** achanda has joined #openstack-nova | 15:58 | |
| baoli | bauzas, it should if the CI is properly set up to do SR-IOV | 15:58 |
| bauzas | baoli: well, it's even not only SR-IOV, it's a global PCI passthru thing | 15:59 |
| *** arnaud____ has joined #openstack-nova | 16:00 | |
| *** dulek has quit IRC | 16:03 | |
| *** Nikolay_St has quit IRC | 16:03 | |
| *** xgsa has quit IRC | 16:03 | |
| *** jlanoux has quit IRC | 16:04 | |
| openstackgerrit | jichenjc proposed openstack/nova: Cleanup HTTPRequest for security_groups test https://review.openstack.org/182630 | 16:04 |
| openstackgerrit | jichenjc proposed openstack/nova: Change group controller of V2 test cases https://review.openstack.org/182613 | 16:04 |
| openstackgerrit | jichenjc proposed openstack/nova: Add schema check for security_groups https://review.openstack.org/182614 | 16:04 |
| ndipanov | dprince, I +2ed it - but would really appreciate more info - I appreciate that the revert is being asked to get things going asap, but it sucks that you guys came up with basically 0 info | 16:04 |
| johnthetubaguy | dprince: sorry, I am missing context here to... | 16:05 |
| johnthetubaguy | dprince: how does this break trippleO but not break the ironic or libvirt based tests? | 16:05 |
| johnthetubaguy | dprince: feels like we have a massive bit of testing missing on nova here | 16:05 |
| dprince | johnthetubaguy: I think the use case is it breaks you if your tests rely on the ephemeral partition | 16:06 |
| johnthetubaguy | dprince: not sure I get you? | 16:07 |
| dansmith | the tests that run ironic against nova patches only starts a single basic guest and doesn't do any real verification of success, AFAIK | 16:07 |
| *** VW has quit IRC | 16:07 | |
| dansmith | ndipanov: this is probably just the ironic driver assuming these BDMs are volumes and not checking if they're swap/ephemeral, right? | 16:07 |
| dansmith | ndipanov: like, the real fix to ironic should be fairly minor right? | 16:07 |
| dprince | johnthetubaguy: The API's all work fine. It only fails if you have something that goes on the instance machine and tries to use the ephemeral partition (/dev/sdb...) | 16:08 |
| *** haypo has left #openstack-nova | 16:08 | |
| johnthetubaguy | dprince: so thats totally a test failures, but this is specific to the ironic driver I guess? | 16:08 |
| dprince | johnthetubaguy: yes, the way Ironic handles ephemeral is a bit different than the normal case | 16:09 |
| johnthetubaguy | dprince: so this is the ironic driver doesn't deal with ephemeral disks correctly after the mentioned change? | 16:09 |
| dprince | johnthetubaguy: haven't tracked that down yet... but I think something along those lines | 16:09 |
| *** asselin_ has joined #openstack-nova | 16:09 | |
| *** Marga_ has joined #openstack-nova | 16:09 | |
| dansmith | I dunno, I haven't seen any ironic logs that show errors | 16:09 |
| dansmith | I see oc-collect-config logs, which I assume is part of tripleo... | 16:10 |
| johnthetubaguy | dprince: problem is, this fixes a major bug most of our users want fixing, so it would be good to know why we should consider the revert, its possible the ironic fix is simpler? | 16:10 |
| *** unicell has quit IRC | 16:10 | |
| *** kiran-r has joined #openstack-nova | 16:10 | |
| ndipanov | dansmith, I just looked at it | 16:10 |
| ndipanov | and it completely disregards the bdms | 16:10 |
| bauzas | alaski: around ? just saw your comment on https://review.openstack.org/#/c/188485/3 ? | 16:10 |
| ndipanov | and only looks at ephemeral_gb | 16:11 |
| bauzas | (the second question mark is optional) | 16:11 |
| ndipanov | so I'm looking if we broke that somewhere | 16:11 |
| dansmith | ndipanov: heh | 16:11 |
| bauzas | alaski: so, I'm removing them since you can get them by using the instance object directly | 16:11 |
| dansmith | ndipanov: doesn't look like it to me | 16:12 |
| bauzas | alaski: that said, I'm thinking on providing some way for d/s people to still have not persisted fields | 16:12 |
| ndipanov | me neither | 16:12 |
| *** tonytan4ever has joined #openstack-nova | 16:12 | |
| ndipanov | wtf | 16:12 |
| bauzas | alaski: but it would mean it wouldn't pass over the wire neither would be persisted | 16:13 |
| *** yamahata has joined #openstack-nova | 16:14 | |
| ndipanov | dprince, u sure that thing fixes your stuff? | 16:14 |
| *** vilobhmm1 has quit IRC | 16:15 | |
| *** Mike_D_laptop has quit IRC | 16:15 | |
| dprince | ndipanov: positive | 16:15 |
| dprince | ndipanov: https://review.openstack.org/#/c/190629/ | 16:16 |
| ndipanov | O_o | 16:16 |
| *** achanda has quit IRC | 16:17 | |
| *** wwriverrat has quit IRC | 16:17 | |
| dansmith | ndipanov: this seems weird: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L703-705 | 16:18 |
| ndipanov | yeah saw that | 16:19 |
| *** wwriverrat has joined #openstack-nova | 16:19 | |
| *** arnaud____ has quit IRC | 16:20 | |
| *** yamahata has quit IRC | 16:20 | |
| *** gszasz has quit IRC | 16:21 | |
| *** kiran-r has quit IRC | 16:22 | |
| *** krtaylor has quit IRC | 16:22 | |
| *** alex_klimov has quit IRC | 16:23 | |
| mriedem | oh there was a nova meeting this morning huh :) | 16:26 |
| mriedem | oops | 16:26 |
| danpb | s/morning/afternoon/ | 16:27 |
| *** r-daneel has joined #openstack-nova | 16:27 | |
| *** subscope has quit IRC | 16:27 | |
| mriedem | oh there was a nova meeting 2 hours and 27 minutes ago huh :) | 16:27 |
| *** moshele has quit IRC | 16:28 | |
| *** atuvenie has joined #openstack-nova | 16:29 | |
| *** matrohon has quit IRC | 16:29 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add common function for v2.1 API flavor_get. https://review.openstack.org/136568 | 16:30 |
| openstackgerrit | jichenjc proposed openstack/nova: Add common function for v2 API flavor_get. https://review.openstack.org/136567 | 16:30 |
| johnthetubaguy | mriedem: morning! | 16:31 |
| *** sambol has joined #openstack-nova | 16:31 | |
| *** jaypipes has joined #openstack-nova | 16:35 | |
| *** achanda has joined #openstack-nova | 16:35 | |
| *** jcoufal has quit IRC | 16:36 | |
| *** unicell has joined #openstack-nova | 16:38 | |
| *** vilobhmm has joined #openstack-nova | 16:40 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 16:40 |
| *** vilobhmm1 has joined #openstack-nova | 16:41 | |
| *** vilobhmm has quit IRC | 16:44 | |
| *** pixelb has quit IRC | 16:45 | |
| samueldmq | hi, we are working in the implementation of hierarchical quotas in nova | 16:45 |
| *** annashen has joined #openstack-nova | 16:45 | |
| samueldmq | this requires nova to call keystone to know the project hierarchy | 16:46 |
| samueldmq | what would be the best approach to instantiate ksclient in nova side ? currently we are getting auth_url + token from the request context | 16:46 |
| samueldmq | ericksonsantos ^ | 16:46 |
| *** Marga_ has quit IRC | 16:47 | |
| alaski | bauzas: just back from lunch | 16:49 |
| alaski | bauzas: if those properties are available in another form, then definitely remove them | 16:49 |
| *** exploreshaifali has joined #openstack-nova | 16:50 | |
| dansmith | mriedem: wanna hit this stable review? https://review.openstack.org/#/c/187740/ | 16:50 |
| *** ndipanov has quit IRC | 16:50 | |
| samueldmq | sdague, hi, any thought on my question above ? :) | 16:50 |
| *** Marga_ has joined #openstack-nova | 16:51 | |
| *** derekh has quit IRC | 16:51 | |
| bauzas | alaski: well, sorry I was unclear, since the Instance object is not provided in the ReqSpec, there would be no way to get these fields unless you're bumping the version and adding the fields | 16:52 |
| *** salv-orl_ has joined #openstack-nova | 16:52 | |
| *** sahid has quit IRC | 16:52 | |
| *** Mike_D_laptop has joined #openstack-nova | 16:52 | |
| bauzas | alaski: I mean os_type and project_id | 16:52 |
| *** eglynn__ has quit IRC | 16:53 | |
| openstackgerrit | Miguel Lavalle proposed openstack/nova-specs: Integrate DNS resolution with Neutron using Nova hostname https://review.openstack.org/90150 | 16:53 |
| bauzas | alaski: that's why I think about downstream people by providing a way to add some non-persisted fields | 16:54 |
| *** yamahata has joined #openstack-nova | 16:54 | |
| *** salv-orlando has quit IRC | 16:54 | |
| *** VW has joined #openstack-nova | 16:55 | |
| alaski | bauzas: perhaps, but if they're important the first time they'll probably be important for later schedules so everything should be persisted | 16:55 |
| sc68cal | johnthetubaguy: hey, have a question about nova-spec repo and permissions - I can comment on https://review.openstack.org/#/c/169638/ but I can't vote? | 16:55 |
| *** dprince has quit IRC | 16:56 | |
| alaski | bauzas: I mean everything that's being used | 16:56 |
| sc68cal | I think my comment is important enough to warrant a -1 - it's a big issue | 16:56 |
| *** achanda has quit IRC | 16:56 | |
| sc68cal | hmmmmmmm | 16:56 |
| sc68cal | now I can vote? whaaaa? | 16:56 |
| mriedem | dansmith: +2 | 16:56 |
| dansmith | mriedem: thanks | 16:57 |
| johnthetubaguy | sc68cal: funky, you should be able to vote, it does go away when the session times out / you get logged out | 16:57 |
| sc68cal | johnthetubaguy: ah, I know why - I was using the link from eavesdrop - I commented on an old patchset | 16:57 |
| johnthetubaguy | sc68cal: oooh, yeah, good point | 16:57 |
| johnthetubaguy | sc68cal: I keep doing that when I update commit messages | 16:58 |
| johnthetubaguy | sc68cal: good comment by the way | 16:59 |
| *** mriedem is now known as mriedem_afk | 17:00 | |
| sc68cal | johnthetubaguy: haha - sorry for the panic there for a second, but yeah I'm only running around like a nut because it was only a couple weeks ago we talked about this issue on the neutron side | 17:00 |
| johnthetubaguy | sc68cal: ah, no worries | 17:00 |
| *** veena has joined #openstack-nova | 17:00 | |
| sc68cal | johnthetubaguy: I'd have to hunt down the review where someone pointed out the issue, but I think we decided that even if they only specificy a single subnet ID on a network that is dual stacked- they're going to get an Ipv6 address no matter what, we're not going to honor that kind of request and start filtering icmpv6 packets to accomplish it | 17:02 |
| sc68cal | just so they get only the ipv4 subnet they specified | 17:02 |
| johnthetubaguy | sc68cal: I think that makes some sense, although it might not be what the user wanted, but yeah, its an issue | 17:03 |
| *** annegentle has joined #openstack-nova | 17:03 | |
| johnthetubaguy | sc68cal: my bigger issue is we want to deprecate those APIs and possibly remove them in a microversion, but probably not till next cycle | 17:03 |
| *** tonytan4ever has quit IRC | 17:05 | |
| sc68cal | johnthetubaguy: understood - I'll be sharing this spec with the neutron team so we can take a look and provide input - credit to anteaya for alerting me to it | 17:06 |
| *** bnemec has joined #openstack-nova | 17:06 | |
| johnthetubaguy | sc68cal: we might want a way to get those neutron specs too you, more generally CPL review worthy specs | 17:06 |
| *** lucasagomes has quit IRC | 17:07 | |
| *** Mike_D_laptop1 has joined #openstack-nova | 17:07 | |
| *** danpb has quit IRC | 17:07 | |
| johnthetubaguy | sc68cal: if you some up with a good way, maybe some git commit tags, they we could try it | 17:07 |
| sc68cal | johnthetubaguy: I think I've got nova-specs subscribed in gertty, if not I'll fix | 17:07 |
| johnthetubaguy | cool | 17:07 |
| *** nic has joined #openstack-nova | 17:07 | |
| * johnthetubaguy has to run to band rehearsal now | 17:08 | |
| *** Mike_D_laptop has quit IRC | 17:09 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova-specs: Servicegroup foundational refactoring for Control Plane https://review.openstack.org/190322 | 17:10 |
| *** mpaolino has quit IRC | 17:11 | |
| *** pbrooko has quit IRC | 17:12 | |
| vilobhmm1 | johnthetubaguy : hi..you have 2 min or so.. | 17:14 |
| *** Mike_D_laptop1 has quit IRC | 17:16 | |
| mnestratov | johnthetubaguy: could you please remove -2 in https://review.openstack.org/#/c/186858/ as soon as its bp got approved? | 17:18 |
| openstackgerrit | Marian Horban proposed openstack/nova: Helper script for running under Apache2 https://review.openstack.org/190727 | 17:18 |
| *** mingy has joined #openstack-nova | 17:19 | |
| openstackgerrit | Alexis Lee proposed openstack/nova-specs: No more soft delete https://review.openstack.org/184645 | 17:21 |
| *** iamjarvo has quit IRC | 17:24 | |
| *** ssurana has joined #openstack-nova | 17:26 | |
| *** mpaolino has joined #openstack-nova | 17:26 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Libvirt: Fix SMBFS volume attach race condition https://review.openstack.org/187619 | 17:27 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Use utf8_bin collation on the flavor extra-specs table in MySQL https://review.openstack.org/190300 | 17:27 |
| *** otter768 has joined #openstack-nova | 17:27 | |
| *** tonytan4ever has joined #openstack-nova | 17:28 | |
| *** lpetrut has quit IRC | 17:30 | |
| openstackgerrit | Qiu Yu proposed openstack/nova: libvirt: Force iSCSI disconnect after timeout https://review.openstack.org/167815 | 17:32 |
| *** otter768 has quit IRC | 17:32 | |
| *** zzzeek has joined #openstack-nova | 17:33 | |
| *** jistr has quit IRC | 17:35 | |
| *** dprince has joined #openstack-nova | 17:41 | |
| *** jlanoux has joined #openstack-nova | 17:43 | |
| *** radek_ has joined #openstack-nova | 17:44 | |
| *** sdake has joined #openstack-nova | 17:46 | |
| *** jang has quit IRC | 17:49 | |
| *** cbader has joined #openstack-nova | 17:51 | |
| *** iamjarvo has joined #openstack-nova | 17:51 | |
| *** iamjarvo has quit IRC | 17:51 | |
| *** iamjarvo has joined #openstack-nova | 17:52 | |
| *** iamjarvo has quit IRC | 17:52 | |
| *** sdake_ has joined #openstack-nova | 17:54 | |
| *** agireud has quit IRC | 17:56 | |
| *** mmedvede has joined #openstack-nova | 17:57 | |
| *** achanda has joined #openstack-nova | 17:57 | |
| *** sdake has quit IRC | 17:58 | |
| *** Marga_ has quit IRC | 17:59 | |
| *** Marga_ has joined #openstack-nova | 17:59 | |
| *** VW has quit IRC | 17:59 | |
| *** lpetrut has joined #openstack-nova | 18:00 | |
| *** destiny has quit IRC | 18:00 | |
| *** hemna is now known as hemnafk | 18:01 | |
| *** VW has joined #openstack-nova | 18:02 | |
| *** marun has quit IRC | 18:03 | |
| *** ociuhandu has quit IRC | 18:06 | |
| *** jlanoux has quit IRC | 18:12 | |
| *** agireud has joined #openstack-nova | 18:13 | |
| *** annegentle has quit IRC | 18:14 | |
| *** jdurgin1 has joined #openstack-nova | 18:15 | |
| *** jdurgin has quit IRC | 18:17 | |
| *** annegentle has joined #openstack-nova | 18:17 | |
| *** veena has quit IRC | 18:18 | |
| *** krtaylor has joined #openstack-nova | 18:18 | |
| *** Marga_ has quit IRC | 18:19 | |
| *** Marga_ has joined #openstack-nova | 18:19 | |
| *** exploreshaifali has quit IRC | 18:20 | |
| *** salv-orl_ has quit IRC | 18:20 | |
| *** salv-orlando has joined #openstack-nova | 18:21 | |
| *** pbrooko has joined #openstack-nova | 18:24 | |
| *** salv-orlando has quit IRC | 18:26 | |
| *** SourabhP has joined #openstack-nova | 18:26 | |
| *** eliqiao has quit IRC | 18:30 | |
| *** achanda has quit IRC | 18:31 | |
| *** eliqiao has joined #openstack-nova | 18:31 | |
| *** Marga_ has quit IRC | 18:32 | |
| *** baoli has quit IRC | 18:32 | |
| *** Marga_ has joined #openstack-nova | 18:32 | |
| *** operator99 has quit IRC | 18:32 | |
| *** iamjarvo has joined #openstack-nova | 18:32 | |
| *** iamjarvo has quit IRC | 18:33 | |
| *** iamjarvo has joined #openstack-nova | 18:33 | |
| *** dprince has quit IRC | 18:34 | |
| *** iamjarvo has joined #openstack-nova | 18:35 | |
| *** iamjarvo has quit IRC | 18:35 | |
| openstackgerrit | Karim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0 https://review.openstack.org/190267 | 18:36 |
| *** achanda has joined #openstack-nova | 18:37 | |
| *** ssurana1 has joined #openstack-nova | 18:38 | |
| *** gyee has joined #openstack-nova | 18:41 | |
| *** ssurana has quit IRC | 18:41 | |
| jroll | is there an etherpad going for the midcycle yet? the one on https://wiki.openstack.org/wiki/Sprints/NovaLibertySprint is empty | 18:41 |
| *** hemnafk is now known as hemna | 18:43 | |
| *** pbrooko_ has joined #openstack-nova | 18:44 | |
| *** annegentle has quit IRC | 18:45 | |
| *** ijw has quit IRC | 18:45 | |
| *** pbrooko has quit IRC | 18:46 | |
| *** jdurgin has joined #openstack-nova | 18:46 | |
| *** jdurgin1 has quit IRC | 18:48 | |
| *** aix has quit IRC | 18:49 | |
| *** achanda has quit IRC | 18:50 | |
| *** achanda has joined #openstack-nova | 18:52 | |
| *** jaypipes has quit IRC | 18:52 | |
| *** mnestratov has quit IRC | 18:53 | |
| *** pbrooko_ has quit IRC | 18:56 | |
| *** claudiub has quit IRC | 18:59 | |
| *** xgsa has joined #openstack-nova | 19:00 | |
| *** achanda has quit IRC | 19:00 | |
| *** marun has joined #openstack-nova | 19:00 | |
| *** winston-d has quit IRC | 19:00 | |
| *** VW_ has joined #openstack-nova | 19:02 | |
| *** achanda has joined #openstack-nova | 19:02 | |
| *** eliqiao1 has joined #openstack-nova | 19:04 | |
| *** VW has quit IRC | 19:04 | |
| *** ijw has joined #openstack-nova | 19:05 | |
| *** exploreshaifali has joined #openstack-nova | 19:05 | |
| *** greghaynes has quit IRC | 19:05 | |
| *** eliqiao has quit IRC | 19:05 | |
| *** greghaynes has joined #openstack-nova | 19:05 | |
| *** marun has quit IRC | 19:05 | |
| *** annashen has quit IRC | 19:08 | |
| *** mriedem_afk is now known as mriedem | 19:08 | |
| *** VW_ has quit IRC | 19:10 | |
| *** VW has joined #openstack-nova | 19:10 | |
| *** pbrooko has joined #openstack-nova | 19:12 | |
| *** mpaolino has quit IRC | 19:14 | |
| *** annashen has joined #openstack-nova | 19:14 | |
| tristanC | Hi folks, can someone familiar with trusted compute pool please have a look at bug 1456228 ? | 19:15 |
| openstack | bug 1456228 in OpenStack Security Advisory "Trusted vm can be powered on untrusted host" [Undecided,Incomplete] https://launchpad.net/bugs/1456228 | 19:15 |
| *** annegentle has joined #openstack-nova | 19:16 | |
| *** ildikov_ has quit IRC | 19:17 | |
| *** pbrooko has quit IRC | 19:19 | |
| *** alex_klimov has joined #openstack-nova | 19:20 | |
| *** marcusvrn has quit IRC | 19:24 | |
| *** VW has quit IRC | 19:25 | |
| *** pbrooko has joined #openstack-nova | 19:25 | |
| *** pbrooko has quit IRC | 19:25 | |
| *** salv-orlando has joined #openstack-nova | 19:26 | |
| *** nova_newbie has joined #openstack-nova | 19:26 | |
| *** ildikov_ has joined #openstack-nova | 19:27 | |
| *** nova_newbie has quit IRC | 19:27 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove/deprecate unused parts of the compute node object https://review.openstack.org/186757 | 19:27 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema https://review.openstack.org/169284 | 19:27 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Handle InstanceNotFound when sending instance update notification https://review.openstack.org/183727 | 19:28 |
| *** otter768 has joined #openstack-nova | 19:28 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker https://review.openstack.org/169288 | 19:29 |
| *** VW has joined #openstack-nova | 19:31 | |
| *** annashen has quit IRC | 19:31 | |
| *** otter768 has quit IRC | 19:33 | |
| *** Marga_ has quit IRC | 19:36 | |
| *** Marga_ has joined #openstack-nova | 19:37 | |
| *** VW has quit IRC | 19:39 | |
| *** mwagner_afk has quit IRC | 19:39 | |
| *** baoli has joined #openstack-nova | 19:41 | |
| *** achanda has quit IRC | 19:41 | |
| *** VW has joined #openstack-nova | 19:42 | |
| *** hemna is now known as hemnafood | 19:44 | |
| *** ZZelle_ has joined #openstack-nova | 19:44 | |
| bauzas | is there a netsplit or is the channel really quiet ? :) | 19:45 |
| *** rotbeard has quit IRC | 19:45 | |
| bauzas | alaski: around ? sorry, was in a family time :) | 19:46 |
| alaski | bauzas: yep | 19:46 |
| bauzas | alaski: just had no time to explain my thoughts | 19:47 |
| bauzas | alaski: so https://review.openstack.org/#/c/188485/3/nova/scheduler/filter_scheduler.py,cm is about to remove os_type and project_id | 19:48 |
| bauzas | alaski: because they are both not in use, and can also be checked by calling the instance_props | 19:48 |
| bauzas | alaski: since you said you're using them d/s, my take is to say we can't add them in the ReqSpec object unless they are in use in-tree | 19:49 |
| *** SourabhP has quit IRC | 19:49 | |
| bauzas | alaski: I mean, adding them as fields | 19:49 |
| alaski | I agree with that | 19:49 |
| bauzas | alaski: given that, I'm fine with providing some getter/setter methods for adding non-fields to the ReqSpec object | 19:50 |
| alaski | the burden is on us to put something in-tree to justify their existence imo | 19:50 |
| bauzas | alaski: a middleground would consist in making sure those could be provided in the object, but not as fields | 19:51 |
| bauzas | alaski: so not persisted neither sent over RPC | 19:51 |
| *** haypo has joined #openstack-nova | 19:52 | |
| bauzas | alaski: if you see https://review.openstack.org/#/c/188485/3/nova/scheduler/filter_scheduler.py,cm I will have to do that for config_options | 19:52 |
| alaski | bauzas: that's somewhat useful as it can help initial scheduling, but means we lose them for a reschedule. I would rather move away from that and just not add things that way | 19:52 |
| vilobhmm1 | bauzas : in your reply to https://review.openstack.org/#/c/190322/5/specs/liberty/approved/servicegroup-api-control-plane.rst you mentioned about asking scheduler team but i dont see anything for scheduler team meetings here https://wiki.openstack.org/wiki/Nova#Active_Sub-teams I know you work for scheduler so thought will check with you… | 19:52 |
| vilobhmm1 | thanks for the review on both the specs | 19:52 |
| bauzas | alaski: that's just a trade-off :) | 19:53 |
| *** sdake_ has quit IRC | 19:53 | |
| bauzas | alaski: again, see the ugly config_options | 19:53 |
| bauzas | alaski: by doing that, we allow out-of-tree filters to know some extra config stuff | 19:53 |
| *** ildikov_ has quit IRC | 19:54 | |
| alaski | bauzas: where are the config options? | 19:54 |
| bauzas | alaski: prepare to be happy | 19:54 |
| *** thangp has quit IRC | 19:55 | |
| bauzas | alaski: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L100 | 19:55 |
| bauzas | alaski: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L126 | 19:55 |
| *** sdake has joined #openstack-nova | 19:56 | |
| bauzas | vilobhmm1: yeah, that's a new stuff for Liberty, we're trying to have subteams directly accounting for their own changes | 19:56 |
| alaski | oh, right. more fun things | 19:57 |
| bauzas | alaski: of course, config_options and context are not in use in-tree, but should I provide the ReqSpec, it would blow out out-of-tree filters | 19:57 |
| bauzas | alaski: so the thing is, let's have some way to set/get some non-fields for the ReqSpec so it would still help them but would also say that's not a contract | 19:58 |
| alaski | bauzas: I think that's fine. especially for the config, since the filters could just load those themselves | 19:59 |
| *** SourabhP has joined #openstack-nova | 19:59 | |
| bauzas | alaski: yup, all of that is on the sched side | 19:59 |
| alaski | I meant I think it's fine to take away things not being used in tree, despite being a victim of that | 20:00 |
| bauzas | alaski: I see | 20:00 |
| alaski | but let's explore the ability to send fields on the first schedule and not persist them | 20:00 |
| vilobhmm1 | bauzas : your -1 is for checking with Scheduler subteam right ? because for every other comment I see you agree with the proposal…sorry if I am missing anything ? | 20:00 |
| bauzas | vilobhmm1: my -1 is for saying that you need to explain more that the SG API should be a abstract layer (or a proxy if you prefer) for both the datastore and the services table | 20:01 |
| alaski | bauzas: would you add the config options, os type, and project id for now? | 20:01 |
| bauzas | vilobhmm1: eg. what are you planning to do about the disabled field ? | 20:02 |
| alaski | bauzas: actually, I'm thinking more and changing my mind | 20:02 |
| bauzas | alaski: config_options is doable, os_type wouldn't, since it's related to the instance, but we're removing the instance from the object | 20:02 |
| bauzas | alaski: okay, fire it | 20:03 |
| alaski | bauzas: I think we should send fields that could reasonably be used for scheduling purposes, like os_type or project_id, whether or not they're used in-tree | 20:03 |
| bauzas | vilobhmm1: I made a long email explaining my thoughts, feel free to read it | 20:03 |
| mriedem | sdague: note to self (and you): when the ec2 api goes out of tree, we should also remove this: https://review.openstack.org/#/c/38081/ :) | 20:04 |
| bauzas | alaski: erm | 20:04 |
| mriedem | b/c w/o the ec2 context, that's very confusing in the compute manager | 20:04 |
| alaski | bauzas: if we're going to send the data, let's do it properly | 20:04 |
| mriedem | oh damn nvm we can't | 20:04 |
| mriedem | b/c ec2 out of tree != ec2 api dead | 20:04 |
| mriedem | :( | 20:04 |
| *** Nikolay_St has joined #openstack-nova | 20:04 | |
| vilobhmm1 | bauzas : sure I can do that…if it is not clear from the spec…my plan is admin-interface and extensions to go through services table and but move the service liveliness handled by the resp data store….I will explicitly mention it..I went through the email thread :) | 20:05 |
| bauzas | alaski: my 'erm' is by saying that I don't see why we should add some fields if they are not in use ? | 20:05 |
| vilobhmm1 | bauzas : basically the sg abstraction layer should take care of it….will update the spec with details.. | 20:06 |
| bauzas | alaski: I mean, that's the whole use of the objects, right? if someone wants a new field, he just has to bump a new ver | 20:06 |
| alaski | bauzas: right. but the scheduler is a component that allows for extensibility | 20:07 |
| alaski | this is one area where it's reasonable to have some very specific filters/weights that may not go upstream | 20:07 |
| bauzas | alaski: I just want to make sure the reqspec object wouldn't be a vector for any out-of-tree need | 20:08 |
| bauzas | alaski: I mean, the contract is to say that you're a full resident if you're using that data, or you get a green card only (non-field) if you don't use it | 20:08 |
| alaski | I agree. I think there's still a boundary of what's reasonable for a scheduler to use. like instance name or uuid, those should have no affect on scheduling | 20:09 |
| bauzas | alaski: that said, I truly admit that os_type and project_id are reasonable | 20:09 |
| alaski | what's the benefit of allowing non-fields? they'll be a vector for out of tree needs, without all the nice things a field provides | 20:10 |
| bauzas | alaski: but for example, config_options is not | 20:10 |
| bauzas | alaski: that's bound by the layer isolation | 20:10 |
| bauzas | alaski: ie. there is no way to use them out of the scheduler, and they won't be persisted | 20:11 |
| alaski | agreed | 20:11 |
| bauzas | alaski: that's just a convenient handler for passing out some data to filters | 20:11 |
| bauzas | alaski: ideally, we should even remove from in-tree the config_options and context and ask the downstream vendors to set those keys elsewhere | 20:12 |
| alaski | probably, though I have no clue what's actually in the config options | 20:13 |
| alaski | the context is because of db calls I'm guessing | 20:13 |
| bauzas | alaski: eg. in a new OutOfTreeFilterScheduler which would inherit from FilterScheduler and would just put those keys in filter_props | 20:13 |
| bauzas | alaski: since both context and config_options are not in use in-tree, we can just speculate about | 20:15 |
| alaski | bauzas: oh yeah. I would remove those, and if a case can be made for pulling some data from those into the requestspec we should look at that | 20:17 |
| *** haypo has left #openstack-nova | 20:17 | |
| *** annashen has joined #openstack-nova | 20:21 | |
| *** matrohon has joined #openstack-nova | 20:24 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls https://review.openstack.org/70485 | 20:24 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: tests: functional tests for keypairs create in API microversion 2.5 https://review.openstack.org/183645 | 20:24 |
| *** Sukhdev has joined #openstack-nova | 20:25 | |
| openstackgerrit | Andre Aranha proposed openstack/nova: Pass Down the Instance Name to Ironic Driver https://review.openstack.org/190675 | 20:25 |
| *** ildikov_ has joined #openstack-nova | 20:30 | |
| *** achanda has joined #openstack-nova | 20:31 | |
| *** alexpilotti has quit IRC | 20:31 | |
| *** rotbart has joined #openstack-nova | 20:34 | |
| *** ijw has quit IRC | 20:35 | |
| *** ijw has joined #openstack-nova | 20:36 | |
| *** vladikr has quit IRC | 20:37 | |
| *** electroc_ has joined #openstack-nova | 20:39 | |
| *** mingy has quit IRC | 20:40 | |
| *** electrocucaracha has quit IRC | 20:42 | |
| *** achanda has quit IRC | 20:43 | |
| *** ildikov has quit IRC | 20:43 | |
| *** dboik has quit IRC | 20:45 | |
| *** annegentle has quit IRC | 20:45 | |
| flwang | mriedem: i have created a bp and will submit a spec today | 20:46 |
| flwang | mriedem: https://blueprints.launchpad.net/nova/+spec/backup-volume-backed-instance | 20:47 |
| *** isd has joined #openstack-nova | 20:47 | |
| *** baoli_ has joined #openstack-nova | 20:48 | |
| *** ivasev has quit IRC | 20:48 | |
| *** mingy has joined #openstack-nova | 20:51 | |
| *** rotbart has quit IRC | 20:51 | |
| *** baoli has quit IRC | 20:51 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Libvirt: vNIC<->network identification https://review.openstack.org/190814 | 20:56 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/nova: Create column allocated in Quota table https://review.openstack.org/151327 | 20:57 |
| *** Marga_ has quit IRC | 20:58 | |
| *** jdurgin1 has joined #openstack-nova | 20:59 | |
| *** Marga_ has joined #openstack-nova | 20:59 | |
| *** rfolco has quit IRC | 20:59 | |
| *** aysyanne has quit IRC | 20:59 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Skip initial sync of block_device_mapping https://review.openstack.org/173502 | 21:00 |
| mingy | edleafe: ping | 21:00 |
| *** jdurgin has quit IRC | 21:00 | |
| *** patrickeast_ has joined #openstack-nova | 21:01 | |
| *** baoli_ has quit IRC | 21:02 | |
| *** patrickeast has quit IRC | 21:02 | |
| *** patrickeast_ is now known as patrickeast | 21:02 | |
| *** raildo has quit IRC | 21:04 | |
| openstackgerrit | Ed Leafe proposed openstack/nova-specs: Remove v3 from nova code tree https://review.openstack.org/189218 | 21:05 |
| edleafe | mingy: pong | 21:05 |
| *** artom has quit IRC | 21:05 | |
| mingy | this is regarding your earlier comment about six.iteritems()... it seems that opinions on the mailing list are divided on its use. what is our take on this? | 21:06 |
| -openstackstatus- NOTICE: Gerrit has been restarted to terminate a persistent looping third-party CI bot | 21:07 | |
| mingy | edleafe: i.e. it seems like the discussion is still premature at this point | 21:08 |
| *** thedodd has quit IRC | 21:08 | |
| edleafe | mingy: the potential gain is tiny unless you are talking about millions of items | 21:09 |
| *** lpetrut has quit IRC | 21:09 | |
| mingy | edleafe: understandably. so knowing this is it desirable that someone rewrites the entire codebase to change six.iteritems(d) into d.items() or vice versa? | 21:10 |
| edleafe | mingy: I think that there are more important issues to tackle first :) | 21:11 |
| edleafe | mingy: but new/changed code should not add to the problem. | 21:11 |
| *** Nikolay_St has quit IRC | 21:11 | |
| mriedem | mingy: it falls into a bigger effort to make things work on py34 | 21:12 |
| mriedem | which is using six for most of it | 21:12 |
| mriedem | this probably got caught up in that | 21:12 |
| *** annegentle has joined #openstack-nova | 21:13 | |
| mingy | mriedem: makes, sense, considering we're talking about six. I was wondering also about functions like itervalues() vs values(), xrange() vs range() etc, that are in also a similar situation but not explicitly mentioned in the thread. | 21:14 |
| edleafe | mingy: if you use six.range(), that translates to xrange in Python 2.x | 21:15 |
| mingy | because if we're going to use items() instead then we should also use values() and range(), and then the module six would not be very useful after that | 21:15 |
| edleafe | mingy: items(), keys(), and values() all changed in 3.x | 21:16 |
| edleafe | six addresses the differences | 21:16 |
| mingy | edleafe: yup I understand, but six addresses their differences in the same way that it does to iteritems() vs items() | 21:17 |
| edleafe | mingy: agreed, but the idea is not to prematurely optimize code. Using items() should be perfectly fine, but if it isn't, you should justify why | 21:18 |
| *** ijw has quit IRC | 21:19 | |
| mriedem | did dhellmann comment on the thread? he wrote the book, literally. | 21:19 |
| *** ijw has joined #openstack-nova | 21:19 | |
| edleafe | mriedem: heh | 21:20 |
| edleafe | mriedem: no, I haven't seen him chime in | 21:20 |
| *** dpaterson has quit IRC | 21:20 | |
| mingy | mriedem: nope, just checked | 21:21 |
| mingy | edleafe: OK, fair enough. | 21:21 |
| *** hemnafood is now known as hemna | 21:21 | |
| *** radek_ has quit IRC | 21:24 | |
| *** otter768 has joined #openstack-nova | 21:29 | |
| *** apuimedo has joined #openstack-nova | 21:29 | |
| *** dpaterson has joined #openstack-nova | 21:32 | |
| *** ociuhandu has joined #openstack-nova | 21:32 | |
| beagles | mriedem, crap I accidentally removed the change id when flipping through the commit message | 21:33 |
| beagles | mriedem, re backport of that ebtables thing | 21:33 |
| mriedem | beagles: i dropped yours after i pushed up mine | 21:33 |
| beagles | k | 21:33 |
| *** otter768 has quit IRC | 21:34 | |
| *** tonytan4ever has quit IRC | 21:35 | |
| beagles | vim is sometimes too vim'y | 21:35 |
| *** eharney has quit IRC | 21:35 | |
| *** ijw_ has joined #openstack-nova | 21:36 | |
| *** annegentle has quit IRC | 21:38 | |
| *** ijw has quit IRC | 21:39 | |
| *** ian_brown has joined #openstack-nova | 21:39 | |
| *** isd has quit IRC | 21:39 | |
| *** dims_ has joined #openstack-nova | 21:44 | |
| *** jwcroppe has joined #openstack-nova | 21:45 | |
| *** dims has quit IRC | 21:46 | |
| *** baoli has joined #openstack-nova | 21:46 | |
| *** jdurgin1 has quit IRC | 21:47 | |
| *** jdurgin has joined #openstack-nova | 21:49 | |
| *** exploreshaifali has quit IRC | 21:50 | |
| *** ijw_ has quit IRC | 21:52 | |
| *** annashen has quit IRC | 21:53 | |
| *** diogogmt has quit IRC | 21:59 | |
| *** mriedem is now known as mriedem_away | 21:59 | |
| *** annegentle has joined #openstack-nova | 22:00 | |
| *** VW has quit IRC | 22:00 | |
| *** baoli has quit IRC | 22:03 | |
| *** matrohon has quit IRC | 22:03 | |
| *** baoli has joined #openstack-nova | 22:03 | |
| *** ijw has joined #openstack-nova | 22:04 | |
| *** apuimedo has quit IRC | 22:04 | |
| *** baoli has quit IRC | 22:09 | |
| *** jecarey has quit IRC | 22:10 | |
| *** electroc_ has quit IRC | 22:10 | |
| *** electrocucaracha has joined #openstack-nova | 22:10 | |
| *** electrocucaracha has quit IRC | 22:13 | |
| tonyb | Oh so it's spec review day there goes that plan :/ | 22:14 |
| *** electrocucaracha has joined #openstack-nova | 22:14 | |
| *** mdrabe has quit IRC | 22:16 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/nova: libvirt: add vzstorage volumes suppport https://review.openstack.org/190843 | 22:22 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: WIP switch to using os-brick https://review.openstack.org/175569 | 22:23 |
| *** annegentle has quit IRC | 22:23 | |
| *** nic has quit IRC | 22:23 | |
| *** Sukhdev has quit IRC | 22:25 | |
| *** agireud has quit IRC | 22:25 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/nova: libvirt: add vzstorage volumes suppport https://review.openstack.org/190843 | 22:28 |
| *** jwcroppe has quit IRC | 22:31 | |
| *** VW has joined #openstack-nova | 22:31 | |
| *** jwcroppe has joined #openstack-nova | 22:31 | |
| *** bnemec has quit IRC | 22:34 | |
| *** salv-orlando has quit IRC | 22:34 | |
| cfriesen | curious what you folks think...would it make sense to update "instance.launched_at" after a stop/start? | 22:42 |
| *** marun has joined #openstack-nova | 22:45 | |
| *** slong has joined #openstack-nova | 22:48 | |
| *** VW has quit IRC | 22:49 | |
| *** jgrimm is now known as zz_jgrimm | 22:51 | |
| *** diogogmt has joined #openstack-nova | 22:52 | |
| *** annashen has joined #openstack-nova | 22:54 | |
| *** annashen has quit IRC | 22:55 | |
| openstackgerrit | Brent Eagles proposed openstack/nova-specs: Amend Wrap Neutron client calls spec https://review.openstack.org/190855 | 23:02 |
| *** krtaylor has quit IRC | 23:04 | |
| *** ijw has quit IRC | 23:05 | |
| *** zzzeek has quit IRC | 23:05 | |
| *** ZZelle_ has quit IRC | 23:06 | |
| *** annashen has joined #openstack-nova | 23:11 | |
| *** markvoelker has quit IRC | 23:11 | |
| openstackgerrit | Brent Eagles proposed openstack/nova-specs: Amend Wrap Neutron client calls spec https://review.openstack.org/190855 | 23:14 |
| *** zhenguo has joined #openstack-nova | 23:15 | |
| *** zzzeek has joined #openstack-nova | 23:15 | |
| *** Sukhdev has joined #openstack-nova | 23:19 | |
| *** marun has quit IRC | 23:21 | |
| *** annashen has quit IRC | 23:21 | |
| *** annegentle has joined #openstack-nova | 23:24 | |
| *** annegentle has quit IRC | 23:29 | |
| openstackgerrit | Brent Eagles proposed openstack/nova-specs: Refactor of the Neutron network adapter https://review.openstack.org/131413 | 23:29 |
| *** otter768 has joined #openstack-nova | 23:30 | |
| *** tjones1 has joined #openstack-nova | 23:32 | |
| *** tjones1 has quit IRC | 23:32 | |
| *** tjones1 has joined #openstack-nova | 23:34 | |
| *** otter768 has quit IRC | 23:34 | |
| mingy | what's the status of bug 1370207 | 23:37 |
| openstack | bug 1370207 in OpenStack Compute (nova) "race condition between nova scheduler and nova compute" [Low,Confirmed] https://launchpad.net/bugs/1370207 | 23:37 |
| *** annashen has joined #openstack-nova | 23:38 | |
| *** SourabhP has quit IRC | 23:42 | |
| *** SourabhP has joined #openstack-nova | 23:45 | |
| *** hemna is now known as hemnafk | 23:45 | |
| * mikal reviews specs | 23:45 | |
| mikal | (It being Friday here) | 23:46 |
| anish | being friday there, shouldn't you be drinking some beer instead ? | 23:47 |
| mikal | Well, its 9:48am | 23:48 |
| mikal | So a bit early for booze. I need the sun to be over the yard arm first. | 23:48 |
| anish | 10am ? are you off the coast of japan somewhere ? | 23:49 |
| anish | or possibly australia | 23:49 |
| *** salv-orlando has joined #openstack-nova | 23:49 | |
| *** tjones1 has quit IRC | 23:49 | |
| *** tjones1 has joined #openstack-nova | 23:50 | |
| mikal | Australia | 23:51 |
| mikal | Morning is when the wild life is most likely to attack, so I have to stay sober for that bit | 23:51 |
| mikal | Its a survival reflex | 23:51 |
| anish | makes sense | 23:52 |
| anish | at least you don't have to guard the rabbit fence | 23:52 |
| *** zzzeek has quit IRC | 23:52 | |
| *** salv-orlando has quit IRC | 23:53 | |
| mikal | True, we gave up on that | 23:54 |
| mikal | (There's actually a lot of rabbit fencing around where I live. By law those had to be put up when rabbits made it to the area, which was about 100 years ago where I am) | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!