Hello,

quick follow up to see if you got any chance to test with latest master.

Cheers,
Daniel

Hello,

can you fetch again the git master, recompile, reinstall and test again? Send here the log messages you get in syslog, because I added some log messages inside the function that fails now.

Cheers,
Daniel

On 23/12/2016 12:39, Yasin CANER wrote:

Hello;

    it is not working. it gives error like below.i added simple  configuration. reload command working fine but dump gives error. i have a look at code but couldnt find why cannot add in structs.

Thanks

13(32039) DEBUG: carrierroute [cr_fifo.c:1177]: cr_rpc_dump_routes(): start processing of data

error: 500 - Internal error - carrier structure

23.12.2016 11:46 tarihinde Daniel-Constantin Mierla yazdı:

Btw, if you use kamctl, then all should just work with the same commands like in the past if you fetch (at this time) latest git repository and load jsonrpcs module in your config. You may need to deploy latest kamctlrc, if you do an update and you did changes in the old one.

Cheers,
Daniel

On 23/12/2016 09:39, Daniel-Constantin Mierla wrote:

Ok, thanks. Note that I just pushed another commit renaming the command from cr.dump to cr.dump_routes in order to match better the mi old command.

Cheers,
Daniel

On 23/12/2016 09:35, Yasin CANER wrote:

Hello;

    I will try it now. Thanks.

23.12.2016 11:33 tarihinde Daniel-Constantin Mierla yazdı:

Hello,

I just pushed a commit for cr.dump rpc command in git master, can you give it a try and report the issues? I don't use the module so I had no testing environment to play with it.

Cheers,
Daniel

On 06/12/2016 08:54, ycaner wrote:

Hello,

you are right. some carrier route functions are missing.important of them are dump and show methods.if i have time ,i can add them.

Cheers.

6.12.2016 10:48 tarihinde Daniel-Constantin Mierla-6 [via SIP Router] yazdı:

Hello,

looking in the code I see the carrierroute module has already rpc
command for reload operations. But there are some other mi commands not
implemented as rpc.

Cheers,
Daniel


On 05/12/2016 11:28, Daniel-Constantin Mierla wrote:

> Hello,
>
> few that are missing can be probably added once we discover them. The
> best is to open an issue on bug tracker whenever you are aware of a mi
> command not implemented in rpc. Even better will be a pull request :-)
> -- migrating from mi to rpc should not be complex and rather easy to
> lean by looking at other variants with both implementations.
>
> Cheers,
> Daniel
>
>
> On 02/12/2016 16:18, ycaner wrote:
>> Hello
>> Carrier route module doesnt have RPC command., we need to add it
>> Cheers.
>>
>>
>>
>> --
>> View this message in context: http://sip-router.1086192.n5.nabble.com/VERY-IMPORTANT-deciding-when-to-remove-the-MI-code-tp153719p153773.html
>> Sent from the Users mailing list archive at Nabble.com.
>>
>> _______________________________________________
>> SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
>> [hidden email]
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

--
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com


_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
[hidden email]
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

---

If you reply to this email, your message will be added to the discussion below:

http://sip-router.1086192.n5.nabble.com/VERY-IMPORTANT-deciding-when-to-remove-the-MI-code-tp153719p153841.html

To unsubscribe from VERY IMPORTANT: deciding when to remove the MI code, click here.
NAML

--

---

View this message in context: Re: VERY IMPORTANT: deciding when to remove the MI code
Sent from the Users mailing list archive at Nabble.com.

_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com

--

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com

--

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com