From 0c38d5558b25f4548ba86702538c48a355e31f20 Mon Sep 17 00:00:00 2001 From: Nick Desaulniers Date: Mon, 2 Dec 2024 14:32:33 -0800 Subject: [PATCH] Revert "Do not create build requests for workers if offline longer than 30 minutes." This reverts commit 618344c61e1cd4aecfa714e65c844b734677506a and fab27e270d1a3754c98175d780bc4793e13fefc5. Otherwise we observe the failure in presubmit: $ cd buildbot/osuosl/master && BUILDBOT_TEST=1 buildbot checkconfig Configuration Errors: Unknown BuildmasterConfig key ignoreOfflineWorkersTimeout FWICT, ignoreOfflineWorkersTimeout is not documented anywhere. It has been implied that LLVM's buildbot has downstream patches. I could not find where these were published, and the issue is now blocking me from further updates to CI. Rip everything out for now. >> LLVM's buildbot seems to be deploying a forked version Link: https://github.com/llvm/llvm-zorg/commit/618344c61e1cd4aecfa714e65c844b734677506a --- buildbot/osuosl/master/master.cfg | 3 --- 1 file changed, 3 deletions(-) diff --git a/buildbot/osuosl/master/master.cfg b/buildbot/osuosl/master/master.cfg index 619d828a8..9f2167695 100644 --- a/buildbot/osuosl/master/master.cfg +++ b/buildbot/osuosl/master/master.cfg @@ -39,9 +39,6 @@ c['workers'] = config.workers.get_all() c['protocols'] = {'pb': {'port': "tcp:9990:interface=127.0.0.1" if test_mode else 9990}} -if not test_mode: # Config key only supported on downstream buildbot fork. - c['ignoreOfflineWorkersTimeout'] = 30 # minutes. - ####### CHANGESOURCES ##from buildbot.changes.pb import PBChangeSource