Ido Barkan has posted comments on this change.

Change subject: hooks: ovs: start every log with 'OVS: '
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/46906/4/vdsm_hooks/ovs/ovs_utils.py
File vdsm_hooks/ovs/ovs_utils.py:

Line 146:             {'connectivityCheck': False, '_inRollback': True})
Line 147: 
Line 148: 
Line 149: def log(message):
Line 150:     hooking.log('OVS: %s' % message)
> i did logging with module name in one of following patches. i just did not 
I think that you should use the logging api directly and just replace the 
formatter. or just create you own custom logger and use it. not a must though.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I477954f0cb18071bf80caa88e241cfabdde6bbe9
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Ido Barkan <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <[email protected]>
Gerrit-Reviewer: Petr Horáček <[email protected]>
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