Dan Kenigsberg has posted comments on this change.
Change subject: fix can't get ssl opt in vdsm sos plugin. ......................................................................
Patch Set 1: I would prefer that you didn't submit this
(2 inline comments)
.................................................... Commit Message Line 9: fix can't get ssl opt in vdsm sos plugin. why is the first line typed twice?
.................................................... File vdsm/sos/vdsm.py.in Line 79: config.read('/etc/vdsm/vdsm.conf') please verify, but I think that the right way to read the default is by
from vdsm import config
-- To view, visit http://gerrit.ovirt.org/4349 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Icbbc9a10202f191d0754bfae5c3f876b6c53a560 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Xu He Jie xuhj@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com