public inbox for util-linux@vger.kernel.org
 help / color / mirror / Atom feed
From: Ruediger Meier <sweet_f_a@gmx.de>
To: Karel Zak <kzak@redhat.com>
Cc: "Thomas Bächler" <thomas@archlinux.org>, util-linux@vger.kernel.org
Subject: Re: [PATCH] fstrim: add systemd units
Date: Wed, 9 Apr 2014 12:07:56 +0200	[thread overview]
Message-ID: <201404091207.56691.sweet_f_a@gmx.de> (raw)
In-Reply-To: <20140409075215.GP22126@x2.net.home>

On Wednesday 09 April 2014, Karel Zak wrote:
> On Tue, Apr 08, 2014 at 02:07:28PM +0200, Ruediger Meier wrote:
> > I don't say something against systemd here and I don't see this
> > black and white only. I'am just against adding sysadmin scripts
> > which actually do only trivial things.
>
> Well, it's not so important if the file is trivial or non-trivial --
> the overhead for distributions is usually almost the same. It's
> always cheaper to maintain distribution independent stuff on one
> shared place.

I understand, but isn't the usefulness of a daily job "fstrim -a" 
debatable at all? Should this be discussed by core util-linux 
developers or not better by sysadmins and distribution developers?
Actually I think daily "fstrim -a" IS not a good idea for most of my 
systems but I feel no need to discuss this here on util-linux because 
the decision whether, how, when and on which devices fstrim should be 
called should not be made here at all.

And I find my other examples in this thread not so stupid.
If we add scripts with one generic use case for fstrim. Why don't we add 
the generic boot and maintenance scripts how and when to mount or fsck 
all filesystems, to activate swap, to get and set hwclock or whatever?

Moreover the portability issue. Why adding scripts for systemd only 
allthough the same could be done without systemd in a more portable 
way.

cu,
Rudi

  reply	other threads:[~2014-04-09 10:07 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-03 21:41 [PATCH] fstrim: add systemd units Thomas Bächler
2014-04-07 10:43 ` Karel Zak
2014-04-08 10:25   ` Ruediger Meier
2014-04-08 10:57     ` Thomas Bächler
2014-04-08 12:07       ` Ruediger Meier
2014-04-08 15:42         ` Dave Reisner
2014-04-08 17:12           ` Ruediger Meier
2014-04-09  7:52         ` Karel Zak
2014-04-09 10:07           ` Ruediger Meier [this message]
2014-04-09 11:02             ` Karel Zak
2014-04-09 12:12               ` Ruediger Meier
2014-04-09 12:49                 ` Thomas Bächler
2014-04-09 15:16                   ` Ruediger Meier
2014-04-09 15:24                     ` Thomas Bächler
2014-04-09 15:44                       ` Ruediger Meier
2014-04-09 14:02               ` Markus Trippelsdorf
2014-04-09 15:48                 ` Ruediger Meier
2014-04-09 15:55                   ` Markus Trippelsdorf
2014-04-09 18:39                     ` Theodore Ts'o
2014-04-10  8:05                       ` Karel Zak
2014-04-10  9:17                         ` Ruediger Meier
2014-04-10 12:49                           ` Karel Zak
2014-04-10 13:16                             ` Ruediger Meier
2014-04-10 13:22                               ` Ruediger Meier
2014-04-08 17:26       ` Karel Zak
2014-04-08 17:30         ` Thomas Bächler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201404091207.56691.sweet_f_a@gmx.de \
    --to=sweet_f_a@gmx.de \
    --cc=kzak@redhat.com \
    --cc=thomas@archlinux.org \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox