Re: [systemd-devel] systemd services via SSH (-H key)

2015-10-22 Thread Ivan Shapovalov
ner]" and it does not allow specifying an explicit port number. Why don't you read manpages before replying? -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part ___ systemd-devel mailing list systemd-

Re: [systemd-devel] Failed at step RUNTIME_DIRECTORY spawning /usr/bin/true: File exists

2015-08-13 Thread Ivan Shapovalov
On 2015-08-10 at 15:46 +0200, Reindl Harald wrote: Am 10.08.2015 um 15:28 schrieb Ivan Shapovalov: On 2015-08-10 at 15:14 +0200, Reindl Harald wrote: Am 10.08.2015 um 15:05 schrieb Ivan Shapovalov: On 2015-08-10 at 11:16 +0200, Reindl Harald wrote: Moreover

Re: [systemd-devel] Failed at step RUNTIME_DIRECTORY spawning /usr/bin/true: File exists

2015-08-10 Thread Ivan Shapovalov
On 2015-08-10 at 11:16 +0200, Reindl Harald wrote: Am 10.08.2015 um 07:57 schrieb Ivan Shapovalov: On 2015-08-06 at 15:01 +0200, Michael Biebl wrote: 2015-08-06 14:43 GMT+02:00 Reindl Harald h.rei...@thelounge.net : well, but Type=simple is default and recommended everywhere

Re: [systemd-devel] Failed at step RUNTIME_DIRECTORY spawning /usr/bin/true: File exists

2015-08-10 Thread Ivan Shapovalov
On 2015-08-10 at 15:14 +0200, Reindl Harald wrote: Am 10.08.2015 um 15:05 schrieb Ivan Shapovalov: On 2015-08-10 at 11:16 +0200, Reindl Harald wrote: Moreover, * RuntimeDirectory is a service configuration * the daemon is started as unprivileged user * RuntimeDirectory

Re: [systemd-devel] Failed at step RUNTIME_DIRECTORY spawning /usr/bin/true: File exists

2015-08-09 Thread Ivan Shapovalov
10hrs after starting due to an internal error. To make things more robust, the daemon should either *properly* implement Type=forking (i. e. fork only after initial start-up) or it should implement Type=notify. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed

Re: [systemd-devel] oneshot service

