tree eeba7f20deaf8158a3ed75e157dfa58e81486e0b
parent 39689a397b274d40abb2181b64d3beca42ca98f2
author Yiwei Zhang <yiwzhang@google.com> 1601968496 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> 1601968496 +0000

Revert "[luci_context] Move writing of LUCI_CONTEXT files to the engine core."

This reverts commit 6126bc2f1092567bb3b0d96f0381330ba01ead75.

Reason for revert: Some steps are wiping out temp_dir which also accidentally deleted LUCI_CONTEXT file. See: https://crbug.com/1135137

Original change's description:
> [luci_context] Move writing of LUCI_CONTEXT files to the engine core.
>
> This removes an odd file-write from 'userspace' recipe code, but also
> enables compuation of 'timeout' for the step much closer to the actual
> step execution. Specifically, this avoids having 'timeout' also cover
> the time that we block for resource contention when running multiple
> greenlets in the recipe.
>
> R=​vadimsh, yiwzhang
>
> Bug: 1127089
> Change-Id: Iff59e7698e9556737a6144508c00cb3342811641
> Reviewed-on: https://chromium-review.googlesource.com/c/infra/luci/recipes-py/+/2441885
> Reviewed-by: Yiwei Zhang <yiwzhang@google.com>
> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>

TBR=iannucci@chromium.org,vadimsh@chromium.org,yiwzhang@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1127089
Change-Id: I9c3dec72e64238397d88293d24508b44c057491c
Reviewed-on: https://chromium-review.googlesource.com/c/infra/luci/recipes-py/+/2451447
Reviewed-by: Yiwei Zhang <yiwzhang@google.com>
Commit-Queue: Yiwei Zhang <yiwzhang@google.com>