ahuston-0/hydra
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

merge/resync-2025-10-20 #10

Merged
ahuston-0 merged 2 commits from merge/resync-2025-10-20 into add-gitea-pulls 2025-10-20 21:20:58 -04:00
Conversation 0 Commits 2 Files Changed 3 +99 -79
3 changed files with 99 additions and 79 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9ebc15e709 - Show all commits

16
flake.lock generated
View File

@@ -3,16 +3,16 @@
"nix": { "nix": {
"flake": false, "flake": false,
"locked": { "locked": {
"lastModified": 1759956402, "lastModified": 1760573252,
"narHash": "sha256-CM27YK+KMi3HLRXqjPaJwkTabmKW+CDXOE3kMMtXH3s=", "narHash": "sha256-mcvNeNdJP5R7huOc8Neg0qZESx/0DMg8Fq6lsdx0x8U=",
"owner": "NixOS", "owner": "NixOS",
"repo": "nix", "repo": "nix",
"rev": "3019db2c87006817b6201113ad4ceee0c53c3b62", "rev": "3c39583e5512729f9c5a44c3b03b6467a2acd963",
"type": "github" "type": "github"
}, },
"original": { "original": {
"owner": "NixOS", "owner": "NixOS",
"ref": "2.31-maintenance", "ref": "2.32-maintenance",
"repo": "nix", "repo": "nix",
"type": "github" "type": "github"
} }
@@ -20,16 +20,16 @@
"nix-eval-jobs": { "nix-eval-jobs": {
"flake": false, "flake": false,
"locked": { "locked": {
"lastModified": 1757626891, "lastModified": 1760478325,
"narHash": "sha256-VrHPtHxVIboqgnw+tlCQepgtBOhBvU5hxbMHsPo8LAc=", "narHash": "sha256-hA+NOH8KDcsuvH7vJqSwk74PyZP3MtvI/l+CggZcnTc=",
"owner": "nix-community", "owner": "nix-community",
"repo": "nix-eval-jobs", "repo": "nix-eval-jobs",
"rev": "c975efc5b2bec0c1ff93c67de4a03306af258ff7", "rev": "daa42f9e9c84aeff1e325dd50fda321f53dfd02c",
"type": "github" "type": "github"
}, },
"original": { "original": {
"owner": "nix-community", "owner": "nix-community",
"ref": "v2.31.0", "ref": "v2.32.1",
"repo": "nix-eval-jobs", "repo": "nix-eval-jobs",
"type": "github" "type": "github"
} }

4
flake.nix
View File

@@ -4,13 +4,13 @@
inputs.nixpkgs.url = "github:NixOS/nixpkgs/nixos-25.05-small"; inputs.nixpkgs.url = "github:NixOS/nixpkgs/nixos-25.05-small";
inputs.nix = { inputs.nix = {
url = "github:NixOS/nix/2.31-maintenance"; url = "github:NixOS/nix/2.32-maintenance";
# We want to control the deps precisely # We want to control the deps precisely
flake = false; flake = false;
}; };
inputs.nix-eval-jobs = { inputs.nix-eval-jobs = {
url = "github:nix-community/nix-eval-jobs/v2.31.0"; url = "github:nix-community/nix-eval-jobs/v2.32.1";
# We want to control the deps precisely # We want to control the deps precisely
flake = false; flake = false;
}; };

66
src/hydra-queue-runner/build-remote.cc
View File

@@ -14,6 +14,7 @@
#include <nix/util/current-process.hh> #include <nix/util/current-process.hh>
#include <nix/util/processes.hh> #include <nix/util/processes.hh>
#include <nix/util/util.hh> #include <nix/util/util.hh>
#include <nix/store/export-import.hh>
#include <nix/store/serve-protocol.hh> #include <nix/store/serve-protocol.hh>
#include <nix/store/serve-protocol-impl.hh> #include <nix/store/serve-protocol-impl.hh>
#include <nix/store/ssh.hh> #include <nix/store/ssh.hh>
@@ -104,7 +105,7 @@ static void copyClosureTo(
std::chrono::seconds(600)); std::chrono::seconds(600));
conn.importPaths(destStore, [&](Sink & sink) { conn.importPaths(destStore, [&](Sink & sink) {
destStore.exportPaths(missing, sink); exportPaths(destStore, missing, sink);
}); });
if (readInt(conn.from) != 1) if (readInt(conn.from) != 1)
@@ -262,6 +263,7 @@ static BuildResult performBuild(
// Since this a `BasicDerivation`, `staticOutputHashes` will not // Since this a `BasicDerivation`, `staticOutputHashes` will not
// do any real work. // do any real work.
auto outputHashes = staticOutputHashes(localStore, drv); auto outputHashes = staticOutputHashes(localStore, drv);
if (auto * successP = result.tryGetSuccess()) {
for (auto & [outputName, output] : drvOutputs) { for (auto & [outputName, output] : drvOutputs) {
auto outputPath = output.second; auto outputPath = output.second;
// We’ve just asserted that the output paths of the derivation // We’ve just asserted that the output paths of the derivation
@@ -269,11 +271,12 @@ static BuildResult performBuild(
assert(outputPath); assert(outputPath);
auto outputHash = outputHashes.at(outputName); auto outputHash = outputHashes.at(outputName);
auto drvOutput = DrvOutput { outputHash, outputName }; auto drvOutput = DrvOutput { outputHash, outputName };
result.builtOutputs.insert_or_assign( successP->builtOutputs.insert_or_assign(
std::move(outputName), std::move(outputName),
Realisation { drvOutput, *outputPath }); Realisation { drvOutput, *outputPath });
} }
} }
}
return result; return result;
} }
@@ -335,54 +338,68 @@ void RemoteResult::updateWithBuildResult(const nix::BuildResult & buildResult)
startTime = buildResult.startTime; startTime = buildResult.startTime;
stopTime = buildResult.stopTime; stopTime = buildResult.stopTime;
timesBuilt = buildResult.timesBuilt; timesBuilt = buildResult.timesBuilt;
errorMsg = buildResult.errorMsg;
isNonDeterministic = buildResult.isNonDeterministic;
switch ((BuildResult::Status) buildResult.status) { std::visit(overloaded{
case BuildResult::Built: [&](const BuildResult::Success & success) {
stepStatus = bsSuccess; stepStatus = bsSuccess;
switch (success.status) {
case BuildResult::Success::Built:
break; break;
case BuildResult::Substituted: case BuildResult::Success::Substituted:
case BuildResult::AlreadyValid: case BuildResult::Success::AlreadyValid:
stepStatus = bsSuccess; case BuildResult::Success::ResolvesToAlreadyValid:
isCached = true; isCached = true;
break; break;
case BuildResult::PermanentFailure: default:
assert(false);
}
},
[&](const BuildResult::Failure & failure) {
errorMsg = failure.errorMsg;
isNonDeterministic = failure.isNonDeterministic;
switch (failure.status) {
case BuildResult::Failure::PermanentFailure:
stepStatus = bsFailed; stepStatus = bsFailed;
canCache = true; canCache = true;
errorMsg = ""; errorMsg = "";
break; break;
case BuildResult::InputRejected: case BuildResult::Failure::InputRejected:
case BuildResult::OutputRejected: case BuildResult::Failure::OutputRejected:
stepStatus = bsFailed; stepStatus = bsFailed;
canCache = true; canCache = true;
break; break;
case BuildResult::TransientFailure: case BuildResult::Failure::TransientFailure:
stepStatus = bsFailed; stepStatus = bsFailed;
canRetry = true; canRetry = true;
errorMsg = ""; errorMsg = "";
break; break;
case BuildResult::TimedOut: case BuildResult::Failure::TimedOut:
stepStatus = bsTimedOut; stepStatus = bsTimedOut;
errorMsg = ""; errorMsg = "";
break; break;
case BuildResult::MiscFailure: case BuildResult::Failure::MiscFailure:
stepStatus = bsAborted; stepStatus = bsAborted;
canRetry = true; canRetry = true;
break; break;
case BuildResult::LogLimitExceeded: case BuildResult::Failure::LogLimitExceeded:
stepStatus = bsLogLimitExceeded; stepStatus = bsLogLimitExceeded;
break; break;
case BuildResult::NotDeterministic: case BuildResult::Failure::NotDeterministic:
stepStatus = bsNotDeterministic; stepStatus = bsNotDeterministic;
canRetry = false; canRetry = false;
canCache = true; canCache = true;
break; break;
default: case BuildResult::Failure::CachedFailure:
case BuildResult::Failure::DependencyFailed:
case BuildResult::Failure::NoSubstituters:
case BuildResult::Failure::HashMismatch:
stepStatus = bsAborted; stepStatus = bsAborted;
break; break;
default:
assert(false);
} }
},
}, buildResult.inner);
} }
/* Utility guard object to auto-release a semaphore on destruction. */ /* Utility guard object to auto-release a semaphore on destruction. */
@@ -404,7 +421,7 @@ void State::buildRemote(ref<Store> destStore,
std::function<void(StepState)> updateStep, std::function<void(StepState)> updateStep,
NarMemberDatas & narMembers) NarMemberDatas & narMembers)
{ {
assert(BuildResult::TimedOut == 8); assert(BuildResult::Failure::TimedOut == 8);
auto [logFile, logFD] = build_remote::openLogFile(logDir, step->drvPath); auto [logFile, logFD] = build_remote::openLogFile(logDir, step->drvPath);
AutoDelete logFileDel(logFile, false); AutoDelete logFileDel(logFile, false);
@@ -513,7 +530,7 @@ void State::buildRemote(ref<Store> destStore,
updateStep(ssBuilding); updateStep(ssBuilding);
BuildResult buildResult = build_remote::performBuild( auto buildResult = build_remote::performBuild(
conn, conn,
*localStore, *localStore,
step->drvPath, step->drvPath,
@@ -555,7 +572,8 @@ void State::buildRemote(ref<Store> destStore,
wakeDispatcher(); wakeDispatcher();
StorePathSet outputs; StorePathSet outputs;
for (auto & [_, realisation] : buildResult.builtOutputs) if (auto * successP = buildResult.tryGetSuccess())
for (auto & [_, realisation] : successP->builtOutputs)
outputs.insert(realisation.outPath); outputs.insert(realisation.outPath);
/* Copy the output paths. */ /* Copy the output paths. */
@@ -589,7 +607,8 @@ void State::buildRemote(ref<Store> destStore,
/* Register the outputs of the newly built drv */ /* Register the outputs of the newly built drv */
if (experimentalFeatureSettings.isEnabled(Xp::CaDerivations)) { if (experimentalFeatureSettings.isEnabled(Xp::CaDerivations)) {
auto outputHashes = staticOutputHashes(*localStore, *step->drv); auto outputHashes = staticOutputHashes(*localStore, *step->drv);
for (auto & [outputName, realisation] : buildResult.builtOutputs) { if (auto * successP = buildResult.tryGetSuccess()) {
for (auto & [outputName, realisation] : successP->builtOutputs) {
// Register the resolved drv output // Register the resolved drv output
destStore->registerDrvOutput(realisation); destStore->registerDrvOutput(realisation);
@@ -600,6 +619,7 @@ void State::buildRemote(ref<Store> destStore,
destStore->registerDrvOutput(unresolvedRealisation); destStore->registerDrvOutput(unresolvedRealisation);
} }
} }
}
/* Shut down the connection. */ /* Shut down the connection. */
child->in = -1; child->in = -1;