2015-07-07 Thread Ivan Shapovalov
way. HTHs, -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-05-15 Thread Ivan Shapovalov
On 2015-05-14 at 22:31 +0200, Lennart Poettering wrote: On Thu, 07.05.15 04:37, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-05-06 at 18:59 +0200, Lennart Poettering wrote: On Wed, 06.05.15 19:53, Andrei Borzenkov (arvidj...@gmail.com) wrote: I still think that being

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-05-15 Thread Ivan Shapovalov
On 2015-05-15 at 11:54 +0200, Lennart Poettering wrote: On Fri, 15.05.15 11:38, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hmm, if , I think this should be fixable though. Already, allocating a unit, loading a unit and starting a unit are three separate steps. It shouldn't be too

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-05-06 Thread Ivan Shapovalov
On 2015-05-06 at 09:16 +0300, Andrei Borzenkov wrote: On Wed, May 6, 2015 at 5:52 AM, Ivan Shapovalov intelfx...@gmail.com wrote: On 2015-04-24 at 11:10 +0200, Lennart Poettering wrote: On Fri, 24.04.15 04:07, Ivan Shapovalov (intelfx...@gmail.com) wrote: - do `systemd-run

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-05-06 Thread Ivan Shapovalov
help here, though they suffer from the same kind of problem: either auxiliary units are read from message and created before the main one, or vice versa. The problems are the same as with two consecutive StartTransientUnit calls. -- Ivan Shapovalov / intelfx / signature.asc Description

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-05-05 Thread Ivan Shapovalov
On 2015-04-24 at 11:10 +0200, Lennart Poettering wrote: On Fri, 24.04.15 04:07, Ivan Shapovalov (intelfx...@gmail.com) wrote: - do `systemd-run` twice and somehow set up the dependencies between two transient units I'd be happy to take a patch that allows configuring deps

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-27 Thread Ivan Shapovalov
On 2015-04-27 at 17:14 +0200, Lennart Poettering wrote: On Sat, 25.04.15 05:48, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-04-25 at 04:00 +0300, Ivan Shapovalov wrote: On 2015-04-24 at 16:04 +0200, Lennart Poettering wrote: [...] Actually, it really is about

[systemd-devel] [PATCHv3] core: coldplug all units which participate in jobs during coldplugging

2015-04-27 Thread Ivan Shapovalov
This is yet another attempt to fix coldplugging order (more especially, the problem which happens when one creates a job during coldplugging and it references a not-yet-coldplugged unit). Now we forcibly coldplug all units which participate in jobs. This is a superset of previously implemented

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-24 Thread Ivan Shapovalov
not yet marked)On 2015-04-24 at 15:52 +0200, Lennart Poettering wrote: On Wed, 25.02.15 21:40, Ivan Shapovalov (intelfx...@gmail.com) wrote: Ivan, Because the order of coldplugging is not defined, we can reference a not-yet-coldplugged unit and read its state while it has not yet

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-24 Thread Ivan Shapovalov
- NB: basic.target is not coldplugged yet! - A.path enters running and starts A.service - transaction_add_job_and_dependencies() adds jobs for all dependencies of A.service - at this point we're fucked up: basic.target is not coldplugged, but a job is added for it -- Ivan Shapovalov / intelfx

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-24 Thread Ivan Shapovalov
On 2015-04-25 at 04:00 +0300, Ivan Shapovalov wrote: On 2015-04-24 at 16:04 +0200, Lennart Poettering wrote: [...] Actually, it really is about the UNIT_TRIGGERS dependencies only, since we don't do the retroactive deps stuff at all when we are coldplugging, it's conditionalized in m

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-24 Thread Ivan Shapovalov
without it you'll still get the above messages due to basic.target being re-started. I'll try to look into code and see why your method fails... -- Ivan Shapovalov / intelfx / -- Ivan Shapovalov / intelfx / -- Ivan Shapovalov / intelfx / -- Ivan Shapovalov / intelfx / signature.asc Description

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-04-24 Thread Ivan Shapovalov
On 2015-04-24 at 20:19 +0200, Lennart Poettering wrote: On Fri, 24.04.15 20:46, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-04-24 at 19:13 +0200, Lennart Poettering wrote: On Fri, 24.04.15 20:06, Ivan Shapovalov (intelfx...@gmail.com) wrote: With this patch applied

Re: [systemd-devel] [PATCH 1/3] sysusers: allow separate alternate roots for configs and modifications

2015-04-23 Thread Ivan Shapovalov
On 2015-04-23 at 14:11 +0200, Lennart Poettering wrote: On Thu, 26.02.15 02:46, Ivan Shapovalov (intelfx...@gmail.com) wrote: This is useful, for example, to create system accounts on an initramfs using the host's configuration. Hmm, but you can already do this, by specifiying

Re: [systemd-devel] [PATCH 1/3] sysusers: allow separate alternate roots for configs and modifications

2015-04-23 Thread Ivan Shapovalov
On 2015-04-23 at 16:48 +0200, Lennart Poettering wrote: On Thu, 23.04.15 17:46, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-04-23 at 14:11 +0200, Lennart Poettering wrote: On Thu, 26.02.15 02:46, Ivan Shapovalov (intelfx...@gmail.com) wrote: This is useful

Re: [systemd-devel] [PATCH 1/3] sysusers: allow separate alternate roots for configs and modifications

2015-04-23 Thread Ivan Shapovalov
On 2015-04-23 at 17:15 +0200, Lennart Poettering wrote: On Thu, 23.04.15 18:09, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-04-23 at 16:48 +0200, Lennart Poettering wrote: On Thu, 23.04.15 17:46, Ivan Shapovalov (intelfx...@gmail.com) wrote: On 2015-04-23 at 14:11

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-04-23 Thread Ivan Shapovalov
On 2015-04-08 at 19:28 +0200, Lennart Poettering wrote: On Mon, 23.03.15 16:04, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hello, is it possible/allowed/desired to support assigning ExecStartPre= and similar options via dbus interface, i. e. in `systemctl set -property

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-03-24 Thread Ivan Shapovalov
On 2015-03-23 at 13:45 +, Zbigniew Jędrzejewski-Szmek wrote: On Mon, Mar 23, 2015 at 04:04:28PM +0300, Ivan Shapovalov wrote: Hello, is it possible/allowed/desired to support assigning ExecStartPre= and similar options via dbus interface, i. e. in `systemctl set-property

[systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-03-23 Thread Ivan Shapovalov
to support, isn't it? Cheers, -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd

Re: [systemd-devel] Supporting ExecStartPre= and friends in `systemctl set-property` or `systemd-run -p`

2015-03-23 Thread Ivan Shapovalov
On 2015-03-23 at 13:45 +, Zbigniew Jędrzejewski-Szmek wrote: On Mon, Mar 23, 2015 at 04:04:28PM +0300, Ivan Shapovalov wrote: Hello, is it possible/allowed/desired to support assigning ExecStartPre= and similar options via dbus interface, i. e. in `systemctl set-property

[systemd-devel] [PATCHv2] sysusers: do not reject users with already present /etc/shadow entries

