[Asterisk-code-review] contrib/ast-db-manage: Add Postgres ENUM type support in aut... (asterisk[master])

Matt Jordan asteriskteam at digium.com
Mon May 4 09:26:17 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: contrib/ast-db-manage: Add Postgres ENUM type support in auto DTMF mode update
......................................................................


contrib/ast-db-manage: Add Postgres ENUM type support in auto DTMF mode update

The upgrade script for auto DTMF mode (31cd4f4891ec) added in 88b0fa7755
failed to add ENUM support for Postgres databases. This requires a
specific import from the sqlalchemy.dialects.postgresql package. This
patch corrects this error, which allows for Postgres update scripts to
be generated.

ASTERISK-24706

Change-Id: I4742ac8efa533cd6f18e0bdd907b339a9aedf015
---
M contrib/ast-db-manage/config/versions/31cd4f4891ec_add_auto_dtmf_mode.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/contrib/ast-db-manage/config/versions/31cd4f4891ec_add_auto_dtmf_mode.py b/contrib/ast-db-manage/config/versions/31cd4f4891ec_add_auto_dtmf_mode.py
index fd1b6c7..0d84390 100644
--- a/contrib/ast-db-manage/config/versions/31cd4f4891ec_add_auto_dtmf_mode.py
+++ b/contrib/ast-db-manage/config/versions/31cd4f4891ec_add_auto_dtmf_mode.py
@@ -11,6 +11,7 @@
 down_revision = '23530d604b96'
 
 from alembic import op
+from sqlalchemy.dialects.postgresql import ENUM
 import sqlalchemy as sa
 
 OLD_ENUM = ['rfc4733', 'inband', 'info']

-- 
To view, visit https://gerrit.asterisk.org/350
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4742ac8efa533cd6f18e0bdd907b339a9aedf015
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list