Nir Soffer has posted comments on this change.

Change subject: sslutils: introduce ssl_enabled
......................................................................


Patch Set 2:

Please check the jenkins failures:
15:08:39 ./vdsm/gluster/storagedev.py:221: undefined name 'rc'
15:08:39 ./vdsm/gluster/storagedev.py:221: undefined name 'out'
15:08:39 ./vdsm/gluster/storagedev.py:221: undefined name 'err'

    I think you need to rebase, this issue was solved on master few days ago

15:08:39 ./lib/vdsm/sslutils.py:340: undefined name 'config'

    Forgot an import?

-- 
To view, visit https://gerrit.ovirt.org/41795
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iec301a55c02943950d5f56702ec1dd58ed954340
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Dima Kuznetsov <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to