2015-03-07 Thread Ivan Shapovalov
This is needed to interoperate firstboot and sysusers. The former one is started first, and it writes only /etc/shadow when it is told to set the root password. It's better to relax checks here than to duplicate functionality in firstboot. --- v2: rebased on top of master

Re: [systemd-devel] [PATCH 0/3] using firstboot and sysusers to construct an initramfs

2015-03-07 Thread Ivan Shapovalov
On 2015-03-07 at 15:01 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Thu, Mar 05, 2015 at 09:42:17PM +0300, Ivan Shapovalov wrote: On 2015-03-05 at 19:16 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Thu, Mar 05, 2015 at 09:09:54PM +0300, Ivan Shapovalov wrote: On 2015-02-26 at 02:53 +0300

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-03-05 Thread Ivan Shapovalov
On 2015-02-28 at 00:50 +0300, Ivan Shapovalov wrote: On 2015-02-27 at 22:25 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Wed, Feb 25, 2015 at 09:40:23PM +0300, Ivan Shapovalov wrote: Because the order of coldplugging is not defined, we can reference a not-yet-coldplugged unit and read its

Re: [systemd-devel] [PATCH 0/3] using firstboot and sysusers to construct an initramfs

2015-03-05 Thread Ivan Shapovalov
On 2015-02-26 at 02:53 +0300, Ivan Shapovalov wrote: On 2015-02-26 at 02:46 +0300, Ivan Shapovalov wrote: Hi there. These patches allow using firstboot and sysusers together to construct an initramfs with a fully functional emergency.service and rescue.service. Moreover, they allow

Re: [systemd-devel] [PATCH 0/3] using firstboot and sysusers to construct an initramfs

2015-03-05 Thread Ivan Shapovalov
On 2015-03-05 at 19:16 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Thu, Mar 05, 2015 at 09:09:54PM +0300, Ivan Shapovalov wrote: On 2015-02-26 at 02:53 +0300, Ivan Shapovalov wrote: On 2015-02-26 at 02:46 +0300, Ivan Shapovalov wrote: Hi there. These patches allow using

Re: [systemd-devel] [PATCH] journal: fix Inappropriate ioctl for device on ext4

2015-03-01 Thread Ivan Shapovalov
= chattr_fd(f-fd, true, FS_NOCOW_FL); if (r 0) -log_warning_errno(errno, Failed to set file attributes: %m); +if(r != -ENOTTY) Maybe it's better to just say if (r 0 r != -ENOTTY) here? Just nitpicking, -- Ivan Shapovalov / intelfx

Re: [systemd-devel] [PATCHv2] core: do not spawn jobs or touch other units during coldplugging

2015-02-27 Thread Ivan Shapovalov
On 2015-02-27 at 22:25 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Wed, Feb 25, 2015 at 09:40:23PM +0300, Ivan Shapovalov wrote: Because the order of coldplugging is not defined, we can reference a not-yet-coldplugged unit and read its state while it has not yet been set to a meaningful

[systemd-devel] [PATCH] user-sessions: move into own subdir and build independently of logind

2015-02-25 Thread Ivan Shapovalov
Suggested by Zbyszek on IRC. --- This does not conditionalize on HAVE_PAM. Don't know whether that's right. Makefile.am | 37 man/systemd-user-sessions.service.xml | 2 +- src/login/Makefile| 1 - src/login/user-sessions.c

[systemd-devel] [PATCH 1/3] sysusers: allow separate alternate roots for configs and modifications

2015-02-25 Thread Ivan Shapovalov
This is useful, for example, to create system accounts on an initramfs using the host's configuration. --- src/sysusers/sysusers.c | 97 + 1 file changed, 66 insertions(+), 31 deletions(-) diff --git a/src/sysusers/sysusers.c

[systemd-devel] [PATCH 0/3] using firstboot and sysusers to construct an initramfs

2015-02-25 Thread Ivan Shapovalov
=rescue.target as a complete alternative to pre-systemd arch-specific break=premount kernel parameter. Ivan Shapovalov (3): sysusers: allow separate alternate roots for configs and modifications firstboot: set all spwd fields to -1 for consistency with sysusers sysusers: do not reject users

[systemd-devel] [PATCH 2/3] firstboot: set all spwd fields to -1 for consistency with sysusers

2015-02-25 Thread Ivan Shapovalov
--- src/firstboot/firstboot.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/firstboot/firstboot.c b/src/firstboot/firstboot.c index a765d6d..a37ca17 100644 --- a/src/firstboot/firstboot.c +++ b/src/firstboot/firstboot.c @@ -525,9 +525,9 @@ static int

[systemd-devel] [PATCH 3/3] sysusers: do not reject users with already present /etc/shadow entries

