Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Use Id=... to check systemd services ...................................................................... packaging: Use Id=... to check systemd services The services managed by systemd can have an unique identifier and then several alternative names, or aliases. The NFS service, for example, has "nfs-server" as identifier but it also has "nfs" and "nfs-server" as aliases. These alternative names can be used for some operations, but not for all of them, in particular the "systemctl enable ..." operation requires the main name, it will not work with alternative names. This meas that we need to make sure that the method that checks the availability of services uses the main name, otherwise it can succeed but then later the call to "systemctl enable" can fail. Change-Id: I99492161d29254e48a0196b3b5cbdfa7c9dc1863 Signed-off-by: Juan Hernandez <juan.hernandez@redhat.com> --- M packaging/fedora/setup/common_utils.py 1 file changed, 14 insertions(+), 7 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12793 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99492161d29254e48a0196b3b5cbdfa7c9dc1863 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib@redhat.com>