Revert "siso: Disable remote timeout on linux-official bots"

This reverts commit f9ca08523db1ea599ecb4b2ce9ac32d7907cf892.

Reason for revert: It seems to cause this build error. https://ci.chromium.org/ui/p/chromium/builders/ci/linux-official/12996/overview

Bug: 427503493
Original change's description:
> siso: Disable remote timeout on linux-official bots
>
> Remote linking for official builds takes much longer than usual.
> e.g. 30 minutes for components_unittests
>
> "no-remote-timeout" config allows disables remote timeout easily
> for the specific builders.
>
> The build speed improves further:
> With remote linking, without remote timeout:
> 52 minutes at compile step
> https://ci.chromium.org/ui/p/chromium/builders/try/linux-official/1543/overview
>
> Only with remote linking:
> 74 minutes at compile step.
> https://ci.chromium.org/ui/p/chromium/builders/try/linux-official/1534/overview
>
> Cq-Include-Trybots: luci.chromium.try:linux-official
> Bug: 427503493
> Change-Id: I4b7bbcc706683ecae2ae756befaf80cf3df00275
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6678433
> Auto-Submit: Junji Watanabe <jwata@google.com>
> Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
> Commit-Queue: Junji Watanabe <jwata@google.com>
> Reviewed-by: Fumitoshi Ukai <ukai@google.com>
> Cr-Commit-Position: refs/heads/main@{#1480346}

Bug: 427503493
Cq-Include-Trybots: luci.chromium.try:linux-official
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I5858c11569b14a623a0daa400dc139c1c8405e01
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6689746
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Junji Watanabe <jwata@google.com>
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1480447}
NOKEYCHECK=True
GitOrigin-RevId: 63bfacd0b4086ed478cef81f94e9604d731f6f54

Author: Jwata

config/siso/config.star

@@ -26,11 +26,6 @@ __KNOWN_CONFIG_OPTIONS = [
     # because developers need objects and tests locally for debugging
     # and testing.
     "remote-link",
-
-    # Unset timeout from rules to allow long remote steps.
-    # This is useful when remote actions are expected to take longer than
-    # the configured timeouts. e.g. remote linking for official builds.
-    "no-remote-timeout",
 ]
 
 def __check(ctx):

config/siso/main.star

@@ -10,7 +10,6 @@ load("@builtin//runtime.star", "runtime")
 load("@builtin//struct.star", "module")
 load("./backend_config/backend.star", "backend")
 load("./blink_all.star", "blink_all")
-load("./config.star", "config")
 load("./gn_logs.star", "gn_logs")
 load("./linux.star", chromium_linux = "chromium")
 load("./mac.star", chromium_mac = "chromium")
@@ -29,13 +28,6 @@ def __disable_remote(ctx, step_config):
         rule["remote"] = False
     return step_config
 
-def __unset_timeout(ctx, step_config):
-    if not config.get(ctx, "no-remote-timeout"):
-        return step_config
-    for rule in step_config["rules"]:
-        rule.pop("timeout", None)
-    return step_config
-
 def init(ctx):
     print("runtime: os:%s arch:%s run:%d" % (
         runtime.os,
@@ -88,7 +80,6 @@ def init(ctx):
         rule["remote_command"] = arg0
 
     step_config = __disable_remote(ctx, step_config)
-    step_config = __unset_timeout(ctx, step_config)
 
     filegroups = {}
     filegroups.update(blink_all.filegroups(ctx))