| *** pratikmallya has quit IRC | 00:01 | |
| *** dimtruck is now known as zz_dimtruck | 00:04 | |
| gangil | nvm I see its deprecated, will remove n-obj from local.conf and restack | 00:05 |
|---|---|---|
| *** zz_dimtruck is now known as dimtruck | 00:05 | |
| *** zhangyufei has joined #openstack-nova | 00:05 | |
| *** zhangyufei has quit IRC | 00:05 | |
| *** pradk has quit IRC | 00:06 | |
| *** markmcclain has quit IRC | 00:09 | |
| *** mragupat has quit IRC | 00:09 | |
| *** markmcclain has joined #openstack-nova | 00:10 | |
| *** zhangjn has joined #openstack-nova | 00:10 | |
| *** pratikmallya has joined #openstack-nova | 00:11 | |
| *** mc_nair has joined #openstack-nova | 00:11 | |
| *** zhangjn has quit IRC | 00:12 | |
| *** smatzek has joined #openstack-nova | 00:14 | |
| *** pratikmallya has quit IRC | 00:14 | |
| *** gangil1 has joined #openstack-nova | 00:17 | |
| *** gangil has quit IRC | 00:17 | |
| *** aginwala has quit IRC | 00:18 | |
| *** markvoelker has quit IRC | 00:20 | |
| *** otter768 has quit IRC | 00:20 | |
| *** irenab_ has joined #openstack-nova | 00:20 | |
| *** ndipanov has joined #openstack-nova | 00:22 | |
| *** irenab has quit IRC | 00:23 | |
| *** irenab_ is now known as irenab | 00:23 | |
| *** aginwala has joined #openstack-nova | 00:23 | |
| *** mc_nair has quit IRC | 00:23 | |
| *** annegentle has quit IRC | 00:24 | |
| *** zenoway has joined #openstack-nova | 00:25 | |
| *** zenoway has quit IRC | 00:29 | |
| *** Sukhdev has quit IRC | 00:31 | |
| *** smatzek has quit IRC | 00:31 | |
| *** shakamunyi has quit IRC | 00:35 | |
| *** shakamunyi has joined #openstack-nova | 00:35 | |
| *** dimtruck is now known as zz_dimtruck | 00:36 | |
| *** diogogmt has quit IRC | 00:37 | |
| *** dims_ has joined #openstack-nova | 00:39 | |
| *** markmcclain has quit IRC | 00:39 | |
| *** dims has quit IRC | 00:40 | |
| *** Sukhdev has joined #openstack-nova | 00:40 | |
| *** rfolco has quit IRC | 00:41 | |
| *** markmcclain has joined #openstack-nova | 00:42 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: add test coverage to _compare_result https://review.openstack.org/263440 | 00:46 |
| *** mc_nair has joined #openstack-nova | 00:47 | |
| *** zenoway has joined #openstack-nova | 00:49 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Pass BDM in swap_volume method of LibvirtDriver https://review.openstack.org/257135 | 00:52 |
| *** angdraug has quit IRC | 00:52 | |
| *** zenoway has quit IRC | 00:54 | |
| *** barra204 has joined #openstack-nova | 00:56 | |
| *** shakamunyi has quit IRC | 00:57 | |
| *** hemna is now known as hemnafk | 00:58 | |
| *** Sukhdev has quit IRC | 00:58 | |
| *** zhangjn has joined #openstack-nova | 00:58 | |
| *** mmedvede has quit IRC | 00:58 | |
| *** eil397 has quit IRC | 01:00 | |
| *** mrkz has quit IRC | 01:04 | |
| *** oomichi has quit IRC | 01:04 | |
| *** eil397 has joined #openstack-nova | 01:05 | |
| *** EinstCrazy has joined #openstack-nova | 01:05 | |
| *** EinstCrazy has quit IRC | 01:06 | |
| *** EinstCrazy has joined #openstack-nova | 01:07 | |
| *** barra204 has quit IRC | 01:14 | |
| *** zenoway has joined #openstack-nova | 01:14 | |
| *** shakamunyi has joined #openstack-nova | 01:15 | |
| *** markmcclain has quit IRC | 01:17 | |
| *** zenoway has quit IRC | 01:18 | |
| *** markvoelker has joined #openstack-nova | 01:21 | |
| *** markmcclain has joined #openstack-nova | 01:21 | |
| *** ljxiash has joined #openstack-nova | 01:22 | |
| *** markvoelker has quit IRC | 01:25 | |
| *** markvoelker has joined #openstack-nova | 01:25 | |
| *** Jeffrey4l has quit IRC | 01:25 | |
| *** unicell1 has quit IRC | 01:27 | |
| *** unicell has joined #openstack-nova | 01:27 | |
| *** aginwala has quit IRC | 01:30 | |
| *** barra204 has joined #openstack-nova | 01:31 | |
| *** shakamunyi has quit IRC | 01:32 | |
| *** shz has quit IRC | 01:34 | |
| *** shz has joined #openstack-nova | 01:35 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type https://review.openstack.org/262584 | 01:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr https://review.openstack.org/262585 | 01:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions https://review.openstack.org/249016 | 01:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs https://review.openstack.org/262586 | 01:35 |
| *** FL1SK has joined #openstack-nova | 01:35 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 01:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests https://review.openstack.org/259254 | 01:35 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver https://review.openstack.org/262583 | 01:35 |
| *** shakamunyi has joined #openstack-nova | 01:36 | |
| eliqiao | alex_xu: ping | 01:37 |
| *** barra204 has quit IRC | 01:38 | |
| *** MVenesio has joined #openstack-nova | 01:38 | |
| *** zenoway has joined #openstack-nova | 01:38 | |
| *** barra204 has joined #openstack-nova | 01:38 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy https://review.openstack.org/149625 | 01:39 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 01:39 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties https://review.openstack.org/149626 | 01:39 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 01:39 |
| *** shakamunyi has quit IRC | 01:39 | |
| *** thorst_ has quit IRC | 01:39 | |
| *** thorst has joined #openstack-nova | 01:40 | |
| *** mmedvede has joined #openstack-nova | 01:41 | |
| *** MVenesio has quit IRC | 01:42 | |
| *** zenoway has quit IRC | 01:43 | |
| *** Sukhdev has joined #openstack-nova | 01:43 | |
| *** aginwala has joined #openstack-nova | 01:44 | |
| *** shakamunyi has joined #openstack-nova | 01:44 | |
| *** barra204 has quit IRC | 01:45 | |
| openstackgerrit | Mark Sturdevant proposed openstack/nova: Provide correct connector for evacuate terminate https://review.openstack.org/266095 | 01:46 |
| *** zz_dimtruck is now known as dimtruck | 01:47 | |
| *** haomaiwang has quit IRC | 01:48 | |
| *** dims_ has quit IRC | 01:48 | |
| *** shakamunyi has quit IRC | 01:48 | |
| *** 14WAAN277 has joined #openstack-nova | 01:48 | |
| *** thorst has quit IRC | 01:48 | |
| *** dims has joined #openstack-nova | 01:48 | |
| *** shakamunyi has joined #openstack-nova | 01:49 | |
| *** 14WAAN277 has quit IRC | 01:50 | |
| *** EinstCrazy has quit IRC | 01:50 | |
| *** EinstCrazy has joined #openstack-nova | 01:51 | |
| *** markmcclain has quit IRC | 01:51 | |
| *** mtanino has quit IRC | 01:52 | |
| alex_xu | eliqiao: pong | 01:53 |
| *** suro-patz has quit IRC | 01:54 | |
| eliqiao | alex_xu: thanks for your reviewing for https://review.openstack.org/#/c/254101/ , I replied in lines. it seems I don't quite understand about rpc api version. | 01:54 |
| eliqiao | alex_xu: would you give a quick explaination? | 01:54 |
| *** dims has quit IRC | 01:54 | |
| *** shakamunyi has quit IRC | 01:54 | |
| *** markmcclain has joined #openstack-nova | 01:55 | |
| *** aginwala has quit IRC | 01:57 | |
| alex_xu | eliqiao: I thought we should return fault when the rpc doesn't support 'None' value for block_migration and disk_overcommit | 01:58 |
| *** Sukhdev has quit IRC | 02:01 | |
| *** EinstCra_ has joined #openstack-nova | 02:03 | |
| eliqiao | alex_xu: you meant define a new exception? | 02:03 |
| *** markmcclain has quit IRC | 02:03 | |
| eliqiao | Also REST api should deal with that exception, right ? | 02:04 |
| *** jhesketh has quit IRC | 02:05 | |
| *** markmcclain has joined #openstack-nova | 02:05 | |
| *** jhesketh has joined #openstack-nova | 02:05 | |
| *** dims has joined #openstack-nova | 02:06 | |
| *** EinstCra_ has quit IRC | 02:06 | |
| *** EinstCrazy has quit IRC | 02:06 | |
| *** eil397 has quit IRC | 02:07 | |
| *** lyanchih has joined #openstack-nova | 02:08 | |
| *** EinstCrazy has joined #openstack-nova | 02:08 | |
| *** shakamunyi has joined #openstack-nova | 02:08 | |
| *** shakamunyi has quit IRC | 02:10 | |
| *** kaisers1 has quit IRC | 02:10 | |
| alex_xu | eliqiao: yea, I think so | 02:10 |
| *** gangil1 has quit IRC | 02:10 | |
| eliqiao | alex_xu: Thank you very much, I will do a revison before today's LM meeting. | 02:11 |
| *** kaisers has joined #openstack-nova | 02:11 | |
| *** qianli has joined #openstack-nova | 02:11 | |
| alex_xu | eliqiao: also about the change of LibvirtMigrateData, as I remember we can't change the field type. I guess the nullable also | 02:11 |
| alex_xu | but I need ensure that first | 02:11 |
| alex_xu | eliqiao: np | 02:11 |
| *** shakamunyi has joined #openstack-nova | 02:12 | |
| eliqiao | alex_xu: Thx | 02:12 |
| *** markstur has quit IRC | 02:12 | |
| *** zenoway has joined #openstack-nova | 02:14 | |
| *** markmcclain has quit IRC | 02:15 | |
| *** gangil has joined #openstack-nova | 02:15 | |
| *** gangil has joined #openstack-nova | 02:15 | |
| *** smatzek has joined #openstack-nova | 02:17 | |
| *** markmcclain has joined #openstack-nova | 02:18 | |
| *** zenoway has quit IRC | 02:19 | |
| *** otter768 has joined #openstack-nova | 02:21 | |
| *** shakamunyi has quit IRC | 02:22 | |
| *** shakamunyi has joined #openstack-nova | 02:23 | |
| *** dimtruck is now known as zz_dimtruck | 02:24 | |
| *** tjones has left #openstack-nova | 02:24 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 02:25 |
| *** gongysh has joined #openstack-nova | 02:26 | |
| *** otter768 has quit IRC | 02:26 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 02:30 |
| *** shakamunyi has quit IRC | 02:32 | |
| *** diogogmt has joined #openstack-nova | 02:33 | |
| *** SourabhP has quit IRC | 02:33 | |
| *** aginwala has joined #openstack-nova | 02:36 | |
| *** rk4n_ has joined #openstack-nova | 02:38 | |
| *** zenoway has joined #openstack-nova | 02:39 | |
| *** rk4n has quit IRC | 02:39 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add "hw_firmware_type" image metadata https://review.openstack.org/262929 | 02:41 |
| *** houming has joined #openstack-nova | 02:41 | |
| *** zenoway has quit IRC | 02:43 | |
| *** zhangjn has quit IRC | 02:43 | |
| *** fawadkhaliq has joined #openstack-nova | 02:44 | |
| openstackgerrit | linwei wu proposed openstack/nova: Docstring change for consistency https://review.openstack.org/265773 | 02:46 |
| *** salv-orlando has quit IRC | 02:47 | |
| *** zhangjn has joined #openstack-nova | 02:48 | |
| *** thorst has joined #openstack-nova | 02:48 | |
| *** rk4n has joined #openstack-nova | 02:50 | |
| *** rk4n_ has quit IRC | 02:50 | |
| *** rk4n has quit IRC | 02:52 | |
| *** rk4n has joined #openstack-nova | 02:53 | |
| *** thorst has quit IRC | 02:53 | |
| *** amotoki has joined #openstack-nova | 02:54 | |
| *** rk4n has quit IRC | 02:54 | |
| *** rk4n_ has joined #openstack-nova | 02:54 | |
| *** rk4n has joined #openstack-nova | 02:56 | |
| *** rk4n_ has quit IRC | 02:56 | |
| *** rk4n_ has joined #openstack-nova | 02:57 | |
| *** rk4n has quit IRC | 02:57 | |
| *** rk4n has joined #openstack-nova | 02:58 | |
| *** rk4n_ has quit IRC | 02:58 | |
| *** zhangjn has quit IRC | 02:58 | |
| *** salv-orlando has joined #openstack-nova | 02:58 | |
| *** rk4n_ has joined #openstack-nova | 03:00 | |
| *** rk4n has quit IRC | 03:00 | |
| *** kevinbenton_ has joined #openstack-nova | 03:00 | |
| *** smatzek has quit IRC | 03:00 | |
| *** apoorvad has quit IRC | 03:02 | |
| *** rk4n has joined #openstack-nova | 03:03 | |
| *** rk4n_ has quit IRC | 03:03 | |
| *** zenoway has joined #openstack-nova | 03:03 | |
| *** oomichi has joined #openstack-nova | 03:03 | |
| *** kevinbenton_ has quit IRC | 03:03 | |
| *** rk4n has quit IRC | 03:04 | |
| *** markmcclain has quit IRC | 03:04 | |
| *** rk4n has joined #openstack-nova | 03:04 | |
| *** zhangjn has joined #openstack-nova | 03:05 | |
| *** zz_dimtruck is now known as dimtruck | 03:06 | |
| *** zenoway has quit IRC | 03:07 | |
| *** markmcclain has joined #openstack-nova | 03:07 | |
| *** ljxiash has quit IRC | 03:08 | |
| *** ljxiash has joined #openstack-nova | 03:08 | |
| *** aginwala has quit IRC | 03:09 | |
| *** jwcroppe has quit IRC | 03:10 | |
| *** jwcroppe has joined #openstack-nova | 03:11 | |
| *** rk4n_ has joined #openstack-nova | 03:12 | |
| *** rk4n has quit IRC | 03:12 | |
| *** aginwala has joined #openstack-nova | 03:12 | |
| *** amotoki has quit IRC | 03:12 | |
| *** markmcclain has quit IRC | 03:12 | |
| *** SourabhP has joined #openstack-nova | 03:12 | |
| *** ljxiash has quit IRC | 03:13 | |
| *** amotoki has joined #openstack-nova | 03:14 | |
| *** rk4n_ has quit IRC | 03:14 | |
| *** rk4n has joined #openstack-nova | 03:15 | |
| *** zhangjn has quit IRC | 03:15 | |
| *** rk4n has quit IRC | 03:15 | |
| *** jwcroppe has quit IRC | 03:15 | |
| *** rk4n has joined #openstack-nova | 03:15 | |
| *** SourabhP_ has joined #openstack-nova | 03:16 | |
| *** aginwala has quit IRC | 03:16 | |
| *** ljxiash has joined #openstack-nova | 03:16 | |
| *** rk4n has quit IRC | 03:16 | |
| *** rk4n has joined #openstack-nova | 03:17 | |
| *** SourabhP has quit IRC | 03:17 | |
| *** SourabhP_ is now known as SourabhP | 03:17 | |
| *** mragupat has joined #openstack-nova | 03:18 | |
| *** rk4n has quit IRC | 03:18 | |
| *** markmcclain has joined #openstack-nova | 03:18 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 03:19 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 03:19 |
| *** rk4n has joined #openstack-nova | 03:19 | |
| *** zhangjn has joined #openstack-nova | 03:19 | |
| *** rk4n has quit IRC | 03:19 | |
| *** rk4n has joined #openstack-nova | 03:19 | |
| *** vilobhmm11 has quit IRC | 03:23 | |
| *** dims has quit IRC | 03:24 | |
| *** rk4n has quit IRC | 03:24 | |
| *** markmcclain has quit IRC | 03:25 | |
| *** SourabhP has quit IRC | 03:25 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 03:26 |
| *** zenoway has joined #openstack-nova | 03:27 | |
| *** vilobhmm11 has joined #openstack-nova | 03:27 | |
| *** doug-fish has joined #openstack-nova | 03:30 | |
| *** zenoway has quit IRC | 03:31 | |
| *** ccard_ has joined #openstack-nova | 03:31 | |
| *** vilobhmm11 has quit IRC | 03:32 | |
| *** EinstCrazy has quit IRC | 03:33 | |
| *** EinstCrazy has joined #openstack-nova | 03:33 | |
| *** ccard__ has quit IRC | 03:34 | |
| *** amotoki has quit IRC | 03:36 | |
| *** MVenesio has joined #openstack-nova | 03:39 | |
| *** suro-patz has joined #openstack-nova | 03:41 | |
| *** MVenesio has quit IRC | 03:44 | |
| *** thorst has joined #openstack-nova | 03:51 | |
| *** zenoway has joined #openstack-nova | 03:51 | |
| *** vivekd has joined #openstack-nova | 03:53 | |
| *** zenoway has quit IRC | 03:55 | |
| *** zhangjn has quit IRC | 03:56 | |
| *** doug-fish has quit IRC | 03:58 | |
| *** thorst has quit IRC | 03:58 | |
| *** Marga_ has joined #openstack-nova | 03:58 | |
| *** Marga_ has quit IRC | 03:59 | |
| *** Marga_ has joined #openstack-nova | 03:59 | |
| *** gangil has quit IRC | 04:02 | |
| *** akshai has joined #openstack-nova | 04:03 | |
| *** amotoki has joined #openstack-nova | 04:04 | |
| *** akshai_ has joined #openstack-nova | 04:04 | |
| *** kevinbenton_ has joined #openstack-nova | 04:07 | |
| *** akshai has quit IRC | 04:08 | |
| *** salv-orl_ has joined #openstack-nova | 04:10 | |
| *** yamahata has quit IRC | 04:10 | |
| openstackgerrit | Chaozhe Chen proposed openstack/python-novaclient: Test: Clean v2 client userwarning https://review.openstack.org/265725 | 04:11 |
| *** salv-orlando has quit IRC | 04:13 | |
| *** lyanchih has quit IRC | 04:15 | |
| *** otter768 has joined #openstack-nova | 04:22 | |
| *** gcb has joined #openstack-nova | 04:22 | |
| *** bwallis_ has quit IRC | 04:22 | |
| *** jwcroppe has joined #openstack-nova | 04:25 | |
| *** links has joined #openstack-nova | 04:26 | |
| *** Marga_ has quit IRC | 04:26 | |
| *** otter768 has quit IRC | 04:26 | |
| *** RA_ has joined #openstack-nova | 04:28 | |
| *** zenoway has joined #openstack-nova | 04:28 | |
| *** coolsvap|away is now known as coolsvap | 04:29 | |
| *** dimtruck is now known as zz_dimtruck | 04:29 | |
| *** zenoway has quit IRC | 04:32 | |
| *** akshai_ has quit IRC | 04:35 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: enable uefi boot for x86_64 https://review.openstack.org/262930 | 04:43 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: update feature support matrix documentation https://review.openstack.org/262931 | 04:44 |
| openstackgerrit | venkatamahesh proposed openstack/nova: Improvements in conditional python code https://review.openstack.org/266143 | 04:45 |
| *** suro-patz has quit IRC | 04:48 | |
| *** sudipto has joined #openstack-nova | 04:49 | |
| *** suro-patz has joined #openstack-nova | 04:49 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 04:49 |
| *** suro-patz has quit IRC | 04:50 | |
| *** gangil has joined #openstack-nova | 04:50 | |
| *** gangil has joined #openstack-nova | 04:50 | |
| *** zenoway has joined #openstack-nova | 04:52 | |
| *** fawadkhaliq has quit IRC | 04:52 | |
| *** zz_dimtruck is now known as dimtruck | 04:55 | |
| *** zenoway has quit IRC | 04:56 | |
| *** thorst has joined #openstack-nova | 04:57 | |
| *** pradipta has joined #openstack-nova | 04:58 | |
| *** ljxiash has quit IRC | 04:58 | |
| *** markvoelker has quit IRC | 04:58 | |
| *** ljxiash has joined #openstack-nova | 04:59 | |
| *** baoli has quit IRC | 04:59 | |
| *** baoli has joined #openstack-nova | 05:00 | |
| *** ljxiash has quit IRC | 05:01 | |
| *** ljxiash has joined #openstack-nova | 05:02 | |
| *** bwallis has joined #openstack-nova | 05:03 | |
| *** thorst has quit IRC | 05:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/266160 | 05:04 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/266173 | 05:05 |
| *** aginwala has joined #openstack-nova | 05:09 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 05:09 |
| *** dimtruck is now known as zz_dimtruck | 05:09 | |
| *** fawadkhaliq has joined #openstack-nova | 05:15 | |
| *** ijw has quit IRC | 05:24 | |
| *** EinstCra_ has joined #openstack-nova | 05:25 | |
| *** EinstCrazy has quit IRC | 05:25 | |
| *** zenoway has joined #openstack-nova | 05:28 | |
| *** sudipto has quit IRC | 05:29 | |
| *** EinstCra_ has quit IRC | 05:29 | |
| *** EinstCrazy has joined #openstack-nova | 05:30 | |
| *** aginwala has quit IRC | 05:30 | |
| *** mragupat has quit IRC | 05:32 | |
| *** aginwala has joined #openstack-nova | 05:32 | |
| *** zenoway has quit IRC | 05:32 | |
| *** zhihais has joined #openstack-nova | 05:33 | |
| *** zhangjn has joined #openstack-nova | 05:35 | |
| *** cfriesen has quit IRC | 05:35 | |
| *** pradipta has quit IRC | 05:37 | |
| *** sudipto has joined #openstack-nova | 05:39 | |
| *** MVenesio has joined #openstack-nova | 05:39 | |
| *** MVenesio has quit IRC | 05:44 | |
| *** moshele has joined #openstack-nova | 05:46 | |
| *** jaosorior has joined #openstack-nova | 05:47 | |
| *** zhangjn has quit IRC | 05:48 | |
| *** zhangjn has joined #openstack-nova | 05:50 | |
| *** yamahata has joined #openstack-nova | 05:51 | |
| openstackgerrit | venkatamahesh proposed openstack/nova: Improvements in conditional python code https://review.openstack.org/266143 | 05:53 |
| *** ildikov has quit IRC | 05:57 | |
| *** markvoelker has joined #openstack-nova | 05:59 | |
| *** bkumar has joined #openstack-nova | 06:01 | |
| *** thorst has joined #openstack-nova | 06:01 | |
| *** edmondsw has quit IRC | 06:02 | |
| *** EinstCrazy has quit IRC | 06:03 | |
| *** EinstCrazy has joined #openstack-nova | 06:03 | |
| *** kevinbenton_ has quit IRC | 06:04 | |
| *** lyanchih has joined #openstack-nova | 06:04 | |
| *** ankit_ag has joined #openstack-nova | 06:04 | |
| *** zenoway has joined #openstack-nova | 06:04 | |
| *** markvoelker has quit IRC | 06:06 | |
| *** thorst has quit IRC | 06:08 | |
| *** zenoway has quit IRC | 06:09 | |
| *** Jeffrey4l has joined #openstack-nova | 06:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/265601 | 06:16 |
| *** qianli has quit IRC | 06:18 | |
| *** qianli has joined #openstack-nova | 06:19 | |
| *** vilobhmm11 has joined #openstack-nova | 06:20 | |
| *** vilobhmm111 has joined #openstack-nova | 06:22 | |
| *** vilobhmm11 has quit IRC | 06:25 | |
| *** 6A4ABMVDU has joined #openstack-nova | 06:26 | |
| *** aginwala_ has joined #openstack-nova | 06:26 | |
| openstackgerrit | ting wang proposed openstack/python-novaclient: Add debug testenv in tox https://review.openstack.org/266219 | 06:26 |
| *** lyanchih has quit IRC | 06:27 | |
| *** sudipto has quit IRC | 06:27 | |
| *** gangil has quit IRC | 06:29 | |
| *** zenoway has joined #openstack-nova | 06:29 | |
| *** nkrinner has joined #openstack-nova | 06:29 | |
| *** aginwala has quit IRC | 06:30 | |
| *** vilobhmm111 has quit IRC | 06:31 | |
| *** lyanchih has joined #openstack-nova | 06:32 | |
| *** aginwala_ has quit IRC | 06:32 | |
| *** vilobhmm11 has joined #openstack-nova | 06:32 | |
| *** zenoway has quit IRC | 06:33 | |
| *** zhangjn has quit IRC | 06:35 | |
| *** zhangjn has joined #openstack-nova | 06:35 | |
| *** zhangjn_ has joined #openstack-nova | 06:38 | |
| *** EinstCrazy has quit IRC | 06:39 | |
| *** EinstCrazy has joined #openstack-nova | 06:39 | |
| *** zhangjn has quit IRC | 06:40 | |
| *** esracelik has joined #openstack-nova | 06:43 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add "hw_firmware_type" image metadata https://review.openstack.org/262929 | 06:43 |
| *** EinstCra_ has joined #openstack-nova | 06:45 | |
| *** EinstCrazy has quit IRC | 06:45 | |
| *** jaosorior has quit IRC | 06:48 | |
| *** fawadkhaliq has quit IRC | 06:49 | |
| *** salv-orl_ has quit IRC | 06:50 | |
| *** salv-orlando has joined #openstack-nova | 06:50 | |
| *** fawadkhaliq has joined #openstack-nova | 06:51 | |
| *** Jialiang has joined #openstack-nova | 06:52 | |
| Jialiang | Hello, | 06:53 |
| Jialiang | I submitted a bug-fix change several days ago, could anybody help to review it? | 06:54 |
| Jialiang | https://review.openstack.org/#/c/264157/2 | 06:54 |
| Jialiang | Thanks | 06:55 |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance. https://review.openstack.org/202617 | 06:56 |
| *** EinstCra_ has quit IRC | 06:56 | |
| *** EinstCrazy has joined #openstack-nova | 06:57 | |
| *** EinstCrazy has quit IRC | 06:57 | |
| *** EinstCrazy has joined #openstack-nova | 06:57 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add checking for new image metadata property 'hw_cpu_realtime_mask' https://review.openstack.org/266233 | 06:58 |
| *** EinstCrazy has quit IRC | 07:00 | |
| *** EinstCra_ has joined #openstack-nova | 07:00 | |
| *** zenoway has joined #openstack-nova | 07:03 | |
| *** 6A4ABMVDU has quit IRC | 07:03 | |
| *** mnestratov has joined #openstack-nova | 07:05 | |
| *** haomaiwang has joined #openstack-nova | 07:05 | |
| *** thorst has joined #openstack-nova | 07:06 | |
| *** paul-carlton2 has joined #openstack-nova | 07:07 | |
| *** mjura has joined #openstack-nova | 07:08 | |
| *** EinstCra_ has quit IRC | 07:08 | |
| *** EinstCrazy has joined #openstack-nova | 07:08 | |
| *** jaosorior has joined #openstack-nova | 07:09 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix deleted non-public flavor lookups for non-admin users https://review.openstack.org/211809 | 07:10 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: Make instance snapshots to respect the force_raw_images setting https://review.openstack.org/226069 | 07:11 |
| *** thorst has quit IRC | 07:13 | |
| *** dedery has joined #openstack-nova | 07:21 | |
| *** dedery has quit IRC | 07:22 | |
| *** dedery has joined #openstack-nova | 07:22 | |
| *** zhangjn_ has quit IRC | 07:23 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Calculate block_migration in virt drivers if it's None https://review.openstack.org/254101 | 07:24 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 07:24 |
| *** haomaiwang has quit IRC | 07:25 | |
| *** suro-patz has joined #openstack-nova | 07:26 | |
| *** haomaiwang has joined #openstack-nova | 07:26 | |
| *** zhangjn has joined #openstack-nova | 07:30 | |
| *** gongysh has quit IRC | 07:30 | |
| *** moshele has quit IRC | 07:31 | |
| *** gongysh has joined #openstack-nova | 07:32 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add swap volume notifications https://review.openstack.org/250283 | 07:32 |
| *** vks_ has joined #openstack-nova | 07:32 | |
| *** sahid has joined #openstack-nova | 07:38 | |
| *** admin0 has joined #openstack-nova | 07:39 | |
| *** MVenesio has joined #openstack-nova | 07:40 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Remove catching of ComputeHostNotFound exception https://review.openstack.org/259470 | 07:40 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 07:40 |
| *** EinstCra_ has joined #openstack-nova | 07:41 | |
| *** EinstCrazy has quit IRC | 07:41 | |
| *** e0ne has joined #openstack-nova | 07:42 | |
| *** Jialiang has quit IRC | 07:42 | |
| *** RA_ has quit IRC | 07:43 | |
| *** vilobhmm11 has quit IRC | 07:43 | |
| *** MVenesio has quit IRC | 07:44 | |
| *** admin0 has quit IRC | 07:47 | |
| *** vivekd_ has joined #openstack-nova | 07:49 | |
| *** mpavone has joined #openstack-nova | 07:49 | |
| *** rk4n has joined #openstack-nova | 07:51 | |
| *** moshele has joined #openstack-nova | 07:51 | |
| *** vivekd has quit IRC | 07:52 | |
| *** vivekd_ is now known as vivekd | 07:52 | |
| *** belmoreira has joined #openstack-nova | 07:53 | |
| *** klkumar has joined #openstack-nova | 07:54 | |
| *** haomaiwang has quit IRC | 07:56 | |
| *** haomaiwa_ has joined #openstack-nova | 07:58 | |
| *** haomaiwa_ has quit IRC | 07:59 | |
| *** haomaiwa_ has joined #openstack-nova | 07:59 | |
| *** otter768 has joined #openstack-nova | 08:00 | |
| *** haomaiwa_ has quit IRC | 08:01 | |
| *** haomaiwa_ has joined #openstack-nova | 08:01 | |
| *** salv-orl_ has joined #openstack-nova | 08:01 | |
| *** otter768_ has joined #openstack-nova | 08:02 | |
| *** rk4n has quit IRC | 08:02 | |
| *** otter768 has quit IRC | 08:02 | |
| *** rk4n has joined #openstack-nova | 08:02 | |
| *** otter768 has joined #openstack-nova | 08:04 | |
| *** salv-orlando has quit IRC | 08:05 | |
| *** salv-orl_ has quit IRC | 08:06 | |
| *** otter76__ has joined #openstack-nova | 08:06 | |
| *** jistr has joined #openstack-nova | 08:06 | |
| *** otter768_ has quit IRC | 08:06 | |
| *** jistr is now known as jistr|doc | 08:07 | |
| *** EinstCrazy has joined #openstack-nova | 08:07 | |
| *** EinstCra_ has quit IRC | 08:07 | |
| *** suro-patz has quit IRC | 08:08 | |
| *** otter768_ has joined #openstack-nova | 08:08 | |
| *** otter76__ has quit IRC | 08:08 | |
| *** jed56 has joined #openstack-nova | 08:08 | |
| *** otter768 has quit IRC | 08:09 | |
| *** otter768_ has quit IRC | 08:09 | |
| *** otter768 has joined #openstack-nova | 08:10 | |
| *** thorst has joined #openstack-nova | 08:11 | |
| *** otter768_ has joined #openstack-nova | 08:12 | |
| *** EinstCrazy has quit IRC | 08:12 | |
| *** matrohon has joined #openstack-nova | 08:12 | |
| *** EinstCrazy has joined #openstack-nova | 08:12 | |
| *** gcb has quit IRC | 08:12 | |
| *** vks_ has quit IRC | 08:13 | |
| *** otter768_ has quit IRC | 08:14 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add "hw_firmware_type" image metadata https://review.openstack.org/262929 | 08:14 |
| *** otter76__ has joined #openstack-nova | 08:14 | |
| *** otter768 has quit IRC | 08:14 | |
| *** zhangjn has quit IRC | 08:14 | |
| *** ildikov has joined #openstack-nova | 08:15 | |
| *** otter768 has joined #openstack-nova | 08:16 | |
| *** e0ne has quit IRC | 08:16 | |
| *** zhangjn has joined #openstack-nova | 08:16 | |
| *** otter768 has quit IRC | 08:17 | |
| *** zakora has joined #openstack-nova | 08:17 | |
| *** otter768 has joined #openstack-nova | 08:17 | |
| *** otter76__ has quit IRC | 08:18 | |
| *** eliqiao has quit IRC | 08:18 | |
| *** thorst has quit IRC | 08:18 | |
| *** otter768 has quit IRC | 08:19 | |
| *** jichen has joined #openstack-nova | 08:19 | |
| *** otter768 has joined #openstack-nova | 08:19 | |
| *** otter768_ has joined #openstack-nova | 08:21 | |
| *** fawadkhaliq has quit IRC | 08:22 | |
| *** fawadkhaliq has joined #openstack-nova | 08:22 | |
| *** otter76__ has joined #openstack-nova | 08:23 | |
| *** otter768 has quit IRC | 08:23 | |
| *** otter76__ has quit IRC | 08:25 | |
| *** otter768 has joined #openstack-nova | 08:25 | |
| *** otter768_ has quit IRC | 08:25 | |
| *** haomaiwa_ has quit IRC | 08:26 | |
| *** otter768 has quit IRC | 08:26 | |
| *** otter768 has joined #openstack-nova | 08:27 | |
| *** otter768_ has joined #openstack-nova | 08:29 | |
| *** otter768 has quit IRC | 08:29 | |
| *** gcb has joined #openstack-nova | 08:29 | |
| *** rdopiera has joined #openstack-nova | 08:30 | |
| *** otter768 has joined #openstack-nova | 08:31 | |
| *** otter768_ has quit IRC | 08:31 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove version hardcoding for 'show' method https://review.openstack.org/228578 | 08:32 |
| *** otter768 has quit IRC | 08:32 | |
| *** otter768 has joined #openstack-nova | 08:33 | |
| *** haomaiwa_ has joined #openstack-nova | 08:33 | |
| *** lyanchih has quit IRC | 08:34 | |
| *** otter768_ has joined #openstack-nova | 08:35 | |
| *** liverpooler has joined #openstack-nova | 08:35 | |
| *** markus_z has joined #openstack-nova | 08:35 | |
| *** otter768_ has quit IRC | 08:36 | |
| *** otter768_ has joined #openstack-nova | 08:37 | |
| *** otter768 has quit IRC | 08:37 | |
| *** otter768_ has quit IRC | 08:38 | |
| *** otter768 has joined #openstack-nova | 08:38 | |
| *** admin0 has joined #openstack-nova | 08:39 | |
| *** otter768 has quit IRC | 08:40 | |
| *** otter768 has joined #openstack-nova | 08:40 | |
| *** ijw has joined #openstack-nova | 08:41 | |
| *** haomaiwa_ has quit IRC | 08:42 | |
| *** otter768_ has joined #openstack-nova | 08:42 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 08:42 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model https://review.openstack.org/176242 | 08:42 |
| *** haomaiwa_ has joined #openstack-nova | 08:44 | |
| *** otter768_ has quit IRC | 08:44 | |
| *** otter768 has quit IRC | 08:45 | |
| *** otter768 has joined #openstack-nova | 08:46 | |
| *** jlanoux has joined #openstack-nova | 08:47 | |
| *** jlanoux has quit IRC | 08:48 | |
| *** otter768 has quit IRC | 08:48 | |
| *** jlanoux has joined #openstack-nova | 08:48 | |
| *** otter768 has joined #openstack-nova | 08:48 | |
| *** otter768_ has joined #openstack-nova | 08:50 | |
| *** zhangjn has quit IRC | 08:50 | |
| *** vks has joined #openstack-nova | 08:50 | |
| dedery | ndipanov: ping | 08:50 |
| *** rotbeard has joined #openstack-nova | 08:51 | |
| *** yamahata has quit IRC | 08:51 | |
| *** otter768_ has quit IRC | 08:51 | |
| *** yamahata has joined #openstack-nova | 08:52 | |
| *** otter768_ has joined #openstack-nova | 08:52 | |
| *** otter768 has quit IRC | 08:53 | |
| *** ijw has quit IRC | 08:54 | |
| *** ijw has joined #openstack-nova | 08:54 | |
| *** otter768 has joined #openstack-nova | 08:56 | |
| *** otter768_ has quit IRC | 08:57 | |
| *** otter768_ has joined #openstack-nova | 08:58 | |
| *** zhangjn has joined #openstack-nova | 08:58 | |
| *** otter768 has quit IRC | 08:58 | |
| vks | hi | 08:59 |
| *** jistr|doc has quit IRC | 08:59 | |
| vks | i am observing one issue with nova, I have 10 ports on Vm, I see after reboot, one port is not there | 08:59 |
| *** otter768 has joined #openstack-nova | 08:59 | |
| *** otter768_ has quit IRC | 09:00 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** belmoreira has quit IRC | 09:01 | |
| *** chaochin has quit IRC | 09:01 | |
| *** 77CAAI2UM has joined #openstack-nova | 09:01 | |
| *** ndipanov has quit IRC | 09:01 | |
| *** otter768 has quit IRC | 09:01 | |
| *** otter768_ has joined #openstack-nova | 09:02 | |
| *** Marga_ has joined #openstack-nova | 09:02 | |
| *** otter768_ has quit IRC | 09:03 | |
| dedery | vks: do you see any error in the nova-compute log? | 09:03 |
| *** otter768_ has joined #openstack-nova | 09:04 | |
| vks | dedery, Don't see any error log, but in instance_cache_info table doesn't has the port entry | 09:04 |
| *** otter768_ has quit IRC | 09:05 | |
| *** chaochin has joined #openstack-nova | 09:05 | |
| *** amotoki has quit IRC | 09:05 | |
| *** otter76__ has joined #openstack-nova | 09:05 | |
| *** ijw has quit IRC | 09:06 | |
| *** amotoki has joined #openstack-nova | 09:07 | |
| *** otter7___ has joined #openstack-nova | 09:07 | |
| *** otter76__ has quit IRC | 09:07 | |
| *** amotoki has quit IRC | 09:07 | |
| *** otter____ has joined #openstack-nova | 09:09 | |
| tobasco | johnthetubaguy: did some digging about the bug we talked about yesterday and setting root_gb to zero in the reqspec that is built by build_request_spec doesn't affect the values that is inserted into the database so the resource tracker, api, horizon, diskfilter etc will still use the wrong values. | 09:09 |
| *** daemontool has joined #openstack-nova | 09:11 | |
| *** otter____ has quit IRC | 09:11 | |
| *** otte_____ has joined #openstack-nova | 09:11 | |
| *** otter7___ has quit IRC | 09:11 | |
| *** otte_____ has quit IRC | 09:13 | |
| *** otter____ has joined #openstack-nova | 09:13 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova: Add host_status attribute for servers/detail and servers/{server_id} https://review.openstack.org/241562 | 09:14 |
| *** otter768 has joined #openstack-nova | 09:15 | |
| *** otter____ has quit IRC | 09:15 | |
| *** thorst has joined #openstack-nova | 09:16 | |
| *** otter768_ has joined #openstack-nova | 09:17 | |
| *** otter768 has quit IRC | 09:17 | |
| *** openstackgerrit has quit IRC | 09:17 | |
| *** openstackgerrit has joined #openstack-nova | 09:18 | |
| *** tpatzig has joined #openstack-nova | 09:18 | |
| *** otter768 has joined #openstack-nova | 09:19 | |
| *** salv-orlando has joined #openstack-nova | 09:19 | |
| dedery | vks: known issue. There's a lot of debate in the reviews around the potential fixes for this issue. You might benefit from manually applying the patches on your own (until it'll get resolved upstream, I have no idea of course which openstack release are you running and how much it's behind the upstream). Take a look at: https://bugs.launchpad.net/nova/+bug/1467581, https://bugs.launchpad.net/nova/+bug/1501735 and a | 09:19 |
| openstack | Launchpad bug 1467581 in OpenStack Compute (nova) "Concurrent interface attachment corrupts info cache" [High,Fix released] - Assigned to Mark Goddard (mgoddard) | 09:19 |
| dedery | bunch of related bugs. Anyway, once the entry is not in instance_info_caches.network_info, there's a debatable piece of code that refreshes it from Neutron (see https://review.openstack.org/#/c/230919/ which reverts it) but the side effect of it (don't know if it's important for you) is that you loose the vNIC ordering (since the only place that holds the order is instance_info_caches.network_info). I can tell you f | 09:19 |
| dedery | rom experience that in our case (VMs with 10-15 ports, large NFV application) it took 3 of these patches to resolve the issue (including my own https://review.openstack.org/#/c/252565). I think that sahid is better qualified to say what needs to be in and what not. | 09:19 |
| openstack | Launchpad bug 1501735 in OpenStack Compute (nova) "Network interface allocation corrupts instance info cache" [Medium,In progress] - Assigned to sahid (sahid-ferdjaoui) | 09:19 |
| *** jistr has joined #openstack-nova | 09:20 | |
| *** otter76__ has joined #openstack-nova | 09:21 | |
| dedery | and mgoddard probably. | 09:21 |
| *** otter768_ has quit IRC | 09:22 | |
| *** otter76__ has quit IRC | 09:22 | |
| *** otter768_ has joined #openstack-nova | 09:23 | |
| *** otter768 has quit IRC | 09:23 | |
| *** thorst has quit IRC | 09:23 | |
| *** otter768 has joined #openstack-nova | 09:24 | |
| *** otter768_ has quit IRC | 09:24 | |
| *** alexander has quit IRC | 09:24 | |
| vks | dedery, so this https://review.openstack.org/#/c/194597/ is the patch which fix the issue? | 09:25 |
| *** otter768_ has joined #openstack-nova | 09:26 | |
| *** otter768 has quit IRC | 09:26 | |
| dedery | vks: you can try it. It worked for us. | 09:27 |
| *** yamahata has quit IRC | 09:27 | |
| vks | dedery, so you mentioned there are other patches also needed/ | 09:27 |
| vks | ? | 09:27 |
| *** lpetrut has joined #openstack-nova | 09:28 | |
| *** otter768 has joined #openstack-nova | 09:28 | |
| *** otter768_ has quit IRC | 09:28 | |
| dedery | vks: again, it depends if you see things like duplicate NICs/NIC order changes. We saw it all and had to fix some of those ourselves. | 09:29 |
| *** otter768 has quit IRC | 09:30 | |
| *** Marga_ has quit IRC | 09:30 | |
| *** ndipanov has joined #openstack-nova | 09:30 | |
| *** otter768 has joined #openstack-nova | 09:30 | |
| ndipanov | dedery, hi | 09:30 |
| *** garyk1 has quit IRC | 09:31 | |
| *** otter768 has quit IRC | 09:31 | |
| *** otter768 has joined #openstack-nova | 09:32 | |
| vks | dedery, thnx will get back to you once I apply the fix in setup | 09:33 |
| *** otter768 has quit IRC | 09:34 | |
| *** otter768 has joined #openstack-nova | 09:34 | |
| *** derekh has joined #openstack-nova | 09:35 | |
| *** yassine has joined #openstack-nova | 09:35 | |
| *** otter768 has quit IRC | 09:35 | |
| dedery | ndipanov: hi, A. thank you for the review (https://review.openstack.org/#/c/252565). B. as you can see from the chat above (vks' observation) the disappearing vNICs/vNIC ordering/vNIC duplication is becoming more of a concern. There are a lot of attempts out there to resolve it but there's no one coherent joint effort. Maybe this is something that should be addressed in the weekly meeting? | 09:36 |
| *** otter768 has joined #openstack-nova | 09:36 | |
| *** zhangjn has quit IRC | 09:37 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Enable volume operations for shelved instances https://review.openstack.org/265898 | 09:37 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: xenapi: simplify swap_xapi_host() https://review.openstack.org/266258 | 09:37 |
| *** aix has joined #openstack-nova | 09:37 | |
| *** otter768_ has joined #openstack-nova | 09:38 | |
| *** otter768_ has quit IRC | 09:39 | |
| *** otter768_ has joined #openstack-nova | 09:40 | |
| *** otter768 has quit IRC | 09:40 | |
| *** otter768_ has quit IRC | 09:41 | |
| *** lyanchih has joined #openstack-nova | 09:41 | |
| vks | dedery, can we discuss this on 14th ? | 09:41 |
| *** otter768 has joined #openstack-nova | 09:41 | |
| *** otter768 has quit IRC | 09:43 | |
| *** otter768 has joined #openstack-nova | 09:44 | |
| *** otter768 has quit IRC | 09:45 | |
| *** otter768 has joined #openstack-nova | 09:45 | |
| *** lyanchih has quit IRC | 09:45 | |
| *** otter768 has quit IRC | 09:47 | |
| *** otter768 has joined #openstack-nova | 09:47 | |
| dedery | vks: we can put it on the agenda. I don't know if all items will be covered + I might have a conflict at 1400UTC so there's a chance that I won't be participating. | 09:48 |
| *** e0ne has joined #openstack-nova | 09:48 | |
| *** otter768 has quit IRC | 09:49 | |
| *** sfinucan has joined #openstack-nova | 09:49 | |
| *** otter768 has joined #openstack-nova | 09:49 | |
| vks | dedery, ok thnx | 09:50 |
| ndipanov | vks, dedery what meeting is that? | 09:50 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Allow block live migration of an instance with attached volumes https://review.openstack.org/227278 | 09:50 |
| vks | ndipanov, openstack nova weekly meeting | 09:50 |
| *** otter768 has quit IRC | 09:51 | |
| *** salv-orlando has quit IRC | 09:51 | |
| ndipanov | ah right yeah | 09:51 |
| *** fawadkhaliq has quit IRC | 09:51 | |
| *** otter768 has joined #openstack-nova | 09:51 | |
| ndipanov | I thought it was a subteam | 09:51 |
| *** fawadkhaliq has joined #openstack-nova | 09:52 | |
| *** n0ano has quit IRC | 09:52 | |
| *** slagle has quit IRC | 09:52 | |
| *** sileht has quit IRC | 09:52 | |
| *** NightKhaos has quit IRC | 09:52 | |
| *** davideag_ has joined #openstack-nova | 09:53 | |
| *** otter768_ has joined #openstack-nova | 09:53 | |
| *** acabot has quit IRC | 09:53 | |
| *** tojuvone_ has quit IRC | 09:53 | |
| ildikov | ndipanov: hi | 09:53 |
| ildikov | ndipanov: I added a new comment to your question on this review: https://review.openstack.org/#/c/193135/ | 09:54 |
| *** derekh has quit IRC | 09:54 | |
| *** vishwana_ has joined #openstack-nova | 09:54 | |
| *** baoli_ has joined #openstack-nova | 09:54 | |
| ildikov | ndipanov: also the patches before this one are ready for review for multi-attach | 09:54 |
| *** slagle has joined #openstack-nova | 09:54 | |
| *** otter768_ has quit IRC | 09:54 | |
| ndipanov | ildikov, on my list :) will look today | 09:55 |
| *** gongysh has quit IRC | 09:55 | |
| *** derekh has joined #openstack-nova | 09:55 | |
| ildikov | ndipanov: great, thanks much :) | 09:55 |
| *** otter768_ has joined #openstack-nova | 09:55 | |
| *** rk4n has quit IRC | 09:55 | |
| *** davideagnello has quit IRC | 09:56 | |
| sfinucan | markus_z: https://bugs.launchpad.net/oslo.config/+bug/1395819 | 09:56 |
| openstack | Launchpad bug 1395819 in oslo.config "oslo-config-generator does not emit group parameter help" [Medium,Fix released] - Assigned to Chris Hoge (hoge) | 09:56 |
| openstackgerrit | Michal Jura proposed openstack/nova: Resolve duplicated html5proxy_host and html5proxy_port options https://review.openstack.org/266268 | 09:56 |
| *** vishwanathj has quit IRC | 09:56 | |
| sfinucan | markus_z: https://github.com/openstack/nova/blob/a4b7f52c1b3f4df996ad334929747bcec298db75/requirements.txt#L38 | 09:56 |
| markus_z | sfinucan: yeah, thanks for the info | 09:56 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Live migration fails when the instance has a config_drive_format=iso9660 https://review.openstack.org/234659 | 09:56 |
| sfinucan | Looks like that bug has been fixed | 09:56 |
| markus_z | oh great, we can finally exploit it :) | 09:56 |
| sfinucan | markus_z: Exactly :) I'll update my commits accordingly | 09:56 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 09:56 |
| markus_z | sfinucan: I'll double-check it today if it works as expected. Thanks for the notice! | 09:56 |
| sfinucan | markus_z: np | 09:56 |
| *** otter768 has quit IRC | 09:56 | |
| *** baoli has quit IRC | 09:56 | |
| *** EinstCrazy has quit IRC | 09:57 | |
| *** otter768 has joined #openstack-nova | 09:57 | |
| *** salv-orlando has joined #openstack-nova | 09:57 | |
| *** NightKhaos has joined #openstack-nova | 09:57 | |
| *** houming has quit IRC | 09:58 | |
| *** otter768 has quit IRC | 09:59 | |
| *** amotoki has joined #openstack-nova | 09:59 | |
| *** otter768_ has quit IRC | 09:59 | |
| *** tojuvone has joined #openstack-nova | 10:00 | |
| *** admin0 has quit IRC | 10:00 | |
| * bauzas waves | 10:00 | |
| *** 77CAAI2UM has quit IRC | 10:01 | |
| *** otter768 has joined #openstack-nova | 10:01 | |
| *** admin0 has joined #openstack-nova | 10:01 | |
| *** fawadkhaliq has quit IRC | 10:01 | |
| *** fawadkhaliq has joined #openstack-nova | 10:01 | |
| *** rk4n has joined #openstack-nova | 10:01 | |
| *** acabot has joined #openstack-nova | 10:02 | |
| *** sileht has joined #openstack-nova | 10:02 | |
| *** markvoelker has joined #openstack-nova | 10:02 | |
| *** rk4n has quit IRC | 10:03 | |
| *** ljxiash has quit IRC | 10:04 | |
| *** otter768 has quit IRC | 10:04 | |
| *** otter768_ has joined #openstack-nova | 10:04 | |
| *** ljxiash has joined #openstack-nova | 10:04 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 10:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 10:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db and obj: Allow multiple volume attachments https://review.openstack.org/193133 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Make xvp proxy into vnc group https://review.openstack.org/235398 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'vnc' options https://review.openstack.org/263763 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'vnc' conf options documentation https://review.openstack.org/266273 | 10:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 10:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 10:06 |
| *** otter768_ has quit IRC | 10:06 | |
| *** otter768 has joined #openstack-nova | 10:07 | |
| *** markvoelker has quit IRC | 10:07 | |
| *** haomaiwa_ has joined #openstack-nova | 10:07 | |
| *** otter768 has quit IRC | 10:08 | |
| *** eliqiao has joined #openstack-nova | 10:08 | |
| *** otter768 has joined #openstack-nova | 10:08 | |
| *** eliqiao has quit IRC | 10:09 | |
| *** ljxiash has quit IRC | 10:09 | |
| *** rdopiera has quit IRC | 10:09 | |
| *** zenoway has quit IRC | 10:09 | |
| *** rdopiera has joined #openstack-nova | 10:09 | |
| *** zenoway has joined #openstack-nova | 10:10 | |
| *** salv-orlando has quit IRC | 10:10 | |
| *** RA has joined #openstack-nova | 10:10 | |
| *** otter768_ has joined #openstack-nova | 10:10 | |
| *** otter768 has quit IRC | 10:10 | |
| *** eliqiao has joined #openstack-nova | 10:10 | |
| *** RA is now known as Guest69175 | 10:10 | |
| *** danpb has joined #openstack-nova | 10:11 | |
| *** otter768_ has quit IRC | 10:12 | |
| *** otter768 has joined #openstack-nova | 10:12 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 10:13 |
| *** otter768 has quit IRC | 10:14 | |
| *** garyk has joined #openstack-nova | 10:16 | |
| *** derekh has quit IRC | 10:16 | |
| *** acabot has quit IRC | 10:16 | |
| *** otter768_ has joined #openstack-nova | 10:16 | |
| *** otter768_ has quit IRC | 10:18 | |
| *** otter768 has joined #openstack-nova | 10:18 | |
| *** derekh has joined #openstack-nova | 10:19 | |
| *** josecastroleon has joined #openstack-nova | 10:19 | |
| *** haomaiwa_ has quit IRC | 10:20 | |
| *** otter768 has quit IRC | 10:20 | |
| *** otter768_ has joined #openstack-nova | 10:20 | |
| *** thorst has joined #openstack-nova | 10:21 | |
| *** otter768_ has quit IRC | 10:21 | |
| *** otter768 has joined #openstack-nova | 10:22 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Flesh out nova.conf.virt documentation https://review.openstack.org/259553 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 10:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 10:22 |
| *** acabot has joined #openstack-nova | 10:22 | |
| *** otter768_ has joined #openstack-nova | 10:24 | |
| *** Guest69175 has quit IRC | 10:25 | |
| *** otter76__ has joined #openstack-nova | 10:25 | |
| *** otter768_ has quit IRC | 10:26 | |
| *** otter768 has quit IRC | 10:26 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Cleanup NovaObjectDictCompat from BandwidthUsage https://review.openstack.org/263638 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Cleanup NovaObjectDictCompat from NumaTopology https://review.openstack.org/260016 | 10:27 |
| *** otter76__ has quit IRC | 10:27 | |
| *** otter768 has joined #openstack-nova | 10:28 | |
| *** thorst has quit IRC | 10:28 | |
| *** salv-orlando has joined #openstack-nova | 10:28 | |
| *** salv-orlando has quit IRC | 10:28 | |
| *** otter768 has quit IRC | 10:29 | |
| *** otter768 has joined #openstack-nova | 10:29 | |
| *** moshele has quit IRC | 10:30 | |
| *** aix has quit IRC | 10:32 | |
| *** otter768 has quit IRC | 10:34 | |
| *** cdent has joined #openstack-nova | 10:36 | |
| openstackgerrit | sandeep nandal proposed openstack/nova: Replaced range() and xrange() by six.moves.range() https://review.openstack.org/264652 | 10:39 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: create vdi via streaming image from glance https://review.openstack.org/266283 | 10:40 |
| *** ildikov has quit IRC | 10:43 | |
| *** ildikov has joined #openstack-nova | 10:45 | |
| *** lpetrut has quit IRC | 10:50 | |
| *** admin0 has quit IRC | 10:50 | |
| *** cdent has quit IRC | 10:51 | |
| *** baoli has joined #openstack-nova | 10:52 | |
| *** fawadkhaliq has quit IRC | 10:52 | |
| *** aix has joined #openstack-nova | 10:53 | |
| *** sudipto has joined #openstack-nova | 10:54 | |
| *** yassine has quit IRC | 10:55 | |
| *** baoli_ has quit IRC | 10:55 | |
| *** jaosorior has quit IRC | 10:55 | |
| *** jaosorior has joined #openstack-nova | 10:55 | |
| *** coolsvap is now known as coolsvap|away | 10:56 | |
| *** carey has joined #openstack-nova | 10:56 | |
| carey | exit | 10:57 |
| *** vks has quit IRC | 10:59 | |
| *** lpetrut has joined #openstack-nova | 10:59 | |
| sfinucan | Can anyone suggest reason why I'm getting a "This change depends on a change that failed to merge." error on just-rebased patches? https://review.openstack.org/#/c/257995/13 | 11:00 |
| *** zhangjn has joined #openstack-nova | 11:00 | |
| *** moshele has joined #openstack-nova | 11:02 | |
| *** shinobu-x has joined #openstack-nova | 11:04 | |
| *** dims has joined #openstack-nova | 11:05 | |
| *** ljxiash has joined #openstack-nova | 11:07 | |
| *** rk4n has joined #openstack-nova | 11:10 | |
| *** admin0 has joined #openstack-nova | 11:10 | |
| *** ljxiash_ has joined #openstack-nova | 11:11 | |
| *** ljxiash has quit IRC | 11:12 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Flesh out nova.conf.virt documentation https://review.openstack.org/259553 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 11:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 11:13 |
| *** josecastroleon has quit IRC | 11:13 | |
| *** achi has joined #openstack-nova | 11:14 | |
| *** achi is now known as Guest99557 | 11:14 | |
| *** bkumar has quit IRC | 11:14 | |
| *** Guest99557 is now known as achinm | 11:15 | |
| *** vivekd has quit IRC | 11:17 | |
| *** openstackgerrit has quit IRC | 11:17 | |
| *** openstackgerrit has joined #openstack-nova | 11:18 | |
| *** lyanchih has joined #openstack-nova | 11:18 | |
| *** ociuhandu has quit IRC | 11:19 | |
| *** admin0 has quit IRC | 11:22 | |
| *** aysyd has joined #openstack-nova | 11:25 | |
| *** thorst has joined #openstack-nova | 11:26 | |
| *** lucas-dinner is now known as lucasagomes | 11:27 | |
| *** jlanoux has quit IRC | 11:29 | |
| *** cdent has joined #openstack-nova | 11:29 | |
| *** zhangjn has quit IRC | 11:30 | |
| *** jlanoux has joined #openstack-nova | 11:30 | |
| *** irenab has quit IRC | 11:31 | |
| *** vladikr has quit IRC | 11:32 | |
| *** irenab has joined #openstack-nova | 11:32 | |
| *** moshele has quit IRC | 11:33 | |
| *** thorst has quit IRC | 11:33 | |
| *** cdent has quit IRC | 11:36 | |
| *** gszasz has joined #openstack-nova | 11:36 | |
| *** smatzek has joined #openstack-nova | 11:36 | |
| *** ParsectiX has joined #openstack-nova | 11:37 | |
| ParsectiX | Hi guys. I have made modifications in /etc/nova/nova.conf What service should I restart to reload the config ? | 11:37 |
| *** admin0 has joined #openstack-nova | 11:38 | |
| *** ildikov has quit IRC | 11:40 | |
| *** vivekd has joined #openstack-nova | 11:40 | |
| andrearosa | ParsectiX: depends on what you have changed. if you look at http://docs.openstack.org/liberty/config-reference/content/list-of-compute-config-options.html you have an explanation of the different keys and you can infer what service uses a specific key. It could be just compute or all nova services. BTW for this kind of questions is better to ask in the #openstack room | 11:41 |
| *** vivekd has quit IRC | 11:41 | |
| *** vivekd has joined #openstack-nova | 11:42 | |
| ParsectiX | andrearosa: Thanks :) | 11:43 |
| *** cdent has joined #openstack-nova | 11:43 | |
| *** rotbeard has quit IRC | 11:44 | |
| *** gongysh has joined #openstack-nova | 11:44 | |
| *** lpetrut has quit IRC | 11:45 | |
| *** moshele has joined #openstack-nova | 11:46 | |
| andrearosa | ParsectiX: np | 11:46 |
| *** lpetrut has joined #openstack-nova | 11:46 | |
| *** lyanchih has quit IRC | 11:46 | |
| *** amotoki has quit IRC | 11:46 | |
| *** jlanoux_ has joined #openstack-nova | 11:47 | |
| *** jlanoux has quit IRC | 11:47 | |
| *** salv-orl_ has joined #openstack-nova | 11:49 | |
| *** jaosorior has quit IRC | 11:49 | |
| *** jaosorior has joined #openstack-nova | 11:50 | |
| *** amotoki has joined #openstack-nova | 11:50 | |
| *** rk4n has quit IRC | 11:51 | |
| sdague | sfinucan: because some of the 3rd party CI systems are doing something wrong | 11:51 |
| sfinucan | sdague: Silly third party CIs | 11:52 |
| sdague | it looks like they aren't processing the commits in order | 11:52 |
| sfinucan | sdague: But actually, even Jenkins was spewing errors | 11:52 |
| sdague | sfinucan: hmm, ok | 11:53 |
| sfinucan | sdague: I resolved it by modifying the commit a little bit, but why I had to do that I don't know :S | 11:53 |
| sfinucan | Maybe a question of #openstack-infra... | 11:53 |
| sdague | yeh, probably | 11:53 |
| sdague | there might be a new edge case with zuul | 11:53 |
| *** vivekd has quit IRC | 11:54 | |
| *** ihrachys has joined #openstack-nova | 11:54 | |
| *** zenoway has quit IRC | 11:56 | |
| * alex_xu reminders api meeting in 4 mins | 11:56 | |
| *** zenoway has joined #openstack-nova | 11:57 | |
| *** zenoway has quit IRC | 11:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/nova: Ironic: Use ironicclient native retries for connection errors https://review.openstack.org/211097 | 11:57 |
| *** zenoway has joined #openstack-nova | 11:57 | |
| *** sudipto has quit IRC | 11:57 | |
| *** rk4n has joined #openstack-nova | 11:57 | |
| *** MVenesio has joined #openstack-nova | 11:57 | |
| *** rk4n has quit IRC | 11:57 | |
| *** rk4n has joined #openstack-nova | 11:57 | |
| *** yassine has joined #openstack-nova | 11:58 | |
| *** ociuhandu has joined #openstack-nova | 11:59 | |
| * alex_xu reminders nova api meeting is running~ | 12:00 | |
| *** sfinucan_ has joined #openstack-nova | 12:01 | |
| *** josecastroleon has joined #openstack-nova | 12:01 | |
| *** amotoki has quit IRC | 12:02 | |
| *** markvoelker has joined #openstack-nova | 12:03 | |
| *** sfinucan has quit IRC | 12:04 | |
| *** vks has joined #openstack-nova | 12:05 | |
| *** markvoelker has quit IRC | 12:08 | |
| *** rk4n has quit IRC | 12:10 | |
| *** josecastroleon has quit IRC | 12:10 | |
| *** rk4n has joined #openstack-nova | 12:11 | |
| *** josecastroleon has joined #openstack-nova | 12:11 | |
| *** zhangjn has joined #openstack-nova | 12:14 | |
| *** qeelee has joined #openstack-nova | 12:14 | |
| *** qeelee has quit IRC | 12:15 | |
| *** tdurakov has joined #openstack-nova | 12:16 | |
| *** qeelee has joined #openstack-nova | 12:16 | |
| *** irenab has quit IRC | 12:17 | |
| *** qeelee has quit IRC | 12:18 | |
| *** qeelee has joined #openstack-nova | 12:18 | |
| *** qeelee has left #openstack-nova | 12:20 | |
| *** qeelee has joined #openstack-nova | 12:20 | |
| *** qeelee has left #openstack-nova | 12:21 | |
| *** PaulMurray has joined #openstack-nova | 12:24 | |
| *** qeelee has joined #openstack-nova | 12:24 | |
| *** eliqiao has quit IRC | 12:24 | |
| *** eliqiao has joined #openstack-nova | 12:25 | |
| *** lpetrut has quit IRC | 12:28 | |
| *** PaulMurray has quit IRC | 12:29 | |
| *** aix has quit IRC | 12:29 | |
| *** irenab has joined #openstack-nova | 12:30 | |
| *** otter768 has joined #openstack-nova | 12:30 | |
| qeelee | Hi there | 12:32 |
| *** aix has joined #openstack-nova | 12:32 | |
| *** ildikov has joined #openstack-nova | 12:32 | |
| *** cdent has quit IRC | 12:34 | |
| *** lbeliveau has quit IRC | 12:34 | |
| *** smoriya_ has quit IRC | 12:35 | |
| *** otter768 has quit IRC | 12:35 | |
| *** lbeliveau has joined #openstack-nova | 12:35 | |
| *** pratikmallya has joined #openstack-nova | 12:37 | |
| ankit_ag | Hi all, can someone please help reviewing https://review.openstack.org/185549 | 12:38 |
| *** rk4n has quit IRC | 12:39 | |
| qeelee | I saw an issue in nova compute. Basically nova-compute failed to start due to an exception raised when disconnecting volume. I tried to fix it in nova-compute manager but others have different view. Could anyone take a look? https://review.openstack.org/#/c/265681/ | 12:39 |
| *** thorst has joined #openstack-nova | 12:39 | |
| *** salv-orl_ has quit IRC | 12:43 | |
| *** rk4n has joined #openstack-nova | 12:44 | |
| *** EinstCrazy has joined #openstack-nova | 12:45 | |
| *** PaulMurray has joined #openstack-nova | 12:45 | |
| *** zhangjn has quit IRC | 12:49 | |
| *** kylek3h has joined #openstack-nova | 12:50 | |
| *** shinobu-x has quit IRC | 12:52 | |
| *** twinklechawla has joined #openstack-nova | 12:52 | |
| *** qianli has quit IRC | 12:53 | |
| *** qianli has joined #openstack-nova | 12:53 | |
| *** haomaiwang has joined #openstack-nova | 12:56 | |
| *** PaulMurray has quit IRC | 12:57 | |
| *** haomaiwang has quit IRC | 13:01 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: HyperV: Add serial console handler class https://review.openstack.org/184039 | 13:02 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: HyperV: Add SerialConsoleOps class https://review.openstack.org/184040 | 13:03 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 13:04 |
| *** markvoelker has joined #openstack-nova | 13:04 | |
| *** shakamunyi has joined #openstack-nova | 13:04 | |
| *** rfolco has joined #openstack-nova | 13:06 | |
| *** haomaiwang has joined #openstack-nova | 13:06 | |
| alex_xu | mriedem_away: alaski appreciate if you can help on this fixing for spec https://review.openstack.org/255122, it need one more +2. thanks in advance! | 13:06 |
| *** baoli has quit IRC | 13:08 | |
| *** ducttape_ has quit IRC | 13:08 | |
| *** baoli has joined #openstack-nova | 13:09 | |
| *** ducttape_ has joined #openstack-nova | 13:09 | |
| *** haomaiwang has quit IRC | 13:11 | |
| *** derekh has quit IRC | 13:11 | |
| *** zakora has quit IRC | 13:11 | |
| *** markvoelker has quit IRC | 13:15 | |
| *** mriedem_away has quit IRC | 13:15 | |
| *** derekh has joined #openstack-nova | 13:16 | |
| alaski | alex_xu: got it | 13:16 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Trace original exception before converting exception https://review.openstack.org/265239 | 13:16 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: objects: added 'os_secure_boot' property to ImageMetaProps object https://review.openstack.org/237593 | 13:18 |
| *** zhihaisong has joined #openstack-nova | 13:19 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'aggregate' https://review.openstack.org/262248 | 13:19 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 13:20 |
| *** zakora has joined #openstack-nova | 13:21 | |
| *** zakora has quit IRC | 13:21 | |
| *** zakora has joined #openstack-nova | 13:22 | |
| *** zhihais has quit IRC | 13:22 | |
| *** haomaiwang has joined #openstack-nova | 13:23 | |
| gerhardqux | any core dev around who can push https://review.openstack.org/#/c/198054 ? | 13:23 |
| *** oomichi is now known as oomichi_away | 13:23 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 13:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 13:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.images' conf options documentation https://review.openstack.org/259553 | 13:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 13:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 13:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 13:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.driver' conf options documentation https://review.openstack.org/266364 | 13:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.firewall' conf options documentation https://review.openstack.org/266365 | 13:25 |
| *** rqw has joined #openstack-nova | 13:25 | |
| tobasco | alaski: please help with follow up on this bug https://bugs.launchpad.net/nova/+bug/1469179 | 13:25 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to Feodor Tersin (ftersin) | 13:25 |
| *** twinklechawla has left #openstack-nova | 13:26 | |
| *** Marga_ has joined #openstack-nova | 13:26 | |
| rqw | danpb: could you help review the patch series for uefi boot spec https://review.openstack.org/#/c/262929/ ? | 13:27 |
| danpb | ok | 13:27 |
| *** diogogmt has quit IRC | 13:27 | |
| *** diogogmt has joined #openstack-nova | 13:27 | |
| *** ducttape_ has quit IRC | 13:27 | |
| *** haomaiwang has quit IRC | 13:28 | |
| *** fawadkhaliq has joined #openstack-nova | 13:28 | |
| *** erlon has joined #openstack-nova | 13:30 | |
| *** dtantsur has joined #openstack-nova | 13:30 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.firewall' conf options documentation https://review.openstack.org/266365 | 13:31 |
| *** rqw has quit IRC | 13:31 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 13:31 |
| *** rqw has joined #openstack-nova | 13:31 | |
| artom | danpb, thanks for the comments on the patches. For the record, I haven't been selectively ignoring some of your feedback, for example about XPath - what happens is I haven't figured out a away to submit only 1 from a series of multiple dependant patches. Ie, in my local repo I do git rebase -i and I have all 5 patches outstanding. | 13:31 |
| *** edmondsw has joined #openstack-nova | 13:32 | |
| *** bkumar has joined #openstack-nova | 13:32 | |
| artom | So every time I submit for review, all 5 get submitted. So for example I'll change the DB patch, and use the rebase opportunity to I address the quick fixes in the others. | 13:33 |
| artom | Putting off the more time consuming for later. | 13:33 |
| *** fawadkhaliq has quit IRC | 13:34 | |
| *** bkumar has quit IRC | 13:34 | |
| *** bkumar has joined #openstack-nova | 13:35 | |
| *** markvoelker has joined #openstack-nova | 13:35 | |
| alaski | tobasco: I'll take a look | 13:36 |
| *** ankit_ag has quit IRC | 13:38 | |
| *** liverpooler has quit IRC | 13:38 | |
| danpb | artom: yes, that's the way gerrit works - i doesn't really make sense to only submit the first patch in a series, as that invalidates the later patches | 13:38 |
| *** eliqiao_ has joined #openstack-nova | 13:39 | |
| *** zhangjn has joined #openstack-nova | 13:39 | |
| danpb | so its generally best to only re-submit the series once you've addressed all comments on all patches | 13:39 |
| *** zhangjn has quit IRC | 13:39 | |
| danpb | or at least add WIP and comment to effect that you've not addressed later patches yet | 13:39 |
| *** ccesario has joined #openstack-nova | 13:39 | |
| *** zhangjn has joined #openstack-nova | 13:39 | |
| *** cdent has joined #openstack-nova | 13:39 | |
| danpb | as its impossible for reviewer to know if you're still working on the later patches or not - we just see the refresh & assume comments will have been addressed | 13:40 |
| *** pkholkin has quit IRC | 13:40 | |
| *** eliqiao has quit IRC | 13:41 | |
| *** dtantsur has left #openstack-nova | 13:42 | |
| artom | danpb, yeah, should have made my intentions clearer in the reviews. | 13:42 |
| *** dslev has joined #openstack-nova | 13:42 | |
| artom | I'm aiming for quick turnaround :) | 13:42 |
| artom | Also, can we discuss the MAC address not being set on LXC vifs? | 13:42 |
| *** eliqiao_ is now known as eliqiao | 13:42 | |
| danpb | what do you mean by that ? all vifs should have a mac address | 13:43 |
| *** diogogmt has quit IRC | 13:43 | |
| danpb | afaik there's no hypervisor specific code related to VIFS | 13:43 |
| artom | Ah, no, you're saying self.address is unset. | 13:43 |
| danpb | that's device address - ie PCI | 13:43 |
| * artom should have gotten coffee first. | 13:43 | |
| danpb | container based virt doesn't provide device addresses | 13:44 |
| danpb | since it isn't emulated hardware buses | 13:44 |
| artom | Yeah, I'm getting that now. | 13:44 |
| artom | Not so much when I first looked at your comment :) | 13:44 |
| *** admin0 has quit IRC | 13:45 | |
| garyk | bauzas: please look at https://review.openstack.org/265645 | 13:45 |
| *** cdent has quit IRC | 13:46 | |
| *** browne has joined #openstack-nova | 13:47 | |
| bauzas | garyk: ack | 13:47 |
| garyk | thanks | 13:47 |
| *** shaohe_feng has joined #openstack-nova | 13:49 | |
| bauzas | garyk: good fix, ty | 13:49 |
| openstackgerrit | Daniel Berrange proposed openstack/os-vif: Import of code from https://github.com/jaypipes/os_vif https://review.openstack.org/266377 | 13:49 |
| *** bkumar has quit IRC | 13:49 | |
| *** cdent has joined #openstack-nova | 13:51 | |
| *** rqw has quit IRC | 13:51 | |
| *** rqw has joined #openstack-nova | 13:51 | |
| *** mhorban_ has joined #openstack-nova | 13:52 | |
| garyk | bauzas: gracias | 13:52 |
| *** browne has quit IRC | 13:52 | |
| jaypipes | danpb: +1 from me on that. (I don't have +2 ability..) | 13:53 |
| danpb | jaypipes: oh dear, you should have, so that's something i screwed up in setup :-) | 13:54 |
| *** PaulMurray has joined #openstack-nova | 13:54 | |
| jaypipes | danpb: :) | 13:54 |
| jaypipes | danpb: jaypipes@gmail.com account, not the jpipes@mirantis.com one, which is for testing CI | 13:54 |
| tdurakov | PaulMurray, hi, are you around? | 13:55 |
| *** bkumar has joined #openstack-nova | 13:55 | |
| PaulMurray | tdurakov, yep | 13:55 |
| *** huats has quit IRC | 13:55 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add "hw_firmware_type" image metadata https://review.openstack.org/262929 | 13:56 |
| *** huats has joined #openstack-nova | 13:56 | |
| PaulMurray | live migration meeting in 5 minutes on #openstack-meeting-3 paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub | 13:56 |
| tdurakov | PaulMurray, wanted to ask about meeting) | 13:56 |
| *** admin0 has joined #openstack-nova | 13:57 | |
| danpb | jaypipes: lol, I don't have +2 either, so i really screwed up | 13:57 |
| jaypipes | haha :) | 13:57 |
| * danpb goes to read the infra docs | 13:57 | |
| *** qeelee has quit IRC | 13:57 | |
| *** haomaiwa_ has joined #openstack-nova | 13:59 | |
| *** aysyd has quit IRC | 14:00 | |
| *** aysyd has joined #openstack-nova | 14:01 | |
| *** femnad has joined #openstack-nova | 14:01 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** carey has quit IRC | 14:01 | |
| *** akshai has joined #openstack-nova | 14:02 | |
| *** burgerk has joined #openstack-nova | 14:02 | |
| *** dslev has quit IRC | 14:02 | |
| *** rqw has quit IRC | 14:02 | |
| *** rqw has joined #openstack-nova | 14:03 | |
| ccarmack | alex_xu: are you there? | 14:04 |
| *** bkumar has quit IRC | 14:05 | |
| *** bkumar has joined #openstack-nova | 14:06 | |
| *** mtanino has joined #openstack-nova | 14:07 | |
| *** adreznec_ is now known as adreznec | 14:09 | |
| *** bkumar has quit IRC | 14:10 | |
| *** rqw has quit IRC | 14:10 | |
| *** ctrath has joined #openstack-nova | 14:10 | |
| *** rqw has joined #openstack-nova | 14:10 | |
| *** salv-orlando has joined #openstack-nova | 14:11 | |
| *** xyang1 has joined #openstack-nova | 14:11 | |
| *** zakora has quit IRC | 14:11 | |
| *** krtaylor has quit IRC | 14:13 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor provider_fw_rule_get_all() https://review.openstack.org/216241 | 14:13 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused provider firewall rules functionality in nova https://review.openstack.org/184027 | 14:13 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove provider_fw_rules tables and associated db api methods https://review.openstack.org/216242 | 14:13 |
| *** rlrossit has joined #openstack-nova | 14:14 | |
| *** esracelik has quit IRC | 14:17 | |
| *** mdrabe has joined #openstack-nova | 14:18 | |
| *** tongli has joined #openstack-nova | 14:18 | |
| *** ZZelle has joined #openstack-nova | 14:18 | |
| *** w_verdugo has joined #openstack-nova | 14:19 | |
| *** jichen has quit IRC | 14:20 | |
| *** amotoki has joined #openstack-nova | 14:20 | |
| *** eharney has joined #openstack-nova | 14:21 | |
| *** amotoki has quit IRC | 14:21 | |
| *** PaulMurray has quit IRC | 14:23 | |
| *** zakora has joined #openstack-nova | 14:23 | |
| *** zz_dimtruck is now known as dimtruck | 14:23 | |
| *** krtaylor has joined #openstack-nova | 14:24 | |
| *** vladikr has joined #openstack-nova | 14:24 | |
| *** mriedem has joined #openstack-nova | 14:25 | |
| *** rqw has quit IRC | 14:26 | |
| *** mtanino has quit IRC | 14:26 | |
| *** rqw has joined #openstack-nova | 14:27 | |
| *** otter768 has joined #openstack-nova | 14:31 | |
| *** achinm has quit IRC | 14:32 | |
| *** aix has quit IRC | 14:33 | |
| *** zakora has quit IRC | 14:33 | |
| *** aix has joined #openstack-nova | 14:33 | |
| *** dimtruck is now known as zz_dimtruck | 14:34 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: xenapi: simplify swap_xapi_host() https://review.openstack.org/266258 | 14:34 |
| *** rqw has quit IRC | 14:35 | |
| *** busterswt has joined #openstack-nova | 14:35 | |
| *** otter768 has quit IRC | 14:36 | |
| *** annegentle has joined #openstack-nova | 14:37 | |
| *** zakora has joined #openstack-nova | 14:37 | |
| *** cfriesen has joined #openstack-nova | 14:38 | |
| *** dslev has joined #openstack-nova | 14:39 | |
| *** amotoki has joined #openstack-nova | 14:40 | |
| *** jaosorior has quit IRC | 14:40 | |
| mriedem | mdbooth: you're going to have to redo https://review.openstack.org/#/c/264815/ | 14:41 |
| mriedem | mdbooth: https://review.openstack.org/#/c/243028/ merged after you proposed that | 14:41 |
| *** salv-orlando has quit IRC | 14:42 | |
| *** burgerk has quit IRC | 14:43 | |
| tojuvone | jaypipes: I updated the policy rule for: https://review.openstack.org/241562 | 14:46 |
| *** zakora has quit IRC | 14:46 | |
| *** kbyrne has quit IRC | 14:46 | |
| jaypipes | tojuvone: coolio, lemme rereview. | 14:46 |
| garyk | mdbooth: why? | 14:47 |
| tojuvone | jaypipes: thx | 14:47 |
| *** amotoki has quit IRC | 14:47 | |
| *** dslev has quit IRC | 14:49 | |
| *** sudipto has joined #openstack-nova | 14:49 | |
| *** Marga_ has quit IRC | 14:50 | |
| *** mrkz has joined #openstack-nova | 14:50 | |
| *** links has quit IRC | 14:50 | |
| *** pratikmallya has quit IRC | 14:50 | |
| *** Marga_ has joined #openstack-nova | 14:50 | |
| *** astupnikov has joined #openstack-nova | 14:51 | |
| dansmith | alex_xu: you still around by chance? | 14:52 |
| *** omnipresent has joined #openstack-nova | 14:52 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Catch errors while creating veth pair https://review.openstack.org/264146 | 14:52 |
| ccarmack | gmann: are you there? | 14:53 |
| *** salv-orlando has joined #openstack-nova | 14:54 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/api directory https://review.openstack.org/258180 | 14:54 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (2) https://review.openstack.org/260587 | 14:54 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (1) https://review.openstack.org/260586 | 14:54 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (3) https://review.openstack.org/260588 | 14:54 |
| *** burt has joined #openstack-nova | 14:54 | |
| *** zenoway has quit IRC | 14:54 | |
| openstackgerrit | Balazs Gibizer proposed openstack/python-novaclient: [microversions] Add support for 2.15 https://review.openstack.org/266413 | 14:54 |
| *** danpb has left #openstack-nova | 14:54 | |
| *** liverpooler has joined #openstack-nova | 14:54 | |
| *** danpb has joined #openstack-nova | 14:54 | |
| tojuvone | dansmith: made spec change still for get-valid-server-state as discussed earlier: https://review.openstack.org/260314 | 14:55 |
| tojuvone | dansmith: Also the code is ready if jaypipes also agree: https://review.openstack.org/241562 | 14:56 |
| kashyap | belliott (Or anyone familiar with resource tracker): Hyou ever come across a scenario where the error on line-796 is reported where "if a user failed to migrate 100 times in total, Nova reports 100 messages per minute forever"? https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L786,L797 | 14:56 |
| *** salv-orlando has quit IRC | 14:56 | |
| *** salv-orlando has joined #openstack-nova | 14:56 | |
| *** annegentle has quit IRC | 14:56 | |
| *** ducttape_ has joined #openstack-nova | 14:57 | |
| *** annegentle has joined #openstack-nova | 14:57 | |
| *** dslev has joined #openstack-nova | 14:58 | |
| *** akshai has quit IRC | 14:58 | |
| *** akshai has joined #openstack-nova | 14:59 | |
| *** edtubill has joined #openstack-nova | 15:00 | |
| dansmith | tojuvone: okay, I'm in the middle of something at the moment | 15:00 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 15:00 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 15:00 |
| *** zakora has joined #openstack-nova | 15:00 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 15:00 |
| tojuvone | dansmith: np, there is time till 21st :D | 15:01 |
| eliqiao | pkoniszewski: hi pawel. | 15:01 |
| pkoniszewski | shaohe_feng: per your question "is there a config sample with optimal options config that suits to most common live migration scenario?" | 15:01 |
| *** jlanoux_ has quit IRC | 15:01 | |
| belliott | kashyap: that doesn't sound familiar | 15:01 |
| tdurakov | pkoniszewski, ping, could we disscuss your patch in an hour? | 15:01 |
| *** jlanoux has joined #openstack-nova | 15:01 | |
| pkoniszewski | with this patch series - https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:deprecate-migration-flags-config - we are trying to create such optimal configuration apart from nova.conf flags | 15:02 |
| *** houming has joined #openstack-nova | 15:02 | |
| *** markmcclain has joined #openstack-nova | 15:02 | |
| pkoniszewski | tdurakov: sure, just ping me and we will talk about it | 15:02 |
| *** doug-fish has joined #openstack-nova | 15:02 | |
| mdbooth | mriedem: Thanks, looking | 15:03 |
| *** annegentle has quit IRC | 15:03 | |
| *** markmcclain has quit IRC | 15:03 | |
| kashyap | belliott: Hmm, such a thing even expected? | 15:03 |
| *** markmcclain1 has joined #openstack-nova | 15:03 | |
| mdbooth | garyk: Did you mean to aim that why at me, btw? | 15:04 |
| kashyap | belliott: Although I see that this message occurs only applicable when DEBUG is enabled though. | 15:04 |
| *** annegentle has joined #openstack-nova | 15:04 | |
| *** mtanino has joined #openstack-nova | 15:04 | |
| *** markmcclain1 is now known as markmcclain | 15:04 | |
| belliott | kashyap: i mean there is logging when the periodic task runs, i'm not sure what specific logging you're referring to | 15:04 |
| *** edtubill has quit IRC | 15:04 | |
| kashyap | belliott: A message such as this - "Migration instance not found: Instance 223343056-bef1-40e5-9aed-5a616f9b81f8 could not be found." | 15:05 |
| *** salv-orl_ has joined #openstack-nova | 15:05 | |
| *** salv-orlando has quit IRC | 15:05 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Add 'hw:cpu_thread_policy=require' scheduling https://review.openstack.org/202651 | 15:06 |
| belliott | kashyap: ah ok, so the migration record is referencing deleted instances there | 15:06 |
| *** zakora has quit IRC | 15:06 | |
| *** lucasagomes is now known as lucas-hungry | 15:06 | |
| kashyap | belliott: Right | 15:06 |
| sfinucan_ | alaski: could I get a +1 workflow on this again? https://review.openstack.org/#/c/202651/ | 15:07 |
| kashyap | belliott: And, apparently (someone who ran tempest tests was reporting to me) that it's thrown a ton of times | 15:07 |
| sfinucan_ | Gerrit's being silly and won't apply patches again :( | 15:07 |
| belliott | kashyap: so yeah that could be chatty, i'd support a patch to make it talk less.. | 15:07 |
| sfinucan_ | ndipanov ^^ | 15:07 |
| *** burgerk has joined #openstack-nova | 15:07 | |
| belliott | kashyap: better query filtering or periodically cleanup useless migration records | 15:07 |
| kashyap | belliott: I see, I've never examined this code, do you mind pointing me where the chatty thing occurs? Just to educate myself | 15:07 |
| alaski | sfinucan_: got it | 15:07 |
| belliott | kashyap: well you linked the line it's logging on :) | 15:08 |
| sfinucan_ | alaski: cheers - appreciate it | 15:08 |
| kashyap | belliott: Heh, apart from that, I thought there's somewhere else (related to periodic tasks) that I should look at :-) | 15:08 |
| belliott | kashyap: sure, in nova/compute/manager you can find functions logged with @periodic_task and there's one called update_available_resource (i think) | 15:09 |
| * kashyap checks | 15:09 | |
| *** zakora has joined #openstack-nova | 15:09 | |
| kashyap | belliott: Yep, you're spot on | 15:10 |
| kashyap | Thanks, I'll check that out. | 15:11 |
| *** salv-orl_ has quit IRC | 15:11 | |
| *** Marga_ has quit IRC | 15:11 | |
| belliott | kashyap: enjoy | 15:11 |
| *** Marga_ has joined #openstack-nova | 15:12 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: WIP: remove EC2 legacy code https://review.openstack.org/266425 | 15:13 |
| *** wznoinsk1 has joined #openstack-nova | 15:16 | |
| mdbooth | mriedem: That extra patch actually makes the backport a ton easier. Take your comments about conflict notes, btw, and I normally do that. I was a bit pushed for time when I did that first time round, though, and I'm afraid I dropped it. | 15:16 |
| *** dave-mccowan has joined #openstack-nova | 15:17 | |
| *** wznoinsk1 has quit IRC | 15:17 | |
| mdbooth | mriedem: Just going to run local tests and I'll push the new backports, thanks. | 15:17 |
| *** houming has quit IRC | 15:17 | |
| mriedem | mdbooth: sounds good, thanks | 15:17 |
| *** dave-mcc_ has joined #openstack-nova | 15:19 | |
| *** pratikmallya has joined #openstack-nova | 15:19 | |
| *** tjones has joined #openstack-nova | 15:21 | |
| *** akshai has quit IRC | 15:21 | |
| *** dave-mccowan has quit IRC | 15:22 | |
| tjones | markus_z: hi i have a few config patches in rebase hell. is there a good way to unblock them? | 15:22 |
| markus_z | tjones: let me have a look | 15:23 |
| *** gszasz has quit IRC | 15:23 | |
| *** akshai has joined #openstack-nova | 15:23 | |
| openstackgerrit | dane-fichter proposed openstack/nova: Add image signature verification https://review.openstack.org/189843 | 15:23 |
| tjones | i have a few i need to rebase and fix too | 15:23 |
| *** mdrabe has quit IRC | 15:23 | |
| tjones | so don't bother with the -1 :-) | 15:23 |
| markus_z | tjones: yesterday merged an item which makes a change necessary http://lists.openstack.org/pipermail/openstack-dev/2016-January/083598.html | 15:23 |
| tjones | ah ok will do | 15:24 |
| *** haomaiwang has quit IRC | 15:24 | |
| tjones | was that the right message? it's about the meeting time | 15:24 |
| tjones | nevermind i see it | 15:25 |
| *** gszasz has joined #openstack-nova | 15:25 | |
| markus_z | tjones: ah, yeah, the right one is the log at around 15:24 http://eavesdrop.openstack.org/meetings/nova_config_options/2016/nova_config_options.2016-01-11-15.01.log.html | 15:25 |
| *** Marga_ has quit IRC | 15:28 | |
| *** ljxiash has joined #openstack-nova | 15:29 | |
| *** kbyrne has joined #openstack-nova | 15:29 | |
| *** kbyrne has quit IRC | 15:29 | |
| *** ljxiash_ has quit IRC | 15:29 | |
| *** kbyrne has joined #openstack-nova | 15:30 | |
| ccesario | hello, is there any restriction about usage Xenserver with OpenStack ? | 15:30 |
| kashyap | - "So the migration record is referencing deleted instances there. | 15:31 |
| kashyap | So yeah that could be chatty, i'd support a patch to make it talk | 15:31 |
| kashyap | less. Better query filtering or periodically cleanup useless | 15:31 |
| kashyap | migration records." | 15:31 |
| kashyap | Urgh | 15:31 |
| kashyap | Wrong copy/paste, I was just taking a note locally | 15:31 |
| *** gszasz has quit IRC | 15:31 | |
| BobBall | ccesario: Use XenServer 6.5 with EXT3 storage ("Optimised for XenDesktop") and install XS65SP1E013 - then you're good to go :) | 15:31 |
| *** breitz has quit IRC | 15:32 | |
| *** breitz has joined #openstack-nova | 15:32 | |
| *** zz_dimtruck is now known as dimtruck | 15:33 | |
| *** gszasz has joined #openstack-nova | 15:33 | |
| bauzas | alaski: good point on https://review.openstack.org/#/c/254434/7/nova/compute/api.py with the RBAC check for v2 | 15:35 |
| ccesario | BobBall, thanks for info !!! | 15:35 |
| bauzas | alaski: that said, I wonder if we could provide the check by the v2 API module like I said | 15:36 |
| bauzas | alex_xu: do you know if possible ? | 15:36 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: :config options: centralize 'spice' options https://review.openstack.org/264271 | 15:36 |
| ccesario | is there any doc comparing openstack with kvm and xenserver? | 15:36 |
| *** gszasz has quit IRC | 15:36 | |
| *** kfarr has joined #openstack-nova | 15:36 | |
| BobBall | ccesario: Do you have specific requirements you're looking at? | 15:36 |
| *** gszasz has joined #openstack-nova | 15:36 | |
| ccesario | BobBall, wich is better ? :) | 15:37 |
| *** mragupat has joined #openstack-nova | 15:37 | |
| alaski | bauzas: we certainly could move it there. feedback from the API folks would be good since none of the other checks were moved like that. | 15:37 |
| bauzas | okay, I just want to make sure it's acceptable or not | 15:38 |
| bauzas | sdague ? | 15:38 |
| sdague | bauzas: what's the question | 15:38 |
| bauzas | sdague: see my point on https://review.openstack.org/#/c/254434/7/nova/compute/api.py sorry | 15:38 |
| bauzas | we already do have a policy check in the v2.1 module but alaski has to provide the same check by the compute API since v2 doesn't do it | 15:38 |
| sdague | bauzas: there are many comments there | 15:38 |
| bauzas | so my question is : why should we provide the policy check by the compute API module while we could add it to the v2 tree ? | 15:39 |
| bauzas | sdague: https://review.openstack.org/#/c/254434/7/nova/compute/api.py L1413 my bad | 15:39 |
| BobBall | ccesario: XenServer is an open source distribution of a hypervisor (xen) with commercial support options from Citrix; KVM is a hypervisor. You'd need to compare XenServer to a KVM distribution such as RHEV to get a meaningful question - but as always, "better" is a subjective term and depends hugely on so many factors that the answer may be different for different people. I think XenServer is better, but I know more about Xen | 15:40 |
| markus_z | tjones: I got distracted, do you need further help? | 15:42 |
| *** mdrabe has joined #openstack-nova | 15:42 | |
| tjones | im not sure - let me pull down my patch, rebase and see | 15:42 |
| tdurakov | pkoniszewski, hey | 15:42 |
| tjones | i think i understand what to do. | 15:42 |
| sdague | bauzas: that's correct form for doing that in our current code | 15:42 |
| markus_z | ok, cool, I'm still here for at least 1 hour | 15:42 |
| tjones | ok im on a plane so my network is slllooooowwww | 15:43 |
| bauzas | sdague: you mean ? it's correct to provide it in the compute API module even it's also done by the v2.1 tree ? fair enough | 15:43 |
| sdague | the policy checks were left in the compute.api for v2. One could argue this is kind of gorpy. | 15:43 |
| sdague | but it's the way we are doing it across the board | 15:43 |
| bauzas | for my knowledge, what's the most acceptable ? do the policy check by the API tree or by the compute API module ? | 15:44 |
| alaski | bauzas: note that it's just being moved from line 1087 in that file | 15:44 |
| bauzas | alaski: yup | 15:44 |
| bauzas | alaski: I saw that | 15:44 |
| sdague | the legacy v2 stuff is in the compute api, behind that skip_policy_check flag | 15:44 |
| bauzas | okay | 15:45 |
| bauzas | thanks for helping me understand | 15:45 |
| sdague | yeh, no prob | 15:45 |
| bauzas | alaski: nevermind my point then, to be clear :) | 15:45 |
| alaski | cool, no worries | 15:45 |
| ccesario | BobBall, I agree with you... I will check more about it..... | 15:46 |
| *** tjones has quit IRC | 15:47 | |
| ccesario | BobBall, is there any openstack feature "limitation" when using Xenserver ? Or all features provide by Openstack works as expected with Xenserver? | 15:47 |
| *** kfarr has quit IRC | 15:47 | |
| *** kfarr has joined #openstack-nova | 15:50 | |
| *** kbyrne has quit IRC | 15:50 | |
| *** rk4n has quit IRC | 15:51 | |
| BobBall | ccesario: Check out http://docs.openstack.org/developer/nova/support-matrix.html | 15:51 |
| *** admin0 has quit IRC | 15:52 | |
| ccesario | thanks :) | 15:52 |
| *** pratikmallya has quit IRC | 15:53 | |
| *** n0ano has joined #openstack-nova | 15:55 | |
| *** carey has joined #openstack-nova | 15:57 | |
| *** pkholkin has joined #openstack-nova | 15:57 | |
| *** houming has joined #openstack-nova | 15:58 | |
| *** fawadkhaliq has joined #openstack-nova | 15:58 | |
| bauzas | alaski: maybe you could review https://review.openstack.org/#/c/264349/ I need another good reviewer for the compute manager | 15:59 |
| bauzas | I could possibly have missed some point | 15:59 |
| alaski | sure | 16:00 |
| *** liverpooler has quit IRC | 16:00 | |
| openstackgerrit | dane-fichter proposed openstack/nova: Add image signature verification https://review.openstack.org/189843 | 16:01 |
| *** carey has quit IRC | 16:02 | |
| *** vilobhmm11 has joined #openstack-nova | 16:02 | |
| *** astupnikov has quit IRC | 16:02 | |
| pkoniszewski | tdurakov: sorry, i'm back | 16:02 |
| *** krtaylor has quit IRC | 16:03 | |
| *** thomasem-away is now known as thomasem | 16:03 | |
| *** garyk has quit IRC | 16:04 | |
| *** lucas-hungry is now known as lucasagomes | 16:05 | |
| *** gokrokve has joined #openstack-nova | 16:05 | |
| *** suro-patz has joined #openstack-nova | 16:06 | |
| *** aix has quit IRC | 16:07 | |
| *** mjura has quit IRC | 16:07 | |
| *** rk4n has joined #openstack-nova | 16:07 | |
| *** rk4n has quit IRC | 16:07 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Move wsgi options into a group https://review.openstack.org/263812 | 16:08 |
| *** rk4n has joined #openstack-nova | 16:08 | |
| *** kbyrne has joined #openstack-nova | 16:09 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: WIP: bug process doc https://review.openstack.org/266453 | 16:10 |
| *** dane-fichter has joined #openstack-nova | 16:11 | |
| markus_z | lxsli: ^ I added you as reviewer, we talked about that some weeks ago. Would be good to have your feedback | 16:12 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'vnc' conf options documentation https://review.openstack.org/266273 | 16:12 |
| *** zhangjn has quit IRC | 16:13 | |
| artom | mriedem, am I mistaken or do current unit tests actually *enforce* the presence of a shadow table for all tables unless explicitly specified as an exception? This is in response to your -1 on the vNIC tagging patch. | 16:14 |
| *** krtaylor has joined #openstack-nova | 16:14 | |
| artom | And that makes soft-delete the default, of sorts. | 16:14 |
| *** fawadkhaliq has quit IRC | 16:14 | |
| mriedem | artom: i don't think so | 16:14 |
| mriedem | shadow tables are only required if you have the soft-delete mixin, which right now only the tags table doesn't in the nova schema | 16:15 |
| artom | mriedem, https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L8258 | 16:15 |
| mriedem | i saw ndipanov's comment on lacking a test for enforcing that | 16:15 |
| ndipanov | mriedem, I don | 16:15 |
| ndipanov | t really care | 16:16 |
| mriedem | artom: ok, so you answered your own question | 16:16 |
| ndipanov | actually I think he should drop it | 16:16 |
| artom | mriedem, did I? :P To me that test checks for shadow_* | 16:16 |
| *** matrohon has quit IRC | 16:16 | |
| artom | Where * is everything | 16:16 |
| mriedem | artom: yeah except for the exceptions | 16:16 |
| mriedem | like tags | 16:16 |
| ndipanov | but I've been for several cycles on the other side of the FF game | 16:16 |
| ndipanov | and I can tell you tahat comments like that one week before the FF are very discouraging | 16:17 |
| ndipanov | there was no way he could have known that | 16:17 |
| ndipanov | actually I completely forgot about it | 16:17 |
| mriedem | the soft delete thing? | 16:17 |
| artom | mriedem, https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L8279 there are currently no exceptions based on that. The tags stuff is taken care of in an if earlier | 16:17 |
| artom | I'll add it manually I guess. | 16:18 |
| artom | Like tags. | 16:18 |
| mriedem | ndipanov: re: the soft-delete thing, i have a ML thread on that because i wanted to get something documented in the devref on that policy, | 16:18 |
| dansmith | ndipanov: but the patches only showed up two weeks before FF, so when were we supposed to provide those comments? | 16:18 |
| artom | But that should really be changed in the future :) The tests, by default, are enforcing something that's no longer wantde. | 16:18 |
| mriedem | but the ML thread piddles out because i couldn't get a conclusion | 16:18 |
| artom | dansmith, well, they've been up as a monolithic WIP for a while. | 16:19 |
| mriedem | artom: it's fine to enforce that shadow tables are emtpy and working on tables that existed before the stance on no more soft delete | 16:19 |
| ndipanov | I am personally scarred by this type of behavior so feel free to diregard me as grumpy | 16:19 |
| artom | dansmith, my bad for leaving them as WIP I guess. | 16:19 |
| mriedem | monolithic WIP for an unapproved bp doesn't get much review exposure | 16:19 |
| ndipanov | fwiw - I consider nova FF and reviewing around it hostile and antisocial | 16:20 |
| ndipanov | i'll just step away from that review | 16:20 |
| *** gszasz has quit IRC | 16:20 | |
| *** diogogmt has joined #openstack-nova | 16:20 | |
| *** pratikmallya has joined #openstack-nova | 16:21 | |
| *** liusheng has quit IRC | 16:21 | |
| mriedem | ndipanov: this is the ML thread where i was trying to figure out how removing soft delete was going to happen and why http://lists.openstack.org/pipermail/openstack-dev/2015-November/080397.html | 16:22 |
| mriedem | which never really concluded | 16:22 |
| *** pratikma_ has joined #openstack-nova | 16:22 | |
| artom | mriedem, yeah I get that now. Essentially this whole process boils down to submit the spec and the full patch at the same time, otherwise you're boned because you'll run out of time before FF. | 16:23 |
| dansmith | artom: you're blowing this out of proportion.. making the change to use the *existing* stuff would take less time than arguing | 16:23 |
| *** hemnafk is now known as hemna | 16:24 | |
| *** liusheng has joined #openstack-nova | 16:24 | |
| artom | dansmith, clearly :) I take it my last comment didn't convince you? Same for mriedem? | 16:24 |
| ndipanov | mriedem, so if we just not inherit from soft delete mixin we're good? | 16:24 |
| ndipanov | (upgrade and future removal in the API aside) | 16:25 |
| bauzas | artom: I just feel that dansmith's point is that modifying the change related to the comment is trivial | 16:25 |
| mriedem | ndipanov: well, that was an easy thing so we don't accidentally add new models that inherit those columns by mistake | 16:25 |
| bauzas | I mean, do the change | 16:25 |
| *** pratikmallya has quit IRC | 16:25 | |
| *** BobBall has quit IRC | 16:25 | |
| mriedem | ndipanov: the question in the thread was more about how do actually drop it, since you can't easily do that w/o breaking backward compat in the APIs that read deleted resources, like flavors and instances | 16:25 |
| dansmith | artom: I still see no reason to add another model that overlaps an existing one of nearly the same name by all but one attribute | 16:26 |
| *** bwallis has quit IRC | 16:26 | |
| mriedem | ndipanov: maybe the how doesn't matter at this point | 16:26 |
| mriedem | i just want to get something in the devref that says, as a policy rule, no more soft delete for new models | 16:26 |
| *** gongysh has quit IRC | 16:26 | |
| ndipanov | mriedem, we could easily add a test for that I think similar to the tests for models mathcing migrations no? | 16:27 |
| *** diogogmt has quit IRC | 16:27 | |
| *** EinstCrazy has quit IRC | 16:28 | |
| mriedem | yeah probably, which would be better than devref | 16:28 |
| artom | mriedem, I would at least modify the test_shadow_tables test to not automatically assume that every table has a shadow. I realise people can add exceptions for their new tables, but it's just to make tests and policy consistent. | 16:28 |
| ndipanov | ndipanov, that's what I was referring to in that comment | 16:29 |
| mriedem | ndipanov: yeah i know | 16:29 |
| *** diogogmt has joined #openstack-nova | 16:29 | |
| mriedem | i think that's a good idea | 16:29 |
| *** eharney has quit IRC | 16:29 | |
| cdent | bauzas, markus_z, n0ano: I've left some comments on this bug to try and get it actionable or dead, please help move it along: https://bugs.launchpad.net/nova/+bug/1427772 | 16:29 |
| openstack | Launchpad bug 1427772 in OpenStack Compute (nova) "Instance that uses force-host still needs to run some filters" [Low,Confirmed] - Assigned to Anant Kaushik (anantkaushik-nsit) | 16:29 |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| openstackgerrit | Balazs Gibizer proposed openstack/python-novaclient: [microversions] Add support for 2.15 https://review.openstack.org/266413 | 16:30 |
| mriedem | dansmith: want to hit this cve backport again? https://review.openstack.org/#/c/264815/ | 16:31 |
| *** josecastroleon has quit IRC | 16:31 | |
| dansmith | mriedem: yep, got it | 16:31 |
| *** cdent has quit IRC | 16:32 | |
| *** sudipto has quit IRC | 16:32 | |
| *** otter768 has joined #openstack-nova | 16:32 | |
| *** mspreitz has joined #openstack-nova | 16:32 | |
| mriedem | i'm also +2 on the 2nd of 3 now https://review.openstack.org/#/c/264816/ | 16:32 |
| markus_z | cdent: I will leave in 30 minutes. Will take a look at it tomorrow. | 16:35 |
| *** otter768 has quit IRC | 16:37 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 16:37 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 16:37 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db and obj: Allow multiple volume attachments https://review.openstack.org/193133 | 16:37 |
| *** thangp has joined #openstack-nova | 16:37 | |
| *** rdopiera has quit IRC | 16:37 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 16:37 |
| *** jamielennox is now known as jamielennox|away | 16:37 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 16:37 |
| *** kfarr_ has joined #openstack-nova | 16:38 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.images' conf options documentation https://review.openstack.org/259553 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.driver' conf options documentation https://review.openstack.org/266364 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt.firewall' conf options documentation https://review.openstack.org/266365 | 16:39 |
| *** ljxiash has quit IRC | 16:39 | |
| *** kfarr has quit IRC | 16:39 | |
| artom | dansmith, last question about vnic DB stuff - the spec calls for multiple tags (eventually). With a vnic_tags table there can be one entry per tag (with duplicate every except for the actual tag column). If I use the existing virtual_interfaces, what would be the correct way to store multiple tags? A JSON array as a text blob? | 16:41 |
| *** pradk has joined #openstack-nova | 16:42 | |
| dansmith | artom: how are you going to do that for bdm? | 16:42 |
| dansmith | artom: and what's the point of multiple tags? | 16:42 |
| * dansmith looks at the spec | 16:42 | |
| dansmith | what line does it say that on? I can't find it | 16:43 |
| artom | dansmith, I'm actually not doing in in BDM - I just have a single new 'tag' column. | 16:43 |
| *** moshele has quit IRC | 16:43 | |
| danpb | i don't think we need to care about multiple tags | 16:43 |
| dansmith | danpb: agreed | 16:43 |
| artom | Err... | 16:43 |
| artom | "Although the syntax supports setting of multiple tags per device, initially the impl will only allow a single tag. The syntax just allows for future extension should there be a need." | 16:43 |
| danpb | we allowed for multiple tags in the metadata schema we expose to the guest os | 16:43 |
| danpb | just as a way to future proof the guest JSON format | 16:44 |
| dansmith | ah, okay.. well, cool for future proofing, but let's not do that now | 16:44 |
| dansmith | yeah | 16:44 |
| danpb | but i think its fine if our impl only stores a single tag in the db | 16:44 |
| artom | Ah... So there'd be another DB migration in the future if that ever becomes implemented? | 16:44 |
| *** rk4n has quit IRC | 16:44 | |
| *** mgoddard__ has joined #openstack-nova | 16:44 | |
| danpb | likewise nova public API only need accept a single tag | 16:44 |
| danpb | artom: if that ever happens yes, but i don't really see it happening | 16:45 |
| dansmith | me either | 16:45 |
| dansmith | especially given the api impact | 16:45 |
| artom | Hrmm, guess I misunderstood the intent of that then. | 16:45 |
| artom | Anyways, makes my work easier. | 16:45 |
| artom | So good times | 16:45 |
| *** derekh has quit IRC | 16:46 | |
| *** browne has joined #openstack-nova | 16:47 | |
| *** mgoddard_ has quit IRC | 16:47 | |
| *** rk4n has joined #openstack-nova | 16:47 | |
| *** rk4n has quit IRC | 16:47 | |
| *** rk4n_ has joined #openstack-nova | 16:47 | |
| *** bwallis_ has joined #openstack-nova | 16:48 | |
| *** fawadkhaliq has joined #openstack-nova | 16:49 | |
| *** cdent has joined #openstack-nova | 16:49 | |
| *** hemna is now known as hemnafk | 16:50 | |
| *** paul-carlton2 has quit IRC | 16:50 | |
| *** tonytan4ever has joined #openstack-nova | 16:54 | |
| *** READ10 has joined #openstack-nova | 16:54 | |
| *** nkrinner has quit IRC | 16:55 | |
| *** bradjones has quit IRC | 16:55 | |
| artom | So that actually makes the DB patch *much* simpler. | 16:56 |
| *** tpatzig has quit IRC | 16:56 | |
| * artom is still afraid of what it'll do to subsequent patches. | 16:57 | |
| dansmith | artom: ...right | 16:57 |
| dansmith | artom: reusing existing stuff often is :) | 16:57 |
| *** suro-patz has quit IRC | 16:57 | |
| ansiwen | mriedem, johnthetubaguy: may I draw your attention to a change that you looked at befor and is stuck since August: https://review.openstack.org/#/c/168024 ? Thanks! | 16:58 |
| artom | dansmith, sunken cost fallacy and all that | 16:58 |
| *** bradjones has joined #openstack-nova | 16:58 | |
| *** bradjones has quit IRC | 16:58 | |
| *** bradjones has joined #openstack-nova | 16:58 | |
| *** bwallis_ has quit IRC | 16:59 | |
| migi | may I ask for +W for https://review.openstack.org/263892 ? | 16:59 |
| *** dslev has quit IRC | 17:01 | |
| *** BobBall has joined #openstack-nova | 17:01 | |
| sdague | something looks off the rails with nova unit tests - https://jenkins05.openstack.org/job/gate-nova-python27/15232/console | 17:01 |
| *** Jeffrey4l has quit IRC | 17:02 | |
| jaypipes | tojuvone: hey, just FYI, I'm still reviewing that patch. I have discovered a few things I had questions on. | 17:02 |
| *** mgoddard__ has quit IRC | 17:02 | |
| dansmith | mmm windows | 17:02 |
| dansmith | sdague: os_win upstream release? | 17:03 |
| sdague | probing now | 17:03 |
| *** pratikma_ has quit IRC | 17:03 | |
| sdague | pypi is kind of slow for my script | 17:03 |
| mtreinish | dansmith: looks that way: https://pypi.python.org/pypi/os-win | 17:03 |
| *** zhihaisong has quit IRC | 17:03 | |
| *** salv-orlando has joined #openstack-nova | 17:03 | |
| *** claudiub has joined #openstack-nova | 17:04 | |
| sdague | yeh | 17:04 |
| jaypipes | os_winning. | 17:04 |
| *** jlanoux has quit IRC | 17:04 | |
| *** szhsong has joined #openstack-nova | 17:05 | |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22NameError%3A%20global%20name%20'wmi'%20is%20not%20defined%5C%22%20AND%20tags%3A%5C%22console%5C%22 | 17:05 |
| mriedem | blacklist it | 17:05 |
| *** vilobhmm11 has quit IRC | 17:06 | |
| mriedem | claudiub: ^ | 17:06 |
| *** zenoway has joined #openstack-nova | 17:06 | |
| *** tongli has quit IRC | 17:07 | |
| *** houming has quit IRC | 17:07 | |
| *** fawadkhaliq has quit IRC | 17:07 | |
| mriedem | https://bugs.launchpad.net/os-win/+bug/1533290 | 17:07 |
| openstack | Launchpad bug 1533290 in os-win "nova unit tests failing with os-win 0.1.0" [Undecided,New] | 17:07 |
| *** zakora has quit IRC | 17:08 | |
| *** omnipresent has quit IRC | 17:09 | |
| claudiub | thanks for letting me know. will fix asap.. | 17:09 |
| *** rk4n_ has quit IRC | 17:10 | |
| mdbooth | mriedem: I was about to ask about that. Wondering why my libvirt patch was failing in hyperv! | 17:11 |
| mriedem | :) | 17:11 |
| mriedem | g-r block https://review.openstack.org/266490 | 17:11 |
| ildikov | mdbooth: same here | 17:11 |
| cfriesen | anyone ever seen nova-compute get a http 408 error when talking to neutron? | 17:13 |
| *** apoorvad has joined #openstack-nova | 17:13 | |
| *** mgoddard has joined #openstack-nova | 17:13 | |
| dansmith | omg, this live migrate patch set is cursed | 17:13 |
| *** jdurgin1 has quit IRC | 17:14 | |
| *** klkumar has quit IRC | 17:15 | |
| *** hemnafk is now known as hemna | 17:15 | |
| mriedem | cfriesen: that doesn't happen to anyone but you | 17:15 |
| *** penick has quit IRC | 17:16 | |
| mriedem | and your forked ovs agent :) | 17:16 |
| *** sfinucan_ has quit IRC | 17:16 | |
| cfriesen | mreidem: I guess I'm special | 17:16 |
| sdague | mriedem: we could also just land this - https://review.openstack.org/#/c/266492/ | 17:16 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Config options: centralize base path configuration https://review.openstack.org/259406 | 17:17 |
| mriedem | sdague: i think you mixed 2 changes in htere | 17:17 |
| sdague | gah | 17:18 |
| mriedem | sdague: btw, the requirements bot update to u-c won't catch this failure | 17:18 |
| mriedem | since it's not running nova unit tests | 17:18 |
| mriedem | so eventually it would have broken us anyway | 17:18 |
| *** akshai has quit IRC | 17:18 | |
| sdague | mriedem: it would have | 17:20 |
| sdague | but this is long overdue anyway | 17:20 |
| mriedem | it would have saved us today | 17:20 |
| sdague | yeh | 17:20 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Hyper-V unit tests for latest os_win release https://review.openstack.org/266495 | 17:20 |
| claudiub | ^ review pls. | 17:21 |
| mriedem | eventually it would be bumped in here though https://review.openstack.org/#/c/265697/ | 17:21 |
| mriedem | and we wouldn't catch it in testing | 17:21 |
| *** daemontool has quit IRC | 17:22 | |
| *** gokrokve has quit IRC | 17:22 | |
| sdague | yeh | 17:24 |
| mriedem | claudiub: fixes b/c of this i guess https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/hyperv/test_base.py#L29 | 17:24 |
| *** sahid has quit IRC | 17:25 | |
| mriedem | so all calls to wmi from within os-win lib are mocked out in nova unit tests | 17:25 |
| mriedem | it feels like we should be mocking out os-win, not wmi within os-win | 17:26 |
| *** rk4n has joined #openstack-nova | 17:26 | |
| *** rk4n has quit IRC | 17:26 | |
| *** rk4n has joined #openstack-nova | 17:26 | |
| *** nic has joined #openstack-nova | 17:28 | |
| *** yassine has quit IRC | 17:29 | |
| *** fawadkhaliq has joined #openstack-nova | 17:29 | |
| *** ccarmack1 has joined #openstack-nova | 17:29 | |
| mriedem | sdague: since this is already approved, you should probably call it out on the ML for nova devs to be aware of https://review.openstack.org/#/c/266492/ | 17:29 |
| claudiub | mriedem: yeah. that mocks the wmi module. | 17:30 |
| mriedem | and remind everyone that they should be running -constraints jobs locally now | 17:30 |
| tojuvone | jaypipes: ok | 17:30 |
| danpb | mriedem: what does that actually mean ? | 17:32 |
| danpb | what are -constraints jobs ?!? | 17:32 |
| mriedem | i think it means, locally, don't run tox -r -e py27 | 17:32 |
| mriedem | b/c that won't be gated upstream | 17:32 |
| mriedem | you run tox -r -e py27-constraints | 17:32 |
| mriedem | fwiw, i think that project-config change was slammed in a bit fast w/o other nova cores reviewing it first | 17:32 |
| danpb | ewww, that's a bit gross - couldn't we just rename py27-constaints to be py27 and not make everyone remember to have to change | 17:33 |
| mriedem | we could | 17:33 |
| mriedem | well, except the jobs are using -constraints | 17:33 |
| mriedem | we could maybe alias py27 to py27-constraints somehow | 17:33 |
| danpb | sigh, why did we have to rush the gate change in so fast | 17:34 |
| mriedem | b/c nova unit tests are broken? | 17:34 |
| mriedem | there is also https://review.openstack.org/#/c/266495/ | 17:34 |
| mriedem | claudiub: so i take it you don't want to update the hyper-v unit tests to be more explicit about mocking out os-win rather than just globally mocking out wmi? | 17:35 |
| *** ccarmack1 has quit IRC | 17:37 | |
| claudiub | mriedem: i think we can do that after the unit tests are working again. that patch will fix them, i've run them on my linux env, seemed fine. | 17:38 |
| sdague | mriedem: please feel free to -1 | 17:39 |
| *** markus_z has quit IRC | 17:39 | |
| sdague | py27 target will still be there | 17:40 |
| sdague | it just isn't protected by constraints | 17:40 |
| sdague | which means it might fail more often | 17:40 |
| *** ccarmack1 has joined #openstack-nova | 17:41 | |
| *** szhsong has quit IRC | 17:42 | |
| *** e0ne has quit IRC | 17:42 | |
| sdague | mriedem: it failed jenkins, I'll just wip it. This has been a long time coming, and nova not being first means we don't get to impact the naming | 17:42 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Allow block live migration of an instance with attached volumes https://review.openstack.org/227278 | 17:42 |
| mriedem | sdague: yeah, i just fear people are so used to running py27 (and still in other projects) so they'll do it in nova and pass locally but fail upstream - or fail locally and pass upstream and be uber confused | 17:42 |
| *** ccarmack1 has quit IRC | 17:42 | |
| mriedem | it's a bridge we have to cross at some point | 17:42 |
| *** matrohon has joined #openstack-nova | 17:42 | |
| mriedem | which is why i asked for the ML thread just as an fyi | 17:43 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Live migration fails when the instance has a config_drive_format=iso9660 https://review.openstack.org/234659 | 17:43 |
| sdague | mriedem: it is extremely unlikely they will pass local and fail upstream | 17:43 |
| *** szhsong has joined #openstack-nova | 17:44 | |
| *** unicell has quit IRC | 17:44 | |
| mriedem | yeah, most likely the other way around | 17:44 |
| *** pratikmallya has joined #openstack-nova | 17:44 | |
| mriedem | py27 fails locally, so someone asks in irc why and we say 'run -constraints' | 17:44 |
| mriedem | we should just alias that if we can | 17:45 |
| sdague | if you do that you always need a network | 17:45 |
| mriedem | pip install -U --force-reinstall {opts} {packages} | 17:46 |
| mriedem | don't we check for updates anyway? | 17:46 |
| *** eil397 has joined #openstack-nova | 17:47 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add tag column to virtual_interfaces table https://review.openstack.org/264014 | 17:50 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 17:50 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: user visible changes https://review.openstack.org/264018 | 17:50 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: compute manager nic tagging support https://review.openstack.org/264017 | 17:50 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: libvirt driver implementation https://review.openstack.org/264016 | 17:50 |
| *** tonytan4ever has quit IRC | 17:51 | |
| mriedem | someone else want to +W this? https://review.openstack.org/#/c/266495/ | 17:51 |
| *** akshai has joined #openstack-nova | 17:53 | |
| dansmith | mriedem: got it | 17:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 17:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 17:55 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 17:55 |
| *** gangil has joined #openstack-nova | 17:55 | |
| *** gangil has joined #openstack-nova | 17:55 | |
| *** ctrath has quit IRC | 17:56 | |
| *** carey has joined #openstack-nova | 17:58 | |
| sdague | mriedem: ok, jenkins fixed, nova folks please weigh in - https://review.openstack.org/#/c/266492/3 | 18:00 |
| *** mspreitz has quit IRC | 18:00 | |
| *** gcb has quit IRC | 18:01 | |
| *** gcb has joined #openstack-nova | 18:02 | |
| *** diogogmt has quit IRC | 18:02 | |
| *** jistr has quit IRC | 18:03 | |
| *** jichen has joined #openstack-nova | 18:03 | |
| *** carey has quit IRC | 18:03 | |
| *** gjayavelu has joined #openstack-nova | 18:03 | |
| *** dave-mcc_ has quit IRC | 18:04 | |
| *** ctrath has joined #openstack-nova | 18:07 | |
| *** diogogmt has joined #openstack-nova | 18:07 | |
| *** tjones has joined #openstack-nova | 18:08 | |
| *** moshele has joined #openstack-nova | 18:13 | |
| claudiub | anyways. sorry about the mess. Also fixed in os_win. Won't happen again. :) | 18:13 |
| dansmith | sdague: so we'll be gating on the constraints job, which will be different from what most people are running, right? | 18:13 |
| mriedem | yes | 18:14 |
| mriedem | so there will be an education curve | 18:14 |
| mriedem | unless we alias | 18:14 |
| *** matrohon has quit IRC | 18:14 | |
| dansmith | alias in tox.ini? | 18:15 |
| mriedem | if that's possible | 18:15 |
| mriedem | but yeah | 18:15 |
| *** yamahata has joined #openstack-nova | 18:15 | |
| dansmith | I guess I'm confused as to why we need both | 18:15 |
| dansmith | if this is the right way (if), why would we not make everyone run this locally? | 18:15 |
| mriedem | i think that's what we need to do | 18:16 |
| mriedem | py27 would now just be -constraints | 18:16 |
| dansmith | okay so this infra change isn't needed if we just change tox.ini? | 18:16 |
| *** angdraug has joined #openstack-nova | 18:16 | |
| mriedem | well i'm assuming the -constraints jobs are calling the -constraints tox target | 18:16 |
| mriedem | and the change also makes them voting | 18:17 |
| *** unicell has joined #openstack-nova | 18:17 | |
| mriedem | yeah http://logs.openstack.org/99/263399/1/check/gate-nova-python27-constraints/20f7f0f/console.html.gz#_2016-01-04_19_40_31_176 | 18:17 |
| mriedem | and we don't want to run both b/c we know the non-constraints one will be busted | 18:17 |
| *** Marga_ has joined #openstack-nova | 18:17 | |
| danpb | oh, so the issue was we have already been running with 2 sets of jobs normal and -constraints | 18:18 |
| mriedem | yeah | 18:18 |
| mriedem | -constraints is non-voting | 18:18 |
| mriedem | and no one probably runs with that locally | 18:18 |
| danpb | and we simply disabled the old jobs and made -constraints ones voting | 18:18 |
| mriedem | in that change, yes | 18:19 |
| *** suro-patz has joined #openstack-nova | 18:19 | |
| mriedem | which isn't approved yet | 18:19 |
| danpb | is the -constraints job a global thing, or nova specific ? | 18:19 |
| mriedem | global | 18:19 |
| *** iurygregory has joined #openstack-nova | 18:19 | |
| danpb | ah, so that's why we're stuck with that tox job naming | 18:19 |
| mriedem | but i don't know the status of it in all projects | 18:19 |
| mriedem | i think neutron is using it as voting now | 18:19 |
| mriedem | yeah | 18:19 |
| *** ctrath has quit IRC | 18:19 | |
| danpb | it wouldn't be the end of the world to just update tox.ini and cut+paste content to make both sets of commands identical | 18:19 |
| danpb | and worry about collapsing them to a single command at some point later | 18:20 |
| mriedem | right | 18:20 |
| *** tonytan4ever has joined #openstack-nova | 18:20 | |
| mriedem | note that -constraints doesn't shield nova unit tests from breaking, at least not right now | 18:21 |
| *** ctrath has joined #openstack-nova | 18:21 | |
| *** tsandquist has joined #openstack-nova | 18:22 | |
| iurygregory | Hi people, i'm trying to use file injecton when creating an instance (http://paste.openstack.org/show/483640/) but the file doesn't appear in the instance | 18:24 |
| *** dane-fichter has quit IRC | 18:25 | |
| *** jaosorior has joined #openstack-nova | 18:27 | |
| jaypipes | tojuvone: ok, review done. thanks for your patience. | 18:28 |
| *** vilobhmm11 has joined #openstack-nova | 18:31 | |
| *** otter768 has joined #openstack-nova | 18:33 | |
| *** eharney has joined #openstack-nova | 18:35 | |
| *** krtaylor has quit IRC | 18:35 | |
| *** e0ne has joined #openstack-nova | 18:35 | |
| *** otter768 has quit IRC | 18:37 | |
| *** dank_ has joined #openstack-nova | 18:38 | |
| *** bwallis_ has joined #openstack-nova | 18:39 | |
| *** mriedem has quit IRC | 18:40 | |
| *** rlrossit has quit IRC | 18:41 | |
| *** fawadkhaliq has quit IRC | 18:42 | |
| tojuvone | jaypipes: Thanks :) | 18:42 |
| *** asselin_ has joined #openstack-nova | 18:42 | |
| dank_ | QQ If I may - we have multiple AZ’s in our cloud, there are no members in the ‘nova’ AZ - when we schedule an instance without an AZ we get an error stating “Availabilty zone not available” I updated the ‘default_schedule_zone’ directive to ‘None’ which should select I a random AZ from what i’ve read - but it isn’t working. Any advice on how to overcome this situation? | 18:43 |
| *** mriedem has joined #openstack-nova | 18:43 | |
| *** rk4n has quit IRC | 18:48 | |
| openstackgerrit | jichenjc proposed openstack/nova: Remove flavor.extra_spec on hw_watchdog_action https://review.openstack.org/254852 | 18:52 |
| *** READ10 has quit IRC | 18:52 | |
| *** rlrossit has joined #openstack-nova | 18:53 | |
| sdague | maybe lifeless can comment on any down side of just patching tox.ini and dumping the other jobs | 18:55 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 18:55 |
| *** femnad has quit IRC | 18:55 | |
| *** gangil has quit IRC | 18:57 | |
| *** danpb has quit IRC | 18:57 | |
| openstackgerrit | Andrew Bogott proposed openstack/nova: Add a disk space weight-based scheduler. https://review.openstack.org/261108 | 18:59 |
| *** jaosorior has quit IRC | 19:00 | |
| *** daemontool has joined #openstack-nova | 19:00 | |
| *** jaosorior has joined #openstack-nova | 19:00 | |
| *** kfarr_ has quit IRC | 19:01 | |
| *** szhsong has quit IRC | 19:02 | |
| *** szhsong has joined #openstack-nova | 19:03 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "guestfs" https://review.openstack.org/248963 | 19:05 |
| *** SourabhP has joined #openstack-nova | 19:05 | |
| openstackgerrit | jichenjc proposed openstack/nova: WIP:remove OS-EXT-IPS and OS-EXT-IPS-MAC extension https://review.openstack.org/266533 | 19:05 |
| *** aginwala has joined #openstack-nova | 19:06 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "wsgi" https://review.openstack.org/252687 | 19:06 |
| *** vks has quit IRC | 19:07 | |
| lifeless | sdague: a) it will not work properly unless you also patch up infra's job definitions | 19:08 |
| *** szhsong has quit IRC | 19:08 | |
| lifeless | - you'd end up running without zuul-cloner on the constraints jobs, so depends-on patches, and patches in branches, would be silently / subtly bust | 19:08 |
| *** szhsong has joined #openstack-nova | 19:09 | |
| lifeless | sdague: b), there's the confusion that can arise where py27 in nova is constrained and py27 in say neutron isn't | 19:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add tag column to virtual_interfaces table https://review.openstack.org/264014 | 19:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 19:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: user visible changes https://review.openstack.org/264018 | 19:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: compute manager nic tagging support https://review.openstack.org/264017 | 19:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: libvirt driver implementation https://review.openstack.org/264016 | 19:09 |
| lifeless | sdague: also it would be arguably inconsistent with the project testing interface, though there's lots of wiggle room there | 19:09 |
| *** diegows has joined #openstack-nova | 19:10 | |
| openstackgerrit | Andrew Bogott proposed openstack/nova: Prevent scheduling of instances on full hosts. https://review.openstack.org/242251 | 19:10 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add tag column to virtual_interfaces table https://review.openstack.org/264014 | 19:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Add metadata objects for device tagging https://review.openstack.org/264015 | 19:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: user visible changes https://review.openstack.org/264018 | 19:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: compute manager nic tagging support https://review.openstack.org/264017 | 19:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: libvirt driver implementation https://review.openstack.org/264016 | 19:12 |
| *** ihrachys has quit IRC | 19:14 | |
| *** diegows has quit IRC | 19:15 | |
| *** aginwala has quit IRC | 19:17 | |
| *** ccarmack has quit IRC | 19:17 | |
| *** salv-orlando has quit IRC | 19:17 | |
| *** salv-orlando has joined #openstack-nova | 19:18 | |
| mriedem | hmm, why doesn't https://github.com/openstack/nova-specs/blob/master/specs/mitaka/approved/vmware-vvol-support.rst show up on http://specs.openstack.org/openstack/nova-specs/specs/mitaka/ | 19:20 |
| *** yarkot has joined #openstack-nova | 19:22 | |
| *** aginwala has joined #openstack-nova | 19:23 | |
| *** yarkot has quit IRC | 19:25 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: ironic driver: Adding Infiniband support https://review.openstack.org/266540 | 19:26 |
| mgagne | pkoniszewski: are you aware of any bug or known issue with QEMU 2.0 where after a ilve migration, the destination qcow file is bloated to around 50% of its original disk size? | 19:27 |
| rlrossit | everyone remember that mriedem owes me $1 | 19:27 |
| *** ccarmack has joined #openstack-nova | 19:29 | |
| mgagne | virsh domblkinfo output: http://paste.openstack.org/show/483651/ | 19:30 |
| *** suro-patz has quit IRC | 19:30 | |
| mriedem | moshele: on this https://review.openstack.org/#/c/261369/ do you plan on ever needing to backport that? | 19:31 |
| mriedem | i.e. it looks like a bug | 19:31 |
| dims | rlrossit : ack i can remind him :) | 19:31 |
| *** baoli has quit IRC | 19:31 | |
| *** jaosorior has quit IRC | 19:31 | |
| *** jaosorior has joined #openstack-nova | 19:32 | |
| moshele | mriedem: I will once it approved | 19:32 |
| mriedem | moshele: my point is the commit message probably needs a bug linked | 19:33 |
| moshele | mriedem: some opened bug in neutron on this do you want me to update the commit msg | 19:33 |
| mriedem | yeah | 19:33 |
| moshele | mriedem: I will do | 19:33 |
| mriedem | thanks | 19:33 |
| mriedem | i'll approve after that | 19:33 |
| mriedem | sc68cal: look good to you? https://review.openstack.org/#/c/261369/ | 19:34 |
| iurygregory | Hi people, i'm trying to use file injecton when creating an instance (http://paste.openstack.org/show/483640/) but the file doesn't appear in the instance. anyone can help me? | 19:35 |
| *** dave-mccowan has joined #openstack-nova | 19:36 | |
| mriedem | iurygregory: please see the channel topic, this is a dev channel, not a support channel | 19:37 |
| * sc68cal looks | 19:37 | |
| mriedem | iurygregory: try ask.openstack.org, openstack ML or #openstack irc channel | 19:37 |
| *** kfarr has joined #openstack-nova | 19:38 | |
| iurygregory | tks mriedem =) in puppet-openstack people ask some questions sorry | 19:38 |
| openstackgerrit | Moshe Levi proposed openstack/nova: Sets binding:profile to empty dic when unbinding port https://review.openstack.org/261369 | 19:38 |
| sc68cal | mriedem: honestly I don't know very much about that part of the code | 19:38 |
| mriedem | c'mon! | 19:39 |
| mriedem | seems fine | 19:39 |
| mriedem | i looked at how it's used and what it's defined as in the neutron db model | 19:39 |
| mriedem | seems sane | 19:39 |
| mriedem | which of the 30 neutron liuetenants would know about that? :) | 19:40 |
| * mriedem doesn't really want to know | 19:40 | |
| *** iurygregory has left #openstack-nova | 19:40 | |
| *** armax has quit IRC | 19:40 | |
| ccarmack | Lieutenant | 19:42 |
| mriedem | thanks chuck | 19:43 |
| ccarmack | yw mriedem | 19:43 |
| moshele | mriedem: I am the pci/sriov :) | 19:43 |
| *** SourabhP has quit IRC | 19:43 | |
| mriedem | heh | 19:43 |
| *** Sukhdev has joined #openstack-nova | 19:43 | |
| moshele | mriedem: you can add baoli to review | 19:44 |
| mriedem | i assume if you throw a dart at an org chart there is a good chance you'll hit a neutron lieutenant | 19:44 |
| moshele | mriedem: I guess you right :) | 19:44 |
| mriedem | last question is, {} vs None | 19:44 |
| mriedem | maybe it doesn't matter? | 19:44 |
| openstackgerrit | jichenjc proposed openstack/nova: Create Nova Scheduler Num Instances Weighter https://review.openstack.org/130526 | 19:45 |
| moshele | mriedem: I check the normal port which doesn't use the binding:proflle and it was set to {} | 19:46 |
| mriedem | ok | 19:46 |
| mriedem | works for me | 19:46 |
| sc68cal | mriedem: binding:profile is only used by mech_sriov it appears? | 19:46 |
| *** armax has joined #openstack-nova | 19:46 | |
| moshele | sc68cal: correct | 19:47 |
| mriedem | +W | 19:47 |
| sc68cal | so, if they break themselves, sure whatever. | 19:47 |
| moshele | mriedem: thanks | 19:47 |
| * sc68cal can't wait for jaypipes to fix all this | 19:48 | |
| *** SourabhP has joined #openstack-nova | 19:50 | |
| *** armax has quit IRC | 19:51 | |
| mriedem | ildikov: so i guess your multiattach series is blocked on https://review.openstack.org/#/c/193133/ right? | 19:51 |
| ildikov | mriedem: it's just the os-win, once it gets through the first three patch should be just fine | 19:52 |
| mriedem | ildikov: well dansmith has a -1 from ps37 | 19:53 |
| ildikov | I had to rebase them today because of an RPC change | 19:53 |
| ildikov | that is why I mentioned the experimental option | 19:53 |
| *** omnipresent has joined #openstack-nova | 19:53 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 19:54 |
| ildikov | mriedem: if you check the discussion the issue is, if I'm not mistaken that you can have a mixed up environment regarding compute services, which can mess up the operations with multiattach as the old ones does not send the attachment_id | 19:54 |
| *** gangil has joined #openstack-nova | 19:54 | |
| *** gangil has joined #openstack-nova | 19:54 | |
| ildikov | mriedem: I answered the comments on that review and asked about alternatives | 19:55 |
| ildikov | mriedem: basically the db and obj patch itself does not do any harm | 19:55 |
| mriedem | send the instance uuid you mean? | 19:56 |
| ildikov | the problem is the chain which enables multiattach by the end | 19:56 |
| ildikov | I mean send the attachment_id to Cinder | 19:56 |
| ildikov | and also using instance_uuid to retrieve the BDM where possible | 19:56 |
| jaypipes | sc68cal: what did I break this time? | 19:57 |
| *** zenoway has quit IRC | 19:57 | |
| ildikov | the only place where it's not is the volume_snapshot_create, but that's blocked for multiattach in compute_api, therefore it's safe from upgrade perspective | 19:57 |
| *** aginwala has quit IRC | 19:57 | |
| sc68cal | jaypipes: nothing - it was just about how nova and neutron handle vif types | 19:57 |
| jaypipes | sc68cal: ah.. | 19:57 |
| ildikov | mriedem: the db functions return the first() match for the BDM query | 19:57 |
| ildikov | mriedem: and I cannot change the old functions, that would break things | 19:58 |
| sc68cal | jaypipes: mostly how you have to do a change in nova, and neutron | 19:58 |
| *** carey has joined #openstack-nova | 19:59 | |
| ildikov | mriedem: but if we can introduce mutliattach as experimental now, then we will have the code in Mitaka and in Newton we can enable it by default and the upgrade should work fine too | 19:59 |
| mriedem | ildikov: i guess i'm not sure what that would look like | 20:00 |
| mriedem | and why it's really needed | 20:01 |
| *** tjones has quit IRC | 20:01 | |
| ildikov | mriedem: it might not and I'm just over reacting it | 20:01 |
| *** gjayavelu has quit IRC | 20:02 | |
| *** aginwala has joined #openstack-nova | 20:02 | |
| *** w_verdugo has quit IRC | 20:02 | |
| *** rfolco has quit IRC | 20:02 | |
| mriedem | so what is the upgrade issue with sending the attachment id? | 20:03 |
| mriedem | is that the rpc thing? | 20:03 |
| mriedem | where we pass the attachment id on detach? | 20:03 |
| *** carey has quit IRC | 20:04 | |
| ildikov | nah, that should be fine | 20:06 |
| ildikov | I was wondering about having an env, where not all the computes are upgraded | 20:07 |
| mriedem | ildikov: in that case we'd detect it in the compute rpcapi and remove the attachment_id kwarg before sending it to detach_volume on the liberty level compute | 20:07 |
| ildikov | but we have some, where multiattach support is added already, so there might be issues | 20:07 |
| ildikov | yeah, but we still have multiattached volume and sending detach to Cinder without attachment_id, which means it will not be able to identify which attachment to remove | 20:08 |
| mriedem | and we get a 400 then? | 20:09 |
| *** aginwala has quit IRC | 20:12 | |
| *** lucasagomes is now known as lucas-dinner | 20:14 | |
| mriedem | so we have a mitaka nova-api and at least one mitaka nova-compute | 20:14 |
| mriedem | we get a request to attach a multiattach volume | 20:14 |
| mriedem | the mitaka nova api and compute get that request and handle it b/c they understand it | 20:15 |
| *** aginwala has joined #openstack-nova | 20:15 | |
| mriedem | then we get a request to detach that volume and it gets to a liberty compute which won't have the attachment_id to pass on to cinder | 20:15 |
| mriedem | so the detach will fail on the cinder side (mitaka cinder code) | 20:15 |
| mriedem | ildikov: is that the right scenario ^ ? | 20:15 |
| ildikov | I would assume, but even Liberty Cinder should behave the same way | 20:16 |
| mriedem | sure, cinder > juno | 20:16 |
| mriedem | >= juno | 20:16 |
| mriedem | when multiattach was added to cinder api | 20:16 |
| *** irenab_ has joined #openstack-nova | 20:16 | |
| mriedem | so everyone but rax works :) | 20:16 |
| openstackgerrit | Moshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex https://review.openstack.org/199488 | 20:17 |
| *** Sukhdev has quit IRC | 20:18 | |
| *** irenab has quit IRC | 20:18 | |
| *** irenab_ is now known as irenab | 20:18 | |
| *** szhsong has quit IRC | 20:19 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Catch errors while creating veth pair https://review.openstack.org/264146 | 20:19 |
| *** szhsong has joined #openstack-nova | 20:19 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 20:19 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 20:19 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db and obj: Allow multiple volume attachments https://review.openstack.org/193133 | 20:19 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 20:19 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 20:19 |
| ildikov | mriedem: :) | 20:20 |
| ildikov | mriedem: imagining Juno Cinder with Mitaka Nova I wonder whether the multiattach will be the biggest issue :) | 20:20 |
| mriedem | ildikov: do we have any tempest changes up yet that have some of the test coverage for this series? | 20:21 |
| mriedem | it'd be really nice to see a tempest change that depends on the top nova change in the series which is testing with multiattach and we could see it working | 20:21 |
| *** dslev has joined #openstack-nova | 20:22 | |
| ildikov | mriedem: not yet | 20:22 |
| ildikov | mriedem: can we make tempest pass, if this change is not merged? | 20:22 |
| *** moshele has quit IRC | 20:22 | |
| *** mrkz has quit IRC | 20:23 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type https://review.openstack.org/262584 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr https://review.openstack.org/262585 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions https://review.openstack.org/249016 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs https://review.openstack.org/262586 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests https://review.openstack.org/259254 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver https://review.openstack.org/262583 | 20:23 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: update the old location parent_addr only if it has value https://review.openstack.org/266560 | 20:23 |
| ildikov | mriedem: question from the other direction: can these patches be merged without Tempest tests up? | 20:23 |
| mriedem | ildikov: yes, using the Depends-On tag in the tempest commit message | 20:23 |
| mriedem | ildikov: i'd really prefer to see integration test coverage for this series before it lands in nova | 20:24 |
| mriedem | which we can surely do with a tempest change that has the Depends-On to the top nova change in the series | 20:24 |
| ildikov | mriedem: the deadline is pretty very close | 20:25 |
| *** dslev has quit IRC | 20:25 | |
| ildikov | I would assume it should not be rocket science based on the already existing tests, but I will not know until I actually get there | 20:25 |
| *** gangil has quit IRC | 20:25 | |
| *** aginwala has quit IRC | 20:26 | |
| mriedem | well, there is 0 tempest testing with multiattach right? | 20:26 |
| *** aginwala has joined #openstack-nova | 20:26 | |
| ndipanov | dansmith, a follow on to that data migration - a small edge case I missed https://review.openstack.org/#/c/266560/1 | 20:27 |
| mriedem | i'd rather this misses mitaka due to lack of test coverage than we release it and volume attachment is broken somehow | 20:27 |
| ildikov | mriedem: currently there's no, only unit and functional tests | 20:27 |
| ildikov | we can still put an experimental flag on it | 20:27 |
| mriedem | ildikov: honestly, the spec said there would be tempest tests, so IMO we should always have a tempest change depending on a nova api change series before we land the nova changes, we just don't always make people honor their commitments to add tempest testing | 20:28 |
| *** tonytan4ever has quit IRC | 20:28 | |
| mriedem | ildikov: i'm still not clear on where the experimental flag would go | 20:28 |
| mriedem | there is code everywhere in this series | 20:28 |
| dansmith | ndipanov: okay | 20:29 |
| ildikov | mriedem: yeah, I get that, but it's only M-2 that's coming so I have 1/3 of a cycle to get the tempest tests and fix whatever needs to be fixed | 20:29 |
| mriedem | can someone else work on the tempest tests? | 20:29 |
| mriedem | in parallel? | 20:29 |
| mriedem | i would if i had time | 20:29 |
| *** lpetrut has joined #openstack-nova | 20:29 | |
| ildikov | mriedem: that would need to go to the attach_check, turn off the two blocks and to block_device just to ensure that every path is covered to not enable the second attachment | 20:30 |
| ildikov | mriedem: this work is not the highest priority in my job currently, but we're pushing this for cycles now, so I spend literally my nights with this now | 20:30 |
| ildikov | mriedem: and business as usual we have resource coverage issues | 20:31 |
| *** mrkz has joined #openstack-nova | 20:31 | |
| ildikov | mriedem: sorry, I stop complaining | 20:32 |
| *** szhsong has quit IRC | 20:32 | |
| *** szhsong has joined #openstack-nova | 20:32 | |
| mriedem | yeah i understand | 20:32 |
| ildikov | mriedem: so I try to find a solution here, believe me, I like better provide quality work, if you check the patches, I added tests, where I could already | 20:33 |
| *** smatzek has quit IRC | 20:33 | |
| *** w_verdugo has joined #openstack-nova | 20:33 | |
| melwitt | ildikov, mriedem: I don't know if you noticed my comment on https://review.openstack.org/#/c/256273/ in the series, but it's not currently passing jenkins yet because an invalid cinder attach call is being made in the patch | 20:33 |
| *** gangil has joined #openstack-nova | 20:33 | |
| *** gangil has joined #openstack-nova | 20:33 | |
| *** suro-patz has joined #openstack-nova | 20:33 | |
| ildikov | melwitt: working on it with hemna: I marked it WIP because of that earlier | 20:33 |
| mriedem | melwitt: yeah, it's WIP and it's not tied to the bp so i wasn't reviewing those yet | 20:34 |
| mriedem | there are really just 3 changes here, the db one, the compute manager one, and the libvirt driver | 20:34 |
| *** otter768 has joined #openstack-nova | 20:34 | |
| mriedem | i'm not sure if we have race/upgrade issues with the db one, there was concern in a previous patch set that i'm not sure is cleared up | 20:34 |
| ildikov | mriedem: this should not introduce more race than what we had already, IMHO | 20:35 |
| mriedem | and i don't have the mixed compute scenario in my head really for where detach fails b/c we don't pass cinder the attachment id | 20:35 |
| dansmith | is it not a new cinder call entirely? | 20:35 |
| melwitt | ildikov, mriedem: oh, didn't realize it wasn't tied to the bp. I happened upon it in logstash flagged by an e-r check I ended up removing for an old bug and I commented on it in case it wasn't noticed yet | 20:35 |
| ildikov | as I said it's not the db patch in either case that causes issues here | 20:35 |
| dansmith | what happens if you have two novas one old and one new, with a multi-attached volume? | 20:35 |
| dansmith | is there anything to guard against that? | 20:35 |
| mriedem | dansmith: that's what i'm trying to sort out | 20:36 |
| ildikov | melwitt: no probs, thanks for your comment, I saw it in the test falures, so I was aware, but better twice than not noticing at all :) | 20:36 |
| *** tjones has joined #openstack-nova | 20:36 | |
| mriedem | dansmith: detach_volume here https://review.openstack.org/#/c/193134/64/nova/compute/manager.py | 20:36 |
| mriedem | that gets an attachment_id now from the api | 20:37 |
| mriedem | if you're api is new enough | 20:37 |
| mriedem | *your | 20:37 |
| *** kfarr has quit IRC | 20:37 | |
| ildikov | the api should always be upgraded first | 20:37 |
| dansmith | ildikov: what, cinder api you mean? | 20:38 |
| ildikov | as much as I know | 20:38 |
| ildikov | no, Nova API | 20:38 |
| ildikov | when the detach goes down from the API to compute | 20:38 |
| *** otter768 has quit IRC | 20:39 | |
| ildikov | we send the attachment_id, if the call chain is not an API detach request and we don't have it, then in cinder.py we have one more attempt to retrieve the attachment_id so Cinder knows which volume to detach | 20:39 |
| *** tonytan4ever has joined #openstack-nova | 20:39 | |
| *** vilobhmm11 has quit IRC | 20:39 | |
| *** smatzek has joined #openstack-nova | 20:39 | |
| ildikov | of course it matters only in case when we have the volume attached twice | 20:39 |
| *** salv-orlando has quit IRC | 20:39 | |
| *** gangil has quit IRC | 20:40 | |
| dansmith | what happens if nova is upgraded before cinder? | 20:40 |
| *** vilobhmm11 has joined #openstack-nova | 20:40 | |
| dansmith | or cinder is never upgraded (like RAX) ? | 20:40 |
| ildikov | the Cinder drivers have capabilities maintained | 20:40 |
| *** salv-orlando has joined #openstack-nova | 20:40 | |
| *** szhsong has quit IRC | 20:40 | |
| ildikov | which means you cannot create a volume with multiattach support if the driver does not support it | 20:41 |
| ildikov | so in case of very old Cinder we will not have multiattach at all | 20:41 |
| mriedem | yeah, i think old cinder wouldn't be a problem | 20:41 |
| *** nic1 has joined #openstack-nova | 20:41 | |
| dansmith | but the detach calls are passing the attachment id? is that handled in cinderclient? | 20:41 |
| ildikov | because Nova uses the multiattach flag from the volume info | 20:41 |
| mriedem | you have to bring a multiattach volume to nova | 20:41 |
| *** aginwala has quit IRC | 20:41 | |
| mriedem | dansmith: yeah it's in cinderclient, at a min version that mitaka requires | 20:41 |
| dansmith | hmm, okay | 20:41 |
| *** nic has quit IRC | 20:42 | |
| mriedem | in the case that nova boots from volume (image/blank/snapshot) and creates the volume in cinder, we don't create it as multiattach=True | 20:42 |
| ildikov | I think oficially we need to support one release back | 20:42 |
| mriedem | False is the default | 20:42 |
| ildikov | yeah | 20:42 |
| ildikov | and boot from multiattach volume is disabled | 20:42 |
| mriedem | so in the n-1 compute case, it looks like the fallback is we get the attachments from the volume and then look for the one that is mapped to the instance uuid here https://review.openstack.org/#/c/193134/64/nova/volume/cinder.py | 20:42 |
| mriedem | ildikov: i thought we were ok with multiattach boot from volume now? i have to go back and read the change on the amended spec | 20:43 |
| *** bwallis_ has quit IRC | 20:43 | |
| ildikov | mriedem: what we don't want is actually boot from that volume, so when it is marked with boot_index=0, that case is disabled | 20:43 |
| dansmith | mriedem: how would that work? | 20:43 |
| ildikov | all the other cases are fine | 20:43 |
| mriedem | ildikov: oh right | 20:44 |
| dansmith | I guess you could readonly boot from multi | 20:44 |
| *** jaosorior has quit IRC | 20:44 | |
| dansmith | you'd have to be super careful though | 20:44 |
| ildikov | I disabled, when destination_type is volume too | 20:44 |
| mriedem | so i think what i'd really like to see is a multinode test where we attach an instance on n-1 compute and current compute to the same volume, then detach both | 20:45 |
| mriedem | that would this attachment_id lookup path | 20:45 |
| mriedem | *hit this | 20:45 |
| mriedem | but maybe that gets tricky, i'm not sure if tempest multinode jobs can tell which is the older host or not | 20:46 |
| mriedem | mtreinish: ^ do you know? | 20:46 |
| *** szhsong has joined #openstack-nova | 20:46 | |
| ildikov | mriedem: do you want this before merging the code? | 20:47 |
| *** cburgess_ has quit IRC | 20:48 | |
| mtreinish | mriedem: unless you can check it from the api not easily | 20:48 |
| *** matrohon has joined #openstack-nova | 20:48 | |
| *** cburgess has joined #openstack-nova | 20:48 | |
| mtreinish | you could play a game with a config variable and have devstack/grenade pass it into tempest. But, I'd probably be -2 that | 20:48 |
| dansmith | I really don't understand how this is not going to confuse any non-mitaka compute in the deployment | 20:49 |
| ildikov | mtreinish: TBH bad news would make me happy here... :) | 20:49 |
| mriedem | mtreinish: i don't think we expose any version in the hosts api | 20:50 |
| ildikov | dansmith: I was wondering to make multiattach experimental and document to enable it with homogeneous Nova env to avoid issues | 20:51 |
| mtreinish | mriedem: yeah, that's what I thought too | 20:51 |
| dansmith | ildikov: the right thing to do there is to make mitaka computes able to handle it and then introduce it in newton, but .. :) | 20:51 |
| ildikov | dansmith: the experimental flag would do this job exactly | 20:52 |
| dansmith | ildikov: well, we wouldn't need an experimental flag if we didn't add the feature until later | 20:52 |
| mtreinish | mriedem: you could also just do it both ways with 4 guests and 2 volumes (or maybe 1) that'll ensure you go the way you want (assuming only 2 computes) | 20:52 |
| *** daemontool has quit IRC | 20:52 | |
| ildikov | this would let us have the checks there for Mitaka and who upgraded fully or have lab env, they can play with it, at least we will have feedback finally as opposed to theoretical scenarios | 20:53 |
| ildikov | dansmith: there's no API request change for this | 20:53 |
| ildikov | dansmith: whenever you introduce multiattach you will face the exact same problem as now | 20:53 |
| *** szhsong has quit IRC | 20:53 | |
| dansmith | mriedem: maybe we should require service_version before any computes will honor the multi-attach semantics | 20:54 |
| *** ctrath has quit IRC | 20:54 | |
| *** szhsong has joined #openstack-nova | 20:54 | |
| ildikov | dansmith: what's new is a few more checks in the code and passing the attachment_id to Cinder | 20:55 |
| dansmith | well, also what is new is having multiple BDMs in our database with the same instance_uuid, right? | 20:55 |
| *** krtaylor has joined #openstack-nova | 20:55 | |
| ildikov | same volume_id | 20:55 |
| dansmith | sorry, right | 20:56 |
| *** edtubill has joined #openstack-nova | 20:56 | |
| ildikov | and all the get_by_volume_id calls are updated | 20:56 |
| ildikov | except the create_volume_snapshot, that's disabled with multiattach, because we don't have instance_uuid there | 20:56 |
| mriedem | dansmith: so on the multiattach attach request, we check the computes to find one that meets a min version to support it? | 20:56 |
| mriedem | it seems to me the unfortunate thing is this orchestration with cinder shouldn't even be in the compute | 20:57 |
| mriedem | unless i'm missing something | 20:57 |
| *** raildo is now known as raildo-afk | 20:57 | |
| *** rk4n has joined #openstack-nova | 20:57 | |
| dansmith | mriedem: no, on that request we see if any computes are below the required level | 20:57 |
| dansmith | mriedem: which is a very quick ask of the db | 20:57 |
| *** smatzek has quit IRC | 20:58 | |
| mriedem | and then fail if there are? so you can't multiattach in nova until all of your computes are upgrade to at least mitaka? | 20:58 |
| dansmith | she said there was no new request, right? so it would just be failing the second attach in that case, right? | 20:59 |
| dansmith | I mean, I don't love it, I'd rather merge support and then expose it, but.. | 20:59 |
| ildikov | dansmith: exactly | 20:59 |
| dansmith | just throwing out ideas | 20:59 |
| *** rk4n has quit IRC | 20:59 | |
| ildikov | the bigger part of the code gets in the better for me TBH as I don't have to rebase it all the time | 21:00 |
| ildikov | the most part of this code is in check_attach and some code in check_detach and enabling the actual attach call to Cinder in one place | 21:01 |
| *** dane-fichter has joined #openstack-nova | 21:01 | |
| *** armax has joined #openstack-nova | 21:01 | |
| ildikov | and adding the attachment_id to the Cinder detach call | 21:01 |
| openstackgerrit | Andrew Bogott proposed openstack/nova: Add a disk space weight-based scheduler. https://review.openstack.org/261108 | 21:01 |
| ildikov | so most parts of the code does not make any harm | 21:01 |
| *** dslevin has joined #openstack-nova | 21:03 | |
| *** jichen has quit IRC | 21:03 | |
| *** vladikr has quit IRC | 21:03 | |
| mriedem | so the 2nd attach fails b/c the libvirt driver (in liberty) doesn't know how to handle that thing | 21:03 |
| mriedem | is that right? | 21:03 |
| *** dslevin has quit IRC | 21:04 | |
| dansmith | that's what I'm saying, but I don't even know if it's possible | 21:04 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create filter_properties earlier in boot request https://review.openstack.org/254434 | 21:04 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Parse availability_zone in API https://review.openstack.org/266573 | 21:04 |
| dansmith | we'd have to do it in a transaction somehow to make sure we don't race our way to multiple attachments I guess | 21:05 |
| *** kfarr has joined #openstack-nova | 21:05 | |
| *** busterswt has quit IRC | 21:05 | |
| *** dave-mccowan has quit IRC | 21:06 | |
| *** dave-mccowan has joined #openstack-nova | 21:07 | |
| *** MVenesio has quit IRC | 21:07 | |
| ildikov | mriedem: how the libvirt driver is upgraded? | 21:08 |
| mriedem | ildikov: when you upgrade nova-compute | 21:08 |
| ildikov | I mean what's the relation between the libvirt driver and the computes? | 21:08 |
| mriedem | 1:1 | 21:08 |
| mriedem | ildikov: from what i can tell in https://review.openstack.org/#/c/193134/ you're not checking to see if the virt driver can even support multiattach | 21:09 |
| mriedem | from the capability flag i mean | 21:09 |
| ildikov | but then if the compute is ok, then we're ok, if not, then it will not get into a bad situation | 21:09 |
| *** krtaylor has quit IRC | 21:09 | |
| ildikov | mriedem: yeah, don't ask me why, but I added that to the libvirt patch | 21:09 |
| ildikov | oh, I know why | 21:09 |
| ildikov | so in the second patch multiattach in block_device is not enabled at all | 21:10 |
| ildikov | so it's the third patch that makes the magic and that's where I added the extra check | 21:10 |
| *** aysyd has quit IRC | 21:11 | |
| mriedem | here https://review.openstack.org/#/c/193135/74/nova/virt/block_device.py | 21:11 |
| mriedem | that should probably be in the 2nd change where the capabilities flag is introduced | 21:11 |
| ildikov | I'm at least the third owner of this code, I rewrote 2/3 of it, but the structure more or less remained what the previous owner gave to me | 21:11 |
| mriedem | the 3rd change just sets it to True for the libvirt driver | 21:11 |
| mriedem | yeah, you might want to add Co-Authored-By in the commit messages for this :) | 21:12 |
| ildikov | without the change in line 307 you don't have multiattach support anyway | 21:12 |
| ildikov | I was wondering to change author just simply | 21:12 |
| *** armax has quit IRC | 21:13 | |
| ildikov | so I can write tempest tests if I don't have to reorganize the code ;) | 21:13 |
| mriedem | tempest tests would live in tempest | 21:14 |
| mriedem | not nova | 21:14 |
| ildikov | mriedem: have you just added back the libvirt version comment? | 21:14 |
| mriedem | yeah, why don't we have a min libvirt version check? | 21:14 |
| ildikov | I meant that I don't have time for both until next Thursday | 21:15 |
| ildikov | mriedem: check the blueprint spec amendment | 21:15 |
| mriedem | oh i see | 21:15 |
| mriedem | "You're are reading the wrong docs here. That 'shareable' flag doc you quote is the doc for the <hostdev> device type, *not* the <disk> device type. There is no min version requirement for the <shareable> flag wrt <disk> devices" | 21:15 |
| ildikov | yeap, so I removed it from the spec also not added to the code | 21:15 |
| ildikov | I was just a few tests away, when I saw that comment... | 21:15 |
| mriedem | ok, like i said, i need to review the spec update again | 21:15 |
| ildikov | mocks are not my friends... | 21:16 |
| *** twinklechawla1 has joined #openstack-nova | 21:16 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: linux_net: retry ovs vif port creation https://review.openstack.org/265973 | 21:16 |
| ildikov | I think that was the major comment on it | 21:16 |
| *** dane-fichter has quit IRC | 21:16 | |
| mriedem | with tempest these wouldn't be mocks | 21:17 |
| *** ctrath has joined #openstack-nova | 21:17 | |
| mriedem | the simplest tempest test would be to just create a single multiattach volume and attach it to 2 instances | 21:17 |
| mriedem | tempest will need a config option so the test is conditional on whether or not the env supports multiattach (depends on which backend you're using for nova) | 21:18 |
| *** MVenesio has joined #openstack-nova | 21:18 | |
| ildikov | mriedem: could I just move to add the flag to the third patch? | 21:18 |
| mriedem | the virt driver flag? | 21:19 |
| ildikov | mriedem: yeah, the mock comment was for the libvirt version check, I had issues there... | 21:19 |
| ildikov | yes, that one | 21:19 |
| mriedem | i think it makes more sense in the compute change, since nova.virt.block_device is really an extension of the compute layer | 21:19 |
| mriedem | plus that's the change that's plumbing in the api changes | 21:19 |
| mriedem | so say we merged that w/o the check in block_device that would cause it to fail if someone tried it, things get weird | 21:19 |
| ildikov | but that change does not allow multiattach | 21:20 |
| ildikov | I mean the second patch | 21:20 |
| ildikov | so that flag is about the virt layer under compute, so I can move the flag to that change, where it belongs | 21:20 |
| ildikov | and then I can do all the test coverage at one place | 21:21 |
| ildikov | I don't get your last comment | 21:22 |
| ildikov | do you mean we fail silently? | 21:22 |
| mriedem | so in the 2nd change a 2nd attach request for the same volume would fail b/c the volume status is already 'in-use' | 21:23 |
| mriedem | and block_device is checking that it's 'detached' | 21:23 |
| ildikov | yes | 21:23 |
| mriedem | i'd be ok with moving the supports_multiattach capability flag to the 3rd change | 21:23 |
| *** pratikmallya has quit IRC | 21:23 | |
| ildikov | ok, great, will do that | 21:23 |
| ildikov | BTW it would be so good if that capability dict would be updated as opposed to just overwrite it in every driver | 21:25 |
| ildikov | I almost missed it | 21:25 |
| *** twinklechawla1 has quit IRC | 21:25 | |
| mriedem | yeah that's annoying | 21:26 |
| *** rotbeard has joined #openstack-nova | 21:28 | |
| mriedem | jaypipes: didn't you have people that had a vested interest in the cinder multiattach series in nova? | 21:29 |
| mriedem | i.e. minions | 21:29 |
| doffm | alaski: cell0. Is there any reason that the top-level cell (The API cell) can't be used as cell0? | 21:29 |
| doffm | Seems like that would make things easier, as we could just keep all the tables around in the api cell for the moment. | 21:29 |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "service" https://review.openstack.org/256569 | 21:29 |
| *** rk4n has joined #openstack-nova | 21:30 | |
| *** dslevin has joined #openstack-nova | 21:30 | |
| doffm | alaski: (I'm sure there is a very good reason, i'm just late to this thought process) | 21:30 |
| *** rk4n has quit IRC | 21:31 | |
| *** aginwala has joined #openstack-nova | 21:31 | |
| alaski | doffm: it does make things easier, and the tradeoff is losing the clear distinction between the api and a cell | 21:31 |
| alaski | I'm not totally opposed to the idea though | 21:32 |
| *** ZZelle_ has joined #openstack-nova | 21:32 | |
| *** gangil has joined #openstack-nova | 21:33 | |
| *** gangil has joined #openstack-nova | 21:33 | |
| *** rosmaita has joined #openstack-nova | 21:33 | |
| ildikov | mriedem: jaypipes: if there are as we discussed it ages ago they would, now they could write tempest tests :) | 21:33 |
| doffm | alaski: I'm not sure that the tradeoff is too bad. Especially as the spec talks about trimming down the tables needed in cell0. | 21:34 |
| doffm | alaski: Eventually the api cell would just have the tables needed for instances in error state. | 21:34 |
| mriedem | ildikov: i might take a stab at an initial tempest change | 21:34 |
| doffm | rather than full duplication. | 21:34 |
| dansmith | doffm: alaski: seems like the abstraction is much cleaner if we don't muddy things in the api cell too much | 21:35 |
| dansmith | the special cell being otherwise a normal cell means it's actually less special and more of just a parking lot | 21:36 |
| alaski | it definitely is, and that's my preference. but I'm open to arguments | 21:36 |
| *** yamahata has quit IRC | 21:36 | |
| alaski | doffm: it should be noted that the api db currently doesn't have the tables necessary, they would all need to be added | 21:36 |
| doffm | dansmith: cell0 is going to be special anyway. Its not going to be like the other cells. | 21:36 |
| dansmith | alaski: and we'd have some models talking to databases with both sets and some with one | 21:37 |
| dansmith | alaski: which means it's easier to accidentally build a relationship or dependency that we don't catch in regular cell-of-one testing | 21:37 |
| dansmith | doffm: yeah I know :0 | 21:37 |
| *** ildikov has quit IRC | 21:37 | |
| alaski | dansmith: yeah, I totally agree | 21:38 |
| doffm | Ok. | 21:39 |
| *** ociuhandu has quit IRC | 21:39 | |
| dansmith | alaski: an important part of this process is making sure that a single-cell deploy and a multi-cell deploy are always basically the same, and so maintaining the distinction in the single case is important | 21:39 |
| doffm | Cell0 when finished seems like something of a rump to me, but maybe its an important one. | 21:40 |
| *** thangp has quit IRC | 21:40 | |
| dansmith | doffm: it's a special implementation of an abstraction | 21:40 |
| alaski | I'm on the side of keeping cell0 a separate thing, but I still haven't shaken the nagging feeling that asking deployers to setup a db and message queue for just a parking lot might be a bit much | 21:40 |
| dansmith | doffm: as opposed to something that is special all the way up past the abstraction | 21:40 |
| *** ildikov has joined #openstack-nova | 21:40 | |
| ildikov | mriedem: sorry, my laptop has just died on me... | 21:41 |
| alaski | with cell0 there is no single cell deployment | 21:41 |
| dansmith | alaski: it just has to be a db on the same host, and I think the mq part is probably equally minimal | 21:41 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add test coverage to functional api tests _compare_result method https://review.openstack.org/263440 | 21:41 |
| ildikov | mriedem: thanks for the tempest test offering in advance | 21:41 |
| dansmith | alaski: eh? | 21:41 |
| alaski | dansmith: cell0 plus a single cell that can host instances is the minimum setup | 21:41 |
| dansmith | alaski: in a single cell, you've got the api db, the cell0 db and the actual cell db right? | 21:41 |
| dansmith | right, okay, thought you meant something else | 21:42 |
| *** lpetrut has quit IRC | 21:42 | |
| dansmith | I also feel like we'll have a LOT fewer single-cell deployments when it's trivial to add a cell | 21:42 |
| dansmith | we have people with like 32 compute nodes that want to have cells | 21:42 |
| alaski | it's just a slightly larger jump from 0 cells to two for the migration path | 21:42 |
| dansmith | alaski: what's the actual cost though? another "create database foo;" call? | 21:43 |
| *** salv-orlando has quit IRC | 21:43 | |
| alaski | yeah. it's not much to ask, just one additional step or so | 21:43 |
| dansmith | alaski: steps are automated :) | 21:44 |
| *** salv-orlando has joined #openstack-nova | 21:44 | |
| dansmith | alaski: you could also offer the option to make cell0 = cells[0] | 21:44 |
| dansmith | alaski: and just collect your cruft into one of the actual cells | 21:44 |
| *** gyee has joined #openstack-nova | 21:44 | |
| doffm | dansmith: Thats the other option. Make one cell the dump cell. | 21:44 |
| dansmith | the point is just that we need a place when we can't pick a cell based on resources | 21:44 |
| *** gjayavelu has joined #openstack-nova | 21:44 | |
| dansmith | doffm: as long as it's configurable to be a separate place, because I expect a lot of people would rather purge the dumping ground frequently | 21:45 |
| alaski | that's an interesting thought. I'd prefer separate, but like using a cell better than colocating with the api db | 21:45 |
| doffm | dansmith: Thats true. | 21:45 |
| *** erlon has quit IRC | 21:46 | |
| *** baoli has joined #openstack-nova | 21:47 | |
| doffm | alaski: The work for cell0 is reduced then right, basically to a configuration for the cell db to use as cell0 and the work to put cells in error state there? | 21:48 |
| dansmith | doffm: you mean instances | 21:48 |
| doffm | dansmith: I do. :) | 21:48 |
| dansmith | doffm: I don't think it's necessarily any reduction | 21:48 |
| doffm | Put instances in error state there. | 21:48 |
| dansmith | doffm: or do you mean for the operator? | 21:48 |
| doffm | For us. | 21:49 |
| dansmith | I don't see how it's less work for us | 21:49 |
| *** pratikmallya has joined #openstack-nova | 21:49 | |
| doffm | Ok. | 21:49 |
| doffm | Sorry to ask these 'design' related questions when things have already been worked on for so long. | 21:50 |
| doffm | Trying to understand it so I can start helping out. | 21:51 |
| alaski | the questions are good | 21:51 |
| dansmith | this design point isn't that old actually | 21:51 |
| alaski | this was just from the last summit | 21:51 |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "guestfs" https://review.openstack.org/248963 | 21:53 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Add better help to the guestfs option https://review.openstack.org/255578 | 21:53 |
| *** pratikmallya has quit IRC | 21:54 | |
| doffm | alaski: dansmith: If we are going to have a special cell0 for error state instances why not for 'building' instances? There is alot of code for getting Instance object from BuildRequest objects. | 21:54 |
| doffm | Why dont the pre-scheduled instances go in to cell0 also? | 21:55 |
| dansmith | doffm: because moving them between databases is a bad thing | 21:55 |
| *** ctrath has quit IRC | 21:55 | |
| rlrossit | dansmith: I dun be stumped on why https://review.openstack.org/#/c/265951/ is failing Jenkins... no (relevant) errors in the api (or any other) logs http://logs.openstack.org/51/265951/2/check/gate-tempest-dsvm-full/ea165dd/logs/screen-n-api.txt.gz?level=ERROR | 21:55 |
| doffm | dansmith: Ok. :) | 21:55 |
| dansmith | doffm: again, with the abstraction.. requests are at the api level, instances are at the cell level.. we don't move one from one db to the other, we merely create an instance in response to a request | 21:55 |
| dansmith | rlrossit: nice | 21:56 |
| *** diogogmt has quit IRC | 21:56 | |
| dansmith | rlrossit: you know that's not some other failure? | 21:56 |
| *** belmoreira has joined #openstack-nova | 21:56 | |
| rlrossit | dansmith: well, that's literally the exact same test failing that I then fixed up in PS2... | 21:57 |
| alaski | doffm: also for abstractions this helps clarify that the input to the scheduler is not an instance, you don't need an instance until you have somewhere to put it | 21:57 |
| dansmith | rlrossit: this related? http://logs.openstack.org/51/265951/2/check/gate-tempest-dsvm-full/ea165dd/logs/screen-n-cond.txt.gz?level=TRACE | 21:57 |
| alaski | doffm: the contortions to get an instance from a buildrequest is because we have to pretend we have an instance for the sake of the API | 21:57 |
| dansmith | rlrossit: maybe fallout | 21:58 |
| doffm | alaski: dansmith: Seems like the right thing to do. I'm just thinking from an existing code point-of-view writing the instance fits better with whats (Wrongly) already there. | 21:58 |
| rlrossit | dansmith: but those aren't even the right tests... | 21:58 |
| dansmith | rlrossit: okay | 21:58 |
| *** baoli has quit IRC | 21:58 | |
| rlrossit | dansmith: I'm just expecting an AttributeError somewhere telling me I forgot another __getitem__.... | 21:58 |
| dansmith | doffm: remember, we're coming at this from years of scars from the "existing code" :) | 21:58 |
| doffm | :) | 21:59 |
| dansmith | rlrossit: well, unless it's failing somewhere, triggering an error path and not logging what the original problem was | 21:59 |
| *** baoli has joined #openstack-nova | 21:59 | |
| * rlrossit grumbles | 21:59 | |
| rlrossit | eating AttributeError == evil | 21:59 |
| dansmith | rlrossit: because it's failing to save instances, any chance we're passing an object down to the db and it's treating it like a dict? | 21:59 |
| *** carey has joined #openstack-nova | 21:59 | |
| *** eil397 has quit IRC | 22:00 | |
| jroll | dansmith: random question, when removing an rpc endpoint, it should be deprecated in the server, obviously, but can it just be deleted from the client I guess? | 22:01 |
| jroll | s/random/dumb/ | 22:01 |
| jroll | since there's no (in-tree) callers | 22:01 |
| dansmith | jroll: yep | 22:01 |
| jroll | cool, ty | 22:02 |
| *** ctrath has joined #openstack-nova | 22:02 | |
| dansmith | jroll: that helps ensure it doesn't grow a new user before you can remove it | 22:02 |
| jroll | right | 22:02 |
| dansmith | jroll: hard part is remembering to remove it and do the major bump | 22:02 |
| dansmith | jroll: so we usually put a comment above it with the actual version in which it can be removed | 22:02 |
| jroll | yeah | 22:02 |
| dansmith | # NOTE(danms): Remove this in version 5.0 of the RPC API | 22:02 |
| jroll | nod | 22:02 |
| jroll | and it can be removed the following cycle yeah? | 22:02 |
| rlrossit | dansmith: oh gosh if this is caused by lack of unit tests in the db api... more rabbit holes | 22:03 |
| jroll | (or 3 months) | 22:03 |
| dansmith | jroll: we bump the major on cycle boundaries, supporting both the new and old one and then remove the old one in the following | 22:04 |
| dansmith | jroll: so if we bumped for mitaka, we'd do it right before we release, | 22:04 |
| dansmith | sending version 5.0, but tolerating 4.x and 5.0 on the server side | 22:04 |
| dansmith | then as soon as we cut mitaka, we drop 4.x support | 22:04 |
| *** carey has quit IRC | 22:05 | |
| *** tsandquist has quit IRC | 22:05 | |
| dansmith | jroll: so we support liberty and mitaka clients in mitaka, but only 5.x (mitaka+) in N | 22:05 |
| *** szhsong has quit IRC | 22:05 | |
| *** krtaylor has joined #openstack-nova | 22:05 | |
| dansmith | jroll: you could do the bump every time even if you barely need to, but in practice we do it when the minor version gets large enough to result in dropping significant cruft, | 22:06 |
| dansmith | or to get rid of something we hate holding | 22:06 |
| *** aginwala has quit IRC | 22:06 | |
| jroll | dansmith: yeah, makes sense. thanks | 22:06 |
| *** szhsong has joined #openstack-nova | 22:07 | |
| *** davidlenwell has quit IRC | 22:09 | |
| *** smatzek has joined #openstack-nova | 22:09 | |
| *** salv-orl_ has joined #openstack-nova | 22:09 | |
| *** thorst has quit IRC | 22:10 | |
| *** baoli has quit IRC | 22:10 | |
| *** thorst has joined #openstack-nova | 22:10 | |
| *** baoli has joined #openstack-nova | 22:11 | |
| *** davidlenwell has joined #openstack-nova | 22:11 | |
| *** eharney has quit IRC | 22:12 | |
| *** salv-orlando has quit IRC | 22:12 | |
| *** szhsong has quit IRC | 22:12 | |
| *** rk4n has joined #openstack-nova | 22:13 | |
| *** matrohon has quit IRC | 22:14 | |
| *** yamahata has joined #openstack-nova | 22:14 | |
| *** szhsong has joined #openstack-nova | 22:14 | |
| *** dims has quit IRC | 22:15 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Persist the request spec during an instance boot https://review.openstack.org/258628 | 22:15 |
| *** admin0_ has joined #openstack-nova | 22:15 | |
| *** thorst has quit IRC | 22:15 | |
| *** eil397 has joined #openstack-nova | 22:16 | |
| *** cdent has quit IRC | 22:16 | |
| *** admin0_ has quit IRC | 22:16 | |
| *** annegentle has quit IRC | 22:16 | |
| rlrossit | if I wanted to work https://github.com/openstack/oslo.messaging/commit/6621b9010e6c46cfff72be4ce0f7f5c2cad81b45 to get a separate notification driver into nova, is that going to require a bp? or is that just "bringing in oslo goodness"? It will be changing some pretty low-level RPC stuff... | 22:17 |
| *** jamielennox|away is now known as jamielennox | 22:17 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 22:17 |
| *** pratikmallya has joined #openstack-nova | 22:18 | |
| *** diogogmt has joined #openstack-nova | 22:20 | |
| *** pratikma_ has joined #openstack-nova | 22:21 | |
| *** baoli has quit IRC | 22:22 | |
| *** baoli has joined #openstack-nova | 22:22 | |
| *** gangil has quit IRC | 22:24 | |
| *** pratikmallya has quit IRC | 22:24 | |
| *** szhsong has quit IRC | 22:25 | |
| *** szhsong has joined #openstack-nova | 22:25 | |
| *** omnipresent has quit IRC | 22:26 | |
| *** aginwala has joined #openstack-nova | 22:28 | |
| *** rosmaita has left #openstack-nova | 22:30 | |
| rlrossit | dansmith: found it! https://github.com/openstack/nova/blob/master/nova/network/floating_ips.py#L125 | 22:30 |
| dansmith | heh, cool | 22:30 |
| *** davideag_ has quit IRC | 22:31 | |
| *** davideagnello has joined #openstack-nova | 22:32 | |
| *** rk4n has quit IRC | 22:33 | |
| *** baoli has quit IRC | 22:34 | |
| *** rk4n has joined #openstack-nova | 22:34 | |
| *** baoli has joined #openstack-nova | 22:34 | |
| *** annegentle has joined #openstack-nova | 22:34 | |
| *** otter768 has joined #openstack-nova | 22:35 | |
| *** davideagnello has quit IRC | 22:35 | |
| rlrossit | dansmith: sooo... I don't think we test that function at all... https://github.com/openstack/nova/blob/master/nova/tests/unit/network/test_api.py#L193 | 22:35 |
| * rlrossit jumps down the rabbit hole | 22:35 | |
| dansmith | rlrossit: not surprising, unfortunately :/ | 22:36 |
| *** davideagnello has joined #openstack-nova | 22:36 | |
| dansmith | rlrossit: but super awesome karma for you fixing this *and* adding tests :) | 22:36 |
| *** belmoreira has quit IRC | 22:36 | |
| rlrossit | yeah, this whole dict compat thing is helping to find quite a few holes | 22:36 |
| *** ociuhandu has joined #openstack-nova | 22:36 | |
| dansmith | like tickets at the arcade.. not useful for much but more has to be better | 22:36 |
| rlrossit | dansmith: can I just open a bug saying it's untested, assign it to myself, then go home? | 22:37 |
| rlrossit | I don't feel like fixing that now | 22:37 |
| dansmith | or just fix it with no bug tomorrow if you wish | 22:37 |
| dansmith | but .. sure :) | 22:37 |
| rlrossit | I want a dumping ground so I don't forget what I'm doing ;) | 22:37 |
| dansmith | dump away | 22:37 |
| *** ccarmack has left #openstack-nova | 22:37 | |
| *** zenoway has joined #openstack-nova | 22:38 | |
| *** otter768 has quit IRC | 22:39 | |
| *** armax has joined #openstack-nova | 22:42 | |
| *** zenoway has quit IRC | 22:42 | |
| *** baoli has quit IRC | 22:46 | |
| *** gangil has joined #openstack-nova | 22:47 | |
| *** gangil has joined #openstack-nova | 22:47 | |
| *** baoli has joined #openstack-nova | 22:47 | |
| *** w_verdugo has quit IRC | 22:48 | |
| *** gjayavelu has quit IRC | 22:49 | |
| *** smatzek has quit IRC | 22:50 | |
| *** thorst has joined #openstack-nova | 22:51 | |
| *** thorst_ has joined #openstack-nova | 22:53 | |
| *** eliqiao has quit IRC | 22:54 | |
| *** davideagnello has quit IRC | 22:54 | |
| *** tjones has left #openstack-nova | 22:54 | |
| *** mriedem has quit IRC | 22:54 | |
| *** eliqiao has joined #openstack-nova | 22:55 | |
| *** thorst has quit IRC | 22:56 | |
| *** davideagnello has joined #openstack-nova | 22:56 | |
| *** baoli has quit IRC | 22:58 | |
| *** baoli has joined #openstack-nova | 22:58 | |
| *** aginwala has quit IRC | 23:00 | |
| rlrossit | dansmith: ok... I guess that's not it. It turns out there's also <class 'nova.network.model.FixedIP'> which uses dict syntaxing, so that isn't the source of my bug... | 23:00 |
| rlrossit | but, it's 5 o'clock somewhere (specifically, here), so I'm going home | 23:01 |
| dansmith | ah yeah | 23:01 |
| rlrossit | I'll leave that for me to get more frustrated tomorrow | 23:01 |
| dansmith | sounds like a plan | 23:01 |
| dansmith | I recommend deferring frustration | 23:02 |
| *** zenoway has joined #openstack-nova | 23:02 | |
| *** mdrabe has quit IRC | 23:02 | |
| *** burgerk has quit IRC | 23:03 | |
| *** tonytan4ever has quit IRC | 23:04 | |
| *** rlrossit has quit IRC | 23:05 | |
| *** aginwala has joined #openstack-nova | 23:05 | |
| *** armax has quit IRC | 23:05 | |
| *** Swanson has left #openstack-nova | 23:05 | |
| *** armax has joined #openstack-nova | 23:06 | |
| *** zenoway has quit IRC | 23:07 | |
| *** markmcclain has quit IRC | 23:07 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available https://review.openstack.org/205282 | 23:07 |
| *** pradk has quit IRC | 23:08 | |
| *** markmcclain has joined #openstack-nova | 23:09 | |
| *** armax has quit IRC | 23:10 | |
| *** baoli has quit IRC | 23:10 | |
| *** baoli has joined #openstack-nova | 23:10 | |
| *** armax has joined #openstack-nova | 23:11 | |
| *** armax has quit IRC | 23:11 | |
| *** burt has quit IRC | 23:11 | |
| cfriesen | if I look at nova-compute I see something like 20 OS-level threads. where are they coming from? is this being done by eventlet? | 23:12 |
| *** rk4n has quit IRC | 23:12 | |
| *** rk4n has joined #openstack-nova | 23:12 | |
| *** edtubill has quit IRC | 23:13 | |
| *** rk4n has quit IRC | 23:14 | |
| *** davideagnello has quit IRC | 23:14 | |
| *** dave-mccowan has quit IRC | 23:15 | |
| *** mragupat has quit IRC | 23:15 | |
| *** davideagnello has joined #openstack-nova | 23:16 | |
| *** davidlenwell has quit IRC | 23:17 | |
| *** kfarr has quit IRC | 23:17 | |
| *** davidlenwell has joined #openstack-nova | 23:18 | |
| *** armax has joined #openstack-nova | 23:18 | |
| *** gjayavelu has joined #openstack-nova | 23:20 | |
| *** aginwala has quit IRC | 23:20 | |
| *** Sukhdev has joined #openstack-nova | 23:22 | |
| *** baoli has quit IRC | 23:22 | |
| *** baoli has joined #openstack-nova | 23:23 | |
| *** markmcclain has quit IRC | 23:25 | |
| *** markmcclain has joined #openstack-nova | 23:26 | |
| *** e0ne has quit IRC | 23:26 | |
| *** zenoway has joined #openstack-nova | 23:27 | |
| *** xyang1 has quit IRC | 23:27 | |
| *** dimtruck is now known as zz_dimtruck | 23:28 | |
| *** browne has quit IRC | 23:29 | |
| *** annegentle has quit IRC | 23:31 | |
| *** zenoway has quit IRC | 23:31 | |
| *** eliqiao has quit IRC | 23:31 | |
| *** eliqiao has joined #openstack-nova | 23:32 | |
| *** aginwala has joined #openstack-nova | 23:32 | |
| *** armax has quit IRC | 23:34 | |
| *** baoli has quit IRC | 23:34 | |
| *** baoli has joined #openstack-nova | 23:35 | |
| *** armax has joined #openstack-nova | 23:35 | |
| auggy | i have a silly openstack question, i'm trying to verify this bug - https://bugs.launchpad.net/nova/+bug/1521467 | 23:36 |
| openstack | Launchpad bug 1521467 in OpenStack Compute (nova) "Snapshot of volume base instance shows type as image not snapshot and size as 0" [Undecided,New] | 23:36 |
| auggy | the reporter says they listed the images and found the type of holly-vm1-snapshot is "Image", which should be "Snapshot" | 23:37 |
| auggy | but the ways i know to list images don't show a type | 23:37 |
| auggy | or the type they show isn't snapshot or image | 23:37 |
| auggy | nova image-list doesn't contain that field, cinder list only contains a volume-type field, and that's not technically listing images i think, and glance image-list only contains id and name | 23:38 |
| auggy | i am happy to ask the reporter to clarify but i didn't want to do that if there's some obvious thing i'm missing here | 23:39 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments https://review.openstack.org/193135 | 23:39 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 23:39 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db and obj: Allow multiple volume attachments https://review.openstack.org/193133 | 23:39 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 23:39 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 23:39 |
| *** armax has quit IRC | 23:40 | |
| alex_xu | alaski: thanks for the +2 | 23:40 |
| alex_xu | dansmith: hey, I'm here now if you still wake up | 23:40 |
| dansmith | alex_xu: just wanted you to take another pass on those live migration patches | 23:41 |
| *** armax has joined #openstack-nova | 23:41 | |
| alex_xu | dansmith: ok, I will do that today | 23:41 |
| dansmith | alex_xu: I just added those two attributes to the object and converted the post path | 23:41 |
| dansmith | alex_xu: we had some random gate failures today, let me see about rechecking them | 23:41 |
| alex_xu | dansmith: cool :) | 23:41 |
| *** logan2 has joined #openstack-nova | 23:42 | |
| alex_xu | dansmith: I have question about libvirtmigrationdata obj. | 23:42 |
| mtreinish | auggy: I would assume that bug is referring to cinder volume type since it's talking about boot from volume which should be doing cinder snapshots iirc | 23:42 |
| dansmith | alex_xu: okay | 23:42 |
| *** logan2 is now known as logan- | 23:42 | |
| auggy | mtreinish: yeah it was just confusing because the repro steps specifically said list the images? idk, i just marked as incomplete since they'd mentioned an attachment with a screenshot and it's missing | 23:42 |
| alex_xu | dansmith: this patch https://review.openstack.org/254101 want to change the block_migration field's nullable. I guess we can't change that directly, right? | 23:42 |
| dansmith | alex_xu: yeah, because if we have null for those values we can't send them to older clients | 23:43 |
| * auggy has a little tech support experience lol | 23:43 | |
| dansmith | alex_xu: so that would break talking to those older clients (i.e. live migrations between those two) | 23:44 |
| *** armax has quit IRC | 23:44 | |
| *** szhsong has quit IRC | 23:44 | |
| alex_xu | dansmith: but for now, your patch https://review.openstack.org/251588 didn't merge yet, do we have chance change that easily for now? or just create another field? | 23:44 |
| auggy | mtreinish: i volunteered for bug skimming duty which technically doesn't start until thursday but i figured i'd get a head start ;) | 23:44 |
| *** akshai has quit IRC | 23:44 | |
| *** szhsong has joined #openstack-nova | 23:45 | |
| dansmith | alex_xu: well, we probably could because we don't send it over the wire yet, BUT, the proper way is not to pass None there, but to leave those unset I think | 23:45 |
| *** armax has joined #openstack-nova | 23:45 | |
| dansmith | alex_xu: being unset and calculated is better than always passing None from here on out | 23:45 |
| dansmith | alex_xu: leaving them unset also lets us deprecate and remove those two fields later much easier | 23:45 |
| *** gangil has quit IRC | 23:46 | |
| mtreinish | auggy: there is a metadef thing in glance which maps resources associated with an image, but as far as I know there isn't an image "type" (besides the disk format etc, which doesn't sound related) | 23:46 |
| *** gangil has joined #openstack-nova | 23:46 | |
| *** gangil has joined #openstack-nova | 23:46 | |
| dansmith | alex_xu: so instead of him doing "if md.block_migration == None" he should do "if 'block_migration' not in md" | 23:46 |
| *** baoli has quit IRC | 23:46 | |
| dansmith | alex_xu: I can comment | 23:46 |
| alex_xu | dansmith: ok, got it, thanks a lot :) | 23:46 |
| mtreinish | auggy: heh, well good luck with that. I normally have a very low tolerance for dealing with bug triage :) | 23:46 |
| *** baoli has joined #openstack-nova | 23:47 | |
| mtreinish | I rage quit after a couple bad ones in a row | 23:47 |
| auggy | mtreinish: hah it's all part of my wax-on wax-off learning about nova process ;D | 23:47 |
| *** armax has quit IRC | 23:47 | |
| *** armax has joined #openstack-nova | 23:47 | |
| *** angdraug has quit IRC | 23:48 | |
| auggy | mtreinish: i think in this case, a screenshot would clarify what they are talking about... i could be wrong but they mentioned using xiv and i wonder if it's going to end up being an issue with that.. | 23:48 |
| *** smoriya_ has joined #openstack-nova | 23:48 | |
| *** oomichi_away is now known as oomichi | 23:49 | |
| auggy | anyways, on to the next bug :D thanks | 23:49 |
| mtreinish | auggy: heh, well if it is an xiv issue, it's unlikely to be a nova bug then :) | 23:49 |
| auggy | yeah exactly | 23:50 |
| *** ducttape_ has quit IRC | 23:50 | |
| *** armax has quit IRC | 23:51 | |
| *** zenoway has joined #openstack-nova | 23:51 | |
| *** mrkz has quit IRC | 23:53 | |
| *** annegentle has joined #openstack-nova | 23:54 | |
| *** zenoway has quit IRC | 23:55 | |
| *** SourabhP has quit IRC | 23:56 | |
| *** baoli has quit IRC | 23:58 | |
| *** baoli has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!