6 Aug
2019
6 Aug
'19
3:52 p.m.
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
<hw(a)skalatan.de> wrote:
Hi Andrew,
comments below inline
Am 06.08.19 um 09:14 schrieb Andrew White:
--
Henning Westerholt - https://skalatan.de/blog/ <https://skalatan.de/blog/>
Kamailio services - https://skalatan.de/services <https://skalatan.de/services>
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 <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:
_______________________________________________
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> 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>