Re: [SR-Users] app_ruby - invalid ruby environment attributes or parameters

Hi Henning, Thanks! I’ll start bisecting now and see what I can find and report back. Awesome, I’ve built and tested a generic config and submitted a pull request - I’m excited to finally be contributing! Thanks again, Andrew > On 6 Aug 2019, at 5:30 pm, Henning Westerholt &lt;hw(a)skalatan.de <mailto:hw@skalatan.de>> wrote: > > Hi Andrew, > > comments below inline > > Am 06.08.19 um 09:14 schrieb Andrew White: >> Hey Daniel, >> >> They certainly seem to be, here’s a snippet: >> >> { >> "func": { >> "ret": "xval", >> "module": "pv", >> "name": "getw", >> "params": "str" >> } >> }, { >> "func": { >> "ret": "xval", >> "module": "pv", >> "name": "gete", >> "params": "str" >> } >> }, { >> "func": { >> "ret": "xval", >> "module": "pv", >> "name": "getvn", >> "params": "str, int" >> } >> }, { >> "func": { >> "ret": "xval", >> "module": "pv", >> "name": "getvs", >> "params": "str, str" >> } >> }, { >> "func": { >> "ret": "bool", >> "module": "pv", >> "name": "seti", >> "params": "str, int" >> } >> }, { >> "func": { >> "ret": "bool", >> "module": "pv", >> "name": "sets", >> "params": "str, str" >> } >> }, >> >> I’ve just done a fresh deploy with the same playbook to a new instance, but using dev-4 (6d43eacd6d5b78fe857267e70b7fa85519a7d5b6) instead of dev-7 (bd477f5bb938a05efab76fb096cf5ac780d65c45) and it works perfectly, so it must be a change in between the two. > Just looked to the app_ruby commits - they are only 10 or so after dev4. Maybe you can just a bi-sect (pick the one in the middle, see if still works, pick the next one in the "new" middle etc..) to find the commit that broke it for you. > >> >> I was going to add in a ruby example config to https://github.com/kamailio/kamailio/tree/master/misc/examples/kemi <https://github.com/kamailio/kamailio/tree/master/misc/examples/kemi> a while ago, but I keep getting caught up with other things. Would it be helpful here for testing if I did this and put in a pull request? I’ll see if I can get that done tonight (I guess today EU time!) >> > Pull requests are always welcome, and an example cfg would make testing of course easier for other developers as well. > Cheers, > > Henning > >>> On 6 Aug 2019, at 2:27 am, Daniel-Constantin Mierla &lt;miconda(a)gmail.com <mailto:miconda@gmail.com>> wrote: >>> >>> Hello, >>> >>> there was a change to use the core PV functions, not those implemented >>> in app_ruby, so this might have broken something. >>> >>> Can you run kamailio master branch with jsonrpcs module loaded and then >>> execute: >>> >>> kamctl rpc app_ruby.api_list >>> >>> See if the PV functions are listed int the output. >>> >>> Cheers, >>> Daniel >>> >>> On 05.08.19 15:19, Andrew White wrote: >>>> Hey all, >>>> >>>> I’m setting up a dev-7 (bd477f) server to test as per the other thread, however I’m getting the following loading my config: >>>> >>>> ERROR: app_ruby [app_ruby_api.c:465]: sr_kemi_ruby_exec_func_ex(): invalid ruby environment attributes or parameters >>>> >>>> I’m unsure what this error means - are there expected environment variables or other values that must be set? I didn’t see this issue in dev-4. >>>> >>>> I thought it might be not able to find the library, but I ran with debug=4 and saw: >>>> >>>> DEBUG: app_ruby [app_ruby_api.c:193]: ruby_sr_init_child(): test execution without error >>>> ERROR: app_ruby [app_ruby_api.c:107]: app_ruby_print_last_exception(): exception: undefined method `>' for false:FalseClass >>>> >>>> I changed kamailio.cfg to load my generic routing script, and during the onreply_manage it runs: >>>> >>>> scode = KSR::PV.get("$rs”) >>>> >>>> This now returns false (which appears to be the second error above, I check if scode is > 599). >>>> >>>> Has anything changed in the way ruby is loaded in dev-7? I’m using the same ruby/ruby-devel library versions on the same OS. I did this deploy the same with devel-4 a few weeks ago and this didn’t happen. >>>> >>>> Thanks! >>>> >>>> Andrew >>>> _______________________________________________ >>>> Kamailio (SER) - Users Mailing List >>>> sr-users(a)lists.kamailio.org <mailto:sr-users@lists.kamailio.org> >>>> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users <https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users> >>> >>> -- >>> Daniel-Constantin Mierla -- www.asipto.com <http://www.asipto.com/> >>> www.twitter.com/miconda <http://www.twitter.com/miconda> -- www.linkedin.com/in/miconda <http://www.linkedin.com/in/miconda> >>> >> >> >> >> _______________________________________________ >> Kamailio (SER) - Users Mailing List >> sr-users(a)lists.kamailio.org <mailto:sr-users@lists.kamailio.org> >> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users <https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users> > -- > Henning Westerholt - https://skalatan.de/blog/ <https://skalatan.de/blog/> > Kamailio services - https://skalatan.de/services <https://skalatan.de/services>