2015-02-25 Thread Ivan Shapovalov
This is needed to interoperate firstboot and sysusers. The former one is started first, and it writes only /etc/shadow when it is told to set the root password. It's better to relax checks here than to duplicate functionality in firstboot. --- src/sysusers/sysusers.c | 23 +--

Re: [systemd-devel] Combining systemd-firstboot and systemd-sysusers

2015-02-21 Thread Ivan Shapovalov
On 2015-02-14 at 08:07 +0300, Ivan Shapovalov wrote: Hi all, I'm trying to adapt systemd-{sysusers,firstboot} for creating the system users in an initramfs (at generation time). (Note: I use systemd-firstboot to set the root password.) The situation - So, I'm running

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-02-21 Thread Ivan Shapovalov
On 2015-02-05 at 02:03 +0300, Ivan Shapovalov wrote: On 2015-01-28 at 22:29 +0300, Ivan Shapovalov wrote: On 2015-01-28 at 20:15 +0100, Lennart Poettering wrote: On Sun, 18.01.15 04:21, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi folks, I'm trying to fix this bug

Re: [systemd-devel] user units and system units behavior

2015-02-17 Thread Ivan Shapovalov
On 2015-02-16 at 11:14 +, Simon McVittie wrote: On 14/02/15 18:26, Ivan Shapovalov wrote: Yes, the per-session bus is there, but it is not used at all for communication with per-user systemd instance. I do want this to work, and I'm working on making it happen. It works on my Debian

Re: [systemd-devel] user units and system units behavior

2015-02-14 Thread Ivan Shapovalov
: cf. src/libsystemd/sd-bus/bus-util.c:bus_open_transport_systemd() and src/libsystemd/sd-bus/bus-util.c:bus_open_user_systemd(). That's why you can't see the traffic using `busctl --user`. And this all is going to change when kdbus becomes finally there. HTHs, -- Ivan Shapovalov / intelfx

[systemd-devel] Combining systemd-firstboot and systemd-sysusers

2015-02-13 Thread Ivan Shapovalov
) - make systemd-sysusers correctly handle entries which are only present in /etc/shadow (how? by preserving the shadow entry? by overwriting it, preserving the password? how else?) The question Which one to implement? Thanks for consideration, -- Ivan Shapovalov / intelfx

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-02-13 Thread Ivan Shapovalov
On 2015-01-28 at 20:15 +0100, Lennart Poettering wrote: On Sun, 18.01.15 04:21, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi folks, I'm trying to fix this bug: https://bugs.freedesktop.org/show_bug.cgi?id=88401 The initial problem (as reported) looks following: performing

[systemd-devel] [PATCHv2 3/4] systemctl: cat, edit: further polish error messages

2015-02-04 Thread Ivan Shapovalov
--- src/systemctl/systemctl.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 567b467..384ae02 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -4578,7 +4578,7 @@ static

[systemd-devel] [PATCHv2 4/4] systemctl: unit_find_paths(): unify error handling in two code pathes

2015-02-04 Thread Ivan Shapovalov
--- src/systemctl/systemctl.c | 63 ++- 1 file changed, 35 insertions(+), 28 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 384ae02..2d70ff1 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c

[systemd-devel] [PATCHv2 2/4] systemctl: cat: fix error handling

2015-02-04 Thread Ivan Shapovalov
- correctly check for local vs. remote transport - return after receiving error from expand_names() --- src/systemctl/systemctl.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 083b618..567b467 100644 ---

[systemd-devel] [PATCHv2 1/4] systemctl: cat, edit: improve unit load error reporting

2015-02-04 Thread Ivan Shapovalov
- report actual load error for units which could not be loaded - make unit_find_paths() report all kinds of errors it encounters (for consistency) - consistently handle not-found errors in cat() and edit() --- src/systemctl/systemctl.c | 54 +++ 1

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-02-04 Thread Ivan Shapovalov
On 2015-01-28 at 22:29 +0300, Ivan Shapovalov wrote: On 2015-01-28 at 20:15 +0100, Lennart Poettering wrote: On Sun, 18.01.15 04:21, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi folks, I'm trying to fix this bug: https://bugs.freedesktop.org/show_bug.cgi?id=88401

Re: [systemd-devel] [PATCH 2/4] systemctl: edit: improve error messages, report actual error for units which could not be loaded

2015-02-03 Thread Ivan Shapovalov
On 2015-02-03 at 23:05 +0100, Lennart Poettering wrote: On Fri, 19.12.14 17:08, Ivan Shapovalov (intelfx...@gmail.com) wrote: What happened to this patch series actually? I think only 1/4 was ever commited, what about the other ones? Ivan, any chance you can rebase the rest with Zbigniew's

