Francesco Romani has posted comments on this change. Change subject: virt: bind console to unix domain socket ......................................................................
Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/40704/3/vdsm/virt/vmdevices/core.py File vdsm/virt/vmdevices/core.py: Line 94: try: Line 95: supervdsm.getProxy().prepareVmChannel(self._path) Line 96: except Exception: Line 97: self.log.exception( Line 98: "Failed to prepare the VM console channel") do not swallow the exception here, even though we are in _domDependentInit path Line 99: Line 100: def cleanup(self): Line 101: if os.path.islink(self._path): Line 102: utils.rmFile(os.path.realpath(self._path)) -- To view, visit https://gerrit.ovirt.org/40704 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0a876cd7f3b848aa9dde8123d459dc292729b303 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Vitor de Lima <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
