Alon Bar-Lev has posted comments on this change. Change subject: services, setup: Serial console proxy [WIP] ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/35906/2/packaging/setup/ovirt_engine_setup/console_proxy/constants.py File packaging/setup/ovirt_engine_setup/console_proxy/constants.py: Line 73: OVIRT_ENGINE_PKI_CONSOLE_CERT = \ Line 74: os.path.join(ocpconfig.VMPROXY_DIR, '.pki/libvirt/clientcert.pem') Line 75: Line 76: OVIRT_ENGINE_PKI_CONSOLE_KEY = \ Line 77: os.path.join(ocpconfig.VMPROXY_DIR, '.pki/libvirt/clientkey.pem') I am unsure why this goes into /var/lib while all other pki artifacts are at /etc/pki. if you are not going to chroot then /etc/pki/ovirt-engine-<something> should be used. Line 78: Line 79: OVIRT_ENGINE_CONSOLE_PROXY_HOST_KEY = \ Line 80: ocpconfig.ENGINE_CONSOLE_PROXY_SSH_HOST_KEY_PATH Line 81: -- To view, visit http://gerrit.ovirt.org/35906 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I034ef8e6d10da5dc93eda61e0c5c518ca13a5a28 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
