Thursday, 2015-10-08

*** openstackgerrit has quit IRC00:01
*** openstackgerrit has joined #openstack-mistral00:02
*** xylan_kong has joined #openstack-mistral00:08
xylan_konghi, guys, i'm back from vacation, start to work00:18
*** achanda has quit IRC00:21
*** achanda has joined #openstack-mistral00:26
*** achanda has quit IRC00:31
*** achanda has joined #openstack-mistral00:36
*** achanda has quit IRC00:37
*** achanda has joined #openstack-mistral00:40
*** bobh has joined #openstack-mistral00:41
*** bobh has quit IRC00:44
*** harlowja has quit IRC00:44
*** bobh has joined #openstack-mistral00:47
*** harlowja has joined #openstack-mistral00:59
*** bobh has quit IRC01:48
*** zhenguo_ has joined #openstack-mistral01:58
*** zhenguo_ is now known as zhenguo01:58
openstackgerritWinson Chan proposed openstack/python-mistralclient: Fix support of dev use case with no auth  https://review.openstack.org/23230602:23
openstackgerritMerged openstack/mistral: Update AUTHORS file  https://review.openstack.org/23090202:53
*** tristanC has quit IRC03:10
*** tristanC has joined #openstack-mistral03:12
*** achanda has quit IRC04:11
*** hparekh has joined #openstack-mistral04:18
*** achanda has joined #openstack-mistral04:59
*** noa_koffman has quit IRC06:13
rakhmerovxylan_kong: how was your vacation? Glad you're back06:23
rakhmerovwe need you very much :)06:23
xylan_kongrakhmerov: hi, how are you06:23
xylan_kongrakhmerov: i'm just stay at home with my kid during the whole holiday06:24
rakhmerovI'm more or less ok06:24
rakhmerovok, it's not a bad thing also )06:24
rakhmerovto spend time with a kid06:24
xylan_kongyeah, i'm happy with him06:24
rakhmerov:)06:24
rakhmerovgood06:24
xylan_kongso, rakhmerov, what's our plan for Liberty?06:24
xylan_kongwhen release rc2?06:25
rakhmerovwe're doing some final patches now and about to start massive QA work06:25
rakhmerovmost likely next week06:25
xylan_kongrakhmerov: yeah, i have already found a critical bug, which is filed by Winson06:25
rakhmerovI created stable/liberty based on RC1 so we'll need to backport the most critical patches into it and make an official release out of it06:26
rakhmerovwhich one?06:26
rakhmerovwhat bug?06:26
xylan_konghttps://bugs.launchpad.net/mistral/+bug/150391506:26
openstackLaunchpad bug 1503915 in Mistral "mistralclient does not work without auth" [Critical,Confirmed] - Assigned to Lingxian Kong (kong)06:26
rakhmerovooh, ok06:27
xylan_kongI can't use mistral without Keystone now06:27
rakhmerovyeah, we need to fix that06:27
xylan_kongrakhmerov: i'm doing that06:27
rakhmerovare you going to work on it, right?06:27
rakhmerovok06:27
rakhmerovcool06:27
rakhmerovalso, just FYI: significant part of time we're now involved in our internal things06:29
rakhmerovso there won't be too many patches from us in the next couple of things06:29
xylan_kongrakhmerov: ok, got you06:30
rakhmerovxylan_kong: I also want to ask you to review and approve a bunch patches sent by me, they are very simple06:30
rakhmerovAUTHORS and .mailmap06:30
xylan_kongrakhmerov: sure, will do that06:30
xylan_kongrakhmerov: btw, what about ALU guys, I didn't see patches from them these days06:30
xylan_kongand I heard ALU was brought by Nokia06:31
xylan_kongbought06:31
rakhmerovxylan_kong: I'm not figuring out their status06:34
rakhmerovI know they had very very long holidays06:34
rakhmerovand I'm not sure they're now over06:34
xylan_kongrakhmerov: ok, thanks for the info06:35
openstackgerrithardik proposed openstack/mistral: Check for trigger before delete wf.  https://review.openstack.org/23199307:06
openstackgerritLingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body  https://review.openstack.org/23239508:04
*** sergmelikyan has joined #openstack-mistral08:07
*** achanda has quit IRC08:07
openstackgerritLingxian Kong proposed openstack/mistral: Delivering error message via header in pecan.abort  https://review.openstack.org/23239808:08
*** achanda has joined #openstack-mistral08:13
*** achanda has quit IRC08:15
rakhmerovxylan_kong, nmakhotkin: please review https://review.openstack.org/#/c/232306/08:18
rakhmerovit's very important08:18
xylan_kongrakhmerov: ...08:23
rakhmerov?08:23
xylan_kongrakhmerov:  I have another patch for that bug. Winson submitted his patch without assigning bug to him08:24
nmakhotkinxylan_kong, hi!08:24
nmakhotkinAFAIK, misralclient should work without auth if OS_AUTH_URL is empty08:24
rakhmerovyeah, but he already sent it08:24
xylan_kongnmakhotkin: hi08:24
rakhmerovsorry08:24
rakhmerovnmakhotkin: yes, that's exactly what this patch does08:25
xylan_kongnmakhotkin: agree08:25
xylan_kongrakhmerov: no, i don't think so08:25
rakhmerovxylan_kong: what do you mean? Anything wrong with this patch?08:25
rakhmerovxylan_kong: can you pls explain?08:25
rakhmerovwe can't set a default value when we define parameters, right?08:26
rakhmerovbecause for the rest of the code it would as if it was provided by user08:27
rakhmerovso that's why this "default" value is removed which is right, IMO08:27
rakhmerovinstead we should manually check if --os-auth-url is really provided by a user08:27
*** sergmelikyan has quit IRC08:27
rakhmerovseems reasonable08:27
openstackgerritMerged openstack/python-mistralclient: Fix support of dev use case with no auth  https://review.openstack.org/23230608:29
xylan_kongwith winson's patch, if i didn't provide auth_url, and mistral service has config 'auth_enable=True', I still can see the html body08:30
xylan_kongthe bug https://bugs.launchpad.net/python-mistralclient/+bug/1502840 is still no resolved08:30
openstackLaunchpad bug 1502840 in python-mistralclient "Http Response is shown, In case of 'OS_AUTH_URL' is not set." [Undecided,Fix committed] - Assigned to hardik (hardik-parekh047)08:30
xylan_kongif so, why we need winson's patch?08:31
xylan_kongwe can just revert https://review.openstack.org/#/c/230981/08:32
rakhmerovwait a sec08:32
rakhmerovbut that's actually a different thing08:32
rakhmerovyes, mostly Winson's patch is a revert of that patch, but not completely08:33
rakhmerovbut this bug describes a different issue08:34
rakhmerovit has nothing to do with auth_enabled=True which is a conf prop on a server side08:34
rakhmerovso I just wanna see your patch, xylan_kong, I think these two issues are complementary08:35
*** LimorStotland has quit IRC08:35
xylan_kongrakhmerov: winson's bug coms from the patch https://review.openstack.org/#/c/230981/. the problem he described is caused by that patch08:36
*** LimorStotland has joined #openstack-mistral08:36
xylan_kongwithout https://review.openstack.org/#/c/230981/, what's the problems there?08:37
hparekhxylan_kong: hi for  https://review.openstack.org/#/c/230981/08:37
hparekhwe can check whther user name and password is provided or not.08:37
hparekhif it is provided then we should set auth_url08:37
rakhmerovxylan_kong: yes, Winson just reverts that patch and adds one more condition for password08:38
rakhmerovwhich makes sense IMO08:38
xylan_konghparekh, rakhmerov : but what if keystone service is not on localhost?08:38
rakhmerovso it's fine08:38
rakhmerovthen just specify where it is08:38
hparekhyeah08:39
hparekhagree08:39
rakhmerovxylan_kong: what I'm trying to say is that, it seems to me that 1) there's no problem with Winson's patch 2) the problem you're working on is orthogonal and it also needs to be fixed08:39
rakhmerovi mean that html body thing..08:39
xylan_kongrakhmerov: I have an 'ugly' patch, we can discuss on that https://review.openstack.org/232398 and https://review.openstack.org/23239808:40
xylan_kongbtw, I hope we do things right. we can assign bug to someone, to avoid effort duplication08:41
openstackgerritMerged openstack/mistral: Replace copy.copy with copy.deepcopy  https://review.openstack.org/23218008:43
openstackgerritLingxian Kong proposed openstack/mistral: Delivering error message via header in pecan.abort  https://review.openstack.org/23239808:43
*** sergmelikyan has joined #openstack-mistral08:44
rakhmerovyeah08:44
rakhmerovthat's ok08:44
openstackgerritLingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body  https://review.openstack.org/23239508:52
rakhmerovxylan_kong: can you add a doc for Service API?09:01
rakhmerovthis is a new functionality but it's not described anywhere yet09:01
rakhmerovif yes, pls add a BP for that09:02
xylan_kongrakhmerov: ok, I'll try, use my terrible English :-)09:02
rakhmerovit's ok, we'll get it corrected if needed )09:02
rakhmerovanyway, we all need to keep improving our skills09:02
xylan_kongno problem09:02
rakhmerovthnx09:02
rakhmerovxylan_kong: a little bit off topic. I carefully read your PTL candidacy email and I have to admit that I agree with a number of important things that you told about09:03
rakhmerovparticularly, openstackclient, spec repository etc. etc.09:03
xylan_kongrakhmerov: thanks for reading, i just want mistral to keep up with the whole community09:04
rakhmerovjust letting you know that I'm not going to forget those things as well and what I'd like to ask you about is to file necessary BPs for all these things so that we could prioritize them09:04
rakhmerovyes, and this is right09:04
rakhmerovI agree with that09:04
xylan_kongrakhmerov: very late congratulations, btw09:05
rakhmerovthe main reason why we haven't done some thing is just resources mostly09:05
rakhmerovooh, thanks09:05
rakhmerovso my point is: thank you very much for all these statements, I hope we'll be working together on all those things09:06
xylan_kongrakhmerov: yes, i also hope so09:06
rakhmerovgood09:06
rakhmerovyou're going to Tokyo, right?09:06
xylan_kongrakhmerov: yeah09:06
rakhmerovI just got my Japanese visa btw09:07
rakhmerovyesterday09:07
rakhmerovso I'm going too09:07
xylan_kongrakhmerov:  greate09:07
rakhmerovyep09:07
xylan_kongwill see you there09:07
xylan_kongI've already get my visa last month09:07
rakhmerovcool09:07
rakhmerovxylan_kong: btw, you can use this etherpad to collect topics for the summit: https://etherpad.openstack.org/p/mistral-tokyo-summit-201509:08
rakhmerovthere's already some in it09:08
xylan_kongrakhmerov: i saw it earlier, but i have not think about it systematically09:08
rakhmerovnp, we still have time to think and shape it out09:09
openstackgerritMerged openstack/mistral-dashboard: Update AUTHORS file  https://review.openstack.org/23090309:09
openstackgerritMerged openstack/mistral-dashboard: Add .mailmap for pbr AUTHORS generation  https://review.openstack.org/23094809:09
*** achanda has joined #openstack-mistral09:16
*** LimorStotland has quit IRC09:16
*** LimorStotland has joined #openstack-mistral09:18
*** LimorStotland_ has joined #openstack-mistral09:20
rakhmerovxylan_kong: Lingxian, can you pls review and approve other patches with AUTHORS and .mailmap? 4 of them09:21
*** achanda has quit IRC09:21
*** LimorStotland has quit IRC09:22
openstackgerritLingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body  https://review.openstack.org/23239510:22
xylan_kongrakhmerov:  i've already done that :-)10:24
openstackgerritMerged openstack/python-mistralclient: Update AUTHORS file  https://review.openstack.org/23091010:31
openstackgerritMerged openstack/python-mistralclient: Add .mailmap for pbr AUTHORS generation  https://review.openstack.org/23094510:32
*** ^Gal^ has quit IRC10:51
*** ^Gal^ has joined #openstack-mistral10:53
*** ^Gal^ has quit IRC11:04
rakhmerovxylan_kong: two left )11:18
*** achanda has joined #openstack-mistral11:19
nmakhotkinxylan_kong, are you here?11:22
nmakhotkinxylan_kong: could you please review https://review.openstack.org/#/c/230415/1 and https://review.openstack.org/#/c/231426/11:23
nmakhotkin?11:23
*** bobh has joined #openstack-mistral11:25
*** achanda has quit IRC11:26
*** sergmelikyan has quit IRC11:44
*** ^Gal^ has joined #openstack-mistral11:48
*** sergmelikyan has joined #openstack-mistral11:55
*** bobh has quit IRC12:27
openstackgerrithardik proposed openstack/mistral: Check for trigger before delete wf.  https://review.openstack.org/23199312:31
openstackgerritNikolay Mahotkin proposed openstack/mistral: Adding functional tests for SSH actions  https://review.openstack.org/23250712:35
*** sergmelikyan has quit IRC12:36
*** ^Gal^ has quit IRC12:37
*** ^Gal^ has joined #openstack-mistral12:39
*** sergmelikyan has joined #openstack-mistral12:45
*** sergmelikyan has quit IRC12:59
*** bobh has joined #openstack-mistral13:14
*** achanda has joined #openstack-mistral13:24
*** achanda has quit IRC13:27
*** achanda has joined #openstack-mistral13:27
*** achanda has quit IRC13:27
*** hparekh has quit IRC13:41
*** hparekh has joined #openstack-mistral13:47
*** sergmelikyan has joined #openstack-mistral13:59
*** sergmelikyan has quit IRC14:06
*** ramishra has joined #openstack-mistral14:07
*** hparekh has quit IRC14:19
*** Piet has joined #openstack-mistral14:25
*** achanda has joined #openstack-mistral14:28
*** achanda has quit IRC14:38
*** tonytan4ever has joined #openstack-mistral14:40
*** bobh has quit IRC14:42
*** achanda has joined #openstack-mistral14:59
*** achanda has quit IRC15:02
*** tonytan4ever has quit IRC15:37
*** gyee has joined #openstack-mistral15:46
*** bobh has joined #openstack-mistral15:55
*** sergmelikyan has joined #openstack-mistral15:59
*** LimorStotland_ has quit IRC16:08
*** sergmelikyan has quit IRC16:46
*** achanda has joined #openstack-mistral17:20
*** tonytan4ever has joined #openstack-mistral17:24
*** sergmelikyan has joined #openstack-mistral17:46
*** sergmelikyan has quit IRC17:52
*** sergmelikyan has joined #openstack-mistral18:02
*** sergmelikyan has quit IRC18:48
*** sergmelikyan has joined #openstack-mistral18:58
*** ryanpetrello has joined #openstack-mistral19:34
*** sergmelikyan has quit IRC19:58
*** tonytan4ever has quit IRC20:03
*** tonytan4ever has joined #openstack-mistral20:09
*** ig0r_ has joined #openstack-mistral20:58
*** ig0r_ has quit IRC21:03
*** ig0r_ has joined #openstack-mistral21:05
*** ig0r_ has quit IRC21:10
*** tonytan4ever has quit IRC21:36
*** bobh has quit IRC21:43
*** harlowja has quit IRC23:52
*** harlowja has joined #openstack-mistral23:52

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