[systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lukas Nykryn
--- src/shared/util.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/shared/util.c b/src/shared/util.c index dd67c22..6f73387 100644 --- a/src/shared/util.c +++ b/src/shared/util.c @@ -1500,7 +1500,8 @@ bool fstype_is_network(const char *fstype) { nfs\0

[systemd-devel] [PATCH] core: add startup resource control option

2014-03-24 Thread WaLyong Cho
Similar to CPUShares= and BlockIOWeight= respectively. However only assign the specified weight during startup. Each control group attribute is re-assigned as weight by CPUShares=weight and BlockIOWeight=weight after startup. If not CPUShares= or BlockIOWeight= be specified, then the attribute is

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 14:00, Michael Biebl (mbi...@gmail.com) wrote: Don't parse Red Hat style chkconfig headers if chkconfig support is disabled via --disable-chkconfig. Hmm, I am not overly hapy with adding even more ifdefs to the code... i'd rather see less ifdefs... Thus, what's the rationale

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 15:55, Lukas Nykryn (lnyk...@redhat.com) wrote: Are you sure this does what you think it does? I mean, AFAIR glusterfs is a fuse FS, and hence will nevershow up like this in fstab nor /proc/self/mountinfo... Or am I missing something? --- src/shared/util.c | 3 ++- 1 file

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:11 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 14:00, Michael Biebl (mbi...@gmail.com) wrote: Don't parse Red Hat style chkconfig headers if chkconfig support is disabled via --disable-chkconfig. Hmm, I am not overly hapy with adding even more ifdefs

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:11 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 14:00, Michael Biebl (mbi...@gmail.com) wrote: Don't parse Red Hat style chkconfig headers if chkconfig support is disabled via --disable-chkconfig. Hmm, I am not overly hapy with adding even more ifdefs

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:17, Michael Biebl (mbi...@gmail.com) wrote: 2014-03-24 17:11 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 14:00, Michael Biebl (mbi...@gmail.com) wrote: Don't parse Red Hat style chkconfig headers if chkconfig support is disabled via

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: Well, but we are a lot more dependent on correct headers than other parsers anyway... For example, the LSB deps have traditionally been much less accurately followed than they are now... I mean, we don't claim compatibility

Re: [systemd-devel] [PATCH 1/1] sd-rtnl: Introduce container parsing

2014-03-24 Thread Tom Gundersen
On Sat, Mar 22, 2014 at 5:13 PM, Susant Sahani sus...@redhat.com wrote: Introducing generic container parsing . Now supported for type FLA_INFO_KIND and IFLA_VLAN_ID which can be extended to other container parsing which is based on table based look up. ---

Re: [systemd-devel] journal handling of process title changes

2014-03-24 Thread Lennart Poettering
On Sun, 23.03.14 21:46, Patrick Donnelly (batr...@batbytes.com) wrote: My problem is not related to race conditions. The issue is that /proc/pid/cmdline is shown instead of /proc/pid/comm for each journal entry. That is: $ journalctl --boot [...] Mar 23 21:39:01 host a.out[10697]: hi Mar

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:25, Michael Biebl (mbi...@gmail.com) wrote: 2014-03-24 17:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: Well, but we are a lot more dependent on correct headers than other parsers anyway... For example, the LSB deps have traditionally been much less accurately

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lukáš Nykrýn
Po 24. březen 2014, 17:12:30 CET, Lennart Poettering napsal: On Mon, 24.03.14 15:55, Lukas Nykryn (lnyk...@redhat.com) wrote: Are you sure this does what you think it does? I mean, AFAIR glusterfs is a fuse FS, and hence will nevershow up like this in fstab nor /proc/self/mountinfo... Or am I

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 17:17, Michael Biebl (mbi...@gmail.com) wrote: 2014-03-24 17:11 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 14:00, Michael Biebl (mbi...@gmail.com) wrote: Don't parse Red Hat

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:36 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 17:25, Michael Biebl (mbi...@gmail.com) wrote: 2014-03-24 17:21 GMT+01:00 Lennart Poettering lenn...@poettering.net: Well, but we are a lot more dependent on correct headers than other parsers anyway...

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:37, Michael Biebl (mbi...@gmail.com) wrote: Hmm, I am not overly hapy with adding even more ifdefs to the code... i'd rather see less ifdefs... I can rip out the chkconfig parsing completely, if that is what you want. Nah, we need to keep it in for fedora/rhel

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:38, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Po 24. březen 2014, 17:12:30 CET, Lennart Poettering napsal: On Mon, 24.03.14 15:55, Lukas Nykryn (lnyk...@redhat.com) wrote: Are you sure this does what you think it does? I mean, AFAIR glusterfs is a fuse FS, and hence

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lukáš Nykrýn
Po 24. březen 2014, 17:48:53 CET, Lennart Poettering napsal: On Mon, 24.03.14 17:38, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Po 24. březen 2014, 17:12:30 CET, Lennart Poettering napsal: On Mon, 24.03.14 15:55, Lukas Nykryn (lnyk...@redhat.com) wrote: Are you sure this does what you think

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 17:46 GMT+01:00 Michael Biebl mbi...@gmail.com: sysv scripts would behave the same way on all systemd installations... I mean that's what we try to do after all, provide a unified interface for developers... Since Debian doesn't have chkconfig, sysv init script already behave

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 18:03, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Well, but how exactly do those lines look in fstab? Are you sure they have glusterfs in the fstype field? And not fuse.glusterfs or so? From RH Knowledgebase (again sorry not a public link)

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:46, Michael Biebl (mbi...@gmail.com) wrote: Turning off chkconfig support does a), not b). And I'd really prefer if sysv scripts would behave the same way on all systemd installations... I mean that's what we try to do after all, provide a unified interface for

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lukáš Nykrýn
Po 24. březen 2014, 18:49:34 CET, Lennart Poettering napsal: On Mon, 24.03.14 18:03, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Well, but how exactly do those lines look in fstab? Are you sure they have glusterfs in the fstype field? And not fuse.glusterfs or so? From RH Knowledgebase (again

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 19:20, Frederic Crozat (fcro...@suse.com) wrote: Le lundi 24 mars 2014 à 18:58 +0100, Lennart Poettering a écrit : It's simply that the PID file info in the chkconfig header is just increadibly useful (since it allows us to identify the main process of a service) and I'd

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 19:17, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Since glusterfs is a FUSE file system I am pretty sure it is called fuse.gluster or so in /proc/self/mountinfo, right? You are right, it shows up as fuse.glusterfs there. So any suggestions? And just drop the patch is also a

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Frederic Crozat
Le lundi 24 mars 2014 à 18:58 +0100, Lennart Poettering a écrit : It's simply that the PID file info in the chkconfig header is just increadibly useful (since it allows us to identify the main process of a service) and I'd really like to make sure we make use of it wherever possible. So that

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 19:04, Michael Biebl (mbi...@gmail.com) wrote: We could probably just drop the priority info parsing entirely, since it is pretty useless these days: as no native units carry a priority number we cannot make any useful use of the chkconfig priority anyway anymore... It

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
We could probably just drop the priority info parsing entirely, since it is pretty useless these days: as no native units carry a priority number we cannot make any useful use of the chkconfig priority anyway anymore... It is only useful to order sysv scripts with such a number against other

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 17:48, Lennart Poettering (lenn...@poettering.net) wrote: On Mon, 24.03.14 17:38, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Po 24. březen 2014, 17:12:30 CET, Lennart Poettering napsal: On Mon, 24.03.14 15:55, Lukas Nykryn (lnyk...@redhat.com) wrote: Are you sure

Re: [systemd-devel] writing a systemd unit for nbd devices

2014-03-24 Thread Lennart Poettering
On Fri, 21.03.14 21:58, Wouter Verhelst (w...@uter.be) wrote: In addition, nbd-client needs to fork() and open() the /dev/nbdX device to support partitioned NBD devices (due to a deadlock issue, that can't be done from the initial NBD_DO_IT ioctl handling, so it is done in the first

Re: [systemd-devel] [PATCH] service: add support for reboot argument when triggered by StartLimitAction=

2014-03-24 Thread Lennart Poettering
On Fri, 21.03.14 15:34, Michael Olbrich (m.olbr...@pengutronix.de) wrote: When rebooting with systemctl, an optional argument can be passed to the reboot system call. This makes it possible the specify the argument in a service file and use it when the service triggers a restart. This is

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 19:21, Lennart Poettering (lenn...@poettering.net) wrote: See the bug report I posted earlier. The discrepancy between the chkconfig head and the LSB header (the latter being the correct one) caused a dependency loop and ultimately dbus.socket to be dropped. So this is

Re: [systemd-devel] [PATCH] build: change tcpwrappers support to disabled by default

2014-03-24 Thread Lennart Poettering
On Thu, 20.03.14 20:29, Lennart Poettering (lenn...@poettering.net) wrote: To figure out what we can do in Fedora I have now started a discussion on fedora-devel, about getting rid of tcpwrap system-wide. Let's see where this goes. Would be interested in feedback about this from other distros

Re: [systemd-devel] sudo -u app_user systemctl --user

2014-03-24 Thread Lennart Poettering
On Thu, 20.03.14 12:43, David Schmitt (da...@dasz.at) wrote: The systemd --user stuff is not complete yet, and will likely be complete only when kdbus support is completed too. Note that systemd --user is only for real login users though, and the systemd instance is a singleton that is

Re: [systemd-devel] [RFC PATCH] service: add Restart option to execute StartLimitAction immediately

2014-03-24 Thread Lennart Poettering
On Thu, 20.03.14 12:52, Michael Olbrich (m.olbr...@pengutronix.de) wrote: The idea is to reboot immediately when a service crashes or the watchdog triggers. This is useful in embedded scenarios when there is only one important service. There are use-cases where rebooting immediately instead

Re: [systemd-devel] [PATCH] build: change tcpwrappers support to disabled by default

2014-03-24 Thread Cristian Rodríguez
El 24/03/14 16:16, Lennart Poettering escribió: However, I figure this doesn't have to stop us from killing it in systemd though, which I now did. This doesn't mean much however, since people who want tcpwrap support can actually easily restore it by plugging tcpd into systemd, the same way is

Re: [systemd-devel] How to manually set service in failed state

2014-03-24 Thread Lennart Poettering
On Tue, 18.03.14 14:59, Usman (deser...@yahoo.com) wrote: Hi Guys, I am trying to test some code with respect to my services setup. I have two services, lets say service_a and service_b. In service_a unit file I have this: OnFailure=service_b And from the man page for OnFailure

Re: [systemd-devel] [PATCH] Don't use class in public headers

2014-03-24 Thread Lennart Poettering
On Wed, 19.03.14 11:20, Marc-Antoine Perennou (marc-anto...@perennou.com) wrote: Applied! Thanks! --- src/systemd/sd-login.h | 2 +- src/systemd/sd-resolve.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/systemd/sd-login.h b/src/systemd/sd-login.h index

Re: [systemd-devel] [PATCH] timedated: add --timezone option to set the default timezone

2014-03-24 Thread Lennart Poettering
On Tue, 18.03.14 02:34, Juho Son (juho80@samsung.com) wrote: This option could changes the default system's time zone. The default time zone is /etc/localtime. If we want to use the specific path, we could use this option. I agree with the others on this thread, this really looks like

Re: [systemd-devel] [PATCH] timedate: fix numerous grammar errors in comments and output/debug messages

2014-03-24 Thread Lennart Poettering
On Tue, 18.03.14 00:27, Jason St. John (jstj...@purdue.edu) wrote: Applied! Thanks! --- src/timedate/timedate-sntp.c | 28 ++-- src/timedate/timedatectl.c | 25 - src/timedate/timedated.c | 25 - 3 files changed,

Re: [systemd-devel] [PATCH] src/systemd: fix grammar and spelling errors in comments

2014-03-24 Thread Lennart Poettering
On Tue, 18.03.14 01:13, Jason St. John (jstj...@purdue.edu) wrote: Doesn't apply anymore, could please rebase? --- src/systemd/_sd-common.h | 4 +-- src/systemd/sd-event.h | 6 ++-- src/systemd/sd-id128.h | 2 +- src/systemd/sd-login.h | 52

Re: [systemd-devel] [PATCH] core: add startup resource control option

2014-03-24 Thread Lennart Poettering
On Tue, 25.03.14 01:03, WaLyong Cho (walyong@samsung.com) wrote: /* Figure out which controllers we need */ -if (c-cpu_accounting || c-cpu_shares != 1024) +if (c-cpu_accounting || +c-startup_cpu_shares != 1024 || +(manager_state(m) !=

Re: [systemd-devel] [PATCH] Do not cache use_smack() value unless /sys is mounted

2014-03-24 Thread Lennart Poettering
On Fri, 14.03.14 09:35, Łukasz Stelmach (l.stelm...@samsung.com) wrote: It was 2014-03-11 wto 05:12, when Lennart Poettering wrote: On Fri, 28.02.14 17:09, Łukasz Stelmach (l.stelm...@samsung.com) wrote: use_smack() is called very early via mkdir_p_label(). This happens before /sys is

Re: [systemd-devel] Help regarding service dependency

2014-03-24 Thread Lennart Poettering
On Thu, 13.03.14 21:25, Amit Saha (as...@redhat.com) wrote: Hello, We have service1 which starts in default.target, and we want it to start After service2 (systemd-readahead-done) which starts after the default.target is reached. So, I think what would happen in this case is the

Re: [systemd-devel] systemd version debus call changed

2014-03-24 Thread Lennart Poettering
On Tue, 04.03.14 01:06, Timothée Ravier (sios...@gmail.com) wrote: On 03/03/2014 14:28, Àlex Fiestas wrote: On Friday 28 February 2014 02:28:20 Lennart Poettering wrote: They should just invoke the methods. If they get org.freedesktop.DBus.Error.UnknownMethod,

Re: [systemd-devel] [PATCH] service: don't create extra cgroup for control process when reloading SysV service

2014-03-24 Thread Lennart Poettering
On Thu, 13.03.14 09:40, Lukáš Nykrýn (lnyk...@redhat.com) wrote: Exactly. Systemd exec /etc/init.d/foo reload in control subgroup. Than the initscript kills the original deamon, starts a new one and quits. Systemd sees that the reload process finished and kills remaining processes in the

Re: [systemd-devel] [PATCH] add keyhandler support to cryptsetup

2014-03-24 Thread Lennart Poettering
On Thu, 13.03.14 18:29, Benjamin SANS (b...@ziirish.info) wrote: Hi list, Following this thread: http://lists.freedesktop.org/archives/systemd-devel/2012-July/005835.html, I understand you don't want to support a keyscript option as implemented in that patch. So I wrote these few

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Michael Biebl
2014-03-24 19:58 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 19:21, Lennart Poettering (lenn...@poettering.net) wrote: See the bug report I posted earlier. The discrepancy between the chkconfig head and the LSB header (the latter being the correct one) caused a

Re: [systemd-devel] journal handling of process title changes

2014-03-24 Thread Patrick Donnelly
On Mon, Mar 24, 2014 at 12:30 PM, Lennart Poettering lenn...@poettering.net wrote: On Sun, 23.03.14 21:46, Patrick Donnelly (batr...@batbytes.com) wrote: My problem is not related to race conditions. The issue is that /proc/pid/cmdline is shown instead of /proc/pid/comm for each journal

Re: [systemd-devel] journal handling of process title changes

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 19:01, Patrick Donnelly (batr...@batbytes.com) wrote: On Mon, Mar 24, 2014 at 12:30 PM, Lennart Poettering lenn...@poettering.net wrote: On Sun, 23.03.14 21:46, Patrick Donnelly (batr...@batbytes.com) wrote: My problem is not related to race conditions. The issue is

Re: [systemd-devel] [PATCH] core: make parsing of chkconfig headers conditional

2014-03-24 Thread Lennart Poettering
On Mon, 24.03.14 23:59, Michael Biebl (mbi...@gmail.com) wrote: 2014-03-24 19:58 GMT+01:00 Lennart Poettering lenn...@poettering.net: On Mon, 24.03.14 19:21, Lennart Poettering (lenn...@poettering.net) wrote: See the bug report I posted earlier. The discrepancy between the chkconfig

Re: [systemd-devel] How to compute a value for a service argument or environment value?

2014-03-24 Thread Lennart Poettering
On Thu, 13.03.14 14:37, Alan Stern (st...@rowland.harvard.edu) wrote: This question has probably been asked many times before, but I didn't see it mentioned anywhere on the systemd web site. I want to create a unit file for a service where the server program requires an argument or

Re: [systemd-devel] How to compute a value for a service argument or environment value?

2014-03-24 Thread Lennart Poettering
On Wed, 19.03.14 16:00, Umut Tezduyar (u...@tezduyar.com) wrote: Hi Alan, As Cristian has suggested, the correct behavior is letting daemons pick up the right argument. Though, there is a hack if you can't re implement the daemon. You could compute the variables in an ExecStartPre= and

Re: [systemd-devel] How to compute a value for a service argument or environment value?

2014-03-24 Thread Lennart Poettering
On Wed, 19.03.14 11:36, Alan Stern (st...@rowland.harvard.edu) wrote: That is, the man page for systemd.exec merely says that files listed in EnvironmentFile= directives will be read shortly before the process is executed. It doesn't say whether shortly before means before or after the

Re: [systemd-devel] Documentation error for systemctl kill?

2014-03-24 Thread Lennart Poettering
On Thu, 13.03.14 14:54, Alan Stern (st...@rowland.harvard.edu) wrote: In systemd 208 (the version currently distributed in Fedora 20), the man page for the systemctl(1) kill command says: Send a signal to one or more processes of the unit. Use --kill-who= to select which

Re: [systemd-devel] Forwarding logs to another systems journal

2014-03-24 Thread Lennart Poettering
On Fri, 14.03.14 01:24, Anand Neeli (anand.ne...@gmail.com) wrote: Hello, I have multiple systems, How do i forward logs from one system running systemd-journald to another remote systems journal service, so that all the logs are stored on a centralized machine. Have went through

Re: [systemd-devel] [PATCH 1/3] Add more password agent information

2014-03-24 Thread Lennart Poettering
On Wed, 12.02.14 23:55, David Härdeman (da...@hardeman.nu) wrote: Sorry for the late review! Add an (optional) Id key in the password agent .ask files. The Id is supposed to be a simple string in subsystem:target form which is used to provide more information on what the requested passphrase

Re: [systemd-devel] [PATCH 2/3] Fix keysize handling in cryptsetup (bits vs. bytes)

2014-03-24 Thread Lennart Poettering
On Wed, 12.02.14 23:55, David Härdeman (da...@hardeman.nu) wrote: The command line key-size is in bits but the libcryptsetup API expects bytes. This doesn't apply anymore :-( Could you rebase please? --- src/cryptsetup/cryptsetup.c |8 ++-- 1 file changed, 6 insertions(+), 2

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Zbigniew Jędrzejewski-Szmek
On Mon, Mar 24, 2014 at 07:43:00PM +0100, Lennart Poettering wrote: On Mon, 24.03.14 17:48, Lennart Poettering (lenn...@poettering.net) wrote: To me it really appears as if _netdev is the right thing to use here, and we really should advertise its use for cases like this, and close the bug as

Re: [systemd-devel] [PATCH 3/3] Fix askpass buffer overflow

2014-03-24 Thread Lennart Poettering
On Wed, 12.02.14 23:55, David Härdeman (da...@hardeman.nu) wrote: Fix askpass overflow in reading a passphrase from a tty. Doesn't seem security sensitive, but add a check for correctness. Ouch, embarassing! --- src/shared/ask-password-api.c |5 + 1 file changed, 5 insertions(+)

Re: [systemd-devel] [PATCH] Fix permissions on new journal files

2014-03-24 Thread Lennart Poettering
On Fri, 14.03.14 03:28, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: On Fri, Mar 14, 2014 at 12:07:35AM +, Greg KH wrote: When starting up journald on a new system, set the proper permissions on the system.journal file, not only on the journal directory. diff --git

Re: [systemd-devel] [PATCH] util: always consider glusterfs as a network filesystem

2014-03-24 Thread Lennart Poettering
On Tue, 25.03.14 01:17, Zbigniew Jędrzejewski-Szmek (zbys...@in.waw.pl) wrote: On Mon, Mar 24, 2014 at 07:43:00PM +0100, Lennart Poettering wrote: On Mon, 24.03.14 17:48, Lennart Poettering (lenn...@poettering.net) wrote: To me it really appears as if _netdev is the right thing to use

Re: [systemd-devel] [PATCH 1/3] Add more password agent information

2014-03-24 Thread David Härdeman
On Tue, Mar 25, 2014 at 01:03:48AM +0100, Lennart Poettering wrote: On Wed, 12.02.14 23:55, David Härdeman (da...@hardeman.nu) wrote: Add an (optional) Id key in the password agent .ask files. The Id is supposed to be a simple string in subsystem:target form which is used to provide more

Re: [systemd-devel] [PATCH 4/4] Add binary password agent protocol

2014-03-24 Thread Lennart Poettering
On Tue, 04.02.14 00:57, David Härdeman (da...@hardeman.nu) wrote: Sorry for the later review! Add binary string handling functions and extend the password agent protocol to support binary strings (using = as a string prefix instead of +). Hmm, I wished there was a different way to implement

Re: [systemd-devel] [PATCH] test-resolve.c: cleanup alittle bit and make use of sockaddr_pretty()

2014-03-24 Thread Lennart Poettering
On Fri, 14.03.14 23:53, Daniel Buch (boogiewasth...@gmail.com) wrote: +struct sockaddr_in sa = { +.sin_family = AF_INET, +.sin_addr.s_addr = inet_addr(argc = 3 ? argv[2] : 193.99.144.71), +.sin_port = htons(80) +}; I'd prefer

Re: [systemd-devel] [PATCH 1/3] Add more password agent information

2014-03-24 Thread Lennart Poettering
On Tue, 25.03.14 01:46, David Härdeman (da...@hardeman.nu) wrote: So yes, I think they're related...as in they are independent implementations of the same thing :) Would be great on settle on a common implementation... ;-) Also, please use strappend() for cases like this, where we just want

Re: [systemd-devel] [PATCH 1/3] Add more password agent information

2014-03-24 Thread David Härdeman
On Tue, Mar 25, 2014 at 01:03:48AM +0100, Lennart Poettering wrote: On Wed, 12.02.14 23:55, David Härdeman (da...@hardeman.nu) wrote: -r = ask_password_auto(text, drive-harddisk, until, accept_cached, passwords); +if (asprintf(id, cryptsetup:%s, name) 0) +return

Re: [systemd-devel] [PATCH 1/1] sd-rtnl: Introduce container parsing

2014-03-24 Thread Susant Sahani
On 03/24/2014 09:58 PM, Tom Gundersen wrote: On Sat, Mar 22, 2014 at 5:13 PM, Susant Sahani sus...@redhat.com wrote: Introducing generic container parsing . Now supported for type FLA_INFO_KIND and IFLA_VLAN_ID which can be extended to other container parsing which is based on table based look

Re: [systemd-devel] [PATCH 1/3] Add more password agent information

2014-03-24 Thread Lennart Poettering
On Tue, 25.03.14 02:13, David Härdeman (da...@hardeman.nu) wrote: Also, please use strappend() for cases like this, where we just want to concatenate two strings. Hmmm, the asprinf use there matches the style of the code of the rest of the functionfor example, with the patch applied

Re: [systemd-devel] [PATCH] timedated: add --timezone option to set the default timezone

2014-03-24 Thread juho son
On 03/25/2014 04:47 AM, Lennart Poettering wrote: On Tue, 18.03.14 02:34, Juho Son (juho80@samsung.com) wrote: This option could changes the default system's time zone. The default time zone is /etc/localtime. If we want to use the specific path, we could use this option. I agree with the

[systemd-devel] [PATCHv2] src/systemd: fix grammar and spelling errors in comments

2014-03-24 Thread Jason St. John
--- This is a resubmit that now should apply cleanly on the latest master. src/systemd/_sd-common.h | 4 +-- src/systemd/sd-event.h | 6 ++-- src/systemd/sd-id128.h | 2 +- src/systemd/sd-login.h | 52 +-- src/systemd/sd-network.h | 14

Re: [systemd-devel] [PATCH] src/systemd: fix grammar and spelling errors in comments

2014-03-24 Thread Jason St. John
On Mon, Mar 24, 2014 at 3:50 PM, Lennart Poettering lenn...@poettering.net wrote: On Tue, 18.03.14 01:13, Jason St. John (jstj...@purdue.edu) wrote: Doesn't apply anymore, could please rebase? Sure. I resubmitted the patch here:

Re: [systemd-devel] Help regarding service dependency

2014-03-24 Thread Amit Saha
- Original Message - From: Lennart Poettering lenn...@poettering.net To: Amit Saha as...@redhat.com Cc: systemd Mailing List systemd-devel@lists.freedesktop.org Sent: Tuesday, March 25, 2014 7:49:08 AM Subject: Re: [systemd-devel] Help regarding service dependency On Thu,

[systemd-devel] Restarting a socket starts the associated service

2014-03-24 Thread Tim Cuthbertson
Hi all, I've got a socket-activated service, called `myapp-main.service` and `myapp-main.socket`. These are part of myapp.target: $ cat myapp.target [Install] WantedBy=multi-user.target $ cat myapp-main.service [Unit] After=local-fs.target After=network.target Requires=myapp-main.socket

Re: [systemd-devel] Restarting a socket starts the associated service

2014-03-24 Thread Tim Cuthbertson
On Tue, Mar 25, 2014 at 3:05 PM, Tim Cuthbertson t...@gfxmonk.net wrote: Hi all, I've got a socket-activated service, called `myapp-main.service` and `myapp-main.socket`. These are part of myapp.target: $ cat myapp.target [Install] WantedBy=multi-user.target $ cat myapp-main.service

Re: [systemd-devel] Help regarding service dependency

2014-03-24 Thread Amit Saha
- Original Message - From: Andrey Borzenkov arvidj...@gmail.com To: Amit Saha as...@redhat.com Cc: systemd Mailing List systemd-devel@lists.freedesktop.org Sent: Friday, March 14, 2014 7:08:28 PM Subject: Re: [systemd-devel] Help regarding service dependency On Fri, Mar 14, 2014

Re: [systemd-devel] Restarting a socket starts the associated service

2014-03-24 Thread David Timothy Strauss
It's already reported (by me): https://bugs.freedesktop.org/show_bug.cgi?id=75185 ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] journal entries for most recent pid?

2014-03-24 Thread Peter Hutterer
I can't seem to find this in the documentation, so I do wonder if it exists: is there an equivalent to journalctl -b -1 that lists the log of the most recent pid? Specifically I'd like to run journalctl /usr/bin/Xorg --pid -1 or similar to get users to attach the most recent log file instead of