<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/10178">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Joshua Colp: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">alembic: fix suppress_q850_reason_headers column name<br><br>In the original commit introducing the feature the column in the alembic<br>script was called 'suppress_q850_reason_header'.<br>In the code however the option is called 'suppress_q850_reason_headers'<br>(trailing 's'). This leads to errors when ARI push configuration is used.<br><br>Change-Id: Ie84808adbca6fcc9136556e4f5d741adbef5d14f<br>---<br>A contrib/ast-db-manage/config/versions/7f85dd44c775_fix_suppress_q850_reason_headers.py<br>1 file changed, 43 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/contrib/ast-db-manage/config/versions/7f85dd44c775_fix_suppress_q850_reason_headers.py b/contrib/ast-db-manage/config/versions/7f85dd44c775_fix_suppress_q850_reason_headers.py</span><br><span>new file mode 100644</span><br><span>index 0000000..911349f</span><br><span>--- /dev/null</span><br><span>+++ b/contrib/ast-db-manage/config/versions/7f85dd44c775_fix_suppress_q850_reason_headers.py</span><br><span>@@ -0,0 +1,43 @@</span><br><span style="color: hsl(120, 100%, 40%);">+"""fix suppress_q850_reason_headers</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Revision ID: 7f85dd44c775</span><br><span style="color: hsl(120, 100%, 40%);">+Revises: fe6592859b85</span><br><span style="color: hsl(120, 100%, 40%);">+Create Date: 2018-09-18 16:16:29.304815</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+"""</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+# revision identifiers, used by Alembic.</span><br><span style="color: hsl(120, 100%, 40%);">+revision = '7f85dd44c775'</span><br><span style="color: hsl(120, 100%, 40%);">+down_revision = 'fe6592859b85'</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+from alembic import op</span><br><span style="color: hsl(120, 100%, 40%);">+import sqlalchemy as sa</span><br><span style="color: hsl(120, 100%, 40%);">+from sqlalchemy.dialects.postgresql import ENUM</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+YESNO_NAME = 'yesno_values'</span><br><span style="color: hsl(120, 100%, 40%);">+YESNO_VALUES = ['yes', 'no']</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+def upgrade():</span><br><span style="color: hsl(120, 100%, 40%);">+    yesno_values = ENUM(*YESNO_VALUES, name=YESNO_NAME, create_type=False)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+    if op.get_context().bind.dialect.name != 'mssql':</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_header', type_=yesno_values,</span><br><span style="color: hsl(120, 100%, 40%);">+                        new_column_name='suppress_q850_reason_headers')</span><br><span style="color: hsl(120, 100%, 40%);">+    else:</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_header', existing_type=yesno_values, type_=sa.String(3),</span><br><span style="color: hsl(120, 100%, 40%);">+                        new_column_name='suppress_q850_reason_headers')</span><br><span style="color: hsl(120, 100%, 40%);">+        yesno_values = ENUM(*YESNO_VALUES, name=YESNO_NAME, create_type=True)</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_headers', existing_type=sa.String(3), type_=yesno_values)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+def downgrade():</span><br><span style="color: hsl(120, 100%, 40%);">+    yesno_values = ENUM(*YESNO_VALUES, name=YESNO_NAME, create_type=False)</span><br><span style="color: hsl(120, 100%, 40%);">+    if op.get_context().bind.dialect.name != 'mssql':</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_headers', type_=yesno_values,</span><br><span style="color: hsl(120, 100%, 40%);">+                        new_column_name='suppress_q850_reason_header')</span><br><span style="color: hsl(120, 100%, 40%);">+    else:</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_headers', existing_type=yesno_values, type_=sa.String(3),</span><br><span style="color: hsl(120, 100%, 40%);">+                        new_column_name='suppress_q850_reason_header')</span><br><span style="color: hsl(120, 100%, 40%);">+        yesno_values = ENUM(*YESNO_VALUES, name=YESNO_NAME, create_type=True)</span><br><span style="color: hsl(120, 100%, 40%);">+        op.alter_column('ps_endpoints', 'suppress_q850_reason_header', existing_type=sa.String(3), type_=yesno_values)</span><br><span>\ No newline at end of file</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/10178">change 10178</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10178"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ie84808adbca6fcc9136556e4f5d741adbef5d14f </div>
<div style="display:none"> Gerrit-Change-Number: 10178 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Florian Floimair <f.floimair@commend.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>