diff options
author | Simon Westphahl <simon.westphahl@bmw.de> | 2022-09-05 15:32:53 +0200 |
---|---|---|
committer | Simon Westphahl <simon.westphahl@bmw.de> | 2022-09-05 15:32:53 +0200 |
commit | 406aee55972ccbce6a1ebc5788e817c9ecf60e9b (patch) | |
tree | 4fdd553a17e8ad91a58971b75d5fc277bdce6a10 | |
parent | df8874a560631d38f988f27c2c4aad731ad96178 (diff) | |
download | zuul-406aee55972ccbce6a1ebc5788e817c9ecf60e9b.tar.gz |
Don't try to report build w/o buildset to DB
When a build result arrives for a non-current buildset we should skip
the reporting as we can no longer create the reference to the buildset.
Traceback (most recent call last):
File "/opt/zuul/lib/python3.10/site-packages/zuul/scheduler.py", line 2654, in _doBuildCompletedEvent
self.sql.reportBuildEnd(
File "/opt/zuul/lib/python3.10/site-packages/zuul/driver/sql/sqlreporter.py", line 143, in reportBuildEnd
db_build = self._createBuild(db, build)
File "/opt/zuul/lib/python3.10/site-packages/zuul/driver/sql/sqlreporter.py", line 180, in _createBuild
tenant=buildset.item.pipeline.tenant.name, uuid=buildset.uuid)
AttributeError: 'NoneType' object has no attribute 'item'
Change-Id: Iccbe9ab8212fbbfa21cb29b84a17e03ca221d7bd
-rw-r--r-- | zuul/driver/sql/sqlreporter.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/zuul/driver/sql/sqlreporter.py b/zuul/driver/sql/sqlreporter.py index cf75a7495..d16f50fcb 100644 --- a/zuul/driver/sql/sqlreporter.py +++ b/zuul/driver/sql/sqlreporter.py @@ -176,6 +176,8 @@ class SQLReporter(BaseReporter): start = datetime.datetime.fromtimestamp(start_time, tz=datetime.timezone.utc) buildset = build.build_set + if not buildset: + return db_buildset = db.getBuildset( tenant=buildset.item.pipeline.tenant.name, uuid=buildset.uuid) if not db_buildset: |