Zhou Zheng Sheng has posted comments on this change. Change subject: Change log message from warning to debug ......................................................................
Patch Set 1: I would prefer that you didn't submit this (1 inline comment) .................................................... Commit Message Line 5: CommitDate: 2013-01-03 15:52:44 +0200 Line 6: Line 7: Change log message from warning to debug Line 8: Line 9: Related to BZ#876558 I have a look at https://bugzilla.redhat.com/show_bug.cgi?id=876558 and its fix (commit bee887345). I guess the purpose of this patch is that after the fix, reloading the invalid volume becomes a normal operation, it should not be treated as something unexpected, so the log level should be "debug". Could you add the bugzilla link and the commit id of the fix and explain the intention here? Line 10: Line 11: Change-Id: I87d78a36abe0751960ecd4e7798d79210a3a0704 -- To view, visit http://gerrit.ovirt.org/10621 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I87d78a36abe0751960ecd4e7798d79210a3a0704 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Eduardo <ewars...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Mark Wu <wu...@linux.vnet.ibm.com> Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Gerrit-Reviewer: humble devassy <hchir...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
