Menu
▾
▴
Re: [Astlinux-devel] CDR Table for Asterisk 13
|
From: Michael K. <mic...@ip...> - 2018-01-07 22:49:53
|
We have built a script which I will use in my upgrade script but I didn't realise I could also do it in the SQL Tab if I needed to so that's cool. Its pretty much a once only task so CLI is fine. PS would it be worth using the native start column rather than the custom calldate column so we don't need to add an alias? Regards Michael Knill -----Original Message----- From: Lonnie Abelbeck <li...@lo...> Reply-To: AstLinux Developers Mailing List <ast...@li...> Date: Monday, 8 January 2018 at 2:35 am To: AstLinux Developers Mailing List <ast...@li...> Subject: Re: [Astlinux-devel] CDR Table for Asterisk 13 Hi Michael, Interesting, Asterisk 11.25.3 properly fills in the following using cdr_adaptive_odbc.so -- 'answer' TEXT 'end' TEXT 'linkedid' TEXT 'peeraccount' TEXT 'sequence' TEXT -- Tip -> Use "module reload cdr_adaptive_odbc.so" to apply any SQL table changes. I guess it is simpler for users/admins to delete unused columns/fields rather than add them using the SQL-Data tab, so adding these 5 fields by default is probably a good thing. BTW, this only effects new installs when /mnt/kd/cdr-sqlite3/cdr-odbc.sqlite3 does not exist. > How would you plan on them being added by an administrator? All manual? SQL-Data tab -> CDR ODBC -> [table] cdr -> Structure There you can add and delete CDR columns/fields . Lonnie On Jan 7, 2018, at 1:34 AM, Michael Knill <mic...@ip...> wrote: > Thanks Lonnie > > I don't see any reason why you wouldn't create all the fields for 11 as well so you don't need to add them later. > Happy for actual fields created to be customised but it's a bit harder when that have already been created in init.d. > How would you plan on them being added by an administrator? All manual? > > Regards > Michael Knill > > -----Original Message----- > From: Lonnie Abelbeck <li...@lo...> > Reply-To: AstLinux Developers Mailing List <ast...@li...> > Date: Sunday, 7 January 2018 at 3:57 pm > To: AstLinux Developers Mailing List <ast...@li...> > Subject: Re: [Astlinux-devel] CDR Table for Asterisk 13 > > Hi Michael, > > Perfectly clear now, without the default /mnt/kd/cdr-sqlite3/cdr-odbc.sqlite3 containing all the fields, the cdr_adaptive_odbc.conf will skip those missing fields. > > So, the missing fields are: > -- > 'answer' TEXT > 'end' TEXT > 'linkedid' TEXT > 'peeraccount' TEXT > 'sequence' REAL > -- > agree ? > all 5 appended at the end of the current fields ? > > Hmmm, I wonder if these added fields should also be included for initial Asterisk 11 installations so later upgrades to Asterisk 13 will populate the added fields ? > > The drawback is these 5 extra fields, would all be empty in Asterisk 11 and the full "cdr" SQL-Data tab would not be very pretty. > > Do Asterisk 13 users always want all 5 extra fields ? Possibly we should simply allow users (administrators) add the fields they want with Asterisk 13 ? > > Comments ? > > Lonnie > > > > On Jan 6, 2018, at 9:57 PM, Michael Knill <mic...@ip...> wrote: > >> So sorry Lonnie for not being clear. >> This is when you are logging to sqlight3. You create the database and table on startup in init.d/asterisk if cdr-odbc.sqlite3 is not there. >> Im thinking however that init.d/asterisk should add the extra fields into the created database table if the Asterisk version is > 12. >> >> Regards >> Michael Knill >> -----Original Message----- >> From: Lonnie Abelbeck <li...@lo...> >> Reply-To: AstLinux Developers Mailing List <ast...@li...> >> Date: Sunday, 7 January 2018 at 7:11 am >> To: AstLinux Developers Mailing List <ast...@li...> >> Subject: Re: [Astlinux-devel] CDR Table for Asterisk 13 >> >> Hi Michael, >> >> AFAIF, all the CDR logged fields are a part of of you asterisk *.conf files. >> >> If not, please explain in the finest detail so I understand. >> >> Lonnie >> >> >> >> On Jan 6, 2018, at 2:04 PM, Michael Knill <mic...@ip...> wrote: >> >>> Hi Lonnie >>> >>> Unfortunately, because of the change to the way CDR’s are generated, the new fields are important. >>> Would it be possible in init.d/asterisk to check the Asterisk version and add the extra fields when using >12. >>> >>> What do you think? >>> >>> Regards >>> Michael Knill >>> >>> Sent from my iPhone so please excuse my brevity. >>> >>>> On 7 Jan 2018, at 12:29 am, Lonnie Abelbeck <li...@lo...> wrote: >>>> >>>> >>>>> On Jan 5, 2018, at 9:25 PM, Michael Knill <mic...@ip...> wrote: >>>>> >>>>> Hi Devs >>>>> >>>>> I have recently found out that a number of Asterisk CDR changes were made in Asterisk 12 including a number of new CDR fields: >>>>> https://wiki.asterisk.org/wiki/display/AST/Asterisk+12+CDR+Specification >>>>> >>>>> We will therefore need to update the automatically created cdr schema to add the new fields! >>>>> >>>>> Regards >>>>> Michael Knill >>>> >>>> Michael, is this a question or comment ? >>>> >>>> Yes the Asterisk CDR innards were changed with Asterisk 12-13, but are basically backward compatible with the Asterisk 11 configs. >>>> >>>> Lonnie > > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > _______________________________________________ > Astlinux-devel mailing list > Ast...@li... > https://lists.sourceforge.net/lists/listinfo/astlinux-devel > > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > _______________________________________________ > Astlinux-devel mailing list > Ast...@li... > https://lists.sourceforge.net/lists/listinfo/astlinux-devel ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Astlinux-devel mailing list Ast...@li... https://lists.sourceforge.net/lists/listinfo/astlinux-devel |
