Dan Kenigsberg has posted comments on this change.

Change subject: certificates: remove config.read call
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/62885/2//COMMIT_MSG
Commit Message:

Line 6: 
Line 7: certificates: remove config.read call
Line 8: 
Line 9: certificates initiated config read from 'vdsm.conf' file, while config
Line 10: should be loaded from multiple configuration files starting from 
sorry, but I fail to parse the English of this commit message. Why is it safe 
to remove the code?
Line 11: https://gerrit.ovirt.org/#/c/58728/
Line 12: config.read call overrides vdsm configuration and should be removed.
Line 13: 
Line 14: Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Irit Goihman <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to