Add oldrev/newrev column to the buildset reporter table
This change adds oldrev/newrev column to the zuul_buildset sql table to be able to query post job for a specific commit. Change-Id: Ic9c0b260560123d080fa47c47a76fcfa31eb3607
This commit is contained in:
parent
305a21586b
commit
8198c0ef45
|
@ -69,7 +69,7 @@ class TestSQLConnection(ZuulDBTestCase):
|
||||||
insp = sa.engine.reflection.Inspector(
|
insp = sa.engine.reflection.Inspector(
|
||||||
self.connections.connections['resultsdb'].engine)
|
self.connections.connections['resultsdb'].engine)
|
||||||
|
|
||||||
self.assertEqual(11, len(insp.get_columns(buildset_table)))
|
self.assertEqual(13, len(insp.get_columns(buildset_table)))
|
||||||
self.assertEqual(10, len(insp.get_columns(build_table)))
|
self.assertEqual(10, len(insp.get_columns(build_table)))
|
||||||
|
|
||||||
def test_sql_results(self):
|
def test_sql_results(self):
|
||||||
|
|
|
@ -0,0 +1,26 @@
|
||||||
|
"""Add oldrev/newrev columns
|
||||||
|
|
||||||
|
Revision ID: ba4cdce9b18c
|
||||||
|
Revises: 5efb477fa963
|
||||||
|
Create Date: 2017-09-27 19:33:21.800198
|
||||||
|
|
||||||
|
"""
|
||||||
|
|
||||||
|
# revision identifiers, used by Alembic.
|
||||||
|
revision = 'ba4cdce9b18c'
|
||||||
|
down_revision = '5efb477fa963'
|
||||||
|
branch_labels = None
|
||||||
|
depends_on = None
|
||||||
|
|
||||||
|
from alembic import op
|
||||||
|
import sqlalchemy as sa
|
||||||
|
|
||||||
|
|
||||||
|
def upgrade():
|
||||||
|
op.add_column('zuul_buildset', sa.Column('oldrev', sa.String(255)))
|
||||||
|
op.add_column('zuul_buildset', sa.Column('newrev', sa.String(255)))
|
||||||
|
|
||||||
|
|
||||||
|
def downgrade():
|
||||||
|
op.drop_column('zuul_buildset', 'newrev')
|
||||||
|
op.drop_column('zuul_buildset', 'oldrev')
|
|
@ -91,6 +91,8 @@ class SQLConnection(BaseConnection):
|
||||||
sa.Column('change', sa.Integer, nullable=True),
|
sa.Column('change', sa.Integer, nullable=True),
|
||||||
sa.Column('patchset', sa.Integer, nullable=True),
|
sa.Column('patchset', sa.Integer, nullable=True),
|
||||||
sa.Column('ref', sa.String(255)),
|
sa.Column('ref', sa.String(255)),
|
||||||
|
sa.Column('oldrev', sa.String(255)),
|
||||||
|
sa.Column('newrev', sa.String(255)),
|
||||||
sa.Column('ref_url', sa.String(255)),
|
sa.Column('ref_url', sa.String(255)),
|
||||||
sa.Column('result', sa.String(255)),
|
sa.Column('result', sa.String(255)),
|
||||||
sa.Column('message', sa.TEXT()),
|
sa.Column('message', sa.TEXT()),
|
||||||
|
|
|
@ -36,6 +36,8 @@ class SQLReporter(BaseReporter):
|
||||||
change = getattr(item.change, 'number', '')
|
change = getattr(item.change, 'number', '')
|
||||||
patchset = getattr(item.change, 'patchset', '')
|
patchset = getattr(item.change, 'patchset', '')
|
||||||
ref = getattr(item.change, 'ref', '')
|
ref = getattr(item.change, 'ref', '')
|
||||||
|
oldrev = getattr(item.change, 'oldrev', '')
|
||||||
|
newrev = getattr(item.change, 'newrev', '')
|
||||||
buildset_ins = self.connection.zuul_buildset_table.insert().values(
|
buildset_ins = self.connection.zuul_buildset_table.insert().values(
|
||||||
zuul_ref=item.current_build_set.ref,
|
zuul_ref=item.current_build_set.ref,
|
||||||
pipeline=item.pipeline.name,
|
pipeline=item.pipeline.name,
|
||||||
|
@ -43,6 +45,8 @@ class SQLReporter(BaseReporter):
|
||||||
change=change,
|
change=change,
|
||||||
patchset=patchset,
|
patchset=patchset,
|
||||||
ref=ref,
|
ref=ref,
|
||||||
|
oldrev=oldrev,
|
||||||
|
newrev=newrev,
|
||||||
ref_url=item.change.url,
|
ref_url=item.change.url,
|
||||||
result=item.current_build_set.result,
|
result=item.current_build_set.result,
|
||||||
message=self._formatItemReport(
|
message=self._formatItemReport(
|
||||||
|
|
Loading…
Reference in New Issue