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: |