| *** baoli has quit IRC | 00:00 | |
| *** gtt116__ has joined #openstack-nova | 00:00 | |
| *** dgonzalez has joined #openstack-nova | 00:01 | |
| *** agireud has quit IRC | 00:02 | |
| *** thorst has quit IRC | 00:03 | |
| *** gtt116_ has quit IRC | 00:04 | |
| *** agireud has joined #openstack-nova | 00:04 | |
| *** thorst has joined #openstack-nova | 00:05 | |
| *** thorst has quit IRC | 00:06 | |
| *** zenoway has joined #openstack-nova | 00:10 | |
| *** otter768 has quit IRC | 00:11 | |
| *** agireud has quit IRC | 00:12 | |
| *** zenoway has quit IRC | 00:14 | |
| *** thorst has joined #openstack-nova | 00:15 | |
| *** dgonzalez has quit IRC | 00:16 | |
| *** dgonzalez has joined #openstack-nova | 00:17 | |
| *** dgonzalez has quit IRC | 00:17 | |
| *** dgonzalez has joined #openstack-nova | 00:18 | |
| *** otter768 has joined #openstack-nova | 00:19 | |
| *** dgonzalez has quit IRC | 00:20 | |
| *** dgonzalez has joined #openstack-nova | 00:23 | |
| *** dgonzalez has quit IRC | 00:29 | |
| *** dgonzalez has joined #openstack-nova | 00:29 | |
| *** dgonzalez has quit IRC | 00:30 | |
| *** dgonzalez has joined #openstack-nova | 00:30 | |
| *** otter768 has quit IRC | 00:30 | |
| *** dgonzalez has quit IRC | 00:31 | |
| *** dgonzalez has joined #openstack-nova | 00:32 | |
| *** dgonzalez has quit IRC | 00:32 | |
| *** dgonzalez has joined #openstack-nova | 00:33 | |
| *** dgonzalez has quit IRC | 00:34 | |
| *** dgonzalez has joined #openstack-nova | 00:34 | |
| *** dgonzalez has quit IRC | 00:35 | |
| *** dgonzalez has joined #openstack-nova | 00:37 | |
| *** rotbeard has quit IRC | 00:40 | |
| *** ducttape_ has quit IRC | 00:43 | |
| *** w_verdugo has quit IRC | 00:44 | |
| *** Sukhdev has joined #openstack-nova | 00:45 | |
| *** zenoway has joined #openstack-nova | 00:45 | |
| *** Sukhdev has quit IRC | 00:46 | |
| *** bklei has joined #openstack-nova | 00:47 | |
| *** zhangjn has joined #openstack-nova | 00:47 | |
| *** zenoway has quit IRC | 00:50 | |
| *** ducttape_ has joined #openstack-nova | 00:52 | |
| *** thorst has quit IRC | 00:53 | |
| *** zhangjn_ has joined #openstack-nova | 00:53 | |
| *** zhangjn has quit IRC | 00:53 | |
| *** thorst has joined #openstack-nova | 00:53 | |
| *** ducttape_ has quit IRC | 00:54 | |
| *** _ducttape_ has joined #openstack-nova | 00:54 | |
| *** bklei has quit IRC | 00:56 | |
| *** houming has joined #openstack-nova | 01:00 | |
| *** thorst has quit IRC | 01:02 | |
| *** dims has quit IRC | 01:03 | |
| *** mriedem_away has quit IRC | 01:04 | |
| *** _ducttape_ has quit IRC | 01:05 | |
| *** agireud has joined #openstack-nova | 01:10 | |
| *** markvoelker has joined #openstack-nova | 01:14 | |
| *** markvoelker has quit IRC | 01:19 | |
| *** EinstCrazy has joined #openstack-nova | 01:21 | |
| *** otter768 has joined #openstack-nova | 01:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 01:27 |
|---|---|---|
| *** agireud has quit IRC | 01:28 | |
| *** agireud has joined #openstack-nova | 01:31 | |
| *** yonglihe has joined #openstack-nova | 01:34 | |
| *** otter768 has quit IRC | 01:37 | |
| *** dims has joined #openstack-nova | 01:37 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP]Add config option "scheduler_compute_node_update_interval" for scheduler https://review.openstack.org/260920 | 01:40 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add expected_task_state for instance.save() used by block_device_mapping https://review.openstack.org/245038 | 01:44 |
| *** ljxiash has joined #openstack-nova | 01:50 | |
| *** rfolco has joined #openstack-nova | 01:53 | |
| *** ljxiash has quit IRC | 01:55 | |
| *** thorst has joined #openstack-nova | 01:59 | |
| *** zhangjn_ has quit IRC | 02:03 | |
| *** agireud has quit IRC | 02:05 | |
| *** ducttape_ has joined #openstack-nova | 02:06 | |
| *** thorst has quit IRC | 02:07 | |
| *** dims has quit IRC | 02:07 | |
| *** agireud has joined #openstack-nova | 02:08 | |
| *** ducttape_ has quit IRC | 02:10 | |
| *** zhangyufei has joined #openstack-nova | 02:10 | |
| *** otter768 has joined #openstack-nova | 02:13 | |
| *** rfolco has quit IRC | 02:14 | |
| *** zhangyufei has quit IRC | 02:15 | |
| *** ljxiash has joined #openstack-nova | 02:15 | |
| *** houming has quit IRC | 02:21 | |
| *** Sukhdev has joined #openstack-nova | 02:21 | |
| *** houming has joined #openstack-nova | 02:21 | |
| *** zhangyufei has joined #openstack-nova | 02:23 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Support neutron security group https://review.openstack.org/251271 | 02:30 |
| *** armax has joined #openstack-nova | 02:32 | |
| *** lyanchih has joined #openstack-nova | 02:35 | |
| *** agireud has quit IRC | 02:38 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 02:39 |
| *** agireud has joined #openstack-nova | 02:40 | |
| *** zhangjn has joined #openstack-nova | 02:43 | |
| *** otter768 has quit IRC | 02:44 | |
| *** zhangjn has quit IRC | 02:45 | |
| *** zz_dimtruck is now known as dimtruck | 02:47 | |
| *** otter768 has joined #openstack-nova | 02:47 | |
| *** zhangyufei has quit IRC | 02:47 | |
| *** salv-orlando has quit IRC | 02:52 | |
| *** salv-orlando has joined #openstack-nova | 02:53 | |
| *** markvoelker has joined #openstack-nova | 02:53 | |
| *** zhangjn has joined #openstack-nova | 02:56 | |
| *** dimtruck is now known as zz_dimtruck | 02:57 | |
| *** rushiagr_away is now known as rushiagr | 02:58 | |
| *** zhangjn has quit IRC | 03:01 | |
| *** zz_dimtruck is now known as dimtruck | 03:02 | |
| *** zhangjn has joined #openstack-nova | 03:03 | |
| *** zhangjn has quit IRC | 03:04 | |
| *** thorst has joined #openstack-nova | 03:04 | |
| *** otter768 has quit IRC | 03:05 | |
| *** zhangjn has joined #openstack-nova | 03:08 | |
| *** agireud has quit IRC | 03:10 | |
| *** zhangjn has quit IRC | 03:12 | |
| *** thorst has quit IRC | 03:12 | |
| *** agireud has joined #openstack-nova | 03:13 | |
| *** dimtruck is now known as zz_dimtruck | 03:15 | |
| *** haomaiwang has joined #openstack-nova | 03:16 | |
| *** zhangjn has joined #openstack-nova | 03:17 | |
| *** yangyapeng has joined #openstack-nova | 03:17 | |
| *** mragupat has joined #openstack-nova | 03:19 | |
| *** rushiagr is now known as rushiagr_away | 03:22 | |
| *** mragupat has quit IRC | 03:22 | |
| *** lyanchih has quit IRC | 03:22 | |
| *** liusheng has quit IRC | 03:22 | |
| *** otter768 has joined #openstack-nova | 03:23 | |
| *** mragupat has joined #openstack-nova | 03:23 | |
| *** liusheng has joined #openstack-nova | 03:23 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: update feature support matrix documentation https://review.openstack.org/262931 | 03:28 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: enable uefi boot for x86_64 https://review.openstack.org/262930 | 03:28 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: add "hw_firmware_type" image metadata https://review.openstack.org/262929 | 03:28 |
| *** zz_dimtruck is now known as dimtruck | 03:38 | |
| *** ljxiash has quit IRC | 03:39 | |
| *** ljxiash has joined #openstack-nova | 03:39 | |
| *** zhangjn has quit IRC | 03:42 | |
| *** ljxiash has quit IRC | 03:44 | |
| *** links has joined #openstack-nova | 03:48 | |
| *** ducttape_ has joined #openstack-nova | 03:49 | |
| openstackgerrit | Felix Ma proposed openstack/nova: Catch exceptions from _destroy_evacuated_instances https://review.openstack.org/265681 | 03:49 |
| *** vilobhmm111 has joined #openstack-nova | 03:52 | |
| *** ducttape_ has quit IRC | 03:53 | |
| *** vilobhmm11 has quit IRC | 03:55 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** jwcroppe has joined #openstack-nova | 04:02 | |
| *** rushiagr_away is now known as rushiagr | 04:06 | |
| *** salv-orl_ has joined #openstack-nova | 04:09 | |
| *** thorst has joined #openstack-nova | 04:09 | |
| *** salv-orlando has quit IRC | 04:11 | |
| *** ljxiash has joined #openstack-nova | 04:16 | |
| *** thorst has quit IRC | 04:17 | |
| *** hunters1094 has joined #openstack-nova | 04:20 | |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: update feature support matrix documentation https://review.openstack.org/262931 | 04:24 |
| openstackgerrit | Ren Qiaowei proposed openstack/nova: enable uefi boot for x86_64 https://review.openstack.org/262930 | 04:24 |
| *** vilobhmm111 has quit IRC | 04:26 | |
| *** agireud has quit IRC | 04:26 | |
| hunters1094 | hi all | 04:26 |
| hunters1094 | i have problem with nova encrypted volume | 04:27 |
| hunters1094 | can anyone help? | 04:27 |
| *** agireud has joined #openstack-nova | 04:28 | |
| *** davidlenwell_ is now known as davidlenwell | 04:35 | |
| *** haomaiwang has quit IRC | 04:37 | |
| *** agireud has quit IRC | 04:39 | |
| *** Jialiang has joined #openstack-nova | 04:39 | |
| *** agireud has joined #openstack-nova | 04:41 | |
| *** coolsvap|away is now known as coolsvap | 04:41 | |
| *** shyama has joined #openstack-nova | 04:43 | |
| *** klkumar has joined #openstack-nova | 04:46 | |
| *** agireud has quit IRC | 04:48 | |
| *** ducttape_ has joined #openstack-nova | 04:50 | |
| *** suro-patz has joined #openstack-nova | 04:50 | |
| *** ljxiash has quit IRC | 04:51 | |
| *** suro-patz has quit IRC | 04:51 | |
| *** ljxiash has joined #openstack-nova | 04:51 | |
| *** agireud has joined #openstack-nova | 04:52 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy https://review.openstack.org/149625 | 04:52 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 04:53 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties https://review.openstack.org/149626 | 04:53 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 04:53 |
| *** otter768 has quit IRC | 04:53 | |
| *** otter768 has joined #openstack-nova | 04:54 | |
| *** ljxiash has quit IRC | 04:56 | |
| *** ducttape_ has quit IRC | 04:56 | |
| *** armax has quit IRC | 04:58 | |
| *** dimtruck is now known as zz_dimtruck | 04:58 | |
| *** EinstCrazy has quit IRC | 04:58 | |
| *** agireud has quit IRC | 04:59 | |
| *** david-lyle has quit IRC | 05:00 | |
| *** david-lyle has joined #openstack-nova | 05:00 | |
| *** agireud has joined #openstack-nova | 05:02 | |
| *** mragupat has quit IRC | 05:03 | |
| *** mragupat has joined #openstack-nova | 05:04 | |
| *** agireud has quit IRC | 05:06 | |
| *** EinstCrazy has joined #openstack-nova | 05:07 | |
| *** agireud has joined #openstack-nova | 05:09 | |
| *** otter768 has quit IRC | 05:10 | |
| *** zz_dimtruck is now known as dimtruck | 05:13 | |
| *** hunters1094 has quit IRC | 05:13 | |
| *** vishwanathj has joined #openstack-nova | 05:13 | |
| *** suro-patz has joined #openstack-nova | 05:13 | |
| *** thorst has joined #openstack-nova | 05:15 | |
| *** ljxiash has joined #openstack-nova | 05:17 | |
| *** thorst has quit IRC | 05:22 | |
| *** zhangjn has joined #openstack-nova | 05:28 | |
| *** vilobhmm11 has joined #openstack-nova | 05:29 | |
| *** gcb_ has joined #openstack-nova | 05:41 | |
| *** deepakcs has joined #openstack-nova | 05:42 | |
| *** lyanchih has joined #openstack-nova | 05:43 | |
| *** gcb has joined #openstack-nova | 05:43 | |
| *** _diana_1 has joined #openstack-nova | 05:45 | |
| *** suro-patz has quit IRC | 05:50 | |
| *** gcb_ has quit IRC | 05:50 | |
| *** jaosorior has joined #openstack-nova | 05:50 | |
| *** dedery has joined #openstack-nova | 05:52 | |
| *** dave-mccowan has joined #openstack-nova | 06:02 | |
| *** dave-mcc_ has joined #openstack-nova | 06:03 | |
| *** dave-mccowan has quit IRC | 06:06 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt - Add log if libguestfs can't read host kernel https://review.openstack.org/237547 | 06:10 |
| openstackgerrit | lyanchih proposed openstack/nova: [db] Instances named invalid regexp string can't been deleted https://review.openstack.org/255130 | 06:11 |
| *** _diana_1 has quit IRC | 06:13 | |
| *** baoli_ has quit IRC | 06:15 | |
| *** dimtruck is now known as zz_dimtruck | 06:18 | |
| *** thorst has joined #openstack-nova | 06:19 | |
| *** dave-mcc_ has quit IRC | 06:21 | |
| *** nkrinner has joined #openstack-nova | 06:23 | |
| *** thorst has quit IRC | 06:27 | |
| *** mragupat has quit IRC | 06:27 | |
| *** salv-orl_ has quit IRC | 06:31 | |
| *** esracelik has joined #openstack-nova | 06:31 | |
| *** salv-orlando has joined #openstack-nova | 06:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/265601 | 06:32 |
| *** ankit_ag has joined #openstack-nova | 06:34 | |
| *** zhangjn has quit IRC | 06:36 | |
| *** EinstCrazy has quit IRC | 06:38 | |
| *** EinstCrazy has joined #openstack-nova | 06:38 | |
| *** lpetrut has joined #openstack-nova | 06:40 | |
| *** zhangjn has joined #openstack-nova | 06:41 | |
| *** _diana_1 has joined #openstack-nova | 06:42 | |
| *** bkumar has joined #openstack-nova | 06:48 | |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add version discovery guideline in api concept doc https://review.openstack.org/264044 | 06:50 |
| openstackgerrit | Alex Xu proposed openstack/nova: doc: add client interactive guideline for microversions https://review.openstack.org/264045 | 06:50 |
| *** EinstCrazy has quit IRC | 06:56 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Merge os-user-data extension into servers https://review.openstack.org/258235 | 06:57 |
| *** EinstCrazy has joined #openstack-nova | 06:57 | |
| *** mjura has joined #openstack-nova | 06:58 | |
| *** EinstCrazy has quit IRC | 06:59 | |
| *** achinm has joined #openstack-nova | 06:59 | |
| *** Marga_ has joined #openstack-nova | 07:01 | |
| *** jwcroppe has quit IRC | 07:02 | |
| *** EinstCrazy has joined #openstack-nova | 07:02 | |
| *** suro-patz has joined #openstack-nova | 07:02 | |
| *** EinstCrazy has quit IRC | 07:05 | |
| *** yamahata has quit IRC | 07:05 | |
| *** EinstCrazy has joined #openstack-nova | 07:06 | |
| *** EinstCrazy has quit IRC | 07:07 | |
| *** EinstCrazy has joined #openstack-nova | 07:07 | |
| *** scheuran has joined #openstack-nova | 07:07 | |
| *** EinstCrazy has quit IRC | 07:08 | |
| *** EinstCrazy has joined #openstack-nova | 07:09 | |
| *** EinstCrazy has quit IRC | 07:10 | |
| *** otter768 has joined #openstack-nova | 07:11 | |
| *** EinstCrazy has joined #openstack-nova | 07:11 | |
| *** gcb has quit IRC | 07:12 | |
| *** EinstCrazy has quit IRC | 07:12 | |
| *** EinstCrazy has joined #openstack-nova | 07:13 | |
| *** EinstCrazy has quit IRC | 07:14 | |
| *** EinstCrazy has joined #openstack-nova | 07:15 | |
| *** irenab has joined #openstack-nova | 07:15 | |
| *** otter768 has quit IRC | 07:16 | |
| *** Marga_ has quit IRC | 07:17 | |
| *** EinstCrazy has quit IRC | 07:17 | |
| *** Marga_ has joined #openstack-nova | 07:18 | |
| *** lpetrut has quit IRC | 07:19 | |
| *** markvoelker has quit IRC | 07:22 | |
| *** EinstCrazy has joined #openstack-nova | 07:22 | |
| *** vilobhmm11 has quit IRC | 07:24 | |
| *** thorst has joined #openstack-nova | 07:27 | |
| *** Marga__ has joined #openstack-nova | 07:30 | |
| *** Marga_ has quit IRC | 07:30 | |
| *** paul-carlton2 has joined #openstack-nova | 07:31 | |
| *** thorst has quit IRC | 07:32 | |
| *** liverpooler has joined #openstack-nova | 07:33 | |
| *** Marga__ has quit IRC | 07:35 | |
| *** liverpooler has quit IRC | 07:35 | |
| *** sahid has joined #openstack-nova | 07:36 | |
| *** Jialiang has quit IRC | 07:37 | |
| *** suro-patz has quit IRC | 07:39 | |
| openstackgerrit | lyanchih proposed openstack/nova: [libvirt] Attaching volume to paused instance error is not clear https://review.openstack.org/252170 | 07:41 |
| *** sahid has quit IRC | 07:44 | |
| *** Marga_ has joined #openstack-nova | 07:46 | |
| *** Marga_ has quit IRC | 07:50 | |
| *** sahid has joined #openstack-nova | 07:51 | |
| *** tpatzig has joined #openstack-nova | 07:52 | |
| *** sahid has quit IRC | 08:00 | |
| *** mpavone has joined #openstack-nova | 08:00 | |
| *** sahid has joined #openstack-nova | 08:00 | |
| *** otter768 has joined #openstack-nova | 08:00 | |
| *** garyk has joined #openstack-nova | 08:00 | |
| *** liverpooler has joined #openstack-nova | 08:01 | |
| garyk | johnthetubaguy: can you please look at https://review.openstack.org/#/c/196171/10. you gave it a -1 due to the fact that there rae no release notes. the release notes were in a parallel patch. i have not based that on this. | 08:01 |
| *** otter768 has quit IRC | 08:01 | |
| *** otter768 has joined #openstack-nova | 08:02 | |
| *** otter768_ has joined #openstack-nova | 08:04 | |
| *** belmoreira has joined #openstack-nova | 08:05 | |
| *** otter768_ has quit IRC | 08:06 | |
| *** otter768_ has joined #openstack-nova | 08:06 | |
| *** otter768 has quit IRC | 08:07 | |
| *** ihrachys has joined #openstack-nova | 08:08 | |
| *** otter768 has joined #openstack-nova | 08:08 | |
| *** otter768_ has quit IRC | 08:08 | |
| *** rdopiera has joined #openstack-nova | 08:08 | |
| *** mpavone has quit IRC | 08:08 | |
| *** Sukhdev has quit IRC | 08:08 | |
| *** otter768_ has joined #openstack-nova | 08:10 | |
| *** otter768 has quit IRC | 08:10 | |
| openstackgerrit | Bo Chi proposed openstack/nova: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/265711 | 08:10 |
| *** otter768_ has quit IRC | 08:11 | |
| *** otter768 has joined #openstack-nova | 08:12 | |
| *** haomaiwang has joined #openstack-nova | 08:12 | |
| *** otter768 has quit IRC | 08:14 | |
| *** jlanoux has joined #openstack-nova | 08:14 | |
| *** otter768 has joined #openstack-nova | 08:15 | |
| *** otter768 has quit IRC | 08:17 | |
| *** otter768 has joined #openstack-nova | 08:17 | |
| *** jichen has joined #openstack-nova | 08:18 | |
| *** otter768 has quit IRC | 08:19 | |
| *** EinstCrazy has quit IRC | 08:19 | |
| *** EinstCrazy has joined #openstack-nova | 08:19 | |
| *** otter768 has joined #openstack-nova | 08:19 | |
| *** zakora has joined #openstack-nova | 08:20 | |
| *** markus_z has joined #openstack-nova | 08:20 | |
| *** otter768 has quit IRC | 08:21 | |
| *** otter768_ has joined #openstack-nova | 08:21 | |
| *** otter768 has joined #openstack-nova | 08:23 | |
| *** otter768_ has quit IRC | 08:23 | |
| *** matrohon has joined #openstack-nova | 08:26 | |
| *** jlanoux has quit IRC | 08:26 | |
| *** otter768_ has joined #openstack-nova | 08:27 | |
| *** haomaiwang has quit IRC | 08:27 | |
| *** jlanoux has joined #openstack-nova | 08:27 | |
| *** otter768 has quit IRC | 08:27 | |
| *** otter768_ has quit IRC | 08:28 | |
| *** boris-42 has joined #openstack-nova | 08:29 | |
| *** otter768 has joined #openstack-nova | 08:29 | |
| *** lpetrut has joined #openstack-nova | 08:30 | |
| *** sudipto has joined #openstack-nova | 08:30 | |
| *** thorst has joined #openstack-nova | 08:30 | |
| *** ildikov has quit IRC | 08:30 | |
| *** otter768 has quit IRC | 08:31 | |
| *** otter768_ has joined #openstack-nova | 08:31 | |
| *** openstackgerrit has quit IRC | 08:32 | |
| *** liverpoo1er has joined #openstack-nova | 08:33 | |
| *** openstackgerrit has joined #openstack-nova | 08:33 | |
| *** otter768_ has quit IRC | 08:33 | |
| *** otter768 has joined #openstack-nova | 08:33 | |
| *** liverpooler has quit IRC | 08:34 | |
| *** otter768 has quit IRC | 08:35 | |
| *** otter768 has joined #openstack-nova | 08:35 | |
| *** otter768 has quit IRC | 08:36 | |
| *** thorst has quit IRC | 08:36 | |
| *** otter768 has joined #openstack-nova | 08:37 | |
| *** liverpoo1er has quit IRC | 08:38 | |
| *** otter768 has quit IRC | 08:38 | |
| *** otter768_ has joined #openstack-nova | 08:38 | |
| *** liverpooler has joined #openstack-nova | 08:39 | |
| *** otter768 has joined #openstack-nova | 08:40 | |
| *** otter768_ has quit IRC | 08:40 | |
| *** tiagogomes__ has joined #openstack-nova | 08:42 | |
| *** otter768 has quit IRC | 08:42 | |
| bauwser | good morning Nova | 08:44 |
| *** bauwser is now known as bauzas | 08:44 | |
| *** lyanchih has quit IRC | 08:44 | |
| *** otter768 has joined #openstack-nova | 08:44 | |
| *** salv-orlando has quit IRC | 08:44 | |
| *** femnad has joined #openstack-nova | 08:45 | |
| *** salv-orlando has joined #openstack-nova | 08:45 | |
| *** otter768_ has joined #openstack-nova | 08:46 | |
| *** otter768_ has quit IRC | 08:48 | |
| *** otter76__ has joined #openstack-nova | 08:48 | |
| *** otter768 has quit IRC | 08:48 | |
| *** otter76__ has quit IRC | 08:50 | |
| *** otter768 has joined #openstack-nova | 08:50 | |
| *** otter768 has quit IRC | 08:51 | |
| *** otter768 has joined #openstack-nova | 08:52 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/nova: Keep py3.X compatibility for urllib https://review.openstack.org/261034 | 08:53 |
| *** otter768 has quit IRC | 08:53 | |
| *** otter768 has joined #openstack-nova | 08:54 | |
| *** rha has joined #openstack-nova | 08:54 | |
| *** rha has joined #openstack-nova | 08:54 | |
| *** markvoelker has joined #openstack-nova | 08:54 | |
| *** otter768 has quit IRC | 08:56 | |
| *** otter768_ has joined #openstack-nova | 08:56 | |
| *** otter768_ has quit IRC | 08:57 | |
| *** otter768 has joined #openstack-nova | 08:57 | |
| *** otter768 has quit IRC | 08:59 | |
| *** markvoelker has quit IRC | 08:59 | |
| *** otter768 has joined #openstack-nova | 09:00 | |
| *** otter768 has quit IRC | 09:01 | |
| *** otter768_ has joined #openstack-nova | 09:02 | |
| *** ociuhandu has quit IRC | 09:03 | |
| *** zhangjn has quit IRC | 09:03 | |
| *** derekh has joined #openstack-nova | 09:03 | |
| *** otter768 has joined #openstack-nova | 09:03 | |
| *** otter768_ has quit IRC | 09:03 | |
| *** paul-carlton2 has quit IRC | 09:05 | |
| *** otter768 has quit IRC | 09:05 | |
| *** otter768_ has joined #openstack-nova | 09:05 | |
| *** e0ne has joined #openstack-nova | 09:05 | |
| *** otter768_ has quit IRC | 09:07 | |
| *** lyanchih has joined #openstack-nova | 09:07 | |
| *** otter768 has joined #openstack-nova | 09:07 | |
| *** otter768 has quit IRC | 09:08 | |
| *** EinstCrazy has quit IRC | 09:09 | |
| *** zhangjn has joined #openstack-nova | 09:09 | |
| *** otter768 has joined #openstack-nova | 09:09 | |
| *** EinstCrazy has joined #openstack-nova | 09:09 | |
| *** EinstCrazy has quit IRC | 09:10 | |
| *** EinstCrazy has joined #openstack-nova | 09:11 | |
| *** otter768_ has joined #openstack-nova | 09:11 | |
| *** EinstCrazy has quit IRC | 09:11 | |
| *** otter768 has quit IRC | 09:11 | |
| *** EinstCrazy has joined #openstack-nova | 09:11 | |
| openstackgerrit | Chaozhe Chen proposed openstack/python-novaclient: Test: Clean v2 client userwarning https://review.openstack.org/265725 | 09:13 |
| *** otter768_ has quit IRC | 09:13 | |
| *** EinstCrazy has quit IRC | 09:13 | |
| *** EinstCrazy has joined #openstack-nova | 09:13 | |
| *** otter768 has joined #openstack-nova | 09:13 | |
| *** EinstCrazy has quit IRC | 09:13 | |
| *** EinstCrazy has joined #openstack-nova | 09:13 | |
| *** kashyap has joined #openstack-nova | 09:14 | |
| *** daemontool has joined #openstack-nova | 09:14 | |
| *** ildikov has joined #openstack-nova | 09:14 | |
| *** otter768 has quit IRC | 09:15 | |
| *** otter768_ has joined #openstack-nova | 09:15 | |
| *** EinstCrazy has quit IRC | 09:16 | |
| *** EinstCrazy has joined #openstack-nova | 09:16 | |
| *** otter768 has joined #openstack-nova | 09:17 | |
| *** otter76__ has joined #openstack-nova | 09:18 | |
| *** Jeffrey4l__ has quit IRC | 09:19 | |
| *** otter768_ has quit IRC | 09:20 | |
| *** otter76__ has quit IRC | 09:20 | |
| *** otter768_ has joined #openstack-nova | 09:21 | |
| *** otter768 has quit IRC | 09:21 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Changed filter_by() to filter() during filtering instances in db API https://review.openstack.org/245677 | 09:22 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support https://review.openstack.org/248805 | 09:22 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 09:22 |
| *** otter768 has joined #openstack-nova | 09:22 | |
| *** yassine__ has joined #openstack-nova | 09:23 | |
| *** acabot has quit IRC | 09:24 | |
| *** EinstCrazy has quit IRC | 09:24 | |
| *** acabot has joined #openstack-nova | 09:24 | |
| *** EinstCra_ has joined #openstack-nova | 09:24 | |
| *** otter76__ has joined #openstack-nova | 09:24 | |
| *** ndipanov has joined #openstack-nova | 09:25 | |
| *** daemontool has quit IRC | 09:25 | |
| *** jistr has joined #openstack-nova | 09:25 | |
| *** daemontool has joined #openstack-nova | 09:25 | |
| *** otter768_ has quit IRC | 09:25 | |
| *** otter76__ has quit IRC | 09:26 | |
| *** ndipanov has quit IRC | 09:26 | |
| *** otter768 has quit IRC | 09:27 | |
| *** paul-carlton2 has joined #openstack-nova | 09:27 | |
| *** daemontool has quit IRC | 09:27 | |
| *** otter768 has joined #openstack-nova | 09:28 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Moved tags filtering tests to TestInstanceTagsFiltering test case https://review.openstack.org/258515 | 09:29 |
| *** daemontool has joined #openstack-nova | 09:29 | |
| *** EinstCrazy has joined #openstack-nova | 09:30 | |
| *** otter768 has quit IRC | 09:30 | |
| *** EinstCra_ has quit IRC | 09:30 | |
| *** otter768 has joined #openstack-nova | 09:32 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 09:34 |
| *** otter768 has quit IRC | 09:34 | |
| *** thorst has joined #openstack-nova | 09:34 | |
| *** otter768_ has joined #openstack-nova | 09:34 | |
| *** yangyapeng has quit IRC | 09:35 | |
| *** otter768 has joined #openstack-nova | 09:36 | |
| *** otter768_ has quit IRC | 09:36 | |
| *** otter768 has quit IRC | 09:37 | |
| *** otter768 has joined #openstack-nova | 09:38 | |
| openstackgerrit | lyanchih proposed openstack/nova-specs: Spec to add resources-links in resource list API Response https://review.openstack.org/264114 | 09:38 |
| *** otter768 has quit IRC | 09:40 | |
| *** otter768_ has joined #openstack-nova | 09:40 | |
| *** zhangjn has quit IRC | 09:40 | |
| *** salv-orlando has quit IRC | 09:40 | |
| *** salv-orlando has joined #openstack-nova | 09:41 | |
| *** pkholkin has quit IRC | 09:41 | |
| *** otter768 has joined #openstack-nova | 09:42 | |
| *** thorst has quit IRC | 09:42 | |
| *** pkholkin has joined #openstack-nova | 09:43 | |
| *** otter768 has quit IRC | 09:43 | |
| *** otter76__ has joined #openstack-nova | 09:43 | |
| *** pkholkin has left #openstack-nova | 09:43 | |
| *** pkholkin has joined #openstack-nova | 09:44 | |
| *** ndipanov has joined #openstack-nova | 09:44 | |
| *** otter768_ has quit IRC | 09:44 | |
| *** otter768 has joined #openstack-nova | 09:45 | |
| *** otter76__ has quit IRC | 09:45 | |
| *** otter768_ has joined #openstack-nova | 09:47 | |
| *** otter768_ has quit IRC | 09:49 | |
| *** ljxiash has quit IRC | 09:49 | |
| *** ljxiash has joined #openstack-nova | 09:50 | |
| *** otter768 has quit IRC | 09:50 | |
| *** otter768 has joined #openstack-nova | 09:51 | |
| *** NightKhaos has quit IRC | 09:52 | |
| openstackgerrit | Sayali Lunkad proposed openstack/nova: Allows to set display_name as instance_name https://review.openstack.org/263353 | 09:52 |
| *** otter768 has quit IRC | 09:53 | |
| *** otter768 has joined #openstack-nova | 09:53 | |
| *** daemontool has quit IRC | 09:53 | |
| *** daemontool has joined #openstack-nova | 09:53 | |
| *** ljxiash has quit IRC | 09:54 | |
| *** otter768_ has joined #openstack-nova | 09:55 | |
| *** daemontool has quit IRC | 09:55 | |
| *** ociuhandu has joined #openstack-nova | 09:55 | |
| *** daemontool has joined #openstack-nova | 09:56 | |
| *** rotbeard has joined #openstack-nova | 09:56 | |
| *** zhangjn has joined #openstack-nova | 09:57 | |
| *** otter768_ has quit IRC | 09:57 | |
| *** daemontool has quit IRC | 09:57 | |
| *** otter76__ has joined #openstack-nova | 09:57 | |
| *** otter76__ has quit IRC | 09:58 | |
| *** otter768_ has joined #openstack-nova | 09:59 | |
| *** daemontool has joined #openstack-nova | 09:59 | |
| *** otter76__ has joined #openstack-nova | 10:01 | |
| *** otter768_ has quit IRC | 10:01 | |
| *** DinaBelova has quit IRC | 10:01 | |
| *** DinaBelova has joined #openstack-nova | 10:01 | |
| *** otter768 has quit IRC | 10:02 | |
| *** NightKhaos has joined #openstack-nova | 10:02 | |
| *** otter76__ has quit IRC | 10:02 | |
| *** otter768 has joined #openstack-nova | 10:02 | |
| *** ptm_away is now known as PaulMurray | 10:03 | |
| PaulMurray | Its quiet today! | 10:03 |
| *** otter768_ has joined #openstack-nova | 10:05 | |
| *** danpb has joined #openstack-nova | 10:05 | |
| *** otter768_ has quit IRC | 10:06 | |
| kashyap | Calm before the storm. . . | 10:06 |
| bauzas | maybe because Bowie ? | 10:07 |
| *** otter768 has quit IRC | 10:07 | |
| *** otter768 has joined #openstack-nova | 10:08 | |
| *** aarefiev has joined #openstack-nova | 10:10 | |
| *** otter768 has quit IRC | 10:10 | |
| *** salv-orl_ has joined #openstack-nova | 10:10 | |
| *** otter768 has joined #openstack-nova | 10:10 | |
| *** daemontool_ has joined #openstack-nova | 10:10 | |
| johnthetubaguy | garyk: sorry, looking, I thought I removed that -1 already | 10:11 |
| *** EinstCrazy has quit IRC | 10:11 | |
| * alex_xu try to make some noise | 10:11 | |
| *** klkumar has quit IRC | 10:11 | |
| *** otter768 has quit IRC | 10:12 | |
| garyk | johnthetubaguy: ok, thanks | 10:12 |
| *** otter768 has joined #openstack-nova | 10:12 | |
| *** jlanoux has quit IRC | 10:12 | |
| *** salv-orlando has quit IRC | 10:12 | |
| johnthetubaguy | alex_xu: should I lend you my tuba? | 10:13 |
| *** daemontool has quit IRC | 10:13 | |
| *** otter768_ has joined #openstack-nova | 10:14 | |
| *** lyanchih has quit IRC | 10:14 | |
| *** pask81 has joined #openstack-nova | 10:15 | |
| *** otter768_ has quit IRC | 10:16 | |
| *** otter768_ has joined #openstack-nova | 10:16 | |
| *** lyanchih has joined #openstack-nova | 10:17 | |
| *** daemontool_ has quit IRC | 10:17 | |
| *** pask81 has quit IRC | 10:17 | |
| *** daemontool_ has joined #openstack-nova | 10:17 | |
| *** otter768 has quit IRC | 10:17 | |
| *** otter768_ has quit IRC | 10:17 | |
| *** otter768 has joined #openstack-nova | 10:18 | |
| *** zhangjn has quit IRC | 10:19 | |
| *** otter768_ has joined #openstack-nova | 10:20 | |
| *** otter76__ has joined #openstack-nova | 10:22 | |
| *** otter768 has quit IRC | 10:23 | |
| *** zhangjn has joined #openstack-nova | 10:23 | |
| *** otter768 has joined #openstack-nova | 10:23 | |
| *** otter76__ has quit IRC | 10:24 | |
| openstackgerrit | lyanchih proposed openstack/nova-specs: Spec to add resources-links in resource list API Response https://review.openstack.org/264114 | 10:24 |
| alex_xu | johnthetubaguy: hehe :) | 10:24 |
| *** otter768_ has quit IRC | 10:24 | |
| *** zhangjn has quit IRC | 10:24 | |
| *** klkumar has joined #openstack-nova | 10:25 | |
| *** zhangjn has joined #openstack-nova | 10:25 | |
| *** otter768_ has joined #openstack-nova | 10:25 | |
| *** otter768 has quit IRC | 10:25 | |
| *** cdent has joined #openstack-nova | 10:26 | |
| *** otter768_ has quit IRC | 10:27 | |
| *** otter768 has joined #openstack-nova | 10:27 | |
| *** otter768 has quit IRC | 10:29 | |
| *** otter768 has joined #openstack-nova | 10:29 | |
| *** qianli has quit IRC | 10:31 | |
| *** otter768 has quit IRC | 10:34 | |
| PaulMurray | pkoniszewski, https://review.openstack.org/#/c/227278/25 failed experimental live migration check on test_live_block_migration | 10:34 |
| PaulMurray | pkoniszewski, obviously related to the change - is it expected? | 10:35 |
| *** aix has joined #openstack-nova | 10:35 | |
| *** Jeffrey4l__ has joined #openstack-nova | 10:37 | |
| *** deepakcs has quit IRC | 10:37 | |
| *** thorst has joined #openstack-nova | 10:40 | |
| *** sudipto has quit IRC | 10:45 | |
| *** zenoway has joined #openstack-nova | 10:46 | |
| *** thorst has quit IRC | 10:47 | |
| alex_xu | dansmith: due to this spec http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/making_live_migration_api_friendly.html, the 'block_migration' and 'disk_over_commit' in LibvirtLiveMigrateData obj should be nullable also. Do we still have chance to update that object now? as it didn't pass over rpc yet. otherwise I guess we need add new fields to add nullable. | 10:50 |
| *** jlanoux has joined #openstack-nova | 10:51 | |
| *** rfolco has joined #openstack-nova | 10:51 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: treat cases where network does not exist on host https://review.openstack.org/265764 | 10:51 |
| *** zhangjn has quit IRC | 10:53 | |
| *** jaosorior has quit IRC | 10:53 | |
| *** jaosorior has joined #openstack-nova | 10:54 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object https://review.openstack.org/249015 | 10:54 |
| *** zenoway has quit IRC | 10:54 | |
| ndipanov | johnthetubaguy, ^ | 10:54 |
| ndipanov | you had some comments on Friday | 10:54 |
| johnthetubaguy | ndipanov: ah, yep, will take a look | 10:54 |
| ndipanov | johnthetubaguy, thanks | 10:54 |
| *** markvoelker has joined #openstack-nova | 10:55 | |
| *** zenoway has joined #openstack-nova | 10:56 | |
| johnthetubaguy | ndipanov: had a thought about this (more a follow up patch thing), do we have a nova-manage command to execute this data migration? When we did the flavor one, I have a feeling it was flavor specific, rather than a general case one. | 10:56 |
| claudiub | sahid: hello. There has been some answers on your comments on this patch: https://review.openstack.org/#/c/179727/ can you take a look pls? | 10:57 |
| *** omnipresent has joined #openstack-nova | 10:57 | |
| ndipanov | johnthetubaguy, well ... | 10:57 |
| *** jaosorior has quit IRC | 10:57 | |
| *** jaosorior has joined #openstack-nova | 10:58 | |
| tobasco | johnthetubaguy: do you know anybody that has insight in resource_tracker.py for the compute service? related to https://bugs.launchpad.net/nova/+bug/1508907 | 10:58 |
| openstack | Launchpad bug 1508907 in OpenStack Compute (nova) "local_gb_used wrong in compute_nodes table when using Dell Cinder backend" [Undecided,Confirmed] | 10:58 |
| ndipanov | johnthetubaguy, in theory we should have it I guess, but in practice it's not very useful. all devices get saved on every run of the RT | 10:59 |
| ndipanov | and they never get undeleted | 10:59 |
| *** markvoelker has quit IRC | 11:00 | |
| *** zenoway has quit IRC | 11:00 | |
| johnthetubaguy | ndipanov: its more needed for the end of release, for when we put a check into the schema migration that its "complete", so we can remove the migration code, but we can sort that later, I think. | 11:01 |
| ndipanov | johnthetubaguy, yeah well in all normal circumstances it will be complete - but yes can be done later | 11:02 |
| *** gszasz has joined #openstack-nova | 11:02 | |
| ndipanov | I really think we should have some kind of a framework for this stuff though | 11:02 |
| johnthetubaguy | tobasco: unsure, have you tried raising the issue in the nova scheduler meeting, the folks in there are good candidates to help out, there were plans to start tracking external storage | 11:02 |
| *** zhangjn has joined #openstack-nova | 11:02 | |
| ndipanov | it's not as trivial as dansmith pitches it imho, and is easy to get wrong | 11:02 |
| ndipanov | hand rolling it even for the simple case like this is not trivial | 11:03 |
| johnthetubaguy | ndipanov: yeah, I agree, I think we said lets do this two or three times so we can understand the partern | 11:03 |
| johnthetubaguy | ndipanov: yeah, it certainly too a few attempts to make the flavor migration work correctly | 11:03 |
| *** salv-orl_ has quit IRC | 11:03 | |
| *** salv-orlando has joined #openstack-nova | 11:04 | |
| johnthetubaguy | ndipanov: its a big help seeing this one, its honestly feeling surpizingly similar | 11:04 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: objects: added 'os_secure_boot' property to ImageMetaProps object https://review.openstack.org/237593 | 11:04 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 11:04 |
| *** salv-orlando has quit IRC | 11:05 | |
| *** aysyd has joined #openstack-nova | 11:05 | |
| *** salv-orlando has joined #openstack-nova | 11:05 | |
| tobasco | johnthetubaguy: ok I'm gonna have a peek in there when the meeting starts | 11:05 |
| danpb | someone remind me the right place to file bugs against openstack's gerrit instance - is it under openstack-ci or somewhere else ? | 11:06 |
| *** aysyd has quit IRC | 11:06 | |
| johnthetubaguy | tobasco: I think it was jaypipe's "resource provider" spec that was the closest to helping with that bug | 11:06 |
| *** dedery has quit IRC | 11:06 | |
| *** aysyd has joined #openstack-nova | 11:07 | |
| johnthetubaguy | danpb: sorry, I am not sure, I have taken to just asking in the infra channel | 11:07 |
| *** tiagogomes__ has left #openstack-nova | 11:07 | |
| johnthetubaguy | ndipanov: that patch looks good now I think, although gerrit seems to be moaning about a merge issue? | 11:11 |
| openstackgerrit | linwei wu proposed openstack/nova: Docstring change for consistency https://review.openstack.org/265773 | 11:11 |
| *** sahid has quit IRC | 11:12 | |
| *** bkumar has quit IRC | 11:12 | |
| ndipanov | johnthetubaguy, that;s because the previous one failed merge :( | 11:14 |
| *** baoli has joined #openstack-nova | 11:14 | |
| ndipanov | I rechecked it | 11:14 |
| ndipanov | ah no seems to be something else | 11:15 |
| *** coolsvap is now known as coolsvap|away | 11:17 | |
| *** dims_ has joined #openstack-nova | 11:18 | |
| gibi | johnthetubaguy: hi! Do you have a minute? | 11:19 |
| gibi | johnthetubaguy: you hade a comment in the first versioned notification patch about the dependency between the notification.publisher and the Service object | 11:20 |
| gibi | johnthetubaguy: I slept on it but I still cannot decide if it is a good idea or a bad one | 11:20 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Add caching of service_min_versions in the conductor https://review.openstack.org/265326 | 11:20 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object https://review.openstack.org/249015 | 11:20 |
| gibi | johnthetubaguy: how likely that querying the db for a Service object to send a notification will cause problems in the future? | 11:21 |
| ndipanov | johnthetubaguy, done | 11:21 |
| johnthetubaguy | gibi: good question... | 11:21 |
| johnthetubaguy | ndipanov: sweet | 11:21 |
| *** lucas-dinner is now known as lucasagomes | 11:23 | |
| johnthetubaguy | gibi: I was wondering about having a service object that uses that schema system, to stop any service changes causing that version to increase? | 11:23 |
| dims_ | sdague : multinode jobs are failing here's a elastic recheck query - https://review.openstack.org/#/c/265640/ - i haven't dug into it yet | 11:24 |
| johnthetubaguy | gibi: but feels a touch live overkill | 11:24 |
| johnthetubaguy | s/live/like/ | 11:24 |
| gibi | johnthetubaguy: you mean an proxy object between the Notification.publisher field and the Service object? | 11:24 |
| gibi | johnthetubaguy: that sounds reasonable for me | 11:25 |
| sdague | dims_: ok, just got up, I'll look | 11:25 |
| sdague | dims_: they just started failing? | 11:25 |
| johnthetubaguy | gibi: maybe worth a try, see if it looks terrible or now :) | 11:25 |
| *** sahid has joined #openstack-nova | 11:25 | |
| johnthetubaguy | s/now/not | 11:25 |
| johnthetubaguy | so I am not using my normal keyboard... arg! | 11:26 |
| gibi | johnthetubaguy: it gives us a possibility to avoid direct dependecy and also make it possible to avoid db query and provide the publisher as a string later if needed | 11:26 |
| dims_ | sdague : i saw them fail over the weekend, don't know when they started to fail | 11:26 |
| sdague | ok | 11:26 |
| johnthetubaguy | gibi: ah, true | 11:26 |
| gibi | johnthetubaguy: I will create a proxy object. Thanks for the consultation | 11:26 |
| dims_ | sdague : they showed up here as well - http://status.openstack.org//elastic-recheck/data/uncategorized.html | 11:26 |
| cdent | johnthetubaguy: do you non-qwerty? which one? | 11:26 |
| gibi | johnthetubaguy: btw, all your other concerns are answered in the versioned notification chain | 11:27 |
| johnthetubaguy | cdent: this is really boring I am afriad. on my laptop keyboard, instead of the desktop one :( | 11:27 |
| johnthetubaguy | gibi: awesome stuff, thanks | 11:27 |
| cdent | :) | 11:27 |
| johnthetubaguy | gibi: ping me when you have that next bit up, and I can take another look,if that works? | 11:28 |
| gibi | johnthetubaguy: sure. thanks! | 11:29 |
| *** sfinucan has joined #openstack-nova | 11:29 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 11:31 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 11:32 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Flesh out nova.conf.virt documentation https://review.openstack.org/259553 | 11:32 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 11:32 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 11:32 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 11:32 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 11:32 |
| *** daemontool_ is now known as daemontool | 11:32 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add FC attached volumes support https://review.openstack.org/258617 | 11:33 |
| *** lyanchih has quit IRC | 11:34 | |
| openstackgerrit | Ayush Garg proposed openstack/nova: Change assertEqual(True/False) to assertTrue/False https://review.openstack.org/265581 | 11:35 |
| *** bauzas has quit IRC | 11:35 | |
| *** jaypipes has joined #openstack-nova | 11:37 | |
| *** thorst has joined #openstack-nova | 11:44 | |
| *** liusheng has quit IRC | 11:45 | |
| *** liusheng has joined #openstack-nova | 11:45 | |
| sdague | johnthetubaguy: going to need your input for this - https://review.openstack.org/#/c/265282 so we can decide how / if we are moving forward there | 11:46 |
| johnthetubaguy | oh right, this is the cells related bits | 11:47 |
| *** aix has quit IRC | 11:47 | |
| *** thorst has quit IRC | 11:51 | |
| sfinucan | johnthetubaguy: Would it be possible to get some eyes on this feature over the next few days? https://review.openstack.org/#/q/topic:bp/virt-driver-cpu-thread-pinning+status:open+project:openstack/nova | 11:52 |
| *** daemontool_ has joined #openstack-nova | 11:53 | |
| sfinucan | johnthetubaguy: Or suggest someone that I should talk to outside of ndipanov? I'm increasingly wary of the closing window and I want my ATC pass to Austin :) | 11:53 |
| ankit_ag | sdague: hi | 11:54 |
| ankit_ag | could you please take a look at https://review.openstack.org/#/c/256343/ | 11:54 |
| ankit_ag | and the related patch https://review.openstack.org/#/c/185549/ | 11:54 |
| sfinucan | sahid: Available to discuss this https://review.openstack.org/#/c/202651/ should you have any questions | 11:54 |
| *** daemontool has quit IRC | 11:55 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Move wsgi options into a group https://review.openstack.org/263812 | 11:55 |
| ndipanov | sfinucan, what happened to the cpu pinning patches - I +2ed most of them right? | 11:56 |
| *** markvoelker has joined #openstack-nova | 11:56 | |
| sfinucan | ndipanov: Yes, that's the ones. I need another core's +2 but I don't know who to ask | 11:56 |
| sfinucan | *they're | 11:57 |
| openstackgerrit | Sayali Lunkad proposed openstack/nova: Allows to set display_name as instance_name https://review.openstack.org/263353 | 11:57 |
| johnthetubaguy | sfinucan: I did take a quick look, and got confused, need to go back and revisit those, just been pulled lots of directions at the moment :( | 11:59 |
| ndipanov | sfinucan, alaski +2ed my patches | 12:00 |
| ndipanov | so I assumed he would look at those as well | 12:00 |
| *** markvoelker has quit IRC | 12:00 | |
| sfinucan | johnthetubaguy: Understandable :) That code just doesn't get that many eyes, it seems | 12:01 |
| *** dedery has joined #openstack-nova | 12:01 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'fixed_ip' https://review.openstack.org/240811 | 12:01 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance' https://review.openstack.org/244146 | 12:01 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'security_group' https://review.openstack.org/246302 | 12:01 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'quota' and 'reservation' https://review.openstack.org/260549 | 12:01 |
| sfinucan | ndipanov: Yeah, alaski +2d the first patch of the series but stopped after that. I'll reach out to him when he comes online and see if he can help | 12:02 |
| johnthetubaguy | sfinucan: yeah, I fear its a complicated area that not too many folks have gone digging in recently | 12:02 |
| ndipanov | sfinucan, yeah - the more the merrier | 12:02 |
| johnthetubaguy | I do hope to get there, yeah | 12:02 |
| johnthetubaguy | +1 to more the merrier | 12:02 |
| *** zhangjn has quit IRC | 12:03 | |
| *** haomaiwang has joined #openstack-nova | 12:04 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Propagate qemu-img errors to compute manager https://review.openstack.org/264349 | 12:05 |
| *** dedery has quit IRC | 12:06 | |
| *** EinstCrazy has joined #openstack-nova | 12:07 | |
| *** dedery has joined #openstack-nova | 12:07 | |
| *** dedery_ has joined #openstack-nova | 12:08 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding volume operations for shelved instances https://review.openstack.org/259528 | 12:09 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Adding guard on None value for some helpers method https://review.openstack.org/264853 | 12:09 |
| *** aysyd has quit IRC | 12:10 | |
| *** daemontool has joined #openstack-nova | 12:12 | |
| *** dedery has quit IRC | 12:12 | |
| *** daemontool_ has quit IRC | 12:14 | |
| *** rushiagr is now known as rushiagr_away | 12:14 | |
| *** dedery_ has quit IRC | 12:14 | |
| *** derekh has quit IRC | 12:15 | |
| johnthetubaguy | sdague: added some comments, that seems a good direction, going to get some food now, and mull it over | 12:17 |
| johnthetubaguy | sdague: I think we need a strong concept of a flavor lifecycle in the API | 12:17 |
| johnthetubaguy | stronger | 12:17 |
| *** irenab has quit IRC | 12:17 | |
| johnthetubaguy | sdague: this seems a good step in that direction | 12:18 |
| *** dedery has joined #openstack-nova | 12:19 | |
| *** irenab has joined #openstack-nova | 12:19 | |
| *** raildo-afk is now known as raildo | 12:21 | |
| *** aix has joined #openstack-nova | 12:22 | |
| *** ljxiash has joined #openstack-nova | 12:22 | |
| *** aysyd has joined #openstack-nova | 12:23 | |
| *** paul-carlton1 has quit IRC | 12:24 | |
| sdague | ankit_ag: that looks better | 12:25 |
| *** derekh has joined #openstack-nova | 12:25 | |
| ankit_ag | sdague: thanks :) | 12:26 |
| *** thorst has joined #openstack-nova | 12:27 | |
| *** HenryG has quit IRC | 12:27 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'server_group' https://review.openstack.org/261316 | 12:27 |
| *** HenryG has joined #openstack-nova | 12:27 | |
| *** rotbeard has quit IRC | 12:30 | |
| *** gszasz has quit IRC | 12:30 | |
| *** otter768 has joined #openstack-nova | 12:30 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Propagate qemu-img errors to compute manager https://review.openstack.org/264349 | 12:31 |
| sdague | dims_: your query is actually nearly 100% success | 12:33 |
| *** jaosorior has quit IRC | 12:33 | |
| sdague | http://logstash.openstack.org/#dashboard/file/logstash.json?query=(message%3A%5C%22(LiveBlockMigrationTestJSON%3Atest_live_block_migration_paused)%5C%22%20%20AND%20message%3A%5C%22(LiveBlockMigrationTestJSON%3Atest_live_block_migration)%5C%22)%20OR%20message%3A%5C%22Current%20status%3A%20MIGRATING.%20Current%20task%20state%3A%20migrating.%5C%22%20tags%3A%5C%22console%5C%22%20AND%20voting%3A1 | 12:33 |
| *** jaosorior has joined #openstack-nova | 12:34 | |
| *** cgascoig has quit IRC | 12:35 | |
| *** otter768 has quit IRC | 12:35 | |
| *** gszasz has joined #openstack-nova | 12:37 | |
| *** dedery has quit IRC | 12:37 | |
| *** dedery has joined #openstack-nova | 12:43 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: removed db_exc.DBDuplicateEntry in bw_usage_update https://review.openstack.org/250807 | 12:43 |
| *** alejandrito has joined #openstack-nova | 12:45 | |
| dims_ | sdague : can you please try just a simple message:"Current status: MIGRATING. Current task state: migrating." | 12:45 |
| sdague | dims_: sure, but nothing there is voting | 12:46 |
| *** alejandrito has quit IRC | 12:46 | |
| *** alejandrito has joined #openstack-nova | 12:46 | |
| *** smatzek has joined #openstack-nova | 12:47 | |
| dims_ | sdague : so just the ones on here http://status.openstack.org//elastic-recheck/data/uncategorized.html count i guess | 12:47 |
| dims_ | which is from 9th | 12:47 |
| dims_ | at least the top 2 | 12:47 |
| sdague | yeh, that's unclassified failures in the gate queue | 12:48 |
| dims_ | so this is probably not a cause for concern right now | 12:48 |
| dims_ | apologies | 12:48 |
| sdague | it might be, if people think the multinode job should pass | 12:48 |
| dims_ | sdague : agree | 12:49 |
| *** andreykurilin__ has joined #openstack-nova | 12:49 | |
| sdague | yeh, though actually, there are other issues with the query | 12:49 |
| *** dedery has quit IRC | 12:49 | |
| sdague | I'm going to revert it with some comments | 12:50 |
| *** dedery has joined #openstack-nova | 12:51 | |
| dims_ | sdague : ack thanks | 12:51 |
| sdague | the AND / OR structure of that actually meant it hit lots of unrelated stuff | 12:51 |
| *** salv-orlando has quit IRC | 12:55 | |
| *** salv-orlando has joined #openstack-nova | 12:56 | |
| *** markvoelker has joined #openstack-nova | 12:57 | |
| *** ankit_ag has quit IRC | 12:59 | |
| dims_ | sdague gah! i think i had the query better in the bug report :) https://bugs.launchpad.net/nova/+bug/1532542 | 13:00 |
| openstack | Launchpad bug 1532542 in tempest "multinode-full's test_live_block_migration and test_live_block_migration_paused are broken" [Undecided,New] | 13:00 |
| *** houming has quit IRC | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** Jeffrey4l__ has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** markvoelker has quit IRC | 13:01 | |
| *** zhangjn has joined #openstack-nova | 13:05 | |
| *** kylek3h_ has quit IRC | 13:06 | |
| *** takedakn has joined #openstack-nova | 13:06 | |
| *** Jeffrey4l__ has joined #openstack-nova | 13:06 | |
| *** daemontool has quit IRC | 13:07 | |
| *** Jeffrey4l__ is now known as Jeffrey4l | 13:08 | |
| *** takedakn has quit IRC | 13:09 | |
| *** doug-fish has joined #openstack-nova | 13:09 | |
| *** daemontool has joined #openstack-nova | 13:10 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Config options: centralize base path configuration https://review.openstack.org/259406 | 13:11 |
| *** haomaiwang has quit IRC | 13:11 | |
| *** edmondsw has joined #openstack-nova | 13:11 | |
| *** salv-orlando has quit IRC | 13:11 | |
| *** salv-orlando has joined #openstack-nova | 13:12 | |
| *** haomaiwang has joined #openstack-nova | 13:12 | |
| *** links has quit IRC | 13:13 | |
| *** ducttape_ has joined #openstack-nova | 13:13 | |
| *** paul-carlton1 has joined #openstack-nova | 13:14 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Handle SetAdminPasswdNotSupported raised by libvirt driver https://review.openstack.org/252850 | 13:14 |
| openstackgerrit | jichenjc proposed openstack/nova: Add description on how to run ./run_test.sh -8 https://review.openstack.org/265262 | 13:17 |
| *** lyanchih has joined #openstack-nova | 13:17 | |
| *** markvoelker has joined #openstack-nova | 13:20 | |
| claudiub | johnthetubaguy: hello. sorry to bother you, but could you review a one-line patch? https://review.openstack.org/#/c/265239/ | 13:21 |
| *** MVenesio has joined #openstack-nova | 13:21 | |
| *** zenoway has joined #openstack-nova | 13:21 | |
| *** dedery has quit IRC | 13:22 | |
| openstackgerrit | jichenjc proposed openstack/nova: Use stub_out and mock to remove mox:part 3 https://review.openstack.org/264753 | 13:23 |
| johnthetubaguy | claudiub: should we not try re-populate the new exception, or something like that? | 13:23 |
| *** erlon has joined #openstack-nova | 13:23 | |
| *** zhangyufei has joined #openstack-nova | 13:23 | |
| *** daemontool_ has joined #openstack-nova | 13:23 | |
| *** omnipresent has quit IRC | 13:24 | |
| johnthetubaguy | claudiub: oh, I see, its kinda not going to work in py2 | 13:24 |
| *** ccarmack has joined #openstack-nova | 13:24 | |
| *** Jeffrey4l has quit IRC | 13:24 | |
| claudiub | johnthetubaguy: i don't know what you mean by repopulating the new exception | 13:25 |
| *** daemontool has quit IRC | 13:26 | |
| *** shyama has quit IRC | 13:26 | |
| *** Jeffrey4l has joined #openstack-nova | 13:28 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 13:29 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new DB API method to retrieve migration for instance https://review.openstack.org/257270 | 13:29 |
| johnthetubaguy | claudiub: I am wondering if we can keep the traceback by using three part raise, but maybe that causes other issues? | 13:31 |
| *** rk4n has joined #openstack-nova | 13:32 | |
| *** obondarev has joined #openstack-nova | 13:32 | |
| *** bauzas has joined #openstack-nova | 13:33 | |
| claudiub | johnthetubaguy: oh, I see what you mean now. let me check. | 13:33 |
| *** omnipresent has joined #openstack-nova | 13:35 | |
| *** ducttape_ has quit IRC | 13:35 | |
| *** zhangyufei has quit IRC | 13:36 | |
| openstackgerrit | jichenjc proposed openstack/nova: Support mac and type for /servers/ips request https://review.openstack.org/257261 | 13:37 |
| *** zenoway has quit IRC | 13:38 | |
| *** EinstCrazy has quit IRC | 13:40 | |
| *** kylek3h has joined #openstack-nova | 13:40 | |
| *** dedery has joined #openstack-nova | 13:43 | |
| *** n0ano has joined #openstack-nova | 13:43 | |
| *** admin0 has joined #openstack-nova | 13:43 | |
| admin0 | in which table does nova stores all flavor-key data ? | 13:43 |
| sdague | johnthetubaguy: so, it turns out to drop project id in url, and avoid some odd edge cases, I'm going to need to put a pattern match on the route. Which I think means fixing a few thousand unit tests that use fake_project or openstack as their project id | 13:45 |
| rdopiera | obondarev: hi, do you have a moment to talk about the live migration network bug that you are working on? | 13:45 |
| ildikov | johnthetubaguy: hi | 13:45 |
| johnthetubaguy | sdague: oh my :S | 13:45 |
| sdague | johnthetubaguy: well, using crappy fake data was always going to come back and bite us eventualy | 13:46 |
| johnthetubaguy | sdague: thats totally true | 13:46 |
| sdague | which is part of why the uuid sentinel effort got started | 13:46 |
| johnthetubaguy | true | 13:46 |
| ildikov | johnthetubaguy: as I see it's not the best timing :) | 13:46 |
| johnthetubaguy | ildikov: you got those patches refreshed? | 13:47 |
| *** dedery has quit IRC | 13:47 | |
| ildikov | johnthetubaguy: evrything is refreshed I was able to fix myself, so the blocking is there for boot_from_volume and create_volume_snapshot | 13:47 |
| ildikov | johnthetubaguy: I also updated the spec amendment, so that should be good now too | 13:48 |
| *** lpetrut has quit IRC | 13:48 | |
| ildikov | johnthetubaguy: I also answered dansmith's comment on the first patch as much as I could | 13:48 |
| ildikov | johnthetubaguy: basically the question I have is what is needed to be able to merge this code for Mitaka? | 13:49 |
| obondarev | rdopiera: hi, sure | 13:49 |
| ildikov | johnthetubaguy: I mean beyind fixing coding and other nits in the code that people might find during review | 13:50 |
| rdopiera | obondarev: I'm wondering if there is anything that I could do to help, and I'm also wondering how to approach backporting it once it landed. | 13:51 |
| *** omnipresent has quit IRC | 13:51 | |
| *** zenoway has joined #openstack-nova | 13:51 | |
| rdopiera | obondarev: as I understand, the migration info is not in DB in earlier versions, right? | 13:51 |
| obondarev | rdopiera: correct | 13:51 |
| obondarev | rdopiera: so this approach only works since liberty | 13:52 |
| johnthetubaguy | ildikov: cool beans, hope to get to those soon, folks have been waiting a long time for that feature | 13:52 |
| dims_ | sdague : here's another logstash query - message:"No lease, failing" AND voting:1 - This catches failure to get DHCP lease which ends up in SSH timeout | 13:53 |
| *** eharney has quit IRC | 13:53 | |
| ildikov | johnthetubaguy: yeah, I know, this is why I hope the code can make it finally | 13:53 |
| dims_ | sdague : 24 hits in last 2 days | 13:53 |
| obondarev | rdopiera: the patch is lacking reviews so it's the best help I can think of | 13:53 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'instance_group' https://review.openstack.org/261316 | 13:53 |
| rdopiera | obondarev: I'm just a beginner in this, but I will try to ask other for reviews | 13:53 |
| johnthetubaguy | dims_: is that client side / serial log error? | 13:54 |
| ildikov | johnthetubaguy: I keep the Cinder guys involved as much as possible too, I hope the session will be good too on the mid-cycle about cross-project topics like this one | 13:54 |
| dims_ | johnthetubaguy : serial log error | 13:54 |
| obondarev | rdopiera: ok, thank you | 13:54 |
| sdague | dims_: that seems better | 13:54 |
| ildikov | johnthetubaguy: and of course thanks much for all the help and review effort with this! | 13:54 |
| sdague | dims_: which file is the No lease coming out of? | 13:54 |
| johnthetubaguy | dims_: cool, I guess that, another of the "it not worky" for the VIF, but its nice and specific | 13:55 |
| *** scheuran has quit IRC | 13:55 | |
| dims_ | sdague : http://logs.openstack.org/97/265697/1/check/gate-grenade-dsvm/6eeced7/console.html#_2016-01-11_07_42_30_838 | 13:55 |
| *** Jeffrey4l has quit IRC | 13:55 | |
| dims_ | sdague : console.html | 13:55 |
| dims_ | johnthetubaguy : right. | 13:55 |
| sdague | dims_: ok, nice, I didn't realize we already had that in there | 13:55 |
| sdague | that's not a new bug, but it's a good one to be tracking | 13:56 |
| sdague | will definitely merge that | 13:56 |
| johnthetubaguy | awesome to capture that one, its a very old friend | 13:56 |
| dims_ | sdague : should i open a new bug to log the elastic-recheck against? | 13:56 |
| *** vladikr has joined #openstack-nova | 13:56 | |
| dims_ | johnthetubaguy : right, it seems like the last dhcp_release for that ip address did not make it to dnsmasq as i don't see entries in syslog for it | 13:57 |
| johnthetubaguy | yeah, sounds like it captures the broken vif quite well | 13:57 |
| *** admin0 has quit IRC | 13:57 | |
| *** rushiagr_away is now known as rushiagr | 13:57 | |
| sdague | dims_: yeh, probably | 13:58 |
| dims_ | k | 13:58 |
| openstackgerrit | jichenjc proposed openstack/nova: Use SpawnIsSynchronousFixture in most unit tests https://review.openstack.org/230953 | 13:59 |
| openstackgerrit | jichenjc proposed openstack/nova: Replace fake_utils by using Fixture https://review.openstack.org/230954 | 13:59 |
| sdague | so, I'm a little confused that we aren't passing network info with config drive | 13:59 |
| *** haomaiwang has quit IRC | 14:01 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions https://review.openstack.org/249016 | 14:01 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests https://review.openstack.org/259254 | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| johnthetubaguy | sdague: oh, true, but I think we had some jobs without config drive? | 14:02 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 14:02 |
| *** rlrossit has joined #openstack-nova | 14:03 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver https://review.openstack.org/262583 | 14:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type https://review.openstack.org/262584 | 14:03 |
| sdague | we do, not this one | 14:03 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr https://review.openstack.org/262585 | 14:04 |
| *** dane-fichter has joined #openstack-nova | 14:04 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs https://review.openstack.org/262586 | 14:04 |
| dane-fichter | danpb: could you take a look at https://review.openstack.org/#/c/189843/ when you get a chance? It's been updated to use ImageMeta per your suggestion. | 14:05 |
| *** krtaylor has quit IRC | 14:06 | |
| sdague | dims_: what's up with this - http://logs.openstack.org/66/265266/3/check/gate-cinder-python34-constraints/8b779ee/console.html#_2016-01-11_13_30_26_594 ? | 14:06 |
| *** liverpooler has quit IRC | 14:06 | |
| *** Jeffrey4l has joined #openstack-nova | 14:07 | |
| dims_ | sdague : the "Zeromq not found", we fixed it last week, waiting for a release | 14:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Trace original exception before converting exception https://review.openstack.org/265239 | 14:07 |
| claudiub | johnthetubaguy: yeap, seems to be fine. thanks for the suggestion. :) | 14:07 |
| *** mdrabe has joined #openstack-nova | 14:07 | |
| *** jang has quit IRC | 14:08 | |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Refactor claim code to eliminate duplication https://review.openstack.org/262939 | 14:08 |
| openstackgerrit | Yingxin Cheng proposed openstack/nova: Scheduler use Claim to check resource consumption https://review.openstack.org/262938 | 14:08 |
| *** jang has joined #openstack-nova | 14:08 | |
| *** omnipresent has joined #openstack-nova | 14:09 | |
| *** jwcroppe has joined #openstack-nova | 14:09 | |
| sdague | dims_: ok, cool, just started noticing it in unit tests | 14:11 |
| johnthetubaguy | claudiub: np | 14:11 |
| cdent | which magic recheck to recheck "check-citrix-xenserver" | 14:11 |
| *** zenoway has quit IRC | 14:12 | |
| *** bklei has joined #openstack-nova | 14:12 | |
| *** dslev has quit IRC | 14:14 | |
| *** med_ has quit IRC | 14:15 | |
| johnthetubaguy | cdent: BobBall will know, I think its just recheck xenserver | 14:15 |
| *** dslevin has quit IRC | 14:15 | |
| johnthetubaguy | cdent: I thought the comment used to say, but I don't totally remember | 14:16 |
| cdent | thanks johnthetubaguy | 14:16 |
| andrearosa | cdent: I think it is xen:recheck | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications https://review.openstack.org/263788 | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Generate doc for versioned notifications https://review.openstack.org/265151 | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add infra for versioned notifications https://review.openstack.org/247024 | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 14:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Make emitting versioned notifications configurable https://review.openstack.org/247564 | 14:16 |
| *** claudiub has quit IRC | 14:17 | |
| *** takedakn has joined #openstack-nova | 14:18 | |
| *** takedakn has quit IRC | 14:18 | |
| *** krtaylor has joined #openstack-nova | 14:18 | |
| *** ducttape_ has joined #openstack-nova | 14:19 | |
| *** ctrath has joined #openstack-nova | 14:19 | |
| *** med_ has joined #openstack-nova | 14:20 | |
| *** med_ is now known as Guest74023 | 14:20 | |
| *** cfriesen__ has joined #openstack-nova | 14:22 | |
| johnthetubaguy | andrearosa: that might be xenproject CI rather than XenServer CI | 14:23 |
| *** jichen has quit IRC | 14:23 | |
| *** lucasagomes is now known as lucas-hungry | 14:23 | |
| *** zhangjn has quit IRC | 14:23 | |
| *** ducttape_ has quit IRC | 14:24 | |
| andrearosa | johnthetubaguy: ops, thanks for the correction | 14:28 |
| johnthetubaguy | so many Xen's :) | 14:28 |
| andrearosa | :) | 14:29 |
| *** dslevin has joined #openstack-nova | 14:30 | |
| *** otter768 has joined #openstack-nova | 14:31 | |
| *** zenoway has joined #openstack-nova | 14:32 | |
| alaski | sfinucan: I'll go through your series in a bit. I think the last time I looked there was a -1 from ndipanov and I stopped there | 14:32 |
| sfinucan | alaski: That would be great. Appreciate it :) | 14:33 |
| *** eharney has joined #openstack-nova | 14:34 | |
| *** busterswt has joined #openstack-nova | 14:35 | |
| *** omnipresent has quit IRC | 14:35 | |
| *** smatzek has quit IRC | 14:35 | |
| BobBall | cdent: Hit "Toggle CI" at the bottom right, then look at the "XenServer CI" comment. A recheck is 'xenserver: recheck'. | 14:36 |
| *** otter768 has quit IRC | 14:36 | |
| *** bkumar has joined #openstack-nova | 14:36 | |
| cdent | ah, okay, thanks BobBall | 14:36 |
| *** ildikov has quit IRC | 14:36 | |
| johnthetubaguy | ah, its in the comment, thats it | 14:37 |
| ndipanov | thanks alaski ! | 14:38 |
| BobBall | And yes, "Toggle CI" is the bottom left not the bottom right... ... ... | 14:38 |
| *** achinm has quit IRC | 14:39 | |
| *** rushiagr is now known as rushiagr_away | 14:40 | |
| *** rushiagr_away is now known as rushiagr | 14:42 | |
| *** zhangjn has joined #openstack-nova | 14:42 | |
| *** ducttape_ has joined #openstack-nova | 14:43 | |
| *** dedery has joined #openstack-nova | 14:43 | |
| *** omnipresent has joined #openstack-nova | 14:44 | |
| *** mriedem has joined #openstack-nova | 14:45 | |
| *** zenoway_ has joined #openstack-nova | 14:45 | |
| *** zenoway has quit IRC | 14:45 | |
| *** mmedvede has quit IRC | 14:46 | |
| *** annegentle has joined #openstack-nova | 14:46 | |
| *** annegentle has quit IRC | 14:47 | |
| *** xyang1 has joined #openstack-nova | 14:47 | |
| *** annegentle has joined #openstack-nova | 14:48 | |
| dansmith | ndipanov: why are you not making the test stub return true (i.e. emulate the future)? https://review.openstack.org/#/c/249015/18/nova/tests/unit/pci/test_manager.py | 14:48 |
| PaulMurray | ndipanov, if you have a minute could you check this attach/detach volumes please: https://review.openstack.org/#/c/259528 (plus trivial preceding patch) | 14:48 |
| *** gszasz has quit IRC | 14:48 | |
| *** cfriesen has joined #openstack-nova | 14:48 | |
| *** omnipresent has quit IRC | 14:48 | |
| *** cfriesen__ has quit IRC | 14:48 | |
| *** dedery has quit IRC | 14:49 | |
| ndipanov | dansmith, I'm conservative like that :) (also it doesn't matter for that test - we just check the number of calls) | 14:49 |
| *** cfarquhar has joined #openstack-nova | 14:50 | |
| *** cfarquhar has quit IRC | 14:50 | |
| *** cfarquhar has joined #openstack-nova | 14:50 | |
| dansmith | ndipanov: just seems weird, but I guess it doesn't matter | 14:50 |
| *** garyk has quit IRC | 14:51 | |
| *** mrkz has joined #openstack-nova | 14:52 | |
| dansmith | ndipanov: one other question | 14:52 |
| dansmith | ndipanov: here: https://review.openstack.org/#/c/249015/18/nova/pci/devspec.py | 14:52 |
| dansmith | ndipanov: is this changing what would go in the pci device whitelist blob? | 14:52 |
| dansmith | ndipanov: i.e. is "dev_dict" the thing we pull out of the whitelist? | 14:53 |
| ndipanov | dansmith, so I don't think we use match() anywhere anymore | 14:55 |
| *** gszasz has joined #openstack-nova | 14:55 | |
| *** gszasz has quit IRC | 14:55 | |
| ndipanov | just match_pci_obj | 14:55 |
| ndipanov | let me check | 14:55 |
| dansmith | ndipanov: well, the same change happens there | 14:55 |
| *** lennyb_ has joined #openstack-nova | 14:56 | |
| ndipanov | dansmith, then I don't get the question | 14:56 |
| dansmith | ndipanov: is this matching thing taking the spec from the pci whitelist and using it to filter devices? | 14:56 |
| *** esracelik has quit IRC | 14:57 | |
| ndipanov | so class PciDeviceSpec instance is generated based on the whitelist | 14:57 |
| *** pradk has joined #openstack-nova | 14:58 | |
| ndipanov | then it is passed either a device_dict (straight from the virt driver) or a device object to see if it can be used at all | 14:58 |
| ndipanov | dansmith, ^ does that make sense? | 14:58 |
| *** pratikmallya has joined #openstack-nova | 14:59 | |
| dansmith | okay, so self.tags is the thing from the whitelist and dev_dict comes from ... the object? | 14:59 |
| ndipanov | yeah | 14:59 |
| ndipanov | or driver | 14:59 |
| *** dedery has joined #openstack-nova | 15:00 | |
| dansmith | okay, I just want to make sure we're not breaking the format of the whitelist in config (again) | 15:00 |
| dansmith | so that's cool, just checking | 15:00 |
| ndipanov | yeah | 15:00 |
| ndipanov | good point | 15:00 |
| *** claudiub has joined #openstack-nova | 15:00 | |
| ndipanov | it actually took quite a bit of thinking to figure out how to do this without breaking it | 15:00 |
| markus_z | edleaf: Kevin_Zheng: tjones: ccarmack: sfinucan: esracelik: Do you have a moment in #openstack-meeting-3 for a quick talk about the config options? | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| sfinucan | markus_z: I'm there | 15:01 |
| ndipanov | tags are basically freeform fields - but one of them is special (physical_network) | 15:01 |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** haomaiwang has quit IRC | 15:01 | |
| ndipanov | which is how neutron ports work - it's a massive hack | 15:01 |
| *** pratikma_ has joined #openstack-nova | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:02 | |
| *** zenoway has joined #openstack-nova | 15:02 | |
| dansmith | nice | 15:03 |
| ndipanov | dansmith, see my comment here - those whitelists are a mistake | 15:04 |
| *** xyang1 has quit IRC | 15:04 | |
| *** pratikmallya has quit IRC | 15:04 | |
| ndipanov | https://review.openstack.org/#/c/242555/ | 15:04 |
| johnthetubaguy | tobasco: bauzas: for that bug we were talking about, have we checked this logic here: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L360 | 15:05 |
| *** mmedvede has joined #openstack-nova | 15:05 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications https://review.openstack.org/263788 | 15:05 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add service status notification https://review.openstack.org/245678 | 15:05 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Generate doc for versioned notifications https://review.openstack.org/265151 | 15:05 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Add infra for versioned notifications https://review.openstack.org/247024 | 15:05 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 15:05 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Make emitting versioned notifications configurable https://review.openstack.org/247564 | 15:05 |
| johnthetubaguy | tobasco: bauzas: feels like it should really check the disk_gb relative to the requested volumes, but I don't see that happening, but thats separate to the resource tracker | 15:05 |
| *** zenoway_ has quit IRC | 15:05 | |
| *** xyang1 has joined #openstack-nova | 15:06 | |
| *** dave-mccowan has joined #openstack-nova | 15:06 | |
| *** Jeffrey4l has quit IRC | 15:06 | |
| *** gszasz has joined #openstack-nova | 15:07 | |
| *** moshele has joined #openstack-nova | 15:07 | |
| *** dave-mcc_ has joined #openstack-nova | 15:07 | |
| *** cdent has quit IRC | 15:10 | |
| *** smatzek has joined #openstack-nova | 15:10 | |
| dansmith | ndipanov: yeah, I'd really like to see that thing broken out into a separate file with more consistent formatting and structure | 15:11 |
| tobasco | johnthetubaguy: bauzas ok, i have yet only checked out the code for the resource tracker to see how it calculates the values that are then updated in the compute_nodes databas table | 15:11 |
| *** dave-mccowan has quit IRC | 15:11 | |
| johnthetubaguy | tobasco: I think its probably both things | 15:11 |
| johnthetubaguy | tobasco: that one is so the disk filter knows its not request any local disk, the resource tracker stuff is for getting the local claim/report correct | 15:12 |
| *** admin0 has joined #openstack-nova | 15:13 | |
| *** obondarev has quit IRC | 15:14 | |
| *** ansiwen has quit IRC | 15:14 | |
| *** mdbooth has quit IRC | 15:14 | |
| *** migi has quit IRC | 15:14 | |
| gibi | johnthetubaguy: I added the proxy object for the publisher field of the notification as we discussed. | 15:14 |
| johnthetubaguy | gibi: ah, cool | 15:14 |
| gibi | gibi: so if you have time then any comment are appreciated. :) | 15:14 |
| *** obondarev has joined #openstack-nova | 15:15 | |
| gibi | I mean s/gibi/johnthetubaguy/ | 15:15 |
| tobasco | johnthetubaguy: ok cool, yeah there would be two changes then, i'm not the optimal person for the job but we need to figure out where to do the changes, for the resource tracker we need to make sure that it can access the is_volume_backed_instance function from the compute api however I cannot see it being instantiated anywhere right now | 15:15 |
| *** migi has joined #openstack-nova | 15:16 | |
| *** ansiwen has joined #openstack-nova | 15:16 | |
| *** mdbooth has joined #openstack-nova | 15:16 | |
| *** artom has quit IRC | 15:17 | |
| johnthetubaguy | tobasco: yeah, it might be a bit trickier, as its really a case of checking all the final BDMs for any "local" device vs "cinder" devices | 15:17 |
| *** bkumar has quit IRC | 15:17 | |
| * bauzas scolling back the convo | 15:18 | |
| bauzas | was otp | 15:18 |
| *** Jeffrey4l has joined #openstack-nova | 15:18 | |
| *** pkholkin has quit IRC | 15:19 | |
| *** admin0 has quit IRC | 15:19 | |
| tobasco | johnthetubaguy: so the conclusion would be two different changes, one too where the diskfilter checks local disk when booting from a volume and another one where the resource tracker is tracking volume size as local disk used | 15:20 |
| tobasco | bauzas: | 15:20 |
| bauzas | not sure I got your discussion, folks | 15:21 |
| bauzas | the DiskFilter is only caring about some information given by host states | 15:22 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'detail', 'download' and 'delete' methods https://review.openstack.org/238309 | 15:22 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'create' and 'update' methods https://review.openstack.org/259097 | 15:22 |
| matrohon | hi all | 15:22 |
| *** pkholkin has joined #openstack-nova | 15:22 | |
| bauzas | if we consider that Cinder provides resources, it's a shared storage | 15:23 |
| matrohon | I'm trying to reproduce the error that I reported for live-migration : https://bugs.launchpad.net/nova/+bug/1441054 | 15:23 |
| openstack | Launchpad bug 1441054 in OpenStack Compute (nova) "live-migration --block-migrate fails with default libvirt flags" [Medium,Incomplete] - Assigned to Mathieu Rohon (mathieu-rohon) | 15:23 |
| bauzas | tobasco: johnthetubaguy: and we know that we suck when it comes to shared resources, hence the resource-providers BP | 15:23 |
| matrohon | however I've got on error from libvirt that I've never seen before | 15:24 |
| *** belmoreira has quit IRC | 15:24 | |
| matrohon | 2016-01-11 15:08:01.765+0000: 28690: debug : virDomainObjListAddLocked:2336 : Requested operation is not valid: domain 'instance-00000001' is already active | 15:24 |
| matrohon | 2016-01-11 15:08:01.765+0000: 28691: debug : virNetClientProgramDispatchError:177 : Requested operation is not valid: domain 'instance-00000001' is already active | 15:24 |
| matrohon | if anybody can help :) | 15:25 |
| *** admin0 has joined #openstack-nova | 15:25 | |
| *** lucas-hungry is now known as lucasagomes | 15:25 | |
| johnthetubaguy | sorry lost all my IRC messages for last 10 mins, stupid parents internet | 15:27 |
| *** nkrinner has quit IRC | 15:27 | |
| *** yamahata has joined #openstack-nova | 15:27 | |
| tobasco | johnthetubaguy: http://paste.openstack.org/show/483441/ | 15:27 |
| bauzas | heh | 15:27 |
| *** mragupat has joined #openstack-nova | 15:27 | |
| *** omnipresent has joined #openstack-nova | 15:28 | |
| tobasco | matrohon: any interesting messages in the nova-compute logs on both sides? | 15:29 |
| *** vivekd has joined #openstack-nova | 15:30 | |
| matrohon | tobasco, well nova is reporting back the libvirt error : Requested operation is not valid: domain 'instance-00000001' is already active | 15:30 |
| *** zenoway has quit IRC | 15:30 | |
| tobasco | matrohon: hm | 15:31 |
| *** thangp has joined #openstack-nova | 15:32 | |
| *** breitz has quit IRC | 15:32 | |
| *** breitz has joined #openstack-nova | 15:32 | |
| matrohon | tobasco, kashyap I'll try to remove VIR_MIGRATE_TUNNELLED flag set by nova, to see if it fixes the issue as stated in the bug report | 15:32 |
| *** zenoway has joined #openstack-nova | 15:32 | |
| *** dane-fichter has quit IRC | 15:33 | |
| matrohon | tobasco, kashyap : actually libvirt is also giving me a warning : warning : qemuMigrationBeginPhase:2332 : NBD in tunnelled migration is currently not supported | 15:33 |
| kashyap | matrohon: Hi | 15:34 |
| kashyap | matrohon: By the way, you might've slighlty misread that OSSN | 15:35 |
| kashyap | Allow me to explain | 15:35 |
| kashyap | matrohon: When you said "indeed, live block migration cannot occur in tunneling mode" -- that's entirely true. | 15:35 |
| *** kfarr has joined #openstack-nova | 15:36 | |
| *** omnipresent has quit IRC | 15:36 | |
| *** mdrabe has quit IRC | 15:36 | |
| kashyap | Since there are _two_ migration approaches, tunneling works fine in the old method (the QMP command '"inc":true' approach). However, it doesn't work with the _new_ method (which is NBD-based). | 15:36 |
| kashyap | Err, I made a typo earlier: s/that's entirely true/that's NOT entirely true/ :-) | 15:37 |
| *** zz_dimtruck is now known as dimtruck | 15:37 | |
| *** garyk has joined #openstack-nova | 15:38 | |
| *** PaulMurray is now known as ptm_away | 15:38 | |
| matrohon | kashyap, ok I see | 15:38 |
| kashyap | That's an important distinction to keep in mind. | 15:38 |
| garyk | bauzas: please look at https://review.openstack.org/265645 | 15:38 |
| *** artom has joined #openstack-nova | 15:38 | |
| *** annegentle has quit IRC | 15:39 | |
| matrohon | kashyap, however the fallback to old method mentioned by markmc doesn't seem to happen with my libvirt : 1.2.9-9 | 15:39 |
| *** Jeffrey4l has quit IRC | 15:40 | |
| kashyap | matrohon: Can you please write a full response in the bug, mention your test, versions, etc. | 15:40 |
| matrohon | kashyap, I'll do some more tests to be sure of that | 15:40 |
| *** omnipresent has joined #openstack-nova | 15:40 | |
| *** zhangyufei has joined #openstack-nova | 15:40 | |
| matrohon | kashyap, I'll do for sure, that's the purpose of my current tests | 15:41 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Use instance in setup_networks_on_host https://review.openstack.org/249110 | 15:41 |
| *** omnipresent has quit IRC | 15:41 | |
| *** zenoway has quit IRC | 15:41 | |
| kashyap | matrohon: As you can see from mine and Mark's tests, with the said versions, libvirt gracefully falls back to the old approach when you try to tunnel. | 15:41 |
| mdbooth | I suspect this is a FAQ: with the deprecation of run_tests.sh, is there a replacement for run_tests.sh -8, which only runs flake8 against files which are currently dirty or modified by the most recent changeset? | 15:42 |
| *** zenoway has joined #openstack-nova | 15:42 | |
| matrohon | kashyap, I see yes, I'm trying to figure out in which version of libvirt this feature has been introduced | 15:43 |
| *** omnipresent has joined #openstack-nova | 15:44 | |
| artom | ndipanov, thanks for the reviews :) I'm aware every patch "beyond" the metadata objects is missing tests. I started by splitting up the initial huge patch, and am now working through the split up series to add tests, address feedback, etc | 15:44 |
| *** Sukhdev has joined #openstack-nova | 15:44 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Only attempt to inject files if the injection disk exists https://review.openstack.org/250872 | 15:45 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove deprecated option libvirt.remove_unused_kernels https://review.openstack.org/265886 | 15:45 |
| kashyap | mdbooth: Something like? tox -epep8 -- -HEAD | 15:46 |
| mdbooth | kashyap: Does that do it? | 15:46 |
| kashyap | mdbooth: I'm no expert, just posted it from some IRC logs :-) | 15:46 |
| *** zhangjn has quit IRC | 15:46 | |
| mdbooth | I said I thought it was a FAQ :) | 15:46 |
| *** dane-fichter has joined #openstack-nova | 15:47 | |
| mdbooth | Well searched :) | 15:47 |
| kashyap | Heh | 15:47 |
| *** zenoway has quit IRC | 15:47 | |
| *** sahid has quit IRC | 15:47 | |
| andrearosa | kashyap: mdbooth just tested and it seems to work | 15:48 |
| kashyap | andrearosa: Thanks | 15:48 |
| mdbooth | kashyap andrearosa: Thanks! | 15:48 |
| ndipanov | artom, cool - there are some inline nits too so try to get them as well | 15:50 |
| *** mdrabe has joined #openstack-nova | 15:52 | |
| *** gcb has joined #openstack-nova | 15:53 | |
| *** tonytan4ever has joined #openstack-nova | 15:53 | |
| *** mtanino has joined #openstack-nova | 15:53 | |
| *** aorourke has joined #openstack-nova | 15:53 | |
| johnthetubaguy | tobasco: bauzas: ah, got those messages now, via paste | 15:54 |
| *** sahid has joined #openstack-nova | 15:54 | |
| johnthetubaguy | bauzas: this is about when you boot from volume, you shouldn't be requesting any local storage in the request_spec / spec_obj, I feel | 15:55 |
| johnthetubaguy | bauzas: this seems totally independent of cinder | 15:55 |
| *** zenoway has joined #openstack-nova | 15:55 | |
| *** Jeffrey4l has joined #openstack-nova | 15:57 | |
| *** sahid has quit IRC | 15:57 | |
| tobasco | johnthetubaguy: do we have the tools to check if a instance is volume backed properly so that we can implement it with the resource tracker, i have confirmed that if we can make sure the root_gb and ephemeral_gb keys is 0 when having a cinder volume the resource tracking should be correct | 15:57 |
| tobasco | the DiskFilter however is a completely different story | 15:58 |
| *** sahid has joined #openstack-nova | 15:58 | |
| johnthetubaguy | tobasco: thats about making sure the spec_obj sent to the scheduler has the correct values in it, I think | 15:58 |
| bauzas | johnthetubaguy: sure, but do you think Cinder should provide the resources ? | 15:59 |
| *** omnipresent has quit IRC | 15:59 | |
| bauzas | johnthetubaguy: because if no, anyone could ask like 10 times an instance with a related volume like 500GB and Cinder backends just beeing full | 16:00 |
| bauzas | unless I missed something | 16:00 |
| *** jlanoux has quit IRC | 16:01 | |
| johnthetubaguy | bauzas: but thats a different issue to this | 16:01 |
| *** jlanoux has joined #openstack-nova | 16:01 | |
| *** femnad has quit IRC | 16:01 | |
| johnthetubaguy | bauzas: the quota should be on the cinder volumes at that point, I believe | 16:01 |
| tobasco | bauzas: ah I see what you mean, that would require a lot of work from Cinder side to provide information about storage states then I guess, I don't think it's nova's job to care if there is storage available or not | 16:01 |
| *** annegentle has joined #openstack-nova | 16:02 | |
| *** cdent has joined #openstack-nova | 16:02 | |
| bauzas | johnthetubaguy: then, agreed | 16:02 |
| tobasco | johnthetubaguy: so the fix would be to modify the payload that the nova-scheduler recieves then? | 16:02 |
| *** jlanoux has quit IRC | 16:02 | |
| bauzas | johnthetubaguy: if people ask for booting a cinder volume, then nova shouldn't care about checking the volume resource | 16:02 |
| johnthetubaguy | bauzas: I am worried about making sure we only worry about local disk usage, when its actually being used | 16:02 |
| johnthetubaguy | bauzas: yeah, that | 16:02 |
| bauzas | okay I see your point then | 16:03 |
| *** jlanoux has joined #openstack-nova | 16:03 | |
| bauzas | honestly, that's again the main problem of being able to have nova creating a cinder volume IMHO | 16:03 |
| johnthetubaguy | bauzas: thats basically my suggestion, here, modfiy spec_obj so it doesn't request local storage when you are not going to use local storage | 16:03 |
| johnthetubaguy | bauzas: well, the API is nuts, but thats a different issue | 16:03 |
| bauzas | because if people would be creating the volume first, and then calling nova saying "eh, take it as the root volume", then I totally agree, we should not care about that | 16:04 |
| johnthetubaguy | bauzas: this affects both cases the same | 16:04 |
| bauzas | johnthetubaguy: I feel the problem is that since we accept that as part of our API, it creates a contract | 16:04 |
| johnthetubaguy | bauzas: in both of those cases, you have no interest in how much hypervisor local disk is available | 16:04 |
| bauzas | johnthetubaguy: sure, I agree on that | 16:04 |
| johnthetubaguy | thats all this bug is about | 16:05 |
| bauzas | johnthetubaguy: okay, lemme be clear => +1 to having DiskFilter just bypassing the root_gb if it's a volume reqspec | 16:05 |
| *** mjura has quit IRC | 16:05 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Enable volume operationa for shelved instances https://review.openstack.org/265898 | 16:06 |
| openstackgerrit | Andrea Rosa proposed openstack/nova: Enable volume operations for shelved instances https://review.openstack.org/265898 | 16:07 |
| tobasco | bauzas: johnthetubaguy would the DiskFilter even need to be modified if it recieves the payload with root_gb already set to 0? | 16:07 |
| johnthetubaguy | bauzas: cool, and matching logic in the resource manager so you don't need to claim stuff | 16:07 |
| johnthetubaguy | tobasco: diskfilter should be unchanged, I think | 16:07 |
| *** dane-fichter has quit IRC | 16:07 | |
| bauzas | mmm | 16:08 |
| bauzas | I wonder the right approach | 16:08 |
| bauzas | it's a boot request problem, not a resource problem, right? | 16:08 |
| bauzas | ie. someone is asking for something that nova doesn't claim | 16:08 |
| tobasco | johnthetubaguy: ah ok got a little confused there | 16:08 |
| johnthetubaguy | well kinda | 16:08 |
| johnthetubaguy | thats where I said the API is crazy | 16:09 |
| bauzas | exactly... | 16:09 |
| bauzas | it's totally confusing, because by some extend, we own some volume resources | 16:09 |
| johnthetubaguy | you say flavor 34 please, except sod that root disk, use volume uuid_5 instead | 16:09 |
| tobasco | the resource tracker wouldn't be an issue if the root_gb is set to 0 already which it should if we do a fix like johnthetubaguy is on | 16:09 |
| *** salv-orl_ has joined #openstack-nova | 16:09 | |
| johnthetubaguy | now the spec object should say | 16:09 |
| johnthetubaguy | OK, no local disk, uuid_5 from cinder, rather than 10Gb local and uuid_5 from cinder | 16:10 |
| *** mriedem has left #openstack-nova | 16:10 | |
| *** lyanchih has quit IRC | 16:10 | |
| *** mriedem has joined #openstack-nova | 16:10 | |
| *** annegentle has quit IRC | 16:11 | |
| *** salv-orlando has quit IRC | 16:12 | |
| tobasco | johnthetubaguy: so the fix would go into build_request_spec func in scheduler/utils.py? setting the instance properties? | 16:13 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from ComputeNode https://review.openstack.org/265521 | 16:13 |
| *** tpatzig has left #openstack-nova | 16:13 | |
| johnthetubaguy | tobasco: thats part of it, I think | 16:13 |
| bauzas | I would say possible yes | 16:14 |
| bauzas | possibly yes | 16:14 |
| bauzas | that sounds the best fit | 16:14 |
| johnthetubaguy | likewise | 16:14 |
| bauzas | because that's where we define what would be the instance | 16:14 |
| bauzas | a VBI doesn't really has root gb, isn't it ? | 16:14 |
| bauzas | but I'm possibly missing the implications of that | 16:14 |
| bauzas | ie. what if we say that the instance_prop.root_gb becomes 0 ? | 16:15 |
| *** hemnafk is now known as hemna | 16:15 | |
| tobasco | we also need access to the is_volume_backed_instance func from the compute api | 16:15 |
| *** Jeffrey4l has quit IRC | 16:16 | |
| tobasco | bauzas: that's the change we need, needs some good testing to make sure it doesn't affect anything else, as you said | 16:16 |
| *** annegentle has joined #openstack-nova | 16:17 | |
| tobasco | johnthetubaguy: or i might be wrong since the is_volume_backed_instance can't be called since we dont have an instance yet, then what variables would be best for checking and being sure the spawned instance is with a volume | 16:17 |
| tobasco | hmm | 16:17 |
| johnthetubaguy | you should have an instance object passed into the conductor I think | 16:18 |
| johnthetubaguy | but you need to check each BDM, I don't think that method helps much | 16:18 |
| johnthetubaguy | hmm, not sure they have been populated at that point though, the BDMs... | 16:19 |
| johnthetubaguy | oh dear | 16:19 |
| johnthetubaguy | tobasco: I have to go do some reviews now, urgently, I will have to come back to this I am afraid, let me know what you find | 16:19 |
| tobasco | johnthetubaguy: bauzas thanks for helping out :) i need to rush now, i'll be sure to read any backlog if you continue on, i have a test environment at my disposal so i'll do some ugly testing | 16:19 |
| johnthetubaguy | hopefully some others on this channel can keep giving oyu a hand | 16:19 |
| tobasco | johnthetubaguy: sure, thanks for helping out | 16:20 |
| * tobasco out | 16:20 | |
| *** daemontool_ has quit IRC | 16:21 | |
| *** _diana_1 has joined #openstack-nova | 16:24 | |
| *** zhangyufei has quit IRC | 16:26 | |
| *** edtubill has joined #openstack-nova | 16:26 | |
| *** admin0 has quit IRC | 16:27 | |
| *** Jeffrey4l has joined #openstack-nova | 16:29 | |
| *** bklei_ has joined #openstack-nova | 16:30 | |
| *** jdurgin1 has joined #openstack-nova | 16:31 | |
| *** ljxiash has quit IRC | 16:31 | |
| *** tongli has joined #openstack-nova | 16:31 | |
| *** gcb has quit IRC | 16:31 | |
| *** otter768 has joined #openstack-nova | 16:32 | |
| *** dedery_ has joined #openstack-nova | 16:33 | |
| *** zakora has quit IRC | 16:33 | |
| *** markstur_ has quit IRC | 16:33 | |
| *** tjones has joined #openstack-nova | 16:34 | |
| *** bklei has quit IRC | 16:34 | |
| *** paul-carlton2 has quit IRC | 16:34 | |
| *** bklei_ has quit IRC | 16:35 | |
| *** jaosorior has quit IRC | 16:36 | |
| *** jaosorior has joined #openstack-nova | 16:36 | |
| *** markstur has joined #openstack-nova | 16:36 | |
| *** otter768 has quit IRC | 16:37 | |
| *** dedery has quit IRC | 16:37 | |
| *** dedery_ has quit IRC | 16:38 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from DNSDomain https://review.openstack.org/265916 | 16:39 |
| *** vivekd has quit IRC | 16:40 | |
| *** _diana_1 has quit IRC | 16:42 | |
| *** danpb has quit IRC | 16:44 | |
| *** pratikma_ has quit IRC | 16:50 | |
| *** garyk1 has joined #openstack-nova | 16:53 | |
| *** moshele has quit IRC | 16:54 | |
| *** garyk has quit IRC | 16:55 | |
| *** armax has joined #openstack-nova | 16:55 | |
| *** ihrachys has quit IRC | 16:55 | |
| *** dims has joined #openstack-nova | 16:56 | |
| *** dims_ has quit IRC | 16:56 | |
| *** matrohon has quit IRC | 16:57 | |
| mriedem | tobasco: you're trying to find out in the compute api if it's going to be a volume-backed instance on boot? (boot from volume) | 16:57 |
| mriedem | the bdm's are created in the db here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1200 | 16:57 |
| *** yamahata has quit IRC | 16:58 | |
| *** migi has quit IRC | 16:58 | |
| *** migi has joined #openstack-nova | 16:58 | |
| *** bklei has joined #openstack-nova | 16:58 | |
| *** tongli_ has joined #openstack-nova | 16:59 | |
| *** mgoddard_ has joined #openstack-nova | 16:59 | |
| *** vishwana_ has joined #openstack-nova | 17:00 | |
| *** mgoddard has quit IRC | 17:01 | |
| *** bklei has quit IRC | 17:01 | |
| *** bklei has joined #openstack-nova | 17:01 | |
| *** tongli has quit IRC | 17:02 | |
| *** vishwanathj has quit IRC | 17:03 | |
| markus_z | sdague: would you +W https://review.openstack.org/#/c/260015/4 later today when xen gives its ok? You already did it last Friday but by mistake a new patchset interrupted the integration gate. | 17:03 |
| *** jaosorior has quit IRC | 17:04 | |
| *** jaosorior has joined #openstack-nova | 17:04 | |
| *** yassine__ has quit IRC | 17:05 | |
| *** sfinucan has quit IRC | 17:06 | |
| *** tongli_ has quit IRC | 17:06 | |
| *** rk4n has quit IRC | 17:06 | |
| *** jlanoux has quit IRC | 17:08 | |
| BobBall | markus_z: Looks like Xen is nearly finished... http://zuul.openstack.xenproject.org/ but it hit an issue (test_create_image_specify_multibyte_character_image_name) | 17:08 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from EC2 objects https://review.openstack.org/265933 | 17:08 |
| *** rdopiera has quit IRC | 17:08 | |
| BobBall | markus_z: Suggest you xen: recheck again rather than waiting for results :) | 17:09 |
| markus_z | BobBall: ah, cool, thanks for the heads up, I'll do that | 17:10 |
| *** ildikov has joined #openstack-nova | 17:10 | |
| *** daemontool has joined #openstack-nova | 17:10 | |
| bauzas | mriedem: yeah, it's about notifying the scheduler that it shouldn't care about volume-backed instances if DiskFilter | 17:11 |
| *** daemontool_ has joined #openstack-nova | 17:12 | |
| BobBall | markus_z: Alternatively, as patch set #3 succeeded, and the #4 failure must have been a race rather than related to your change I'm sure sdague could +W without needing to wait for the CI? | 17:12 |
| *** paul-carlton2 has joined #openstack-nova | 17:12 | |
| *** bklei has quit IRC | 17:13 | |
| *** rk4n has joined #openstack-nova | 17:14 | |
| markus_z | It's not a critical thing, let's wait for the result | 17:14 |
| BobBall | Fair enough. | 17:14 |
| *** daemontool has quit IRC | 17:15 | |
| *** burt has joined #openstack-nova | 17:18 | |
| *** yamahata has joined #openstack-nova | 17:22 | |
| *** markus_z has quit IRC | 17:22 | |
| *** tonytan4ever has quit IRC | 17:23 | |
| sdague | markus_z done | 17:31 |
| sdague | I did read the xenserver ci fail logs, but it didn't seem related | 17:31 |
| sdague | though it did expose an unrelated oslo.db logging issue | 17:31 |
| sdague | so was worth the time | 17:31 |
| dims | :) | 17:31 |
| *** apoorvad has joined #openstack-nova | 17:32 | |
| *** mrkz has quit IRC | 17:41 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from FixedIP https://review.openstack.org/265951 | 17:41 |
| *** doug-fish has quit IRC | 17:41 | |
| *** doug-fish has joined #openstack-nova | 17:42 | |
| *** akshai has joined #openstack-nova | 17:43 | |
| *** vishwanathj has joined #openstack-nova | 17:46 | |
| *** doug-fish has quit IRC | 17:47 | |
| *** claudiub has quit IRC | 17:47 | |
| *** rk4n has quit IRC | 17:47 | |
| *** vishwana_ has quit IRC | 17:48 | |
| *** rk4n has joined #openstack-nova | 17:49 | |
| *** pratikmallya has joined #openstack-nova | 17:50 | |
| *** derekh has quit IRC | 17:51 | |
| *** penick has joined #openstack-nova | 17:55 | |
| *** ijw has joined #openstack-nova | 17:59 | |
| *** sahid has quit IRC | 18:00 | |
| *** e0ne has quit IRC | 18:01 | |
| *** ihrachys has joined #openstack-nova | 18:04 | |
| *** ijw has quit IRC | 18:04 | |
| *** suro-patz has joined #openstack-nova | 18:05 | |
| *** klkumar has quit IRC | 18:06 | |
| *** jdurgin1 has quit IRC | 18:10 | |
| *** pradk_ has joined #openstack-nova | 18:10 | |
| *** omnipresent has joined #openstack-nova | 18:11 | |
| *** pradk has quit IRC | 18:11 | |
| *** pradk_ is now known as pradk | 18:12 | |
| *** unicell1 has joined #openstack-nova | 18:12 | |
| *** lpetrut has joined #openstack-nova | 18:12 | |
| *** zenoway_ has joined #openstack-nova | 18:13 | |
| *** zenoway has quit IRC | 18:14 | |
| *** unicell has quit IRC | 18:14 | |
| *** rk4n_ has joined #openstack-nova | 18:14 | |
| *** yassine__ has joined #openstack-nova | 18:15 | |
| *** jistr has quit IRC | 18:15 | |
| *** omnipresent has quit IRC | 18:16 | |
| *** sileht has quit IRC | 18:17 | |
| *** krtaylor has quit IRC | 18:17 | |
| *** Sukhdev has quit IRC | 18:17 | |
| *** rk4n has quit IRC | 18:18 | |
| *** Sukhdev has joined #openstack-nova | 18:20 | |
| openstackgerrit | Neetu Jain proposed openstack/python-novaclient: Default quota-show to current tenant and user https://review.openstack.org/265971 | 18:20 |
| openstackgerrit | Neetu Jain proposed openstack/python-novaclient: Default quota-show to current tenant and user https://review.openstack.org/265971 | 18:21 |
| *** andreykurilin__ has quit IRC | 18:22 | |
| *** moshele has joined #openstack-nova | 18:25 | |
| *** Sukhdev has quit IRC | 18:28 | |
| *** krtaylor has joined #openstack-nova | 18:28 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy https://review.openstack.org/149625 | 18:29 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 18:29 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties https://review.openstack.org/149626 | 18:29 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 18:29 |
| *** daemontool_ has quit IRC | 18:30 | |
| *** lpetrut has quit IRC | 18:31 | |
| *** lpetrut has joined #openstack-nova | 18:32 | |
| *** sileht has joined #openstack-nova | 18:33 | |
| *** otter768 has joined #openstack-nova | 18:33 | |
| mriedem | cburgess_: reading up on the efficient rbd snapshots spec again before reviewing the code, re: show_image_direct_url in glance, is that a glance v1 and v2 thing, or api version doesn't matter? | 18:33 |
| suro-patz | johnthetubaguy: should I abandon the https://review.openstack.org/#/c/254423/ for Mitaka? suggestion? | 18:34 |
| cburgess_ | mriedem Good question... been a while let me check. | 18:34 |
| cburgess_ | mriedem Been a while can you re-link me the spec? | 18:34 |
| mriedem | http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/rbd-instance-snapshots.html | 18:34 |
| *** aix has quit IRC | 18:35 | |
| mriedem | cburgess_: looks like only glance v2 | 18:36 |
| *** paul-carlton2 has quit IRC | 18:36 | |
| mriedem | flaper87: ^ could you confirm? | 18:36 |
| *** jichen has joined #openstack-nova | 18:36 | |
| cburgess_ | mriedem Yeah I think so. | 18:37 |
| mriedem | https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L748 | 18:37 |
| mriedem | and it's not in glance v3 | 18:37 |
| mriedem | heh | 18:37 |
| cburgess_ | Oh cute.... | 18:38 |
| *** otter768 has quit IRC | 18:38 | |
| cburgess_ | and I have the stupid _ again. I need to fix that.... | 18:38 |
| cburgess_ | mriedem OK well so is lack of v3 support a potential issue? | 18:38 |
| *** rk4n_ has quit IRC | 18:38 | |
| mriedem | idk | 18:38 |
| openstackgerrit | Sean Dague proposed openstack/nova: API sample testing without project_id https://review.openstack.org/256089 | 18:38 |
| mriedem | i'm jumping to the glance channel to ask about it | 18:38 |
| openstackgerrit | Sean Dague proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 18:38 |
| openstackgerrit | Sean Dague proposed openstack/nova: WIP: fixing routes issue https://review.openstack.org/265983 | 18:38 |
| cburgess_ | mriedem OK | 18:39 |
| mriedem | cburgess_: my concern is around all of the things that can fail with the rbd snapshot, | 18:39 |
| mriedem | and whether or not nova will be smart enough to say what actually failed and how the admin should go about fixing it | 18:39 |
| mriedem | like, i'm on glance v1, so move to glance v2 | 18:39 |
| mriedem | but i don't want to enable show_image_direct_url for security reasons | 18:39 |
| cburgess_ | mriedem Right | 18:39 |
| mriedem | but i also don't want to see exceptions in n-cpu logs every time someone creates a snapshot of an instance | 18:40 |
| cburgess_ | Hmm so might need a flag to force it to fail back? | 18:40 |
| mriedem | well, the spec says that if anything in the efficient snapshot operation fails, we cleanup and revert to the old copy method | 18:40 |
| mriedem | full copy | 18:40 |
| cburgess_ | Oh thats right.. we did do something to detect that and fail back. | 18:40 |
| cburgess_ | Forgot we did it that way. | 18:40 |
| mriedem | but what i'm wondering about is if we want a flag to squash even trying the new path until we know it could work, like knowing that i'm using at least glance v2 and have show_image_direct_url=True | 18:41 |
| mriedem | and show_image_direct_url isn't exposed in the glance API from what i can tell, so it's not really discoverable | 18:41 |
| mriedem | well i guess the image would have the direct_url meta property | 18:41 |
| cburgess_ | Yeah | 18:41 |
| mriedem | but is that needed before or after we're doing the snapshot? | 18:42 |
| mriedem | i thought that was when uploading the cloned snapshot to glance | 18:42 |
| cburgess_ | mriedem But if you want a flag that can turn it off I don't think thats terrible. I would argue it should default to try new fall back to old and the flag should be to suppress ever trying new. | 18:42 |
| cburgess_ | mriedem Let me check | 18:42 |
| mriedem | basically step 6 here http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/rbd-instance-snapshots.html#proposed-change | 18:43 |
| *** annegentle has quit IRC | 18:43 | |
| cburgess_ | mriedem Right checking | 18:43 |
| openstackgerrit | Chuck Carmack proposed openstack/nova: config options: centralize 'spice' options https://review.openstack.org/264271 | 18:44 |
| *** davideag_ has quit IRC | 18:46 | |
| *** burgerk has joined #openstack-nova | 18:47 | |
| openstackgerrit | jichenjc proposed openstack/nova: Support mac and type for /servers/ips request https://review.openstack.org/257261 | 18:48 |
| *** ZZelle has joined #openstack-nova | 18:48 | |
| *** e0ne has joined #openstack-nova | 18:49 | |
| *** burgerk has quit IRC | 18:49 | |
| cburgess_ | mriedem Let me check with nic and get back to you. Stuck in a conf call right now and splitting my attention between trying to re-read the review and the call isn't working. So give me a bit. | 18:49 |
| *** burgerk has joined #openstack-nova | 18:49 | |
| artom | Where do the random-looking alphadecimal strings in releasenotes come from? | 18:50 |
| artom | ex: nova/releasenotes/parallels_support_snapshot-29b4ffae300c1f05.yaml | 18:50 |
| *** davideagnello has joined #openstack-nova | 18:50 | |
| mriedem | cburgess_: ok, np | 18:50 |
| artom | git blame seems to indicate the files themselves are added by people, but it looks like there's a tool that generates them? | 18:51 |
| cburgess_ | mriedem I think the only place use it is in determine what pool the image lives in... So see.... | 18:51 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'instance_group' https://review.openstack.org/261316 | 18:51 |
| mriedem | artom: reno | 18:51 |
| cburgess_ | mriedem https://review.openstack.org/#/c/205282/24/nova/virt/libvirt/imagebackend.py line 906 - 921 | 18:51 |
| artom | mriedem, cheers! | 18:51 |
| mriedem | artom: http://docs.openstack.org/developer/reno/usage.html | 18:52 |
| cburgess_ | mriedem I'll confirm with nic | 18:52 |
| cburgess_ | mriedem If thats the only place we could work around that by having a config option that just tells nova which pool all images should live in. Its not as nice as dynamically figuring it out but workable. | 18:53 |
| *** ijw has joined #openstack-nova | 18:53 | |
| *** jaosorior has quit IRC | 18:53 | |
| *** jaosorior has joined #openstack-nova | 18:54 | |
| sdague | migi / dansmith is there a uuidsentinal invocation that gives the uuid without out the '-' in it? | 18:54 |
| dansmith | sdague: no | 18:54 |
| sdague | or should I just run the replace('-','') after if I care | 18:54 |
| *** Guest41804 is now known as mgagne | 18:54 | |
| cburgess_ | mriedem The whole try except block could have a 3 condition which is to fall back on a config option if set. But like I said I'll sync up with nic once he pops online. | 18:54 |
| *** zenoway_ has quit IRC | 18:54 | |
| dansmith | sdague: if there are lots of cases we could have something inbuilt, but if there's just a few, I'd .replace | 18:54 |
| sdague | lets see how many I trip over here | 18:55 |
| *** mgagne is now known as Guest84419 | 18:55 | |
| mriedem | cburgess_: ok, he also has to fix unit tests :) | 18:55 |
| migi | sdague: not at the moment, why? | 18:55 |
| *** barra204 has joined #openstack-nova | 18:55 | |
| cburgess_ | mriedem Also pretty sure that lines 908-909 prevent us from exploding if the direct url stuff isn't enabled. | 18:55 |
| cburgess_ | mriedem Did they bit rot? | 18:55 |
| mriedem | cburgess_: he addressed some comment but didn't update unit tests it looks like | 18:55 |
| cburgess_ | mriedem Yeah just saw that. | 18:56 |
| mriedem | anyway, i'm making comments on it right now so don't really need him updating today | 18:56 |
| *** shakamunyi has quit IRC | 18:56 | |
| cburgess_ | OK I'll have him come talk to you about the direct questions and remind him to fix the unit tests. | 18:56 |
| cburgess_ | mriedem OK yeah if you ask the questions in the review he is pretty good about reviewing and responding to that stuff. | 18:56 |
| mriedem | nikhil is also looking at the spec | 18:56 |
| cburgess_ | But... pretty sure the answer to your question is that we use it to determine the pool and we could fall back to using a config option if needed. | 18:57 |
| *** tongli has joined #openstack-nova | 18:57 | |
| *** SourabhP has joined #openstack-nova | 18:57 | |
| cburgess_ | mriedem Whats the deadline? I know its coming up soon. | 18:58 |
| mriedem | 1/21 | 18:58 |
| nikhil | mriedem: looks like a fair spec. one thing though that may fall in the icky spot is not keeping the feature optional (for RBD in this case). features like image signing, checksum consistencies can potentially break | 18:58 |
| mriedem | nikhil: yeah was talking to cburgess_ about adding a flag to make it optional | 18:59 |
| mriedem | given how many things you have to have configured before this will work | 18:59 |
| cburgess_ | mriedem OK... I'll raise some awareness here. Sounds like we have a few minor issues but looks like we have commitment from you all to get this merged assuming we can work through the last few issues? | 18:59 |
| nikhil | I don't see a clear mention of the config flag so .. yeah that is a must have I would say | 18:59 |
| cburgess_ | mriedem nikhil Yeah we can add a config option if thats whats wanted. We try and avoid that since there is an aversion to adding yet more config options to nova. | 19:00 |
| mriedem | nikhil: there wasn't a mention in the spec | 19:00 |
| mriedem | cburgess_: i wouldn't say 'you all' :) | 19:00 |
| mriedem | i'm just trying to review it | 19:00 |
| nikhil | cburgess_: I know!! nova and configs :) but this is a breaker w/o config | 19:00 |
| mriedem | because i know it missed liberty | 19:00 |
| nikhil | mriedem: gotcha | 19:00 |
| cburgess_ | mriedem Some people seem to like to provide a perfunctory -1 if there is a new option. Better? | 19:00 |
| cburgess_ | nikhil Not at all. It doesn't break at all. It just traps the error and falls back. | 19:01 |
| *** diogogmt has joined #openstack-nova | 19:01 | |
| mriedem | cburgess_: new config options are ok when they make sense | 19:01 |
| mriedem | i agree that dynamically doing things is best if possible | 19:01 |
| cburgess_ | mriedem Well so sounds like possibly a third condition that checks for the config option and falls back to using that if it fails to dynamically determine it? | 19:02 |
| nikhil | cburgess_: I see, you are saying if a error is detected then there is a failover but I was thinking of some corner cases when those errors are not caught | 19:02 |
| cburgess_ | I suppose its possible but we both catch all exceptions and have handling for getting back an empty URL. | 19:02 |
| *** nelsnels_ has quit IRC | 19:02 | |
| *** nelsnelson has joined #openstack-nova | 19:03 | |
| *** doug-fish has joined #openstack-nova | 19:03 | |
| mriedem | let me keep reviewing this before talking any more about config options :) | 19:03 |
| mriedem | i think nic can at least split up some of the refactoring into a dependent change to make this more digestable | 19:03 |
| nikhil | one of them being the signed images, the uploaded images could potentially be useless if not configured properly but then we are also saying the education for a complexly configured nova + glance + backend deployment | 19:04 |
| sdague | migi: because I'm trying to use the uuidsentinel bits on project_id in urls | 19:04 |
| sdague | but that has the form without slashes | 19:04 |
| sdague | it also makes me a little sad that our context modules don't normalize form | 19:05 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: virtual interface tags DB table https://review.openstack.org/264014 | 19:05 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: metadata objects https://review.openstack.org/264015 | 19:05 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: user visible changes https://review.openstack.org/264018 | 19:05 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: compute manager nic tagging support https://review.openstack.org/264017 | 19:05 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging: libvirt driver implementation https://review.openstack.org/264016 | 19:05 |
| nikhil | anyways, I would refrain from making any more judgements on what nova should choose to do . Glance POV: I think it's easier for ops to enable advanced glance features if we have the explicit config. | 19:05 |
| cburgess_ | nikhil Fair enough | 19:06 |
| *** erlon has quit IRC | 19:06 | |
| *** doug-fish has quit IRC | 19:07 | |
| *** doug-fish has joined #openstack-nova | 19:07 | |
| *** annegentle has joined #openstack-nova | 19:09 | |
| *** eil397 has joined #openstack-nova | 19:11 | |
| *** vilobhmm11 has joined #openstack-nova | 19:11 | |
| *** vilobhmm11 has quit IRC | 19:11 | |
| migi | sdague: I would keep the uuid from sentinel as they are and if required convert the comparison to the ones with slashes or vice-versa | 19:11 |
| *** vilobhmm11 has joined #openstack-nova | 19:11 | |
| sdague | migi: yeh, that's what I'm doing right now | 19:12 |
| alaski | sdague: are you expecting that project_ids are uuids? | 19:12 |
| sdague | uuids.project_id.replace('-', '') | 19:12 |
| migi | sdague: is that correct? | 19:12 |
| sdague | alaski: yes.... is that a problem? | 19:12 |
| *** shakamunyi has joined #openstack-nova | 19:12 | |
| *** barra204 has quit IRC | 19:12 | |
| migi | ah ok project uuids in urls got it | 19:13 |
| alaski | well, they're just freeform strings I thought | 19:13 |
| alaski | they certainly aren't uuids at rax | 19:13 |
| sdague | that's because rax doesn't use keystone? | 19:13 |
| migi | project_id must be a valid uuid | 19:13 |
| alaski | they use a "keystone compatible" thing | 19:14 |
| alaski | but that must be a keystone req they weren't aware of | 19:14 |
| sdague | I guess, for some definition of keystone... | 19:14 |
| sdague | ok, well, what format are they there | 19:14 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add helper shim for getting items https://review.openstack.org/265995 | 19:14 |
| alaski | int I think :| | 19:14 |
| sdague | here is the problem, free from project_id means we can't build a route table that works both with and without project_id | 19:15 |
| sdague | because where does /v2.1/extensions/os-agents go? | 19:16 |
| migi | hmm they are in here as xsd:string http://developer.openstack.org/api-ref-identity-v3.html | 19:16 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from ComputeNode https://review.openstack.org/265521 | 19:16 |
| *** zenoway has joined #openstack-nova | 19:17 | |
| sdague | that's actually the issue that tripped over here in the 11th hour of the optional project_id url path | 19:17 |
| *** Guest84419 is now known as mgagne | 19:18 | |
| *** mgagne has quit IRC | 19:18 | |
| *** mgagne has joined #openstack-nova | 19:18 | |
| alaski | what affect does the project_id currently have for /v2/extensions/os-agents ? | 19:19 |
| alaski | I think what I'm missing is why does the project_id affect routing | 19:19 |
| alaski | I thought it was only checked against the context | 19:20 |
| *** zenoway has quit IRC | 19:21 | |
| sdague | /v2/{project_id}/os-agents is a thing | 19:22 |
| sdague | sorry, /v2.1/{project_id}/os-agents is a thing | 19:23 |
| sdague | as is /v2.1/extensions/os-agents | 19:23 |
| sdague | but if project id can be anything, it can be 'extensions' | 19:23 |
| alaski | ah, I see | 19:23 |
| sdague | and the way python routes work is it compiles the mother of all regexes with all the match/variable values | 19:23 |
| ildikov | mriedem: hi | 19:24 |
| sdague | so you have a regex with over 100 branches | 19:24 |
| sdague | plus backtracking | 19:24 |
| openstackgerrit | jichenjc proposed openstack/nova: Add DiskFormat as Enum in fields https://review.openstack.org/229854 | 19:24 |
| notmorgan | sdague: oooh did the glance things merge? cause if so, yay! | 19:24 |
| sdague | and sometimes /v2.1/extensions/os-agents matches to the extensions controller | 19:24 |
| sdague | and sometimes to the os-agents controller | 19:25 |
| sdague | notmorgan: yes, all the full url in api_servers did | 19:25 |
| notmorgan | sdague: WOOOT | 19:25 |
| alaski | sdague: yeah, I see the issue now | 19:25 |
| notmorgan | sdague: :) | 19:25 |
| sdague | notmorgan: :) | 19:25 |
| ildikov | mriedem: I added a comment about maybe introducing multiattach as experimental to one of the patches | 19:26 |
| ildikov | mriedem: what is your opinion about this idea? | 19:27 |
| alaski | sdague: forcing project_id to a uuid is going to cause complications, at least for rax | 19:27 |
| sdague | alaski: yeh, I wonder who else is affected | 19:28 |
| alaski | sdague: would it be possible to just build one routing table, without project_id, and then silently drop it from the URL when received? | 19:28 |
| *** shz has quit IRC | 19:28 | |
| sdague | alaski: how would you know when it was safe to drop? | 19:28 |
| sdague | if we have no idea what a project id looks like | 19:29 |
| migi | sdague: how about then adding uuid_to_uri kind of function | 19:29 |
| sdague | what part of the url can we throw away? | 19:29 |
| *** shz has joined #openstack-nova | 19:29 | |
| *** zenoway has joined #openstack-nova | 19:29 | |
| dansmith | sdague: duh, the project_id part! | 19:29 |
| * dansmith runs | 19:29 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 19:29 |
| alaski | I keep thinking of routing like a tree, and at the first non match you could drop. but a regex obviously won't allow that | 19:30 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 19:30 |
| sdague | alaski: right, except it's not a tree | 19:30 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 19:30 |
| dansmith | alaski: if you could ever get a project called "extensions" your cloud would stop working | 19:30 |
| sdague | honestly, I didn't really understand how python routes fully worked until this morning when I finally got to the bottom of how to address this | 19:30 |
| alaski | dansmith: yeah | 19:31 |
| dansmith | alaski: you could make a two hours hollywood movie on that premise, as long as you had john travolta and vertical falling green text on some screens | 19:31 |
| dansmith | s/hours/hour/ | 19:31 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Auto converge as a default LM parameter https://review.openstack.org/266001 | 19:32 |
| sdague | ok, so we could put a deprecated config option in that lets you override the regex, and give fair warning that it will be deleted in 2 cycles. | 19:32 |
| alaski | dansmith: heh | 19:33 |
| sdague | the regex is a little gross right now because of a routes issue | 19:33 |
| sdague | https://github.com/bbangert/routes/pull/57 | 19:33 |
| *** jaosorior has quit IRC | 19:33 | |
| *** Jeffrey4l has quit IRC | 19:33 | |
| sdague | https://review.openstack.org/#/c/265983/1/nova/api/openstack/__init__.py - L203 | 19:33 |
| alaski | what about comparing whatever comes after /v2/ or /v2.1/ to context.project_id and if they match it's a safe assumption it's a project_id? | 19:34 |
| alaski | and then big warnings about potential breakage if project_id matches a valid part of a path | 19:34 |
| sdague | alaski: that's way after routing happens | 19:34 |
| sdague | we've punted all processing of routes off to the routes module, kind of by design | 19:35 |
| *** ducttape_ has quit IRC | 19:36 | |
| dansmith | I kinda think that if we ever have to actually break old clients then it's not worth making this change | 19:36 |
| sdague | I get the concept, but to the best of my reading we don't have all these pieces together at once | 19:36 |
| dansmith | I think the assumption has always been that we can work around this by just letting them specify it if they want right? | 19:36 |
| sdague | dansmith: specifying which thing? | 19:36 |
| *** moshele has quit IRC | 19:36 | |
| dansmith | sdague: project_id in the url | 19:37 |
| dansmith | er, wait, I might be misremembering | 19:37 |
| mriedem | ildikov: i haven't looked at those changes recently, how would making it experimental work? | 19:37 |
| dansmith | this doesn't break the client, it just means you have to configure your keystone differently, right? | 19:37 |
| sdague | dansmith: right | 19:37 |
| dansmith | without the variable in the url or whatever? | 19:37 |
| dansmith | right, okay | 19:37 |
| sdague | no clients break | 19:37 |
| migi | sorry for maybe foolish question, but why one would ever do any sort of matching rather then exact comparison | 19:37 |
| dansmith | sdague: so ignore what I just said | 19:37 |
| sdague | the challenge is doing the thing that makes no clients break, because we've got these 2 namespaces overlapping. Which works fine if you have a contract about what a project_id is | 19:38 |
| sdague | and is an intractable problem if you don't | 19:38 |
| alaski | sdague: you've adequately convinced me that this is a very tough problem | 19:38 |
| dansmith | sdague: if a deployer removes it from their keystone and configures their nova to not allow it simultaneously, then everything's fine right? | 19:39 |
| migi | how about making a very simple thing of forcing new api to do exact matching and if that fails do the fallback on supporting old api | 19:39 |
| dansmith | sdague: the problem is just operational overlap for making that change? | 19:39 |
| migi | this way old support will eventually go away | 19:39 |
| sdague | migi: it's not really an api, it happens before the api layer | 19:39 |
| sdague | migi: and this is a library, we don't control all the processing details | 19:40 |
| sdague | dansmith: right, assuming that all clients are well behaved and always start with keystone | 19:40 |
| sdague | instead of hardcoding in their api url | 19:40 |
| sdague | because it is computable | 19:40 |
| dansmith | sdague: yeah | 19:40 |
| dansmith | sdague: well, that is a concern | 19:40 |
| sdague | which is an assumption I was not making | 19:41 |
| sdague | right now there is no configuration for this to be one way or the other, it's just always both ways after this microversion | 19:41 |
| ildikov | mriedem: it's a quick idea on top of my head currently | 19:41 |
| *** yassine__ has quit IRC | 19:41 | |
| dansmith | yeah | 19:41 |
| migi | dansmith: do I remember you telling me that this uuid thing is the simplest thing in the entire world ? | 19:41 |
| dansmith | migi: this part of the uuid thing is not your responsibility :P | 19:42 |
| ildikov | mriedem: basically we can add the experimental config option check to the two blocking checks for volume snapshot and boot from volume | 19:42 |
| dansmith | migi: and i was certainly not including this in that "easy" thing :) | 19:42 |
| migi | :) | 19:42 |
| sdague | the best idea I currently have is exposing a validation regex in config for override | 19:42 |
| ildikov | mriedem: and also check it when we enable to multiattach a volume | 19:42 |
| *** mrkz has joined #openstack-nova | 19:42 | |
| sdague | so rax could set it to \d+ | 19:42 |
| ildikov | mriedem: the db, bdm object and detach changes are not harmful in any sense | 19:43 |
| dansmith | sdague: yeah, I dunno | 19:43 |
| sdague | anyway, I've got to fix some unit tests to not all have 'fake' for project regardless I think | 19:43 |
| alaski | sdague: I can't think of anything better, unless we could add further constraints | 19:43 |
| dansmith | or not do this :) | 19:43 |
| alaski | or that | 19:44 |
| sdague | well if we don't do this, we kind of give up fixing the service catalog entirely | 19:44 |
| alaski | v3 was going to remove project_id from the url | 19:44 |
| sdague | this is step 1 for that | 19:44 |
| migi | can't we have a convert function whic converts api2 non valid uuid to an internal correct uuid for all other components to be used | 19:44 |
| alaski | I think a better patch would be to not have this be optional past a given point in time | 19:44 |
| patrickeast | mriedem: hey, back again to bug about https://review.openstack.org/#/c/205726/ :D lost your +2 from a rebase... | 19:44 |
| alaski | s/patch/path/ | 19:44 |
| migi | so this way there is a mapping which will dissapear later | 19:44 |
| dansmith | sdague: can we give people something in paste to handle this if they need it or is that too late again? | 19:45 |
| dansmith | sdague: meaning, a thing that just strips out the project from the Nth spot, where maybe N is configurable | 19:45 |
| sdague | right, but you still need a definition of project_id | 19:45 |
| dansmith | because otherwise paste is broken? | 19:46 |
| ildikov | mriedem: thinking it through again in check_attach we can add the experimental config option check and raise an error if it's not enabled and the volume already attached | 19:46 |
| sdague | because what do you do with /v2.1/extensions/os-agents | 19:46 |
| dansmith | well, | 19:46 |
| sdague | that's actually the test case that caught this | 19:46 |
| dansmith | this would be a thing to that would just let them keep the old url format forever, to some degree | 19:46 |
| *** Jeffrey4l has joined #openstack-nova | 19:47 | |
| *** omnipresent has joined #openstack-nova | 19:47 | |
| dansmith | sdague: so if I'm a deployer, can I configure my nova service entry to be /foo instead of /{project_id} and then get stats on who is hard coding the project? | 19:47 |
| dansmith | like, do that today I mean | 19:47 |
| migi | isn't there a response field telling that some request will be deprecated by XYZ ? | 19:48 |
| *** rushiagr is now known as rushiagr_away | 19:49 | |
| migi | dansmith: so you could have - yes you can use this untill some date, where the /foo will get depricated and removed | 19:49 |
| *** zenoway has quit IRC | 19:49 | |
| dansmith | migi: not that I know of, and certainly not that would indicate to clients that already aren't consulting the service catalog *what* is being deprecated | 19:49 |
| migi | ok | 19:50 |
| dansmith | in the past, we've done this by giving the deployers some way to measure use of a given piece of the api, determine the impact of the change, and let them consult with the problematic parts of their user base | 19:50 |
| dansmith | prior to making the change | 19:50 |
| *** mspreitz has joined #openstack-nova | 19:51 | |
| dansmith | which is why I asked about hard-coding a fake project into the url as a sentinel | 19:51 |
| * migi found the answer http://developer.openstack.org/api-guide/compute/versions.html | 19:51 | |
| dansmith | that's not the answer for communicating this kind of change, IMHO | 19:51 |
| migi | not communicating, but having it in current release of api as status DEPRICATED will give some advice for developers that this will change | 19:52 |
| migi | and communicating is on top of that | 19:52 |
| dansmith | we can't do that unless we raise the minimum version of the api, which we may never do | 19:53 |
| dansmith | and even still, I don't think this is really something we want to cover under microversions | 19:53 |
| migi | ah, ok | 19:53 |
| *** lucasagomes is now known as lucas-dinner | 19:53 | |
| sdague | dansmith: so let me ponder what you are suggesting. Put another middleware in place that pillages the url to add / remove sections pre routing? | 19:54 |
| dansmith | sdague: well, that's one suggestion | 19:54 |
| dansmith | sdague: the other is to have them hard-code a fake project in place of the variable | 19:54 |
| dansmith | sdague: so they can see anyone not using /foo/... is clearly doing it client-side | 19:54 |
| sdague | ok, so if we do that, then we have to throw out our actual check of project_id vs. context.project_id, which we still do | 19:55 |
| dansmith | right | 19:55 |
| dansmith | so we drop that for the moment and say we're going with the context, but still tolerate/ignore the url project, | 19:55 |
| dansmith | gather data and then work on a drop procedure | 19:56 |
| sdague | right, and we still need to know what a url looks like with | 19:56 |
| sdague | sorry, what the project_id looks like for the site specific drop | 19:56 |
| migi | sdague: if we are talking about this project_id should as well something be done with tenant ? I think this is another part that may cause issues | 19:56 |
| auggy | dansmith: sdague: would it be worth possibly trying to get user information about how people are setting the project_id and what values they are using? | 19:57 |
| sdague | before going down this path, I guess the real question is if you use keystone can project_id ever not be a uuid | 19:57 |
| sdague | notmorgan: ? ^^^ | 19:57 |
| notmorgan | yes | 19:57 |
| notmorgan | it can | 19:57 |
| sdague | notmorgan: ok | 19:57 |
| sdague | it can be anything? | 19:58 |
| dansmith | auggy: that'd be the point of my hardcode hack to let people measure how many clients are doing the wrong thing | 19:58 |
| notmorgan | it tehcnically can be anything | 19:58 |
| notmorgan | now. | 19:58 |
| dansmith | sdague: you're eff'd.. give up now :P | 19:58 |
| notmorgan | let me put an asterisk next to that | 19:58 |
| notmorgan | that is mostly history | 19:58 |
| notmorgan | we don't have LDAP assignment | 19:58 |
| auggy | dansmith: i mean for them to provide to us to make sure we don't have any surprise edge cases but yeah that was what i was building on ;) | 19:58 |
| dansmith | auggy: yep | 19:59 |
| notmorgan | ldap assignment is where that mostly occured, but if someone has a custom driver, the id could be anything. in 99% of the cases it will be uuid | 19:59 |
| jroll | can it include '/'? | 19:59 |
| notmorgan | jroll: in the past, yes. | 19:59 |
| jroll | because that would make this regex really wicked | 19:59 |
| sdague | notmorgan: so by "technically anything" you mean by an out of tree driver? | 19:59 |
| notmorgan | jroll: if you're talking about the project_id | 19:59 |
| notmorgan | sdague: yep. | 19:59 |
| jroll | notmorgan: yep | 19:59 |
| sdague | jroll: well that doesn't work in nova today anyway | 19:59 |
| jroll | sure, good point | 20:00 |
| sdague | you can not access nova resources with a project_id that has a '/' in it | 20:00 |
| *** angdraug has joined #openstack-nova | 20:00 | |
| notmorgan | sdague: you'd have a hard time accessing that in keystone | 20:00 |
| notmorgan | too | 20:00 |
| notmorgan | so i think it's safe to say project_id must be URL safe | 20:01 |
| dansmith | group%2Fsubgroup <- done! :P | 20:01 |
| sdague | notmorgan: is there any way with in tree keystone to have non uuid values ? | 20:01 |
| notmorgan | which means '/' is out | 20:01 |
| notmorgan | sdague: not without mucking around in the DB. | 20:01 |
| notmorgan | with sql | 20:01 |
| sdague | ldap? | 20:01 |
| notmorgan | LDAP resource/assignment is dead | 20:01 |
| notmorgan | in tree | 20:01 |
| dansmith | we know of one major public cloud where it's not a uuid | 20:02 |
| dansmith | not using keystone, and generally evil anyway, BUT | 20:02 |
| dansmith | we know it's in the wild, | 20:02 |
| notmorgan | also migration from LDAP -> SQL might make non-uuid ids | 20:02 |
| dansmith | so acting like it's not seems like a fun but not very useful exercise to me | 20:02 |
| notmorgan | fwiw | 20:02 |
| alaski | working on migrating towards keystone, which raises the question on how they're going to handle the non-uuid project_ids | 20:02 |
| notmorgan | requiring it to be a UUID might break deployments that did upgrades | 20:02 |
| notmorgan | or are doing upgrades from <non-sql/non-keystone> -> keystone | 20:03 |
| notmorgan | sdague: in our schemas, project_id is sql.String(length=64) | 20:04 |
| notmorgan | sdague: so it is a safe bet that project_ids will always be <64 chars | 20:04 |
| notmorgan | jroll, alaski, ^ cc | 20:04 |
| notmorgan | but not guaranteed to be UUID. | 20:04 |
| sdague | yeh, length doesn't really matter here | 20:04 |
| sdague | dansmith: right, but they have a well defined schema for it | 20:05 |
| notmorgan | and they will be URL Safe | 20:05 |
| sdague | it's just different | 20:05 |
| notmorgan | beyond that i can't assert anything else. | 20:05 |
| dansmith | sdague: sure, I just mean not providing for that pattern (if thats where you were headed) seems like a non-starter | 20:05 |
| sdague | right | 20:05 |
| sdague | no, I'm not saying not provide anything | 20:05 |
| alaski | just checked, and my rax project_id is a six digit int | 20:06 |
| sdague | I'm trying to scope the number of ways this deviates from our hopeful norm | 20:06 |
| notmorgan | alaski: thats not too bad. | 20:06 |
| notmorgan | alaski: glad it's not >64char ;) | 20:06 |
| sdague | because if we have upstream as uuid | 20:06 |
| sdague | and plugable to others, that feels like we can provide a path through this | 20:07 |
| notmorgan | if it helps there is talk of allowing cloud_admin define the project_id when creating a project | 20:07 |
| notmorgan | sdague: so that we can address an issue where someone deleted an old project. | 20:07 |
| notmorgan | and it needs to be restored. | 20:07 |
| *** lpetrut has quit IRC | 20:07 | |
| notmorgan | ut that also opens the door for the id to be *anything* that is url-safe. | 20:07 |
| sdague | notmorgan: unless you restrict it on the keystone api call | 20:07 |
| sdague | which, please do | 20:07 |
| notmorgan | except we can't 100% restrict it | 20:08 |
| sdague | then I would like my project_id to be 'extensions' | 20:08 |
| notmorgan | because if rax moves to keystone, and needs to restore a project it's either <SQL queries> or create with ID <6 digits> | 20:08 |
| notmorgan | if that makes sense | 20:08 |
| notmorgan | sdague: technically as a cloud admin, if we do this, you could do that. | 20:09 |
| notmorgan | sdague: it's still being discussed for N | 20:09 |
| notmorgan | wont be a feature in mitaka | 20:09 |
| sdague | notmorgan: right, which is a recipe for complete failure | 20:09 |
| sdague | because keystone doesn't know all the reserve words in projects | 20:09 |
| notmorgan | sdague: it's mostly a break-glass scenario | 20:09 |
| notmorgan | that API call will be explicit/different from the normal create project one | 20:10 |
| notmorgan | and if someone wants to shoot themselves in the foot, we can't really stop them | 20:10 |
| sdague | well the problem is they won't know they shot themselves in the foot until a stream of hard to understand helpdesk tickets | 20:10 |
| notmorgan | but just keep in mind that project ids are fluid-ish from historical deployments | 20:10 |
| notmorgan | modern deployments are unlikely to have anything that isn't a SHA256 or UUID value | 20:11 |
| notmorgan | very unlikely | 20:11 |
| notmorgan | no one except HP, RAX, and CERN afaik run custom backends these days | 20:11 |
| *** kmartin has quit IRC | 20:11 | |
| *** markmcclain has quit IRC | 20:12 | |
| *** ihrachys has quit IRC | 20:12 | |
| *** markmcclain has joined #openstack-nova | 20:12 | |
| auggy | sdague: sorry to interrupt, i noticed the microversion bump on https://review.openstack.org/#/c/233076/ missed the test_versions tests and i can do a quick patch to fix those | 20:12 |
| *** mspreitz has quit IRC | 20:13 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model https://review.openstack.org/149627 | 20:14 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties https://review.openstack.org/149626 | 20:14 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus https://review.openstack.org/149628 | 20:14 |
| openstackgerrit | Neetu Jain proposed openstack/python-novaclient: Default quota-show to current tenant and user https://review.openstack.org/265971 | 20:15 |
| *** whenry has joined #openstack-nova | 20:15 | |
| *** omnipresent has quit IRC | 20:18 | |
| sdague | auggy: sure | 20:19 |
| auggy | sdague: also i'm still trying to fully understand the problem, what i'm understanding is the routing logic is looking for the project id in the url and attempting to remove it and there's some difficulty defining what the project_id is exactly due to these edge cases? | 20:21 |
| auggy | would the project_id be in the context at that point or only if the caller used keystone for auth? | 20:21 |
| * auggy might be confused ;) | 20:22 | |
| auggy | what i understood is due to this inconsistency /v2.1/extensions/os-agents sometimes gets interpreted as a call to os-agents to a project called "extensions" | 20:23 |
| *** aginwala has joined #openstack-nova | 20:23 | |
| auggy | what i'm unsure about is if the correct project id would always be in the context and if there was a way to check against that? | 20:24 |
| *** ducttape_ has joined #openstack-nova | 20:24 | |
| *** dslevin has quit IRC | 20:25 | |
| *** ZZelle_ has joined #openstack-nova | 20:27 | |
| *** Sukhdev has joined #openstack-nova | 20:29 | |
| *** zenoway has joined #openstack-nova | 20:33 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: config options: centralize 'spice' options https://review.openstack.org/264271 | 20:33 |
| *** otter768 has joined #openstack-nova | 20:34 | |
| *** ociuhandu has quit IRC | 20:35 | |
| *** ctrath has quit IRC | 20:35 | |
| sdague | auggy: right, so by convention in all the upstream work project_id is always a uuid | 20:37 |
| sdague | and in all the docs | 20:37 |
| *** ZZelle has quit IRC | 20:37 | |
| sdague | for reference, this is the routes regex (line breaks inserted so you can read it) - http://paste.openstack.org/show/483489/ | 20:37 |
| *** zenoway has quit IRC | 20:37 | |
| *** ctrath has joined #openstack-nova | 20:38 | |
| auggy | wow crazy town | 20:38 |
| *** otter768 has quit IRC | 20:38 | |
| *** lpetrut has joined #openstack-nova | 20:40 | |
| *** aginwala has quit IRC | 20:41 | |
| *** gszasz has quit IRC | 20:44 | |
| sdague | yeh, it also looks like we forgot an xml removal | 20:49 |
| *** ctrath has quit IRC | 20:49 | |
| *** markmcclain has quit IRC | 20:50 | |
| *** amotoki has joined #openstack-nova | 20:52 | |
| *** markmcclain has joined #openstack-nova | 20:52 | |
| sdague | you know, I guess we could go the other direction | 20:54 |
| *** ociuhandu has joined #openstack-nova | 20:54 | |
| sdague | and blacklist project_id values | 20:54 |
| *** doug-fish has quit IRC | 20:54 | |
| *** doug-fish has joined #openstack-nova | 20:55 | |
| *** whenry is now known as ipbabble | 20:55 | |
| *** lpetrut has quit IRC | 20:56 | |
| *** aginwala has joined #openstack-nova | 20:57 | |
| *** thangp has quit IRC | 20:58 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 20:59 |
| *** doug-fish has quit IRC | 20:59 | |
| *** doug-fish has joined #openstack-nova | 21:00 | |
| *** apoorvad has quit IRC | 21:02 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add description for server query https://review.openstack.org/259565 | 21:02 |
| sdague | hmmmm no, that's not going to work | 21:04 |
| * cdent passes sdague some sustaining brain food | 21:06 | |
| *** apoorvad has joined #openstack-nova | 21:07 | |
| *** jichen has quit IRC | 21:08 | |
| sdague | cdent: you don't happen to know, or know anyone, that knows the routes maintainer do you? | 21:08 |
| sdague | because there is a patch there which would open up some options here I just sent a PR for | 21:09 |
| cdent | sdague: sorry, I don't | 21:10 |
| cdent | you might check with lukasa, he seems to know a lot of the web-related crowd | 21:11 |
| *** ihrachys has joined #openstack-nova | 21:11 | |
| *** kmartin has joined #openstack-nova | 21:11 | |
| *** zenoway has joined #openstack-nova | 21:11 | |
| *** mrkz has quit IRC | 21:12 | |
| mriedem | tjones: does nsx ci run on stable/liberty? https://review.openstack.org/#/c/259693/ | 21:15 |
| mriedem | tjones: rechecked last week but never got any results on that one | 21:15 |
| tjones | mriedem: i'll check - i think we do periodically but focus on current | 21:15 |
| tjones | just a sec | 21:15 |
| tjones | we can prob trigger that manully if needed | 21:16 |
| *** zenoway has quit IRC | 21:16 | |
| *** aysyd has quit IRC | 21:17 | |
| *** aginwala has quit IRC | 21:17 | |
| *** ctrath1 has joined #openstack-nova | 21:23 | |
| *** pratikmallya has quit IRC | 21:23 | |
| tjones | mriedem: we run on master only. checking to see how much work to trigger manually on liberty | 21:23 |
| *** smatzek has quit IRC | 21:24 | |
| *** e0ne has quit IRC | 21:25 | |
| mriedem | ok | 21:25 |
| dansmith | mriedem: while you wait, feel free to re-ack my live migration patches :P | 21:26 |
| mriedem | on it | 21:27 |
| mriedem | dansmith: while i do that, you can review the first 2 in this backport series for a security bug https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/liberty+topic:bug/1524274 | 21:27 |
| mriedem | the 3rd change isn't merged yet on master | 21:27 |
| dansmith | oh yeah, I'm supposed to be looking at these anyway | 21:28 |
| mriedem | tsk tsk | 21:28 |
| *** dslev has joined #openstack-nova | 21:28 | |
| *** FL1SK has quit IRC | 21:29 | |
| *** suro-patz has quit IRC | 21:32 | |
| mriedem | huh, has anyone noticed this? http://logs.openstack.org/53/247853/21/check/gate-tempest-dsvm-multinode-full/909458c/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-11_20_17_18_542 | 21:32 |
| *** mrkz has joined #openstack-nova | 21:33 | |
| *** suro-patz has joined #openstack-nova | 21:33 | |
| mriedem | looks like broken by https://github.com/openstack/nova/commit/f0d5fc61916f41214da580097a09136e4ed2c99a | 21:35 |
| *** zenoway has joined #openstack-nova | 21:35 | |
| mriedem | dims: isn't this what you were failing on http://logs.openstack.org/53/247853/21/check/gate-tempest-dsvm-multinode-full/909458c/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-01-11_20_17_18_542 ? | 21:36 |
| mriedem | live migration is all fubar since 1/6 | 21:36 |
| *** MVenesio has quit IRC | 21:36 | |
| dims | mriedem: ha! that would do it :) | 21:36 |
| mriedem | i'll fix it | 21:37 |
| *** rk4n has joined #openstack-nova | 21:37 | |
| dims | thanks Matt | 21:38 |
| mriedem | dansmith: +2 on your xen live migration object one | 21:39 |
| dansmith | mriedem: just acked the third one on master | 21:39 |
| dansmith | mriedem: thanks | 21:39 |
| *** zenoway has quit IRC | 21:40 | |
| *** ihrachys has quit IRC | 21:41 | |
| *** ihrachys has joined #openstack-nova | 21:42 | |
| *** aginwala has joined #openstack-nova | 21:42 | |
| cdent | Doies phil day still hang out in parts nova? | 21:43 |
| mriedem | nope | 21:43 |
| cdent | thanks mriedem | 21:43 |
| cdent | I was hoping to ask him about https://review.openstack.org/#/c/28179/ | 21:44 |
| cdent | Oh well, I shall have to use my brain I guess. | 21:44 |
| *** pratikmallya has joined #openstack-nova | 21:46 | |
| bauzas | cdent: what do you want to know ? | 21:47 |
| cdent | bauzas: Just curious about the thing behind destroying the generators. | 21:47 |
| cdent | s/thing/thinking/ | 21:47 |
| *** mrkz has quit IRC | 21:47 | |
| *** pratikma_ has joined #openstack-nova | 21:48 | |
| *** Sukhdev has quit IRC | 21:48 | |
| bauzas | cdent: honestly I don't feel a generator here is super useful | 21:48 |
| cdent | The reason I even looked at it was because I was doing some experimenting with: nova/tests/unit/scheduler/test_caching_scheduler.py with the number of hosts raised up much higher | 21:49 |
| cdent | under those conditions the number of useless calls to filter_all goes way way up | 21:49 |
| cdent | at the high end of the scale picture that's relevant | 21:50 |
| cdent | actually not even "much". I set up 2000 hosts | 21:50 |
| *** pratikmallya has quit IRC | 21:51 | |
| *** ndipanov has quit IRC | 21:51 | |
| cdent | bauzas: why not? | 21:52 |
| *** ducttape_ has quit IRC | 21:54 | |
| bauzas | cdent: because I feel that using generators for 2000 hosts could help, but that's not the main problem | 21:55 |
| cdent | eventually the main problem will be solved, then whatever is next will be the main problem ;) | 21:55 |
| *** ihrachys has quit IRC | 21:55 | |
| mriedem | dansmith: i can already imagine how terrible it's going to be backporting mdbooth's cve fix down through grizzly :( | 21:56 |
| cdent | bauzas: What do you consider the main problem at this point? | 21:56 |
| *** ihrachys has joined #openstack-nova | 21:56 | |
| alaski | bauzas: I'm going through your comments still, but at a high level the reason I'm passing filter_properties into compute/api is because when parsing the AZ we get host/node there and if filter_props aren't created there those need to be passed down https://review.openstack.org/#/c/254434 | 21:57 |
| dansmith | mriedem: want a job? | 21:57 |
| mriedem | mikal: sdague: dims: dansmith: ^ easy fix to unbreak live block migration tests in the multinode jobs | 21:57 |
| mriedem | oops, early | 21:57 |
| mriedem | dansmith: job as backport monkey? | 21:57 |
| mriedem | does it pay in bananas? | 21:57 |
| dansmith | mriedem: heh, no | 21:57 |
| mriedem | gerrit seems....sad | 21:58 |
| *** Sukhdev has joined #openstack-nova | 21:58 | |
| mriedem | slow | 21:58 |
| tjones | mriedem: yes we can trigger manually. doiing it now | 21:58 |
| mriedem | tjones: ok - note that i already approved it :) | 21:58 |
| mriedem | but sure | 21:58 |
| bauzas | cdent: because using generators for helping the memory is good, but I feel that the problem with 2000 hosts is not memory-bound, but rather io-bound | 21:59 |
| bauzas | hence the caching scheduler | 21:59 |
| tjones | :-) | 21:59 |
| bauzas | alaski: sure, I understood that | 21:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: fix TypeError calling _live_migration_copy_disk_paths https://review.openstack.org/266057 | 22:00 |
| mriedem | there we go ^ | 22:00 |
| bauzas | alaski: honestly, I just feel proving forced_host/node to the compute/api seems not really bad IMHO | 22:00 |
| cdent | bauzas: exactly, so wouldn't you want to be able to use generators to exit the filtering process before visiting all the hoststates, if possible? Or is weighing (of all the hosts) really so important? | 22:00 |
| *** suro-patz has quit IRC | 22:01 | |
| mikal | mriedem: you found someone for that review yet? | 22:01 |
| mriedem | mikal: i have not | 22:01 |
| bauzas | alaski: like I said in my summary text, I think that parsing the AZ string is good to be in the API tree but I also feel that hydrating filt_props should be done in the compute module | 22:01 |
| mikal | mriedem: https://review.openstack.org/266057 is the one, yes? | 22:02 |
| mriedem | yes'r | 22:02 |
| * mikal looks | 22:02 | |
| bauzas | cdent: mmm | 22:02 |
| bauzas | cdent: sec, verifying something | 22:02 |
| alaski | bauzas: sure. I liked that compute/api:create was cleaner passing in filter_props, but I can go for reducing duplication | 22:03 |
| bauzas | alaski: tbh, that's just one opinion | 22:04 |
| alaski | I don't have a strong position on it, and all that matters is filter_props is created earlier than now. either place works for me | 22:05 |
| *** davidlenwell has quit IRC | 22:05 | |
| -openstackstatus- NOTICE: Gerrit is restarting to resolve java memory issues | 22:06 | |
| bauzas | alaski: but I feel that the API tree modules should be there for verifying some related stuff like RBAC or parsing the options, not for creating new objects that could be taken by southbound modules | 22:06 |
| *** salv-orlando has joined #openstack-nova | 22:09 | |
| rlrossit | mriedem: ^ clearly it wasn't just you that thought it was slow | 22:09 |
| mriedem | i'm the canary | 22:09 |
| *** thorst has quit IRC | 22:09 | |
| *** thorst has joined #openstack-nova | 22:10 | |
| alaski | bauzas: creating filter_properties is borderline parsing of options, but I'll try it the other way and see how that looks | 22:11 |
| *** salv-orl_ has quit IRC | 22:12 | |
| bauzas | alaski: yeah, you could be right, I dunno honestly | 22:12 |
| *** doug-fish has quit IRC | 22:12 | |
| *** doug-fish has joined #openstack-nova | 22:13 | |
| bauzas | alaski: I don't want to be very opiniated, you know, just I feel that direction | 22:14 |
| *** thorst has quit IRC | 22:14 | |
| bauzas | alaski: so if you see some bad reason, just leave that out and I'll understand | 22:14 |
| bauzas | alaski: tbh, I hate the forced_host API we currently have, and that's something I'd like to modify in a next microversion | 22:15 |
| alaski | it's just personal preference. moving it into compute/api and reducing duplication is worthwhile | 22:15 |
| alaski | I was blisfully ignorant until this patch, I now hate it as well | 22:16 |
| bauzas | alaski: but that's something mid-term | 22:16 |
| bauzas | alaski: you remember the check-destination-on-migrations BP, right ? | 22:16 |
| alaski | yeah | 22:16 |
| bauzas | alaski: it's part of the big API modifications I'd like to see - with forced_host to be killed also | 22:17 |
| *** doug-fish has quit IRC | 22:17 | |
| bauzas | the UX for forced_host is terrible, but that's not the main reason I hate it | 22:17 |
| bauzas | I hate it because it doesn't really claim the resources correctly | 22:17 |
| *** eharney has quit IRC | 22:18 | |
| alaski | ahh. I was just shocked by how bad the ux for it was | 22:18 |
| rlrossit | these dict compat changes help me find quite a bit of untested code. It opens up so many rabbit holes | 22:19 |
| cdent | rlrossit: I was having a similar reaction recently: "where's the test for this...oh" | 22:21 |
| rlrossit | cdent: luckily tempest is catching me on these, but every time I see one of those, I'm like "wow unit tests really should've caught me on this" | 22:21 |
| openstackgerrit | Tracy Jones proposed openstack/nova: change set_stubs to use stub_out in vmwareapi/stubs.py https://review.openstack.org/266060 | 22:21 |
| rlrossit | and it's in the api too, which really should have strong unit tests... | 22:21 |
| cdent | oh well, each day, in every way, we're making it better and better | 22:23 |
| cdent | or something | 22:23 |
| *** jed56 has quit IRC | 22:23 | |
| *** Guest74023 is now known as med_ | 22:23 | |
| rlrossit | cheesy | 22:23 |
| *** med_ has quit IRC | 22:23 | |
| *** med_ has joined #openstack-nova | 22:23 | |
| bauzas | rlrossit: call it Continuous Improvement | 22:23 |
| cdent | rlrossit: that's a bastardization from a pink panther movie | 22:24 |
| *** zenoway has joined #openstack-nova | 22:24 | |
| cdent | (amongst other things) | 22:24 |
| *** aginwala has quit IRC | 22:25 | |
| cdent | https://www.youtube.com/watch?v=Xze_izjdEYk | 22:25 |
| rlrossit | cdent: prepare to feel old, but this movie was way before I was born http://www.imdb.com/title/tt0072081/ | 22:26 |
| rlrossit | so I wouldn't know that quote :P | 22:26 |
| *** pratikma_ has quit IRC | 22:26 | |
| cdent | Sure, but I've seen Casablanca, and it's from way before I was born ;) | 22:26 |
| *** ihrachys has quit IRC | 22:26 | |
| rlrossit | I make mriedem feel old all the time | 22:26 |
| cdent | uh oh | 22:27 |
| cdent | mriedem is a wee babe | 22:27 |
| *** ihrachys has joined #openstack-nova | 22:28 | |
| mriedem | i don't feel old | 22:28 |
| mriedem | only when i talk about staying up until 3am playing video games and twitter | 22:28 |
| *** aginwala has joined #openstack-nova | 22:28 | |
| *** zenoway has quit IRC | 22:28 | |
| * cdent his rocker and goes to the front porch | 22:28 | |
| mriedem | e.g. "idk, don't you <30 year old just stay up until 3am on saturdays playing video games and playing on the twitter" | 22:29 |
| *** davidlenwell has joined #openstack-nova | 22:29 | |
| *** markmcclain has quit IRC | 22:29 | |
| rlrossit | mriedem: yes, the twitter | 22:30 |
| rlrossit | mriedem: you remember Gary Anderson at least right? | 22:30 |
| *** mrkz has joined #openstack-nova | 22:30 | |
| rlrossit | that was '98, I was too young then | 22:30 |
| *** tongli has quit IRC | 22:31 | |
| mriedem | i was watching that game live | 22:31 |
| mriedem | before people could tweet their rage about it | 22:31 |
| *** tongli has joined #openstack-nova | 22:31 | |
| mriedem | it was a simpler time | 22:31 |
| *** Swanson has joined #openstack-nova | 22:31 | |
| *** ihrachys has quit IRC | 22:32 | |
| auggy | they'd just call you and leave a message on your answering machine | 22:32 |
| rlrossit | I very very vaguely remember playing games online on my parent's windows 95 computer | 22:33 |
| mriedem | i've write a letter to my grandma about my displeasure with the outcome of the vikings 98 season | 22:34 |
| mriedem | and she'd send back a check for $4 | 22:34 |
| mriedem | *i'd | 22:34 |
| *** otter768 has joined #openstack-nova | 22:35 | |
| auggy | i used to play hunt the wumpus on my ti-99 ;) | 22:35 |
| mriedem | i think that's a dating site now | 22:35 |
| * auggy will never think of that game the same way ever again | 22:35 | |
| *** tongli has quit IRC | 22:35 | |
| rlrossit | auggy: thank goodness those calculators haven't changed in price ever | 22:36 |
| openstackgerrit | Jake Yip proposed openstack/python-novaclient: Fix broken link https://review.openstack.org/266063 | 22:36 |
| auggy | rlrossit: oh it wasn't a calculator... https://en.wikipedia.org/wiki/Texas_Instruments_TI-99/4A | 22:36 |
| rlrossit | holy cow | 22:36 |
| rlrossit | wow | 22:36 |
| rlrossit | definitely not what I expected | 22:36 |
| openstackgerrit | Jake Yip proposed openstack/python-novaclient: Fix broken link in documentation https://review.openstack.org/266063 | 22:36 |
| mriedem | bauzas: how about fixing live block migration jobs https://review.openstack.org/#/c/266057/ | 22:37 |
| rlrossit | auggy: the only thing I've ever known TI for is making calculators :P | 22:37 |
| rlrossit | this thing did all of calc 3 for me https://en.wikipedia.org/wiki/TI-Nspire_series | 22:38 |
| *** suro-patz has joined #openstack-nova | 22:38 | |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: libvirt: deprecate live/block_migration_flag opts https://review.openstack.org/263436 | 22:39 |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: libvirt: remove the live/block_migration_flag opts https://review.openstack.org/263437 | 22:39 |
| openstackgerrit | Mark McLoughlin proposed openstack/nova: libvirt: add a new live_migration_tunnelled config https://review.openstack.org/263434 | 22:39 |
| rlrossit | I'm pretty sure you can load pokemon on that thing | 22:39 |
| bauzas | mriedem: +W'd before I say hello to my bed | 22:39 |
| mriedem | bauzas: <3 | 22:39 |
| auggy | rlrossit: :) i was very confused when i needed to get a ti-81 for a class haha | 22:39 |
| *** otter768 has quit IRC | 22:40 | |
| openstackgerrit | Eric Brown proposed openstack/nova: Fix config option types https://review.openstack.org/168481 | 22:40 |
| * cdent waves goodnight | 22:40 | |
| *** cdent has quit IRC | 22:40 | |
| auggy | rlrossit: ooh fancy | 22:40 |
| *** markmcclain has joined #openstack-nova | 22:41 | |
| *** doug-fis_ has joined #openstack-nova | 22:42 | |
| *** burgerk_ has joined #openstack-nova | 22:42 | |
| dansmith | auggy: lol, rlrossit is a youngun' | 22:42 |
| rlrossit | dansmith: yet I still find a way to complain about kids these days... | 22:43 |
| dansmith | auggy: ask him what a Commodore PET is and I bet he points you at a tamagotchi | 22:43 |
| *** thorst has joined #openstack-nova | 22:43 | |
| *** pratikmallya has joined #openstack-nova | 22:43 | |
| auggy | or a nintendog ;D | 22:43 |
| rlrossit | auggy: ^ | 22:43 |
| rlrossit | I never had either, but I remember nintendogs much better | 22:43 |
| rlrossit | the commodore PET is as much a pet as a pet rock | 22:44 |
| rlrossit | pet rocks were a thing once right? | 22:44 |
| *** pratikma_ has joined #openstack-nova | 22:45 | |
| *** burgerk has quit IRC | 22:45 | |
| *** rfolco has quit IRC | 22:46 | |
| *** doug-fis_ has quit IRC | 22:46 | |
| *** rk4n_ has joined #openstack-nova | 22:46 | |
| *** rk4n_ has quit IRC | 22:46 | |
| *** burgerk_ has quit IRC | 22:47 | |
| artom | dansmith, so, is there a cost to having both virtual_interfaces and virtual_interface_tags? | 22:48 |
| *** pratikmallya has quit IRC | 22:48 | |
| *** thorst has quit IRC | 22:48 | |
| *** edtubill has quit IRC | 22:48 | |
| *** zenoway has joined #openstack-nova | 22:49 | |
| *** busterswt has quit IRC | 22:49 | |
| *** rk4n has quit IRC | 22:49 | |
| dansmith | artom: sure, more code, more schema | 22:51 |
| mriedem | more confusion | 22:51 |
| mriedem | if i'm new and writing virtual interface looking code, which table do i need to use? | 22:51 |
| artom | dansmith, more code? It makes the code more straightforward because there's no need to differentiate Neutron vs nova-network | 22:52 |
| artom | I mean, as I said in the review, clearly the consensus is to use the existing virtual_interfaces, so I'll do that :) | 22:53 |
| dansmith | artom: yeah but mriedem's point is good.. both have the name "virtual interface" yet one would be exclusively for neutron and one for nova-net | 22:53 |
| dansmith | but yeah, I'm sure it'll be less code | 22:53 |
| *** zenoway has quit IRC | 22:53 | |
| dansmith | your change will be one new object field, one new model field, adding the column and that's mostly it :) | 22:53 |
| artom | Yeah, but then when the tag is written there needs to be some extra logic. | 22:54 |
| artom | If vif exists (because if was added by nova-network) add the tag, otherwise create it. | 22:54 |
| dansmith | artom: extra logic you mean .. "one if statement" right? :) | 22:55 |
| artom | Hey, if statements are hard ;) | 22:55 |
| *** rk4n has joined #openstack-nova | 22:56 | |
| dansmith | if statements are hard, then what? | 22:56 |
| * dansmith makes a joke | 22:56 | |
| artom | And actually, the spec calls for eventually changing/adding tags on existing interfaces, so that logic would probably have to be added eventually... | 22:56 |
| *** kfarr has quit IRC | 22:56 | |
| * artom appreciates the joke and fetches a Rimshot. | 22:57 | |
| dansmith | the spec definitely seems to aim towards adding a tag field to the relevant objects | 22:57 |
| * rlrossit throws a tomato | 22:57 | |
| *** mriedem is now known as mriedem_away | 22:57 | |
| * artom will see if he can crank that out tonight once baby is sleeping... | 22:58 | |
| *** eil397 has quit IRC | 23:00 | |
| *** dslev has quit IRC | 23:00 | |
| *** dave-mcc_ has quit IRC | 23:00 | |
| *** eil397 has joined #openstack-nova | 23:02 | |
| *** thorst has joined #openstack-nova | 23:02 | |
| *** mdrabe has quit IRC | 23:02 | |
| *** thorst_ has joined #openstack-nova | 23:04 | |
| *** smatzek has joined #openstack-nova | 23:05 | |
| *** thorst has quit IRC | 23:07 | |
| *** kylek3h has quit IRC | 23:08 | |
| *** pratikma_ has quit IRC | 23:14 | |
| *** akshai has quit IRC | 23:14 | |
| *** akshai has joined #openstack-nova | 23:16 | |
| *** dimtruck is now known as zz_dimtruck | 23:16 | |
| *** smatzek has quit IRC | 23:16 | |
| *** otter768 has joined #openstack-nova | 23:18 | |
| *** gtt116__ has quit IRC | 23:19 | |
| *** gtt116__ has joined #openstack-nova | 23:19 | |
| *** burt has quit IRC | 23:19 | |
| *** akshai has quit IRC | 23:20 | |
| *** ctrath1 has quit IRC | 23:21 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from FixedIP https://review.openstack.org/265951 | 23:21 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Add tests for metadata functions https://review.openstack.org/266070 | 23:21 |
| *** rlrossit has quit IRC | 23:22 | |
| *** ducttape_ has joined #openstack-nova | 23:25 | |
| *** zenoway has joined #openstack-nova | 23:25 | |
| *** amotoki has quit IRC | 23:25 | |
| *** markmcclain has quit IRC | 23:26 | |
| *** markmcclain has joined #openstack-nova | 23:26 | |
| *** zenoway has quit IRC | 23:29 | |
| *** FL1SK has joined #openstack-nova | 23:29 | |
| *** otter768 has quit IRC | 23:29 | |
| *** aginwala has quit IRC | 23:31 | |
| *** xyang1 has quit IRC | 23:32 | |
| *** aginwala has joined #openstack-nova | 23:35 | |
| *** MVenesio has joined #openstack-nova | 23:37 | |
| *** FL1SK has quit IRC | 23:37 | |
| *** pratikmallya has joined #openstack-nova | 23:37 | |
| *** Sukhdev has quit IRC | 23:37 | |
| *** mragupat has quit IRC | 23:38 | |
| *** alejandrito has quit IRC | 23:40 | |
| *** Sukhdev has joined #openstack-nova | 23:40 | |
| *** MVenesio has quit IRC | 23:42 | |
| *** otter768 has joined #openstack-nova | 23:44 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "wsgi" https://review.openstack.org/252687 | 23:47 |
| *** zenoway has joined #openstack-nova | 23:49 | |
| *** ZZelle_ has quit IRC | 23:50 | |
| *** zz_dimtruck is now known as dimtruck | 23:51 | |
| *** bwallis_ has joined #openstack-nova | 23:53 | |
| *** zenoway has quit IRC | 23:53 | |
| *** gangil has joined #openstack-nova | 23:56 | |
| gangil | Hi, the commit https://github.com/openstack/nova/commit/4140eb40048ab8f7ebbc684b03c64a15bb39aed9 removes nova/cmd/objectstore.py module | 23:57 |
| gangil | this causes failure in starting the n-obj service | 23:57 |
| *** smoriya_ has joined #openstack-nova | 23:57 | |
| gangil | since that imports nova.cmd.objectstore | 23:57 |
| *** rfolco has joined #openstack-nova | 23:58 | |
| *** mragupat has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!