Re: [systemd-devel] [PATCH 3/4] systemctl: edit: further reword error messages

2015-02-03 Thread Ivan Shapovalov
On 2015-01-05 at 17:08 +0100, Zbigniew Jędrzejewski-Szmek wrote: On Fri, Dec 19, 2014 at 05:08:09PM +0300, Ivan Shapovalov wrote: --- src/systemctl/systemctl.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/systemctl/systemctl.c b/src

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-01-28 Thread Ivan Shapovalov
On 2015-01-28 at 20:15 +0100, Lennart Poettering wrote: On Sun, 18.01.15 04:21, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi folks, I'm trying to fix this bug: https://bugs.freedesktop.org/show_bug.cgi?id=88401 The initial problem (as reported) looks following: performing

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-01-23 Thread Ivan Shapovalov
On 2015-01-18 at 04:21 +0300, Ivan Shapovalov wrote: Hi folks, I'm trying to fix this bug: https://bugs.freedesktop.org/show_bug.cgi?id=88401 The initial problem (as reported) looks following: performing a reload (maybe implicitly) re-starts alsa-restore.service if it is enabled

Re: [systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-01-17 Thread Ivan Shapovalov
On 2015-01-18 at 04:21 +0300, Ivan Shapovalov wrote: [...] Questions: - is my analysis correct? - if yes, then how to fix this? Maybe add a similar if (UNIT(p)-manager-n_reloading = 0) check to path_enter_running() to avoid calling manager_add_job() during reloading? Here's what I

[systemd-devel] bug 88401 / daemon-reload causes Type=oneshot services to be re-started / path_coldplug() is non-deserialization-aware

2015-01-17 Thread Ivan Shapovalov
analysis correct? - if yes, then how to fix this? Maybe add a similar if (UNIT(p)-manager-n_reloading = 0) check to path_enter_running() to avoid calling manager_add_job() during reloading? Thanks, -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part

Re: [systemd-devel] Typo fix in libabc (you missed my prev. letter?)

2015-01-08 Thread Ivan Shapovalov
: https://git.kernel.org/cgit/linux/kernel/git/kay/libabc.git/tree/README#n26 However, Askar, what you have provided isn't a patch, it is just two lines of text. You'd better send a correct git-formatted patch. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed

Re: [systemd-devel] Delegate=yes and user instance's resource controlling

2015-01-06 Thread Ivan Shapovalov
On Monday 05 January 2015 at 18:32:35, Lennart Poettering wrote: On Thu, 25.12.14 23:38, Ivan Shapovalov (intelfx...@gmail.com) wrote: Hi! Judging from commit a931ad47a8623163a29d898224d8a8c1177ffdaf, the systemd user instance is intentionally restricted from touching

Re: [systemd-devel] [PATCH 1/4] path-lookup, systemctl: export lookup_paths_init_from_scope() from shared/install.c and use it

2015-01-02 Thread Ivan Shapovalov
On Friday 26 December 2014 at 00:27:16, Ivan Shapovalov wrote: On Friday 19 December 2014 at 17:08:07, Ivan Shapovalov wrote: --- src/shared/install.c | 16 src/shared/path-lookup.c | 16 src/shared/path-lookup.h | 4 src

Re: [systemd-devel] PrivateDevices=true blocks use of ttys?

2014-12-26 Thread Ivan Shapovalov
with only a few files inside, and ttys aren't among these files. At least, that's how I understand it. Maybe you can do an mknod from ExecStartPre=, if you know the major:minor (4:0 for /dev/tty0) beforehand? -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed

[systemd-devel] Delegate=yes and user instance's resource controlling

2014-12-25 Thread Ivan Shapovalov
? If not, are there any alternative ways to achieve resource controlling per-user-service? Thanks in advance. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel

Re: [systemd-devel] [PATCH 1/4] path-lookup, systemctl: export lookup_paths_init_from_scope() from shared/install.c and use it

2014-12-25 Thread Ivan Shapovalov
On Friday 19 December 2014 at 17:08:07, Ivan Shapovalov wrote: --- src/shared/install.c | 16 src/shared/path-lookup.c | 16 src/shared/path-lookup.h | 4 src/systemctl/systemctl.c | 6 +- 4 files changed, 21 insertions(+), 21 deletions

Re: [systemd-devel] logind, su - sessions and initscripts compatibility

2014-12-21 Thread Ivan Shapovalov
in logind. (Please correct me if my analysis is wrong.) -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman

Re: [systemd-devel] Improving module loading

2014-12-21 Thread Ivan Shapovalov
-module@foo.service. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] [PATCH 1/4] path-lookup, systemctl: export lookup_paths_init_from_scope() from shared/install.c and use it

