Eduardo has posted comments on this change.

Change subject: sdc: Change error messages to info
......................................................................


Patch Set 1: Code-Review-2

Already discussed.

There is no spontaneous SD generation.

Should not be unfetched domains, then if such thing exists, they should be 
reported as ERROR.

An ERROR log is not a bug, is for help developers to understand degraded 
conditions or raise eyebrows.

This log is no so frequent nor is the root cause of the issue here.
Changing to info or debug is misleading and improve nothing.

grep ERROR for opening bugs is a bad practice.

The log dissapeared from 3.2.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1aaf82604beef7ec9abbeb83b6a0c2b0b708853b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to