| *** Daisy has joined #openstack-nova | 00:02 | |
| *** alejandrito has quit IRC | 00:08 | |
| *** vishwana_ has quit IRC | 00:08 | |
| *** Daisy has quit IRC | 00:09 | |
| *** ziggit has joined #openstack-nova | 00:09 | |
| *** sturlry_ has quit IRC | 00:09 | |
| *** Daisy has joined #openstack-nova | 00:09 | |
| *** artom has quit IRC | 00:10 | |
| *** signed8bit has joined #openstack-nova | 00:10 | |
| *** signed8bit has quit IRC | 00:11 | |
| *** ctrath has joined #openstack-nova | 00:12 | |
| *** mwagner_lap has joined #openstack-nova | 00:14 | |
| *** Daisy has quit IRC | 00:14 | |
| *** artom has joined #openstack-nova | 00:15 | |
| *** EinstCrazy has quit IRC | 00:25 | |
| *** signed8bit has joined #openstack-nova | 00:25 | |
| *** ctrath has quit IRC | 00:27 | |
| *** sturlry has joined #openstack-nova | 00:28 | |
| *** sturlry_ has joined #openstack-nova | 00:30 | |
| *** agireud has quit IRC | 00:30 | |
| *** otter768 has joined #openstack-nova | 00:30 | |
| *** sturlry_ has quit IRC | 00:30 | |
| *** sturlry_ has joined #openstack-nova | 00:31 | |
| *** zhangjn has joined #openstack-nova | 00:32 | |
| *** signed8bit has quit IRC | 00:32 | |
| *** sturlry has quit IRC | 00:32 | |
| *** rohit_ has quit IRC | 00:33 | |
| *** salv-orl_ has quit IRC | 00:34 | |
| *** rohit_ has joined #openstack-nova | 00:34 | |
| *** sturlry_ is now known as sturlry | 00:35 | |
| *** agireud has joined #openstack-nova | 00:36 | |
| *** agireud has quit IRC | 00:36 | |
| *** agireud has joined #openstack-nova | 00:36 | |
| *** pratikmallya has joined #openstack-nova | 00:37 | |
| *** RichardRaseley has quit IRC | 00:39 | |
| *** markvoelker has quit IRC | 00:39 | |
| *** aginwala has quit IRC | 00:43 | |
| *** aginwala has joined #openstack-nova | 00:46 | |
| *** gongysh has quit IRC | 00:48 | |
| *** hemna has quit IRC | 00:49 | |
| *** suro-patz has quit IRC | 00:50 | |
| *** agireud has quit IRC | 00:51 | |
| *** davideagnello has quit IRC | 00:52 | |
| *** davideagnello has joined #openstack-nova | 00:52 | |
| *** markvoelker has joined #openstack-nova | 00:55 | |
| *** amotoki has joined #openstack-nova | 00:57 | |
| *** otter768 has quit IRC | 01:00 | |
| *** EinstCrazy has joined #openstack-nova | 01:02 | |
| *** dims has quit IRC | 01:04 | |
| *** gongysh has joined #openstack-nova | 01:05 | |
| *** gongysh has quit IRC | 01:05 | |
| *** doug-fish has quit IRC | 01:05 | |
| *** mtanino has quit IRC | 01:06 | |
| *** aorourke is now known as aorourke-afk | 01:06 | |
| *** daemontool has quit IRC | 01:08 | |
| *** ljxiash has joined #openstack-nova | 01:09 | |
| *** dims has joined #openstack-nova | 01:09 | |
| *** aginwala has quit IRC | 01:13 | |
| *** suro-patz has joined #openstack-nova | 01:13 | |
| *** jwcroppe has joined #openstack-nova | 01:13 | |
| *** darrenc_afk is now known as darrenc | 01:13 | |
| *** arunkant_ has quit IRC | 01:14 | |
| *** zhangjn has quit IRC | 01:17 | |
| *** zhangjn has joined #openstack-nova | 01:18 | |
| *** ziggit has quit IRC | 01:19 | |
| *** ziggit has joined #openstack-nova | 01:20 | |
| *** ziggit has quit IRC | 01:24 | |
| *** aginwala has joined #openstack-nova | 01:28 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Make scheduler_hints schema allow list of id https://review.openstack.org/259247 | 01:30 |
|---|---|---|
| oomichi | alex_xu: hello | 01:32 |
| oomichi | alex_xu: Nova v2.1 validation still has an backwards incompatible issue. | 01:33 |
| oomichi | alex_xu: can you take a look at https://review.openstack.org/#/c/259247/ ? | 01:33 |
| oomichi | sdague: johnthetubaguy: hi, Nova v2.1 validation still has an backwards incompatible issue, can you take a look at https://review.openstack.org/#/c/259247/ ? | 01:34 |
| *** takashin has joined #openstack-nova | 01:35 | |
| *** ljxiash has quit IRC | 01:35 | |
| *** ljxiash has joined #openstack-nova | 01:35 | |
| *** aginwala has quit IRC | 01:36 | |
| *** aginwala has joined #openstack-nova | 01:38 | |
| oomichi | sdague: related to the above problem, the scheduler_hint is not available on the gate now. To test it on the gate, https://review.openstack.org/#/c/258882/ is devstack patch. | 01:38 |
| oomichi | sdague: can you take a look at https://review.openstack.org/#/c/258882/ ? | 01:39 |
| *** ljxiash has quit IRC | 01:40 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a compute API to trigger crash dump in instance. https://review.openstack.org/202605 | 01:40 |
| *** thorst has joined #openstack-nova | 01:41 | |
| *** otter768 has joined #openstack-nova | 01:44 | |
| openstackgerrit | yuntongjin proposed openstack/nova: _can_fallocate should throw a warning instead of error https://review.openstack.org/257638 | 01:45 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 01:46 |
| *** oomichi is now known as oomichi_away | 01:48 | |
| tangchen | Hi alex_xu, I updated that patch, and would you please help to see if I used the mock correctly as you think ? It's not in a hurry, please review when you are free. https://review.openstack.org/#/c/202605/29/nova/tests/unit/compute/test_compute_api.py | 01:49 |
| tangchen | alex_xu: thanks. :) | 01:49 |
| *** thorst has quit IRC | 01:49 | |
| *** browne has quit IRC | 01:49 | |
| *** rfolco has joined #openstack-nova | 01:50 | |
| *** agireud has joined #openstack-nova | 01:51 | |
| *** zenoway has joined #openstack-nova | 01:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 01:53 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "keymgr" https://review.openstack.org/255044 | 01:54 |
| *** rk4n has quit IRC | 01:54 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: Centralise floating ip options https://review.openstack.org/257087 | 01:54 |
| *** ssurana has quit IRC | 01:55 | |
| *** haomaiwa_ has joined #openstack-nova | 01:56 | |
| *** tjones has quit IRC | 01:56 | |
| *** zenoway has quit IRC | 01:57 | |
| *** mrkz has quit IRC | 01:59 | |
| *** agireud has quit IRC | 01:59 | |
| *** zhangjn has quit IRC | 02:00 | |
| *** haomaiwa_ has quit IRC | 02:01 | |
| *** rfolco has quit IRC | 02:02 | |
| *** agireud has joined #openstack-nova | 02:02 | |
| *** haomaiwang has joined #openstack-nova | 02:03 | |
| *** pratikmallya has quit IRC | 02:04 | |
| *** penick has quit IRC | 02:05 | |
| *** yamahata has quit IRC | 02:07 | |
| *** zenoway has joined #openstack-nova | 02:07 | |
| *** agireud has quit IRC | 02:07 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions https://review.openstack.org/249016 | 02:11 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 02:11 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: adding columns to PciDevice table https://review.openstack.org/249013 | 02:11 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object https://review.openstack.org/249015 | 02:11 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: querry to retrieve all pci device by parent address https://review.openstack.org/249014 | 02:11 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests https://review.openstack.org/259254 | 02:11 |
| *** zenoway has quit IRC | 02:12 | |
| *** Jeffrey4l has joined #openstack-nova | 02:13 | |
| *** vilobhmm11 has quit IRC | 02:20 | |
| *** agireud has joined #openstack-nova | 02:23 | |
| *** Marga_ has quit IRC | 02:23 | |
| *** Marga_ has joined #openstack-nova | 02:23 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 02:24 |
| *** ducttape_ has joined #openstack-nova | 02:28 | |
| *** pratikmallya has joined #openstack-nova | 02:30 | |
| *** zenoway has joined #openstack-nova | 02:33 | |
| *** edtubill has joined #openstack-nova | 02:33 | |
| *** suro-patz has quit IRC | 02:33 | |
| *** signed8bit has joined #openstack-nova | 02:34 | |
| *** zenoway has quit IRC | 02:37 | |
| *** edtubill has quit IRC | 02:38 | |
| *** salv-orlando has joined #openstack-nova | 02:38 | |
| *** jwcroppe has quit IRC | 02:38 | |
| *** jwcroppe has joined #openstack-nova | 02:39 | |
| *** signed8bit has quit IRC | 02:40 | |
| *** kragniz has quit IRC | 02:41 | |
| *** jwcroppe has quit IRC | 02:43 | |
| *** apoorvad has quit IRC | 02:46 | |
| *** kragniz has joined #openstack-nova | 02:46 | |
| *** zhenq has quit IRC | 02:47 | |
| *** dims has quit IRC | 02:50 | |
| *** aginwala has quit IRC | 02:51 | |
| *** zhangjn has joined #openstack-nova | 02:55 | |
| *** agireud has quit IRC | 02:57 | |
| *** zenoway has joined #openstack-nova | 02:58 | |
| *** pratikmallya has quit IRC | 02:58 | |
| *** unicell has quit IRC | 02:58 | |
| *** baoli_ has quit IRC | 02:59 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** salv-orl_ has joined #openstack-nova | 03:02 | |
| *** JohnnyCI has joined #openstack-nova | 03:02 | |
| *** agireud has joined #openstack-nova | 03:02 | |
| *** JohnnyCI has left #openstack-nova | 03:02 | |
| *** zenoway has quit IRC | 03:03 | |
| *** salv-orlando has quit IRC | 03:05 | |
| *** houming has joined #openstack-nova | 03:05 | |
| *** RA has joined #openstack-nova | 03:05 | |
| *** RA is now known as Guest39668 | 03:06 | |
| *** Guest39668 is now known as _RA | 03:08 | |
| *** sturlry has quit IRC | 03:11 | |
| *** salv-orl_ has quit IRC | 03:11 | |
| *** aginwala has joined #openstack-nova | 03:12 | |
| *** cfarquhar has quit IRC | 03:12 | |
| *** aginwala has quit IRC | 03:13 | |
| *** Jeffrey4l has quit IRC | 03:14 | |
| *** links has joined #openstack-nova | 03:14 | |
| *** amotoki has quit IRC | 03:15 | |
| *** angdraug has quit IRC | 03:21 | |
| *** gongysh has joined #openstack-nova | 03:22 | |
| *** zenoway has joined #openstack-nova | 03:22 | |
| *** jyuso1 has joined #openstack-nova | 03:23 | |
| *** shz has quit IRC | 03:24 | |
| *** vilobhmm11 has joined #openstack-nova | 03:26 | |
| *** zenoway has quit IRC | 03:27 | |
| *** Jeffrey4l has joined #openstack-nova | 03:27 | |
| *** vilobhmm111 has joined #openstack-nova | 03:28 | |
| *** vilobhmm11 has quit IRC | 03:31 | |
| *** Jeffrey4l has quit IRC | 03:33 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 03:38 |
| *** gyee has quit IRC | 03:40 | |
| *** ducttape_ has quit IRC | 03:40 | |
| *** Jeffrey4l has joined #openstack-nova | 03:40 | |
| *** vilobhmm111 has quit IRC | 03:40 | |
| *** amotoki has joined #openstack-nova | 03:43 | |
| *** gongysh has quit IRC | 03:45 | |
| *** markvoelker has quit IRC | 03:47 | |
| *** oomichi has joined #openstack-nova | 03:48 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Fix VIF plug and unplug problem https://review.openstack.org/254022 | 03:49 |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 03:52 |
| *** shz has joined #openstack-nova | 03:55 | |
| *** ducttape_ has joined #openstack-nova | 03:55 | |
| *** zhangjn has quit IRC | 03:56 | |
| *** Jeffrey4l__ has joined #openstack-nova | 03:57 | |
| *** rushiagr_away is now known as rushiagr | 03:57 | |
| *** rushiagr is now known as rushiagr_away | 03:58 | |
| *** Jeffrey4l has quit IRC | 03:58 | |
| *** amotoki has quit IRC | 03:58 | |
| *** Jeffrey4l__ is now known as Jeffrey4l | 03:58 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-nova | 04:01 | |
| *** gongysh has joined #openstack-nova | 04:04 | |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions https://review.openstack.org/249016 | 04:09 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment https://review.openstack.org/249018 | 04:09 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests https://review.openstack.org/259254 | 04:09 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: adding columns to PciDevice table https://review.openstack.org/249013 | 04:09 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object https://review.openstack.org/249015 | 04:09 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: querry to retrieve all pci device by parent address https://review.openstack.org/249014 | 04:09 |
| *** ducttape_ has quit IRC | 04:12 | |
| *** haomaiwang has quit IRC | 04:15 | |
| *** haomaiwang has joined #openstack-nova | 04:16 | |
| *** artom has quit IRC | 04:16 | |
| *** otter768 has quit IRC | 04:18 | |
| *** salv-orlando has joined #openstack-nova | 04:26 | |
| *** sturlry has joined #openstack-nova | 04:27 | |
| *** zhangjn has joined #openstack-nova | 04:28 | |
| *** albertom has quit IRC | 04:28 | |
| *** zenoway has joined #openstack-nova | 04:28 | |
| *** albertom has joined #openstack-nova | 04:30 | |
| *** sturlry_ has joined #openstack-nova | 04:30 | |
| *** salv-orlando has quit IRC | 04:30 | |
| *** sturlry has quit IRC | 04:31 | |
| *** amotoki has joined #openstack-nova | 04:32 | |
| *** zenoway has quit IRC | 04:33 | |
| *** zhangjn has quit IRC | 04:33 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Make scheduler_hints schema allow list of id https://review.openstack.org/259247 | 04:36 |
| *** pixelbeat_ has quit IRC | 04:38 | |
| *** markvoelker has joined #openstack-nova | 04:47 | |
| *** david-lyle has quit IRC | 04:50 | |
| *** markvoelker has quit IRC | 04:52 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** davideagnello has quit IRC | 05:02 | |
| *** oomichi has quit IRC | 05:04 | |
| *** zenoway has joined #openstack-nova | 05:05 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 05:06 |
| *** davideagnello has joined #openstack-nova | 05:08 | |
| *** zenoway has quit IRC | 05:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Add wrapper classes for return-request-id-to-caller https://review.openstack.org/258802 | 05:12 |
| notmorgan | mikal: ping re https://review.openstack.org/#/c/254214/6 and how you'd want to make that "deployable" [i commented, and am happy to see it through] | 05:14 |
| notmorgan | mikal: just want to keep the momentum here so we can have it all landed and happy as quickly as reasonably possible | 05:14 |
| *** ducttape_ has joined #openstack-nova | 05:16 | |
| *** ducttape_ has quit IRC | 05:17 | |
| *** deepakcs has joined #openstack-nova | 05:31 | |
| *** houming has quit IRC | 05:37 | |
| *** jwcroppe has joined #openstack-nova | 05:38 | |
| *** armax has quit IRC | 05:44 | |
| *** gongysh has quit IRC | 05:46 | |
| *** markvoelker has joined #openstack-nova | 05:49 | |
| *** yamahata has joined #openstack-nova | 05:50 | |
| *** markvoelker has quit IRC | 05:54 | |
| *** suro-patz has joined #openstack-nova | 05:55 | |
| *** amotoki has quit IRC | 05:57 | |
| *** zenoway has joined #openstack-nova | 05:58 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a compute API to trigger crash dump in instance. https://review.openstack.org/202605 | 06:02 |
| *** amotoki has joined #openstack-nova | 06:03 | |
| *** zenoway has quit IRC | 06:03 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 06:05 |
| *** serverascode has quit IRC | 06:05 | |
| *** sballe has quit IRC | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/252197 | 06:09 |
| *** sballe has joined #openstack-nova | 06:11 | |
| *** serverascode has joined #openstack-nova | 06:11 | |
| *** otter768 has joined #openstack-nova | 06:18 | |
| *** david-lyle has joined #openstack-nova | 06:21 | |
| *** zenoway has joined #openstack-nova | 06:23 | |
| *** otter768 has quit IRC | 06:23 | |
| *** vilobhmm11 has joined #openstack-nova | 06:25 | |
| *** zenoway has quit IRC | 06:27 | |
| *** coolsvap|away is now known as coolsvap | 06:27 | |
| *** takashin has left #openstack-nova | 06:28 | |
| *** haomaiw__ has joined #openstack-nova | 06:32 | |
| *** haomaiwang has quit IRC | 06:34 | |
| *** mjura has joined #openstack-nova | 06:41 | |
| *** jwcroppe_ has joined #openstack-nova | 06:45 | |
| *** jwcroppe has quit IRC | 06:46 | |
| *** achinm has joined #openstack-nova | 06:47 | |
| *** zenoway has joined #openstack-nova | 06:47 | |
| *** zenoway has quit IRC | 06:52 | |
| alex_xu | tangchen: yea, will do | 06:52 |
| *** jwcroppe_ has quit IRC | 06:54 | |
| alex_xu | oomichi_away: checking... | 06:55 |
| *** zenoway has joined #openstack-nova | 06:55 | |
| openstackgerrit | lyanchih proposed openstack/nova: NUMATopologyFilter raise exception and not continue filter next node https://review.openstack.org/200630 | 06:55 |
| *** zenoway has quit IRC | 06:56 | |
| *** zenoway has joined #openstack-nova | 06:57 | |
| *** jianghuaw has joined #openstack-nova | 07:00 | |
| *** haomaiw__ has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** salv-orlando has joined #openstack-nova | 07:02 | |
| *** zenoway has quit IRC | 07:02 | |
| *** suro-patz has quit IRC | 07:03 | |
| *** jianghuaw has quit IRC | 07:04 | |
| *** amotoki has quit IRC | 07:04 | |
| *** alexschm has joined #openstack-nova | 07:07 | |
| *** salv-orlando has quit IRC | 07:07 | |
| *** amotoki has joined #openstack-nova | 07:12 | |
| *** gongysh has joined #openstack-nova | 07:12 | |
| *** vilobhmm11 has quit IRC | 07:17 | |
| *** gongysh has quit IRC | 07:17 | |
| *** jwcroppe has joined #openstack-nova | 07:19 | |
| *** scheuran has joined #openstack-nova | 07:20 | |
| *** paul-carlton2 has joined #openstack-nova | 07:22 | |
| *** jwcroppe has quit IRC | 07:24 | |
| *** gongysh has joined #openstack-nova | 07:30 | |
| *** rk4n has joined #openstack-nova | 07:33 | |
| *** cfriesen has quit IRC | 07:35 | |
| *** sahid has joined #openstack-nova | 07:36 | |
| *** e0ne has joined #openstack-nova | 07:36 | |
| *** paul-carlton2 has quit IRC | 07:37 | |
| *** paul-carlton2 has joined #openstack-nova | 07:39 | |
| *** rcernin has joined #openstack-nova | 07:41 | |
| *** markvoelker has joined #openstack-nova | 07:50 | |
| *** markvoelker has quit IRC | 07:55 | |
| *** unicell has joined #openstack-nova | 07:55 | |
| *** browne has joined #openstack-nova | 07:55 | |
| *** korean-101 has joined #openstack-nova | 07:56 | |
| korean-101 | hi guys | 07:56 |
| korean-101 | can i ask a question? | 07:56 |
| korean-101 | i setup openstack kilo release | 07:57 |
| korean-101 | and ceph storage (10Gbps) | 07:57 |
| korean-101 | i set writeback cache modes | 07:57 |
| korean-101 | and got a nearline 10GB/s performance | 07:57 |
| *** rk4n has quit IRC | 07:57 | |
| korean-101 | but i reboot the all openstack & ceph hosts | 07:58 |
| korean-101 | i got a 700MB/s performance | 07:58 |
| korean-101 | but i don't know what's wrong | 07:58 |
| korean-101 | any suggestions? | 07:58 |
| *** _RA has quit IRC | 07:58 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** jianghuaw has joined #openstack-nova | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** markus_z has joined #openstack-nova | 08:02 | |
| *** jianghuaw has quit IRC | 08:05 | |
| *** jianghuaw has joined #openstack-nova | 08:11 | |
| *** jianghuaw has quit IRC | 08:11 | |
| *** paul-carlton2 has quit IRC | 08:12 | |
| *** unicell1 has joined #openstack-nova | 08:14 | |
| *** unicell has quit IRC | 08:14 | |
| *** jyuso1 has quit IRC | 08:15 | |
| *** jyuso1 has joined #openstack-nova | 08:15 | |
| *** zakora has joined #openstack-nova | 08:15 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Calculate block_migration in virt drivers if it's None https://review.openstack.org/254101 | 08:18 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Return host and block_migration in task_api._execute() https://review.openstack.org/254102 | 08:18 |
| openstackgerrit | Eli Qiao proposed openstack/nova: WIP: API: Improve os-migrateLive input and output values https://review.openstack.org/259319 | 08:18 |
| *** otter768 has joined #openstack-nova | 08:19 | |
| *** Marga_ has quit IRC | 08:23 | |
| markus_z | Anyone from the XEN CI present? | 08:24 |
| *** Marga_ has joined #openstack-nova | 08:24 | |
| *** otter768 has quit IRC | 08:24 | |
| *** paul-carlton2 has joined #openstack-nova | 08:25 | |
| *** Thelo has quit IRC | 08:26 | |
| *** ildikov has quit IRC | 08:27 | |
| *** Thelo has joined #openstack-nova | 08:29 | |
| *** rdopiera has joined #openstack-nova | 08:30 | |
| *** rdopiera has quit IRC | 08:30 | |
| *** rdopiera has joined #openstack-nova | 08:30 | |
| *** lpetrut has joined #openstack-nova | 08:33 | |
| *** houming has joined #openstack-nova | 08:36 | |
| *** jianghuaw has joined #openstack-nova | 08:42 | |
| gongysh | sdague, hi | 08:45 |
| *** jlanoux has joined #openstack-nova | 08:45 | |
| *** paul-carlton2 has quit IRC | 08:45 | |
| *** jianghuaw has quit IRC | 08:46 | |
| *** oomichi_away is now known as oomichi | 08:49 | |
| oomichi | alex_xu: thanks! | 08:49 |
| *** matrohon has joined #openstack-nova | 08:50 | |
| oomichi | johnthetubaguy: hi | 08:50 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cleanup needless code from oslo-incubator https://review.openstack.org/259044 | 08:51 |
| *** ducttape_ has joined #openstack-nova | 08:51 | |
| *** BobBall has joined #openstack-nova | 08:52 | |
| *** vlaza has joined #openstack-nova | 08:52 | |
| *** rk4n has joined #openstack-nova | 08:53 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 08:55 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new DB API method to retrieve migration for instance https://review.openstack.org/257270 | 08:55 |
| *** rk4n has quit IRC | 08:59 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** e0ne has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** Jeffrey4l has quit IRC | 09:01 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Finish post live migration even when exception occurs https://review.openstack.org/235994 | 09:03 |
| *** Jeffrey4l has joined #openstack-nova | 09:04 | |
| *** derekh has joined #openstack-nova | 09:05 | |
| *** browne has quit IRC | 09:05 | |
| *** macsz has joined #openstack-nova | 09:08 | |
| *** macsz has quit IRC | 09:10 | |
| *** rohit_ has quit IRC | 09:10 | |
| tangchen | Hi oomichi, alex_xu, I made my patch pass the CI. Please help to review when you are free. https://review.openstack.org/#/c/202605/ | 09:12 |
| tangchen | oomichi, alex_xu: Thanks. :) | 09:12 |
| bauzas | oh my manners | 09:15 |
| *** bauzas is now known as bauwser | 09:15 | |
| *** lpetrut has quit IRC | 09:16 | |
| *** e0ne has joined #openstack-nova | 09:17 | |
| *** yassine__ has joined #openstack-nova | 09:17 | |
| *** coolsvap is now known as coolsvap|away | 09:17 | |
| bauwser | markus_z: I missed your question about Xen CI, I'd defer to BobBall | 09:19 |
| bauwser | wazzzup ? | 09:20 |
| markus_z | bauwser: no worries, I mailed them. Something looks odd with their CI. | 09:20 |
| *** salv-orlando has joined #openstack-nova | 09:20 | |
| *** salv-orlando has quit IRC | 09:20 | |
| bauwser | like? | 09:21 |
| markus_z | bauwser: http://zuul.openstack.xenproject.org/scoreboard/?project=openstack%2Fnova&user=jenkins%2Ccitrix_xenserver_ci%2CXenProject-CI&timeframe=24&start=&end=&page_size= | 09:21 |
| *** salv-orlando has joined #openstack-nova | 09:21 | |
| bauwser | oh I see | 09:21 |
| markus_z | The CI is giving a -1 to almost every patch in the last hours | 09:21 |
| markus_z | I did samples of the reviews and it was always a "merge conflict" | 09:22 |
| *** jistr has joined #openstack-nova | 09:25 | |
| *** gongysh has quit IRC | 09:25 | |
| *** meh_di has joined #openstack-nova | 09:26 | |
| *** danpb has joined #openstack-nova | 09:28 | |
| *** paul-carlton2 has joined #openstack-nova | 09:28 | |
| *** paul-carlton1 has joined #openstack-nova | 09:29 | |
| *** coolsvap|away is now known as coolsvap | 09:29 | |
| meh_di | hi, in module nova/virt/libvirt/guest.py there is a method calling in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/guest.py#L145 . I want to know when method destroy () is called , which method of what module is invoked. | 09:31 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add policy check to avoid using qga in realtime context https://review.openstack.org/247585 | 09:32 |
| openstackgerrit | sahid proposed openstack/nova: conductor: fix unbound local variable request_spec https://review.openstack.org/257344 | 09:32 |
| openstackgerrit | sahid proposed openstack/nova: conductor: initialize request_spec to handle expected exception https://review.openstack.org/258011 | 09:32 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add realtime support https://review.openstack.org/197569 | 09:32 |
| *** openstackgerrit has quit IRC | 09:32 | |
| *** openstackgerrit has joined #openstack-nova | 09:32 | |
| *** teju has joined #openstack-nova | 09:34 | |
| *** e0ne has quit IRC | 09:35 | |
| *** amotoki has quit IRC | 09:35 | |
| *** macsz has joined #openstack-nova | 09:35 | |
| *** gszasz has joined #openstack-nova | 09:36 | |
| *** mnestratov has joined #openstack-nova | 09:39 | |
| *** EinstCrazy has quit IRC | 09:40 | |
| *** amotoki has joined #openstack-nova | 09:41 | |
| *** cdent has joined #openstack-nova | 09:41 | |
| *** houming has quit IRC | 09:46 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Replace unicode with six.text_type https://review.openstack.org/259351 | 09:48 |
| *** ndipanov has joined #openstack-nova | 09:48 | |
| *** markvoelker has joined #openstack-nova | 09:51 | |
| *** jianghuaw has joined #openstack-nova | 09:53 | |
| *** links has quit IRC | 09:54 | |
| *** markvoelker has quit IRC | 09:55 | |
| *** pixelbeat_ has joined #openstack-nova | 09:57 | |
| *** jianghuaw has quit IRC | 09:58 | |
| *** pkholkin has joined #openstack-nova | 09:58 | |
| *** pkholkin has quit IRC | 09:58 | |
| *** chenzeng has quit IRC | 09:58 | |
| *** amotoki has quit IRC | 09:58 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| *** pkholkin has joined #openstack-nova | 10:01 | |
| *** sfinucan has joined #openstack-nova | 10:02 | |
| *** zhenguo has quit IRC | 10:02 | |
| *** rushiagr_away is now known as rushiagr | 10:04 | |
| johnthetubaguy | oomichi: sorry, wasn't around earlier, can I help now? | 10:06 |
| *** links has joined #openstack-nova | 10:07 | |
| johnthetubaguy | markus_z: BobBall or anthonyper might be able to answer XenCI question, I know there were previous fighting the tox issue | 10:07 |
| pkholkin | johnthetubaguy, alaski: good day guys! after so many rechecks depending patch was merged! we updated our 'migration' patch according to small question about 'use_slave' arg, please take a look one more time!) https://review.openstack.org/#/c/243496/ thansk! | 10:08 |
| *** danpb has left #openstack-nova | 10:09 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt - Add log if libguestfs can't read host kernel https://review.openstack.org/237547 | 10:13 |
| *** jianghuaw has joined #openstack-nova | 10:15 | |
| openstackgerrit | lyanchih proposed openstack/nova: libvirt - Flavor's quota instance resource not observed for non-ephemeral storage https://review.openstack.org/201019 | 10:15 |
| johnthetubaguy | pkholkin: hey, got a quick question, how does migration_update work here: https://review.openstack.org/#/c/243496/18/nova/db/sqlalchemy/api.py | 10:15 |
| *** ildikov has joined #openstack-nova | 10:16 | |
| johnthetubaguy | pkholkin: does the read context only apply during the get method call, then we revert back to the writer context? | 10:16 |
| markus_z | johnthetubaguy: bauwser: JFYI, BobBall sent me an answer per mail. He restarted the XenProject CI, hopefully this will fix things. | 10:17 |
| johnthetubaguy | cool | 10:18 |
| pkholkin | johnthetubaguy: firstly writer is applied and then reader will not change something | 10:20 |
| pkholkin | writer is stronger than reader) | 10:20 |
| johnthetubaguy | pkholkin: oh, thats simple enough, cool | 10:20 |
| pkholkin | johnthetubaguy: yes, no problem here | 10:20 |
| *** otter768 has joined #openstack-nova | 10:20 | |
| pkholkin | johnthetubaguy: I think that 'context' should be always in the args, in other case smth is really wrong, so I don't feel it is a problem | 10:23 |
| pkholkin | johnthetubaguy: oh, +2, thanks) | 10:24 |
| *** Daisy has joined #openstack-nova | 10:24 | |
| pkholkin | writer beats reader, yes_ | 10:25 |
| *** otter768 has quit IRC | 10:25 | |
| johnthetubaguy | pkholkin: yeah, this looks good to me, but I am curious what others think about it | 10:25 |
| pkholkin | johnthetubaguy: yes, that's great that still you leave comments after +2 to other guys | 10:26 |
| alex_xu | johnthetubaguy: oomichi founds a bug for v2.1 incompatible https://review.openstack.org/#/c/259247/ | 10:26 |
| ndipanov | johnthetubaguy, daily nag: https://review.openstack.org/#/c/229574/9 | 10:28 |
| *** Daisy has quit IRC | 10:29 | |
| *** jwcroppe has joined #openstack-nova | 10:31 | |
| johnthetubaguy | alex_xu: I thought we agreed to add a microversion for those things | 10:38 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Remove redundant migration status update https://review.openstack.org/247519 | 10:39 |
| *** e0ne has joined #openstack-nova | 10:40 | |
| *** mnestratov|2 has joined #openstack-nova | 10:41 | |
| *** EinstCrazy has joined #openstack-nova | 10:41 | |
| *** jwcroppe has quit IRC | 10:43 | |
| *** mnestratov has quit IRC | 10:44 | |
| oomichi | johnthetubaguy: alex_xu: thanks for considering it. | 10:45 |
| oomichi | johnthetubaguy: alex_xu: I feel now we don't need a microversion bump because v2comp also has this backward incompatible issue | 10:46 |
| johnthetubaguy | oomichi: but how can a user work out which clouds has the bug fixed? | 10:46 |
| oomichi | johnthetubaguy: alex_xu: need to backport it to liberty also | 10:47 |
| johnthetubaguy | agreed with apply to older versions of the API, its just the advertising bit thats tricky | 10:47 |
| johnthetubaguy | hmm, not sure about a backport | 10:47 |
| oomichi | johnthetubaguy: nice point, this problem is reported from users who are using v2.0 with the sched_hint. | 10:48 |
| oomichi | johnthetubaguy: they switched to v2.1 and found this problem with their apps. | 10:48 |
| johnthetubaguy | oomichi: agreed, still think we should bump the microversion to advertise the fix, but apply the fix to all microversions, we could consider dropping the version bump on the backport, but thats a question for mriedem | 10:49 |
| oomichi | johnthetubaguy: very nice point. agree with you. | 10:50 |
| oomichi | johnthetubaguy: oh, sorry I missed one thing. now scheduler_hints is out of microversion bump as a rule. | 10:51 |
| *** ekarlso has quit IRC | 10:51 | |
| *** ekarlso has joined #openstack-nova | 10:51 | |
| johnthetubaguy | oomichi: well thats when I assumed it wasn't validated | 10:52 |
| johnthetubaguy | oomichi: now we have validation, if we change it, we need to bump I think, so users of the API can find out what version of the validation a cloud is using | 10:53 |
| paul-carlton1 | johnthetubaguy, do you remember discussion in Toyko of some keystone and/or cross project activity to address token expiry issues? | 10:53 |
| oomichi | johnthetubaguy: the validation way is wrong on the problem | 10:53 |
| *** rk4n has joined #openstack-nova | 10:53 | |
| *** ducttape_ has quit IRC | 10:54 | |
| johnthetubaguy | paul-carlton1: yes I do, I think jamielennox was going to look into that, on the keystone midlewere side | 10:54 |
| *** lpetrut has joined #openstack-nova | 10:54 | |
| johnthetubaguy | paul-carlton1: did you have a specific question on the idea I can help with? | 10:55 |
| oomichi | johnthetubaguy: nice to discuss it on -dev ml anyways. I will send a mail, thanks for your time :) | 10:55 |
| johnthetubaguy | oomichi: yes, its worth thinking about, I suspect we should do all the things we discussed | 10:55 |
| *** yamahata has quit IRC | 10:55 | |
| oomichi | johnthetubaguy: yeah, that is a nice sample for our microversion future. | 10:56 |
| paul-carlton1 | Thanks, will reach out to him, just want to track progress, it is an issue HPE have encounter with respect to live migration and so I have an internal ticket assigned to me that I want to update with details of upstream activity designed to address it | 10:56 |
| *** dims has joined #openstack-nova | 10:57 | |
| alex_xu | johnthetubaguy: I thought our initial API contract is allow that, but we break it, so we should fix the contract and without microversion | 10:58 |
| alex_xu | or something bug in the initial API contract, we should fix it with microversion | 10:59 |
| *** ociuhandu has joined #openstack-nova | 10:59 | |
| alex_xu | anyway let's discussion it in the ml | 10:59 |
| paul-carlton1 | johnthetubaguy, also, I believe there is work being done to address issues with scheduler not honouring all instance attributes when being rescheduled as in resize/live migration, am I remembering right that jay Pipes was progressing this? | 10:59 |
| johnthetubaguy | alex_xu: I think the user of the API needs to know when the fix is applied | 11:00 |
| *** ociuhandu has quit IRC | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| johnthetubaguy | paul-carlton1: I think that one is with bauwser https://blueprints.launchpad.net/nova/+spec/check-destination-on-migrations | 11:01 |
| paul-carlton1 | ta | 11:01 |
| alex_xu | johnthetubaguy: emm... but we don't have way to notice user when we break it | 11:01 |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| johnthetubaguy | paul-carlton1: yeah, I would check with keystone on the progress. FWIW, its a big issues for me around snapshot uploads to glance | 11:01 |
| oomichi | alex_xu: yeah. I think advirtisement is nice idea with microversion bump :) | 11:01 |
| oomichi | alex_xu: I didn't expect the usage of microversion to be honest, but that is the same as removed-project-id. | 11:02 |
| johnthetubaguy | alex_xu: true, well the users request will start failing. We could actually find out the microversion after the regression was applied and document the bug "window" if we wanted to | 11:02 |
| oomichi | johnthetubaguy: alex_xu: at the end of liberty, we fixed this problem without microversion bump actually | 11:03 |
| oomichi | johnthetubaguy: alex_xu: nice to discuss it with all things on ml | 11:04 |
| johnthetubaguy | oomichi: sounds like a mistake, but I don't remember the logic we applied on that one, yeah its a good ML thread | 11:04 |
| oomichi | johnthetubaguy: the middle of space in server name, etc. | 11:04 |
| oomichi | johnthetubaguy: maybe "don't break the existing users" as primary rule. I agree with that. | 11:05 |
| *** rfolco has joined #openstack-nova | 11:06 | |
| *** jianghuaw has quit IRC | 11:07 | |
| johnthetubaguy | oomichi: yeah, that was crazy edge case. I would have preferred an advertising microversion there too, I can't remember why we didn't do that now | 11:08 |
| oomichi | johnthetubaguy: to be honest, I could not find the usage of microversion for advitising at the time. | 11:09 |
| *** rk4n has quit IRC | 11:09 | |
| *** oomichi is now known as oomichi_away | 11:10 | |
| johnthetubaguy | oomichi_away: yeah, not sure we have done that yet | 11:11 |
| *** coolsvap is now known as coolsvap|away | 11:13 | |
| *** haomaiwang has quit IRC | 11:14 | |
| * alex_xu stuck in thinking | 11:14 | |
| sdague | mikal: did you really stop reviewing based on a theoretical exception use that is never needed? | 11:17 |
| *** oomichi_away has quit IRC | 11:19 | |
| tojuvone | oomichi: Hi, like to get agreement with that host_status handling in: https://review.openstack.org/241562 | 11:21 |
| johnthetubaguy | sdague: :S | 11:21 |
| tojuvone | Is it only me or do you also have this failing: nova.tests.unit.test_signature_utils.TestSignatureUtils.test_verify_signature_ECC | 11:26 |
| bauwser | alex_xu: johnthetubaguy: hey, quick q, I'm wondering how I could get the list of instances per host using the novaclient ? It seems that nova.servers.list(search_opts={'hypervisor': host}) is incorrect | 11:27 |
| johnthetubaguy | bauwser: not sure I could tell you what the REST API is honestly | 11:30 |
| *** lucas-dinner is now known as lucasagomes | 11:30 | |
| johnthetubaguy | brain is a big pile of pre-holiday mush | 11:31 |
| bauwser | LOL | 11:31 |
| *** vladikr has quit IRC | 11:31 | |
| sdague | mikal I really don't think we need an else branch. All uses of that class are carefully controlled. Please revisit your -1 on https://review.openstack.org/#/c/254214 | 11:32 |
| johnthetubaguy | bauwser: maybe try host: <hostname> based on this: http://developer.openstack.org/api-ref-compute-v2.1.html#listServers | 11:33 |
| bauwser | johnthetubaguy: cool, it seems to work | 11:34 |
| sdague | ok, can I get second opinions on this - https://review.openstack.org/#/c/254214 from other nova cores | 11:34 |
| johnthetubaguy | sdague: taking a peak for you | 11:34 |
| sdague | as mikal would like me to respin that patch based on a potential fall through condition, which is not triggered over the extremely short lifespan of this class | 11:35 |
| *** cdent has quit IRC | 11:37 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova: Add host_status attribute for servers/detail and servers/{server_id} https://review.openstack.org/241562 | 11:37 |
| *** amotoki has joined #openstack-nova | 11:38 | |
| *** erlon has joined #openstack-nova | 11:39 | |
| alex_xu | bauwser: have you try "?host=" | 11:41 |
| alex_xu | but it is admin-only | 11:41 |
| alex_xu | ah, johnthetubaguy already pointed it out | 11:42 |
| johnthetubaguy | alex_xu: do we not have those all moved over to policy yet? | 11:42 |
| johnthetubaguy | still does is_admin I guess? | 11:42 |
| alex_xu | johnthetubaguy: no, it still admin-only in code | 11:42 |
| bauwser | alex_xu: thanks, that works | 11:43 |
| alex_xu | bauwser: np | 11:44 |
| bauwser | alex_xu: I was missing that the client is passing the search_opts as a query string of the URI | 11:44 |
| *** jwcroppe has joined #openstack-nova | 11:44 | |
| bauwser | speaking of http://docs.openstack.org/developer/python-novaclient/ref/v2/servers.html | 11:45 |
| bauwser | btw. I wonder if we shouldn't document that | 11:45 |
| ansiwen | garyk: I addressed your comment on 258020, but after pushing I saw the comment from Radoslav, that the fix is not sufficient. He pointed me to another one, that is stalled since August, because there is no bug report... confused... | 11:45 |
| *** amotoki has quit IRC | 11:46 | |
| *** rotbeard has joined #openstack-nova | 11:48 | |
| garyk | ansiwen: yeah, it does not address the issue that he mentioned | 11:48 |
| BobBall | markus_z: Took a while to start, but your change is now rechecking on the xenproject CI | 11:48 |
| garyk | ansiwen: i think that he has yet to add the bug or maybe missed it. | 11:49 |
| *** salv-orlando has quit IRC | 11:49 | |
| alex_xu | emm..not sure | 11:49 |
| *** links has quit IRC | 11:50 | |
| markus_z | BobBall: great, thanks! I've seen that your CI is starting to return some +1's, so I assume whatever the issue was, it has disappeared. | 11:50 |
| sdague | sahid: if you can throw a comment in here - https://review.openstack.org/#/c/254709 - we can turn it around quick | 11:50 |
| BobBall | I've looked a bit further and I'm seriously confused why there are no logs at all | 11:50 |
| BobBall | disk wasn't even close to full - but a whole bunch of zuul logs seem to be missing | 11:51 |
| * alex_xu will leave early today, but want to say Merry Christmas to everyone who is vacation from next week | 11:51 | |
| *** markvoelker has joined #openstack-nova | 11:51 | |
| garyk | ansiwen: i suggest that you reach out to rado | 11:52 |
| andrearosa | any cells expert otu there? I have an rpccall which is not supported in the cells.rpcapi, the call is "reserve_block_device_name". is that mean that that call is not supported by cell yet? | 11:52 |
| *** rk4n has joined #openstack-nova | 11:54 | |
| *** teju has quit IRC | 11:55 | |
| ansiwen | garyk: I don't mind which change goes in, I'm just worried, because it's not moving for so long | 11:55 |
| andrearosa | I am writing a unit test which verifies that that method is called, do I need to skip that check for the ComputeAPIAPICellUnitTestCase? | 11:55 |
| garyk | ansiwen: i am not sure what the stats are. but they are very long when it comes to the driver patches. | 11:56 |
| *** markvoelker has quit IRC | 11:56 | |
| garyk | ansiwen: in some cases it is more than a cycle and it becomes a chellenge to stable backport | 11:56 |
| ansiwen | garyk: I con provide a reproducer for the stream_optimized case... not for vSAN though | 11:56 |
| garyk | ansiwen: understaood, but that code wil lbreak the vsan. | 11:57 |
| garyk | i suggest trying to get rado's patch reviewed | 11:57 |
| johnthetubaguy | bauwser: document which bit, search_opts? | 11:57 |
| bauwser | johnthetubaguy: yup, and refer them to the API doc | 11:57 |
| ansiwen | garyk: it breaks it or it just doesn't address it? | 11:57 |
| garyk | ansiwen: rados patch should address it if i recall correctly | 11:58 |
| bauwser | johnthetubaguy: I had to review the code to understand that it was serializing the params so that it was matching with the API optional attrs | 11:58 |
| bauwser | johnthetubaguy: but maybe I'm overthinking | 11:59 |
| johnthetubaguy | bauwser: oh, so python-novaclient docs, rather than REST API docs? | 11:59 |
| bauwser | johnthetubaguy: yup | 11:59 |
| bauwser | johnthetubaguy: I'm fine with the API doc | 11:59 |
| sdague | bauwser: yeh, we could add docs there | 11:59 |
| ansiwen | garyk: sure, I'm just wondering if my patch would break something that works already, or if it just doesn't fix the same bug for vSAN. | 11:59 |
| johnthetubaguy | bauwser: so I would love python-novaclient examples eventually, to match to the same requests | 11:59 |
| sdague | honestly, it should probably be just a better interface and in the help | 11:59 |
| sdague | johnthetubaguy: well, for servers list, it would honestly be better to get folks over to openstackclient | 12:00 |
| ansiwen | garyk: of course, if rados patch is "mergable" and more complete, I also would prefer that one | 12:00 |
| bauwser | johnthetubaguy: sdague: just a brief explanation in https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L572 | 12:00 |
| johnthetubaguy | sdague: true, we do need to move folks over at some point | 12:00 |
| ansiwen | garyk: what is radoslavs irc nick? | 12:01 |
| bauwser | johnthetubaguy: sdague: at least before adding more explanation in the doc tree | 12:01 |
| sdague | bauwser: yeh, sure, is this for CLI or API usage? | 12:01 |
| bauwser | sdague: for API usage, not the CLI | 12:01 |
| bauwser | hence me talking of novaclient | 12:01 |
| bauwser | unless I misunderstood | 12:01 |
| *** baoli has joined #openstack-nova | 12:02 | |
| sdague | oh, I thought you meant about search on the command line | 12:02 |
| ansiwen | garyk: it also means there are vSAN tests missing... | 12:02 |
| bauwser | IIUC, we agreed to have openstackclient as a global CLI that passing thru novaclient for calling, right? | 12:02 |
| sdague | yeh, documentation on search opts would be good, I can take a stab | 12:02 |
| bauwser | sdague: no, I was just playing with the python bindings :) | 12:02 |
| bauwser | not the CLI | 12:03 |
| *** baoli has quit IRC | 12:03 | |
| *** achinm has quit IRC | 12:04 | |
| garyk | ansiwen: the tests are covered. the CI is being worked on | 12:04 |
| sdague | bauwser: while I am writing that up, you want to look at https://review.openstack.org/#/c/254214/ - one +2 will land 4 patches :) | 12:04 |
| garyk | his nickname is rgerganov | 12:04 |
| *** baoli has joined #openstack-nova | 12:04 | |
| openstackgerrit | Chaozhe Chen proposed openstack/python-novaclient: Deprecate 'get_client_class' in test_client https://review.openstack.org/259401 | 12:05 |
| *** baoli has quit IRC | 12:05 | |
| bauwser | sdague: sure, I just need a bit of time for loading a glance related patch :) | 12:05 |
| bauwser | s/patch/reviews | 12:05 |
| *** links has joined #openstack-nova | 12:06 | |
| *** Jeffrey4l has quit IRC | 12:06 | |
| sdague | ok, remind me again why foo={} is terrible python arg defaults. It has been explained to me before, but I forget now | 12:07 |
| *** zenoway has joined #openstack-nova | 12:07 | |
| johnthetubaguy | sdague: if you modify foo, you modify the next call? | 12:10 |
| andrearosa | sdague: I think because the argument are initialized once | 12:10 |
| johnthetubaguy | sdague: I think there is only a single instance of the defaults | 12:10 |
| migi | sdague: http://stackoverflow.com/questions/26320899/why-is-the-empty-dictionary-a-dangerous-default-value-in-python | 12:11 |
| sdague | migi: thanks | 12:11 |
| sdague | ah, ok, it's a state corruption thing | 12:12 |
| sdague | great | 12:12 |
| sdague | thanks folks | 12:12 |
| tojuvone | Should I report bug, if unit tests do not reset time back after using timeutils.set_time_override()? Now next unit test needs to make timeutils.clear_time_override() in order to work. | 12:13 |
| *** cdent has joined #openstack-nova | 12:13 | |
| sahid | sdague: ok i will address you comment, btw thanks for your reviews | 12:15 |
| tojuvone | so basically timeutils.utcnow() will be broken in next test case | 12:15 |
| *** links has quit IRC | 12:16 | |
| sahid | johnthetubaguy: any chance to +W this one ? https://review.openstack.org/#/c/257344/ | 12:16 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Move nova/opts.py to nova/conf/ https://review.openstack.org/255436 | 12:17 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Config options: centralize base path configuration https://review.openstack.org/259406 | 12:17 |
| bauwser | migi: oh interesting, a closure side-effect | 12:18 |
| bauwser | we thank mutable objects | 12:19 |
| migi | bauwser: right, interesting - quick grep and 260 found in nova git | 12:20 |
| *** mjura has quit IRC | 12:21 | |
| migi | bauwser: I'm wrong those are not in func defs | 12:21 |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient: document search_opts parameter https://review.openstack.org/259408 | 12:21 |
| *** otter768 has joined #openstack-nova | 12:21 | |
| sdague | bauwser: ^^ | 12:21 |
| sdague | how about that | 12:21 |
| migi | 0 found | 12:21 |
| sdague | migi: yeh, dansmith has smacked me down about it before | 12:22 |
| sdague | I just forgot why | 12:22 |
| bauwser | sdague: that's exactly what I was going to write | 12:22 |
| migi | sdague: was grepping with .tox so there are some | 12:22 |
| migi | but other libs | 12:22 |
| sdague | migi: yeh | 12:22 |
| bauwser | should we write an hacking path for getting rid of foo={} as an arg ? | 12:22 |
| sdague | migi: git grep is your fried | 12:22 |
| * migi looking | 12:22 | |
| bauwser | I mean, writing the hacking check and piling a list of fix patches | 12:22 |
| sdague | bauwser: honestly, we don't have any instances of it | 12:22 |
| bauwser | oh cool then | 12:23 |
| bauwser | so we can write the hacking check I guess | 12:23 |
| migi | sdague: nice one, thanks | 12:23 |
| sdague | bauwser: or realize it's not a problem | 12:23 |
| bauwser | heh | 12:23 |
| bauwser | :p | 12:23 |
| sdague | we've been catching it correctly in review | 12:23 |
| sdague | knowing why a thing is bad and building that into the culture is better than a hacking rule people just code around until it passes :) | 12:24 |
| bauwser | fair enough | 12:24 |
| sdague | bauwser: I did find an api-site bug while looking at this doc. We document host as a param, but don't say it's typically only available to cloud admins | 12:24 |
| bauwser | tbh, I wasn't catching that although I understand the problem | 12:24 |
| bauwser | sdague: mmm, right | 12:25 |
| sdague | time to patch that | 12:25 |
| bauwser | +1 | 12:25 |
| johnthetubaguy | still think these should all move to policy, like we did for the DB layer | 12:25 |
| *** otter768 has quit IRC | 12:26 | |
| ansiwen | garyk: well CI tests is what I meant. ;-) | 12:26 |
| ansiwen | garyk: in which TZ he is? | 12:27 |
| *** smatzek has joined #openstack-nova | 12:28 | |
| *** salv-orlando has joined #openstack-nova | 12:28 | |
| cdent | johnthetubaguy: before you head out on your end of year recovery (good luck with that): you cool with me being the cross project liaison if nobody else has stepped up? | 12:30 |
| *** thorst has joined #openstack-nova | 12:31 | |
| sdague | https://review.openstack.org/#/c/259414/1 | 12:35 |
| sdague | johnthetubaguy: sure, it's just policy is a whole other giant thing to have to fix | 12:36 |
| sdague | johnthetubaguy: on https://review.openstack.org/#/c/254785 - fixing the reno is the only thing you care about? | 12:38 |
| *** baoli has joined #openstack-nova | 12:38 | |
| *** baoli has quit IRC | 12:38 | |
| sdague | just making sure before I submit the new change | 12:38 |
| *** baoli has joined #openstack-nova | 12:38 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Add upload/download vhd2 interfaces https://review.openstack.org/254785 | 12:39 |
| *** rushiagr is now known as rushiagr_away | 12:39 | |
| sdague | ok, that's fixed | 12:40 |
| *** mjura has joined #openstack-nova | 12:41 | |
| *** mjura_ has joined #openstack-nova | 12:41 | |
| *** kylek3h has quit IRC | 12:47 | |
| *** markvoelker has joined #openstack-nova | 12:52 | |
| *** rsquared has joined #openstack-nova | 12:53 | |
| *** markvoelker has quit IRC | 12:57 | |
| *** BobBall has quit IRC | 12:57 | |
| *** andreykurilin__ has joined #openstack-nova | 12:57 | |
| *** BobBall has joined #openstack-nova | 12:57 | |
| *** fesp has joined #openstack-nova | 12:59 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.ironic' options https://review.openstack.org/259014 | 13:02 |
| openstackgerrit | Dina Belova proposed openstack/nova: == WIP == Integrate OSprofiler and Nova https://review.openstack.org/254703 | 13:02 |
| *** fesp has quit IRC | 13:03 | |
| *** ljxiash has joined #openstack-nova | 13:03 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused pipeline_factory_v3 alias https://review.openstack.org/259185 | 13:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove vestigial XML_NS_V11 variable https://review.openstack.org/259191 | 13:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused EXTENSION_DESERIALIZE_* constants https://review.openstack.org/259189 | 13:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused extends_name attribute https://review.openstack.org/259179 | 13:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused methods from integrated_helpers test class https://review.openstack.org/259180 | 13:03 |
| sdague | markus_z: if you respin https://review.openstack.org/#/c/245789/7 with the rewording, I'll +2 it | 13:05 |
| johnthetubaguy | cdent: hey, that would be cool | 13:07 |
| cdent | ✔ | 13:07 |
| *** salv-orlando has quit IRC | 13:07 | |
| johnthetubaguy | cdent: do you want to put yourself on the CPL wiki? | 13:08 |
| cdent | yup, I'll take care of that | 13:08 |
| *** baoli_ has joined #openstack-nova | 13:08 | |
| *** baoli has quit IRC | 13:11 | |
| *** Daisy_ has joined #openstack-nova | 13:12 | |
| johnthetubaguy | cdent: thanks, let me know if you need anything | 13:13 |
| cdent | a pony? | 13:13 |
| pkholkin | johnthetubaguy, bauwser: hello once again, have a question about this method https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L408-L409, why it doesn't support 'use_slave'? Other methods inside 'get_all_by_host' function support and pass it. | 13:13 |
| pkholkin | maybe it was forgotten by mistake? | 13:14 |
| *** lpetrut has quit IRC | 13:14 | |
| openstackgerrit | Dina Belova proposed openstack/nova: == WIP == Integrate OSprofiler and Nova https://review.openstack.org/254703 | 13:15 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Nuke EC2 API from api-paste and remove wsgi support https://review.openstack.org/232325 | 13:16 |
| *** ducttape_ has joined #openstack-nova | 13:17 | |
| *** raildo-afk is now known as raildo | 13:23 | |
| *** pradk has joined #openstack-nova | 13:24 | |
| bauwser | johnthetubaguy: meh, I forgot to add myself on the wiki for the Release guy (thanks cdent for lemme point that) | 13:24 |
| *** deepakcs has quit IRC | 13:24 | |
| cdent | I actually couldn't find a list for cross-project cpl | 13:25 |
| johnthetubaguy | pkholkin: looks that way | 13:25 |
| johnthetubaguy | hmm, I wonder where they are putting those | 13:25 |
| *** links has joined #openstack-nova | 13:25 | |
| johnthetubaguy | pkholkin: oh wait, thats stuff that will be removed soon, so its probably not worth fixing | 13:25 |
| johnthetubaguy | bauwser: can we drop that code now, that pkholkin is pointing at | 13:26 |
| *** ducttape_ has quit IRC | 13:27 | |
| pkholkin | johnthetubaguy: in my current understanding we should also add 'allow_async' to 'compute_nodes_get_by_service_id' because the code looks similar in 'get_all_by_host' method | 13:27 |
| *** rushiagr_away is now known as rushiagr | 13:28 | |
| pkholkin | johnthetubaguy: I mean that 'get_all_by_host' should fully support 'use_slave' regarding to all db calls in it, so that is your 'impl' concept | 13:29 |
| *** renatoarmani has joined #openstack-nova | 13:29 | |
| *** renatoarmani has quit IRC | 13:29 | |
| *** ctrath has joined #openstack-nova | 13:32 | |
| *** thumpba has joined #openstack-nova | 13:32 | |
| *** breitz has quit IRC | 13:33 | |
| *** breitz has joined #openstack-nova | 13:34 | |
| bauwser | johnthetubaguy: heh, you're right, we could drop now all the service/compute old ship | 13:34 |
| bauwser | sink it even | 13:34 |
| bauwser | and we could even provide a ComputeNodeV2 which would get rid the service_id | 13:35 |
| johnthetubaguy | pkholkin: yeah, so I think we can just drop the stuff in the exception clause now, which is even easier :) | 13:35 |
| johnthetubaguy | bauwser: not a bad idea | 13:35 |
| bauwser | it was a Kilo BP IIRC, we can axe that | 13:36 |
| bauwser | \o/ | 13:36 |
| *** jyuso1 has quit IRC | 13:36 | |
| *** Daisy_ has quit IRC | 13:38 | |
| pkholkin | johnthetubaguy, bauwser: do you mean that we can remove all these lines? https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L399-L415 | 13:38 |
| bauwser | pkholkin: that's one of the sections to be removed, but all of that | 13:38 |
| bauwser | not* all of that I mean | 13:39 |
| *** ctrath has quit IRC | 13:39 | |
| bauwser | there is also a compat code in the CN object | 13:39 |
| bauwser | and in the Service object too | 13:39 |
| bauwser | that plus the ResourceTracker, but I need to check | 13:39 |
| bauwser | pkholkin: the idea was that old computes were not having a new field called 'host' hence all the lookups done by the exception | 13:40 |
| *** vladikr has joined #openstack-nova | 13:41 | |
| *** artom has joined #openstack-nova | 13:42 | |
| *** boris-42_ has quit IRC | 13:43 | |
| pkholkin | johnthetubaguy, bauwser: the idea is clear, but we were working here on switching to enginefacade, will someone remove this old stuff? because it definitely should not be in our switch-patches | 13:44 |
| pkholkin | we can just make our switch for now | 13:44 |
| *** jaypipes has joined #openstack-nova | 13:44 | |
| johnthetubaguy | pkholkin: stepping back, I would just make the switch to enginefacade, and not worry about adding extra use_slave (knowing that stuff will soon be deleted)? Now it would be great if you could do the removal patch while you are there, but good not to get too distracted from the goal | 13:46 |
| *** haomaiwa_ has joined #openstack-nova | 13:46 | |
| *** doug-fish has joined #openstack-nova | 13:47 | |
| *** meh_di has left #openstack-nova | 13:47 | |
| *** e0ne has quit IRC | 13:48 | |
| *** alaski is now known as lascii | 13:49 | |
| *** e0ne has joined #openstack-nova | 13:50 | |
| pkholkin | johnthetubaguy: ok, we will try to create some removal patch for this particular place and not add extra use_slave here | 13:50 |
| johnthetubaguy | pkholkin: that sounds great. bauwser: I know there are more bits to remove, but that can be a follow up patch. Maybe raise a bug to remember about this in the new year? | 13:52 |
| markus_z | sdague: that would be cool, thanks. I'm going to ping you when it's done. | 13:53 |
| *** scheuran has quit IRC | 13:53 | |
| sdague | johnthetubaguy: also, I fixed your reno concern on - https://review.openstack.org/#/c/254785/ | 13:54 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Move nova/opts.py to nova/conf/ https://review.openstack.org/255436 | 13:54 |
| sdague | go go gerrit inline editor | 13:54 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Config options: centralize base path configuration https://review.openstack.org/259406 | 13:54 |
| *** links has quit IRC | 13:54 | |
| sdague | markus_z: also, could you review the various conf move patches - https://review.openstack.org/#/c/256344 (as an example) | 13:55 |
| sdague | I'd feel more comfortable +2ing them if you had a +1 on them to know they were being done like you envisioned | 13:55 |
| markus_z | sdague: yepp | 13:57 |
| sdague | cool, thanks | 13:57 |
| sdague | I'm happy to proxy your +1 into a +2 | 13:57 |
| markus_z | thanks for having an eye on that | 13:58 |
| ndipanov | vladikr, https://review.openstack.org/#/c/249017/ see my comment on that | 13:58 |
| ndipanov | vladikr, I think we need a 3rd approach | 13:58 |
| ndipanov | I'll try to cook something up today | 13:58 |
| vladikr | ndipanov, reading | 13:58 |
| openstackgerrit | linbing proposed openstack/nova: Fix the TypeError：when all_cluster is None https://review.openstack.org/257674 | 13:59 |
| ndipanov | vladikr, so what you can do is take the changes from PS4 that are not related to passing device_type and put them in a separate patch | 14:00 |
| ndipanov | since device_type is needed for the neutron side as well | 14:00 |
| *** haomaiwa_ has quit IRC | 14:01 | |
| vladikr | ndipanov, also, I have updated the patch that I previously had as ps4 in a different change : https://review.openstack.org/#/c/259254/2 - it has all the tests and I showed showed that device_type in whitelist has no effect really on anything | 14:01 |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| vladikr | ndipanov, maybe we can sync on that over hangouts or bluejeans? | 14:02 |
| vladikr | if you have the time | 14:02 |
| ndipanov | vladikr, in 2h I need to step out | 14:02 |
| ndipanov | ping you when we're back | 14:02 |
| vladikr | ndipanov, great :) | 14:02 |
| ndipanov | er I'm back | 14:03 |
| *** alexsch__ has joined #openstack-nova | 14:03 | |
| johnthetubaguy | sdague: oh, I see the inline editor now, cool | 14:04 |
| *** ankit_ag has quit IRC | 14:05 | |
| *** alexschm has quit IRC | 14:05 | |
| *** dustins|out is now known as dustins | 14:05 | |
| *** Jeffrey4l has joined #openstack-nova | 14:11 | |
| *** vlaza has quit IRC | 14:12 | |
| sfinucan | markus_z: So these config options - is there a better way to do the 'services which consume this' section? | 14:12 |
| sfinucan | I have a feeling I asked that question before, but I don't recall the response :/ | 14:13 |
| sfinucan | I'm concerned about how manual it is, and therefore prone to getting outdated. It really seems like something that would benefit from structured data from...somewhere... | 14:14 |
| *** pradk_ has joined #openstack-nova | 14:14 | |
| *** pradk has quit IRC | 14:14 | |
| *** salv-orlando has joined #openstack-nova | 14:14 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step https://review.openstack.org/254709 | 14:14 |
| *** pradk_ is now known as pradk | 14:14 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce block_stats to return device stats https://review.openstack.org/251288 | 14:14 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a root block device https://review.openstack.org/249139 | 14:14 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: wrapper list_guests to Host's object https://review.openstack.org/249835 | 14:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce exceptions handler https://review.openstack.org/257991 | 14:15 |
| *** lucasagomes is now known as lucas-hungry | 14:15 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce get_job_info to Guest's object https://review.openstack.org/250740 | 14:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: configure migrate max downtime https://review.openstack.org/250739 | 14:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "abort_job" to Guest's object https://review.openstack.org/250738 | 14:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "migrate" to Guest's object https://review.openstack.org/250365 | 14:15 |
| markus_z | sfinucan: right now I don't have a solution to that. | 14:15 |
| markus_z | sfinucan: I thought of some kind of dependency analysis with the /nova/cmd modules as a base but I didn't have time to go further on that road | 14:16 |
| markus_z | sfinucan: I agree that the current way has potential to get stale | 14:16 |
| sfinucan | markus_z: So that leads to another question: what's the value proposition? Not that it's a bad idea, but I don't grasp why it would matter that nova-compute is affected by an option and not nova-scheduler, for example | 14:17 |
| markus_z | sfinucan: It won't be perfect but much more better as before | 14:17 |
| markus_z | sfinucan: My point of view was, "only configure what's necessary" and to achieve that you need to know which service consumes it. | 14:18 |
| johnthetubaguy | its about what deployers put in the nova.conf file on each of their machines, and partly to help folks understand when they change a config which services need to get SIGHUPed, etc | 14:18 |
| *** bnemec has quit IRC | 14:18 | |
| johnthetubaguy | now we probably need to decide what we want here, as there was a big push to assume the same config file across all services, by default | 14:18 |
| markus_z | sfinucan: I had doubts that there is a general purpose "nova.conf" which fits for controller node and compute node or more complex topologies | 14:18 |
| johnthetubaguy | markus_z: so that was an aim at some point, not sure how far away we are from that | 14:19 |
| johnthetubaguy | it was an upgrade assumption at one point | 14:19 |
| markus_z | johnthetubaguy: was that ever feasible with complex topologies with mixed hypervisors? | 14:20 |
| *** ducttape_ has joined #openstack-nova | 14:20 | |
| sfinucan | markus_z: Ah - gotcha | 14:21 |
| johnthetubaguy | markus_z: you could have the same base everywhere, with a few differences for the specific hypervisor notes, I think | 14:21 |
| openstackgerrit | jichenjc proposed openstack/nova: Support mac and type for /servers/ips request https://review.openstack.org/257261 | 14:21 |
| *** otter768 has joined #openstack-nova | 14:22 | |
| *** ducttape_ has quit IRC | 14:22 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: review guide: add location details for config options https://review.openstack.org/245789 | 14:23 |
| *** markvoelker has joined #openstack-nova | 14:23 | |
| markus_z | johnthetubaguy: A crazy voice at the back of my head was also whispering that the "service consumption" information could become useful later. | 14:24 |
| *** mrkz has joined #openstack-nova | 14:25 | |
| markus_z | for example when the scheduler decides to jump out of the tree to become its own project | 14:25 |
| markus_z | dan was also proposing the idea to split out the compute drivers | 14:25 |
| cdent | thanks for sharing your review dashboard sdague. some of those sections are rather long... :( | 14:25 |
| *** lpetrut has joined #openstack-nova | 14:26 | |
| johnthetubaguy | markus_z: its useful, just thinking out loud really | 14:26 |
| *** otter768 has quit IRC | 14:26 | |
| sdague | cdent: they are | 14:26 |
| johnthetubaguy | cdent: yeah, looks like we just hit over 700 in the review queue | 14:27 |
| *** toure|biab is now known as toure | 14:27 | |
| * cdent sharpens his pencil | 14:27 | |
| *** markvoelker has quit IRC | 14:27 | |
| markus_z | johnthetubaguy: FWIW, I wanted to post on the ops ML in January and get their feedback to this too. Maybe the say "meh, skip that piece of info". | 14:29 |
| johnthetubaguy | markus_z: good plan | 14:30 |
| johnthetubaguy | markus_z: that an the folks that do the install/orchestration stuff | 14:30 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Remove catching of ComputeHostNotFound exception https://review.openstack.org/259470 | 14:31 |
| andreykurilin | sdague: https://review.openstack.org/#/c/211206/ hi! Can you look at update for a novaclient's microversion spec? | 14:31 |
| pkholkin | johnthetubaguy, bauwser: please take a look this one https://review.openstack.org/#/c/259470/1 | 14:32 |
| markus_z | johnthetubaguy: Is there a special ML for them? I cannot find one here: http://lists.openstack.org/cgi-bin/mailman/listinfo | 14:32 |
| openstackgerrit | esracelik proposed openstack/nova: Config options: centralize section "hyperv" https://review.openstack.org/255124 | 14:32 |
| dansmith | migi: +W on the bottom one, but there is an issue in the top one | 14:33 |
| bauwser | pkholkin: ack, I feel we could write a specless BP for that, johnthetubaguy thoughts ? | 14:33 |
| johnthetubaguy | bauwser: maybe just do a bug? not sure we need to worry too much about it | 14:34 |
| johnthetubaguy | could add a reno note if there is user impact | 14:34 |
| bauwser | johnthetubaguy: fair if it's a wishlist one | 14:34 |
| bauwser | johnthetubaguy: I don't really want to log a "real" bug for that one | 14:35 |
| bauwser | johnthetubaguy: I don't guess we would have more impact than just that mitaka wouldn't support liberty computes, but that's already part of the contract | 14:35 |
| bauwser | removing the field from the DB would have an impact tho | 14:36 |
| bauwser | here, I'm just speaking of removing the compat code | 14:36 |
| *** dslevin has quit IRC | 14:36 | |
| bauwser | oooooh | 14:38 |
| bauwser | we can have a custom menu in Gerrit ! | 14:38 |
| bauwser | <3 | 14:38 |
| bauwser | https://review.openstack.org/#/settings/preferences | 14:38 |
| bauwser | sdague: bingo, I'm able to add your dash into my own Gerrit menu | 14:39 |
| bauwser | no need to keep URL bookmarks | 14:40 |
| bauwser | that makes me the day | 14:40 |
| dansmith | you're the day now? | 14:41 |
| *** dansmith is now known as superdan | 14:41 | |
| lascii | heh | 14:41 |
| *** amotoki has joined #openstack-nova | 14:41 | |
| superdan | someone please remind me to revert my nick before the end of the day so I don't stay superdan until january :D | 14:41 |
| sdague | superdan: it's better than being the night | 14:41 |
| bauwser | ergh, stupid frenchies | 14:41 |
| bauwser | that makes me *my* day | 14:41 |
| superdan | sdague: I dunno, sounds cooler actually | 14:42 |
| superdan | "I am the night" -- has a certain ring to it | 14:42 |
| pkholkin | lascii: not usual nick) hello! could you please take make another try on this patch) https://review.openstack.org/#/c/243496/18 | 14:42 |
| pkholkin | lascii: after several rechecks your patch was merged!! | 14:42 |
| *** paul-carlton1 has quit IRC | 14:43 | |
| *** yassine__ has quit IRC | 14:43 | |
| lascii | pkholkin: yeah, I managed to hit every gate failure over the past week. I have your patch open in a tab, will get to it soon | 14:43 |
| pkholkin | lascii: that's great!) I also had opened your patch and rechecked it so many times) | 14:44 |
| lascii | superdan: http://i.imgur.com/BMLlmaI.jpg | 14:45 |
| superdan | lascii: nice | 14:45 |
| *** dslev has joined #openstack-nova | 14:45 | |
| sfinucan | markus_z: Another one :) | 14:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 14:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make libvirt driver return migrate data objects for source and dest checks https://review.openstack.org/247720 | 14:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 14:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 14:45 |
| *** ljxiash_ has joined #openstack-nova | 14:46 | |
| sfinucan | Related options. Surely we can implement this via OptGroups? | 14:46 |
| sfinucan | Maybe "Related options" in the message would be a good short term option, but putting them together in a group would provide a more structured approach IMO | 14:47 |
| sfinucan | I don't know how difficult moving options into groups is though - those VNC changes have been spinning around for a long time... | 14:47 |
| markus_z | sfinucan: Some options are cross cutting. For example SSL/Cert stuff. Which means they are "related" but not in the same group. | 14:48 |
| *** ljxiash has quit IRC | 14:48 | |
| markus_z | sfinucan: moving options to another group is an API change and needs to deprecation policy compliant | 14:48 |
| markus_z | *to be | 14:49 |
| sfinucan | markus_z: Damn - that would explain it | 14:49 |
| johnthetubaguy | sfinucan: there is a deprecated opt group name, which helps with moving | 14:49 |
| sfinucan | johnthetubaguy: Yeah - I just saw that this morning actually :) | 14:49 |
| johnthetubaguy | yeah, we say for those who upgrade, they don't need to change their config until after the upgrade is complete | 14:49 |
| sdague | pkholkin: are you still about? https://review.openstack.org/#/c/240253/10 - some comments in there | 14:50 |
| sfinucan | didn't know it's purpose | 14:50 |
| sfinucan | *its | 14:50 |
| johnthetubaguy | ah, cool | 14:50 |
| johnthetubaguy | yeah, alias basically | 14:50 |
| johnthetubaguy | now there is another bit here | 14:50 |
| johnthetubaguy | any move we make forces all our users to do work | 14:50 |
| *** mriedem has joined #openstack-nova | 14:50 | |
| garyk | ansiwen: he is in europe | 14:50 |
| *** ljxiash_ has quit IRC | 14:51 | |
| johnthetubaguy | sfinucan: markus_z: so I am tempted we do this in two steps, try group them in code, then once we are done, and deprecate the ones we don't want, do some moves? | 14:51 |
| *** ljxiash has joined #openstack-nova | 14:51 | |
| markus_z | johnthetubaguy: yepp | 14:52 |
| *** jed56 has joined #openstack-nova | 14:52 | |
| sfinucan | johnthetubaguy: markus_z: Yeah, I'm not going to do any deprecations or regrouping while all this churn is happening | 14:53 |
| pkholkin | sdague: also look some time ago on @require_context and it seemed strange to me, in general we must have context in the args, but I don't think e.g. of removing it | 14:53 |
| markus_z | johnthetubaguy: I'd also like to replace some StrOpt, MultiStrOpt with custom Opts to do the validation checking when the options get slurped in, which is pretty early in the time line. | 14:53 |
| markus_z | sfinucan: cool! I think that would be too confusing | 14:54 |
| johnthetubaguy | markus_z: yeah, did you see the choices param on stropt? | 14:54 |
| sfinucan | But there's got to be a better way to specify interdependencies than in the help message - I share your dislike of strings with "special meaning", markus_z | 14:54 |
| sfinucan | Lemme thing about it | 14:54 |
| *** cfriesen has joined #openstack-nova | 14:54 | |
| johnthetubaguy | sfinucan: yeah, agreed, I like the create a group idea for that | 14:54 |
| markus_z | johnthetubaguy: yepp, I've seen it. Sometimes it's not enough. When you see the "pci_passthrough_whitelist" logic you will see that. | 14:55 |
| johnthetubaguy | markus_z: heh, yeah, that one is special | 14:55 |
| pkholkin | sdague: as for me @require_context is smth old, but when I did the patches I didn't think of doing smth with it | 14:55 |
| markus_z | sfinucan: I had plans to make an update in "oslo.config" itself, but just enhancing the string was the fast version. | 14:56 |
| *** amotoki_ has joined #openstack-nova | 14:56 | |
| pkholkin | I think we should ask other guys about it | 14:56 |
| *** ljxiash has quit IRC | 14:56 | |
| *** ljxiash has joined #openstack-nova | 14:56 | |
| *** jwcroppe has quit IRC | 14:57 | |
| sfinucan | markus_z: Yeah, and I'll use that approach myself for now | 14:57 |
| sfinucan | but I'll look into doing something in oslo_config also | 14:57 |
| *** sturlry_ has quit IRC | 14:57 | |
| markus_z | sfinucan: So that an Opt has a list attribute for related options which can be checked programatically (at least for its existence) | 14:57 |
| *** jwcroppe has joined #openstack-nova | 14:58 | |
| *** cdent has quit IRC | 14:58 | |
| *** zhenq has joined #openstack-nova | 14:58 | |
| *** mtanino has joined #openstack-nova | 14:58 | |
| sfinucan | Already attempting to remove "deprecation reasons" out of those messages https://review.openstack.org/#/c/259447/ | 14:58 |
| sdague | pkholkin: yeh, the problem with leaving it there is people writing new code are going to be confused if they need it or not | 14:58 |
| *** amotoki has quit IRC | 14:58 | |
| sfinucan | markus_z: Sounds good to me | 14:58 |
| *** zenoway has quit IRC | 14:59 | |
| sfinucan | I might try implementing that myself if I get positive noises about the 'deprecation_reason' change | 14:59 |
| sfinucan | ...and you don't do it first | 14:59 |
| *** zenoway has joined #openstack-nova | 14:59 | |
| *** ducttape_ has joined #openstack-nova | 14:59 | |
| pkholkin | sdague: ok, thanks for pointing this, will ask other opinions | 15:00 |
| *** dane-fichter has joined #openstack-nova | 15:00 | |
| *** alexsch__ has quit IRC | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** alexschm has joined #openstack-nova | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** lucas-hungry is now known as lucasagomes | 15:01 | |
| pkholkin | sdague: if you have time you can review this one https://review.openstack.org/#/c/247395/, it seems not to have any problems) | 15:01 |
| markus_z | sfinucan: I'd like to have as less simultaneously open tasks as possible, otherwise I screw things up. Which means I'd like to finish the move, then finish the help text, then finish deprecation+validation (slight overlap is fine). | 15:02 |
| *** Kevin_Zheng has quit IRC | 15:02 | |
| *** eharney has joined #openstack-nova | 15:03 | |
| *** zenoway has quit IRC | 15:03 | |
| markus_z | sfinucan: I'm just reviewing your patch https://review.openstack.org/#/c/257956/4 | 15:04 |
| *** Kevin_Zheng has joined #openstack-nova | 15:04 | |
| *** alexschm_ has joined #openstack-nova | 15:05 | |
| markus_z | sfinucan: What's the reason that you didn't replace the "import_opts" with "import nova.conf"? | 15:05 |
| *** alexschm has quit IRC | 15:05 | |
| *** ildikov has quit IRC | 15:05 | |
| sdague | pkholkin: looking | 15:05 |
| *** sturlry has joined #openstack-nova | 15:05 | |
| *** alexschm_ has quit IRC | 15:05 | |
| sdague | pkholkin: +2 | 15:05 |
| *** alexschm has joined #openstack-nova | 15:06 | |
| pkholkin | sdague: nice to +2) thanks! | 15:07 |
| *** david-lyle has quit IRC | 15:08 | |
| *** cfarquhar has joined #openstack-nova | 15:09 | |
| *** cfarquhar has quit IRC | 15:09 | |
| *** cfarquhar has joined #openstack-nova | 15:09 | |
| *** thangp has joined #openstack-nova | 15:11 | |
| pkholkin | nice to see* | 15:12 |
| openstackgerrit | Dina Belova proposed openstack/nova: == WIP == Integrate OSprofiler and Nova https://review.openstack.org/254703 | 15:12 |
| *** yassine__ has joined #openstack-nova | 15:13 | |
| *** yassine__ has quit IRC | 15:13 | |
| *** bklei has joined #openstack-nova | 15:14 | |
| *** salv-orlando has quit IRC | 15:18 | |
| superdan | johnthetubaguy: still around? | 15:18 |
| *** salv-orlando has joined #openstack-nova | 15:18 | |
| sdague | sahid: so when you are fixing a stack like that, you only really need to rebase the first one, it saves a lot of ci time to not repush the whole stack | 15:19 |
| superdan | johnthetubaguy: we'd really like to get a new novaclient tagged so it can be in the pipeline over the slow next couple weeks, because of several features that are going to be enabled by it | 15:19 |
| superdan | johnthetubaguy: are you cool with us doing one of those todayish? | 15:19 |
| openstackgerrit | jichenjc proposed openstack/nova: Remove stub_compute config options for console https://review.openstack.org/234217 | 15:22 |
| openstackgerrit | jichenjc proposed openstack/nova: Use instance object instead of db record https://review.openstack.org/234218 | 15:22 |
| *** yamahata has joined #openstack-nova | 15:22 | |
| *** fmccrthy has quit IRC | 15:22 | |
| *** carl_baldwin_ has joined #openstack-nova | 15:22 | |
| *** carl_baldwin has quit IRC | 15:23 | |
| *** carl_baldwin_ is now known as carl_baldwin | 15:23 | |
| *** alexschm has quit IRC | 15:23 | |
| *** fmccrthy has joined #openstack-nova | 15:24 | |
| sdague | can we get to the bottom of what needs to happen for this patch - https://review.openstack.org/#/c/171098/2 ? | 15:24 |
| sdague | that seems deadlocked in min libvirt discussions | 15:25 |
| sdague | and at this point I'm not sure I understand how we unwind it | 15:25 |
| *** zakora has quit IRC | 15:25 | |
| sahid | sdague: i do not know how to rebase a patch in bottom of serie without to reapply patches on top of it | 15:26 |
| sahid | i'm using rebase -i master, then pick the one i need to update | 15:27 |
| sdague | sahid: just create a new branch | 15:27 |
| sdague | that ends with the one you need to fix | 15:27 |
| sdague | you can rebase -i master that | 15:28 |
| sahid | i see | 15:28 |
| sahid | why gerrit is able to keep votes where CI relaunch tests? | 15:28 |
| sdague | git checkout -b new_branch; git reset --hard $fixid; git rebase -i master | 15:29 |
| *** burgerk has joined #openstack-nova | 15:29 | |
| *** amotoki_ has quit IRC | 15:30 | |
| sdague | sahid: example? | 15:33 |
| bauwser | sahid: in general, that's because the patch hasn't been touched | 15:36 |
| bauwser | while the others have been | 15:37 |
| bauwser | but that still needs a full Jenkins pass, because that's rebased on top of something changed - so the result is different and needs to be reverified | 15:37 |
| openstackgerrit | Sean Dague proposed openstack/nova: libvirt: update to min required version to 0.10.2 https://review.openstack.org/231917 | 15:37 |
| sdague | superdan: https://review.openstack.org/231917 addresses your concerns on that patch I believe, plus I added the reno in that mriedem wanted | 15:38 |
| sfinucan | markus_z: sorry - call | 15:38 |
| sahid | bauwser: so according to your reponse, i do not understand complaints from sdague | 15:38 |
| *** ljxiash has quit IRC | 15:38 | |
| *** ljxiash has joined #openstack-nova | 15:39 | |
| sfinucan | Yeah, agree on simultaneous thing. I won't be doing anything with that option until this blueprint is completed. Just trying to get feedback on the idea for now | 15:39 |
| sdague | sahid: no, votes from Verified are not kept after trivial rebase | 15:39 |
| sfinucan | markus_z: As for that review, let me refresh myself on that first | 15:39 |
| sdague | all your patches now need tests rerun on them | 15:40 |
| *** ociuhandu has joined #openstack-nova | 15:40 | |
| *** amotoki has joined #openstack-nova | 15:41 | |
| migi | superdan: thanks, I've replied to your comment with comment | 15:42 |
| sfinucan | markus_z: Ah. OK, I wasn't sure what to do about 'import_opt' so I guessed | 15:43 |
| *** ljxiash has quit IRC | 15:43 | |
| sfinucan | It may have been on the Etherpad, but I didn't see it. I'll update those files now | 15:43 |
| markus_z | sfinucan: cool, thanks! I'm going to check the etherpad to make it clear for the future. | 15:44 |
| sfinucan | markus_z: Wait - can I make that change before the other imported options are migrated? | 15:46 |
| sfinucan | *moved to 'nova/conf' | 15:46 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'vif' and 'task_log' https://review.openstack.org/247545 | 15:46 |
| dane-fichter | Anyone know if Kevin Mitchell is out for the holidays? | 15:47 |
| markus_z | sfinucan: if it's in "nova/conf/*", the import_opt is not necessary if that module also did an "import nova.conf". | 15:48 |
| markus_z | sfinucan: At the end of the day they are all in the same global object | 15:48 |
| *** danpb has joined #openstack-nova | 15:48 | |
| danpb | sdague: thanks for updating that libvirt min version patch of mine | 15:48 |
| *** xyang2 has joined #openstack-nova | 15:48 | |
| jed56 | Hello everybody | 15:48 |
| jed56 | I'm working on the project Watcher (i.e: resource optimization) https://github.com/openstack/watcher. | 15:49 |
| markus_z | sdague: I reworded the code-review guide as suggested: https://review.openstack.org/#/c/245789/ | 15:49 |
| jed56 | I would like to take advantage of all of the existing scheduler filters. We don't want to calculate the capacity requirements, etc., which can quickly turn into a headache. | 15:49 |
| dane-fichter | sdague: If you have some time, could you take a look at https://review.openstack.org/#/c/189843/ | 15:49 |
| jed56 | Can I use FilterScheduler to determine of the available hosts given the constraints? For instance, we have a strategy (i.e: an algorithm implementation able to find some actions (e.g Live migration, shutdown,...) but for example we don't want to manage capacity requirements. | 15:49 |
| *** Daisy_ has joined #openstack-nova | 15:50 | |
| sdague | danpb: no prob | 15:50 |
| bauwser | jed56: I'm not sure I get your question | 15:50 |
| sdague | danpb: any idea about what needs to happen with https://review.openstack.org/#/c/171098/2 to make it fit into our current min requirements? | 15:50 |
| *** cdent has joined #openstack-nova | 15:51 | |
| jed56 | bauwser: nova is initial placement but we are working on dynamic virtual machine placement | 15:51 |
| bauwser | jed56: I know the Watcher project, I just wonder what you want :) | 15:51 |
| jed56 | ok :) | 15:52 |
| * bauwser is bauzas btw. | 15:52 | |
| jed56 | ah! | 15:52 |
| pkholkin | sdague: I understand that we have already several merged patches with '@require_context' left above the function. So we should return and maybe e.g. remove it if we come to this decision | 15:52 |
| *** ctina has joined #openstack-nova | 15:52 | |
| danpb | sdague: sigh, in general i wish we didn't have those freaking config options that expose libvirt API flags as admin changable settings | 15:52 |
| danpb | we really ought to look at just deleting them and having Nova choose the right flags at runtime | 15:53 |
| sdague | danpb: I'd be ok with that | 15:53 |
| pkholkin | sdague: I will ask other guys for the opinions and keep it in my mind | 15:53 |
| *** pumaranikar has joined #openstack-nova | 15:53 | |
| sdague | danpb: deprecating the config options and setting them to the right thing if they are set at the defaults seems reasonable | 15:54 |
| pkholkin | sdague: so maybe now it is not a problem, but I will take care of it for consistency | 15:54 |
| *** Daisy_ has quit IRC | 15:54 | |
| danpb | we'd need to add some other config options in their place to control migration setup at a logical level, which would inform nova which flags to pick | 15:54 |
| jed56 | bauzas: I would like to call nova-scheduler to help me to find a set hosts with for example enough available resource to host a new VM that I want to migrate | 15:56 |
| sdague | danpb: ok, still seems like a better approac | 15:56 |
| *** ildikov has joined #openstack-nova | 15:57 | |
| mriedem | danpb: the operator thread on min libvirt kind of pittled out, are we ok with 0.10.2 being the min and moving to 1.2.1 in N? | 15:58 |
| sdague | mriedem: yes | 15:59 |
| sdague | mriedem: aren't you supposed to be drinking eggnog? | 15:59 |
| danpb | mriedem: yep, we did get agreement on that in the end | 15:59 |
| bauwser | jed56: thru the API ? | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| jed56 | yes if possible | 16:01 |
| mriedem | danpb: cool | 16:01 |
| mriedem | sdague: i don't do anything with "nog" in the name | 16:01 |
| sdague | jed56: that's not exposed at this point | 16:01 |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| *** browne has joined #openstack-nova | 16:01 | |
| mriedem | danpb: sdague: commented on the change, my only question is if 1.2.1 is really OK for N or if that's meant for O | 16:01 |
| bauwser | jed56: so you would like to ask the scheduler to give you a destination without actually booting or doing a migration ? | 16:02 |
| andrearosa | mriedem: thanks for https://bugs.launchpad.net/nova/+bug/1527623 | 16:02 |
| openstack | Launchpad bug 1527623 in OpenStack Compute (nova) "Nova might orphan volumes when it's racing to delete a volume-backed instance" [Medium,Triaged] | 16:02 |
| bauwser | jed56: that's not possible atm | 16:02 |
| sdague | mriedem: yes, 1.2.1 is ok for N | 16:02 |
| sdague | O is going to be higher | 16:02 |
| *** mjura_ has quit IRC | 16:03 | |
| *** mjura has quit IRC | 16:03 | |
| sdague | basically the moment we go above 0.10.2 there aren't any distros we support that aren't on 1.2.x | 16:03 |
| *** hemna has joined #openstack-nova | 16:03 | |
| sdague | so any value < 1.2.1 doesn't support any more people | 16:03 |
| sdague | and it just leaves more cruft | 16:03 |
| mriedem | ok, i'm probably overthinking the comment in the driver code about when it's ok to bump again | 16:04 |
| *** amotoki has quit IRC | 16:04 | |
| sdague | which comment? | 16:04 |
| mriedem | # MIN_LIBVIRT_VERSION can be updated to match this after | 16:04 |
| mriedem | # NEXT_MIN_LIBVIRT_VERSION has been at a higher value for | 16:04 |
| mriedem | # one cycle | 16:04 |
| openstackgerrit | Samuel Matzek proposed openstack/nova: WIP Allow modification of delete_on_termination https://review.openstack.org/259518 | 16:04 |
| mriedem | i take 'one cycle' to mean one full cycle, but that could just be a release (mitaka) | 16:04 |
| sdague | yeh, I think we're really talking about a release | 16:04 |
| mriedem | but for CDers that's less than a full cycle | 16:04 |
| mriedem | like i said, probably overthinking this | 16:05 |
| sdague | right, but that falls back to our deprecation policy in openstack | 16:05 |
| sdague | which is one cycle && 90 days | 16:05 |
| sdague | the CDers get 90 days notice at least | 16:05 |
| jed56 | bauwser: okay and it-is possible to propose a list of hosts and let the scheduler choose in this list the best one host. | 16:05 |
| sdague | if you are CDing and not updating more frequenctly than every 90 days, you are really Discontinously Deploying | 16:06 |
| sdague | so I think it's consistent | 16:06 |
| mriedem | yeah i'm fine now, +2 | 16:06 |
| jed56 | bauwser : live_migrate(hosts= | 16:09 |
| *** matrohon has quit IRC | 16:10 | |
| danpb | mriedem: i tried to explain it in the comment too | 16:10 |
| mriedem | yeah, makes sense | 16:11 |
| mriedem | i'm holding up on the +W just b/c it seems like a thing that could use other agreement | 16:11 |
| *** ziggit has joined #openstack-nova | 16:11 | |
| danpb | i just checked the thread again and both CERN & GoDaddy said they'd be on RHEL-7/CentoS-7 by N, so it wouldn't be an issue for them | 16:12 |
| *** rcernin has quit IRC | 16:15 | |
| sdague | mriedem: ok, I felt like we agreed on all this over a month ago | 16:16 |
| mriedem | i thought it was stalled in the ML, but danpb just confirmed it | 16:16 |
| bauwser | jed56: sorry on a call | 16:16 |
| cdent | Is the HostManager updated for each select_destinations call? | 16:17 |
| sdague | so, there isn't any nice "Live Migration Complete" message in n-cpu? | 16:17 |
| sdague | I just wanted to double check things were really working in the live migration tests | 16:17 |
| mriedem | danpb: sdague: +W with links to the ML | 16:18 |
| markus_z | johnthetubaguy: If you find time https://review.openstack.org/#/c/255436/ | 16:19 |
| jed56 | bauwser: don't worry :) | 16:19 |
| *** ziggit has quit IRC | 16:20 | |
| kashyap | sdague: If you have libvirt debug log, I can give you an obscure message indicating it is successfully complete :-) | 16:20 |
| *** armax has joined #openstack-nova | 16:20 | |
| kashyap | It'll be something like: | 16:21 |
| kashyap | 2015-09-29 18:47:45.048+0000: 22661: debug : qemuDomainObjSetJobPhase:1231 : Setting 'migration in' phase to 'finish3' | 16:21 |
| sdague | kashyap: well, it would be nice if it exposed in n-cpu logs | 16:21 |
| kashyap | (On destination) | 16:21 |
| *** tjones has joined #openstack-nova | 16:21 | |
| kashyap | Yeah, I hear you - you'd like a nice message from a Nova perspective | 16:21 |
| danpb | sdague: yes, it should be there - let me find an example | 16:21 |
| danpb | LOG.info(_LI("Migration operation has completed"), | 16:21 |
| danpb | instance=instance) | 16:21 |
| danpb | sdague: that's what libvirt driver reports | 16:21 |
| danpb | conversely there is also "Migration operation was cancelled" @ WARN and "Migration operation has aborted" at ERROR if it fails | 16:22 |
| *** otter768 has joined #openstack-nova | 16:23 | |
| *** gszasz has quit IRC | 16:23 | |
| openstackgerrit | Andrea Rosa proposed openstack/nova: WIP Enable volume operations for shelved instances https://review.openstack.org/259528 | 16:23 |
| danpb | you'll also see some progress messages from libvirt driver while migration takes place, every 5 seconds | 16:23 |
| *** cdent has quit IRC | 16:23 | |
| *** markvoelker has joined #openstack-nova | 16:24 | |
| kashyap | Oh, nice, I should've looked at this _live_migration_monitor() | 16:25 |
| *** paul-carlton2 has quit IRC | 16:25 | |
| *** otter768 has quit IRC | 16:27 | |
| *** markvoelker has quit IRC | 16:28 | |
| *** hemna has quit IRC | 16:30 | |
| *** jdurgin1 has joined #openstack-nova | 16:30 | |
| *** andreykurilin__ has quit IRC | 16:32 | |
| *** xyang2 has quit IRC | 16:33 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "guestfs" https://review.openstack.org/248963 | 16:33 |
| sdague | danpb: is that on the target or source? | 16:34 |
| sdague | out of some strange coincidence it looks like our live migrated instances always seem to be going in one direction in this multinode test | 16:35 |
| * markus_z is going to leave for today. Back on monday. | 16:35 | |
| danpb | sdague: all on the source | 16:35 |
| *** markus_z has quit IRC | 16:35 | |
| sdague | from subnode -> controller | 16:35 |
| sdague | danpb: ok, cool | 16:35 |
| sdague | I guess I had expected that chance would show them in the controller n-cpu, but got unlucky | 16:36 |
| sdague | http://logs.openstack.org/88/251588/15/check/gate-tempest-dsvm-multinode-full/236d959/logs/subnode-2/screen-n-cpu.txt.gz#_2015-12-18_15_26_41_562 | 16:36 |
| sdague | so superdan's migration objects patches have at least not completely ruined live migration :) | 16:36 |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "wsgi" https://review.openstack.org/252687 | 16:37 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Get Hierarchy info https://review.openstack.org/242568 | 16:37 |
| *** dims_ has joined #openstack-nova | 16:37 | |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver https://review.openstack.org/242626 | 16:38 |
| *** dims has quit IRC | 16:39 | |
| mriedem | we have a commit message job now? | 16:41 |
| * andrearosa is on holidays mode now and wish merry Xmas to all of you | 16:42 | |
| *** zhenq has quit IRC | 16:42 | |
| mriedem | sdague: are the volume-backed live migration tests known to be failing in the multi-node job? | 16:43 |
| *** zhenq has joined #openstack-nova | 16:43 | |
| *** andrearosa is now known as andrearosa_holid | 16:43 | |
| sdague | mriedem: they seem to be failing pretty regularly in stuff that I'm seeing | 16:45 |
| sdague | at least as often as the volumes tests themselves | 16:45 |
| sdague | although this is currious - http://logs.openstack.org/20/247720/15/check/gate-tempest-dsvm-multinode-full/497e483/logs/subnode-2/screen-n-cpu.txt.gz#_2015-12-18_15_26_37_364 | 16:45 |
| *** andreykurilin__ has joined #openstack-nova | 16:45 | |
| sdague | the rollback seems to fail | 16:46 |
| sdague | superdan: actually, that looks like a real issue with your patches | 16:46 |
| *** rdopiera has quit IRC | 16:47 | |
| openstackgerrit | Tracy Jones proposed openstack/nova: config options: centralize section "volume" https://review.openstack.org/253747 | 16:47 |
| sdague | superdan: I put a -1 on the base patch with a log link to that | 16:47 |
| openstackgerrit | Tracy Jones proposed openstack/nova: Add better help to the guestfs option https://review.openstack.org/255578 | 16:47 |
| *** jistr has quit IRC | 16:49 | |
| *** jwcroppe has quit IRC | 16:49 | |
| *** ctina has quit IRC | 16:49 | |
| *** jwcroppe has joined #openstack-nova | 16:49 | |
| *** mrkz has quit IRC | 16:50 | |
| *** mrkz has joined #openstack-nova | 16:51 | |
| mriedem | sdague: there are a couple of other bugs in that change i think | 16:51 |
| mriedem | sdague: ah yeah, i just pointed that out | 16:52 |
| mriedem | http://logs.openstack.org/20/247720/15/check/gate-tempest-dsvm-multinode-full/497e483/logs/subnode-2/screen-n-cpu.txt.gz#_2015-12-18_15_26_37_364 | 16:52 |
| mriedem | the object doesn't have dict compat | 16:52 |
| mriedem | which is why that blows up | 16:52 |
| openstackgerrit | Tracy Jones proposed openstack/nova: WIP: config options: centralize section "service" https://review.openstack.org/256569 | 16:52 |
| sdague | well this is a blowup after it would have failed anyway right | 16:52 |
| sdague | I mean, it's not good, but I'm also trying to understand why we fail and trigger a rollback | 16:52 |
| mriedem | yeah, but still | 16:53 |
| *** jdurgin1 has quit IRC | 16:53 | |
| mriedem | there is another bug in there too that needs fixed | 16:53 |
| mriedem | on setting the boot index in the bdm | 16:53 |
| mriedem | i'm also struggling with how new gerrit hijacks ctrl+f | 16:53 |
| *** rwmjones is now known as rwmjones|holiday | 16:54 | |
| *** jwcroppe has quit IRC | 16:54 | |
| *** salv-orlando has quit IRC | 16:55 | |
| *** salv-orlando has joined #openstack-nova | 16:55 | |
| *** pumaranikar has quit IRC | 16:56 | |
| *** pumaranikar has joined #openstack-nova | 16:56 | |
| *** rk4n has quit IRC | 16:56 | |
| *** jlanoux has quit IRC | 16:56 | |
| sdague | it turns it into it's own search in it's editor | 16:57 |
| *** rk4n has joined #openstack-nova | 16:57 | |
| *** ssurana has joined #openstack-nova | 16:58 | |
| *** nikhil_k has joined #openstack-nova | 16:59 | |
| mriedem | yeah i know, i'm not crazy about it | 17:00 |
| mriedem | soon new gerrit will be an OS | 17:00 |
| mriedem | and become self-aware | 17:00 |
| *** pumaranikar has quit IRC | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| *** nikhil has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-nova | 17:01 | |
| sdague | http://logs.openstack.org/20/247720/15/check/gate-tempest-dsvm-multinode-full/497e483/logs/screen-n-cpu.txt.gz?#_2015-12-18_15_26_26_411 - is the reason it fails for some low level libvirt thing | 17:01 |
| sdague | ? | 17:01 |
| *** pumaranikar has joined #openstack-nova | 17:01 | |
| mriedem | sdague: known thing | 17:02 |
| mriedem | regXboi had a bug reported for it | 17:02 |
| mriedem | i thought i had an e-r query up, checking | 17:02 |
| sdague | ok, but, do we know why libvirt fails? | 17:02 |
| sdague | because "libvirt randomly fails" isn't so useful | 17:02 |
| sdague | and it only seems to be on the block live migration tests | 17:02 |
| *** paul-carlton_ has joined #openstack-nova | 17:02 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1368844 | 17:03 |
| openstack | Launchpad bug 1368844 in OpenStack Compute (nova) "Block migration fails with volume size 2 GB and more" [Medium,Incomplete] - Assigned to Timofey Durakov (tdurakov) | 17:03 |
| *** gjayavelu has joined #openstack-nova | 17:03 | |
| mriedem | danpb had looked when we brought this up before, there was something screwing with qemu in the libvirtd logs | 17:03 |
| sdague | so, our volumes in this test aren't > 2GB | 17:04 |
| sdague | are they? | 17:04 |
| sdague | I thought it was always 1 G volumes | 17:04 |
| mriedem | oh wait that's not the right bug | 17:04 |
| mriedem | thanks lp search | 17:04 |
| *** pradk has quit IRC | 17:05 | |
| mriedem | sdague: this one https://bugs.launchpad.net/nova/+bug/1524898 | 17:05 |
| openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | 17:05 |
| *** bnemec has joined #openstack-nova | 17:06 | |
| *** ctrath has joined #openstack-nova | 17:06 | |
| mriedem | see comment 3 from dan | 17:06 |
| sdague | interesting that it's volume related only | 17:06 |
| *** gjayavelu has quit IRC | 17:06 | |
| mriedem | i'll get an e-r query up for that one | 17:07 |
| sdague | yeh, it's only in non-voting jobs though | 17:07 |
| sdague | so .... not hugely useful in ER | 17:07 |
| mriedem | sdague: and it's almost exclusively OVH nodes | 17:07 |
| sdague | oh, is it/ | 17:07 |
| mriedem | oh yeah maybe that's why i didn't put up a query | 17:07 |
| sdague | so, ovh is the slowest nodes we have | 17:08 |
| mriedem | yeah, mostly OVH | 17:08 |
| sdague | some bluebox after that? | 17:08 |
| mriedem | yeah, and 2 hp | 17:08 |
| mriedem | 1 bluebox | 17:08 |
| mriedem | otherwise ovh | 17:08 |
| sdague | if this is speed related ovh -> bluebox -> hp -> rax is our speed gradient | 17:09 |
| mriedem | 0 rax | 17:09 |
| bauwser | sdague: mriedem: you need OVH support ? | 17:09 |
| sdague | yeh, that would indicate there was a speed component | 17:09 |
| sdague | bauwser: not really | 17:09 |
| bauwser | okay | 17:09 |
| sdague | it just exposing some races because their nodes are slower | 17:09 |
| bauwser | you mean nodepool is taking more time to build them, or because they're less powerful ? | 17:10 |
| sdague | they are less powerful | 17:10 |
| sdague | I think they gave us some dedicated hosts, but it means we are running flat out against ourselves in VMs | 17:11 |
| sdague | so we are our own noisy neighbors | 17:11 |
| bauwser | I see | 17:11 |
| *** mriedem is now known as mriedem_lunch | 17:11 | |
| *** e0ne has quit IRC | 17:13 | |
| *** paul-carlton2 has joined #openstack-nova | 17:13 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Use TimeFixture from oslo_utils to override time in tests https://review.openstack.org/237616 | 17:15 |
| sdague | lascii: fyi ^^^ was old enough there was a pep8 error after the rebase on unused imports | 17:17 |
| sdague | I just fixed it and reapproved | 17:17 |
| *** devananda has quit IRC | 17:17 | |
| kashyap | sdague: Does your log message earlier mean live block migration completely fails? Or is it likely a fault in Tempest? | 17:18 |
| sdague | kashyap: live migration with volumes seems to fail kind of often | 17:18 |
| sdague | I don't think Tempest is doing anything wrong | 17:18 |
| kashyap | Yeah, was looking at the bug linked above | 17:18 |
| sdague | the non volume live migration seems pretty happy | 17:19 |
| kashyap | sdague: Happen to know what is an "OVH node" (from that bug)? | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.hardware' options https://review.openstack.org/256344 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt' conf options documentation https://review.openstack.org/256368 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 17:19 |
| sdague | kashyap: one of our 4 cloud providers for upstream nodes | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 17:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: WIP: Flesh out nova.conf.virt documentation https://review.openstack.org/259553 | 17:19 |
| openstackgerrit | sonu proposed openstack/nova: Replaces xrange() with six.moves.range() to provide py3 compatibility https://review.openstack.org/205307 | 17:20 |
| *** devananda has joined #openstack-nova | 17:21 | |
| *** rohit_ has joined #openstack-nova | 17:24 | |
| *** markvoelker has joined #openstack-nova | 17:24 | |
| *** apoorvad has joined #openstack-nova | 17:25 | |
| *** markvoelker has quit IRC | 17:29 | |
| *** lpetrut has quit IRC | 17:31 | |
| *** nic has joined #openstack-nova | 17:32 | |
| *** nic1 has joined #openstack-nova | 17:34 | |
| *** nic1 has quit IRC | 17:35 | |
| *** unicell1 has quit IRC | 17:35 | |
| superdan | sdague: ah, yeah, that new failure is in the stuff I changed for mriedem_lunch yesterday I think | 17:36 |
| sdague | superdan: so he made you break it? | 17:36 |
| superdan | sdague: yes | 17:36 |
| sdague | extra nog for mriedem_lunch | 17:36 |
| *** nic has quit IRC | 17:36 | |
| superdan | mriedem_lunch: did you see my query to johnthetubaguy above about a novaclient release? | 17:37 |
| *** ctrath has quit IRC | 17:39 | |
| *** penick has joined #openstack-nova | 17:40 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add description for server query https://review.openstack.org/259565 | 17:41 |
| *** sfinucan has quit IRC | 17:42 | |
| *** ociuhandu has quit IRC | 17:42 | |
| *** ihrachys has joined #openstack-nova | 17:45 | |
| *** ociuhandu has joined #openstack-nova | 17:45 | |
| *** diogogmt has joined #openstack-nova | 17:47 | |
| *** ociuhandu has quit IRC | 17:47 | |
| openstackgerrit | Eric Brown proposed openstack/nova: VMware: speed up start of n-cpu https://review.openstack.org/255586 | 17:48 |
| *** paul-carlton2 has quit IRC | 17:49 | |
| *** suro-patz has joined #openstack-nova | 17:49 | |
| *** ljxiash has joined #openstack-nova | 17:51 | |
| *** jwcroppe has joined #openstack-nova | 17:52 | |
| *** david-lyle has joined #openstack-nova | 17:54 | |
| *** ljxiash has quit IRC | 17:56 | |
| *** pixelbeat_ has quit IRC | 17:58 | |
| *** david-lyle has quit IRC | 17:59 | |
| openstackgerrit | Patrick East proposed openstack/nova: libvirt: add discard support for attached volumes https://review.openstack.org/205726 | 17:59 |
| *** e0ne has joined #openstack-nova | 18:00 | |
| *** david-lyle has joined #openstack-nova | 18:00 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 18:00 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make libvirt driver return migrate data objects for source and dest checks https://review.openstack.org/247720 | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 18:01 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 18:01 |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| *** paul-carlton2 has joined #openstack-nova | 18:01 | |
| *** diogogmt has quit IRC | 18:02 | |
| *** yamahata has quit IRC | 18:02 | |
| *** diogogmt has joined #openstack-nova | 18:03 | |
| superdan | sdague: you have any arguments against a client tag today? keeping in mind I just +Wed your last glance client patch of course... :P | 18:03 |
| johnthetubaguy | superdan: which query about nova-client? | 18:04 |
| superdan | johnthetubaguy: ah, you're here | 18:04 |
| johnthetubaguy | ah, I see your comment now | 18:04 |
| superdan | johnthetubaguy: the mark-host-down stuff in the client is stranded at the moment because we need a release tagged since it got re-enabled | 18:04 |
| johnthetubaguy | yeah, about to go on holiday, which apparently means block up my calendar with back to back meetings :) | 18:05 |
| superdan | and we have a timing thing where it would be really advantageous if we could get that tag in our QA pipeline before we all disappear for xmas | 18:05 |
| superdan | johnthetubaguy: I know, I have too many today myself :( | 18:05 |
| sdague | superdan: nope, we should do it | 18:05 |
| sdague | I think it moves our microversion support up to 2.12 as well | 18:05 |
| superdan | cool, who can do that? just melwitt and mriedem or others? | 18:05 |
| sdague | which is nice | 18:05 |
| superdan | sdague: right, we want that too | 18:05 |
| johnthetubaguy | so I think I can just +1 a patch to the releases repo, to get the release done | 18:05 |
| *** rcernin has joined #openstack-nova | 18:06 | |
| superdan | oh, right, I forgot that changed | 18:06 |
| johnthetubaguy | then the release team (eventually automtation) does the rest | 18:06 |
| *** rushiagr is now known as rushiagr_away | 18:06 | |
| sdague | johnthetubaguy: yeh, just someone from the release team needs to do it | 18:06 |
| johnthetubaguy | I think we just need to fine the git sha | 18:06 |
| superdan | so I should propose a patch quickly so johnthetubaguy can +1? | 18:06 |
| *** tjones1 has joined #openstack-nova | 18:06 | |
| sdague | superdan: yeh | 18:06 |
| johnthetubaguy | superdan: yeah, that works | 18:06 |
| sdague | and then we should poke dims_ | 18:06 |
| johnthetubaguy | good point | 18:06 |
| sdague | and ask him to spin it | 18:06 |
| sdague | I think dhellmann is on a plane back from .it atm | 18:06 |
| superdan | sdague: and this is 3.0.0 yes? | 18:07 |
| sdague | we didn't do the 3.0 already? | 18:07 |
| sdague | if not, yes this is 3.0 | 18:07 |
| johnthetubaguy | ah, did we merge those patches from monty? | 18:07 |
| superdan | I don't see 3.0 | 18:07 |
| sdague | as we removed some stuff | 18:07 |
| *** unicell has joined #openstack-nova | 18:08 | |
| johnthetubaguy | yeah, I haven't seen 3.0.0 yet, was waiting on mriedem_lunch | 18:08 |
| *** derekh has quit IRC | 18:08 | |
| * dims_ peeks | 18:11 | |
| superdan | sdague: johnthetubaguy: dims_ https://review.openstack.org/259574 | 18:11 |
| dims_ | superdan : ack, need to wait for the jobs to run | 18:12 |
| johnthetubaguy | superdan: you are missing a period at the end of the release summary I guess? | 18:12 |
| dane-fichter | reviews on this would be much appreciated: https://review.openstack.org/#/c/189843/ | 18:12 |
| superdan | johnthetubaguy: I like how you're giving _me_ grammar advice :) | 18:13 |
| superdan | johnthetubaguy: fixed | 18:13 |
| johnthetubaguy | superdan: it feels wrong! | 18:13 |
| superdan | johnthetubaguy: even though it's right! I'm so confused :) | 18:13 |
| johnthetubaguy | heh | 18:13 |
| superdan | johnthetubaguy: there's no hacking test that fails with no period, fwiw | 18:14 |
| johnthetubaguy | true | 18:14 |
| johnthetubaguy | superdan: sdague: I think we may have originally been waiting for this patch: https://review.openstack.org/#/c/245200/ | 18:15 |
| johnthetubaguy | I am not sure if we should be waiting for that one | 18:15 |
| *** lpetrut has joined #openstack-nova | 18:16 | |
| *** garyk has quit IRC | 18:16 | |
| superdan | johnthetubaguy: unless it needs to be in 3.0 for some compat reason, I'd say we shouldn't | 18:17 |
| superdan | but if it does, then ... | 18:17 |
| *** kfarr has joined #openstack-nova | 18:17 | |
| johnthetubaguy | yeah, it doesn't look like we actually need it. There was talk of deprecating some stuff, but I couldn't see it | 18:17 |
| johnthetubaguy | oh, wait, maybe it was depreating NOVACLIENT_INSECURE, etc | 18:19 |
| *** browne has quit IRC | 18:19 | |
| johnthetubaguy | deprecating | 18:19 |
| *** ctrath has joined #openstack-nova | 18:19 | |
| superdan | johnthetubaguy: this changes no code.. is it just the doc stuff? | 18:19 |
| *** vilobhmm11 has joined #openstack-nova | 18:20 | |
| superdan | oh, nm | 18:20 |
| superdan | I'm blind | 18:20 |
| johnthetubaguy | yeah, thats what I did on first read | 18:20 |
| johnthetubaguy | its line 346 in shell | 18:20 |
| *** ctrath has quit IRC | 18:21 | |
| superdan | I'm a little afraid of blocking the release on that patch based on the comments :/ | 18:21 |
| superdan | I barely understand | 18:22 |
| *** e0ne has quit IRC | 18:22 | |
| superdan | sdague: ? | 18:22 |
| johnthetubaguy | superdan: similar view, honestly | 18:22 |
| johnthetubaguy | v4.0.0 is a nice round number anyways | 18:22 |
| superdan | heh | 18:22 |
| *** paul-carlton2 has quit IRC | 18:23 | |
| superdan | so mriedem_lunch was around earlier, but I thought he was supposed to be on vacation | 18:24 |
| *** ssurana has quit IRC | 18:24 | |
| *** otter768 has joined #openstack-nova | 18:24 | |
| johnthetubaguy | superdan: ah, good point | 18:24 |
| *** burgerk has quit IRC | 18:24 | |
| johnthetubaguy | like me tomorrow | 18:24 |
| *** tjones1 has quit IRC | 18:24 | |
| superdan | it's only noon-thirty there now, so maybe he really is at lunch | 18:24 |
| johnthetubaguy | honestly, we have delayed weeks, and its not done | 18:25 |
| *** diogogmt has quit IRC | 18:25 | |
| johnthetubaguy | we have to go at some point, and well, christmas is a good point to pick :) | 18:25 |
| superdan | yeah, we're kinda up against a wall needing a client with mark-host-down, hence the rush today :/ | 18:26 |
| *** Thelo has quit IRC | 18:27 | |
| sdague | it needs to be 3.0 for a compat reason | 18:28 |
| *** ihrachys has quit IRC | 18:28 | |
| *** yamahata has joined #openstack-nova | 18:28 | |
| superdan | sdague: monty's patches do? | 18:28 |
| sdague | no, we deleted some v1 symbols | 18:28 |
| superdan | right, we get that | 18:28 |
| *** otter768 has quit IRC | 18:28 | |
| superdan | we're talking about those patches that are still pending | 18:28 |
| johnthetubaguy | email me, if more stuff crops up, can try sort that out later, have to run | 18:29 |
| sdague | which patches? | 18:29 |
| superdan | johnthetubaguy: okay | 18:29 |
| superdan | sdague: https://review.openstack.org/#/c/245200/ | 18:29 |
| *** ihrachys has joined #openstack-nova | 18:29 | |
| *** ssurana has joined #openstack-nova | 18:29 | |
| *** HenryG is now known as HenryG_afk | 18:29 | |
| *** nikhil_k is now known as nikhil | 18:29 | |
| *** Thelo has joined #openstack-nova | 18:29 | |
| sdague | we can release without that | 18:29 |
| superdan | okay sweet | 18:29 |
| superdan | dims_: ^ | 18:30 |
| sdague | 245200 won't trigger a major bump | 18:30 |
| superdan | okay | 18:30 |
| sdague | it would have been nice if that was in, but it's not, and that's fine, we'll pick it up in the new year | 18:30 |
| *** ctrath has joined #openstack-nova | 18:30 | |
| *** tjones has quit IRC | 18:30 | |
| *** kfarr has quit IRC | 18:30 | |
| *** sahid has quit IRC | 18:31 | |
| sdague | we'll need to 4.0 release once we remove that backwards compat stuff | 18:31 |
| *** e0ne has joined #openstack-nova | 18:31 | |
| sdague | but that has to be 3 - 6 months out anyway, as mordred is adding new deprecations in that patch | 18:32 |
| *** tjones has joined #openstack-nova | 18:32 | |
| dims_ | superdan : ack, watching zuul | 18:32 |
| superdan | sdague: cool | 18:32 |
| superdan | dims_: yep, just pointing to the discussion of that dep, not trying to subvert zuul I promise :) | 18:32 |
| *** Marga_ has quit IRC | 18:32 | |
| superdan | dims_: I do not want to cause a major failure the day I'm trying to go on vacation :) | 18:32 |
| dims_ | superdan : haha yes i saw the 245200 discussion | 18:33 |
| superdan | cool | 18:33 |
| *** lpetrut has quit IRC | 18:33 | |
| mordred | sdague: well | 18:34 |
| mordred | we've removed things that were deprecated, and added new deprecations | 18:34 |
| mordred | oh | 18:34 |
| mordred | wait | 18:34 |
| mordred | I see what you're saying | 18:34 |
| sdague | mordred: right, but we need to follow deprecation window | 18:34 |
| mordred | YES totally agree | 18:34 |
| mordred | 3.0 has the current deprecation removals | 18:34 |
| sdague | right | 18:34 |
| mordred | 4.0 has the new deprecation removals | 18:34 |
| sdague | yes | 18:34 |
| mordred | I'm hoping to have the patch with the new deprecations updated in just a little bit | 18:35 |
| sdague | it's not like landing your patch in 3.0 would really impact when we could remove that chunk | 18:35 |
| *** rcernin has quit IRC | 18:36 | |
| *** nic has joined #openstack-nova | 18:36 | |
| *** diogogmt has joined #openstack-nova | 18:41 | |
| *** e0ne has quit IRC | 18:42 | |
| *** tjones has quit IRC | 18:52 | |
| *** andreykurilin__ has quit IRC | 18:52 | |
| *** burt has quit IRC | 18:55 | |
| *** aix has quit IRC | 18:56 | |
| *** danpb has quit IRC | 18:58 | |
| *** davideagnello has quit IRC | 18:58 | |
| *** nic has quit IRC | 18:59 | |
| *** nic has joined #openstack-nova | 18:59 | |
| *** ihrachys has quit IRC | 19:00 | |
| *** davideagnello has joined #openstack-nova | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** ihrachys has joined #openstack-nova | 19:01 | |
| *** ducttape_ has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| *** nic has quit IRC | 19:02 | |
| *** nic has joined #openstack-nova | 19:02 | |
| *** zhenq has quit IRC | 19:05 | |
| *** zhenq has joined #openstack-nova | 19:09 | |
| *** vilobhmm11 has quit IRC | 19:11 | |
| *** vilobhmm11 has joined #openstack-nova | 19:11 | |
| *** paul-carlton_ has quit IRC | 19:12 | |
| *** salv-orlando has quit IRC | 19:12 | |
| *** bklei has quit IRC | 19:15 | |
| *** lucasagomes is now known as lucas-dinner | 19:15 | |
| *** openstackgerrit has quit IRC | 19:17 | |
| *** openstackgerrit has joined #openstack-nova | 19:18 | |
| *** ducttape_ has joined #openstack-nova | 19:18 | |
| *** tjones has joined #openstack-nova | 19:23 | |
| *** markvoelker has joined #openstack-nova | 19:25 | |
| *** xyang1 has joined #openstack-nova | 19:26 | |
| *** rfolco has quit IRC | 19:27 | |
| *** markvoelker has quit IRC | 19:30 | |
| *** ildikov has quit IRC | 19:33 | |
| openstackgerrit | Kevin L. Mitchell proposed openstack/python-novaclient: Add a mechanism to call out deprecated options https://review.openstack.org/257595 | 19:33 |
| *** thangp has quit IRC | 19:36 | |
| *** ihrachys has quit IRC | 19:36 | |
| *** browne has joined #openstack-nova | 19:38 | |
| *** tjones has quit IRC | 19:38 | |
| *** aix has joined #openstack-nova | 19:38 | |
| *** e0ne has joined #openstack-nova | 19:39 | |
| *** ihrachys has joined #openstack-nova | 19:39 | |
| *** HenryG_afk is now known as HenryG | 19:42 | |
| *** e0ne has quit IRC | 19:42 | |
| *** pratikmallya has joined #openstack-nova | 19:47 | |
| *** yonglihe has joined #openstack-nova | 19:49 | |
| *** aginwala has joined #openstack-nova | 19:49 | |
| *** ihrachys has quit IRC | 19:49 | |
| *** ihrachys_ has joined #openstack-nova | 19:49 | |
| *** pratikma_ has joined #openstack-nova | 19:49 | |
| *** tjones has joined #openstack-nova | 19:50 | |
| *** pratikmallya has quit IRC | 19:53 | |
| *** aginwala has quit IRC | 19:55 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| john5223 | has anyone seen a bug with libvirt not being able to update an apparmor profile when trying to take a snapshot ? | 20:02 |
| john5223 | on liberty with libvirt 1.2.16 | 20:02 |
| *** dane-fichter has quit IRC | 20:07 | |
| *** tjones has quit IRC | 20:07 | |
| *** baoli_ has quit IRC | 20:07 | |
| *** hemna has joined #openstack-nova | 20:07 | |
| *** ihrachys_ has quit IRC | 20:08 | |
| *** Marga_ has joined #openstack-nova | 20:09 | |
| bauwser | superdan: am I blind or https://github.com/openstack/python-novaclient/blob/3.0.0/novaclient/__init__.py is API v2.9 ? | 20:09 |
| *** aginwala has joined #openstack-nova | 20:10 | |
| bauwser | I can see in http://git.openstack.org/cgit/openstack/python-novaclient/log/ that the 3.0.0 should contain v2.12 so me wonders what the heck is | 20:10 |
| bauwser | http://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/__init__.py?h=3.0.0 cgit shows me the same thing | 20:10 |
| bauwser | WTF? | 20:11 |
| *** ihrachys has joined #openstack-nova | 20:11 | |
| *** superdan has quit IRC | 20:12 | |
| *** ihrachys has quit IRC | 20:12 | |
| *** dansmith has joined #openstack-nova | 20:12 | |
| bauwser | http://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/__init__.py?id=6b0fe2a424630f6ebcb1cf0c42d5ac8590705875 too... | 20:12 |
| *** aginwala has quit IRC | 20:13 | |
| * bauwser is puzzled | 20:13 | |
| *** aginwala has joined #openstack-nova | 20:13 | |
| *** ihrachys has joined #openstack-nova | 20:14 | |
| *** ssurana has quit IRC | 20:15 | |
| *** Marga_ has quit IRC | 20:15 | |
| bauwser | mriedem_lunch: around? | 20:15 |
| *** tjones has joined #openstack-nova | 20:17 | |
| bauwser | dansmith: oooooh I think I see the problem | 20:18 |
| bauwser | dansmith: you provided a SHA1 for a patch that was not rebased | 20:18 |
| dansmith | Driving hang on | 20:19 |
| bauwser | np, I got the problme | 20:19 |
| bauwser | dhellmann: around ? | 20:19 |
| bauwser | dhellmann: we have a client versioning issue with a tag that was cut on a SHA1 which was not a merge rebase | 20:20 |
| dansmith | Aren't we supposed to take the half of the top merge commit actual commit? | 20:21 |
| dansmith | I thought I was told not to use the merge commit | 20:21 |
| bauwser | dansmith: unfortunately not | 20:21 |
| *** jamielennox is now known as jamielennox|away | 20:21 | |
| bauwser | dansmith: we always need to cut against a merge commit, or it's not rebased if so | 20:22 |
| dansmith | So we are supposed to use the merge commit | 20:22 |
| *** mrkz has quit IRC | 20:22 | |
| bauwser | yup | 20:22 |
| dansmith | Ok sorry | 20:22 |
| bauwser | no worries | 20:22 |
| bauwser | will handle that, drive safely | 20:22 |
| bauwser | dims_: still around ? | 20:22 |
| dims_ | bauwser : what's up? | 20:23 |
| bauwser | dims_: https://review.openstack.org/#/c/259574/ has a wrong SHA1 | 20:23 |
| bauwser | dims_: since it's not a merge commit, it's not rebased | 20:24 |
| *** davideagnello has quit IRC | 20:24 | |
| *** ssurana has joined #openstack-nova | 20:24 | |
| bauwser | dims_: see http://git.openstack.org/cgit/openstack/python-novaclient/log/ | 20:24 |
| *** rk4n has quit IRC | 20:24 | |
| dims_ | bauwser : look in this http://logs.openstack.org/74/259574/2/check/gate-releases-tox-list-changes/de30c52/console.html#_2015-12-18_19_20_27_080 | 20:24 |
| *** otter768 has joined #openstack-nova | 20:25 | |
| dims_ | does that have all you need? | 20:25 |
| bauwser | dims_: unfortunately not | 20:25 |
| bauwser | we wanted to add v2.11 support | 20:25 |
| dansmith | Dims I really thought I was spanked for using a merge commit at one point | 20:25 |
| *** bklei has joined #openstack-nova | 20:26 | |
| bauwser | dims_: since the patch on the local branch was at the v2.9 point in time, we shipped by this branch | 20:26 |
| bauwser | dims_: even if 2.11 merge commits were earlier | 20:26 |
| *** bklei has quit IRC | 20:27 | |
| dims_ | bauwser : so we can cut a 3.1.0 with a correct SHA and blacklist 3.0.0 in g-r? | 20:27 |
| *** bklei has joined #openstack-nova | 20:27 | |
| dims_ | does it need to be black listed? | 20:27 |
| bauwser | dims_: I dunno | 20:27 |
| dansmith | No | 20:27 |
| dims_ | let's cut 3.1.0 with what we really need then | 20:27 |
| bauwser | okay, writing a change | 20:28 |
| bauwser | dims_: for libs, I guess no .z ? | 20:28 |
| bauwser | writing the changelog would be a bit hard to expain | 20:28 |
| bauwser | explain | 20:28 |
| dansmith | Sorry guys I suck | 20:28 |
| dims_ | dansmith : no worries, i've messed this up multiple times | 20:29 |
| *** ssurana has quit IRC | 20:29 | |
| bauwser | dansmith: heh, no worries, we all are humans | 20:29 |
| *** davideagnello has joined #openstack-nova | 20:29 | |
| dims_ | bauwser : we should cut the new one as 3.1.0 | 20:29 |
| bauwser | at least I guess :D | 20:29 |
| *** otter768 has quit IRC | 20:29 | |
| bauwser | dims_: ack | 20:29 |
| bauwser | writing the yaml | 20:29 |
| *** davideagnello has quit IRC | 20:29 | |
| dims_ | bauwser : dansmith : you can run "tox -e list-changes" locally and review the output | 20:30 |
| *** davideagnello has joined #openstack-nova | 20:30 | |
| dansmith | Okay | 20:30 |
| *** bklei has quit IRC | 20:30 | |
| bauwser | dims_: ack, verifying locally | 20:30 |
| *** aginwala has quit IRC | 20:31 | |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Get Hierarchy info https://review.openstack.org/242568 | 20:32 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver https://review.openstack.org/242626 | 20:32 |
| bauwser | dims_: on a side note, https://github.com/openstack/releases/blob/master/deliverables/mitaka/python-novaclient.yaml#L41 is now incorrect, shall I amend it ? | 20:33 |
| dims_ | bauwser : sure if you wish | 20:33 |
| bauwser | dims_: on the same change? | 20:33 |
| dims_ | bauwser : y | 20:34 |
| bauwser | ack | 20:34 |
| bauwser | (just wanted to make sure I'm not going to push a new call for getting a tar) | 20:34 |
| *** ihrachys has quit IRC | 20:35 | |
| dims_ | won't hurt anything bauwser | 20:36 |
| bauwser | coolness | 20:37 |
| *** smatzek has quit IRC | 20:37 | |
| *** rotbeard has quit IRC | 20:38 | |
| *** hemna has quit IRC | 20:39 | |
| *** rotbeard has joined #openstack-nova | 20:40 | |
| *** e0ne has joined #openstack-nova | 20:40 | |
| *** ihrachys has joined #openstack-nova | 20:42 | |
| bauwser | FWIW here is the difference between 3.0.0 and the merge patch | 20:42 |
| bauwser | https://github.com/openstack/python-novaclient/compare/3.0.0...master | 20:42 |
| *** signed8bit has joined #openstack-nova | 20:43 | |
| bauwser | well, rather https://github.com/openstack/python-novaclient/compare/3.0.0...36e125a | 20:43 |
| *** raildo is now known as raildo-afk | 20:44 | |
| *** ssurana has joined #openstack-nova | 20:45 | |
| *** rsquared has left #openstack-nova | 20:46 | |
| *** signed8b_ has joined #openstack-nova | 20:47 | |
| *** signed8bit has quit IRC | 20:48 | |
| *** signed8b_ has quit IRC | 20:48 | |
| *** ctrath has joined #openstack-nova | 20:48 | |
| *** ssurana has quit IRC | 20:49 | |
| bauwser | dims_: https://review.openstack.org/#/c/259622/ fer yur plizur | 20:51 |
| bauwser | mriedem_lunch: in case you could +1 ^ | 20:52 |
| bauwser | I guess johnthetubaguy being away | 20:52 |
| *** e0ne has quit IRC | 20:53 | |
| *** aginwala has joined #openstack-nova | 20:58 | |
| *** aginwala has quit IRC | 20:58 | |
| *** ctrath has quit IRC | 20:59 | |
| dansmith | bauwser: presumably we've got the PTL ack, this is now a bug fix :) | 20:59 |
| bauwser | dansmith: that makes me think that I offered for the release job desc, I should amend the wiki for better clarity | 21:00 |
| *** haomaiwang has quit IRC | 21:01 | |
| bauwser | although I don't like putting lights on me | 21:01 |
| *** 17WABC36L has joined #openstack-nova | 21:01 | |
| *** cdent has joined #openstack-nova | 21:01 | |
| *** pratikma_ has quit IRC | 21:01 | |
| cdent | zzzeek: awesome post, thanks for that | 21:02 |
| *** rk4n has joined #openstack-nova | 21:03 | |
| zzzeek | cdent: awesome, hope it's correct | 21:03 |
| *** rotbeard has quit IRC | 21:03 | |
| zzzeek | i guess avg. 1.4 sec -> .014 is *two* orders of magnitude, huh | 21:03 |
| * zzzeek <--- dropout | 21:04 | |
| cdent | It rings true, but yeah, needs some independent confirmation | 21:04 |
| *** ihrachys has quit IRC | 21:04 | |
| zzzeek | cdent: well my script games things a little bit, real world there's a lot mroe than just DB access going on | 21:04 |
| cdent | the long term advice is the part that I most care about | 21:04 |
| zzzeek | cdent: but we did see that error in real load testing, so it's a thing | 21:04 |
| *** ihrachys has joined #openstack-nova | 21:05 | |
| zzzeek | we brought down greenlets and brought max_overflow way up and we get a lot more throughput + no more errors | 21:05 |
| zzzeek | now its rabbitmq falling oevr instaed :) | 21:05 |
| dansmith | sdague: I expect you're expiring soon, but the xen patch passed live migration tests, so the libvirt one will too, and the very top patch is full pass as well | 21:07 |
| dansmith | hmm, expiring means something else | 21:07 |
| dansmith | s/expiring/pumpkin-ing. | 21:07 |
| cdent | zzzeek: keep pushing and you'll find the secret deep flaw and there will be much rejoicing | 21:08 |
| * cdent is worried there are some weird magicks in oslo.messaging that will not stand up to rigorous scrutiny | 21:09 | |
| *** aginwala has joined #openstack-nova | 21:09 | |
| *** pratikmallya has joined #openstack-nova | 21:09 | |
| cdent | (in the way concurrency™ is handled) | 21:09 |
| *** aginwala has quit IRC | 21:10 | |
| *** pratikma_ has joined #openstack-nova | 21:10 | |
| dims_ | zzzeek : rabbitmq server itself failing? | 21:10 |
| zzzeek | dims_: no clue, errors in the logs, someone else here is looking at it | 21:11 |
| zzzeek | problems hit rabbitmq and i go on break :) | 21:12 |
| zzzeek | my db is getting you the data, enjoy !:) | 21:12 |
| *** burt has joined #openstack-nova | 21:13 | |
| *** ildikov has joined #openstack-nova | 21:13 | |
| dims_ | haha | 21:13 |
| zzzeek | dims_: this is with a "boot 20000 instances" kind of test | 21:14 |
| *** pratikmallya has quit IRC | 21:14 | |
| dims_ | zzzeek : ack :) | 21:14 |
| *** salv-orlando has joined #openstack-nova | 21:16 | |
| dims_ | zzzeek : we were able to relieve pressure on rabbitmq by turning off HA for rpc quite a bit in fuel - https://review.openstack.org/#/c/249180/15/files/fuel-ha-utils/ocf/set_rabbitmq_policy.sh | 21:17 |
| *** ihrachys has quit IRC | 21:17 | |
| *** suro-patz has quit IRC | 21:22 | |
| dims_ | dansmith : bauwser : http://logs.openstack.org/22/259622/1/check/gate-releases-tox-list-changes/1f6a72b/console.html looks good. | 21:23 |
| bauwser | dims_: yup, I checked locally | 21:23 |
| dims_ | ok | 21:23 |
| dims_ | pushing button now | 21:23 |
| bauwser | ack thanks | 21:23 |
| *** aginwala has joined #openstack-nova | 21:24 | |
| *** changbl has quit IRC | 21:26 | |
| *** markvoelker has joined #openstack-nova | 21:26 | |
| *** vladikr has quit IRC | 21:26 | |
| dims_ | bauwser : dansmith : done | 21:28 |
| dansmith | dims_: thanks | 21:28 |
| bauwser | dims_: thanks | 21:28 |
| *** ihrachys has joined #openstack-nova | 21:29 | |
| *** pumaranikar has quit IRC | 21:30 | |
| *** markvoelker has quit IRC | 21:31 | |
| *** mtanino has quit IRC | 21:31 | |
| *** diogogmt has quit IRC | 21:32 | |
| *** diogogmt has joined #openstack-nova | 21:33 | |
| *** xyang1 has quit IRC | 21:34 | |
| *** ssurana has joined #openstack-nova | 21:34 | |
| *** ducttape_ has quit IRC | 21:35 | |
| *** ihrachys has quit IRC | 21:35 | |
| sdague | dansmith: heh, yeh, just checking back in | 21:36 |
| dansmith | sdague: thanks | 21:36 |
| sdague | so the tests seem to work fine when the migration works, it's actually during rollback of a failed migration that things exploded in a new way | 21:37 |
| sdague | at least that's what I saw in the old patch | 21:37 |
| dansmith | sdague: but only in the bottom patch | 21:37 |
| dansmith | sdague: that code is removed in the last patch of that series, | 21:37 |
| dansmith | and adding a new test for that method seemed unnecessary given it's not tested now and I remove the code three patches later | 21:38 |
| *** ihrachys has joined #openstack-nova | 21:38 | |
| dansmith | sdague: I've also been thinking about how we might test this more functionally so we can simulate a fail | 21:38 |
| dansmith | because it's one of those things that is hard to do in tempest, | 21:38 |
| dansmith | and we really don't have good coverage for it | 21:38 |
| *** ssurana has quit IRC | 21:38 | |
| *** changbl has joined #openstack-nova | 21:39 | |
| sdague | oh, ok, I guess I didn't realize it goes away later | 21:39 |
| dansmith | which is bad because a failed live migration that is wedged is like... bad | 21:39 |
| dansmith | yeah, the bit that was failing there goes away | 21:39 |
| *** ducttape_ has joined #openstack-nova | 21:39 | |
| dansmith | it's just to transition us past the get-a-dict-expect-an-object phas | 21:39 |
| sdague | right, so that is the live migration dedicated job that tdurakov has started in on | 21:39 |
| sdague | we should probably just huddle up a set of interested people early next year and figure out what that should do | 21:39 |
| sdague | including some of these scenarios | 21:40 |
| dansmith | he is going to have failures? | 21:40 |
| sdague | it could | 21:40 |
| dansmith | okay | 21:40 |
| dansmith | that would be awesome | 21:40 |
| sdague | it will be a multinode job that is a big grey box, so we can poke things | 21:40 |
| sdague | and will only be for running live migration | 21:40 |
| dansmith | okay | 21:40 |
| dansmith | that would be cool for sure | 21:40 |
| dansmith | I also really worry about not having a grenade job for live migrate, | 21:41 |
| dansmith | because we rely on that for clearing nodes during upgrades | 21:41 |
| dansmith | and if we break that, people are going to be mad | 21:41 |
| dansmith | i.e. people that pay me | 21:41 |
| sdague | right, that's fair | 21:41 |
| sdague | however I'd like to get live migration voting somewhere less complicated first | 21:41 |
| dansmith | sure | 21:41 |
| dansmith | the reason I did this in this order, | 21:41 |
| dansmith | is that we get the most coverage of legacy-vs-current | 21:42 |
| sdague | and once that's the case, I'm ok looking at putting it into the grenade multinode case | 21:42 |
| dansmith | okay | 21:42 |
| sdague | there are just so many things that end up in flight in the grenade job, and you and are I are the only ones that debug them, that this can't be our primary voting point for live migration working or not :) | 21:42 |
| dansmith | yeah, I know | 21:43 |
| sdague | ok, looking at the diff from 15 -> 16, that seems to cover the explodes I was seeing | 21:44 |
| sdague | +2 | 21:45 |
| dansmith | thanks | 21:45 |
| * cdent puts sdague and dansmith on the nice, not naughty list | 21:47 | |
| dansmith | cdent: big mistake ... | 21:48 |
| cdent | It's very context dependent | 21:48 |
| *** mrkz has joined #openstack-nova | 21:49 | |
| bauwser | weekends, folks | 21:51 |
| *** bauwser is now known as bauzas | 21:51 | |
| *** sturlry has quit IRC | 21:51 | |
| andrewbogott | If any nova core devs are at loose ends this Friday afternoon… https://review.openstack.org/#/c/242251/ has a bunch of +1s and could use a merge. | 21:51 |
| *** sturlry has joined #openstack-nova | 21:55 | |
| *** thorst has quit IRC | 21:56 | |
| *** salv-orlando has quit IRC | 21:56 | |
| openstackgerrit | Kevin L. Mitchell proposed openstack/python-novaclient: document search_opts parameter https://review.openstack.org/259408 | 21:58 |
| *** sturlry_ has joined #openstack-nova | 21:59 | |
| *** sturlry has quit IRC | 21:59 | |
| *** 17WABC36L has quit IRC | 22:01 | |
| *** lascii is now known as alaski | 22:01 | |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| *** ssurana has joined #openstack-nova | 22:02 | |
| *** aginwala has quit IRC | 22:03 | |
| migi | dansmith: could we discuss your comment before weekend? :) https://review.openstack.org/#/c/258878/ | 22:05 |
| *** jwcroppe has quit IRC | 22:05 | |
| *** aginwala has joined #openstack-nova | 22:05 | |
| *** jwcroppe has joined #openstack-nova | 22:05 | |
| migi | you are proposing to leave it as it was, but it won't work if I keep it as it was | 22:06 |
| migi | so some fix must apply | 22:06 |
| dansmith | migi: I looked a bit, and I'm concerned that you're getting that stack trace | 22:06 |
| migi | well, it's not one place | 22:06 |
| *** ssurana has quit IRC | 22:06 | |
| dansmith | because I'm pretty sure that's a flavor id, so I'm worried that it's getting passed somewhere | 22:06 |
| dansmith | so I just wanted to dig deeper first | 22:06 |
| migi | this is just required to run those two gentelmens | 22:06 |
| migi | ok | 22:07 |
| migi | dansmith: to be honest I had really hard time finding where those 1 is not valid uuid coming from the different .commit | 22:07 |
| migi | as it's not clear from stack trace - also dbx is not a big help to me here | 22:07 |
| dansmith | yeah | 22:08 |
| migi | this was quite easy, but there are some where it's nowhere neer the stack trace | 22:08 |
| migi | s/neer/near | 22:08 |
| migi | ok so I will then not bother you and wish great weekend | 22:08 |
| dansmith | I'll still try to get to it before I go, but no promises.. | 22:09 |
| dansmith | you can put other patches under it in the meantime | 22:09 |
| migi | no worries if you won't | 22:09 |
| migi | will do but not today | 22:09 |
| dansmith | sure.. thanks again for doing this work! | 22:10 |
| *** jwcroppe has quit IRC | 22:10 | |
| *** apoorvad has quit IRC | 22:10 | |
| *** ssurana has joined #openstack-nova | 22:10 | |
| *** alex_xu has quit IRC | 22:11 | |
| *** vilobhmm111 has joined #openstack-nova | 22:12 | |
| *** alex_xu has joined #openstack-nova | 22:13 | |
| *** vilobhmm11 has quit IRC | 22:14 | |
| *** thorst has joined #openstack-nova | 22:15 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: use oslo.db[fixtures] instead of testresources https://review.openstack.org/237359 | 22:18 |
| *** ctrath has joined #openstack-nova | 22:19 | |
| *** ihrachys has quit IRC | 22:25 | |
| *** otter768 has joined #openstack-nova | 22:25 | |
| *** ihrachys has joined #openstack-nova | 22:28 | |
| *** thorst has quit IRC | 22:28 | |
| *** thorst has joined #openstack-nova | 22:28 | |
| *** otter768 has quit IRC | 22:30 | |
| *** alex_xu has quit IRC | 22:34 | |
| *** ihrachys has quit IRC | 22:35 | |
| *** alex_xu has joined #openstack-nova | 22:36 | |
| *** aginwala has quit IRC | 22:36 | |
| *** ihrachys has joined #openstack-nova | 22:36 | |
| *** thorst has quit IRC | 22:36 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Just trying gate-nova-requirements job https://review.openstack.org/259645 | 22:38 |
| *** aginwala has joined #openstack-nova | 22:38 | |
| *** rk4n has quit IRC | 22:38 | |
| *** aginwala has quit IRC | 22:39 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: use oslo.db[fixtures] instead of testresources https://review.openstack.org/237359 | 22:41 |
| *** ducttape_ has quit IRC | 22:44 | |
| *** aginwala has joined #openstack-nova | 22:47 | |
| *** aginwala_ has joined #openstack-nova | 22:49 | |
| *** aginwala has quit IRC | 22:49 | |
| *** rcernin has joined #openstack-nova | 22:52 | |
| *** burt has quit IRC | 22:53 | |
| auggy | ok this is probably something really obvious i'm just missing, but when i'm running tox -e py27, i keep getting this no module found error - | 22:56 |
| auggy | No module named castellan.common.exception | 22:56 |
| *** pratikma_ has quit IRC | 22:56 | |
| auggy | this just happened today, i pulled from master and just tried to run the tests, and as far as i can tell the castellan module is installed but i must have missed something | 22:56 |
| *** apoorvad has joined #openstack-nova | 22:59 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** salv-orlando has joined #openstack-nova | 23:01 | |
| *** haomaiwang has joined #openstack-nova | 23:01 | |
| *** pratikmallya has joined #openstack-nova | 23:02 | |
| *** pratikmallya has quit IRC | 23:03 | |
| *** pratikmallya has joined #openstack-nova | 23:03 | |
| *** tjones has quit IRC | 23:04 | |
| *** tjones has joined #openstack-nova | 23:05 | |
| *** pratikmallya has quit IRC | 23:05 | |
| *** rcernin has quit IRC | 23:08 | |
| *** rk4n has joined #openstack-nova | 23:09 | |
| *** mwagner_lap has quit IRC | 23:10 | |
| auggy | ok nm it's working now :/ | 23:10 |
| *** rohit_ has quit IRC | 23:10 | |
| * auggy has no idea | 23:10 | |
| *** ianbrown has quit IRC | 23:12 | |
| *** markvoelker has joined #openstack-nova | 23:12 | |
| *** ianbrown has joined #openstack-nova | 23:12 | |
| *** pixelbeat_ has joined #openstack-nova | 23:12 | |
| *** aginwala_ has quit IRC | 23:12 | |
| *** suro-patz has joined #openstack-nova | 23:15 | |
| *** markvoelker has quit IRC | 23:17 | |
| openstackgerrit | Anita Kuno proposed openstack/nova: [WIP] Just trying gate-nova-requirements job https://review.openstack.org/259645 | 23:17 |
| *** salv-orlando has quit IRC | 23:18 | |
| *** ljxiash has joined #openstack-nova | 23:18 | |
| *** suro-patz1 has joined #openstack-nova | 23:22 | |
| *** rk4n has quit IRC | 23:22 | |
| *** rk4n has joined #openstack-nova | 23:22 | |
| *** ljxiash has quit IRC | 23:22 | |
| *** suro-patz has quit IRC | 23:24 | |
| *** tjones has quit IRC | 23:25 | |
| *** alex_xu has quit IRC | 23:30 | |
| *** rushiagr_away is now known as rushiagr | 23:31 | |
| *** alex_xu has joined #openstack-nova | 23:31 | |
| *** zenoway has joined #openstack-nova | 23:32 | |
| *** thorst has joined #openstack-nova | 23:34 | |
| *** aginwala has joined #openstack-nova | 23:35 | |
| *** zenoway has quit IRC | 23:37 | |
| *** rk4n has quit IRC | 23:37 | |
| *** unicell1 has joined #openstack-nova | 23:38 | |
| *** pratikmallya has joined #openstack-nova | 23:39 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: use oslo.db[fixtures] instead of testresources https://review.openstack.org/237359 | 23:39 |
| *** unicell has quit IRC | 23:40 | |
| *** thorst has quit IRC | 23:42 | |
| *** ducttape_ has joined #openstack-nova | 23:44 | |
| *** vilobhmm111 has quit IRC | 23:46 | |
| *** dims has joined #openstack-nova | 23:46 | |
| *** dims_ has quit IRC | 23:48 | |
| *** ducttape_ has quit IRC | 23:49 | |
| openstackgerrit | Dongcan Ye proposed openstack/nova: VMware: Using constants for rdm and rdmp disk https://review.openstack.org/259652 | 23:50 |
| *** cdent has quit IRC | 23:54 | |
| *** mrkz has quit IRC | 23:56 | |
| *** dims has quit IRC | 23:57 | |
| *** rushiagr is now known as rushiagr_away | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
