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

[Henning Westerholt]

Thank you Andrew, already merged.  :-)

Am 06.08.19 um 14:52 schrieb Andrew White:
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 at skalatan.de<mailto:hw at 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 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 at gmail.com<mailto:miconda at 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 at lists.kamailio.org<mailto:sr-users at lists.kamailio.org>
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 at lists.kamailio.org<mailto:sr-users at lists.kamailio.org>
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users


--
Henning Westerholt - https://skalatan.de/blog/
Kamailio services - https://skalatan.de/services


--
Henning Westerholt - https://skalatan.de/blog/
Kamailio services - https://skalatan.de/services
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-users/attachments/20190806/6cbbb91e/attachment.html>