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@redhat.com
Gerrit-Reviewer: Allon Mureinik
amureini@redhat.com
Gerrit-Reviewer: Federico Simoncelli
fsimonce@redhat.com
Gerrit-Reviewer: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No