*** openstackgerrit has quit IRC | 00:01 | |
*** openstackgerrit has joined #openstack-mistral | 00:02 | |
*** xylan_kong has joined #openstack-mistral | 00:08 | |
xylan_kong | hi, guys, i'm back from vacation, start to work | 00:18 |
---|---|---|
*** achanda has quit IRC | 00:21 | |
*** achanda has joined #openstack-mistral | 00:26 | |
*** achanda has quit IRC | 00:31 | |
*** achanda has joined #openstack-mistral | 00:36 | |
*** achanda has quit IRC | 00:37 | |
*** achanda has joined #openstack-mistral | 00:40 | |
*** bobh has joined #openstack-mistral | 00:41 | |
*** bobh has quit IRC | 00:44 | |
*** harlowja has quit IRC | 00:44 | |
*** bobh has joined #openstack-mistral | 00:47 | |
*** harlowja has joined #openstack-mistral | 00:59 | |
*** bobh has quit IRC | 01:48 | |
*** zhenguo_ has joined #openstack-mistral | 01:58 | |
*** zhenguo_ is now known as zhenguo | 01:58 | |
openstackgerrit | Winson Chan proposed openstack/python-mistralclient: Fix support of dev use case with no auth https://review.openstack.org/232306 | 02:23 |
openstackgerrit | Merged openstack/mistral: Update AUTHORS file https://review.openstack.org/230902 | 02:53 |
*** tristanC has quit IRC | 03:10 | |
*** tristanC has joined #openstack-mistral | 03:12 | |
*** achanda has quit IRC | 04:11 | |
*** hparekh has joined #openstack-mistral | 04:18 | |
*** achanda has joined #openstack-mistral | 04:59 | |
*** noa_koffman has quit IRC | 06:13 | |
rakhmerov | xylan_kong: how was your vacation? Glad you're back | 06:23 |
rakhmerov | we need you very much :) | 06:23 |
xylan_kong | rakhmerov: hi, how are you | 06:23 |
xylan_kong | rakhmerov: i'm just stay at home with my kid during the whole holiday | 06:24 |
rakhmerov | I'm more or less ok | 06:24 |
rakhmerov | ok, it's not a bad thing also ) | 06:24 |
rakhmerov | to spend time with a kid | 06:24 |
xylan_kong | yeah, i'm happy with him | 06:24 |
rakhmerov | :) | 06:24 |
rakhmerov | good | 06:24 |
xylan_kong | so, rakhmerov, what's our plan for Liberty? | 06:24 |
xylan_kong | when release rc2? | 06:25 |
rakhmerov | we're doing some final patches now and about to start massive QA work | 06:25 |
rakhmerov | most likely next week | 06:25 |
xylan_kong | rakhmerov: yeah, i have already found a critical bug, which is filed by Winson | 06:25 |
rakhmerov | I 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 it | 06:26 |
rakhmerov | which one? | 06:26 |
rakhmerov | what bug? | 06:26 |
xylan_kong | https://bugs.launchpad.net/mistral/+bug/1503915 | 06:26 |
openstack | Launchpad bug 1503915 in Mistral "mistralclient does not work without auth" [Critical,Confirmed] - Assigned to Lingxian Kong (kong) | 06:26 |
rakhmerov | ooh, ok | 06:27 |
xylan_kong | I can't use mistral without Keystone now | 06:27 |
rakhmerov | yeah, we need to fix that | 06:27 |
xylan_kong | rakhmerov: i'm doing that | 06:27 |
rakhmerov | are you going to work on it, right? | 06:27 |
rakhmerov | ok | 06:27 |
rakhmerov | cool | 06:27 |
rakhmerov | also, just FYI: significant part of time we're now involved in our internal things | 06:29 |
rakhmerov | so there won't be too many patches from us in the next couple of things | 06:29 |
xylan_kong | rakhmerov: ok, got you | 06:30 |
rakhmerov | xylan_kong: I also want to ask you to review and approve a bunch patches sent by me, they are very simple | 06:30 |
rakhmerov | AUTHORS and .mailmap | 06:30 |
xylan_kong | rakhmerov: sure, will do that | 06:30 |
xylan_kong | rakhmerov: btw, what about ALU guys, I didn't see patches from them these days | 06:30 |
xylan_kong | and I heard ALU was brought by Nokia | 06:31 |
xylan_kong | bought | 06:31 |
rakhmerov | xylan_kong: I'm not figuring out their status | 06:34 |
rakhmerov | I know they had very very long holidays | 06:34 |
rakhmerov | and I'm not sure they're now over | 06:34 |
xylan_kong | rakhmerov: ok, thanks for the info | 06:35 |
openstackgerrit | hardik proposed openstack/mistral: Check for trigger before delete wf. https://review.openstack.org/231993 | 07:06 |
openstackgerrit | Lingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body https://review.openstack.org/232395 | 08:04 |
*** sergmelikyan has joined #openstack-mistral | 08:07 | |
*** achanda has quit IRC | 08:07 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral: Delivering error message via header in pecan.abort https://review.openstack.org/232398 | 08:08 |
*** achanda has joined #openstack-mistral | 08:13 | |
*** achanda has quit IRC | 08:15 | |
rakhmerov | xylan_kong, nmakhotkin: please review https://review.openstack.org/#/c/232306/ | 08:18 |
rakhmerov | it's very important | 08:18 |
xylan_kong | rakhmerov: ... | 08:23 |
rakhmerov | ? | 08:23 |
xylan_kong | rakhmerov: I have another patch for that bug. Winson submitted his patch without assigning bug to him | 08:24 |
nmakhotkin | xylan_kong, hi! | 08:24 |
nmakhotkin | AFAIK, misralclient should work without auth if OS_AUTH_URL is empty | 08:24 |
rakhmerov | yeah, but he already sent it | 08:24 |
xylan_kong | nmakhotkin: hi | 08:24 |
rakhmerov | sorry | 08:24 |
rakhmerov | nmakhotkin: yes, that's exactly what this patch does | 08:25 |
xylan_kong | nmakhotkin: agree | 08:25 |
xylan_kong | rakhmerov: no, i don't think so | 08:25 |
rakhmerov | xylan_kong: what do you mean? Anything wrong with this patch? | 08:25 |
rakhmerov | xylan_kong: can you pls explain? | 08:25 |
rakhmerov | we can't set a default value when we define parameters, right? | 08:26 |
rakhmerov | because for the rest of the code it would as if it was provided by user | 08:27 |
rakhmerov | so that's why this "default" value is removed which is right, IMO | 08:27 |
rakhmerov | instead we should manually check if --os-auth-url is really provided by a user | 08:27 |
*** sergmelikyan has quit IRC | 08:27 | |
rakhmerov | seems reasonable | 08:27 |
openstackgerrit | Merged openstack/python-mistralclient: Fix support of dev use case with no auth https://review.openstack.org/232306 | 08:29 |
xylan_kong | with winson's patch, if i didn't provide auth_url, and mistral service has config 'auth_enable=True', I still can see the html body | 08:30 |
xylan_kong | the bug https://bugs.launchpad.net/python-mistralclient/+bug/1502840 is still no resolved | 08:30 |
openstack | Launchpad 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_kong | if so, why we need winson's patch? | 08:31 |
xylan_kong | we can just revert https://review.openstack.org/#/c/230981/ | 08:32 |
rakhmerov | wait a sec | 08:32 |
rakhmerov | but that's actually a different thing | 08:32 |
rakhmerov | yes, mostly Winson's patch is a revert of that patch, but not completely | 08:33 |
rakhmerov | but this bug describes a different issue | 08:34 |
rakhmerov | it has nothing to do with auth_enabled=True which is a conf prop on a server side | 08:34 |
rakhmerov | so I just wanna see your patch, xylan_kong, I think these two issues are complementary | 08:35 |
*** LimorStotland has quit IRC | 08:35 | |
xylan_kong | rakhmerov: winson's bug coms from the patch https://review.openstack.org/#/c/230981/. the problem he described is caused by that patch | 08:36 |
*** LimorStotland has joined #openstack-mistral | 08:36 | |
xylan_kong | without https://review.openstack.org/#/c/230981/, what's the problems there? | 08:37 |
hparekh | xylan_kong: hi for https://review.openstack.org/#/c/230981/ | 08:37 |
hparekh | we can check whther user name and password is provided or not. | 08:37 |
hparekh | if it is provided then we should set auth_url | 08:37 |
rakhmerov | xylan_kong: yes, Winson just reverts that patch and adds one more condition for password | 08:38 |
rakhmerov | which makes sense IMO | 08:38 |
xylan_kong | hparekh, rakhmerov : but what if keystone service is not on localhost? | 08:38 |
rakhmerov | so it's fine | 08:38 |
rakhmerov | then just specify where it is | 08:38 |
hparekh | yeah | 08:39 |
hparekh | agree | 08:39 |
rakhmerov | xylan_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 fixed | 08:39 |
rakhmerov | i mean that html body thing.. | 08:39 |
xylan_kong | rakhmerov: I have an 'ugly' patch, we can discuss on that https://review.openstack.org/232398 and https://review.openstack.org/232398 | 08:40 |
xylan_kong | btw, I hope we do things right. we can assign bug to someone, to avoid effort duplication | 08:41 |
openstackgerrit | Merged openstack/mistral: Replace copy.copy with copy.deepcopy https://review.openstack.org/232180 | 08:43 |
openstackgerrit | Lingxian Kong proposed openstack/mistral: Delivering error message via header in pecan.abort https://review.openstack.org/232398 | 08:43 |
*** sergmelikyan has joined #openstack-mistral | 08:44 | |
rakhmerov | yeah | 08:44 |
rakhmerov | that's ok | 08:44 |
openstackgerrit | Lingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body https://review.openstack.org/232395 | 08:52 |
rakhmerov | xylan_kong: can you add a doc for Service API? | 09:01 |
rakhmerov | this is a new functionality but it's not described anywhere yet | 09:01 |
rakhmerov | if yes, pls add a BP for that | 09:02 |
xylan_kong | rakhmerov: ok, I'll try, use my terrible English :-) | 09:02 |
rakhmerov | it's ok, we'll get it corrected if needed ) | 09:02 |
rakhmerov | anyway, we all need to keep improving our skills | 09:02 |
xylan_kong | no problem | 09:02 |
rakhmerov | thnx | 09:02 |
rakhmerov | xylan_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 about | 09:03 |
rakhmerov | particularly, openstackclient, spec repository etc. etc. | 09:03 |
xylan_kong | rakhmerov: thanks for reading, i just want mistral to keep up with the whole community | 09:04 |
rakhmerov | just 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 them | 09:04 |
rakhmerov | yes, and this is right | 09:04 |
rakhmerov | I agree with that | 09:04 |
xylan_kong | rakhmerov: very late congratulations, btw | 09:05 |
rakhmerov | the main reason why we haven't done some thing is just resources mostly | 09:05 |
rakhmerov | ooh, thanks | 09:05 |
rakhmerov | so my point is: thank you very much for all these statements, I hope we'll be working together on all those things | 09:06 |
xylan_kong | rakhmerov: yes, i also hope so | 09:06 |
rakhmerov | good | 09:06 |
rakhmerov | you're going to Tokyo, right? | 09:06 |
xylan_kong | rakhmerov: yeah | 09:06 |
rakhmerov | I just got my Japanese visa btw | 09:07 |
rakhmerov | yesterday | 09:07 |
rakhmerov | so I'm going too | 09:07 |
xylan_kong | rakhmerov: greate | 09:07 |
rakhmerov | yep | 09:07 |
xylan_kong | will see you there | 09:07 |
xylan_kong | I've already get my visa last month | 09:07 |
rakhmerov | cool | 09:07 |
rakhmerov | xylan_kong: btw, you can use this etherpad to collect topics for the summit: https://etherpad.openstack.org/p/mistral-tokyo-summit-2015 | 09:08 |
rakhmerov | there's already some in it | 09:08 |
xylan_kong | rakhmerov: i saw it earlier, but i have not think about it systematically | 09:08 |
rakhmerov | np, we still have time to think and shape it out | 09:09 |
openstackgerrit | Merged openstack/mistral-dashboard: Update AUTHORS file https://review.openstack.org/230903 | 09:09 |
openstackgerrit | Merged openstack/mistral-dashboard: Add .mailmap for pbr AUTHORS generation https://review.openstack.org/230948 | 09:09 |
*** achanda has joined #openstack-mistral | 09:16 | |
*** LimorStotland has quit IRC | 09:16 | |
*** LimorStotland has joined #openstack-mistral | 09:18 | |
*** LimorStotland_ has joined #openstack-mistral | 09:20 | |
rakhmerov | xylan_kong: Lingxian, can you pls review and approve other patches with AUTHORS and .mailmap? 4 of them | 09:21 |
*** achanda has quit IRC | 09:21 | |
*** LimorStotland has quit IRC | 09:22 | |
openstackgerrit | Lingxian Kong proposed openstack/python-mistralclient: Show exact error message when authentication falied instead of HTML body https://review.openstack.org/232395 | 10:22 |
xylan_kong | rakhmerov: i've already done that :-) | 10:24 |
openstackgerrit | Merged openstack/python-mistralclient: Update AUTHORS file https://review.openstack.org/230910 | 10:31 |
openstackgerrit | Merged openstack/python-mistralclient: Add .mailmap for pbr AUTHORS generation https://review.openstack.org/230945 | 10:32 |
*** ^Gal^ has quit IRC | 10:51 | |
*** ^Gal^ has joined #openstack-mistral | 10:53 | |
*** ^Gal^ has quit IRC | 11:04 | |
rakhmerov | xylan_kong: two left ) | 11:18 |
*** achanda has joined #openstack-mistral | 11:19 | |
nmakhotkin | xylan_kong, are you here? | 11:22 |
nmakhotkin | xylan_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-mistral | 11:25 | |
*** achanda has quit IRC | 11:26 | |
*** sergmelikyan has quit IRC | 11:44 | |
*** ^Gal^ has joined #openstack-mistral | 11:48 | |
*** sergmelikyan has joined #openstack-mistral | 11:55 | |
*** bobh has quit IRC | 12:27 | |
openstackgerrit | hardik proposed openstack/mistral: Check for trigger before delete wf. https://review.openstack.org/231993 | 12:31 |
openstackgerrit | Nikolay Mahotkin proposed openstack/mistral: Adding functional tests for SSH actions https://review.openstack.org/232507 | 12:35 |
*** sergmelikyan has quit IRC | 12:36 | |
*** ^Gal^ has quit IRC | 12:37 | |
*** ^Gal^ has joined #openstack-mistral | 12:39 | |
*** sergmelikyan has joined #openstack-mistral | 12:45 | |
*** sergmelikyan has quit IRC | 12:59 | |
*** bobh has joined #openstack-mistral | 13:14 | |
*** achanda has joined #openstack-mistral | 13:24 | |
*** achanda has quit IRC | 13:27 | |
*** achanda has joined #openstack-mistral | 13:27 | |
*** achanda has quit IRC | 13:27 | |
*** hparekh has quit IRC | 13:41 | |
*** hparekh has joined #openstack-mistral | 13:47 | |
*** sergmelikyan has joined #openstack-mistral | 13:59 | |
*** sergmelikyan has quit IRC | 14:06 | |
*** ramishra has joined #openstack-mistral | 14:07 | |
*** hparekh has quit IRC | 14:19 | |
*** Piet has joined #openstack-mistral | 14:25 | |
*** achanda has joined #openstack-mistral | 14:28 | |
*** achanda has quit IRC | 14:38 | |
*** tonytan4ever has joined #openstack-mistral | 14:40 | |
*** bobh has quit IRC | 14:42 | |
*** achanda has joined #openstack-mistral | 14:59 | |
*** achanda has quit IRC | 15:02 | |
*** tonytan4ever has quit IRC | 15:37 | |
*** gyee has joined #openstack-mistral | 15:46 | |
*** bobh has joined #openstack-mistral | 15:55 | |
*** sergmelikyan has joined #openstack-mistral | 15:59 | |
*** LimorStotland_ has quit IRC | 16:08 | |
*** sergmelikyan has quit IRC | 16:46 | |
*** achanda has joined #openstack-mistral | 17:20 | |
*** tonytan4ever has joined #openstack-mistral | 17:24 | |
*** sergmelikyan has joined #openstack-mistral | 17:46 | |
*** sergmelikyan has quit IRC | 17:52 | |
*** sergmelikyan has joined #openstack-mistral | 18:02 | |
*** sergmelikyan has quit IRC | 18:48 | |
*** sergmelikyan has joined #openstack-mistral | 18:58 | |
*** ryanpetrello has joined #openstack-mistral | 19:34 | |
*** sergmelikyan has quit IRC | 19:58 | |
*** tonytan4ever has quit IRC | 20:03 | |
*** tonytan4ever has joined #openstack-mistral | 20:09 | |
*** ig0r_ has joined #openstack-mistral | 20:58 | |
*** ig0r_ has quit IRC | 21:03 | |
*** ig0r_ has joined #openstack-mistral | 21:05 | |
*** ig0r_ has quit IRC | 21:10 | |
*** tonytan4ever has quit IRC | 21:36 | |
*** bobh has quit IRC | 21:43 | |
*** harlowja has quit IRC | 23:52 | |
*** harlowja has joined #openstack-mistral | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!