Wednesday, 2017-01-11

*** jamielennox is now known as jamielennox|away00:07
*** harlowja has quit IRC00:09
*** jamielennox|away is now known as jamielennox00:11
*** thrash has joined #openstack-mistral00:13
*** thrash has joined #openstack-mistral00:13
*** harlowja has joined #openstack-mistral00:25
openstackgerritJizhaoxuan proposed openstack/mistral: Support i18n for LOG.warning  https://review.openstack.org/41449801:56
*** thrash is now known as thrash|g0ne02:20
*** chlong has quit IRC02:32
openstackgerritrajiv proposed openstack/mistral: Removed unnecessary utf-8 encoding  https://review.openstack.org/41835502:58
openstackgerritrajiv proposed openstack/mistral-dashboard: Removed unnecessary utf-8 encoding  https://review.openstack.org/41837503:05
*** sharatss has quit IRC03:15
*** sharatss has joined #openstack-mistral03:15
*** gongysh has joined #openstack-mistral03:47
openstackgerritSharat Sharma proposed openstack/python-mistralclient: Removes unnecessary utf-8 encoding  https://review.openstack.org/41868705:37
openstackgerritSharat Sharma proposed openstack/mistral-specs: Removes unnecessary utf-8 encoding  https://review.openstack.org/41869505:42
openstackgerritSharat Sharma proposed openstack/mistral-extra: Removes unnecessary utf-8 encoding  https://review.openstack.org/41869805:47
*** gongysh has quit IRC05:54
*** ist has joined #openstack-mistral05:55
openstackgerritMerged openstack/python-mistralclient: Initial commit for python-mistralclient document  https://review.openstack.org/40473606:13
*** gongysh has joined #openstack-mistral06:21
*** dturner has quit IRC06:24
openstackgerritMerged openstack/mistral: Using sys.exit(main()) instead of main()  https://review.openstack.org/41638806:29
openstackgerritMerged openstack/mistral: Added gnocchi action pack  https://review.openstack.org/41612306:34
*** gongysh has quit IRC06:39
openstackgerritSharat Sharma proposed openstack/mistral: Added congress actions into Mistral  https://review.openstack.org/41873706:52
*** AlexeyAbashkin has joined #openstack-mistral07:42
openstackgerritIstvan Imre proposed openstack/mistral: External OpenStack action mapping file support added  https://review.openstack.org/41876108:13
*** jpich has joined #openstack-mistral08:51
openstackgerritIstvan Imre proposed openstack/mistral: External OpenStack action mapping file support added  https://review.openstack.org/41876108:54
*** shardy has joined #openstack-mistral09:13
openstackgerritLingxian Kong proposed openstack/python-mistralclient: Pass target insecure flag to msitral service  https://review.openstack.org/41830509:41
openstackgerritSharat Sharma proposed openstack/mistral: Add script for unit test coverage job  https://review.openstack.org/41788109:43
openstackgerritSharat Sharma proposed openstack/mistral: Added congress actions into Mistral  https://review.openstack.org/41873709:45
*** tuan_ has joined #openstack-mistral10:11
tuan_ddeja: Hi ddeja10:16
tuan_hope you are online?10:16
*** Alexey_Abashkin has joined #openstack-mistral10:18
*** AlexeyAbashkin has quit IRC10:19
*** enykeev has quit IRC10:19
ddejatuan_: Hi, yes10:21
ddejaI'm on-line10:21
ddejahow can I help you?10:22
tuan_hi10:22
tuan_just about a yesterday launchpad i have written10:22
tuan_Renat told me that we have changed the order of the error message before10:22
tuan_is it on master branch now10:23
tuan_?10:23
tuan_i think that we are using newton version10:23
ddejaI think it is10:23
tuan_do you know which patch i can find10:24
tuan_?10:24
ddejaunfortunately, I have no idea10:25
ddejaRenat said it was in October or November10:25
ddejabut that's all I know10:25
tuan_thanks ddeja10:29
tuan_i will ping him for more information10:29
sharatssrakhmerov: ddeja what do you think about this? https://blueprints.launchpad.net/mistral/+spec/mistral-hacking10:30
*** Alexey_Abashkin has quit IRC10:31
ddejasharatss: looking10:34
sharatssddeja: thanks10:34
ddejasharatss: I'm ok with everything but this: 'mutables should not be used as default args'10:35
ddejaI don't remember when and in which project, but I used once something like 'def foo(l=[])'10:36
ddejaknowing the consequances10:37
ddejaand 2 thing - do you think it would be possible to also add style checks for multiline function calls?10:37
ddejalike, closing bracket in separate line and so on?10:38
sharatssddeja: i can try that.. so mutables can be used as default args?10:38
sharatssddeja: i can add style checks for multiline function calls10:39
ddejasharatss: in 99.99% of situation they should not be used as deafult args10:39
ddejabut then this 1 in a 10000 situation comes, when you want to use it  ;)10:39
ddejaand therefore, I'd like to not add such rule10:40
sharatssddeja: i was going through some pep8 doc and i found that it should not be used\10:40
ddejasharatss: yes, I know10:40
sharatssso added it.. didnt think about the exceptional cases ;)10:40
ddejawell, I'm not going to be very picky about it10:40
ddejasharatss: style checks about multiline functions calls would be very nice, so it would be great if you add it10:41
sharatssddeja: i will definitely try to add it10:41
ddejacool, thanks10:42
openstackgerritMerged openstack/mistral-extra: Removes unnecessary utf-8 encoding  https://review.openstack.org/41869810:43
sharatssddeja: thanks for reviewing the BP10:44
ddejano problem :)10:45
openstackgerritMerged openstack/mistral-dashboard: Removed unnecessary utf-8 encoding  https://review.openstack.org/41837510:46
openstackgerritMerged openstack/mistral-specs: Removes unnecessary utf-8 encoding  https://review.openstack.org/41869510:47
*** sharatss has quit IRC10:55
*** sharatss has joined #openstack-mistral10:55
*** AlexeyAbashkin has joined #openstack-mistral11:02
*** Alexey_Abashkin has joined #openstack-mistral11:06
*** AlexeyAbashkin has quit IRC11:07
*** Alexey_Abashkin_ has joined #openstack-mistral11:09
*** Alexey_Abashkin has quit IRC11:11
openstackgerritMerged openstack/python-mistralclient: Removes unnecessary utf-8 encoding  https://review.openstack.org/41868711:32
openstackgerritMerged openstack/mistral: Removed unnecessary utf-8 encoding  https://review.openstack.org/41835511:34
*** sharatss_ has joined #openstack-mistral11:34
rakhmerovsharatss: I would also remove the item about mutables11:57
*** AlexeyAbashkin has joined #openstack-mistral11:58
*** Alexey_Abashkin_ has quit IRC11:59
sharatss_rakhmerov: yes.. i will remove it12:09
tuan_rakhmerov: When you online, could you please send me the patch of change the order of error message if you do not mind12:10
rakhmerovtuan_: it's not so easy to find, really12:11
rakhmerovbut I'll try12:11
rakhmerovtuan_: these patches: https://review.openstack.org/#/q/project:openstack/mistral+%22Fix+error+message+format%2212:13
*** AlexeyAbashkin has quit IRC12:20
*** shardy is now known as shardy_lunch12:43
*** rbrady-afk is now known as rbrady12:50
openstackgerritSharat Sharma proposed openstack/mistral: Initial commit for mistral api-ref  https://review.openstack.org/41891312:56
*** thrash|g0ne is now known as thrash13:00
*** dprince has joined #openstack-mistral13:04
*** sharatss_ has quit IRC13:05
*** chlong has joined #openstack-mistral13:08
*** shardy_lunch has quit IRC13:12
tuan_rakhmerov: Thanks Renat, i will check this patch on our system13:13
*** tuan_ has quit IRC13:13
*** shardy_lunch has joined #openstack-mistral13:19
*** shardy_lunch is now known as shardy13:28
*** shardy has quit IRC13:28
*** shardy has joined #openstack-mistral13:29
*** AlexeyAbashkin has joined #openstack-mistral13:35
*** AlexeyAbashkin has quit IRC13:57
*** AlexeyAbashkin has joined #openstack-mistral13:59
*** ist has quit IRC14:05
*** tuan_ has joined #openstack-mistral14:13
*** jamielennox is now known as jamielennox|away14:14
*** chlong has quit IRC14:37
*** tuan_ has quit IRC14:39
*** bobh has joined #openstack-mistral15:02
*** jaosorior has joined #openstack-mistral15:13
*** shardy has quit IRC15:43
thrashd0ugal: any clue on those local settings?16:03
d0ugalthrash: sorry, in a call16:09
thrashd0ugal: no worries..16:09
thrashand no hurries.. lol16:09
d0ugallol16:10
*** chlong has joined #openstack-mistral16:11
d0ugalthrash: okay, I can help now.16:22
d0ugalthrash: so you are trying to start Mistral manually?16:23
thrashd0ugal: yes16:23
thrashd0ugal: from a venv... In "non-openstack" mode16:23
d0ugalthrash: have you disabled auth?16:23
thrashd0ugal: yes16:23
thrashthat's the other thing I set... in the pecan section16:24
d0ugalthrash: generally that is all you need to do :/ Maybe something has changed.16:24
thrashd0ugal: let me try with nothing but that...16:24
d0ugalcool16:25
d0ugalI am just giving it a shot too16:25
thrashd0ugal: and you're launching via launch.py?16:26
d0ugalthrash: you should have a mistral-server command16:26
thrashd0ugal: ok.. so different now... It's looping on trying to connect to rabbit..16:26
thrashd0ugal: let me try that.16:26
d0ugalthrash: https://github.com/openstack/mistral/blob/master/setup.cfg#L4116:26
d0ugalthrash: but it uses the launch.py file.16:26
d0ugalthrash: (I don't think we expose that command when installed via the rpm for some reason...)16:27
thrashd0ugal: right... this is from source tho16:27
d0ugalyup16:27
d0ugalJust if you wondered why it was normally missing16:27
thrashso yeah. same thing with mistral-server16:27
thrashahh16:27
thrashnow it is just looping looking for rabbit.16:27
d0ugalInteresting.16:28
d0ugalI am just waiting for pip to finish installing.16:28
thrashobviously I'd prefer not to have to set up rabbit... But I can start a container if necessary.16:28
d0ugalIt shouldn't be needed.16:28
d0ugalunless my memory is failing me16:28
d0ugalthrash: actually, #7 here, maybe it is: http://docs.openstack.org/developer/mistral/guides/configuration_guide.html16:29
d0ugalI've not done this since 2016 :P so that might be why16:30
d0ugalthrash: did you try changing the rpc_backend?16:31
*** AlexeyAbashkin has quit IRC16:32
d0ugalthrash: https://github.com/openstack/mistral/blob/master/README.rst#debugging16:34
d0ugalthrash: can you show me the config you have?16:36
*** shardy has joined #openstack-mistral16:38
thrashd0ugal: sure16:41
thrashd0ugal: I did change the rpc_backend to fake16:42
thrashd0ugal: and still got the original error I sent you16:42
d0ugalhmm16:42
thrashd0ugal: right now... http://paste.openstack.org/show/594588/16:44
thrashand that just loops looking for a rabbit server.16:44
*** FL1SK has quit IRC16:45
d0ugaloh, almost empty16:45
thrashd0ugal: yep16:45
d0ugalthrash: I have been doing my testing lately in a tripleo undercloud, I thought it was a pain but this might be worse :P16:47
*** FL1SK has joined #openstack-mistral16:47
openstackgerritDougal Matthews proposed openstack/mistral: [WIP] Don't create actions with empty arg_lists  https://review.openstack.org/41243316:47
d0ugalthrash: yeah, so, I get the same thing as you now.16:50
d0ugalthrash: if I set rpc_backend=fake I get a different error16:51
thrashd0ugal: what error do you get?16:51
thrashd0ugal: this? http://paste.openstack.org/show/594590/16:52
d0ugalthrash: yeah16:52
d0ugalthrash: and I have no why :)16:52
d0ugalthrash: I can start the api (--server api) but none of the others.16:54
thrashstrange16:55
d0ugalthrash: it might be that we have broken everything but rabbit16:56
d0ugalI'm not sure how well the others are tested16:56
d0ugalkombu is in the gate, but I have never used it16:56
thrashd0ugal: it does appear to make some assumptions about rabbit...16:56
d0ugalthrash: https://github.com/openstack/mistral/blob/master/mistral/utils/rpc_utils.py#L43-L5516:57
d0ugalthrash: lol "fake" == "rabbit"16:58
d0ugalso it looks like the valid options are amqp, zmq, rabbit or rabbit16:58
d0ugalI'd guess the "fake" backend has been broken.16:58
thrashd0ugal: this seems odd to me: https://github.com/openstack/mistral/blob/master/mistral/utils/rpc_utils.py#L5917:01
d0ugalthrash: Yeah, I don't really understand that17:01
d0ugalI'll ask ddeja in the morning I think :)17:02
thrashd0ugal: I'll play with it for another 10 minutes.. then punt and use a rabbit container.17:02
d0ugalthrash: sounds good. Let me know how you get on17:04
*** FL1SK has quit IRC17:15
*** FL1SK has joined #openstack-mistral17:16
*** FL1SK has quit IRC17:24
*** FL1SK has joined #openstack-mistral17:28
*** FL1SK has quit IRC17:32
*** FL1SK has joined #openstack-mistral17:33
*** rbrady has quit IRC17:46
*** jpich has quit IRC17:46
*** FL1SK has quit IRC17:50
*** FL1SK has joined #openstack-mistral17:53
*** AlexeyAbashkin has joined #openstack-mistral17:55
*** AlexeyAbashkin has quit IRC17:58
*** AlexeyAbashkin has joined #openstack-mistral17:58
*** bobh has quit IRC18:12
*** Alexey_Abashkin has joined #openstack-mistral18:21
*** AlexeyAbashkin has quit IRC18:24
thrashd0ugal: got it running with rabbit container18:30
*** FL1SK has quit IRC18:32
*** chlong has quit IRC18:46
*** shardy has quit IRC18:48
*** FL1SK has joined #openstack-mistral18:51
*** chlong has joined #openstack-mistral18:59
*** jaosorior has quit IRC19:46
*** Alexey_Abashkin has quit IRC19:48
*** AlexeyAbashkin has joined #openstack-mistral19:51
*** rbrady has joined #openstack-mistral19:52
*** rbrady has quit IRC19:52
*** rbrady has joined #openstack-mistral19:52
*** jaosorior has joined #openstack-mistral19:56
*** Alexey_Abashkin has joined #openstack-mistral19:56
*** AlexeyAbashkin has quit IRC19:57
*** ist has joined #openstack-mistral20:03
*** Alexey_Abashkin_ has joined #openstack-mistral20:11
*** Alexey_Abashkin has quit IRC20:13
*** Alexey_Abashkin_ has quit IRC20:20
*** Alexey_Abashkin_ has joined #openstack-mistral20:28
*** bobh has joined #openstack-mistral20:32
*** bobh has quit IRC20:32
*** jaosorior has quit IRC20:39
*** ist has quit IRC20:51
*** Alexey_Abashkin_ has quit IRC21:00
*** jamielennox|away is now known as jamielennox21:01
*** jaosorior has joined #openstack-mistral21:44
*** sharatss has quit IRC22:24
*** sharatss has joined #openstack-mistral22:25
*** catintheroof has quit IRC22:50
*** catintheroof has joined #openstack-mistral22:51
*** chlong has quit IRC22:55
*** catintheroof has quit IRC22:55
*** FL1SK has quit IRC23:41
*** jaosorior has quit IRC23:54

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