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 <ykaplan(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com>
Gerrit-Reviewer: humble devassy <hchiramm(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server