2014-12-19 Thread Ivan Shapovalov
--- src/shared/install.c | 16 src/shared/path-lookup.c | 16 src/shared/path-lookup.h | 4 src/systemctl/systemctl.c | 6 +- 4 files changed, 21 insertions(+), 21 deletions(-) diff --git a/src/shared/install.c b/src/shared/install.c index

[systemd-devel] [PATCH 4/4] systemctl: unit_find_paths(): unify error handling in two code pathes

2014-12-19 Thread Ivan Shapovalov
--- src/systemctl/systemctl.c | 63 ++- 1 file changed, 35 insertions(+), 28 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 20c367c..2a4e2a2 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c

[systemd-devel] [PATCH 2/4] systemctl: edit: improve error messages, report actual error for units which could not be loaded

2014-12-19 Thread Ivan Shapovalov
Not found condition in find_paths_to_edit() has been made non-fatal because the error message in edit() (Cannot find any units to edit) suggests that. Error messages in edit() themselves have been removed because - result of expand_names() is not checked anywhere else - an extra cannot find any

[systemd-devel] [PATCH 3/4] systemctl: edit: further reword error messages

2014-12-19 Thread Ivan Shapovalov
--- src/systemctl/systemctl.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 658793e..20c367c 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -4776,7 +4776,7 @@

Re: [systemd-devel] Using `systemctl edit` on invalid unit names

2014-12-17 Thread Ivan Shapovalov
On Tuesday, December 16, 2014 at 06:35:09 AM, Zbigniew Jędrzejewski-Szmek wrote: On Sun, Dec 14, 2014 at 04:21:32PM +0300, Ivan Shapovalov wrote: On Saturday 13 December 2014 at 15:34:01, Ronny Chevalier wrote: 2014-12-13 11:33 GMT+01:00 Ivan Shapovalov intelfx...@gmail.com

[systemd-devel] Using `systemctl edit` on invalid unit names

2014-12-13 Thread Ivan Shapovalov
a separate question: is this failure valid or is it a bug?) But well. I guess that `edit` operation should always work with unit files directly, just like enable/disable commands do. Is this all correct? Can anyone please comment on these two issues? -- Ivan Shapovalov / intelfx / signature.asc

Re: [systemd-devel] multi-user.target only wants network.service if I'm doing 'systemct show runlevel3.target'

2014-12-09 Thread Ivan Shapovalov
) before anything forces systemd to load that unit. After you issue `systemctl show runlevel3.target`, systemd loads that unit, loads its dependencies and merges them to multi-user.target (because runlevel3.target is an alias for multi-user.target). HTHs, -- Ivan Shapovalov / intelfx / Also

Re: [systemd-devel] a way to limit restarts?

2014-12-09 Thread Ivan Shapovalov
enters failed state, i. e. at the same time it is restarted. There is apparently no way to start a special unit when the start limit is reached. However, I may have missed it as well... -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part

[systemd-devel] Option fail mentioned in systemd.mount(5)

2014-12-04 Thread Ivan Shapovalov
, or what? -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH] rules: rerun vconsole-setup when switching from vgacon to fbcon

2014-12-01 Thread Ivan Shapovalov
On Friday 07 November 2014 at 16:45:02, Lennart Poettering wrote: On Fri, 07.11.14 17:45, Ivan Shapovalov (intelfx...@gmail.com) wrote: On Thursday 06 November 2014 at 11:02:44, David Herrmann wrote: Hi Ray On Thu, Nov 6, 2014 at 10:40 AM, David Herrmann dh.herrm

[systemd-devel] [PATCH] unit-name: fix escaping logic in unit_name_mangle_with_suffix().

2014-11-25 Thread Ivan Shapovalov
Make screened character set consistent with unit_name_mangle() by splitting off the escaping loop into a separate function. Before this fix, unit names such as `foo@bar.target` would get transformed into `foo\x40bar.target` when unit_name_mangle_with_suffix() is used.

Re: [systemd-devel] remote-fs ordering, iSCSI and _netdev

2014-11-09 Thread Ivan Shapovalov
for on shutdown so network could be teared down before filesystem is unmounted. Isn't this (unmount before teardown) achieved by After=network.target? That target is passive, so it is pulled in by a provider, and all should work even in case of manually mounted filesystems. Am I wrong somewhere? -- Ivan

Re: [systemd-devel] [PATCH] rules: rerun vconsole-setup when switching from vgacon to fbcon

2014-11-07 Thread Ivan Shapovalov
, these additional systemd-vconsole-setup instances aren't supervised by systemd... -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http

Re: [systemd-devel] systemd and power management

2014-10-29 Thread Ivan Shapovalov
already done something along these lines for my own purposes, see https://github.com/intelfx/power-management However, I still want to know if I this is OK wrt systemd spirit. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part

