Change in vdsm[master]: storage: use info log level to report missing checksum

Federico Simoncelli fsimonce at redhat.com
Mon Jun 9 08:15:00 UTC 2014


Federico Simoncelli has posted comments on this change.

Change subject: storage: use info log level to report missing checksum
......................................................................


Patch Set 1: Code-Review-1

The warning must remain a warning.

What we should do is to check if we need to display it or not, because when the metadata is empty (e.g. bz1073989) there's also no checksum to check.

The metadata being empty is something expected and it may deserve just a debug message.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic6627fef453a62603d93bfb8aee66ebedcbbf190
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xavi Francisco <xfrancis at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list