| *** achanda has quit IRC | 00:01 | |
| *** achanda_ has quit IRC | 00:02 | |
| *** dims has quit IRC | 00:03 | |
| *** takedakn has joined #openstack-nova | 00:05 | |
| *** mtanino has quit IRC | 00:05 | |
| *** pv_ has quit IRC | 00:05 | |
| *** jdurgin has quit IRC | 00:06 | |
| *** baoli has joined #openstack-nova | 00:07 | |
| *** jdurgin has joined #openstack-nova | 00:07 | |
| *** mtanino has joined #openstack-nova | 00:09 | |
| *** tjones has joined #openstack-nova | 00:10 | |
| *** rushil has quit IRC | 00:10 | |
| *** takedakn has quit IRC | 00:12 | |
| *** Piet has quit IRC | 00:16 | |
| *** tjones has left #openstack-nova | 00:17 | |
| *** diogogmt has quit IRC | 00:18 | |
| *** sachin has quit IRC | 00:21 | |
| *** sachin has joined #openstack-nova | 00:22 | |
| *** sachin has quit IRC | 00:24 | |
| *** sachin has joined #openstack-nova | 00:24 | |
| *** vivekd has joined #openstack-nova | 00:27 | |
| *** sachin has quit IRC | 00:29 | |
| *** takedakn has joined #openstack-nova | 00:29 | |
| *** mtanino has quit IRC | 00:29 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: API: Handle InstanceUnknownCell exceptions https://review.openstack.org/212965 | 00:30 |
|---|---|---|
| *** tojuvone has joined #openstack-nova | 00:32 | |
| *** mtanino has joined #openstack-nova | 00:32 | |
| *** sachin has joined #openstack-nova | 00:33 | |
| *** meghal has quit IRC | 00:33 | |
| *** sachin has joined #openstack-nova | 00:33 | |
| *** Piet has joined #openstack-nova | 00:37 | |
| *** tri2sing has joined #openstack-nova | 00:38 | |
| *** tpeoples has quit IRC | 00:38 | |
| *** sachin has quit IRC | 00:38 | |
| *** tri2sing_ has joined #openstack-nova | 00:42 | |
| *** sachin has joined #openstack-nova | 00:43 | |
| *** thorst has joined #openstack-nova | 00:45 | |
| *** nic has quit IRC | 00:46 | |
| *** choyj has joined #openstack-nova | 00:46 | |
| *** aysyd has quit IRC | 00:47 | |
| *** Sukhdev has joined #openstack-nova | 00:47 | |
| *** pixelb has joined #openstack-nova | 00:49 | |
| *** yuntongjin has joined #openstack-nova | 00:50 | |
| *** tri2sing has quit IRC | 00:52 | |
| *** tri2sing_ has quit IRC | 00:52 | |
| *** liusheng has joined #openstack-nova | 00:57 | |
| *** zhangjn has joined #openstack-nova | 00:57 | |
| *** takedakn has quit IRC | 00:59 | |
| *** irenab_ has joined #openstack-nova | 01:01 | |
| *** skylerberg has quit IRC | 01:03 | |
| *** vivekd_ has joined #openstack-nova | 01:03 | |
| *** Hazelesque has quit IRC | 01:03 | |
| *** Hazelesque has joined #openstack-nova | 01:03 | |
| *** irenab has quit IRC | 01:03 | |
| *** irenab_ is now known as irenab | 01:03 | |
| *** Hazelesque has quit IRC | 01:03 | |
| *** Hazelesque has joined #openstack-nova | 01:03 | |
| *** vivekd has quit IRC | 01:04 | |
| *** vivekd_ is now known as vivekd | 01:04 | |
| *** dave-mccowan has quit IRC | 01:04 | |
| mriedem | cfriesen: yes | 01:07 |
| *** asettle is now known as asettle-afk | 01:07 | |
| mriedem | cfriesen: http://status.openstack.org/elastic-recheck/index.html#1490740 | 01:07 |
| *** baoli has quit IRC | 01:08 | |
| *** yuntongjin1 has joined #openstack-nova | 01:10 | |
| *** yuntongjin has quit IRC | 01:11 | |
| *** Apoorva has quit IRC | 01:14 | |
| *** yuntongjin has joined #openstack-nova | 01:16 | |
| *** VW_ has joined #openstack-nova | 01:17 | |
| *** zhangjn_ has joined #openstack-nova | 01:17 | |
| *** zhangjn has quit IRC | 01:19 | |
| *** yuntongjin1 has quit IRC | 01:19 | |
| *** dims has joined #openstack-nova | 01:20 | |
| *** vilobhmm1 has quit IRC | 01:22 | |
| *** hanzhang has quit IRC | 01:22 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 01:22 |
| *** hanzhang has joined #openstack-nova | 01:23 | |
| *** sachin has quit IRC | 01:23 | |
| *** sachin has joined #openstack-nova | 01:25 | |
| *** sachin has quit IRC | 01:26 | |
| *** sachin has joined #openstack-nova | 01:27 | |
| *** baoli has joined #openstack-nova | 01:29 | |
| *** mtanino has quit IRC | 01:29 | |
| *** Piet has quit IRC | 01:30 | |
| *** meghal has joined #openstack-nova | 01:30 | |
| *** tpeoples has joined #openstack-nova | 01:32 | |
| *** haomaiwang has joined #openstack-nova | 01:33 | |
| *** shuquan has joined #openstack-nova | 01:33 | |
| *** baoli has quit IRC | 01:34 | |
| *** baoli has joined #openstack-nova | 01:34 | |
| *** shuquan has quit IRC | 01:38 | |
| *** SourabhP has quit IRC | 01:38 | |
| *** meghal has quit IRC | 01:39 | |
| *** vivekd has quit IRC | 01:42 | |
| *** vivekd has joined #openstack-nova | 01:44 | |
| *** mriedem is now known as mriedem_away | 01:44 | |
| *** haigang has quit IRC | 01:44 | |
| *** Sukhdev has quit IRC | 01:51 | |
| openstackgerrit | Sachi King proposed openstack/nova: Add constraint target to tox.ini https://review.openstack.org/205931 | 01:51 |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 01:53 |
| *** shangxdy has joined #openstack-nova | 01:55 | |
| *** sachin has quit IRC | 01:55 | |
| *** zhangjn_ has quit IRC | 01:58 | |
| *** zhangjn has joined #openstack-nova | 01:59 | |
| *** zhangjn has quit IRC | 01:59 | |
| *** zhangjn has joined #openstack-nova | 02:00 | |
| *** lyanchih has joined #openstack-nova | 02:04 | |
| *** zzzeek has quit IRC | 02:04 | |
| *** qianli has quit IRC | 02:06 | |
| *** asettle-afk has quit IRC | 02:07 | |
| *** Piet has joined #openstack-nova | 02:12 | |
| *** vilobhmm1 has joined #openstack-nova | 02:12 | |
| *** qianli has joined #openstack-nova | 02:16 | |
| *** vilobhmm1 has left #openstack-nova | 02:18 | |
| *** I has joined #openstack-nova | 02:20 | |
| *** bkopilov has quit IRC | 02:20 | |
| *** I is now known as Guest10903 | 02:20 | |
| *** thorst has quit IRC | 02:29 | |
| garyk | mriedem_away: i rebased and updtaed that one (did that count as a review?) | 02:29 |
| *** baoli has quit IRC | 02:30 | |
| *** VW_ has quit IRC | 02:33 | |
| *** VW_ has joined #openstack-nova | 02:34 | |
| *** rushiagr_away is now known as rushiagr | 02:36 | |
| *** baoli has joined #openstack-nova | 02:37 | |
| *** pixelb has quit IRC | 02:38 | |
| *** hparekh has joined #openstack-nova | 02:38 | |
| *** VW_ has quit IRC | 02:38 | |
| *** meghal has joined #openstack-nova | 02:45 | |
| *** rushiagr is now known as rushiagr_away | 02:45 | |
| *** meghal1 has joined #openstack-nova | 02:46 | |
| *** meghal has quit IRC | 02:49 | |
| *** yamahata has quit IRC | 02:50 | |
| *** baoli has quit IRC | 02:52 | |
| *** hakimo has joined #openstack-nova | 02:52 | |
| *** wanghua has joined #openstack-nova | 02:54 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** rushiagr_away is now known as rushiagr | 02:55 | |
| *** baoli has joined #openstack-nova | 02:55 | |
| *** VW_ has joined #openstack-nova | 02:57 | |
| *** humble_ has joined #openstack-nova | 02:57 | |
| *** asettle has joined #openstack-nova | 02:58 | |
| *** deepthi has joined #openstack-nova | 03:01 | |
| *** wanghua has quit IRC | 03:01 | |
| *** venkat_p has joined #openstack-nova | 03:03 | |
| *** achanda has joined #openstack-nova | 03:07 | |
| *** meghal1 has quit IRC | 03:07 | |
| *** baoli has quit IRC | 03:08 | |
| *** baoli has joined #openstack-nova | 03:10 | |
| *** eglynn__ has quit IRC | 03:10 | |
| *** xyang1 has quit IRC | 03:13 | |
| *** baoli has quit IRC | 03:14 | |
| *** humble__ has joined #openstack-nova | 03:14 | |
| *** rushil has joined #openstack-nova | 03:15 | |
| *** meghal has joined #openstack-nova | 03:17 | |
| *** humble_ has quit IRC | 03:18 | |
| *** yuntongjin has quit IRC | 03:23 | |
| openstackgerrit | Wen Zhi Yu proposed openstack/nova-specs: Spec for nova API blueprint correct-volume-attachment-id-in-nova-api-volumes https://review.openstack.org/214021 | 03:24 |
| *** meghal1 has joined #openstack-nova | 03:25 | |
| *** meghal has quit IRC | 03:26 | |
| *** mriedem_away has quit IRC | 03:27 | |
| *** baoli has joined #openstack-nova | 03:28 | |
| *** unicell has quit IRC | 03:31 | |
| *** yuntongjin has joined #openstack-nova | 03:34 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 03:38 |
| *** yuntongjin has quit IRC | 03:38 | |
| *** yuntongjin has joined #openstack-nova | 03:39 | |
| *** tpeoples has quit IRC | 03:39 | |
| *** yuntongjin1 has joined #openstack-nova | 03:40 | |
| *** yuntongjin has quit IRC | 03:40 | |
| *** humble_ has joined #openstack-nova | 03:42 | |
| *** vivekd has quit IRC | 03:44 | |
| *** humble__ has quit IRC | 03:44 | |
| *** links has joined #openstack-nova | 03:46 | |
| *** humble__ has joined #openstack-nova | 03:47 | |
| *** rushil has quit IRC | 03:50 | |
| *** humble_ has quit IRC | 03:50 | |
| *** achanda has quit IRC | 03:51 | |
| *** haomaiwang has quit IRC | 03:51 | |
| *** haomaiwang has joined #openstack-nova | 03:52 | |
| *** yuntongjin1 has quit IRC | 03:57 | |
| *** markvoelker has joined #openstack-nova | 04:00 | |
| *** bkopilov has joined #openstack-nova | 04:00 | |
| *** markvoelker_ has joined #openstack-nova | 04:02 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports https://review.openstack.org/216917 | 04:04 |
| *** markvoelker has quit IRC | 04:04 | |
| *** rushiagr is now known as rushiagr_away | 04:04 | |
| *** ildikov has joined #openstack-nova | 04:04 | |
| *** RuiChen has joined #openstack-nova | 04:05 | |
| *** Sree has joined #openstack-nova | 04:08 | |
| *** Sree has quit IRC | 04:08 | |
| *** Sree has joined #openstack-nova | 04:09 | |
| *** garyk has quit IRC | 04:13 | |
| *** takedakn has joined #openstack-nova | 04:14 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Expose keystoneclient's session and auth plugin loading parameters https://review.openstack.org/214423 | 04:15 |
| jamielennox | dims: i think you should just stick to the common opts, ^ | 04:18 |
| jamielennox | if you register everything possible for every plugin on the system that can become ugly | 04:18 |
| *** meghal1 has quit IRC | 04:22 | |
| *** Sree_ has joined #openstack-nova | 04:22 | |
| *** tpeoples has joined #openstack-nova | 04:22 | |
| *** Sree_ is now known as Guest41206 | 04:22 | |
| *** takedakn has quit IRC | 04:23 | |
| *** Sree has quit IRC | 04:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/216968 | 04:26 |
| *** dims has quit IRC | 04:30 | |
| *** irenab has quit IRC | 04:31 | |
| *** rushiagr_away is now known as rushiagr | 04:31 | |
| *** baoli has quit IRC | 04:34 | |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 04:38 |
| *** yonglihe has joined #openstack-nova | 04:41 | |
| *** rushiagr is now known as rushiagr_away | 04:44 | |
| *** meghal has joined #openstack-nova | 04:44 | |
| *** achanda has joined #openstack-nova | 04:45 | |
| alex_xu | oomichi: hope you can take a look at this https://review.openstack.org/218103 when you have time, thanks! | 04:47 |
| *** shaohe_feng has joined #openstack-nova | 04:48 | |
| *** meghal1 has joined #openstack-nova | 04:49 | |
| *** meghal has quit IRC | 04:49 | |
| oomichi | alex_xu: done, nice work :) | 04:53 |
| alex_xu | oomichi: thanks :) | 04:53 |
| *** Sukhdev has joined #openstack-nova | 04:58 | |
| *** unicell has joined #openstack-nova | 05:02 | |
| *** vivekd has joined #openstack-nova | 05:05 | |
| *** yuntongjin has joined #openstack-nova | 05:05 | |
| *** shaohe_feng1 has joined #openstack-nova | 05:11 | |
| *** yuntongjin1 has joined #openstack-nova | 05:11 | |
| *** ildikov has quit IRC | 05:12 | |
| *** yuntongjin has quit IRC | 05:13 | |
| *** shaohe_feng has quit IRC | 05:13 | |
| *** yuntongjin has joined #openstack-nova | 05:16 | |
| *** yuntongjin1 has quit IRC | 05:16 | |
| *** alex_klimov has joined #openstack-nova | 05:17 | |
| *** shaohe_feng has joined #openstack-nova | 05:25 | |
| *** shaohe_feng1 has quit IRC | 05:26 | |
| *** alex_klimov has quit IRC | 05:26 | |
| *** lpetrut has joined #openstack-nova | 05:31 | |
| *** VW_ has quit IRC | 05:31 | |
| *** VW_ has joined #openstack-nova | 05:32 | |
| *** irenab has joined #openstack-nova | 05:34 | |
| *** ildikov has joined #openstack-nova | 05:38 | |
| *** lyanchih has quit IRC | 05:40 | |
| *** garyk has joined #openstack-nova | 05:40 | |
| *** moshele has joined #openstack-nova | 05:40 | |
| *** meghal1 has quit IRC | 05:40 | |
| *** sudipto has joined #openstack-nova | 05:41 | |
| *** Guest10903 has quit IRC | 05:42 | |
| *** SourabhP has joined #openstack-nova | 05:46 | |
| *** hparekh2 has joined #openstack-nova | 05:46 | |
| *** hparekh has quit IRC | 05:46 | |
| *** cfriesen has quit IRC | 05:46 | |
| *** lyanchih has joined #openstack-nova | 05:48 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Object: Fix KeyError when loading instance from db https://review.openstack.org/205917 | 05:49 |
| *** nkrinner has joined #openstack-nova | 05:51 | |
| *** markvoelker_ has quit IRC | 05:53 | |
| *** garyk1 has joined #openstack-nova | 05:55 | |
| *** garyk has quit IRC | 05:57 | |
| *** I has joined #openstack-nova | 05:58 | |
| *** I is now known as Guest92023 | 05:58 | |
| *** dims has joined #openstack-nova | 05:59 | |
| *** lyanchih has quit IRC | 06:01 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova: Allow force-delete irrespective of VM task_state https://review.openstack.org/163798 | 06:03 |
| *** dims has quit IRC | 06:04 | |
| *** eglynn__ has joined #openstack-nova | 06:05 | |
| *** lpetrut has quit IRC | 06:10 | |
| *** Nikolay_St has quit IRC | 06:11 | |
| *** unicell has quit IRC | 06:15 | |
| *** unicell has joined #openstack-nova | 06:15 | |
| *** lyanchih has joined #openstack-nova | 06:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 06:16 |
| *** shoutm has quit IRC | 06:17 | |
| *** vilobhmm11 has joined #openstack-nova | 06:18 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 06:20 |
| *** garyk has joined #openstack-nova | 06:20 | |
| openstackgerrit | Joe Cropper proposed openstack/nova: Fix nova metric cache TTL https://review.openstack.org/219153 | 06:21 |
| *** RuiChen has quit IRC | 06:22 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 06:22 |
| *** garyk1 has quit IRC | 06:23 | |
| *** asettle has quit IRC | 06:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/212367 | 06:27 |
| *** Murali has joined #openstack-nova | 06:28 | |
| *** sudipto has quit IRC | 06:29 | |
| *** hparekh2 has quit IRC | 06:30 | |
| *** hparekh has joined #openstack-nova | 06:31 | |
| *** deepakcs has joined #openstack-nova | 06:34 | |
| *** SourabhP has quit IRC | 06:35 | |
| *** afazekas__ has joined #openstack-nova | 06:36 | |
| *** armax has quit IRC | 06:37 | |
| *** lpetrut has joined #openstack-nova | 06:38 | |
| *** emagana has joined #openstack-nova | 06:41 | |
| jwcroppe | alex_hu: you familiar with the metrics framework? | 06:41 |
| jwcroppe | alex_xu: ^^^ (darn typo) :-) | 06:42 |
| *** kiran-r has joined #openstack-nova | 06:43 | |
| alex_xu | jwcroppe: I know a little. I think jaypipe is expert | 06:43 |
| jwcroppe | alex_xu: wondering if you can look at https://review.openstack.org/#/c/219153/ if you get a sec? I ran into this - would like your feedback if you have a sec? | 06:44 |
| jwcroppe | I'll hunt down jaypipes tomorrow too :) | 06:44 |
| alex_xu | jwcroppe: no problem, let me try if I can help | 06:44 |
| jwcroppe | it's a simple change, IMO. The bug causes some sporadically weird behavior when it crops up | 06:45 |
| jwcroppe | alex_xu: thanks for your time | 06:45 |
| *** RuiChen has joined #openstack-nova | 06:45 | |
| alex_xu | jwcroppe: np | 06:46 |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix incomplete error message of quota exceeded https://review.openstack.org/197460 | 06:46 |
| *** Sukhdev has quit IRC | 06:46 | |
| *** josecastroleon has quit IRC | 06:50 | |
| openstackgerrit | Huang Zhiteng proposed openstack/nova: libvirt: Force iSCSI disconnect after timeout https://review.openstack.org/167815 | 06:50 |
| *** leseb_ has quit IRC | 06:52 | |
| openstackgerrit | Joe Cropper proposed openstack/nova: Fix nova metric cache TTL https://review.openstack.org/219153 | 06:52 |
| *** kiran-r has quit IRC | 06:53 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata https://review.openstack.org/208277 | 06:54 |
| *** alexschm has joined #openstack-nova | 06:54 | |
| openstackgerrit | Joe Cropper proposed openstack/nova: Fix nova metric cache TTL https://review.openstack.org/219153 | 06:54 |
| *** leseb_ has joined #openstack-nova | 06:56 | |
| *** Nikolay_St has joined #openstack-nova | 06:58 | |
| *** lpetrut has quit IRC | 07:01 | |
| *** alex_klimov has joined #openstack-nova | 07:02 | |
| *** markus_z has joined #openstack-nova | 07:03 | |
| *** rdopiera has joined #openstack-nova | 07:05 | |
| *** slagle has quit IRC | 07:06 | |
| *** stefan_amann has joined #openstack-nova | 07:09 | |
| *** alex_klimov has quit IRC | 07:10 | |
| *** achanda has quit IRC | 07:11 | |
| *** hparekh2 has joined #openstack-nova | 07:12 | |
| *** hparekh has quit IRC | 07:13 | |
| *** sirushti has quit IRC | 07:15 | |
| *** rotbeard has joined #openstack-nova | 07:16 | |
| *** tpeoples has quit IRC | 07:16 | |
| *** ndipanov has quit IRC | 07:16 | |
| *** achanda has joined #openstack-nova | 07:17 | |
| *** hparekh has joined #openstack-nova | 07:17 | |
| *** VW_ has quit IRC | 07:17 | |
| *** pbrooko has joined #openstack-nova | 07:18 | |
| *** VW_ has joined #openstack-nova | 07:18 | |
| *** VW_ has quit IRC | 07:18 | |
| *** hparekh2 has quit IRC | 07:18 | |
| jwcroppe | alex_xu: moving the timestamp allows the stats to be collected before we start the 'counter' | 07:18 |
| *** VW_ has joined #openstack-nova | 07:19 | |
| *** jichen has joined #openstack-nova | 07:19 | |
| jwcroppe | alex_xu: if you get cases where the stat collection takes > 1 second, the cache expires on the next metric (e.g., time X we try to get cpu.user.percent with data Y, but then cpu.kernel.percent is calculated with data Z -- leads to inconsistent values since the intent is to have all metrics based off the same data set) | 07:20 |
| alex_xu | jwcroppe: yea, I'm a little confuse on what problem resolved? | 07:20 |
| jwcroppe | alex_xu: see previous | 07:21 |
| *** eglynn__ has quit IRC | 07:21 | |
| *** deepakcs has quit IRC | 07:22 | |
| jwcroppe | alex_xu: basically the way the metrics framework works is that you have a list of metrics you want to retrieve, and then it calls get_metric(name) for each metric. The idea is that the same underlying data is used to calculate all of those values, but due to this bug, that's not necessarily the case | 07:22 |
| *** shoutm has joined #openstack-nova | 07:24 | |
| alex_xu | jwcroppe: it's more like the libvirt can get the result back amtoic | 07:25 |
| *** achanda has quit IRC | 07:25 | |
| jwcroppe | alex_xu: yes, that's what this patch is trying to help assist with across all 10 metrics | 07:25 |
| *** achanda has joined #openstack-nova | 07:26 | |
| jwcroppe | alex_xu: right now under certain cases (heavy system load) only a subset of the 10 come back using the same underlying data, which leads to inconsistent data readings | 07:26 |
| *** ndipanov has joined #openstack-nova | 07:27 | |
| *** matrohon has joined #openstack-nova | 07:28 | |
| jwcroppe | I'm sure jaypipes will look at it in the next few days, but I hope that explains a bit more, alex_xu. :) | 07:28 |
| alex_xu | jwcroppe: the '10 metrics' you point to what? | 07:29 |
| *** achanda has quit IRC | 07:30 | |
| jwcroppe | alex_xu: https://github.com/openstack/nova/blob/master/nova/compute/monitors/base.py#L71 | 07:30 |
| jwcroppe | alex_hu: and this bug is caused when get_metric(name) (where name is each of those metrics) crosses the 1 second TTL ... inconsistent results | 07:31 |
| alex_xu | jwcroppe: ^ this word probably let me get the point, let me check the code | 07:33 |
| jwcroppe | e.g., so you might get something like cpu_user_percent = 15, cpu_kernel_percent = 10, cpu_idle_percent = 85, cpu_iowait_percent = 0 for the same 'sample period' -- which doesn't make any sense :) | 07:33 |
| *** ishant has joined #openstack-nova | 07:34 | |
| *** kjelly has quit IRC | 07:35 | |
| alex_xu | jwcroppe: ah, I think I got you now :) | 07:35 |
| jwcroppe | cool | 07:35 |
| jwcroppe | yes, it takes a min to wrap your head around. | 07:35 |
| alex_xu | jwcroppe: I think the fix make sense to me, let me review the code detail again | 07:35 |
| jwcroppe | k | 07:36 |
| *** kjelly has joined #openstack-nova | 07:36 | |
| alex_xu | jwcroppe: ah, thanks for the explain | 07:36 |
| jwcroppe | alex_xu: no problem | 07:36 |
| jwcroppe | basically the patch pushes the timestamp cache after the 'expensive' call and then the rest of the metrics can come right in and use the data quickly | 07:37 |
| *** kiran-r has joined #openstack-nova | 07:38 | |
| *** pbrooko has quit IRC | 07:38 | |
| jwcroppe | alex_xu: e.g., it would be like me telling you I will cache DNS entry for 15 minutes *before* I do the look-up, but it takes 16 minutes to perform the look-up. By the time I create the cached entry, the TTL expires...not very useful :) | 07:39 |
| alex_xu | jwcroppe: yea | 07:39 |
| *** emagana has quit IRC | 07:39 | |
| *** rotbeard has quit IRC | 07:40 | |
| *** shaohe_feng has quit IRC | 07:44 | |
| jwcroppe | alex_xu: thanks for looking - it's bed time for me now... let me know if you have more questions | 07:44 |
| alex_xu | jwcroppe: np, have good rest :) | 07:45 |
| *** jwcroppe is now known as jwcroppe_zzz | 07:45 | |
| *** vivekd has quit IRC | 07:46 | |
| *** pixelb has joined #openstack-nova | 07:49 | |
| *** rotbeard has joined #openstack-nova | 07:51 | |
| *** jlanoux has joined #openstack-nova | 07:53 | |
| *** romainh has joined #openstack-nova | 07:56 | |
| *** alex_klimov has joined #openstack-nova | 07:56 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix a few typos https://review.openstack.org/208749 | 07:57 |
| *** rotbeard has quit IRC | 07:58 | |
| *** vilobhmm11 has quit IRC | 07:58 | |
| *** lucas-dinner has joined #openstack-nova | 08:00 | |
| *** lucas-dinner has quit IRC | 08:00 | |
| *** lucasagomes has joined #openstack-nova | 08:00 | |
| *** yassine has joined #openstack-nova | 08:02 | |
| *** karimb has joined #openstack-nova | 08:03 | |
| *** jistr has joined #openstack-nova | 08:08 | |
| *** pbrooko has joined #openstack-nova | 08:08 | |
| *** sudipto has joined #openstack-nova | 08:10 | |
| *** lennyb has joined #openstack-nova | 08:11 | |
| garyk | johnthetubaguy: yo, you around for a question - and no its not about -2's (i can defer that till later) | 08:20 |
| *** eglynn__ has joined #openstack-nova | 08:20 | |
| johnthetubaguy | ndipanov: just looking at the migration patches, are we down to just the bug fix part now, it looks a bit that way? | 08:21 |
| johnthetubaguy | garyk: hi, happy to answer | 08:21 |
| garyk | is nova going to change to a feature model in the futue or we stickin to the 6 month release cycle? | 08:22 |
| *** danpb has joined #openstack-nova | 08:22 | |
| johnthetubaguy | garyk: so my preference is for more regular time bounded releases, with a stable branch (and upgrade limits, i.e. major bump) every 6 months | 08:23 |
| johnthetubaguy | garyk: but the real answer is, I don't think its decided yet, its a good debate for the summit | 08:23 |
| johnthetubaguy | I think ironic is testing the waters of the alternatives, I am interested to see how things have gone for them | 08:24 |
| garyk | ok, understood | 08:24 |
| johnthetubaguy | garyk: what would you prefer? | 08:24 |
| ndipanov | johnthetubaguy, well provided the object stuff merges - we are | 08:24 |
| ndipanov | johnthetubaguy, https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1417667,n,z | 08:25 |
| johnthetubaguy | ndipanov: it looks like thats merged, or am I missing some patches? | 08:25 |
| ndipanov | the lower two are missing | 08:25 |
| ndipanov | one is approved | 08:25 |
| garyk | johnthetubaguy: i am in favor of the current release cycles that we have | 08:25 |
| ndipanov | the other one has a +2 | 08:25 |
| ndipanov | but the gate :( | 08:25 |
| garyk | i have yet to understand the other models - not sure what was proposed. | 08:26 |
| johnthetubaguy | ndipanov: yeah, its that time of the release, but its not looking too bad right now | 08:26 |
| ndipanov | ok hitting recheck | 08:26 |
| *** yamahata has joined #openstack-nova | 08:26 | |
| ndipanov | this one needs a +W | 08:26 |
| ndipanov | all others are bugfixes basically | 08:26 |
| ndipanov | on the other branch too | 08:26 |
| ndipanov | (the evacuate stuff you reviewed and I split off to get more reviews | 08:27 |
| johnthetubaguy | garyk: so the only change I am thinking is to give the milestones real names, I guess, so they don't look tottally like betas | 08:27 |
| ndipanov | johnthetubaguy, https://review.openstack.org/#/c/216733/ <- that's the last one | 08:27 |
| garyk | johnthetubaguy: my concern with that is the whole review process - that features take weeks and months to land | 08:28 |
| johnthetubaguy | garyk: you and me both | 08:28 |
| bauzas | garyk: are we sure it's a good time for discussing that now ? | 08:29 |
| johnthetubaguy | ndipanov: OK, that looks right, just getting confused with what the blueprint is telling me somehow | 08:29 |
| bauzas | garyk: in particular when most of the US guys are just afk | 08:29 |
| garyk | bauzas: :) you crack me up. this talk is takin away valuable review ccyles. | 08:29 |
| bauzas | garyk: but we agreed during last summit to see what's happening with Ironic | 08:30 |
| garyk | bauzas: i was just asking to try and get some clarification. this is a community dicsussion that will take place at some time or another | 08:30 |
| bauzas | garyk: and to reconsider that during the next Summit | 08:30 |
| garyk | bauzas: that is why i was asking as i saw the mail about ironic. | 08:30 |
| bauzas | garyk: okay, my bad then :) | 08:30 |
| johnthetubaguy | garyk: yup, thats cool, its something for the summit at this point, and once mikta opens | 08:30 |
| garyk | what would alanis maoriset say (most prob spelt wrong) | 08:30 |
| bauzas | garyk: so, this is planned to be discussed during Tokyo I guess | 08:30 |
| johnthetubaguy | bauzas: thats my answer, yup | 08:31 |
| garyk | bauzas: yes, i think so | 08:31 |
| *** zhangjn_ has joined #openstack-nova | 08:32 | |
| ndipanov | johnthetubaguy, what do you mean? | 08:32 |
| *** sirushti has joined #openstack-nova | 08:32 | |
| johnthetubaguy | ndipanov: sorry, what do I mean about which bit? | 08:33 |
| *** jcoufal has joined #openstack-nova | 08:34 | |
| ndipanov | the blueprint is confusing | 08:34 |
| *** zhangjn has quit IRC | 08:34 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Scheduler RPC API select_dests() https://review.openstack.org/199205 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler https://review.openstack.org/188487 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Prepare filters for using RequestSpec object https://review.openstack.org/197912 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify left filters for RequestSpec https://review.openstack.org/202678 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify conductor to use RequestSpec object https://review.openstack.org/203078 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify metric-related filters for RequestSpec https://review.openstack.org/202676 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify NUMA, PCI and num_instances filters for RequestSpec https://review.openstack.org/202677 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec in the ChanceScheduler https://review.openstack.org/199051 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Aggregate filters for RequestSpec https://review.openstack.org/202674 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify default filters for RequestSpec https://review.openstack.org/202675 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Affinity filter for RequestSpec https://review.openstack.org/202673 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec object in HostManager https://review.openstack.org/191251 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager https://review.openstack.org/191250 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Remove reservation_id from the logs when a schedule fails https://review.openstack.org/217798 | 08:34 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix RequestSpec.instance_group hydratation https://review.openstack.org/219182 | 08:34 |
| ndipanov | nice | 08:35 |
| johnthetubaguy | ndipanov: oh, the release status and links are confusing me, I just added a note saying the last patch we need to merge, so I don't get confused next time | 08:35 |
| johnthetubaguy | ndipanov: I have +Wed that last objects patch, will need some recheck I am guessing, but not totally sure | 08:35 |
| johnthetubaguy | ndipanov: just being sure about which patches we wait for before we tag the release | 08:36 |
| johnthetubaguy | ndipanov: I think its these patches I found: https://review.openstack.org/#/c/214713/7 | 08:37 |
| ndipanov | yeah those -1s are all from some weird pbr issue I think | 08:37 |
| *** lyanchih has quit IRC | 08:37 | |
| ndipanov | yeah so those are the ones that will now fix it for evacuate and live migrate (soon) | 08:37 |
| ndipanov | but they rely on resource tracking doing the right thing | 08:38 |
| ndipanov | which is the other branch | 08:38 |
| ndipanov | the reason they are not dependant on eachother | 08:38 |
| openstackgerrit | lalit dagre proposed openstack/nova: Flavor object which support both DB with active migration https://review.openstack.org/213041 | 08:38 |
| openstackgerrit | lalit dagre proposed openstack/nova: Creates flavor* tables in API database https://review.openstack.org/201606 | 08:38 |
| ndipanov | is that they don't conflict code-wise | 08:38 |
| ndipanov | i.e - we don't test how resource tracking works in the compute manager business logic - just that it gets called | 08:39 |
| ndipanov | if that makes sense | 08:39 |
| ndipanov | johnthetubaguy, ^ | 08:39 |
| johnthetubaguy | ndipanov: yeah, kinda | 08:39 |
| ndipanov | they really should be | 08:39 |
| ndipanov | but I split them up so as to get more reviews | 08:40 |
| ndipanov | :) | 08:40 |
| *** apuimedo has joined #openstack-nova | 08:40 | |
| *** dims has joined #openstack-nova | 08:42 | |
| *** shangxdy has quit IRC | 08:43 | |
| sudipto | ndipanov, i have a question around the memory bw filter work that we plan to do in the next release. (and a subsequent one around the NUMA placement logic currently) | 08:45 |
| *** IanGovett has joined #openstack-nova | 08:45 | |
| ndipanov | sudipto, yes :) | 08:45 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Add some tests for map_dev https://review.openstack.org/215568 | 08:46 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Wait for device to be mapped https://review.openstack.org/215613 | 08:46 |
| openstackgerrit | Alex Xu proposed openstack/nova: Remove the v3 word from help message of api_rate_limit option https://review.openstack.org/219188 | 08:47 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor provider_fw_rule_get_all() https://review.openstack.org/216241 | 08:47 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused provider firewall rules functionality in nova https://review.openstack.org/184027 | 08:47 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove provider_fw_rules tables and associated db api methods https://review.openstack.org/216242 | 08:47 |
| *** lyanchih has joined #openstack-nova | 08:47 | |
| sudipto | ndipanov, the mem bw utilization data can be calculated by dividing the maximum theoretical memory bw on each of the numa nodes and the current usage data on each of those NUMA nodes. If we would like to take a NUMA placement decison based on this metric - would you suggest that we enhance the https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L756 numa_fit_instance_cell method - or would you suggest a different approach? | 08:48 |
| ndipanov | sudipto, hmmm | 08:48 |
| ndipanov | that's a good question | 08:48 |
| *** dims has quit IRC | 08:48 | |
| ndipanov | sudipto, I feel I need to think about it a bit | 08:48 |
| sudipto | ndipanov, ok sure... | 08:49 |
| ndipanov | how about sending a ML post about this | 08:49 |
| ndipanov | and I'll be sure to respond | 08:49 |
| sudipto | ndipanov, sure i can do that. | 08:49 |
| *** derekh has joined #openstack-nova | 08:49 | |
| ndipanov | also outline the usecase a bit | 08:49 |
| ndipanov | as I am not sure I fully grasp it | 08:49 |
| sudipto | ndipanov, sure i will. | 08:49 |
| sudipto | ndipanov, another quick one - if you have a couple of mins... | 08:49 |
| ndipanov | go | 08:50 |
| *** aix has quit IRC | 08:50 | |
| sudipto | ndipanov, https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L724 I feel that this line makes an assumption that each cpu_set on the host corresponds to a VCPU thread on the VM - is that a correct way of looking at it? | 08:52 |
| *** rotbeard has joined #openstack-nova | 08:52 | |
| sudipto | as in there's a 1 on 1 mapping between the cpuset on the host and the number of vcpus requested? | 08:52 |
| ndipanov | sudipto, well that method only runs when CPU pinning was requested | 08:53 |
| ndipanov | so in that case yes (because that's the point) | 08:53 |
| ndipanov | but that's only when flavor has hw:cpu_policy=dedicated | 08:53 |
| *** ociuhandu has quit IRC | 08:54 | |
| *** Marga_ has joined #openstack-nova | 08:56 | |
| *** Marga__ has joined #openstack-nova | 08:57 | |
| *** kiran-r has quit IRC | 08:57 | |
| *** boris-42 has quit IRC | 09:00 | |
| sudipto | ndipanov, ok, there are cpu architectures where the scheduling is done on the core level, and then there can be dynamic SMT threads created/switched as required by the Virtual machines...for example the powerpc architecture, so i was wondering if i could propose something that would take care of that situation... | 09:01 |
| *** Marga_ has quit IRC | 09:01 | |
| *** marzif has joined #openstack-nova | 09:01 | |
| *** yamahata has quit IRC | 09:01 | |
| sudipto | ndipanov, here with the current code, it's taking a decision based on the scheduling done at the hw thread level - which is perfectly fine. | 09:02 |
| *** ndipanov has quit IRC | 09:03 | |
| *** kiran-r has joined #openstack-nova | 09:04 | |
| *** marzif has quit IRC | 09:04 | |
| *** marzif has joined #openstack-nova | 09:05 | |
| *** yuntongjin has quit IRC | 09:06 | |
| *** sirushti has quit IRC | 09:08 | |
| *** sirushti has joined #openstack-nova | 09:11 | |
| *** sahid has joined #openstack-nova | 09:12 | |
| *** ildikov has quit IRC | 09:13 | |
| yonglihe | hi, PCI CI is disabled for accident introduced bug, and will be recover tomorrow , to ensure this CI work correctly, following things will be done: | 09:14 |
| yonglihe | 1. we rolling back to original scripts, that's running for long time | 09:14 |
| yonglihe | 2. second, we will closely monitoring the states to make sure it will running as it did before. jianyu(jyso) will own this. | 09:15 |
| yonglihe | sorry for this trouble, thanks. | 09:15 |
| *** ociuhandu has joined #openstack-nova | 09:16 | |
| *** VW__ has joined #openstack-nova | 09:18 | |
| *** aix has joined #openstack-nova | 09:19 | |
| *** VW_ has quit IRC | 09:22 | |
| *** VW__ has quit IRC | 09:23 | |
| *** lucasagomes is now known as lucas-afk-bank | 09:23 | |
| *** Marga__ has quit IRC | 09:32 | |
| *** Marga_ has joined #openstack-nova | 09:32 | |
| *** chenzeng has joined #openstack-nova | 09:36 | |
| *** Guest92023 has quit IRC | 09:40 | |
| *** lpetrut has joined #openstack-nova | 09:47 | |
| *** asettle has joined #openstack-nova | 09:48 | |
| *** ildikov has joined #openstack-nova | 09:48 | |
| *** Marga_ has quit IRC | 09:52 | |
| *** rajesht has joined #openstack-nova | 09:53 | |
| *** claudiub has joined #openstack-nova | 09:56 | |
| *** coolsvap has quit IRC | 10:01 | |
| *** coolsvap has joined #openstack-nova | 10:02 | |
| *** ndipanov has joined #openstack-nova | 10:03 | |
| *** Guest41206 has quit IRC | 10:03 | |
| *** karimb has quit IRC | 10:04 | |
| *** Sree has joined #openstack-nova | 10:04 | |
| *** chenzeng has quit IRC | 10:04 | |
| *** aix has quit IRC | 10:05 | |
| *** aix has joined #openstack-nova | 10:05 | |
| ndipanov | sudipto, sorry dropped off | 10:06 |
| ndipanov | what were the changes you wanted to make? | 10:06 |
| *** xgsa has quit IRC | 10:06 | |
| *** dave-mccowan has joined #openstack-nova | 10:06 | |
| sudipto | ndipanov, ok, I have not drafted the change as yet, but need a bit of consultancy from you before i do so. | 10:07 |
| *** xgsa has joined #openstack-nova | 10:07 | |
| ndipanov | I didn't get the problem you are trying to solve really | 10:07 |
| sudipto | ok - so let me try an attempt at explaining. | 10:07 |
| ndipanov | :) | 10:08 |
| *** Sree has quit IRC | 10:08 | |
| *** karimb has joined #openstack-nova | 10:09 | |
| sudipto | the current pinning logic is working on the assumption (well i wouldn't call a more widely accepted norm as assumption but just for now) - that the number of cpusets on the host translate to the number of online hardware threads...For example the cpusets reported are 160 - so we assume that there are 160 pcpus available for scheduling. | 10:09 |
| ndipanov | right | 10:10 |
| ndipanov | actually maybe - but go on | 10:11 |
| sudipto | but in powerpc - the scheduling is done on the core , each core can contain upto 8 pcpus, but only one pcpu is kept online - so if we see the virsh capabilities output - we would see only 5 online CPUs per NUMA node as opposed to 40 in case of intel. | 10:11 |
| ndipanov | hmmm | 10:11 |
| openstackgerrit | Cedric Brandily proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere https://review.openstack.org/186716 | 10:12 |
| *** matrohon has quit IRC | 10:12 | |
| sudipto | the core scheduling allows dynamic allocation of the pcpus - as and when needed. so if we let's say we want to pin a host with 8 vcpus - we can effectively pin it on one single core (that is shown online)... | 10:12 |
| sudipto | but if we keep the scheduling at the threads level - then the number of vcpus must be equal or less than the number of pcpus on the host...that's i think is the philosophy behind the code present in hardware.py (i maybe wrong) | 10:13 |
| ndipanov | ok I'd have to read up on ppc a bit more | 10:14 |
| sudipto | ok let me search for a video that explains this...if that's ok... | 10:14 |
| ndipanov | sure | 10:14 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Libvirt: Fix SMBFS volume attach race condition https://review.openstack.org/187619 | 10:15 |
| *** claudiub has quit IRC | 10:18 | |
| sudipto | ndipanov, i will get to better documentation, but here's one http://www.linux-kvm.org/images/7/77/01x09a-KVMPower.pdf slide number 7...that somewhat explains it. | 10:22 |
| sudipto | and slide 9 | 10:23 |
| *** marzif has quit IRC | 10:24 | |
| *** _gryf has quit IRC | 10:26 | |
| ndipanov | sudipto, hmmm | 10:29 |
| *** Sree has joined #openstack-nova | 10:30 | |
| *** dims has joined #openstack-nova | 10:30 | |
| *** Sree has quit IRC | 10:30 | |
| *** karimb has quit IRC | 10:31 | |
| *** Sree has joined #openstack-nova | 10:31 | |
| *** karimb has joined #openstack-nova | 10:32 | |
| ndipanov | sudipto, that's a good resource | 10:32 |
| sudipto | ndipanov, ok yeah :) | 10:33 |
| *** VikasC has joined #openstack-nova | 10:33 | |
| *** apuimedo is now known as apuimedo|lunch | 10:34 | |
| *** dims has quit IRC | 10:36 | |
| openstackgerrit | garyk proposed openstack/nova: Revert "Remove references to suds" https://review.openstack.org/219225 | 10:39 |
| ndipanov | sudipto, so I'd say you should definitely raise this on the ML | 10:40 |
| *** gszasz has joined #openstack-nova | 10:40 | |
| ndipanov | there seem to be several approaches we could take here | 10:41 |
| *** _gryf has joined #openstack-nova | 10:42 | |
| *** h00327910__ has quit IRC | 10:43 | |
| *** lyanchih has quit IRC | 10:43 | |
| *** marzif has joined #openstack-nova | 10:44 | |
| sudipto | ndipanov, sure...i will work on a draft idea of solving this as well, will that be ok? | 10:44 |
| *** ociuhandu_ has joined #openstack-nova | 10:44 | |
| *** karimb has quit IRC | 10:44 | |
| *** andreykurilin__ has joined #openstack-nova | 10:45 | |
| *** ociuhandu has quit IRC | 10:45 | |
| *** ociuhandu_ is now known as ociuhandu | 10:45 | |
| *** qianli has quit IRC | 10:47 | |
| *** zhangjn_ has quit IRC | 10:47 | |
| *** qianli has joined #openstack-nova | 10:48 | |
| *** ZZelle has joined #openstack-nova | 10:48 | |
| *** _gryf has quit IRC | 10:49 | |
| garyk | johnthetubaguy: rie | 10:49 |
| garyk | johnthetubaguy: any chance that you or one of the other cores please look at https://review.openstack.org/219225 . this patch really fills the log files with all kinds of junk from the suds module | 10:50 |
| garyk | it is terribly difficult to debug | 10:50 |
| garyk | its about 80G for each tempest run | 10:50 |
| andreykurilin__ | sdague, alex_xu, melwitt: hi! I'll be so happy, if you spend a bit of your time for review update of novaclient spec( https://review.openstack.org/#/c/211206/ ). The change is small(+52, -45) and includes only updates related to "latest" version and several typos. The patch was published 3 weeks ago and no one reviewed it yet. :( | 10:50 |
| *** amann has joined #openstack-nova | 10:51 | |
| *** _gryf has joined #openstack-nova | 10:52 | |
| andreykurilin__ | oomichi alaski : ^ | 10:52 |
| *** stefan_amann has quit IRC | 10:54 | |
| *** smatzek has joined #openstack-nova | 10:56 | |
| *** Sree has quit IRC | 10:58 | |
| *** Sree has joined #openstack-nova | 10:59 | |
| *** deepthi has quit IRC | 11:01 | |
| openstackgerrit | Noel Nelson Dsouza proposed openstack/nova: Delete flavor should raise error, if it is inuse https://review.openstack.org/217674 | 11:02 |
| *** marzif has quit IRC | 11:02 | |
| *** Sree has quit IRC | 11:03 | |
| *** eglynn__ has quit IRC | 11:04 | |
| *** karimb has joined #openstack-nova | 11:05 | |
| sdague | andreykurilin__: honestly, I was waiting until we get past freeze to go take a look. I'm going to assume it's fine at this point, but we'll just look after | 11:08 |
| *** haomaiwang has quit IRC | 11:08 | |
| andreykurilin__ | sdague: ok, I'll wait | 11:09 |
| sdague | garyk: info is fine, or should it be pumped up to warn? | 11:09 |
| *** eglynn has joined #openstack-nova | 11:10 | |
| garyk | sdague: i think that the INFO is ok. it is kind of what we were using till the change was done. | 11:11 |
| *** lucas-afk-bank is now known as lucasagomes | 11:11 | |
| garyk | my real concern is that when people upgrade it is going to be very painful for them if this is not reverted | 11:11 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 11:12 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V vTPM Devices https://review.openstack.org/218642 | 11:12 |
| sdague | garyk: yep, I'm fine with it, +2 | 11:12 |
| *** slagle has joined #openstack-nova | 11:12 | |
| garyk | sdague: thanks! much appreciated | 11:12 |
| *** claudiub has joined #openstack-nova | 11:14 | |
| *** jaypipes has joined #openstack-nova | 11:14 | |
| *** sayali has quit IRC | 11:15 | |
| *** karimb has quit IRC | 11:16 | |
| *** sayali has joined #openstack-nova | 11:17 | |
| *** alex_klimov has quit IRC | 11:17 | |
| *** pbrooko has quit IRC | 11:18 | |
| *** shoutm has quit IRC | 11:19 | |
| *** asettle has quit IRC | 11:20 | |
| *** dims has joined #openstack-nova | 11:25 | |
| jaypipes | morning all. | 11:26 |
| ndipanov | halloooo jay | 11:27 |
| jaypipes | ndipanov: dobro jutro! | 11:28 |
| *** dims has quit IRC | 11:29 | |
| ndipanov | :) | 11:29 |
| *** thorst has joined #openstack-nova | 11:33 | |
| *** pbrooko has joined #openstack-nova | 11:34 | |
| *** exploreshaifali has joined #openstack-nova | 11:35 | |
| *** Kevin_Zheng has joined #openstack-nova | 11:37 | |
| rajesht | ndipanov, jaypipes: could you please look at https://review.openstack.org/#/c/170118 | 11:44 |
| rajesht | nidpanov, jaypipes: thanks :) | 11:44 |
| jaypipes | rajesht: sure thing. | 11:44 |
| *** sudipto has quit IRC | 11:47 | |
| sdague | fyi, nova patches are 100% blocked right now because of the ironic job | 11:47 |
| sdague | so don't hit +A on anything until we get that job flipped to non voting (should be about 45 minutes to propogate) | 11:48 |
| alex_xu | andreykurilin__: yea, this isn't focus at this point for the nova | 11:48 |
| *** Marga_ has joined #openstack-nova | 11:49 | |
| *** amann has quit IRC | 11:50 | |
| *** alex_klimov has joined #openstack-nova | 11:51 | |
| pkholkin | alaski, oomichi: Good day guys! I want to ask you to make two small reviews, it will take a few minutes! Thanks a lot in advance! https://review.openstack.org/#/c/204082/ https://review.openstack.org/#/c/215658/ | 11:52 |
| openstackgerrit | Cedric Brandily proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere https://review.openstack.org/186716 | 11:52 |
| gibi | sdague: Hi! I had to rebase https://review.openstack.org/#/c/147514/ so I lost your +2, could you please re-add it? | 11:53 |
| jaypipes | sdague: got it. thx for the heads up. | 11:53 |
| *** rfolco has joined #openstack-nova | 11:55 | |
| *** ctrath has joined #openstack-nova | 11:56 | |
| *** diegows has joined #openstack-nova | 11:58 | |
| *** Marga_ has quit IRC | 11:59 | |
| *** pbrooko has quit IRC | 12:00 | |
| *** gmann_ has joined #openstack-nova | 12:00 | |
| alex_xu | nova api meeting is running~ | 12:00 |
| *** lucasagomes is now known as lucas-hungry | 12:01 | |
| *** stefan_amann has joined #openstack-nova | 12:02 | |
| pkholkin | oomichi: Thanks a lot for review! | 12:02 |
| *** Marga_ has joined #openstack-nova | 12:02 | |
| *** bkopilov has quit IRC | 12:02 | |
| openstackgerrit | Cedric Brandily proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere https://review.openstack.org/186716 | 12:03 |
| *** matrohon has joined #openstack-nova | 12:05 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Remove wsgi functionality https://review.openstack.org/219248 | 12:07 |
| *** mwagner_lap has quit IRC | 12:08 | |
| *** Marga_ has quit IRC | 12:09 | |
| *** hparekh has quit IRC | 12:09 | |
| *** ferest has joined #openstack-nova | 12:10 | |
| *** pbrooko has joined #openstack-nova | 12:10 | |
| *** annegentle has joined #openstack-nova | 12:11 | |
| *** salv-orlando has quit IRC | 12:14 | |
| *** VW_ has joined #openstack-nova | 12:16 | |
| *** ferest has quit IRC | 12:16 | |
| *** VW_ has quit IRC | 12:17 | |
| *** ishant has quit IRC | 12:17 | |
| *** humble__ has quit IRC | 12:17 | |
| *** jcoufal has quit IRC | 12:17 | |
| *** VW_ has joined #openstack-nova | 12:17 | |
| *** gmann__ has joined #openstack-nova | 12:19 | |
| *** gmann_ has quit IRC | 12:20 | |
| *** VW_ has quit IRC | 12:21 | |
| *** baoli has joined #openstack-nova | 12:21 | |
| bauzas | jaypipes: morning | 12:22 |
| *** salv-orlando has joined #openstack-nova | 12:22 | |
| bauzas | jaypipes: so, I made a rebase for https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/request-spec-object,n,z | 12:22 |
| bauzas | jaypipes: but thanks this time to the ironic job, -1... | 12:22 |
| *** salv-orl_ has joined #openstack-nova | 12:23 | |
| bauzas | jaypipes: if you have time for reviewing https://review.openstack.org/#/c/219182/, would be awesome | 12:23 |
| bauzas | sdague: so you made an -infra change for ironic non-voting, right? | 12:24 |
| bauzas | sdague: if so, could you please give me the change ? | 12:24 |
| *** zhangjn has joined #openstack-nova | 12:26 | |
| sdague | bauzas: https://review.openstack.org/219240 | 12:26 |
| bauzas | sdague: ack, thanks | 12:26 |
| *** salv-orlando has quit IRC | 12:26 | |
| bauzas | okay, so propagating | 12:26 |
| bauzas | sdague: I guess either a recheck or a new patch should be fine ? | 12:27 |
| *** zhangjn has quit IRC | 12:27 | |
| sdague | bauzas: hopefully, there is a time delay on it | 12:27 |
| sdague | so I don't know if it's in the clear yet or not | 12:28 |
| bauzas | sdague: okay, will see | 12:28 |
| bauzas | sdague: thanks for handling that | 12:28 |
| sdague | bauzas: no prob | 12:29 |
| bauzas | sdague: I guess the bug is logged, so I can recheck against it? | 12:29 |
| *** aysyd has joined #openstack-nova | 12:30 | |
| sdague | bauzas: yeh | 12:30 |
| *** burgerk has joined #openstack-nova | 12:30 | |
| bauzas | sdague: okay, will figure out | 12:30 |
| *** dprince has joined #openstack-nova | 12:31 | |
| bauzas | gotcha https://bugs.launchpad.net/diskimage-builder/+bug/1490864 | 12:31 |
| openstack | Launchpad bug 1490864 in diskimage-builder "Ubuntu mirror file renaming broken "ubuntu" element" [Undecided,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 12:31 |
| *** venkat_p has quit IRC | 12:31 | |
| *** dims has joined #openstack-nova | 12:33 | |
| *** MarcF has quit IRC | 12:33 | |
| *** MarcF has joined #openstack-nova | 12:33 | |
| *** salv-orlando has joined #openstack-nova | 12:33 | |
| sdague | yep that's the one | 12:34 |
| jaypipes | bauzas: yeah, I'll get to that once the gate clears. | 12:36 |
| bauzas | jaypipes: yup, agreed | 12:36 |
| *** VikasC has quit IRC | 12:36 | |
| *** salv-orl_ has quit IRC | 12:37 | |
| *** zhangjn has joined #openstack-nova | 12:38 | |
| *** jcoufal has joined #openstack-nova | 12:40 | |
| *** lucas-hungry is now known as lucasagomes | 12:40 | |
| *** raildo is now known as raildo-afk | 12:40 | |
| *** MarcF has quit IRC | 12:42 | |
| *** Sree has joined #openstack-nova | 12:44 | |
| *** shuquan has joined #openstack-nova | 12:44 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: api: deprecate the api v2 extension configuration https://review.openstack.org/214601 | 12:46 |
| *** dprince has quit IRC | 12:46 | |
| *** ferest has joined #openstack-nova | 12:46 | |
| *** dprince has joined #openstack-nova | 12:48 | |
| gmann__ | oomichi: on master file structure is changed for v3 cleanup | 12:48 |
| *** haomaiwang has joined #openstack-nova | 12:48 | |
| sdague | we should be back in the clear on approving things | 12:48 |
| *** shuquan has quit IRC | 12:49 | |
| alex_xu | edleafe: thanks for the update! | 12:49 |
| gmann__ | oomichi: sdague : that's why cherry pick did not work on https://review.openstack.org/#/c/198944/ | 12:49 |
| *** ctrath has quit IRC | 12:50 | |
| gmann__ | sdague: Thanks | 12:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Scheduler RPC API select_dests() https://review.openstack.org/199205 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify left filters for RequestSpec https://review.openstack.org/202678 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Prepare filters for using RequestSpec object https://review.openstack.org/197912 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify conductor to use RequestSpec object https://review.openstack.org/203078 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify metric-related filters for RequestSpec https://review.openstack.org/202676 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec in the ChanceScheduler https://review.openstack.org/199051 | 12:51 |
| alex_xu | sdague: two seconds review for https://review.openstack.org/#/c/219188/ | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify NUMA, PCI and num_instances filters for RequestSpec https://review.openstack.org/202677 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Aggregate filters for RequestSpec https://review.openstack.org/202674 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify default filters for RequestSpec https://review.openstack.org/202675 | 12:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Affinity filter for RequestSpec https://review.openstack.org/202673 | 12:51 |
| alex_xu | sdague: which we should fix before string freeze | 12:51 |
| oomichi | gmann__: yeah, the commit message is different from the other commits on the stable branch | 12:51 |
| *** ferest has quit IRC | 12:51 | |
| *** derekh has quit IRC | 12:52 | |
| *** diegows has quit IRC | 12:52 | |
| *** MarcF has joined #openstack-nova | 12:55 | |
| *** dustins has joined #openstack-nova | 12:56 | |
| *** romainh has left #openstack-nova | 12:57 | |
| *** zzzeek has joined #openstack-nova | 12:57 | |
| *** oomichi has quit IRC | 12:58 | |
| lbeliveau | sahid: FYI, I updated tox unit tests for CI 216049 | 13:00 |
| *** shuquan has joined #openstack-nova | 13:00 | |
| bauzas | jaypipes: had a question on https://review.openstack.org/#/c/213591/9 | 13:02 |
| bauzas | jaypipes: I don't want to block your series for something quite nitpicking, but would you have time for answering me ? | 13:02 |
| *** oomichi has joined #openstack-nova | 13:03 | |
| *** stefan_amann has quit IRC | 13:03 | |
| *** salv-orlando has quit IRC | 13:04 | |
| *** mjturek1 has joined #openstack-nova | 13:07 | |
| sdague | alex_xu: +A | 13:08 |
| alex_xu | sdague: thanks | 13:08 |
| jaypipes | bauzas: gimme a bit. I will change that patch. FYI, I didn't write it originally (lxsli did, AFAICR) | 13:08 |
| bauzas | jaypipes: sure, no worries | 13:08 |
| bauzas | jaypipes: I mean, that's basically unittest questions | 13:08 |
| bauzas | jaypipes: so I don't want to block for that | 13:09 |
| bauzas | jaypipes: because we can just change that later | 13:09 |
| openstackgerrit | Pranali Deore proposed openstack/nova: Fix order of arguments in assertEqual https://review.openstack.org/219265 | 13:09 |
| bauzas | jaypipes: but if you agree and have time to rework that, that's cool :) | 13:09 |
| openstackgerrit | Pranali Deore proposed openstack/nova: Fix order of arguments in assertEqual https://review.openstack.org/219266 | 13:10 |
| openstackgerrit | Pranali Deore proposed openstack/nova: Fix order of arguments in assertEqual https://review.openstack.org/219267 | 13:10 |
| *** mdrabe has joined #openstack-nova | 13:10 | |
| *** stefan_amann has joined #openstack-nova | 13:11 | |
| jaypipes | bauzas: yeah, I will change them... | 13:11 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Fixed duplicated key '0' in cpu_pinning https://review.openstack.org/219268 | 13:11 |
| jaypipes | bauzas: took me a while to figure out what you were getting at, but I understand now. | 13:11 |
| bauzas | jaypipes: yeah sorry... | 13:11 |
| jaypipes | it's ok, my friend :) | 13:11 |
| openstackgerrit | Pranali Deore proposed openstack/nova: Fix order of arguments in assertEqual https://review.openstack.org/219269 | 13:12 |
| openstackgerrit | Pranali Deore proposed openstack/nova: Fix order of arguments in assertEqual https://review.openstack.org/219270 | 13:13 |
| *** gmann__ has quit IRC | 13:14 | |
| alex_xu | edleafe: I added comment on https://review.openstack.org/#/c/217727/ | 13:14 |
| *** ccarmack has joined #openstack-nova | 13:14 | |
| alex_xu | edleafe: just a small thing need to fix | 13:14 |
| *** VW_ has joined #openstack-nova | 13:16 | |
| *** VW_ has quit IRC | 13:17 | |
| *** VW_ has joined #openstack-nova | 13:17 | |
| *** burgerk has quit IRC | 13:17 | |
| *** jichen has quit IRC | 13:19 | |
| *** sudipto has joined #openstack-nova | 13:23 | |
| edleafe | alex_xu: thanks - I'll look at it in a little while | 13:24 |
| alex_xu | edleafe: thanks | 13:24 |
| *** mriedem has joined #openstack-nova | 13:24 | |
| *** changbl has quit IRC | 13:28 | |
| jwcroppe_zzz | jaypipes: morning... you around? | 13:29 |
| *** mtanino has joined #openstack-nova | 13:31 | |
| jwcroppe_zzz | jaypipes: whenever you get a spare cycle, would like you feedback on https://review.openstack.org/#/c/219153/ (metrics related) ... ran into this pesky issue, but it was a pretty simple fix | 13:31 |
| *** shyu has quit IRC | 13:32 | |
| jaypipes | jwcroppe_zzz: will do! as soon as the gate winds down a bit. | 13:32 |
| *** jwcroppe_zzz is now known as jwcroppe | 13:32 | |
| jwcroppe | jaypipes: thanks | 13:32 |
| jaypipes | np | 13:32 |
| bauzas | jaypipes: FWIW, I think the gate is much better now | 13:33 |
| bauzas | jaypipes: the ironic job is non-voting now as per http://status.openstack.org/zuul/ | 13:33 |
| bauzas | jaypipes: the cells job is in a better shape thanks to a couple of alaski's changes | 13:34 |
| mriedem | bauzas: i assume grenade is still borked | 13:34 |
| mriedem | until https://review.openstack.org/#/c/219076/ is merged | 13:34 |
| bauzas | mriedem: well, I got a couple of rechecks and new runs, and it seems it's pretty stablre | 13:34 |
| jaypipes | bauzas: k, thanks for the heads up. | 13:35 |
| mriedem | yeah logstash is showing things passing, i'm surprised | 13:35 |
| bauzas | what's pretty cool with having a long chain is that you become a weather speaker | 13:35 |
| bauzas | (not sure if it's the right word) | 13:35 |
| mriedem | weather speaker certainly sounds more mystical than joe the weather man on your local news | 13:36 |
| bauzas | yeah, was thinking of an anchorman :) | 13:36 |
| bauzas | so, I guess 'weather anchorman" is better? | 13:37 |
| *** tpeoples has joined #openstack-nova | 13:37 | |
| bauzas | anyway | 13:37 |
| bauzas | mriedem: I guess you know https://graphite.openstack.org ? | 13:37 |
| mriedem | the anchorman is the guy that sits in front and does the news | 13:38 |
| mriedem | the weather man is the guy that just does the weather | 13:38 |
| mriedem | and they have banter | 13:38 |
| *** mwagner_lap has joined #openstack-nova | 13:38 | |
| *** Piet has quit IRC | 13:38 | |
| mriedem | don't forget about pat on sports | 13:38 |
| *** shuquan has quit IRC | 13:38 | |
| bauzas | hah | 13:38 |
| bauzas | I only know Ron Burgundy | 13:38 |
| dansmith | it's all you need to know | 13:38 |
| mriedem | right | 13:39 |
| *** vladikr has joined #openstack-nova | 13:40 | |
| *** kiran-r has quit IRC | 13:40 | |
| *** bfic has joined #openstack-nova | 13:41 | |
| *** kiran-r has joined #openstack-nova | 13:41 | |
| gibi | dansmith: hi! You had comments in https://review.openstack.org/#/c/147514/, sdague has already put +2 on it. Could you please check it and approve it if is seems OK to you? | 13:41 |
| *** kiran-r has quit IRC | 13:41 | |
| dansmith | gibi: that can go in any time and I've focusing on things that can't this week | 13:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: Fixed incorrect keys in cpu_pinning https://review.openstack.org/219268 | 13:42 |
| *** ctrath has joined #openstack-nova | 13:43 | |
| gibi | dansmith: OK. Then I will ping you next week. | 13:43 |
| gibi | dansmith: thanks. | 13:43 |
| *** eharney has joined #openstack-nova | 13:44 | |
| *** shuquan has joined #openstack-nova | 13:44 | |
| *** rushil has joined #openstack-nova | 13:47 | |
| *** edmondsw has joined #openstack-nova | 13:47 | |
| *** annegentle has quit IRC | 13:49 | |
| sdague | alex_xu / oomichi / gmann - https://etherpad.openstack.org/p/liberty-nova-api-testing | 13:52 |
| mriedem | sdague: does anything about the microversion support in novaclient cause a major version bump to 3.x? | 13:52 |
| sdague | how does that look? | 13:52 |
| *** smatzek has quit IRC | 13:52 | |
| sdague | mriedem: I don't think so | 13:52 |
| * alex_xu is checking | 13:52 | |
| mriedem | ok, i didn't think so either, since you have to opt in | 13:52 |
| mriedem | there are just a lot of novaclient changes since 2.26.0 | 13:52 |
| mriedem | hmm https://review.openstack.org/#/c/198257/ | 13:53 |
| sdague | yeh, I'd make it a 2.27 | 13:53 |
| mriedem | that removes a thing that heat used to use | 13:53 |
| mriedem | so kilo heat + 2.27 novaclient wouldn't work, let me check g-r on stable/kilo | 13:54 |
| mriedem | python-novaclient>=2.22.0,<2.24.0 | 13:54 |
| mriedem | so i guess we're ok | 13:54 |
| *** bkopilov has joined #openstack-nova | 13:56 | |
| *** zul has quit IRC | 13:56 | |
| *** rlrossit has joined #openstack-nova | 13:56 | |
| *** alex_klimov has quit IRC | 13:56 | |
| *** zul has joined #openstack-nova | 13:56 | |
| *** pbrooko has quit IRC | 13:58 | |
| *** pbrooko has joined #openstack-nova | 13:58 | |
| jroll | sdague: mornin', looking at this ironic gate thing | 13:58 |
| jroll | sdague: we have a job if you s/pxe_ssh/pxe_ipa/ that uses a prebuilt ramdisk, is stable, and currently votes on ironic. can we switch nova to use that instead of making pxe_ssh non-voting? | 13:59 |
| mriedem | johnthetubaguy: novaclient 2.27.0 release requested: https://review.openstack.org/#/c/219283/ | 14:00 |
| *** artom has joined #openstack-nova | 14:00 | |
| *** thangp has joined #openstack-nova | 14:00 | |
| *** mriedem is now known as mriedem_meetings | 14:01 | |
| *** artom has quit IRC | 14:01 | |
| *** pbrooko has quit IRC | 14:02 | |
| *** Kennan has joined #openstack-nova | 14:03 | |
| *** pbrooko has joined #openstack-nova | 14:03 | |
| jroll | sdague: anyway, opinions welcome here https://review.openstack.org/#/c/219285/ | 14:03 |
| *** Kennan2 has quit IRC | 14:03 | |
| alex_xu | sdague: looks ok for me. not familar with tempest job, but I will work with gmann | 14:04 |
| sdague | jroll: 1) we already made it non voting, because it was ruining the world | 14:05 |
| jroll | sdague: yep, found that bit. | 14:06 |
| *** burgerk has joined #openstack-nova | 14:06 | |
| sdague | 2) before making another ironic job voting on nova I'd really like to see a test plan about what's going into it. The other job had a high failure rate, and didn't actually test basically any of what nova wanted tested | 14:06 |
| sdague | and my complaints about that bit fell on deaf ears | 14:07 |
| dansmith | jroll: did the coverage ever get addressed? | 14:08 |
| sdague | jroll: also - http://graphite.openstack.org/render/?from=-200hours&height=500&until=now&width=800&bgcolor=ffffff&fgcolor=000000&yMax=100&yMin=0&target=color(alias(movingAverage(asPercent(stats.zuul.pipeline.gate.job.gate-tempest-dsvm-ironic-pxe_ipa.FAILURE,sum(stats.zuul.pipeline.gate.job.gate-tempest-dsvm-ironic-pxe_ipa.{SUCCESS,FAILURE})),%2712hours%27),%20%27gate-tempest-dsvm-ironic-pxe_ipa%27),%27orange%27) | 14:08 |
| sdague | is not what I consider stable | 14:08 |
| sdague | that's failure rates | 14:09 |
| sdague | in the gate queue | 14:09 |
| *** smatzek has joined #openstack-nova | 14:09 | |
| *** RuiChen has quit IRC | 14:09 | |
| *** RuiChen has joined #openstack-nova | 14:09 | |
| jroll | dansmith: sdague we have an experimental job for full tempest that just needs some poking to skip some tests we don't support etc, need to catch up with mtreinish about it | 14:09 |
| dansmith | okay | 14:10 |
| mtreinish | jroll: do we have logs from a recent run? | 14:10 |
| sdague | jroll: right, so that's the thing that should be worked on | 14:10 |
| jroll | sdague: and yeah our gate has not been awesome lately in general | 14:10 |
| dansmith | sdague: agreed, which is why I brought it up | 14:10 |
| jroll | yeah I agree as well | 14:10 |
| sdague | jroll: right, and the problem is that in the current job definition that generates a lot of collateral damage | 14:10 |
| sdague | which is just not ok | 14:11 |
| jroll | we can deal with non-voting for now, it's just painful because we've had many instances in the past with nova approving ironic driver changes that don't pass ironic jobs | 14:11 |
| sdague | jroll: so I hear people say that | 14:11 |
| jroll | yeah. I hear ya. | 14:11 |
| sdague | and then I ask for patch lists | 14:11 |
| *** artom has joined #openstack-nova | 14:11 | |
| sdague | and no one provides any evidence that it has happened recently | 14:11 |
| jroll | we haven't had a non-voting job recently :P | 14:11 |
| sdague | you have a job that's nearly as good as non voting, it only boots one guest | 14:12 |
| sdague | and it doesn't even test the nova api, so it's not possible to catch the kind of issues you are concerned about | 14:12 |
| jroll | sure. but if it 100% breaks the ironic job it doesn't allow the patch to merge. | 14:12 |
| *** dave-mccowan has quit IRC | 14:13 | |
| *** exploreshaifali has quit IRC | 14:13 | |
| jroll | mtreinish: no, I can trigger one | 14:13 |
| mtreinish | jroll: ok cool | 14:14 |
| jroll | mtreinish: https://review.openstack.org/#/c/219225/ | 14:14 |
| jroll | triggered there | 14:14 |
| *** angdraug has joined #openstack-nova | 14:15 | |
| *** andreykurilin__ has quit IRC | 14:15 | |
| jroll | sdague: so I clearly can't make y'all make this voting, so I'll leave it up to your team | 14:16 |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 14:16 |
| jroll | I know what the goal is, the question is what we do in the meantime | 14:16 |
| jroll | and I'm fine with voting or non-voting | 14:16 |
| jroll | I'd still like to land 219285 as we're deprecating the dib bash ramdisk thing, just need to decide whether it should vote or not. | 14:16 |
| *** liusheng has quit IRC | 14:17 | |
| jroll | dansmith: ^ | 14:17 |
| sdague | so, I think we should be non-voting until the good answer is here, because there have been a lot of promisses around a good answer for a while, but because there is incorrect belief that the current job provides any substantial test coverage so it's not been a priority to get the right thing done | 14:17 |
| *** pbrooko has quit IRC | 14:17 | |
| pkholkin | alaski: Thanks a lot for review! | 14:18 |
| *** rushil has quit IRC | 14:18 | |
| *** irenab has quit IRC | 14:18 | |
| sdague | jroll: that patch doesn't do any of the things you think it does in the commit message :) | 14:18 |
| jroll | sdague: be nice, still on first coffee :P | 14:20 |
| sdague | :) | 14:20 |
| *** kumar has joined #openstack-nova | 14:20 | |
| artom | sdague, (and others how handle stable maint), while you're here, https://review.openstack.org/#/c/189889/ and https://review.openstack.org/#/c/189923/ which depends on it | 14:20 |
| artom | Both are for juno, which IIUC is security only at this point... | 14:20 |
| jroll | sdague: you're right though, heh | 14:20 |
| *** amotoki has joined #openstack-nova | 14:20 | |
| sdague | yeh, sorry, I've been digging back out of this hole for the last 4 hrs, and it's day 3 of dealing with 100% breaking gate issues | 14:21 |
| artom | Why the -1 and not just -2 to block it entirely, they're not security related patches... | 14:21 |
| *** takedakn has joined #openstack-nova | 14:21 | |
| jroll | sdague: so I'll fix that, make it non-voting, and then do the actual job we want to do | 14:21 |
| jroll | sdague: (and I was joking about you being mean :P) | 14:21 |
| *** bfic has quit IRC | 14:22 | |
| sdague | jroll: yeh, I'm fine with using the ipa job instead of the ssh one. I think we should keep it non-voting for now though especially as it's changing out jobs, and we always give them break in periods | 14:22 |
| *** lbeliveau has quit IRC | 14:22 | |
| sdague | and, I'd like that to be incentive to actually get the right job pulled together | 14:23 |
| jroll | sdague: yep, sounds good to me | 14:23 |
| *** links has quit IRC | 14:23 | |
| *** alex_klimov has joined #openstack-nova | 14:23 | |
| sdague | we can fast merge anything we miss and break on the ironic side, but I actually will be pretty surprised if we merge something that that test job would have caught | 14:23 |
| jroll | agree, I hope that's true | 14:24 |
| jroll | I'll try to remember to send a mail about it so people are sure to check the job results | 14:24 |
| *** annegentle has joined #openstack-nova | 14:25 | |
| *** alejandrito has joined #openstack-nova | 14:26 | |
| *** gszasz has quit IRC | 14:27 | |
| *** alex_klimov has quit IRC | 14:27 | |
| *** takedakn has quit IRC | 14:28 | |
| *** alex_klimov has joined #openstack-nova | 14:28 | |
| *** dustins_ has joined #openstack-nova | 14:28 | |
| *** dave-mccowan has joined #openstack-nova | 14:28 | |
| *** mdrabe has quit IRC | 14:29 | |
| *** shuquan has quit IRC | 14:29 | |
| *** matrohon has quit IRC | 14:30 | |
| *** dustins has quit IRC | 14:31 | |
| *** electrocucaracha has joined #openstack-nova | 14:34 | |
| *** kumar has quit IRC | 14:35 | |
| *** Kevin_Zheng has quit IRC | 14:35 | |
| *** garyk has quit IRC | 14:35 | |
| *** gszasz has joined #openstack-nova | 14:37 | |
| *** afazekas__ has quit IRC | 14:41 | |
| *** Nikolay_St has quit IRC | 14:41 | |
| *** meghal has joined #openstack-nova | 14:41 | |
| *** boris-42 has joined #openstack-nova | 14:42 | |
| *** cfriesen has joined #openstack-nova | 14:42 | |
| *** meghal1 has joined #openstack-nova | 14:43 | |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot https://review.openstack.org/209581 | 14:45 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova: Hyper-V vTPM Devices https://review.openstack.org/218642 | 14:45 |
| *** meghal has quit IRC | 14:46 | |
| *** Murali has quit IRC | 14:46 | |
| *** mdrabe has joined #openstack-nova | 14:46 | |
| *** pbrooko has joined #openstack-nova | 14:48 | |
| *** shoutm has joined #openstack-nova | 14:49 | |
| *** rushil has joined #openstack-nova | 14:50 | |
| *** dave-mccowan has quit IRC | 14:51 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Implements attach_interface and detach_interface method https://review.openstack.org/214493 | 14:51 |
| *** jlanoux has quit IRC | 14:52 | |
| *** nkrinner has quit IRC | 14:53 | |
| *** armax has joined #openstack-nova | 14:53 | |
| *** meghal1 has quit IRC | 14:53 | |
| *** irenab has joined #openstack-nova | 14:54 | |
| *** ijuwang has joined #openstack-nova | 14:55 | |
| *** tonytan4ever has joined #openstack-nova | 14:56 | |
| *** links has joined #openstack-nova | 14:57 | |
| *** zhangjn has quit IRC | 14:57 | |
| *** electrocucaracha has quit IRC | 14:57 | |
| mriedem_meetings | vmware ci seems to be gone | 14:57 |
| *** dave-mccowan has joined #openstack-nova | 14:58 | |
| *** jistr is now known as jistr|call | 14:58 | |
| *** angdraug_ has joined #openstack-nova | 14:58 | |
| *** erlon has quit IRC | 14:58 | |
| *** thangp has quit IRC | 14:58 | |
| *** hemna has joined #openstack-nova | 14:59 | |
| openstackgerrit | Tristan Cacqueray proposed openstack/nova: Delete orphaned instance files from compute nodes https://review.openstack.org/219299 | 15:00 |
| *** angdraug has quit IRC | 15:00 | |
| *** jichen has joined #openstack-nova | 15:02 | |
| *** pbrooko has quit IRC | 15:02 | |
| *** electrocucaracha has joined #openstack-nova | 15:04 | |
| *** diogogmt has joined #openstack-nova | 15:04 | |
| *** emagana has joined #openstack-nova | 15:06 | |
| *** sudipto has quit IRC | 15:07 | |
| *** diogogmt has quit IRC | 15:07 | |
| *** MarcF has quit IRC | 15:08 | |
| *** Marga_ has joined #openstack-nova | 15:08 | |
| *** matrohon has joined #openstack-nova | 15:09 | |
| *** diogogmt has joined #openstack-nova | 15:09 | |
| *** burgerk has quit IRC | 15:09 | |
| *** shoutm has quit IRC | 15:10 | |
| *** pbrooko has joined #openstack-nova | 15:11 | |
| *** diogogmt has quit IRC | 15:11 | |
| *** vivekd has joined #openstack-nova | 15:12 | |
| *** alexschm has quit IRC | 15:13 | |
| *** salv-orlando has joined #openstack-nova | 15:13 | |
| *** thangp has joined #openstack-nova | 15:14 | |
| *** gmann has quit IRC | 15:14 | |
| *** yamahata has joined #openstack-nova | 15:15 | |
| openstackgerrit | Alexis Lee proposed openstack/nova: Wait for device to be mapped https://review.openstack.org/215613 | 15:16 |
| *** links has quit IRC | 15:16 | |
| *** dims has quit IRC | 15:17 | |
| *** salv-orlando has quit IRC | 15:18 | |
| *** rajesht has quit IRC | 15:19 | |
| *** sayali has quit IRC | 15:19 | |
| *** salv-orlando has joined #openstack-nova | 15:19 | |
| *** MarcF has joined #openstack-nova | 15:21 | |
| *** diogogmt has joined #openstack-nova | 15:21 | |
| *** sayali has joined #openstack-nova | 15:21 | |
| *** mjturek1 has quit IRC | 15:22 | |
| *** annegentle has quit IRC | 15:23 | |
| *** annegentle has joined #openstack-nova | 15:23 | |
| *** diogogmt has quit IRC | 15:24 | |
| *** moshele has quit IRC | 15:24 | |
| *** gmann has joined #openstack-nova | 15:24 | |
| *** Marga_ has quit IRC | 15:25 | |
| *** garyk has joined #openstack-nova | 15:25 | |
| *** Marga_ has joined #openstack-nova | 15:25 | |
| *** salv-orlando has quit IRC | 15:26 | |
| *** jlanoux has joined #openstack-nova | 15:27 | |
| *** haomaiwang has quit IRC | 15:27 | |
| *** vladikr has quit IRC | 15:27 | |
| *** tojuvone has quit IRC | 15:28 | |
| mtreinish | jroll: so the full job is hitting the same dib thing: http://logs.openstack.org/57/209957/3/experimental/gate-tempest-dsvm-ironic-pxe_ssh-full-nv/413de78//logs/devstacklog.txt.gz#_2015-09-01_15_19_50_970 | 15:30 |
| jroll | damn | 15:31 |
| jroll | lol | 15:31 |
| *** wanghua has joined #openstack-nova | 15:32 | |
| *** gszasz has quit IRC | 15:33 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Make get_pagination_params respect osapi_max_limit https://review.openstack.org/179947 | 15:34 |
| mtreinish | heh, I guess it makes sense since it's doing the same devstack setup | 15:34 |
| *** Marga_ has quit IRC | 15:35 | |
| jroll | mtreinish: yeah, I can get that flipped around later | 15:35 |
| *** salv-orlando has joined #openstack-nova | 15:38 | |
| *** burgerk has joined #openstack-nova | 15:41 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add persistence to the RequestSpec object https://review.openstack.org/211753 | 15:43 |
| *** dustins_ has quit IRC | 15:43 | |
| lxsli | dammit I just rebased on that | 15:46 |
| alaski | I had a few conflicts I needed to fix | 15:47 |
| lxsli | gerrit life | 15:47 |
| openstackgerrit | Alexis Lee proposed openstack/nova: WIP check destination on migrations https://review.openstack.org/212510 | 15:49 |
| *** j_king has left #openstack-nova | 15:50 | |
| *** mriedem_meetings is now known as mriedem | 15:51 | |
| *** j_king has joined #openstack-nova | 15:52 | |
| j_king | dansmith garyk mriedem_meetings wrt https://review.openstack.org/#/c/210126/ would you recommend starting a new patch? I have a few q's wrt feedback so far if anyone has a moment | 15:52 |
| *** dustins has joined #openstack-nova | 15:54 | |
| mriedem | j_king: why would you start with a new change? | 15:54 |
| mriedem | j_king: if you have questions i have some time, | 15:57 |
| j_king | mriedem: it sounds like dansmith was suggesting that we can wait for events in the driver. | 15:57 |
| *** eglynn has quit IRC | 15:57 | |
| mriedem | j_king: you can, you're just not doing it today | 15:57 |
| mriedem | j_king: see https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4512 | 15:57 |
| dansmith | j_king: s/can/can't/ | 15:57 |
| mriedem | j_king: for one thing, the wait_for_instance_event context in your patch isn't actually waiting for anything | 15:58 |
| mriedem | whereas in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4512 it's calling plug_vifs and then waiting on that | 15:58 |
| *** vladikr has joined #openstack-nova | 15:59 | |
| mriedem | dansmith: said "Presumably the actual port delete is happening in compute manager, which means you can't do this here." | 15:59 |
| *** alias015_ has joined #openstack-nova | 15:59 | |
| mriedem | i think the port delete happens in the call to _unplug_vifs here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L837 | 16:00 |
| *** zhenguo has quit IRC | 16:00 | |
| mriedem | or is triggered from that | 16:00 |
| *** ferest has joined #openstack-nova | 16:00 | |
| *** skylerberg has joined #openstack-nova | 16:00 | |
| *** eharney has quit IRC | 16:01 | |
| *** vilobhmm1 has joined #openstack-nova | 16:01 | |
| mriedem | i think i could actually push a change that does what i'm thinking of and vladikr was pointing out | 16:01 |
| *** alias015___ has joined #openstack-nova | 16:01 | |
| *** ivasev has joined #openstack-nova | 16:02 | |
| *** alias015_ has quit IRC | 16:04 | |
| *** baoli has quit IRC | 16:04 | |
| *** baoli has joined #openstack-nova | 16:05 | |
| *** ferest has quit IRC | 16:05 | |
| *** dims has joined #openstack-nova | 16:05 | |
| *** ferest has joined #openstack-nova | 16:06 | |
| *** browne has joined #openstack-nova | 16:06 | |
| alias015___ | Hi all, if I'd like to parser manually http://169.254.169.254 , where I can find a good doc about it? | 16:07 |
| *** jichen has quit IRC | 16:07 | |
| mriedem | alias015___: good python urlparse ? | 16:07 |
| mriedem | 16:07 | |
| j_king | mriedem: I think I'm starting to understand the suggestions here. So we've seen stray ports being created for other operations than just rebuild which is why, iirc, we created this patch to target delete. Presently it doesn't do much other than log and we have to manually clean up the ports when we see this happen | 16:07 |
| j_king | and it usually only happens when we do a large number of such operations since it's kind of racey | 16:07 |
| mriedem | what other operations? migration? | 16:08 |
| *** stefan_amann has quit IRC | 16:08 | |
| mriedem | details are kind of important here | 16:08 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Make rebuild/evacuate use resource claims https://review.openstack.org/214770 | 16:08 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: compute: Make rebuild consider the node that was passed in https://review.openstack.org/214713 | 16:08 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: compute: Make rebuild_instance RPC take additional args https://review.openstack.org/200485 | 16:08 |
| mriedem | j_king: and for rebuild, you're saying it only happens when doing like multiple rebuilds concurrently? i thought someone said the other day you could recreate it if you rebuilt an instance with several ports attached? | 16:09 |
| *** changbl has joined #openstack-nova | 16:10 | |
| *** baoli has quit IRC | 16:10 | |
| *** alias015___ has quit IRC | 16:11 | |
| j_king | mriedem: I think jordan tardif's comment on patchset 7 sums it up. at our scale neutron isn't updating as fast as nova can destroy/create instances | 16:11 |
| *** Sukhdev has joined #openstack-nova | 16:11 | |
| mriedem | sc68cal: beagles: is this orphaned port from rebuild thing something you've heard of ^? | 16:11 |
| j_king | s/updating/doing port syncs and updates/ | 16:11 |
| *** burgerk has quit IRC | 16:11 | |
| ndipanov | jaypipes, if you happen to have a spare moment - this one is ahead of one that has 2x +2 https://review.openstack.org/#/c/218385/ | 16:11 |
| j_king | so we presently patch neutron to log a warning but it could be better | 16:12 |
| *** Piet has joined #openstack-nova | 16:12 | |
| jwcroppe | bauzas: replied to your comment in https://review.openstack.org/#/c/219153/ ... the periodic task works fine, it's just the caching logic has some poor assumptions with the current implementation | 16:13 |
| mriedem | j_king: from jordan's comment, "The trace you would get back is something about port in use ( been a while since I have seen one of these since we patch nova locally with this commit)." | 16:13 |
| mriedem | j_king: if you had that trace in the bug report it'd be helpful | 16:13 |
| *** ferest has quit IRC | 16:13 | |
| j_king | mriedem: I'll see what I can do about that. | 16:13 |
| jaypipes | ndipanov: on that series, it looks like https://review.openstack.org/#/c/216732/ is the one that is stuck and needs a recheck. | 16:14 |
| jaypipes | ndipanov: I just triggered a recheck on that. | 16:14 |
| jwcroppe | bauzas: the code assumes that the driver.get_host_cpu_stats() always runs 'fast' ... so the easiest change is to just move the timestamp cache *after* the libvirt call (without doing a massive re-write on the entire framework) | 16:14 |
| *** nic has joined #openstack-nova | 16:14 | |
| jwcroppe | jaypipes: ^^^ JFYI | 16:14 |
| mriedem | j_king: but again you're on juno right? | 16:15 |
| *** jaypipes has quit IRC | 16:16 | |
| ndipanov | jaypipes, hmm I thought it was in the gate queue | 16:16 |
| *** eharney has joined #openstack-nova | 16:16 | |
| *** jlanoux has quit IRC | 16:17 | |
| *** burgerk has joined #openstack-nova | 16:20 | |
| mriedem | j_king: i'll see if i can slap something together for what i'm thinking should be done here | 16:20 |
| j_king | mriedem: yeah, but I'm pretty sure the issue is still present. so to clarify what happens in rebuild is that nova tries to re-use a port that neutron thinks is still plugged in. | 16:20 |
| *** tjones has joined #openstack-nova | 16:20 | |
| j_king | i'm working on getting some good logs together for the bug report | 16:21 |
| j_king | s/stray ports/ports in an incorrect state/ | 16:22 |
| j_king | i guess | 16:22 |
| j_king | if that makes sense? | 16:22 |
| mriedem | j_king: it's not clear to me, but i don't know the guts of neutron that well - if the race scenario could be described in the commit message that would be helpful | 16:24 |
| *** Nikolay_St has joined #openstack-nova | 16:25 | |
| mriedem | like, there are orphaned ports, but why? because neutron gets messed up by how fast nova is doing things? how does that affect neutron to cause the issue, etc | 16:25 |
| *** hemna_ has joined #openstack-nova | 16:27 | |
| *** jaypipes has joined #openstack-nova | 16:27 | |
| *** jistr|call has quit IRC | 16:28 | |
| *** tonytan4ever has quit IRC | 16:28 | |
| *** dannywilson has joined #openstack-nova | 16:28 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: api: allow any scheduler hints https://review.openstack.org/217727 | 16:28 |
| mriedem | would be nice to get another core review on this before FF https://review.openstack.org/#/c/192396/ | 16:29 |
| *** dannywilson has quit IRC | 16:30 | |
| *** meghal has joined #openstack-nova | 16:31 | |
| *** alex_klimov has quit IRC | 16:34 | |
| *** eglynn has joined #openstack-nova | 16:34 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Update objects test infrastructure for multiple versions https://review.openstack.org/217818 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance and InstanceList v2.0 objects https://review.openstack.org/217819 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix an issue with NovaObjectRegistry hook https://review.openstack.org/217816 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make the conductor fixture use version manifests https://review.openstack.org/217851 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Refactor Instance tests to use objects.Instance https://review.openstack.org/217817 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Pull out the common bits of Instance into _BaseInstance https://review.openstack.org/217815 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Implement manifest-based backports https://review.openstack.org/217239 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Pull out the common bits of InstanceList into _BaseInstanceList https://review.openstack.org/219337 | 16:35 |
| *** dustins has quit IRC | 16:39 | |
| *** lpetrut has quit IRC | 16:39 | |
| dansmith | ndipanov: so, I was just thinking something about this migration context | 16:40 |
| dansmith | ndipanov: if we have a resize in progress during an upgrade, | 16:40 |
| dansmith | which would be the case if we're using migration to move things off of older compute nodes to new ones, | 16:40 |
| dansmith | the older nodes will not be setting up the context on the instance | 16:40 |
| dansmith | so a bunch of your code on the newer nodes will fail because they can't do instance.migration_context.foo, if migration_context is None | 16:41 |
| *** thedodd has joined #openstack-nova | 16:41 | |
| *** dustins has joined #openstack-nova | 16:41 | |
| *** Apoorva has joined #openstack-nova | 16:41 | |
| *** SourabhP has joined #openstack-nova | 16:43 | |
| *** SourabhP_ has joined #openstack-nova | 16:45 | |
| dansmith | ndipanov: I commented on the RT patch, and also reverted my +2 on the instance hook patch as I found another issue | 16:45 |
| *** aix has quit IRC | 16:45 | |
| *** exploreshaifali has joined #openstack-nova | 16:46 | |
| *** diogogmt has joined #openstack-nova | 16:46 | |
| *** skylerberg has quit IRC | 16:46 | |
| *** SourabhP has quit IRC | 16:48 | |
| *** SourabhP_ is now known as SourabhP | 16:48 | |
| *** david-ly_ has joined #openstack-nova | 16:49 | |
| *** david-lyle has quit IRC | 16:49 | |
| *** tonytan4ever has joined #openstack-nova | 16:50 | |
| *** amotoki has quit IRC | 16:52 | |
| *** tjones has quit IRC | 16:54 | |
| *** VW_ has quit IRC | 16:59 | |
| *** humble_ has joined #openstack-nova | 16:59 | |
| *** VW_ has joined #openstack-nova | 16:59 | |
| *** markus_z has quit IRC | 17:00 | |
| *** meghal has quit IRC | 17:00 | |
| *** Sukhdev has quit IRC | 17:00 | |
| *** lpetrut has joined #openstack-nova | 17:01 | |
| *** meghal has joined #openstack-nova | 17:01 | |
| *** skylerberg has joined #openstack-nova | 17:01 | |
| sahid | 4 | 17:02 |
| *** ZZelle_ has joined #openstack-nova | 17:02 | |
| *** wanghua has quit IRC | 17:02 | |
| *** vilobhmm1 has quit IRC | 17:03 | |
| *** unicell has quit IRC | 17:03 | |
| *** lbeliveau has joined #openstack-nova | 17:03 | |
| *** VW_ has quit IRC | 17:04 | |
| *** VW_ has joined #openstack-nova | 17:04 | |
| *** vilobhmm1 has joined #openstack-nova | 17:04 | |
| *** electrocucaracha has quit IRC | 17:05 | |
| *** vilobhmm1 has quit IRC | 17:06 | |
| *** david-ly_ has quit IRC | 17:07 | |
| *** SourabhP has quit IRC | 17:09 | |
| *** hemna has quit IRC | 17:09 | |
| *** tri2sing has joined #openstack-nova | 17:09 | |
| dansmith | mriedem: why does that patch need to wait? | 17:09 |
| mriedem | dansmith: obj_to_primitive was added to ovo in 0.9.0 | 17:10 |
| mriedem | so we need that min version | 17:10 |
| *** vivekd_ has joined #openstack-nova | 17:10 | |
| mriedem | https://github.com/openstack/oslo.versionedobjects/commit/00c503efbb90f03d00c72dae8baeeab6ae4289fc | 17:10 |
| dansmith | the gate doesn't, but you mean just to be legit? | 17:10 |
| mriedem | right | 17:10 |
| mriedem | my man | 17:10 |
| mriedem | there are third party CIs that apparently don't always pull latest packages | 17:11 |
| mriedem | and we burn them from time to time when we don't bump min required versions | 17:11 |
| *** vivekd has quit IRC | 17:11 | |
| *** vivekd_ is now known as vivekd | 17:11 | |
| *** lpetrut has quit IRC | 17:11 | |
| dansmith | yeah, yeah, okay | 17:11 |
| sc68cal | mriedem: no, haven't heard of that | 17:12 |
| mriedem | dansmith: i didn't want to -1/-2/-W your change | 17:13 |
| dansmith | it's cool, I figure it's not likely merging today anyway | 17:13 |
| mriedem | i'm watching 216968 in zuul though | 17:13 |
| mriedem | not with that attitude :P | 17:14 |
| *** lucasagomes is now known as lucas-dinner | 17:14 | |
| *** sahid has quit IRC | 17:14 | |
| dansmith | heh | 17:14 |
| *** tjones has joined #openstack-nova | 17:17 | |
| *** humble_ has quit IRC | 17:18 | |
| *** tonytan4ever has quit IRC | 17:18 | |
| *** tjones has left #openstack-nova | 17:24 | |
| ndipanov | dansmith, why would it be None - we refresh the instance on every run... | 17:25 |
| *** angdraug_ has quit IRC | 17:25 | |
| *** achanda has joined #openstack-nova | 17:25 | |
| ndipanov | ah you mean older node | 17:25 |
| ndipanov | OK | 17:25 |
| *** aix has joined #openstack-nova | 17:25 | |
| ndipanov | yes you are right we need to make sure it's not None | 17:25 |
| mriedem | i wish there was someone around that knew how libvirt worked with bdms..... | 17:26 |
| dansmith | ndipanov: yeah | 17:26 |
| * j_king going to work on a repro for this race condition this afternoon | 17:27 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance and InstanceList v2.0 objects https://review.openstack.org/217819 | 17:27 |
| *** tonytan4ever has joined #openstack-nova | 17:27 | |
| ndipanov | dansmith, not sure I get the other complaint - but it;s also true for numa then and vcpu_model | 17:28 |
| dansmith | ndipanov: you meant s/complaint/helpful insight/ right? | 17:29 |
| ndipanov | yeah | 17:29 |
| ndipanov | that's true for all of those in extra atm | 17:30 |
| dansmith | ndipanov: yeah, the other ones are broken too | 17:30 |
| dansmith | ndipanov: not all of them, flavor is good | 17:30 |
| *** unicell has joined #openstack-nova | 17:30 | |
| dansmith | ndipanov: regardless of the others being broken, we should do it right here :) | 17:30 |
| ndipanov | yeah | 17:31 |
| *** emagana has quit IRC | 17:31 | |
| mriedem | ndipanov: i need help and you're the only person that can help me | 17:32 |
| dansmith | ndipanov: if you want to set the precedent for what the fix looks like, I can go back and fix the others to match | 17:32 |
| *** timello has quit IRC | 17:33 | |
| *** maurosr has quit IRC | 17:34 | |
| *** ekhugen has quit IRC | 17:34 | |
| *** electrocucaracha has joined #openstack-nova | 17:35 | |
| *** emagana has joined #openstack-nova | 17:36 | |
| *** baoli has joined #openstack-nova | 17:36 | |
| *** penick has joined #openstack-nova | 17:38 | |
| *** promulo has joined #openstack-nova | 17:38 | |
| *** lpetrut has joined #openstack-nova | 17:38 | |
| *** baoli has quit IRC | 17:40 | |
| ndipanov | dansmith, I | 17:40 |
| dansmith | how existential | 17:40 |
| ndipanov | ll just check if it's None before calling | 17:41 |
| ndipanov | like you do with Flavor | 17:41 |
| *** baoli has joined #openstack-nova | 17:41 | |
| dansmith | okay cool | 17:41 |
| dansmith | I have to run off for a bit, but I'll look when I get back | 17:41 |
| cfriesen | dansmith: what's the rationale for adding v2 instance? | 17:41 |
| *** Sukhdev has joined #openstack-nova | 17:41 | |
| dansmith | cfriesen: dropping baggage? | 17:41 |
| * dansmith & | 17:42 | |
| ndipanov | dansmith, actually having a sentinel is maybe a better idea | 17:42 |
| ndipanov | I don't know | 17:42 |
| ndipanov | will fix it | 17:42 |
| cfriesen | dansmith: okay, so you're going to have a bunch of followup patches to actually drop the baggage from the v2 version? | 17:42 |
| *** exploreshaifali has quit IRC | 17:46 | |
| *** afazekas__ has joined #openstack-nova | 17:46 | |
| *** mmedvede has joined #openstack-nova | 17:47 | |
| cfriesen | dansmith: never mind, I found the patch series | 17:48 |
| *** Sukhdev has quit IRC | 17:49 | |
| *** afazekas__ has quit IRC | 17:51 | |
| *** vivekd_ has joined #openstack-nova | 17:52 | |
| *** vivekd has quit IRC | 17:53 | |
| *** vivekd_ is now known as vivekd | 17:53 | |
| *** ssurana has joined #openstack-nova | 17:53 | |
| *** tpeoples has quit IRC | 17:53 | |
| *** rotbeard has quit IRC | 17:53 | |
| *** david-lyle has joined #openstack-nova | 17:54 | |
| *** afazekas__ has joined #openstack-nova | 17:54 | |
| *** penick has quit IRC | 17:55 | |
| *** vilobhmm1 has joined #openstack-nova | 17:55 | |
| *** skylerberg has left #openstack-nova | 17:56 | |
| *** SourabhP has joined #openstack-nova | 17:58 | |
| *** pbrooko has quit IRC | 17:58 | |
| *** gyee has joined #openstack-nova | 17:59 | |
| *** jichen has joined #openstack-nova | 17:59 | |
| *** david-lyle has quit IRC | 18:01 | |
| *** VW_ has quit IRC | 18:02 | |
| *** VW_ has joined #openstack-nova | 18:02 | |
| *** tjones has joined #openstack-nova | 18:03 | |
| *** ssurana has quit IRC | 18:03 | |
| *** tpeoples has joined #openstack-nova | 18:05 | |
| *** promulo has quit IRC | 18:06 | |
| *** VW_ has quit IRC | 18:07 | |
| *** promulo has joined #openstack-nova | 18:07 | |
| *** afazekas__ has quit IRC | 18:07 | |
| *** alex_xu has quit IRC | 18:08 | |
| *** sachin has joined #openstack-nova | 18:09 | |
| *** penick has joined #openstack-nova | 18:10 | |
| *** lpetrut has quit IRC | 18:10 | |
| *** meghal has quit IRC | 18:10 | |
| *** VW_ has joined #openstack-nova | 18:11 | |
| *** alex_xu has joined #openstack-nova | 18:11 | |
| *** Sukhdev has joined #openstack-nova | 18:12 | |
| *** sachin has quit IRC | 18:14 | |
| *** sachin has joined #openstack-nova | 18:16 | |
| *** sachin has quit IRC | 18:16 | |
| *** SourabhP has quit IRC | 18:16 | |
| *** SourabhP has joined #openstack-nova | 18:17 | |
| *** hparekh has joined #openstack-nova | 18:19 | |
| *** david-lyle has joined #openstack-nova | 18:19 | |
| *** promulo__ has joined #openstack-nova | 18:20 | |
| *** Marga_ has joined #openstack-nova | 18:21 | |
| *** promulo has quit IRC | 18:21 | |
| *** danpb has quit IRC | 18:21 | |
| *** thorst has quit IRC | 18:23 | |
| *** thorst has joined #openstack-nova | 18:23 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: RT: Migration resource tracking uses migration context https://review.openstack.org/218938 | 18:25 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: compute: migrate/resize paths properly handle stashed numa_topology https://review.openstack.org/218500 | 18:25 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Claims: Make sure move claims create a migration context records https://review.openstack.org/218385 | 18:25 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: objects: Hook migration object into Instance https://review.openstack.org/216733 | 18:25 |
| ndipanov | dansmith, ^ when you're back | 18:25 |
| dansmith | I just got back, will look | 18:25 |
| *** rushil has quit IRC | 18:27 | |
| ndipanov | dansmith, I found another issue with the last patch - good catch on that | 18:28 |
| *** meghal has joined #openstack-nova | 18:29 | |
| dansmith | ndipanov: the RT patch isn't updated yet? | 18:29 |
| *** tjones has quit IRC | 18:30 | |
| *** thangp_ has joined #openstack-nova | 18:31 | |
| *** aysyd has quit IRC | 18:32 | |
| *** aysyd has joined #openstack-nova | 18:33 | |
| *** meghal has quit IRC | 18:34 | |
| *** exploreshaifali has joined #openstack-nova | 18:34 | |
| *** thangp has quit IRC | 18:34 | |
| *** baoli has quit IRC | 18:35 | |
| *** baoli has joined #openstack-nova | 18:37 | |
| *** dims has quit IRC | 18:40 | |
| *** meghal has joined #openstack-nova | 18:41 | |
| *** meghal1 has joined #openstack-nova | 18:43 | |
| *** meghal has quit IRC | 18:43 | |
| *** electrocucaracha has quit IRC | 18:44 | |
| *** diogogmt has quit IRC | 18:48 | |
| *** ssurana has joined #openstack-nova | 18:48 | |
| ndipanov | dansmith, nope it's a bit more complicated | 18:50 |
| ndipanov | well not a lot | 18:51 |
| dansmith | okay | 18:51 |
| ndipanov | but need to cook dinner | 18:51 |
| dansmith | sure just wanted to make sure I wasn't missing it | 18:51 |
| ndipanov | which adds to complexity | 18:51 |
| *** vishwanathj has joined #openstack-nova | 18:52 | |
| *** ivasev has quit IRC | 18:52 | |
| jwcroppe | dansmith: you mind checking out https://bugs.launchpad.net/nova/+bug/1490837 and the fix I proposed if/when you get a sec? Another groovy metrics thing I ran across :) | 18:53 |
| openstack | Launchpad bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Undecided,In progress] - Assigned to Joe Cropper (jwcroppe) | 18:53 |
| *** Marga_ has quit IRC | 18:53 | |
| *** raildo-afk is now known as raildo | 18:53 | |
| dansmith | in a bit | 18:53 |
| *** eharney has quit IRC | 18:53 | |
| jwcroppe | 10-4 | 18:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/216968 | 18:53 |
| *** diogogmt has joined #openstack-nova | 18:54 | |
| *** hparekh has quit IRC | 18:56 | |
| *** tri2sing has quit IRC | 18:57 | |
| *** salv-orlando has quit IRC | 18:57 | |
| *** SourabhP has quit IRC | 18:58 | |
| *** jhesketh has quit IRC | 18:59 | |
| *** annegent_ has joined #openstack-nova | 19:00 | |
| *** lpetrut has joined #openstack-nova | 19:00 | |
| *** Apoorva has quit IRC | 19:01 | |
| *** tri2sing has joined #openstack-nova | 19:02 | |
| *** changbl has quit IRC | 19:02 | |
| *** marcusvrn_ has joined #openstack-nova | 19:03 | |
| *** annegentle has quit IRC | 19:03 | |
| *** jhesketh has joined #openstack-nova | 19:03 | |
| *** Apoorva has joined #openstack-nova | 19:04 | |
| *** Piet has quit IRC | 19:04 | |
| *** ssurana has quit IRC | 19:05 | |
| *** salv-orlando has joined #openstack-nova | 19:06 | |
| *** annegent_ has quit IRC | 19:08 | |
| *** annegentle has joined #openstack-nova | 19:11 | |
| *** krtaylor has quit IRC | 19:11 | |
| *** penick has quit IRC | 19:12 | |
| *** penick has joined #openstack-nova | 19:14 | |
| *** hemna_ is now known as hemnafk | 19:15 | |
| mriedem | ffs https://review.openstack.org/#/c/216968/ | 19:15 |
| mriedem | just merge damn you | 19:15 |
| alaski | needs more +2s | 19:16 |
| mriedem | more cowbell | 19:16 |
| *** tri2sing has quit IRC | 19:16 | |
| *** dims has joined #openstack-nova | 19:18 | |
| *** tri2sing has joined #openstack-nova | 19:19 | |
| *** krtaylor has joined #openstack-nova | 19:21 | |
| *** dims has quit IRC | 19:25 | |
| *** pbrooko has joined #openstack-nova | 19:25 | |
| *** lpetrut has quit IRC | 19:25 | |
| *** armax has quit IRC | 19:26 | |
| *** tri2sing has quit IRC | 19:27 | |
| *** promulo__ has quit IRC | 19:30 | |
| *** artom has quit IRC | 19:30 | |
| *** tri2sing has joined #openstack-nova | 19:30 | |
| *** tri2sing has quit IRC | 19:30 | |
| *** tri2sing has joined #openstack-nova | 19:31 | |
| cfriesen | is the Hyper-V CI voting? the review page doesn't say non-voting, but http://ci-watch.tintri.com/project?project=nova shows it as failing an awful lot | 19:33 |
| mriedem | cfriesen: i think they took it off voting a week or two back b/c of issues | 19:33 |
| mriedem | claudiub: ^ | 19:33 |
| *** krtaylor has quit IRC | 19:34 | |
| *** aysyd has quit IRC | 19:34 | |
| *** thorst has quit IRC | 19:34 | |
| *** VW_ has quit IRC | 19:34 | |
| *** tpeoples has quit IRC | 19:34 | |
| *** claudiub has quit IRC | 19:34 | |
| *** ZZelle has quit IRC | 19:34 | |
| *** pixelb has quit IRC | 19:34 | |
| *** hakimo has quit IRC | 19:34 | |
| *** hanzhang has quit IRC | 19:34 | |
| *** jwcroppe has quit IRC | 19:34 | |
| *** lifeless has quit IRC | 19:34 | |
| *** mcarden has quit IRC | 19:34 | |
| *** tobasco_ has quit IRC | 19:34 | |
| *** rmstar has quit IRC | 19:34 | |
| *** mtreinish has quit IRC | 19:34 | |
| *** tdurakov has quit IRC | 19:34 | |
| *** mrodden has quit IRC | 19:34 | |
| *** raginbajin has quit IRC | 19:34 | |
| *** n0ano has quit IRC | 19:34 | |
| *** harlowja has quit IRC | 19:34 | |
| *** xiaohui has quit IRC | 19:34 | |
| *** eliqiao has quit IRC | 19:34 | |
| *** goodygum has quit IRC | 19:34 | |
| *** nithyag__ has quit IRC | 19:34 | |
| *** whydidyoustealmy has quit IRC | 19:34 | |
| *** IanGovett has quit IRC | 19:34 | |
| *** sc68cal has quit IRC | 19:34 | |
| *** sdague has quit IRC | 19:34 | |
| *** jamiec has quit IRC | 19:34 | |
| *** jhesketh has quit IRC | 19:34 | |
| *** Piet has joined #openstack-nova | 19:34 | |
| *** baoli has quit IRC | 19:34 | |
| ociuhandu | cfriesen: we’re working now on looking why a few of the tempest tests fail very often | 19:34 |
| *** mriedem is now known as mattykins | 19:34 | |
| *** IanGovett has joined #openstack-nova | 19:34 | |
| *** sc68cal has joined #openstack-nova | 19:34 | |
| *** sdague has joined #openstack-nova | 19:34 | |
| *** jamiec has joined #openstack-nova | 19:34 | |
| *** promulo has joined #openstack-nova | 19:35 | |
| *** rushil has joined #openstack-nova | 19:35 | |
| *** krtaylor has joined #openstack-nova | 19:35 | |
| *** aysyd has joined #openstack-nova | 19:35 | |
| *** thorst has joined #openstack-nova | 19:35 | |
| *** VW_ has joined #openstack-nova | 19:35 | |
| *** tpeoples has joined #openstack-nova | 19:35 | |
| *** claudiub has joined #openstack-nova | 19:35 | |
| *** ZZelle has joined #openstack-nova | 19:35 | |
| *** pixelb has joined #openstack-nova | 19:35 | |
| *** hakimo has joined #openstack-nova | 19:35 | |
| *** hanzhang has joined #openstack-nova | 19:35 | |
| *** jwcroppe has joined #openstack-nova | 19:35 | |
| *** lifeless has joined #openstack-nova | 19:35 | |
| *** mcarden has joined #openstack-nova | 19:35 | |
| *** tobasco_ has joined #openstack-nova | 19:35 | |
| *** rmstar has joined #openstack-nova | 19:35 | |
| *** mtreinish has joined #openstack-nova | 19:35 | |
| *** tdurakov has joined #openstack-nova | 19:35 | |
| *** mrodden has joined #openstack-nova | 19:35 | |
| *** raginbajin has joined #openstack-nova | 19:35 | |
| *** n0ano has joined #openstack-nova | 19:35 | |
| *** xiaohui has joined #openstack-nova | 19:35 | |
| *** eliqiao has joined #openstack-nova | 19:35 | |
| *** goodygum has joined #openstack-nova | 19:35 | |
| *** nithyag__ has joined #openstack-nova | 19:35 | |
| *** whydidyoustealmy has joined #openstack-nova | 19:35 | |
| *** harlowja has joined #openstack-nova | 19:35 | |
| *** whydidyoustealmy has quit IRC | 19:35 | |
| *** harlowja has quit IRC | 19:35 | |
| *** harlowja has joined #openstack-nova | 19:35 | |
| *** jhesketh has joined #openstack-nova | 19:35 | |
| *** dannywilson has joined #openstack-nova | 19:35 | |
| *** baoli has joined #openstack-nova | 19:35 | |
| *** dims has joined #openstack-nova | 19:35 | |
| *** eharney has joined #openstack-nova | 19:35 | |
| *** whydidyoustealmy has joined #openstack-nova | 19:35 | |
| bauzas | jwcroppe: got 2 secs for discussing about https://review.openstack.org/#/c/219153/3 ? | 19:36 |
| bauzas | jwcroppe: while I was trying to explain my thoughts, it seems that dansmith provided another view btw. | 19:37 |
| bauzas | jwcroppe: the problem is with stale data, IMHO | 19:37 |
| *** unicell has quit IRC | 19:38 | |
| *** rdopiera has quit IRC | 19:39 | |
| ociuhandu | cfriesen: the CI was offlined for some hardware and network updates and in the mean time some code got in, we’re now checking how it impacted the CI as for neutron the same buld environment works quite reliably | 19:39 |
| *** erlon has joined #openstack-nova | 19:44 | |
| jwcroppe | bauzas: I replied to your comment | 19:45 |
| bauzas | jwcroppe: just reading it | 19:46 |
| jwcroppe | bauzas: ok | 19:46 |
| bauzas | jwcroppe: well, I really think the 1-sec interval is for returning the cache if under the interval | 19:46 |
| bauzas | jwcroppe: because the timestamp is identical | 19:47 |
| bauzas | jwcroppe: but all of that happens within one RT call | 19:47 |
| jwcroppe | I think the cache is to keep the same underlying data so you get meaningful metrics | 19:47 |
| *** mattykins is now known as mriedem | 19:47 | |
| jwcroppe | Right | 19:47 |
| anish | yay, I made the cutoff ! | 19:47 |
| *** harlowja has quit IRC | 19:47 | |
| *** promulo has quit IRC | 19:47 | |
| *** rushil has quit IRC | 19:47 | |
| *** krtaylor has quit IRC | 19:47 | |
| *** aysyd has quit IRC | 19:47 | |
| *** thorst has quit IRC | 19:47 | |
| *** VW_ has quit IRC | 19:47 | |
| *** tpeoples has quit IRC | 19:47 | |
| *** claudiub has quit IRC | 19:47 | |
| *** ZZelle has quit IRC | 19:47 | |
| *** pixelb has quit IRC | 19:47 | |
| *** hakimo has quit IRC | 19:47 | |
| *** hanzhang has quit IRC | 19:47 | |
| *** jwcroppe has quit IRC | 19:47 | |
| *** lifeless has quit IRC | 19:47 | |
| *** mcarden has quit IRC | 19:47 | |
| *** tobasco_ has quit IRC | 19:47 | |
| *** rmstar has quit IRC | 19:47 | |
| *** mtreinish has quit IRC | 19:47 | |
| *** tdurakov has quit IRC | 19:47 | |
| *** mrodden has quit IRC | 19:47 | |
| *** raginbajin has quit IRC | 19:47 | |
| *** n0ano has quit IRC | 19:47 | |
| *** xiaohui has quit IRC | 19:47 | |
| *** eliqiao has quit IRC | 19:47 | |
| *** goodygum has quit IRC | 19:47 | |
| *** nithyag__ has quit IRC | 19:47 | |
| bauzas | ergh, netsplit in the middle of a convo :/ | 19:48 |
| dansmith | count your blessings :) | 19:48 |
| *** meghal1 has quit IRC | 19:48 | |
| bauzas | dansmith: I promise, I haven't asked some hackers to do that :p | 19:49 |
| *** penick has quit IRC | 19:49 | |
| dansmith | suuuure | 19:49 |
| bauzas | but that could be a good thing for the next time :) | 19:49 |
| * bauzas just puts a note | 19:50 | |
| *** davideagnello has quit IRC | 19:50 | |
| dansmith | heh | 19:50 |
| *** harlowja has joined #openstack-nova | 19:52 | |
| *** promulo has joined #openstack-nova | 19:52 | |
| *** rushil has joined #openstack-nova | 19:52 | |
| *** krtaylor has joined #openstack-nova | 19:52 | |
| *** aysyd has joined #openstack-nova | 19:52 | |
| *** thorst has joined #openstack-nova | 19:52 | |
| *** VW_ has joined #openstack-nova | 19:52 | |
| *** tpeoples has joined #openstack-nova | 19:52 | |
| *** claudiub has joined #openstack-nova | 19:52 | |
| *** ZZelle has joined #openstack-nova | 19:52 | |
| *** pixelb has joined #openstack-nova | 19:52 | |
| *** hakimo has joined #openstack-nova | 19:52 | |
| *** hanzhang has joined #openstack-nova | 19:52 | |
| *** jwcroppe has joined #openstack-nova | 19:52 | |
| *** lifeless has joined #openstack-nova | 19:52 | |
| *** mcarden has joined #openstack-nova | 19:52 | |
| *** tobasco_ has joined #openstack-nova | 19:52 | |
| *** rmstar has joined #openstack-nova | 19:52 | |
| *** mtreinish has joined #openstack-nova | 19:52 | |
| *** tdurakov has joined #openstack-nova | 19:52 | |
| *** mrodden has joined #openstack-nova | 19:52 | |
| *** raginbajin has joined #openstack-nova | 19:52 | |
| *** n0ano has joined #openstack-nova | 19:52 | |
| *** xiaohui has joined #openstack-nova | 19:52 | |
| *** eliqiao has joined #openstack-nova | 19:52 | |
| *** goodygum has joined #openstack-nova | 19:52 | |
| *** nithyag__ has joined #openstack-nova | 19:52 | |
| *** yassine has quit IRC | 19:53 | |
| jwcroppe | bauzas: the problem is that... within a "batch of metrics"... sometimes the first call to get_metric()-->_update_data() forces the cache timestamp to get set before libvirt call and therefore if it's slow (read: > 1 second), the next n-1 calls use different data. | 19:53 |
| jwcroppe | bauzas: I think it was originally used as a way to make the metric retrievals "atomic", albeit subject to this error | 19:53 |
| jwcroppe | sorry for the repeat - folks got booted it seemed :) | 19:53 |
| *** davideagnello has joined #openstack-nova | 19:53 | |
| *** penick has joined #openstack-nova | 19:54 | |
| *** ekhugen has joined #openstack-nova | 19:54 | |
| *** ociuhandu has quit IRC | 19:55 | |
| *** maurosr has joined #openstack-nova | 19:55 | |
| bauzas | jwcroppe: yeah we had a netsplit | 19:55 |
| *** burt has joined #openstack-nova | 19:55 | |
| *** meghal has joined #openstack-nova | 19:55 | |
| bauzas | jwcroppe: well, you agree with libvirt as the potential blocker, right? | 19:56 |
| *** timello has joined #openstack-nova | 19:56 | |
| bauzas | jwcroppe: because I guess it's an IO op | 19:56 |
| jwcroppe | bauzas: right | 19:56 |
| bauzas | jwcroppe: okay | 19:56 |
| *** pbrooko has quit IRC | 19:57 | |
| bauzas | jwcroppe: so given that, why do you think it's good to return the same metrics for two calls if the first call was in a blocked state? | 19:57 |
| jwcroppe | bauzas: there's no reason to not just move the cache timestamp *after* it for a quick fix on this - I don't think it was the author's original intention to have call 1 use libvirt data X and then calls n-1 to use libvirt data Y for the same 'group' of metrics | 19:57 |
| bauzas | jwcroppe: again, I just think we're overthinking the original time check | 19:58 |
| jwcroppe | bauzas: so you get a coherent set of metrics | 19:58 |
| jwcroppe | bauzas: all of those metrics are an atomic set of values | 19:58 |
| jwcroppe | bauzas: recall they're not surfaced as individual metrics to the outside world - it's a group | 19:59 |
| bauzas | jwcroppe: each _update_data() call is atomic, I agree | 19:59 |
| *** electrocucaracha has joined #openstack-nova | 19:59 | |
| bauzas | jwcroppe: but there is no reason to accept that long delayed libvirt calls should get the same metrics | 20:00 |
| bauzas | that's IMHO a bad thinh | 20:00 |
| bauzas | thing even | 20:00 |
| *** boris-42 has quit IRC | 20:00 | |
| *** meghal has quit IRC | 20:00 | |
| bauzas | does anyone know who is Shane Wang ? :) | 20:00 |
| jwcroppe | bauzas: let's back up... when you get an RPC notification for a set of metrics, it's a json-encoded dict. with me so far? | 20:00 |
| *** krtaylor has quit IRC | 20:01 | |
| bauzas | jwcroppe: if you're speaking about https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L436, yup | 20:02 |
| *** krtaylor has joined #openstack-nova | 20:02 | |
| jwcroppe | bauzas: in that dict, you have things like: {'cpu.kernel.percent': A, 'cpu.iowait.percent': B, 'cpu.user.percent': C, 'cpu.idle.percent': D, 'cpu.percent': E} (and more entries...just not typing for brevity) ... this is what you get from RPC as well as if you read the data in compute_nodes.metrics. Agreed so far? | 20:02 |
| *** hemnafk is now known as hemna | 20:03 | |
| openstackgerrit | jichenjc proposed openstack/nova-specs: Add display-flavor-id-in-migration.rst https://review.openstack.org/219431 | 20:03 |
| jwcroppe | bauzas: and yep, that's what I was talking about | 20:03 |
| bauzas | jwcroppe: yup, because of https://github.com/openstack/nova/blob/master/nova/compute/monitors/base.py#L59 | 20:03 |
| jwcroppe | bauzas: Right. Ok. So, if you were consuming that data. Would you ever expect to see a case where A + B + C + D != 100? | 20:04 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance and InstanceList v2.0 objects https://review.openstack.org/217819 | 20:04 |
| openstackgerrit | Dan Smith proposed openstack/nova: Teach conductor to do manifest-based object_class_action() things https://review.openstack.org/219432 | 20:04 |
| jwcroppe | bauzas: (i.e., for the same record of data... again, from a *consumer* perspective) | 20:04 |
| bauzas | jwcroppe: then the whole stuff is crap and needs to be dumped IMHO | 20:05 |
| jwcroppe | bauzas: I didn't write the code :) | 20:05 |
| bauzas | jwcroppe: I see your point | 20:05 |
| *** armax has joined #openstack-nova | 20:05 | |
| *** unicell has joined #openstack-nova | 20:05 | |
| bauzas | jwcroppe: but I still think it's just bad to directly call the cache if there is something which was stale | 20:05 |
| jwcroppe | bauzas: then we agree that this bug exists. | 20:06 |
| bauzas | jwcroppe: yup | 20:06 |
| bauzas | jwcroppe: and I understand your problem | 20:06 |
| bauzas | jwcroppe: but I still -1 the change :p | 20:06 |
| jwcroppe | bauzas: I don't dispute that this isn't the prettiest code, I am just suggesting a quick fix until we can find a better comprehensive solution | 20:06 |
| jwcroppe | bauzas: The problem I have is that we are delivering metrics that don't make any sense from a mathematical standpoint, and this patch resolves that in the cases I've seen. And it's a 4 line change. :) | 20:07 |
| bauzas | jwcroppe: 4 lines change on a bad thing | 20:07 |
| *** nic1 has joined #openstack-nova | 20:07 | |
| jwcroppe | bauzas: so it's better to just leave the bug and the bad thing? :) | 20:07 |
| bauzas | jwcroppe: no, it's just better to discuss about something much better :) | 20:08 |
| bauzas | jwcroppe: I'm there, see :) | 20:08 |
| *** nic has quit IRC | 20:08 | |
| bauzas | even when netsplits :p | 20:08 |
| jwcroppe | bauzas: Haha, yes. I think this framework needs to be overhauled a bit | 20:08 |
| jwcroppe | bauzas: probably more blueprint'ish, I was hoping we could reach common ground on an interim fix for L... I'd like to hear jaypipes input | 20:09 |
| *** ociuhandu has joined #openstack-nova | 20:09 | |
| *** nic1 is now known as nic | 20:09 | |
| bauzas | jwcroppe: so, the thing is, there are some resources that need to be contiguous to some others | 20:10 |
| openstackgerrit | Marian Horban proposed openstack/nova: [WIP]Remove wsgi functionality https://review.openstack.org/219248 | 20:10 |
| bauzas | jwcroppe: like you said, groups | 20:10 |
| jwcroppe | bauzas: yep | 20:10 |
| *** jcoufal has quit IRC | 20:11 | |
| bauzas | jwcroppe: ie. user/iowait/kernel/idle time and percentage, right? | 20:11 |
| bauzas | I count 2 groups | 20:11 |
| jwcroppe | well, the percentages are derived from the former set | 20:11 |
| bauzas | jwcroppe: oh right | 20:12 |
| jwcroppe | bauzas: e.g., https://github.com/openstack/nova/blob/master/nova/compute/monitors/cpu/virt_driver.py#L84 | 20:12 |
| *** Sukhdev has quit IRC | 20:12 | |
| bauzas | jwcroppe: yup | 20:12 |
| bauzas | jwcroppe: I forgot that | 20:12 |
| jwcroppe | so it's one mega group | 20:12 |
| *** exploreshaifali has quit IRC | 20:12 | |
| *** vivekd has quit IRC | 20:13 | |
| bauzas | jwcroppe: but even the percent metrics are called, right? | 20:13 |
| jwcroppe | yep | 20:13 |
| jwcroppe | bauzas: so there are 5 actual libvirt metrics... 10 total metrics (5 from libvirt and 5 derived percentages). And then we call get_metric(name) 10 times to build the 'group' | 20:14 |
| bauzas | jwcroppe: okay, so if we need to care about some semantics, then I would make sure that some metrics are within the same libvirt call, while the others are not | 20:14 |
| *** eharney has quit IRC | 20:15 | |
| jwcroppe | bauzas: I'm not sure I agree -- if we are producing a single 'record' of metrics (e.g., a notification/DB entry), the data should be all based on the same libvirt call | 20:15 |
| bauzas | jwcroppe: but we would still get the problem I mentioned | 20:15 |
| openstackgerrit | jichenjc proposed openstack/nova-specs: Add display-flavor-id-in-migration.rst https://review.openstack.org/219431 | 20:16 |
| jwcroppe | bauzas: do you agree with my assertion on the 1 metric record == 1 libvirt call ? | 20:16 |
| bauzas | no :) | 20:16 |
| jwcroppe | bauzas: ? What good are the datapoints if they span several time windows? | 20:17 |
| bauzas | I'm fine with the same set of metrics having the same libvirt call | 20:17 |
| jwcroppe | bauzas: allow me to clarify... the metrics should be based off of consistent time periods, otherwise they're kinda pointless if you ask me (whether that means 1 libvirt call is another story) | 20:17 |
| jwcroppe | bauzas: I think with the metrics that we have today, that does mean 1 libvirt call | 20:18 |
| bauzas | jwcroppe: well, that's why we set timestamps :) | 20:19 |
| *** meghal has joined #openstack-nova | 20:19 | |
| bauzas | jwcroppe: I mean, 2 metrics can have 2 different timestamps, that's fine | 20:19 |
| jwcroppe | bauzas: right, agree | 20:19 |
| *** salv-orl_ has joined #openstack-nova | 20:20 | |
| *** meghal has quit IRC | 20:20 | |
| *** meghal1 has joined #openstack-nova | 20:21 | |
| bauzas | jwcroppe: so the problem you mention is only related to the tuple (iowait, user, idle, kernel) | 20:21 |
| jwcroppe | bauzas: right, and its derived metrics (percentage counterparts) | 20:21 |
| *** IanGovett has quit IRC | 20:21 | |
| bauzas | jwcroppe: yup | 20:21 |
| jwcroppe | bauzas: read: every metric we support today ;-) | 20:21 |
| bauzas | jwcroppe: yup | 20:22 |
| *** eharney has joined #openstack-nova | 20:22 | |
| jwcroppe | bauzas: when I looked at the cache code, I assume Shane Wang (do we know where to find him?) did that to keep the group consistent | 20:22 |
| jwcroppe | otherwise I have no idea what the purpose of that code is... https://github.com/openstack/nova/blob/master/nova/compute/monitors/cpu/virt_driver.py#L48-L53 | 20:23 |
| *** jichen has quit IRC | 20:23 | |
| bauzas | jwcroppe: yeah | 20:24 |
| bauzas | jwcroppe: it misses some good docstring fer sur | 20:24 |
| *** salv-orlando has quit IRC | 20:24 | |
| bauzas | jwcroppe: but I think we can have a better fine-grained query here by discriminating per group | 20:24 |
| *** dave-mccowan has quit IRC | 20:25 | |
| *** meghal1 has quit IRC | 20:25 | |
| jwcroppe | bauzas: in my mind, what we want is... "hey, for this group of metrics... use the same underlying data" | 20:26 |
| *** jcoufal has joined #openstack-nova | 20:26 | |
| bauzas | hah, just discovered that => https://www.openstack.org/community/members/profile/8407 | 20:26 |
| bauzas | lemme see Shane's IRC nick | 20:26 |
| *** tri2sing has quit IRC | 20:26 | |
| jwcroppe | bauzas: are you secretly shane wang as well? | 20:27 |
| mriedem | apmelton: i think i'm going to update my devstack wip change to run without lvm backend (using nbd) and depends on your nova-network bridge patch | 20:27 |
| mriedem | apmelton: i'm running with that locally and so far i've been able to boot a few instances and do some things on them w/o issues | 20:27 |
| bauzas | jwcroppe: okay, he's not online | 20:28 |
| bauzas | jwcroppe: anyway, I think you can just move on and rework on that | 20:29 |
| jwcroppe | bauzas: did you agree with my assertion in my last comment about the group? | 20:29 |
| *** tri2sing has joined #openstack-nova | 20:29 | |
| *** vivekd has joined #openstack-nova | 20:29 | |
| mriedem | apmelton: omfg even resize worked | 20:30 |
| mriedem | i'm amazed :) | 20:30 |
| *** Nikolay_St has quit IRC | 20:31 | |
| *** SourabhP has joined #openstack-nova | 20:31 | |
| bauzas | (22:26:18) jwcroppe: bauzas: in my mind, what we want is... "hey, for this group of metrics... use the same underlying data" | 20:31 |
| jwcroppe | bauzas: yep | 20:32 |
| bauzas | jwcroppe: yes to that ^ | 20:32 |
| jwcroppe | bauzas: and then https://github.com/openstack/nova/blob/master/nova/compute/monitors/base.py#L58-L64 would basically ensure that ... somehow | 20:33 |
| *** baoli has quit IRC | 20:34 | |
| *** rlrossit has left #openstack-nova | 20:34 | |
| *** baoli has joined #openstack-nova | 20:34 | |
| jwcroppe | bauzas: IMO, we'd just rm -f that timestamp stuff too | 20:35 |
| jwcroppe | bauzas: welp, the <= 1 sec part | 20:35 |
| *** baoli has quit IRC | 20:36 | |
| bauzas | jwcroppe: well, maybe, I need to see your updated change :) | 20:37 |
| jwcroppe | bauzas: I'm only doing this if I get to delete code nobody understands ;-) | 20:37 |
| *** baoli has joined #openstack-nova | 20:37 | |
| *** dustins has quit IRC | 20:37 | |
| *** dave-mccowan has joined #openstack-nova | 20:38 | |
| *** dprince has quit IRC | 20:38 | |
| bauzas | jwcroppe: you'll get a good karma if you're doing such thing (deleting code that nobody understands) | 20:38 |
| jwcroppe | bauzas: thanks for the discussion - go to sleep now. I'll see how soon I can get back to this - this gives us something to go on tho | 20:38 |
| bauzas | too early for sleeping | 20:39 |
| bauzas | I'm a night owl | 20:39 |
| jwcroppe | bauzas: I know how that goes :) | 20:39 |
| *** baoli has quit IRC | 20:43 | |
| *** meghal has joined #openstack-nova | 20:44 | |
| *** ssurana has joined #openstack-nova | 20:45 | |
| *** afranc202 has joined #openstack-nova | 20:46 | |
| *** tri2sing has quit IRC | 20:47 | |
| *** meghal1 has joined #openstack-nova | 20:47 | |
| *** tri2sing has joined #openstack-nova | 20:48 | |
| *** ssurana has quit IRC | 20:48 | |
| *** meghal has quit IRC | 20:49 | |
| *** yamahata has quit IRC | 20:49 | |
| *** Apoorva has quit IRC | 20:50 | |
| *** rlrossit has joined #openstack-nova | 20:51 | |
| *** Apoorva has joined #openstack-nova | 20:52 | |
| *** thedodd has quit IRC | 20:53 | |
| *** achanda has quit IRC | 20:54 | |
| *** earlephilhower has quit IRC | 20:55 | |
| *** alex_klimov has joined #openstack-nova | 20:56 | |
| *** meghal1 has quit IRC | 21:02 | |
| *** changbl has joined #openstack-nova | 21:02 | |
| *** raildo is now known as raildo-afk | 21:02 | |
| *** meghal has joined #openstack-nova | 21:02 | |
| *** meghal has quit IRC | 21:02 | |
| *** meghal has joined #openstack-nova | 21:02 | |
| *** aysyd has quit IRC | 21:04 | |
| *** rfolco has quit IRC | 21:05 | |
| *** ssurana has joined #openstack-nova | 21:06 | |
| *** thangp_ has quit IRC | 21:07 | |
| bauzas | alaski: jaypipes: dansmith: by seeing https://review.openstack.org/#/c/194059/2/nova/scheduler/filters/disk_filter.py,cm I think we need to somehow make sure that filter_props is not updated until we merge the ReqSpec object | 21:07 |
| *** achanda has joined #openstack-nova | 21:07 | |
| *** changbl has quit IRC | 21:09 | |
| alaski | bauzas: is bdm in filter_properties? | 21:09 |
| bauzas | alaski: nope | 21:10 |
| *** lbeliveau has quit IRC | 21:10 | |
| bauzas | alaski: https://review.openstack.org/#/c/194059/2/nova/compute/api.py,cm | 21:10 |
| bauzas | alaski: ^ he's adding it there | 21:10 |
| *** vilobhmm1 has quit IRC | 21:10 | |
| *** vilobhmm1 has joined #openstack-nova | 21:11 | |
| bauzas | fortunately, my change broke his change | 21:11 |
| alaski | ok | 21:11 |
| bauzas | alaski: anyway, I think modifying either filt_props or getting a new filter which adds a new hint needs a spec IMHO | 21:12 |
| bauzas | alaski: because it's part of an API contract | 21:12 |
| openstackgerrit | Kevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction https://review.openstack.org/186617 | 21:12 |
| alaski | at the very least a blueprint would be nice | 21:13 |
| *** changbl has joined #openstack-nova | 21:13 | |
| bauzas | alaski: yeah | 21:13 |
| alaski | I have other issues with the change which I'll comment on, and ask that they wait for the scheduler object work to be complete | 21:13 |
| ccarmack | mriedem: why did you upload https://review.openstack.org/#/c/219448/ if you don't want it merged? | 21:13 |
| bauzas | alaski: even with that, I'm not sure getting the BDMs in the scheduler is a good thing | 21:13 |
| bauzas | that way or any other way | 21:14 |
| alaski | yeah, that's my other issue | 21:14 |
| mriedem | ccarmack: b/c i want a gate-tempest-dsvm-full run on it so i can see the results | 21:14 |
| ccarmack | ah, I see | 21:14 |
| mriedem | ccarmack: because i need to know what is going to fail so those tests can be backlisted in an lxc job if they are operations not supported by that configuration | 21:15 |
| alaski | bauzas: either root_gb should be 0 for the instance, or there could be a flag indicating bfv. can be discussed on a spec/bp later | 21:15 |
| ccarmack | blacklisted? | 21:15 |
| mriedem | ccarmack: yeah, like we do for the dsvm-tempest cells job http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc | 21:15 |
| mriedem | http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n1011 | 21:16 |
| mriedem | for example cells doesn't support security groups so dont' run those tests in the cells job | 21:16 |
| bauzas | alaski: yeah, if that's for root_gb, the RT can track that | 21:17 |
| ccarmack | mriedem: ok, got it | 21:17 |
| mriedem | ccarmack: and based on http://docs.openstack.org/developer/nova/support-matrix.html - lxc backend won't support attaching volumes, for example | 21:17 |
| mtreinish | mriedem: the cells job is a bad example, sec groups and other features should be tempest flags if they're configurable | 21:17 |
| mriedem | mtreinish: that was a todo yeah | 21:17 |
| mriedem | once we got the cells backlist | 21:18 |
| mriedem | after we got the cells job voting | 21:18 |
| mriedem | so, | 21:18 |
| mriedem | ctrath: ccarmack: rlrossit: i guess that's another thing that could be worked on | 21:18 |
| ccarmack | mriedem: I'll take a look | 21:19 |
| mriedem | i feel like we had a thing in the mailing list about this one, or something in an etherpad | 21:19 |
| mriedem | alaski: ^ do you remember tracking a todo for adding tempest config options for some of the blacklisted stuff in http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc ? | 21:19 |
| mriedem | melwitt: bauzas: ^? | 21:19 |
| mtreinish | mriedem: I've ranted about it since alaski pushed up that first change to add the blacklist regex | 21:19 |
| ccarmack | mriedem: so, I need to get the cells blacklist then change the cells job to voting? | 21:20 |
| mriedem | yeah but you're just some crazy old man that rants about everything | 21:20 |
| mtreinish | it probably made it to ML at some point too | 21:20 |
| *** irenab_ has joined #openstack-nova | 21:20 | |
| *** rushil has quit IRC | 21:20 | |
| mriedem | ccarmack: no, the cells job is voting already | 21:20 |
| alaski | mriedem: I don't know of any official tracking on it. I think it's just been discussion so far | 21:20 |
| mriedem | ccarmack: tempest can be configured for feature flags | 21:20 |
| mriedem | like pause/resume | 21:20 |
| mriedem | if youre virt driver doesn't support pause/resume, configure tempest to not run those tests | 21:20 |
| *** irenab has quit IRC | 21:20 | |
| *** irenab_ is now known as irenab | 21:20 | |
| mriedem | *your | 21:20 |
| *** burt has quit IRC | 21:21 | |
| mriedem | ccarmack: so the idea was adding config options to tempest for these things that tempest tests but doesn't work today with cells | 21:21 |
| alaski | I could open a wishlist bug for it. But I don't actually understand the tempest mechanism we're supposed to switch to using | 21:21 |
| bauzas | mriedem: none I remember, sorry | 21:21 |
| mriedem | alaski: ccarmack: e.g. http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n340 | 21:22 |
| mtreinish | alaski: it's a config option in http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n319 | 21:22 |
| mriedem | tempest has the compute-feature-enabled group | 21:22 |
| mtreinish | or many config opts | 21:22 |
| mtreinish | for all the things cells v1 doesnt do | 21:22 |
| bauzas | mtreinish: so we would add new flags for explicitely not calling what we want ? | 21:22 |
| mriedem | alaski: ccarmack: and then the tests are not run if you don't support those options in your configuration | 21:22 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/servers/test_server_actions.py#n418 | 21:22 |
| mtreinish | then the tests that do those things just have to be set to raise a skip if that flag is false | 21:23 |
| bauzas | I see | 21:23 |
| alaski | okay, so it's a mix of config and flagging tests | 21:23 |
| alaski | potentially | 21:23 |
| mtreinish | alaski: well you already have the list of tests | 21:23 |
| mtreinish | it's just mapping those to the features and adding a flag for each | 21:24 |
| ccarmack | mriedem: does the config option replace what tests used to be blacklisted ... sorry, trying to keep up | 21:24 |
| mriedem | ccarmack: yeah | 21:24 |
| openstackgerrit | Marian Horban proposed openstack/nova: [WIP]Remove wsgi functionality https://review.openstack.org/219248 | 21:24 |
| bauzas | alaski: I'm pretty free those times | 21:24 |
| bauzas | alaski: I can volunteer for that | 21:24 |
| mriedem | ccarmack: so for example, there are 3 shelve-related tests skipped in http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc | 21:25 |
| alaski | mtreinish: yeah. that's easy enough | 21:25 |
| mriedem | ccarmack: so if you added a compute-feature-enabled flag in tempest for shelve, then in the cells job config we just set that to False | 21:25 |
| mtreinish | mriedem: there is already a flag for that | 21:25 |
| mriedem | else default to True | 21:25 |
| mriedem | orly | 21:25 |
| mtreinish | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n343 | 21:25 |
| mtreinish | I think devstack just doesn't know to set these things if cells | 21:26 |
| mriedem | dibs on the change | 21:26 |
| mriedem | mtreinish: the project-config change and pass them through to devstack right? | 21:26 |
| * mtreinish doubts users do either | 21:26 | |
| mtreinish | mriedem: nah, it's probably a lib/tempest change in devstack | 21:26 |
| alaski | bauzas: okay. we should make sure melwitt is aware as well in case she wants to get in on the action | 21:26 |
| mriedem | mtreinish: ok, fair enough | 21:26 |
| mriedem | so devstack change, project-config change, nova change | 21:26 |
| mtreinish | if is_service_enabled n-cells ; do iniset shelve false | 21:26 |
| bauzas | alaski: sure, we can discuss that tomorrow | 21:26 |
| mriedem | or that | 21:26 |
| mriedem | sure | 21:26 |
| mtreinish | mriedem: probably not a project-config change needed | 21:27 |
| melwitt | I'm reading the flurry of messages | 21:27 |
| mriedem | melwitt: hi stranger | 21:27 |
| melwitt | for me, I knew it's a thing we needed to get done but there was an ignorance barrier on how to do it. with these details I finally understand all of it. I think | 21:28 |
| mriedem | ccarmack: so you want to handle the shelve thing as a first step? | 21:28 |
| ccarmack | mriedem: yes | 21:28 |
| bauzas | it's a thing we wanted to do but then we had some other stuff to do | 21:28 |
| bauzas | :) | 21:28 |
| alaski | I'm still missing the piece on how to configure the cells job, but I'm sure I can figure that out. I'm just hoping it's still something we can control from the nova repo like the blacklist | 21:28 |
| mriedem | ccarmack: ok, i'll open a low priority bug with the details | 21:28 |
| melwitt | I had missed the part where we would need to add new config options in tempest, and how to tie them to the tests themselves | 21:28 |
| mriedem | and you can track against that | 21:28 |
| ccarmack | mriedem: once I understand this chat :) | 21:28 |
| mtreinish | alaski: devstack configures tempest and it knows you run cells | 21:29 |
| *** thorst has quit IRC | 21:29 | |
| mriedem | devstack knows all | 21:29 |
| mtreinish | so it knows to set all these flags to false | 21:29 |
| bauzas | melwitt: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n319 mapped to eg. http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/servers/test_server_actions.py#n418 | 21:29 |
| mriedem | ccarmack: we'll just pair program this into the ground | 21:29 |
| bauzas | melwitt: alaski: and then we just change the conf flags for the devstack job | 21:29 |
| melwitt | bauzas: yeah, sorry I meant until today I didnt' get the whole picture | 21:29 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance and InstanceList v2.0 objects https://review.openstack.org/217819 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova: Teach conductor to do manifest-based object_class_action() things https://review.openstack.org/219432 | 21:30 |
| ccarmack | mriedem: are you saying our 4 days in dev/ops class will pay off? | 21:30 |
| alaski | mtreinish: so "if n-cells enabled set these flags"? or does devstack have a way to inject that from the outside? | 21:30 |
| mtreinish | alaski: the former | 21:30 |
| *** achanda has quit IRC | 21:30 | |
| *** promulo has quit IRC | 21:30 | |
| mtreinish | I don't think we have the config injection mechanism working for tempest in devstack | 21:30 |
| mtreinish | like we do with the local.conf and nova | 21:31 |
| bauzas | the iniset? | 21:31 |
| mtreinish | (we also don't use the local.conf in the gate) | 21:31 |
| *** promulo has joined #openstack-nova | 21:31 | |
| alaski | okay | 21:31 |
| mtreinish | bauzas: it should be a manual iniset line in lib/tempest | 21:31 |
| clarkb | I think we can switch to local.conf now that all devstack branches support it fwiw | 21:32 |
| clarkb | that work has never been much of a priority though | 21:32 |
| openstackgerrit | Kevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction https://review.openstack.org/186617 | 21:32 |
| mtreinish | clarkb: yeah I guess we probably can | 21:32 |
| mriedem | ccarmack: bauzas: alaski: mtreinish: melwitt: https://bugs.launchpad.net/devstack/+bug/1491152 | 21:33 |
| openstack | Launchpad bug 1491152 in OpenStack Compute (nova) "Don't run shelve tests in tempest if cells is enabled" [Undecided,New] | 21:33 |
| bauzas | mtreinish: yeah okay, that's what I understood | 21:33 |
| mriedem | the instructions are in there | 21:33 |
| mriedem | ccarmack: has dibs | 21:33 |
| bauzas | mriedem: looks low-hanging-fruit :) | 21:34 |
| mriedem | it is | 21:34 |
| ccarmack | I call dibs, yes | 21:34 |
| mtreinish | mriedem: there might be a tempest component there too, ie we don't honor the config flag in all the places its needed | 21:34 |
| mriedem | did you verify that? i was about to check | 21:34 |
| mtreinish | although I guess the rc removal patch will catch that | 21:34 |
| mriedem | until we remove those from the nova rc file | 21:35 |
| mriedem | which is part of that bug | 21:35 |
| mriedem | mtreinish: i checked, tempest is ok | 21:35 |
| mriedem | it checks the option in all of the shelve tests | 21:35 |
| mtreinish | ok, cool | 21:36 |
| * mtreinish is kinda surprised | 21:36 | |
| mriedem | me too | 21:36 |
| alaski | bauzas: melwitt https://bugs.launchpad.net/nova/+bug/1491153 | 21:36 |
| openstack | Launchpad bug 1491153 in OpenStack Compute (nova) "Cells: Use tempest configs to turn off groups of tests that can't pass" [Wishlist,Confirmed] | 21:36 |
| mriedem | ccarmack: join this bug team https://launchpad.net/~devstack-drivers | 21:36 |
| mriedem | so i can assign the devstack part of this to you | 21:36 |
| mtreinish | mriedem: you should be able to assign bugs to an lp user | 21:37 |
| melwitt | alaski: thanks for the bug | 21:37 |
| ccarmack | mriedem: my request is awaiting approval | 21:37 |
| mriedem | mtreinish: nope | 21:37 |
| mriedem | mtreinish: i'm not on the bug team | 21:37 |
| *** rlrossit has left #openstack-nova | 21:37 | |
| bauzas | alaski: meh, the remote call to the nova repo for setting the flags is doable but probably not nice :) | 21:39 |
| ccarmack | mriedem: my request to join https://launchpad.net/~devstack-drivers is waiting for approval | 21:40 |
| mriedem | yeah me too | 21:41 |
| alaski | bauzas: yeah, that may not be important as long as we have the regex | 21:41 |
| mtreinish | mriedem: I excpect dtroyer controls that group | 21:42 |
| mtreinish | probably with an iron fist too | 21:42 |
| *** SourabhP has quit IRC | 21:42 | |
| mriedem | sounds right | 21:43 |
| mriedem | i always think of a hawaiian shirt when i think of dtroyer | 21:44 |
| mriedem | i'm not sure if that's something he's actually ever worn though | 21:44 |
| *** mriedem is now known as mriedem_away | 21:46 | |
| *** ssurana1 has joined #openstack-nova | 21:48 | |
| *** ssurana has quit IRC | 21:48 | |
| *** smatzek has quit IRC | 21:48 | |
| *** annegentle has quit IRC | 21:48 | |
| *** burgerk has quit IRC | 21:51 | |
| *** alex_klimov has quit IRC | 21:51 | |
| *** ccarmack has left #openstack-nova | 21:52 | |
| *** meghal has quit IRC | 21:55 | |
| *** tri2sing has quit IRC | 21:57 | |
| *** asettle has joined #openstack-nova | 21:57 | |
| *** changbl has quit IRC | 21:57 | |
| *** meghal has joined #openstack-nova | 21:58 | |
| *** Piet has quit IRC | 22:01 | |
| mikal | Morning | 22:06 |
| bauzas | g'night everyone, heading off | 22:07 |
| *** jcoufal has quit IRC | 22:09 | |
| *** ctrath has quit IRC | 22:09 | |
| *** ssurana1 has quit IRC | 22:11 | |
| *** SourabhP has joined #openstack-nova | 22:11 | |
| *** VW_ has quit IRC | 22:12 | |
| *** tri2sing has joined #openstack-nova | 22:14 | |
| *** smatzek has joined #openstack-nova | 22:14 | |
| *** tri2sing has quit IRC | 22:16 | |
| *** mdrabe has quit IRC | 22:17 | |
| *** smatzek has quit IRC | 22:20 | |
| *** ssurana has joined #openstack-nova | 22:22 | |
| *** tri2sing has joined #openstack-nova | 22:22 | |
| *** ssurana has quit IRC | 22:22 | |
| *** tri2sing has quit IRC | 22:24 | |
| *** tri2sing has joined #openstack-nova | 22:24 | |
| *** tri2sing has quit IRC | 22:26 | |
| *** mwagner_lap has quit IRC | 22:26 | |
| *** edmondsw has quit IRC | 22:28 | |
| *** tri2sing has joined #openstack-nova | 22:28 | |
| *** earlephilhower has joined #openstack-nova | 22:32 | |
| *** zzzeek has quit IRC | 22:36 | |
| *** emagana has quit IRC | 22:36 | |
| *** jaypipes has quit IRC | 22:40 | |
| *** annegentle has joined #openstack-nova | 22:40 | |
| *** meghal has quit IRC | 22:41 | |
| *** meghal has joined #openstack-nova | 22:41 | |
| *** meghal has quit IRC | 22:41 | |
| *** meghal has joined #openstack-nova | 22:41 | |
| *** meghal has quit IRC | 22:43 | |
| *** alejandrito_ has joined #openstack-nova | 22:43 | |
| *** alejandrito_ has quit IRC | 22:45 | |
| *** alejandrito has quit IRC | 22:45 | |
| *** smatzek has joined #openstack-nova | 22:46 | |
| *** tpeoples has quit IRC | 22:48 | |
| *** baoli has joined #openstack-nova | 22:48 | |
| melwitt | does anyone know if it's expected for an instance to be able to go from power state 0 (None) to 3 (Paused) to 1 (Running) during a create? https://bugs.launchpad.net/nova/+bug/1486475 | 22:50 |
| openstack | Launchpad bug 1486475 in tempest "Transient mismatch between OS-EXT-STS:vm_state and OS-EXT-STS:power_state" [Undecided,New] | 22:50 |
| *** meghal has joined #openstack-nova | 22:51 | |
| *** meghal1 has joined #openstack-nova | 22:52 | |
| *** meghal has quit IRC | 22:52 | |
| *** achanda has joined #openstack-nova | 22:54 | |
| *** Sukhdev has joined #openstack-nova | 22:55 | |
| *** boris-42 has joined #openstack-nova | 22:58 | |
| *** apuimedo|lunch is now known as apuimedo | 22:58 | |
| *** tri2sing has quit IRC | 22:58 | |
| *** achanda has quit IRC | 22:59 | |
| *** jogo has quit IRC | 22:59 | |
| *** vivekd has quit IRC | 23:00 | |
| *** vivekd has joined #openstack-nova | 23:01 | |
| *** ijuwang has quit IRC | 23:02 | |
| *** dims has quit IRC | 23:03 | |
| *** jogo has joined #openstack-nova | 23:05 | |
| *** jogo has quit IRC | 23:05 | |
| *** jogo has joined #openstack-nova | 23:05 | |
| *** asselin has joined #openstack-nova | 23:06 | |
| *** Piet has joined #openstack-nova | 23:08 | |
| openstackgerrit | Sachi King proposed openstack/nova: Add constraint target to tox.ini https://review.openstack.org/205931 | 23:11 |
| *** tri2sing has joined #openstack-nova | 23:13 | |
| *** asselin has quit IRC | 23:14 | |
| *** asselin has joined #openstack-nova | 23:14 | |
| *** bkopilov has quit IRC | 23:17 | |
| *** bkopilov has joined #openstack-nova | 23:18 | |
| *** mdbooth has quit IRC | 23:21 | |
| *** tonytan4ever has quit IRC | 23:22 | |
| *** ZZelle_ has quit IRC | 23:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/216968 | 23:24 |
| *** dims__ has joined #openstack-nova | 23:24 | |
| *** mdbooth has joined #openstack-nova | 23:26 | |
| *** dannywil_ has joined #openstack-nova | 23:30 | |
| *** dannywilson has quit IRC | 23:33 | |
| *** diogogmt has quit IRC | 23:40 | |
| *** diogogmt has joined #openstack-nova | 23:43 | |
| *** salv-orlando has joined #openstack-nova | 23:43 | |
| gmann | sdague: Thanks. looks like you have all up in gerrit :). ll check if anything remaining cleanup part etc | 23:43 |
| *** diogogmt has quit IRC | 23:45 | |
| *** salv-orl_ has quit IRC | 23:47 | |
| *** choyj has quit IRC | 23:47 | |
| *** shoutm has joined #openstack-nova | 23:48 | |
| *** y_sawai has joined #openstack-nova | 23:48 | |
| *** IanGovett has joined #openstack-nova | 23:50 | |
| *** hemna is now known as hemnafk | 23:51 | |
| *** mnestratov has quit IRC | 23:53 | |
| *** jhesketh has quit IRC | 23:54 | |
| *** jhesketh has joined #openstack-nova | 23:55 | |
| *** y_sawai has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!