Yair Zaslavsky has posted comments on this change.

Change subject: engine: Removing flood message from engine.log
......................................................................


Patch Set 1: Looks good to me, approved

(2 inline comments)

....................................................
Commit Message
Line 5: CommitDate: 2013-02-28 14:22:45 +0200
Line 6: 
Line 7: engine: Removing flood message from engine.log
Line 8: 
Line 9: The following message is writtent to engine.log when
s/writtent/written
Line 10: we are trying to add some audit log message and it is illegal - 
meaning should
Line 11: be skipped, the message is misleading and also for case of very common 
event
Line 12: we just flooding engine.log
Line 13: 


Line 6: 
Line 7: engine: Removing flood message from engine.log
Line 8: 
Line 9: The following message is writtent to engine.log when
Line 10: we are trying to add some audit log message and it is illegal - 
meaning should
please add "it" between meaning and should
Line 11: be skipped, the message is misleading and also for case of very common 
event
Line 12: we just flooding engine.log
Line 13: 
Line 14: Change-Id: I0ec84555c0473a2e051d68f27cbbbbbd84fa6c9a


--
To view, visit http://gerrit.ovirt.org/12521
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0ec84555c0473a2e051d68f27cbbbbbd84fa6c9a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to