Douglas Schilling Landgraf has posted comments on this change.

Change subject: vdsm-tool: Add register verb
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/40966/1/lib/vdsm/tool/register.py
File lib/vdsm/tool/register.py:

Line 193:         """
Line 194:         The logging settings
Line 195:         Saving log in: /var/log/vdsm/register.log
Line 196:         """
Line 197:         # Avoid node side effects of logging module not been working
> The issue is that log is not created at all when using  from ovirt.node.uti
Completing my answer, so vdsm utils import this module and it affects the 
register.py too. That's why I used this approach until I find this root cause.
Line 198:         logging.shutdown()
Line 199:         reload(logging)
Line 200: 
Line 201:         logging.basicConfig(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica800027beec1e5a20165bb5e1e78baf9283c1da
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to