From: Thomas Deutschmann <whissi@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] [PATCH] acct-user.eclass: don't modify existing user by default
Date: Mon, 4 Jan 2021 15:05:46 +0100 [thread overview]
Message-ID: <4338b8fd-d09b-be58-5b0f-c768db2b6cdb@gentoo.org> (raw)
In-Reply-To: <809f727af51f7dcd9aec97a800c0ecd89f60eaa6.camel@gentoo.org>
On 2021-01-04 10:23, Michał Górny wrote:
> Not modifying an existing user is a horrible default that has already
> bricked one system (by removing /dev/null). So, over my dead commit
> access.
Have you seen how many user were hit caused by the recent rebuilt on
2020-12-28 and are already complaining/asking for help through various
channels?
It's like asking for service auto-restart support in PMS as requested as
part of current OpenSSH upgrade because if you move from <8.3_p1 to
>=8.3_p1 and don't restart OpenSSH in time, you can get locked out.
However, an easily looking solution like
> Just add something like
>
> if [[ -d /run/systemd/system ]]; then
> systemctl try-restart sshd
> else
> rc-service -q --ifstarted sshd restart
> fi
>
> to pkg_postinst
is wrong because even if it works for *some* users it won't work for all
users but has the potential to cause major problems.
That's why we have elog and newitem system. However, 8.3 is in
repository for while and multiple people forgot about the newitem and
didn't pay attention to elog messages. While I agree that it's a problem
when you lose access to a remote box you don't have physical access to,
this reached a level where I have to say,
> We cannot rescue/protect everyone.
Back to topic, acct-* stuff:
Like already said in February 2020 when I joined a thread created by a
user posting same concerns:
There is a reason why *no* distribution on this planet is trying to mess
with existing data/configurations: Every attempt trying to analyze given
setup to apply required changes to fix/migrate something automatically
has been prone to fail the long run.
Please get some experience from real world. Preferable from running
headless systems not just for yourself and where you are not the only
person touching the system.
When I worked on bug 605008 long time ago for example, I also ended up
over-engineering. There is stuff you cannot fix. I am still thinking
about creating everything the way it should look like in $D and report
any difference like changed file permissions to user on merge to allow
them to notice (an improvement, now user only have to pay attention and
you need to solve the additional problem that the more information you
present all the time, the more information will be ignored). But
sometimes users are making changes we wouldn't do, not recommend or just
don't understand at first. That all doesn't matter: We have to keep in
mind that these aren't our systems and we have to respect whatever the
user did on their system.
--
Regards,
Thomas Deutschmann / Gentoo Linux Developer
fpr: C4DD 695F A713 8F24 2AA1 5638 5849 7EE5 1D5D 74A5
next prev parent reply other threads:[~2021-01-04 14:05 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-04 1:35 [gentoo-dev] [PATCH] acct-user.eclass: don't modify existing user by default Thomas Deutschmann
2021-01-04 2:41 ` Mike Gilbert
2021-01-04 3:17 ` Alec Warner
2021-01-04 3:18 ` Michael Orlitzky
2021-01-04 14:46 ` Thomas Deutschmann
2021-01-04 15:24 ` Michael Orlitzky
2021-01-04 15:55 ` David Seifert
2021-01-04 16:18 ` Thomas Deutschmann
2021-01-04 16:28 ` Michał Górny
2021-01-04 16:30 ` Thomas Deutschmann
2021-01-04 16:34 ` Thomas Deutschmann
2021-01-04 16:38 ` Michał Górny
2021-01-04 16:50 ` Thomas Deutschmann
2021-01-04 16:56 ` Michał Górny
2021-01-04 16:56 ` Mike Gilbert
2021-01-04 16:54 ` Mike Gilbert
2021-01-04 7:32 ` Robin H. Johnson
2021-01-04 16:45 ` [gentoo-dev] " James Cloos
2021-01-04 18:07 ` Michael Orlitzky
2021-01-04 18:20 ` Michał Górny
2021-01-04 18:38 ` Michael Orlitzky
2021-01-04 18:23 ` Thomas Deutschmann
2021-01-04 18:27 ` Michael Orlitzky
2021-01-04 18:32 ` Thomas Deutschmann
2021-01-04 9:23 ` [gentoo-dev] " Michał Górny
2021-01-04 14:05 ` Thomas Deutschmann [this message]
2021-01-04 16:10 ` Mike Gilbert
2021-01-04 16:14 ` Michał Górny
2021-01-04 16:20 ` Thomas Deutschmann
2021-01-08 18:11 ` Fabian Groffen
2021-01-08 18:14 ` Michał Górny
2021-01-08 18:23 ` Thomas Deutschmann
2021-01-08 18:32 ` Michał Górny
2021-01-08 15:48 ` Thomas Deutschmann
2021-01-08 16:03 ` Mike Gilbert
2021-01-08 16:29 ` Thomas Deutschmann
2021-01-08 16:50 ` Mike Gilbert
2021-01-08 17:06 ` Mike Gilbert
2021-01-08 18:10 ` Thomas Deutschmann
2021-01-08 18:31 ` Michał Górny
2021-01-08 19:15 ` Mike Gilbert
2021-01-08 17:16 ` Michał Górny
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=4338b8fd-d09b-be58-5b0f-c768db2b6cdb@gentoo.org \
--to=whissi@gentoo.org \
--cc=gentoo-dev@lists.gentoo.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