Attention is currently required from: laforge, fixeria, msuraev.
osmith has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-hlr/+/29311
)
Change subject: Debian: add osmocom user during package install
......................................................................
Patch Set 7: Code-Review-1
(3 comments)
File debian/postinst:
https://gerrit.osmocom.org/c/osmo-hlr/+/29311/comment/3c4e6158_1d9a4cf6
PS7, Line 9: # install.
as you mentioned in the issue, this also need to be added to the rpm spec or else the
systemd file tries to run it with a non-existing user and is then broken
https://gerrit.osmocom.org/c/osmo-hlr/+/29311/comment/89ad7198_1bfb3878
PS7, Line 29: --no-create-home \
why --no-create-home, do we already create the directory structure?
https://gerrit.osmocom.org/c/osmo-hlr/+/29311/comment/54dda13d_e12481be
PS7, Line 37: chperms root osmocom 2775 /etc/osmocom
/var/lib/osmocom/hlr.db* also need to be changed from root -> osmocom.
hlr.db is the sqlite database, and related files hlr.db-shm and hlr.db-wal may exist e.g.
if osmo-hlr is currently running or if it crashed.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hlr/+/29311
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I625c993ab03dfe32976c651acca9c35c33a768e7
Gerrit-Change-Number: 29311
Gerrit-PatchSet: 7
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: msuraev <msuraev(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 19 Sep 2022 09:29:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment