Monday, 2016-08-22

*** zhurong has quit IRC00:06
*** zhurong has joined #senlin01:13
*** yanyanhu has joined #senlin01:24
*** Drago has joined #senlin01:37
*** Drago has joined #senlin01:38
*** jdandrea has quit IRC02:04
*** elynn has joined #senlin02:16
*** Drago has quit IRC02:28
*** Drago has joined #senlin02:31
*** yuanying has quit IRC02:46
*** yuanying has joined #senlin02:50
Qiminglixinhui, the latest change still contains some blank lines at the end of the template, maybe that is the reason pep still fails03:27
openstackgerritQiming Teng proposed openstack/senlin: Remove usage of mox/mox3  https://review.openstack.org/35838303:41
*** yuanying has quit IRC03:45
lixinhuiQiming, done03:47
*** yuanying has joined #senlin03:48
Qimingthx!03:49
openstackgerritYanyan Hu proposed openstack/senlin: Add version support to schema and spec  https://review.openstack.org/34870903:56
*** elynn has quit IRC04:35
*** elynn has joined #senlin04:57
Qiminghi, just submitted a patch to oslo.db to fix the node list problem06:09
Qimingwhen the created_at field is None, oslo.db is messing things up by doing a '<' or '>' comparison on it06:10
Qiminghttps://review.openstack.org/35841006:10
Qimingelynn, there?06:12
elynnQiming, yes06:12
Qiminghi, what's the plan to get profile-validate and policy-validate added to sdk?06:13
Qimingwe will need that asap before 0.9.3 is cut06:13
QimingI don't think we will have a chance to release a 0.9.4 in coming weeks06:14
elynnIf I submit a patch to add it today, will it get merged in 0.9.3?06:14
Qimingthat chance is big06:14
elynnokay, then today I will submit patches to sdk06:14
Qiminggreat, thanks06:15
elynnand also add api for policy validation.06:15
elynnMight be tonight.06:15
yanyanhuhi, Qiming, thanks a lot for proposing this patch 35841006:19
yanyanhuthere is bug reported submitted before: https://bugs.launchpad.net/senlin/+bug/159435206:19
openstackLaunchpad bug 1594352 in senlin "could not list node sorted by "created_at:desc" " [High,Triaged]06:19
Qimingyes, was proposing this to fix that bug06:19
yanyanhumaybe we can add the link to the commit msg06:19
yanyanhuthanks06:19
Qimingwe don't have a bug report to oslo.db06:20
yanyanhuoh, right06:20
yanyanhuthat bug is reported in senlin side06:20
Qimingbut I have added oslo.db as an affected project06:20
yanyanhucool,06:20
yanyanhuactually, ruijie from ddtdream asked me for this problem last week06:20
yanyanhuI think this will be helpful for him as well06:21
yanyanhuthey met this issue when testing senlin06:21
Qimingyep06:25
Qimingthey can have a local fix before oslo.db merges it06:25
yanyanhuyes, will make them to be aware of this fix06:34
openstackgerritQiming Teng proposed openstack/senlin: Improve server metadata format  https://review.openstack.org/35630206:56
yanyanhuhi, Qiming I noticed gate jobs all failed today07:03
Qimingyes07:03
yanyanhuI checked the log and found it was caused by weird request from keystoneauth to 127.0.0.1 url07:03
yanyanhunot sure why it happened since that is the home page of dashboard07:04
Qimingwe have got quite some patches not reviewed before07:04
Qimingnow blocked by the gate ...07:04
Qimingit is a bug in sdk07:04
yanyanhuI see...07:05
Qiminghttps://review.openstack.org/#/c/354492/07:05
Qimingfixed by this one07:05
Qimingbut it is not part of 0.9.2 ...07:05
yanyanhuI see07:05
yanyanhuhope sdk 0.9.3 comes soon07:06
Qimingbut if sdk cuts 0.9.3 today, then we won't have a chance to get profile/policy validate out07:06
yanyanhuyes, ethan still need some time to finish it07:07
yanyanhuso 0.9.3 will be the final release of sdk in N cycle?07:07
yanyanhufor feature freeze is coming07:07
QimingI am not sure we can persuade brian to do another release07:09
*** zhurong has quit IRC07:09
Qiminghe has got pretty short a window too07:09
*** zhurong has joined #senlin07:13
yanyanhuI see07:22
openstackgerritQiming Teng proposed openstack/senlin: Close health manager loop  https://review.openstack.org/35630307:27
openstackgerritQiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN  https://review.openstack.org/35845407:43
openstackgerritQiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN  https://review.openstack.org/35845408:18
openstackgerritQiming Teng proposed openstack/senlin: Improve server metadata format  https://review.openstack.org/35630208:18
openstackgerritQiming Teng proposed openstack/senlin: Close health manager loop  https://review.openstack.org/35630308:18
openstackgerritQiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN  https://review.openstack.org/35845408:22
openstackgerritQiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN  https://review.openstack.org/35845408:27
*** yuanying has quit IRC08:34
*** yuanying has joined #senlin08:35
*** yuanying has quit IRC08:47
*** yuanying has joined #senlin08:49
openstackgerritEthan Lynn proposed openstack/senlin: Add new api for profile validation  https://review.openstack.org/35454908:56
openstackgerritEthan Lynn proposed openstack/senlin: Fix do_validate in heat profile  https://review.openstack.org/35578008:56
openstackgerritYanyan Hu proposed openstack/senlin: Add version support to schema and spec  https://review.openstack.org/34870909:17
*** yanyanhu has quit IRC09:21
elynnCan't push patches to review, will go home an continue....09:48
openstackgerritQiming Teng proposed openstack/senlin: Enable health policy to handle CLUSTER_SCALE_IN  https://review.openstack.org/35845409:49
*** elynn has quit IRC09:49
Qimingelynn, what happened?09:53
Qimingshit ... the whole git history is now corrupted09:54
Qimingno one ever helped review the health manager patch ...09:54
Qimingspent a whole damn day on fixing this ...09:55
*** zhurong has quit IRC09:58
*** xuhaiwei has quit IRC10:40
*** lixinhui has quit IRC10:40
*** lixinhui has joined #senlin10:40
*** openstackgerrit has quit IRC11:03
*** openstackgerrit has joined #senlin11:04
*** ChanServ sets mode: +v openstackgerrit11:04
*** openstackstatus has quit IRC11:36
*** openstackstatus has joined #senlin11:39
*** ChanServ sets mode: +v openstackstatus11:39
*** shu-mutou-OFF has quit IRC12:08
*** rvasilets__ has left #senlin12:45
openstackgerritMerged openstack/python-senlinclient: Reorder required parameters  https://review.openstack.org/35410012:58
*** elynn has joined #senlin13:24
*** elynn has quit IRC13:26
*** elynn has joined #senlin13:31
elynnHi13:38
elynndo you know why our jenkins were failed?13:39
*** elynn has quit IRC13:52
*** elynn has joined #senlin14:02
*** elynn has quit IRC14:06
*** elynn has joined #senlin14:08
*** elynn has quit IRC14:17
*** elynn has joined #senlin14:22
openstackgerritEthan Lynn proposed openstack/senlin: Add policy_validate function to engine  https://review.openstack.org/35868614:42
Qimingelynn, because sdk has a buggy patch merged I think15:09
elynnokay, so we are waiting for new sdk version?15:09
Qimingyes15:10
Qimingyou don't have to worry about the gate15:11
Qimingjust push the changes needed for profile/policy validate15:11
elynnyes, I will, working on it now.15:11
*** Drago has joined #senlin15:11
Qimingthen when sdk has a 0.9.3 release, everything will be fine15:12
Qimingspeaking of validation15:12
QimingI'm wondering if they are meant to be synchronous calls15:12
Qimingis stack-preview a synchronous call?15:12
elynnstack-preview is synchronous15:13
Qimingokay15:13
elynnNo other way to check result15:13
Qiminglet's have validation synchronous as well15:13
Qimingit doesn't make a lot senses to make them asynchronous15:13
elynnokay~15:14
Qimingit feels very strange if users have to check the action to find out if validation passes15:14
DragoJust wanted to note, the Heat team has discussed making stack preview asynchronous because for very large stacks, validation can take a long time (minutes)15:15
elynnHmm... interesting...15:16
DragoThe plan was to validate just enough synchronously to know if the Heat template was well-formed enough that Heat could store *something* in the database for the user to check back to15:16
DragoThis was discussed at the Austin summit. I don't know if has been implemented yet15:17
elynnI would say that it's better to make validation process parallels, then it would get faster ;)15:18
elynnHaven't seen any changes on validation yet...15:18
DragoSee https://etherpad.openstack.org/p/heat-newton-validation-improvements15:18
elynnDrago, thanks for remind :)15:21
elynnSeems they are talking about speed up validation process, so  that user can get a quick api response.15:24
Qimingthanks, drago, there is always a gray area when we want to draw a line between "well-formed" and "working"15:24
DragoYes, the rationale was "just enough that we can store something in the DB"15:30
Qimingsee you guys, going to bed now, has been a long day ...15:31
elynnsee u~15:40
openstackgerritEthan Lynn proposed openstack/senlin: Add new api for policy validation  https://review.openstack.org/35875916:34
*** Drago has quit IRC16:55
*** elynn has quit IRC17:02
*** Drago has joined #senlin18:08
*** Drago has quit IRC18:29
*** Drago has joined #senlin18:29
*** Drago has quit IRC18:30
*** Drago has joined #senlin18:31
*** Drago1 has joined #senlin19:35
*** Drago has quit IRC19:36
*** Drago1 is now known as Drago20:36
*** Drago has quit IRC21:03
*** Drago has joined #senlin21:03
*** Drago has quit IRC21:43
*** Drago has joined #senlin21:43
*** Drago has quit IRC22:10
*** Drago has joined #senlin22:11
*** elynn has joined #senlin22:59
*** haiwei has joined #senlin23:31
*** shu-mutou has joined #senlin23:42
*** Drago has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!