grahamc/hydra - Change GPUIIYUKOCYHVF5VWXSGSW2LT6WWTQY6XZPWLFKZJJFIKHWO4Q2QC

Fix persistent hash mismatch errors when importing

This would start happening if the network connection between the Hydra server and the remote build server breaks after sucessfully importing at least one output of a derivation, but before having finished importing all outputs.

Fixes #816.

Created by Ricardo M. Correia on November 9, 2020

GPUIIYUKOCYHVF5VWXSGSW2LT6WWTQY6XZPWLFKZJJFIKHWO4Q2QC

Dependencies

In channels

main

upstream

Change contents

Deletion in src/hydra-queue-runner/build-remote.cc at line 484 [4.85]

B:BD[3.1896] → [3.1896:2000]

                to << cmdDumpStorePath << localStore->printStorePath(path);
                to.flush();

Insertion in src/hydra-queue-runner/build-remote.cc at line 490 [4.85]

[2.491]

                    /* Note: we should only send the command to dump the store
                       path to the remote if the NAR is actually going to get read
                       by the destination store, which won't happen if this path
                       is already valid on the destination store. Since this
                       lambda function only gets executed if someone tries to read
                       from source2, we will send the command from here rather
                       than outside the lambda. */
                    to << cmdDumpStorePath << localStore->printStorePath(path);
                    to.flush();