Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Roel Janssen
Ludovic Courtès writes: > Heya, > > Roel Janssen skribis: > >> From fcbe7ebb3d205cf7310700e62b78b9aafd94f76f Mon Sep 17 00:00:00 2001 >> From: Roel Janssen >> Date: Thu, 19 Apr 2018 17:11:30 +0200 >> Subject: [PATCH] guix-daemon: Disable garbage

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Ludovic Courtès
Heya, Roel Janssen skribis: > From fcbe7ebb3d205cf7310700e62b78b9aafd94f76f Mon Sep 17 00:00:00 2001 > From: Roel Janssen > Date: Thu, 19 Apr 2018 17:11:30 +0200 > Subject: [PATCH] guix-daemon: Disable garbage collection for remote > connections. > > *

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Marius Bakke
Roel Janssen writes: > I included a test, but I don't know how I can properly run this test. > Could you elaborate on how I can test the test(s)? [...] > diff --git a/tests/guix-daemon.sh b/tests/guix-daemon.sh > index 6f91eb58b..438c79c26 100644 > --- a/tests/guix-daemon.sh >

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Roel Janssen
Ludovic Courtès writes: > Roel Janssen skribis: > >> Ludovic Courtès writes: > > [...] > diff --git a/nix/nix-daemon/nix-daemon.cc b/nix/nix-daemon/nix-daemon.cc index deb7003d7..65770ba95 100644 ---

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Ludovic Courtès
Roel Janssen skribis: > Ludovic Courtès writes: [...] >>> diff --git a/nix/nix-daemon/nix-daemon.cc b/nix/nix-daemon/nix-daemon.cc >>> index deb7003d7..65770ba95 100644 >>> --- a/nix/nix-daemon/nix-daemon.cc >>> +++ b/nix/nix-daemon/nix-daemon.cc >>> @@

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Roel Janssen
Ludovic Courtès writes: > Hello Roel, > > Roel Janssen skribis: > [...] > >> From 00f489d6303720c65571fdf0bc9ee810a20f70e0 Mon Sep 17 00:00:00 2001 >> From: Roel Janssen >> Date: Wed, 11 Apr 2018 09:52:11 +0200 >> Subject: [PATCH]

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-19 Thread Ludovic Courtès
Hello Roel, Roel Janssen skribis: > Roel Janssen writes: > >> Ludovic Courtès writes: [...] >>> In this case, I thought guix-daemon could explicitly check whether the >>> peer is remote, and disable GC in that case. That is, ‘guix gc’

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-18 Thread Ludovic Courtès
Hello! Roel Janssen skribis: > I'm not sure this made it to the mailing list. Is the proposed patch > fine to disable the GC for remote connections? It did make it to the list, I’ll look at it real soon. Ludo’.

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-17 Thread Roel Janssen
Hello there, I'm not sure this made it to the mailing list. Is the proposed patch fine to disable the GC for remote connections? Thanks! Kind regards, Roel Janssen Roel Janssen writes: > Roel Janssen writes: > >> Ludovic Courtès

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-11 Thread Roel Janssen
Roel Janssen writes: > Ludovic Courtès writes: > >> Hello Roel, >> >> Roel Janssen skribis: >> >>> The patch adds a “disableGarbageCollection” boolean variable to the >>> guix-daemon settings, and on each occasion where a store item may be

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-03 Thread Roel Janssen
Ludovic Courtès writes: > Hello Roel, > > Roel Janssen skribis: > >> The patch adds a “disableGarbageCollection” boolean variable to the >> guix-daemon settings, and on each occasion where a store item may be >> deleted, it checks this option. >> >> This

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-03 Thread Ludovic Courtès
Hello Roel, Roel Janssen skribis: > The patch adds a “disableGarbageCollection” boolean variable to the > guix-daemon settings, and on each occasion where a store item may be > deleted, it checks this option. > > This option can be set using “--disable-gc”. > > It would be great

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-03 Thread Roel Janssen
Adam Van Ymeren writes: > Just out of curiosity, what is your situation where the daemon can't > see all GC roots? Are you sharing the store over NFS or something? Yes. And the underlying storage system is configured in such a way that “root” is not allowed to look into user's

Re: [PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-03 Thread Adam Van Ymeren
Just out of curiosity, what is your situation where the daemon can't see all GC roots? Are you sharing the store over NFS or something?On 3 Apr 2018 6:12 a.m., Roel Janssen wrote: > > Dear Guix, > > I have an interesting situation where the guix-daemon cannot see all >

[PATCH] guix-daemon: Add option to disable garbage collection.

2018-04-03 Thread Roel Janssen
TCH] guix-daemon: Add option to disable garbage collection. * nix/libstore/gc.cc: Return early on deleterious functions. * nix/libstore/globals.hh (disableGarbageCollection): New settings variable. * nix/nix-daemon/guix-daemon.cc: Implement new settings variable. * nix/nix-daemon/nix-daemon.cc: Show ap