*** jamielennox is now known as jamielennox|away | 00:07 | |
*** harlowja has quit IRC | 00:09 | |
*** jamielennox|away is now known as jamielennox | 00:11 | |
*** thrash has joined #openstack-mistral | 00:13 | |
*** thrash has joined #openstack-mistral | 00:13 | |
*** harlowja has joined #openstack-mistral | 00:25 | |
openstackgerrit | Jizhaoxuan proposed openstack/mistral: Support i18n for LOG.warning https://review.openstack.org/414498 | 01:56 |
---|---|---|
*** thrash is now known as thrash|g0ne | 02:20 | |
*** chlong has quit IRC | 02:32 | |
openstackgerrit | rajiv proposed openstack/mistral: Removed unnecessary utf-8 encoding https://review.openstack.org/418355 | 02:58 |
openstackgerrit | rajiv proposed openstack/mistral-dashboard: Removed unnecessary utf-8 encoding https://review.openstack.org/418375 | 03:05 |
*** sharatss has quit IRC | 03:15 | |
*** sharatss has joined #openstack-mistral | 03:15 | |
*** gongysh has joined #openstack-mistral | 03:47 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Removes unnecessary utf-8 encoding https://review.openstack.org/418687 | 05:37 |
openstackgerrit | Sharat Sharma proposed openstack/mistral-specs: Removes unnecessary utf-8 encoding https://review.openstack.org/418695 | 05:42 |
openstackgerrit | Sharat Sharma proposed openstack/mistral-extra: Removes unnecessary utf-8 encoding https://review.openstack.org/418698 | 05:47 |
*** gongysh has quit IRC | 05:54 | |
*** ist has joined #openstack-mistral | 05:55 | |
openstackgerrit | Merged openstack/python-mistralclient: Initial commit for python-mistralclient document https://review.openstack.org/404736 | 06:13 |
*** gongysh has joined #openstack-mistral | 06:21 | |
*** dturner has quit IRC | 06:24 | |
openstackgerrit | Merged openstack/mistral: Using sys.exit(main()) instead of main() https://review.openstack.org/416388 | 06:29 |
openstackgerrit | Merged openstack/mistral: Added gnocchi action pack https://review.openstack.org/416123 | 06:34 |
*** gongysh has quit IRC | 06:39 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral: Added congress actions into Mistral https://review.openstack.org/418737 | 06:52 |
*** AlexeyAbashkin has joined #openstack-mistral | 07:42 | |
openstackgerrit | Istvan Imre proposed openstack/mistral: External OpenStack action mapping file support added https://review.openstack.org/418761 | 08:13 |
*** jpich has joined #openstack-mistral | 08:51 | |
openstackgerrit | Istvan Imre proposed openstack/mistral: External OpenStack action mapping file support added https://review.openstack.org/418761 | 08:54 |
*** shardy has joined #openstack-mistral | 09:13 | |
openstackgerrit | Lingxian Kong proposed openstack/python-mistralclient: Pass target insecure flag to msitral service https://review.openstack.org/418305 | 09:41 |
openstackgerrit | Sharat Sharma proposed openstack/mistral: Add script for unit test coverage job https://review.openstack.org/417881 | 09:43 |
openstackgerrit | Sharat Sharma proposed openstack/mistral: Added congress actions into Mistral https://review.openstack.org/418737 | 09:45 |
*** tuan_ has joined #openstack-mistral | 10:11 | |
tuan_ | ddeja: Hi ddeja | 10:16 |
tuan_ | hope you are online? | 10:16 |
*** Alexey_Abashkin has joined #openstack-mistral | 10:18 | |
*** AlexeyAbashkin has quit IRC | 10:19 | |
*** enykeev has quit IRC | 10:19 | |
ddeja | tuan_: Hi, yes | 10:21 |
ddeja | I'm on-line | 10:21 |
ddeja | how can I help you? | 10:22 |
tuan_ | hi | 10:22 |
tuan_ | just about a yesterday launchpad i have written | 10:22 |
tuan_ | Renat told me that we have changed the order of the error message before | 10:22 |
tuan_ | is it on master branch now | 10:23 |
tuan_ | ? | 10:23 |
tuan_ | i think that we are using newton version | 10:23 |
ddeja | I think it is | 10:23 |
tuan_ | do you know which patch i can find | 10:24 |
tuan_ | ? | 10:24 |
ddeja | unfortunately, I have no idea | 10:25 |
ddeja | Renat said it was in October or November | 10:25 |
ddeja | but that's all I know | 10:25 |
tuan_ | thanks ddeja | 10:29 |
tuan_ | i will ping him for more information | 10:29 |
sharatss | rakhmerov: ddeja what do you think about this? https://blueprints.launchpad.net/mistral/+spec/mistral-hacking | 10:30 |
*** Alexey_Abashkin has quit IRC | 10:31 | |
ddeja | sharatss: looking | 10:34 |
sharatss | ddeja: thanks | 10:34 |
ddeja | sharatss: I'm ok with everything but this: 'mutables should not be used as default args' | 10:35 |
ddeja | I don't remember when and in which project, but I used once something like 'def foo(l=[])' | 10:36 |
ddeja | knowing the consequances | 10:37 |
ddeja | and 2 thing - do you think it would be possible to also add style checks for multiline function calls? | 10:37 |
ddeja | like, closing bracket in separate line and so on? | 10:38 |
sharatss | ddeja: i can try that.. so mutables can be used as default args? | 10:38 |
sharatss | ddeja: i can add style checks for multiline function calls | 10:39 |
ddeja | sharatss: in 99.99% of situation they should not be used as deafult args | 10:39 |
ddeja | but then this 1 in a 10000 situation comes, when you want to use it ;) | 10:39 |
ddeja | and therefore, I'd like to not add such rule | 10:40 |
sharatss | ddeja: i was going through some pep8 doc and i found that it should not be used\ | 10:40 |
ddeja | sharatss: yes, I know | 10:40 |
sharatss | so added it.. didnt think about the exceptional cases ;) | 10:40 |
ddeja | well, I'm not going to be very picky about it | 10:40 |
ddeja | sharatss: style checks about multiline functions calls would be very nice, so it would be great if you add it | 10:41 |
sharatss | ddeja: i will definitely try to add it | 10:41 |
ddeja | cool, thanks | 10:42 |
openstackgerrit | Merged openstack/mistral-extra: Removes unnecessary utf-8 encoding https://review.openstack.org/418698 | 10:43 |
sharatss | ddeja: thanks for reviewing the BP | 10:44 |
ddeja | no problem :) | 10:45 |
openstackgerrit | Merged openstack/mistral-dashboard: Removed unnecessary utf-8 encoding https://review.openstack.org/418375 | 10:46 |
openstackgerrit | Merged openstack/mistral-specs: Removes unnecessary utf-8 encoding https://review.openstack.org/418695 | 10:47 |
*** sharatss has quit IRC | 10:55 | |
*** sharatss has joined #openstack-mistral | 10:55 | |
*** AlexeyAbashkin has joined #openstack-mistral | 11:02 | |
*** Alexey_Abashkin has joined #openstack-mistral | 11:06 | |
*** AlexeyAbashkin has quit IRC | 11:07 | |
*** Alexey_Abashkin_ has joined #openstack-mistral | 11:09 | |
*** Alexey_Abashkin has quit IRC | 11:11 | |
openstackgerrit | Merged openstack/python-mistralclient: Removes unnecessary utf-8 encoding https://review.openstack.org/418687 | 11:32 |
openstackgerrit | Merged openstack/mistral: Removed unnecessary utf-8 encoding https://review.openstack.org/418355 | 11:34 |
*** sharatss_ has joined #openstack-mistral | 11:34 | |
rakhmerov | sharatss: I would also remove the item about mutables | 11:57 |
*** AlexeyAbashkin has joined #openstack-mistral | 11:58 | |
*** Alexey_Abashkin_ has quit IRC | 11:59 | |
sharatss_ | rakhmerov: yes.. i will remove it | 12:09 |
tuan_ | rakhmerov: When you online, could you please send me the patch of change the order of error message if you do not mind | 12:10 |
rakhmerov | tuan_: it's not so easy to find, really | 12:11 |
rakhmerov | but I'll try | 12:11 |
rakhmerov | tuan_: these patches: https://review.openstack.org/#/q/project:openstack/mistral+%22Fix+error+message+format%22 | 12:13 |
*** AlexeyAbashkin has quit IRC | 12:20 | |
*** shardy is now known as shardy_lunch | 12:43 | |
*** rbrady-afk is now known as rbrady | 12:50 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral: Initial commit for mistral api-ref https://review.openstack.org/418913 | 12:56 |
*** thrash|g0ne is now known as thrash | 13:00 | |
*** dprince has joined #openstack-mistral | 13:04 | |
*** sharatss_ has quit IRC | 13:05 | |
*** chlong has joined #openstack-mistral | 13:08 | |
*** shardy_lunch has quit IRC | 13:12 | |
tuan_ | rakhmerov: Thanks Renat, i will check this patch on our system | 13:13 |
*** tuan_ has quit IRC | 13:13 | |
*** shardy_lunch has joined #openstack-mistral | 13:19 | |
*** shardy_lunch is now known as shardy | 13:28 | |
*** shardy has quit IRC | 13:28 | |
*** shardy has joined #openstack-mistral | 13:29 | |
*** AlexeyAbashkin has joined #openstack-mistral | 13:35 | |
*** AlexeyAbashkin has quit IRC | 13:57 | |
*** AlexeyAbashkin has joined #openstack-mistral | 13:59 | |
*** ist has quit IRC | 14:05 | |
*** tuan_ has joined #openstack-mistral | 14:13 | |
*** jamielennox is now known as jamielennox|away | 14:14 | |
*** chlong has quit IRC | 14:37 | |
*** tuan_ has quit IRC | 14:39 | |
*** bobh has joined #openstack-mistral | 15:02 | |
*** jaosorior has joined #openstack-mistral | 15:13 | |
*** shardy has quit IRC | 15:43 | |
thrash | d0ugal: any clue on those local settings? | 16:03 |
d0ugal | thrash: sorry, in a call | 16:09 |
thrash | d0ugal: no worries.. | 16:09 |
thrash | and no hurries.. lol | 16:09 |
d0ugal | lol | 16:10 |
*** chlong has joined #openstack-mistral | 16:11 | |
d0ugal | thrash: okay, I can help now. | 16:22 |
d0ugal | thrash: so you are trying to start Mistral manually? | 16:23 |
thrash | d0ugal: yes | 16:23 |
thrash | d0ugal: from a venv... In "non-openstack" mode | 16:23 |
d0ugal | thrash: have you disabled auth? | 16:23 |
thrash | d0ugal: yes | 16:23 |
thrash | that's the other thing I set... in the pecan section | 16:24 |
d0ugal | thrash: generally that is all you need to do :/ Maybe something has changed. | 16:24 |
thrash | d0ugal: let me try with nothing but that... | 16:24 |
d0ugal | cool | 16:25 |
d0ugal | I am just giving it a shot too | 16:25 |
thrash | d0ugal: and you're launching via launch.py? | 16:26 |
d0ugal | thrash: you should have a mistral-server command | 16:26 |
thrash | d0ugal: ok.. so different now... It's looping on trying to connect to rabbit.. | 16:26 |
thrash | d0ugal: let me try that. | 16:26 |
d0ugal | thrash: https://github.com/openstack/mistral/blob/master/setup.cfg#L41 | 16:26 |
d0ugal | thrash: but it uses the launch.py file. | 16:26 |
d0ugal | thrash: (I don't think we expose that command when installed via the rpm for some reason...) | 16:27 |
thrash | d0ugal: right... this is from source tho | 16:27 |
d0ugal | yup | 16:27 |
d0ugal | Just if you wondered why it was normally missing | 16:27 |
thrash | so yeah. same thing with mistral-server | 16:27 |
thrash | ahh | 16:27 |
thrash | now it is just looping looking for rabbit. | 16:27 |
d0ugal | Interesting. | 16:28 |
d0ugal | I am just waiting for pip to finish installing. | 16:28 |
thrash | obviously I'd prefer not to have to set up rabbit... But I can start a container if necessary. | 16:28 |
d0ugal | It shouldn't be needed. | 16:28 |
d0ugal | unless my memory is failing me | 16:28 |
d0ugal | thrash: actually, #7 here, maybe it is: http://docs.openstack.org/developer/mistral/guides/configuration_guide.html | 16:29 |
d0ugal | I've not done this since 2016 :P so that might be why | 16:30 |
d0ugal | thrash: did you try changing the rpc_backend? | 16:31 |
*** AlexeyAbashkin has quit IRC | 16:32 | |
d0ugal | thrash: https://github.com/openstack/mistral/blob/master/README.rst#debugging | 16:34 |
d0ugal | thrash: can you show me the config you have? | 16:36 |
*** shardy has joined #openstack-mistral | 16:38 | |
thrash | d0ugal: sure | 16:41 |
thrash | d0ugal: I did change the rpc_backend to fake | 16:42 |
thrash | d0ugal: and still got the original error I sent you | 16:42 |
d0ugal | hmm | 16:42 |
thrash | d0ugal: right now... http://paste.openstack.org/show/594588/ | 16:44 |
thrash | and that just loops looking for a rabbit server. | 16:44 |
*** FL1SK has quit IRC | 16:45 | |
d0ugal | oh, almost empty | 16:45 |
thrash | d0ugal: yep | 16:45 |
d0ugal | thrash: I have been doing my testing lately in a tripleo undercloud, I thought it was a pain but this might be worse :P | 16:47 |
*** FL1SK has joined #openstack-mistral | 16:47 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral: [WIP] Don't create actions with empty arg_lists https://review.openstack.org/412433 | 16:47 |
d0ugal | thrash: yeah, so, I get the same thing as you now. | 16:50 |
d0ugal | thrash: if I set rpc_backend=fake I get a different error | 16:51 |
thrash | d0ugal: what error do you get? | 16:51 |
thrash | d0ugal: this? http://paste.openstack.org/show/594590/ | 16:52 |
d0ugal | thrash: yeah | 16:52 |
d0ugal | thrash: and I have no why :) | 16:52 |
d0ugal | thrash: I can start the api (--server api) but none of the others. | 16:54 |
thrash | strange | 16:55 |
d0ugal | thrash: it might be that we have broken everything but rabbit | 16:56 |
d0ugal | I'm not sure how well the others are tested | 16:56 |
d0ugal | kombu is in the gate, but I have never used it | 16:56 |
thrash | d0ugal: it does appear to make some assumptions about rabbit... | 16:56 |
d0ugal | thrash: https://github.com/openstack/mistral/blob/master/mistral/utils/rpc_utils.py#L43-L55 | 16:57 |
d0ugal | thrash: lol "fake" == "rabbit" | 16:58 |
d0ugal | so it looks like the valid options are amqp, zmq, rabbit or rabbit | 16:58 |
d0ugal | I'd guess the "fake" backend has been broken. | 16:58 |
thrash | d0ugal: this seems odd to me: https://github.com/openstack/mistral/blob/master/mistral/utils/rpc_utils.py#L59 | 17:01 |
d0ugal | thrash: Yeah, I don't really understand that | 17:01 |
d0ugal | I'll ask ddeja in the morning I think :) | 17:02 |
thrash | d0ugal: I'll play with it for another 10 minutes.. then punt and use a rabbit container. | 17:02 |
d0ugal | thrash: sounds good. Let me know how you get on | 17:04 |
*** FL1SK has quit IRC | 17:15 | |
*** FL1SK has joined #openstack-mistral | 17:16 | |
*** FL1SK has quit IRC | 17:24 | |
*** FL1SK has joined #openstack-mistral | 17:28 | |
*** FL1SK has quit IRC | 17:32 | |
*** FL1SK has joined #openstack-mistral | 17:33 | |
*** rbrady has quit IRC | 17:46 | |
*** jpich has quit IRC | 17:46 | |
*** FL1SK has quit IRC | 17:50 | |
*** FL1SK has joined #openstack-mistral | 17:53 | |
*** AlexeyAbashkin has joined #openstack-mistral | 17:55 | |
*** AlexeyAbashkin has quit IRC | 17:58 | |
*** AlexeyAbashkin has joined #openstack-mistral | 17:58 | |
*** bobh has quit IRC | 18:12 | |
*** Alexey_Abashkin has joined #openstack-mistral | 18:21 | |
*** AlexeyAbashkin has quit IRC | 18:24 | |
thrash | d0ugal: got it running with rabbit container | 18:30 |
*** FL1SK has quit IRC | 18:32 | |
*** chlong has quit IRC | 18:46 | |
*** shardy has quit IRC | 18:48 | |
*** FL1SK has joined #openstack-mistral | 18:51 | |
*** chlong has joined #openstack-mistral | 18:59 | |
*** jaosorior has quit IRC | 19:46 | |
*** Alexey_Abashkin has quit IRC | 19:48 | |
*** AlexeyAbashkin has joined #openstack-mistral | 19:51 | |
*** rbrady has joined #openstack-mistral | 19:52 | |
*** rbrady has quit IRC | 19:52 | |
*** rbrady has joined #openstack-mistral | 19:52 | |
*** jaosorior has joined #openstack-mistral | 19:56 | |
*** Alexey_Abashkin has joined #openstack-mistral | 19:56 | |
*** AlexeyAbashkin has quit IRC | 19:57 | |
*** ist has joined #openstack-mistral | 20:03 | |
*** Alexey_Abashkin_ has joined #openstack-mistral | 20:11 | |
*** Alexey_Abashkin has quit IRC | 20:13 | |
*** Alexey_Abashkin_ has quit IRC | 20:20 | |
*** Alexey_Abashkin_ has joined #openstack-mistral | 20:28 | |
*** bobh has joined #openstack-mistral | 20:32 | |
*** bobh has quit IRC | 20:32 | |
*** jaosorior has quit IRC | 20:39 | |
*** ist has quit IRC | 20:51 | |
*** Alexey_Abashkin_ has quit IRC | 21:00 | |
*** jamielennox|away is now known as jamielennox | 21:01 | |
*** jaosorior has joined #openstack-mistral | 21:44 | |
*** sharatss has quit IRC | 22:24 | |
*** sharatss has joined #openstack-mistral | 22:25 | |
*** catintheroof has quit IRC | 22:50 | |
*** catintheroof has joined #openstack-mistral | 22:51 | |
*** chlong has quit IRC | 22:55 | |
*** catintheroof has quit IRC | 22:55 | |
*** FL1SK has quit IRC | 23:41 | |
*** jaosorior has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!