[asterisk-dev] [Code Review] 3167: alembic: script modifications due to errors

[Kevin Harwell]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3167/
-----------------------------------------------------------

(Updated Jan. 31, 2014, 11:29 a.m.)


Review request for Asterisk Developers.


Changes
-------

Fixed typo and added warning text to the UPGRADE.txt about the changes to the extensions migration script.


Repository: Asterisk


Description
-------

A couple of the scripts had errors that would not allow a full migration to take place.  The extensions table needed to make its 'id' column a primary key in order to work with mysql.  The other script ...add_endpoints... was missing tables that it was trying to add columns to.

Added the primary key on id for extensions and added the tables in for the missing pjsip configuration options. 


Diffs (updated)
-----

  branches/12/contrib/ast-db-manage/config/versions/581a4264e537_adding_extensions.py 406846 
  branches/12/contrib/ast-db-manage/config/versions/2fc7930b41b3_add_pjsip_endpoint_options_for_12_1.py 406846 
  branches/12/UPGRADE.txt 406846 

Diff: https://reviewboard.asterisk.org/r/3167/diff/


Testing
-------

Ran the alembic scripts against postgres and mysql database servers to make sure everything was created correctly.


Thanks,

Kevin Harwell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140131/1079f98b/attachment.html>