ShaoHe Feng has posted comments on this change.
Change subject: Checking accessibility to vdsm log file
......................................................................
Patch Set 6: (1 inline comment)
....................................................
File vdsm/vdsmd.init.in
Line 88:
Line 89: accessing_log_file() {
Line 90: su vdsm > /dev/null 2>&1 << EOF
Line 91: echo -n >> @VDSMLOGDIR@/vdsm.log
Line 92: EOF
I create a user "vdsmtest1" and make the sell as "/sbin/nologin" same with
"vdsm" user.
and I execute the follow command:
$ su vdsmtest1 -c ls
it report error as follow.
This account is currently not available.
Then I create a user "vdsmtest0" and make the sell as "/bin/bash".
$ su vdsmtest0 -c ls
it is successful.
Line 93: }
Line 94:
Line 95: get_libvirt_conf_item() {
Line 96: local cfile key
--
To view, visit http://gerrit.ovirt.org/9118
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8bd379803b01627d6897992ee8798c6a22195b0f
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: ShaoHe Feng <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
