Icinga / icinga-packaging

Packaging documentation and issues for the official Icinga repository
https://packages.icinga.com
44 stars 15 forks source link

Icinga DB - adduser: Warning: The home directory "/" is not owned by the user you are creating #181

Closed mcktr closed 4 years ago

mcktr commented 4 years ago

The warning message appears when I install IcingaDB on Debian 10 "buster". Below you'll find the full output (unfortunately only in German).

I guess setting the home directory to / isn't a good at all, isn't it?

root@icinga2:~# apt install icingadb
Paketlisten werden gelesen... Fertig
Abhängigkeitsbaum wird aufgebaut.       
Statusinformationen werden eingelesen.... Fertig
Die folgenden NEUEN Pakete werden installiert:
  icingadb
0 aktualisiert, 1 neu installiert, 0 zu entfernen und 0 nicht aktualisiert.
Es müssen 3.061 kB an Archiven heruntergeladen werden.
Nach dieser Operation werden 11,5 MB Plattenplatz zusätzlich benutzt.
Holen:1 https://packages.icinga.com/debian icinga-buster-testing/main amd64 icingadb amd64 1.0.0~rc1-1.buster [3.061 kB]
Es wurden 3.061 kB in 1 s geholt (4.138 kB/s).
Vormals nicht ausgewähltes Paket icingadb wird gewählt.
(Lese Datenbank ... 39238 Dateien und Verzeichnisse sind derzeit installiert.)
Vorbereitung zum Entpacken von .../icingadb_1.0.0~rc1-1.buster_amd64.deb ...
Entpacken von icingadb (1.0.0~rc1-1.buster) ...
icingadb (1.0.0~rc1-1.buster) wird eingerichtet ...
Adding system-user for icingadb
adduser: Warnung: Das Home-Verzeichnis »/« gehört nicht dem Benutzer, den Sie gerade anlegen.
Al2Klimov commented 4 years ago

It doesn't matter as this is a system user:

root@b4d3ac20dfea:/# su -ls /bin/bash bin
bin@b4d3ac20dfea:~$ echo $HOME
/bin
bin@b4d3ac20dfea:~$ ls -la /bin
total 5712
drwxr-xr-x 2 root root    4096 Feb 24 00:00 .
lippserd commented 4 years ago

@Al2Klimov Just for the records, why aren't fixing this? We should avoid warnings because it leads to uncertainties whether everything is right upon installing Icinga DB.

N-o-X commented 4 years ago

It's an unnecessary warning, so we should just fix it.

Al2Klimov commented 4 years ago
N-o-X commented 4 years ago

New revisions have been released (including those PRs)