[systemd-devel] [PATCH] man: fix typos in description of SELinuxContextFromNet=

2014-10-27 Thread Ivan Shapovalov
--- man/systemd.socket.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/man/systemd.socket.xml b/man/systemd.socket.xml index ce04b0b..57f769f 100644 --- a/man/systemd.socket.xml +++ b/man/systemd.socket.xml @@ -678,7 +678,7 @@ varlistentry

Re: [systemd-devel] [PATCH] Bootchart: allow parse LABEL, UUID, PARTUUID for svg info

2014-10-27 Thread Ivan Shapovalov
function canonicalize_file_name() in glibc which resolves symlinks and (hopefully) returns /dev/sdXY. I'm not the one to judge, but your code seems pretty messy, esp. with extensive use of magic constants and numbers... -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally

[systemd-devel] systemd in initramfs: /etc/passwd, /etc/group, emergency.service and sulogin

2014-10-24 Thread Ivan Shapovalov
/trunk/initcpio-install-systemd?h=packages/systemd#n141 [2]: https://projects.archlinux.org/svntogit/packages.git/tree/trunk/initcpio-install-systemd?h=packages/systemd#n147 [3]: http://cgit.freedesktop.org/systemd/systemd/tree/units/emergency.service.in#n21 Thanks, -- Ivan Shapovalov / intelfx

Re: [systemd-devel] rpcbind static, want enabled

2014-10-24 Thread Ivan Shapovalov
after rpcbind.service. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] How soon after login can I rely on systemd --user having reached sockets.target?

2014-10-23 Thread Ivan Shapovalov
there. it shouldn't try to do access control for individual sessions but for users on specific sessions. Wasn't this already fixed in polkit.git recently? Oh, if this indeed was, then... does it mean that the last significant blocker for widely using `systemd --user` has gone away? -- Ivan

Re: [systemd-devel] systemd-cron: retrigger generator after /var is mounted

2014-10-22 Thread Ivan Shapovalov
is the preferred approach. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] Timers without matching units?

2014-10-22 Thread Ivan Shapovalov
), I don't need the timer unit to start something else. Does this sound like a valid reason to make timers' Unit= optional? -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list

Re: [systemd-devel] Unicode support in console after boot

2014-10-21 Thread Ivan Shapovalov
switching the framebuffer driver. And the latter condition is not mandated by anything, it's just a coincidence... -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd

Re: [systemd-devel] Unicode support in console after boot

2014-10-21 Thread Ivan Shapovalov
On Tuesday 21 October 2014 at 21:57:09, Michal Sekletar wrote: On Tue, Oct 21, 2014 at 09:39:46PM +0400, Ivan Shapovalov wrote: On Tuesday 21 October 2014 at 19:03:17, Michal Sekletar wrote: On Tue, Oct 14, 2014 at 09:04:56AM +0200, Jan Synacek wrote: Michal Sekletar msekl

Re: [systemd-devel] [PATCH v2] udev hwdb: Support shipping pre-compiled database in system images

2014-10-19 Thread Ivan Shapovalov
On Sunday 19 October 2014 at 05:42:28, Ivan Shapovalov wrote: On Friday 17 October 2014 at 15:44:51, Martin Pitt wrote: Hello again, the previous patch had a typo in the manpage (it said /lib/udev instead of /usr/lib/udev at one place), and also forgot to adjust systemd-udev

Re: [systemd-devel] [PATCH v2] udev hwdb: Support shipping pre-compiled database in system images

2014-10-18 Thread Ivan Shapovalov
it possible with these two lines? ConditionPathExists=|!@udevlibexecdir@/hwdb.bin ConditionDirectoryNotEmpty=|/etc/udev/hwdb.d/ This will succeed if EITHER @udevlibexecdir@/hwdb.bin does not exist OR /etc/udev/hwdb.d/ is not empty. Or have I misunderstood you? -- Ivan Shapovalov / intelfx

Re: [systemd-devel] Unicode support in console after boot

2014-10-13 Thread Ivan Shapovalov
). It is also possible that settings get reset after the framebuffer driver is changed (e. g. i915 module is loaded)... BTW, I'm still seeking for a correct method to handle this. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part

Re: [systemd-devel] [PATCH] journalctl: allow customizable output formats

2014-09-22 Thread Ivan Shapovalov
found it useful many times. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Re: [systemd-devel] [PATCH] build-sys: make hibernation support configure option also handle hybrid-sleep; fix indentation

