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 <xfran...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches