Merge pull request #13268 from NixOS/derivation-builder-cleanup

Remove `buildUser` from `DerivationBuilder`

Author: Ericson2314

src/libstore/build/derivation-building-goal.cc

@@ -655,8 +655,8 @@ Goal::Co DerivationBuildingGoal::tryToBuild()
         builder->startBuilder();
 
     } catch (BuildError & e) {
+        builder.reset();
         outputLocks.unlock();
-        builder->buildUser.reset();
         worker.permanentFailure = true;
         co_return done(BuildResult::InputRejected, {}, std::move(e));
     }

src/libstore/unix/build/derivation-builder.cc

@@ -106,6 +106,11 @@ class DerivationBuilderImpl : public DerivationBuilder, DerivationBuilderParams
 
 private:
 
+    /**
+     * User selected for running the builder.
+     */
+    std::unique_ptr<UserLock> buildUser;
+
     /**
      * The cgroup of the builder, if any.
      */
@@ -251,30 +256,10 @@ class DerivationBuilderImpl : public DerivationBuilder, DerivationBuilderParams
 
 public:
 
-    /**
-     * Set up build environment / sandbox, acquiring resources (e.g.
-     * locks as needed). After this is run, the builder should be
-     * started.
-     *
-     * @returns true if successful, false if we could not acquire a build
-     * user. In that case, the caller must wait and then try again.
-     */
     bool prepareBuild() override;
 
-    /**
-     * Start building a derivation.
-     */
-    void startBuilder() override;;
+    void startBuilder() override;
 
-    /**
-     * Tear down build environment after the builder exits (either on
-     * its own or if it is killed).
-     *
-     * @returns The first case indicates failure during output
-     * processing. A status code and exception are returned, providing
-     * more information. The second case indicates success, and
-     * realisations for each output of the derivation are returned.
-     */
     std::variant<std::pair<BuildResult::Status, Error>, SingleDrvOutputs> unprepareBuild() override;
 
 private:
@@ -306,10 +291,6 @@ class DerivationBuilderImpl : public DerivationBuilder, DerivationBuilderParams
 
 public:
 
-    /**
-     * Stop the in-process nix daemon thread.
-     * @see startDaemon
-     */
     void stopDaemon() override;
 
 private:
@@ -341,15 +322,8 @@ class DerivationBuilderImpl : public DerivationBuilder, DerivationBuilderParams
 
 public:
 
-    /**
-     * Delete the temporary directory, if we have one.
-     */
     void deleteTmpDir(bool force) override;
 
-    /**
-     * Kill any processes running under the build user UID or in the
-     * cgroup of the build.
-     */
     void killSandbox(bool getStats) override;
 
 private:

src/libstore/unix/include/nix/store/build/derivation-builder.hh

@@ -138,11 +138,6 @@ struct DerivationBuilderCallbacks
  */
 struct DerivationBuilder : RestrictionContext
 {
-    /**
-     * User selected for running the builder.
-     */
-    std::unique_ptr<UserLock> buildUser;
-
     /**
      * The process ID of the builder.
      */