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 <[email protected]> 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 <[email protected] >>> <mailto:[email protected]>> 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 >>>> [email protected] <mailto:[email protected]> >>>> 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 >> [email protected] <mailto:[email protected]> >> 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>
_______________________________________________ Kamailio (SER) - Users Mailing List [email protected] https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
