From Dan Kenigsberg <dan...@redhat.com>: Dan Kenigsberg has posted comments on this change.
Change subject: janitorial: drop once-used P_VDSM_KEYS ...................................................................... Patch Set 2: Code-Review-1 Verified-1 (1 comment) https://gerrit.ovirt.org/#/c/77468/2/lib/vdsm/constants.py.in File lib/vdsm/constants.py.in: Line 150: EXT_KVM_2_OVIRT = '@LIBEXECDIR@/kvm2ovirt' # NOQA: E501 (potentially long line) Line 151: EXT_SYSTEMD_RUN = '@SYSTEMD_RUN_PATH@' Line 152: Line 153: # location of the certificates Line 154: PKI_DIR = os.path.join(SYSCONF_PATH, 'pki', 'vdsm') nope, PKI_DIR does not have the /keys/ in the end. Line 155: KEY_FILE = os.path.join(PKI_DIR, 'keys', 'vdsmkey.pem') Line 156: CERT_FILE = os.path.join(PKI_DIR, 'certs', 'vdsmcert.pem') -- To view, visit https://gerrit.ovirt.org/77468 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I588a94534d2813bc9ffdd1688364cea0e7516170 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org