Improvement of the systemd unit files test

Make the systemd unit files test more robus, to avoid false positives in
case there are non-unit files in the sytemd directory.
This commit is contained in:
Jan Kurik 2021-11-10 16:11:00 +01:00
parent 9daf757d8e
commit b263d4070b

View File

@ -31,6 +31,7 @@
PACKAGE="pcp" PACKAGE="pcp"
PKGS="pcp pcp-zeroconf" PKGS="pcp pcp-zeroconf"
UNITS="" UNITS=""
UNITFILESDIR="/usr/lib/systemd/system/"
rlJournalStart rlJournalStart
rlPhaseStartSetup rlPhaseStartSetup
@ -42,13 +43,14 @@ rlJournalStart
rlAssertRpm "${P}" rlAssertRpm "${P}"
done done
# Generate list of unit files # Generate list of unit files
UNITS="$(rpm -ql ${PKGS} | grep /usr/lib/systemd/system/)" UNITS=$(rpm -ql ${PKGS} | grep "${UNITFILESDIR}")
rlLog "Available unit files: ${UNITS}" rlLog "Available unit files: ${UNITS}"
rlPhaseEnd rlPhaseEnd
rlPhaseStartTest rlPhaseStartTest
FAILEDU="" FAILEDU=""
for U in ${UNITS}; do for U in ${UNITS}; do
if [[ -f ${U} ]] && [[ "$(dirname ${U})" == "${UNITFILESDIR%/}" ]]; then
if ! rlRun -s "systemd-analyze --no-pager verify ${U}"; then if ! rlRun -s "systemd-analyze --no-pager verify ${U}"; then
rlFail "Error in ${U}: $(cat ${rlRun_LOG})" rlFail "Error in ${U}: $(cat ${rlRun_LOG})"
FAILEDU="${FAILEDU} $(basename ${U})" FAILEDU="${FAILEDU} $(basename ${U})"
@ -57,6 +59,9 @@ rlJournalStart
rlFail "Error in ${U}: $(cat ${rlRun_LOG})" rlFail "Error in ${U}: $(cat ${rlRun_LOG})"
FAILEDU="${FAILEDU} $(basename ${U})" FAILEDU="${FAILEDU} $(basename ${U})"
fi fi
else
rlLog "${U} is not a regular unit file ... skipping"
fi
done done
[[ -n "${FAILEDU}" ]] && rlLog "List of failed units: ${FAILEDU}" [[ -n "${FAILEDU}" ]] && rlLog "List of failed units: ${FAILEDU}"
rlPhaseEnd rlPhaseEnd