Xavi Francisco has uploaded a new change for review. Change subject: storage: use info log level to report missing checksum ......................................................................
storage: use info log level to report missing checksum When logging that the storage metadata is missing the checksum the info level should be used instead of the warning level. This may happen when, for recovery purposes, the metadata must be edited by hand and the checksum information removed. In that case, as it is done in purpose, a warning should not be used. Change-Id: Ic6627fef453a62603d93bfb8aee66ebedcbbf190 Bug-Url: https://bugzilla.redhat.com/1073989 Signed-off-by: Xavi Francisco <xfran...@redhat.com> --- M vdsm/storage/persistentDict.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/28/27528/1 diff --git a/vdsm/storage/persistentDict.py b/vdsm/storage/persistentDict.py index 3464823..c8a86df 100644 --- a/vdsm/storage/persistentDict.py +++ b/vdsm/storage/persistentDict.py @@ -253,7 +253,7 @@ # FIXME : This is ugly but necessary, What we need is a class # method that creates the initial metadata. Then we can assume # that empty metadata is always invalid. - self.log.warn("data has no embedded checksum - " + self.log.info("data has no embedded checksum - " "trust it as it is") self._isValid = True self._metadata = newMD -- To view, visit http://gerrit.ovirt.org/27528 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic6627fef453a62603d93bfb8aee66ebedcbbf190 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Xavi Francisco <xfran...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
