[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-21 Thread Daniel-Constantin Mierla via sr-dev
Merged #3639 into master. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3639#event-11022345523 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-21 Thread Daniel-Constantin Mierla via sr-dev
Thanks, merging! -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3639#issuecomment-1820630494 You are receiving this because you are subscribed to this thread. Message ID: ___ Kamailio (SER) -

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-20 Thread Xenofon Karamanos via sr-dev
@xkaraman pushed 1 commit. 76ad686d2c268040a30dfc02096d29be28d7eedf topos: Add version_table documentation -- View it on GitHub: https://github.com/kamailio/kamailio/pull/3639/files/0d8052ed91b20d23af2b6c6becbecd1bf31af18a..76ad686d2c268040a30dfc02096d29be28d7eedf You are receiving this

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-20 Thread Xenofon Karamanos via sr-dev
@xkaraman pushed 3 commits. 1f0f181645175e780cad19f0224d605dc0f45906 topos: Add db version check 7ab00b2513301361dea6c057d92d597a14ca6ae8 topos: Close db connection 0d8052ed91b20d23af2b6c6becbecd1bf31af18a topos: Add version_table parameter -- View it on GitHub:

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-16 Thread Daniel-Constantin Mierla via sr-dev
You also need to add documentation for the new module parameter in the xml file from doc/ subfolder. To avoid globals conflict, add `_tps_` prefix to the new global variable in the C code, to be similar to the other variables that are made modparms by topos. -- Reply to this email directly or

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-15 Thread Xenofon Karamanos via sr-dev
Updated to close the db connection and added a module parameter for being able to disable the check. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3639#issuecomment-1812438827 You are receiving this because you are subscribed to this thread.

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-15 Thread Xenofon Karamanos via sr-dev
@xkaraman pushed 2 commits. bf0ef03ce0b55fb8a7998f06bbac6434e188ca9a topos: Close db connection 0f05b4f143bc3e46e89ed477c35532fb509dbcdb topos: Add version_table parameter -- View it on GitHub:

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-15 Thread Xenofon Karamanos via sr-dev
@xkaraman pushed 2 commits. a5773fefa83a8a74399211793a6a4404a9e3d32d topos: Close db connection 02974ce0d805cbb093ddf2c5d4ac39fa8d373ee8 topos: Add version_table parameter -- View it on GitHub:

[sr-dev] Re: [kamailio/kamailio] topos: Add db version check (PR #3639)

2023-11-14 Thread Daniel-Constantin Mierla via sr-dev
If I haven't overlooked something, it seems to be missing closing db connection on mod_init, which results in inheriting it on children after fork and conflicts during runtime. Maybe it is better to move it in child_init with rank 1, we discussed a bit about that during K Devs meeting.