Re: [systemd-devel] [RFC PATCH] service: add FailureAction= option

2014-04-11 Thread Michael Olbrich
On Fri, Apr 11, 2014 at 03:34:42AM +0200, Lennart Poettering wrote: On Wed, 26.03.14 10:02, Michael Olbrich (m.olbr...@pengutronix.de) wrote: It has the same possible values as StartLimitAction= and is executed immediately if a service fails. I think the enum type should probably be

Re: [systemd-devel] [RFC PATCH] service: add FailureAction= option

2014-04-11 Thread Lennart Poettering
On Fri, 11.04.14 09:48, Michael Olbrich (m.olbr...@pengutronix.de) wrote: +else if (allow_restart I would drop the else here, I think. Is there a reason not to do the restart thing anyway? If it is configured, it should run I think, just in case the failure action doesn't

Re: [systemd-devel] Systemd 212 RemoveIPC=yes breaks postgresql

2014-04-11 Thread Reindl Harald
Am 11.04.2014 03:01, schrieb Lennart Poettering: Tradditionally on Linux this is done via login.defs, but I am really not convinced this should be runtime configurable. I'd prefer to detect the minimal normal UID at compile time, and then use that to distuingish normal from system users

Re: [systemd-devel] auditd.service and RefuseManualStop

2014-04-11 Thread Steve Grubb
On Friday, April 11, 2014 03:14:31 AM Lennart Poettering wrote: On Thu, 03.04.14 18:00, Colin Guthrie (gm...@colin.guthr.ie) wrote: Alternatively we can do systemctl kill in this case prior to uninstall and that will work (systemctl kill does not respect RefuseManualStop). Yeah, this is

Re: [systemd-devel] auditd.service and RefuseManualStop

2014-04-11 Thread Lennart Poettering
On Fri, 11.04.14 10:29, Steve Grubb (sgr...@redhat.com) wrote: On Friday, April 11, 2014 03:14:31 AM Lennart Poettering wrote: On Thu, 03.04.14 18:00, Colin Guthrie (gm...@colin.guthr.ie) wrote: Alternatively we can do systemctl kill in this case prior to uninstall and that will work

Re: [systemd-devel] [PATCH] fsck: Search for fsck.type in PATH

2014-04-11 Thread Lennart Poettering
On Wed, 09.04.14 10:07, Mike Gilbert (flop...@gentoo.org) wrote: Matches default behavior in recent util-linux. Quite frankly, this is really really broken in Gentoo. Randomly moving packages from /sbin to /usr/sbin that are required during early boot is just wrong. Either you keep the

Re: [systemd-devel] Two git trees

2014-04-11 Thread David Timothy Strauss
Tests are back to passing, so GitHub is now in sync with the freedesktop.org git. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH 1/1] networkd: Introduce ipip tunnel

2014-04-11 Thread Susant Sahani
On 04/09/2014 01:33 AM, Umut Tezduyar Lindskog wrote: Hi, Few minor suggestions, if you care. Hi sorry for replying late :) On Tue, Apr 8, 2014 at 5:22 AM, Susant Sahani sus...@redhat.com wrote: This patch enables basic ipip tunnel support. It works with kernel module ipip Example

Re: [systemd-devel] [PATCH 1/1] networkd: Introduce ipip tunnel

2014-04-11 Thread Susant Sahani
On 04/08/2014 12:54 PM, Jóhann B. Guðmundsson wrote: On 04/08/2014 03:22 AM, Susant Sahani wrote: file: ipip.netdev -- [NetDev] Name=ipip-tun Kind=ipip [Tunnel] Local=192.168.8.102 Remote=10.4.4.4 TTL=64 MTUBytes=1480 file: ipip.network --

Re: [systemd-devel] Systemd 212 RemoveIPC=yes breaks postgresql

2014-04-11 Thread David Timothy Strauss
On Thu, Apr 10, 2014 at 9:01 PM, Lennart Poettering lenn...@poettering.net wrote: Yuck, I figure we need to ignore RemoveIPC for all system users, not just for root. This still seems dangerous to me. I'm sure I have services running under users where I've forgotten the system flag for the

Re: [systemd-devel] Systemd 212 RemoveIPC=yes breaks postgresql

2014-04-11 Thread David Timothy Strauss
Maybe if any service is running something as a user *or* it's a system user, that user is immune to RemoveIPC? ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] Get exit status inside ExecStopPost ?

2014-04-11 Thread Anand Patil
Hi everyone, In a service's ExecStopPost, is there an easy way to access the exit status of the service's ExecStart process? Thanks, Anand Patil ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org

[systemd-devel] [PATCH] silence warning

2014-04-11 Thread Thomas H.P. Andersen
From: Thomas Hindoe Paaboel Andersen pho...@gmail.com The error message logged in finish: will not be very informative but on the other hand I don't see how this should ever happen. --- src/network/networkd-link.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

Re: [systemd-devel] [PATCH] fsck: Search for fsck.type in PATH

2014-04-11 Thread Mike Gilbert
On Fri, Apr 11, 2014 at 11:48 AM, Lennart Poettering lenn...@poettering.net wrote: On Wed, 09.04.14 10:07, Mike Gilbert (flop...@gentoo.org) wrote: Matches default behavior in recent util-linux. Quite frankly, this is really really broken in Gentoo. Randomly moving packages from /sbin to

Re: [systemd-devel] [PATCH] silence warning

2014-04-11 Thread Tom Gundersen
On Fri, Apr 11, 2014 at 10:17 PM, Thomas H.P. Andersen pho...@gmail.com wrote: From: Thomas Hindoe Paaboel Andersen pho...@gmail.com The error message logged in finish: will not be very informative but on the other hand I don't see how this should ever happen. This can never happen, unless

[systemd-devel] [PATCH] build: if -fstack-protector-strong is available, use it.

2014-04-11 Thread Cristian Rodríguez
--- configure.ac | 1 + 1 file changed, 1 insertion(+) diff --git a/configure.ac b/configure.ac index 5074114..d40fb38 100644 --- a/configure.ac +++ b/configure.ac @@ -156,6 +156,7 @@ CC_CHECK_FLAGS_APPEND([with_cflags], [CFLAGS], [\ -ffunction-sections \ -fdata-sections \

[systemd-devel] [PATCH 2/2] path-lookup: don't hardcode .config

2014-04-11 Thread Tanu Kaskinen
If XDG_CONFIG_HOME is set, then we should respect that. --- src/shared/path-lookup.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/shared/path-lookup.c b/src/shared/path-lookup.c index 63af43c..a885b66 100644 --- a/src/shared/path-lookup.c +++

[systemd-devel] [PATCH 1/2] man: mention XDG_CONFIG_HOME in systemd.unit

2014-04-11 Thread Tanu Kaskinen
--- man/systemd.unit.xml | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/man/systemd.unit.xml b/man/systemd.unit.xml index 07a73fd..bcd4ba8 100644 --- a/man/systemd.unit.xml +++ b/man/systemd.unit.xml @@ -70,7 +70,8 @@ filename.../filename

[systemd-devel] [PATCH 0/2] XDG_CONFIG_HOME fixes

2014-04-11 Thread Tanu Kaskinen
I was reading the systemd.unit man page, and noticed that it mentioned $HOME/.config/systemd/user, but not $XDG_CONFIG_HOME/systemd/user, even though the latter is used instead of the former when possible. The first patch fixes this documentation issue. I then looked for hardcoded instances of