wb | remove eventlogged flag

Author: fmaste

nix/workbench/backend/nomad.nix

@@ -2,8 +2,6 @@
 , lib
 , stateDir
 , subBackendName
-# TODO: Fetch this from config services inside materialise-profile !
-, eventlogged ? true
 , ...
 }:
 let
@@ -147,20 +145,15 @@ let
       # the one used to enter the shell ??????????
       cardano-node = rec {
         # Local reference only used if not "cloud".
-        nix-store-path = with pkgs;
-          if eventlogged
-            then cardanoNodePackages.cardano-node.passthru.eventlogged
-            else cardanoNodePackages.cardano-node.passthru.noGitRev
-        ;
+        # Avoid rebuilding on every commit because of `set-git-rev`.
+        nix-store-path = pkgs.cardanoNodePackages.cardano-node.passthru.noGitRev;
         flake-reference = "github:intersectmbo/cardano-node";
-        flake-output =
-          if eventlogged
-            then "cardanoNodePackages.cardano-node.passthru.eventlogged"
-            else "cardanoNodePackages.cardano-node.passthru.noGitRev"
+        flake-output = "cardanoNodePackages.cardano-node.passthru.noGitRev"
         ;
       };
       cardano-cli = rec {
         # Local reference only used if not "cloud".
+        # Avoid rebuilding on every commit because of `set-git-rev`.
         nix-store-path = pkgs.cardanoNodePackages.cardano-cli.passthru.noGitRev;
         flake-reference = "github:input-output-hk/cardano-cli";
         flake-output = "cardanoNodePackages.cardano-cli.passthru.noGitRev";

nix/workbench/service/healthcheck.nix

@@ -2,7 +2,6 @@
 , backend
 , profile
 , nodeSpecs
-, eventlogged ? true
 }:
 
 with pkgs.lib;
@@ -14,18 +13,16 @@ let
   grep            = pkgs.gnugrep;
   jq              = pkgs.jq;
   # Avoid rebuilding the script on every commit.
-  # Both `eventlogged` and `noGitRev` do not have `set-git-rev`.
+  # `noGitRev` does not have `set-git-rev` that is set on every commit.
   cardano-node = with pkgs;
     if backend.useCabalRun
     then "cardano-node"
-    else if eventlogged
-         then cardanoNodePackages.cardano-node.passthru.eventlogged + "/bin/cardano-node"
-         else cardanoNodePackages.cardano-node.passthru.noGitRev    + "/bin/cardano-node"
+    else cardanoNodePackages.cardano-node.passthru.noGitRev + "/bin/cardano-node"
   ;
   cardano-cli  = with pkgs;
     if backend.useCabalRun
     then "cardano-cli"
-    else cardanoNodePackages.cardano-cli.passthru.noGitRev          + "/bin/cardano-cli";
+    else cardanoNodePackages.cardano-cli.passthru.noGitRev + "/bin/cardano-cli";
 in {
   start =
     # Assumptions:

nix/workbench/service/nodes.nix

@@ -137,11 +137,11 @@ let
     } // optionalAttrs (profiling != "none") {
       inherit profiling;
     } // optionalAttrs (profiling == "none") {
-      # Switch to `cardano-node.passthru.noGitRev` if eventlog is not needed.
-      eventlog               = mkForce true;
+      # Switch to `noGitRev` to avoid rebuilding with every commit.
+      package    = pkgs.cardano-node.passthru.noGitRev;
     } // optionalAttrs backend.useCabalRun {
       # Allow the shell function to take precedence.
-      executable             = "cardano-node";
+      executable = "cardano-node";
     } // optionalAttrs isProducer {
       operationalCertificate = "../genesis/node-keys/node${toString i}.opcert";
       kesKey                 = "../genesis/node-keys/node-kes${toString i}.skey";