2014-09-11 Thread Ivan Shapovalov
On Tuesday 09 September 2014 at 01:40:51, Ivan Shapovalov wrote: --- The patch by Umut did miss at least hybrid-sleep -- it involves hibernation as well (hybrid sleep is a hibernation followed by S3 rather than S4 powerdown). Also, it messed up indentation a bit (Makefile.am seems

[systemd-devel] [PATCH] build-sys: make hibernation support configure option also handle hybrid-sleep; fix indentation

2014-09-08 Thread Ivan Shapovalov
--- The patch by Umut did miss at least hybrid-sleep -- it involves hibernation as well (hybrid sleep is a hibernation followed by S3 rather than S4 powerdown). Also, it messed up indentation a bit (Makefile.am seems to use tabs), which I fixed as well. And I wonder, maybe it makes sense to

Re: [systemd-devel] [PATCH] build: configure option to disable hibernation

2014-09-07 Thread Ivan Shapovalov
). However, I think that the name you've picked for the switch (and commit msg) is too broad. We're configuring support for hibernation resume, not hibernation itself. -- Ivan Shapovalov / intelfx / signature.asc Description: This is a digitally signed message part

Re: [systemd-devel] [PATCHv6 0/3] hibernate-resume: implement support for resuming from hibernation

2014-08-28 Thread Ivan Shapovalov
On Thursday 28 August 2014 at 06:25:51, Jan Janssen wrote: Ivan Shapovalov intelfx100 at gmail.com writes: On Wednesday 27 August 2014 at 03:16:10, Zbigniew Jędrzejewski-Szmek wrote: On Tue, Aug 26, 2014 at 10:21:59PM +0200, Lennart Poettering wrote: On Wed, 27.08.14 00:17

Re: [systemd-devel] [PATCHv6 2/3] hibernate-resume: add a tool to write a device node's major:minor to /sys/power/resume.

2014-08-27 Thread Ivan Shapovalov
On Wednesday 27 August 2014 at 08:18:38, Thomas Bächler wrote: Am 26.08.2014 um 22:17 schrieb Ivan Shapovalov: This can be used to initiate a resume from hibernation by path to a swap device containing the hibernation image. The respective templated unit is also added

Re: [systemd-devel] [PATCHv6 0/3] hibernate-resume: implement support for resuming from hibernation

2014-08-27 Thread Ivan Shapovalov
On Wednesday 27 August 2014 at 03:16:10, Zbigniew Jędrzejewski-Szmek wrote: On Tue, Aug 26, 2014 at 10:21:59PM +0200, Lennart Poettering wrote: On Wed, 27.08.14 00:17, Ivan Shapovalov (intelfx...@gmail.com) wrote: This patchset allows systemd to parse resume= kernel command line

[systemd-devel] [PATCH 1/2] units: add ConditionInitrd=

2014-08-27 Thread Ivan Shapovalov
--- man/systemd.unit.xml | 13 + src/core/condition.c | 17 + src/core/load-fragment-gperf.gperf.m4 | 1 + src/shared/condition-util.c | 1 + src/shared/condition-util.h | 1 + 5 files changed, 33 insertions(+)

[systemd-devel] [PATCH 0/2] units: add and use ConditionInitrd= instead of checking for /etc/initrd-release.

2014-08-27 Thread Ivan Shapovalov
This is as proposed by Thomas in review of my hibernate-resume patchset. The objective benefit of this change is that in_initrd() function is used for checking, which not only checks for /etc/initrd-release, but also verifies that the rootfs is on a virtual device. Ivan Shapovalov (2): units

[systemd-devel] [PATCH 2/2] units: use ConditionInitrd=true instead of ConditionPathExists=/etc/initrd-release

2014-08-27 Thread Ivan Shapovalov
--- units/initrd-cleanup.service.in| 2 +- units/initrd-fs.target | 2 +- units/initrd-parse-etc.service.in | 2 +- units/initrd-root-fs.target| 2 +- units/initrd-switch-root.service.in| 2 +- units/initrd-switch-root.target

[systemd-devel] [PATCH] units: remove unnecessary ordering dependencies in systemd-hibernate-resume@.service

2014-08-27 Thread Ivan Shapovalov
They were left from an early review iteration, when hibernate-resume functionality was intended to work also outside of initramfs. Now this is not the case, and these dependencies became redundant as systemd-fsck-root.service can never be part of initramfs, and systemd-remount-fs.service makes

Re: [systemd-devel] [PATCH 0/2] units: add and use ConditionInitrd= instead of checking for /etc/initrd-release.

2014-08-27 Thread Ivan Shapovalov
On Wednesday 27 August 2014 at 20:19:45, Lennart Poettering wrote: On Wed, 27.08.14 20:26, Ivan Shapovalov (intelfx...@gmail.com) wrote: This is as proposed by Thomas in review of my hibernate-resume patchset. The objective benefit of this change is that in_initrd() function

  1   2   >