Skip to content

Commit

Permalink
config(modules/baseServiceConfig): Revert ProcSubset=pid (default is …
Browse files Browse the repository at this point in the history
…all)

`ProcSubset=pid` causes various problems for both nimbus and geth
(and possibly other services), so it's better to revert that change.
  • Loading branch information
MartinNikov authored and PetarKirov committed Jan 18, 2024
1 parent 6381f10 commit c44d87e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion modules/geth/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,6 @@ in {
serviceConfig = mkMerge [
baseServiceConfig
{
ProcSubset = "all";
User = serviceName;
StateDirectory = serviceName;
ExecStart = "${cfg.package}/bin/geth ${scriptArgs}";
Expand Down
1 change: 0 additions & 1 deletion modules/lib.nix
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@ lib: let
# * ProtectHome = "read-only"
DynamicUser = mkDefault true;

ProcSubset = mkDefault "pid";
ProtectClock = mkDefault true;
ProtectProc = mkDefault "noaccess";
ProtectKernelLogs = mkDefault true;
Expand Down

0 comments on commit c44d87e

Please sign in to comment.