Ondřej Svoboda 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 150: hooking.log('OVS: %s' % message)
Is it not apparent that messages come from OVS, from module names (which are 
prefixed by ovs_)? Just asking, I like this improvement to logs.


-- 
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