phase1: fix crash on exception during Git update v2
authorPetr Štetiar <ynezz@true.cz>
Mon, 15 May 2023 14:18:16 +0000 (16:18 +0200)
committerPetr Štetiar <ynezz@true.cz>
Mon, 15 May 2023 15:36:01 +0000 (17:36 +0200)
Fixes following exception:

 2023-05-15 14:08:56+0000 [-] <Build master_apm821xx/nand number:1 results:success> build got exception when running step Git(name='git', repourl='https://git.openwrt.org/openwrt/openwrt.git', mode='full', method='fresh', locks=renderer(<function NetLockDl at 0x7f5d33351ee0>), haltOnFailure=True)
 2023-05-15 14:08:56+0000 [-] Unhandled Error
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/twisted/internet/defer.py", line 1792, in gotResult
    _inlineCallbacks(r, gen, status, context)
  File "/usr/local/lib/python3.9/dist-packages/twisted/internet/defer.py", line 1693, in _inlineCallbacks
    result = context.run(
  File "/usr/local/lib/python3.9/dist-packages/twisted/python/failure.py", line 518, in throwExceptionIntoGenerator
    return g.throw(self.type, self.value, self.tb)
  File "/usr/local/lib/python3.9/dist-packages/buildbot/process/build.py", line 576, in _start_next_step_impl
    log.err(e)
--- <exception caught here> ---
  File "/usr/local/lib/python3.9/dist-packages/buildbot/process/build.py", line 562, in _start_next_step_impl
    results = yield step.startStep(self.conn)
builtins.KeyError: None

as None is valid property value, but cant be used as dict index.

Signed-off-by: Petr Štetiar <ynezz@true.cz>
phase1/master.cfg

index f511e03a5391b184073ce5453cbe2cbc0b584e48..c18ee74d746e201fbf3afa20f7bc9d2ce271442f 100644 (file)
@@ -142,7 +142,7 @@ def ini_parse_workers(section):
                log.msg("invalid worker configuration ignored: {}".format(repr(section)))
                return
 
-       sl_props = { 'dl_lock':None, 'ul_lock':None, 'tag_only':tagonly }
+       sl_props = { 'tag_only':tagonly }
        if "dl_lock" in section:
                lockname = section.get("dl_lock")
                sl_props['dl_lock'] = lockname