Change in vdsm[master]: sdc: Change error messages to info

sgotliv at redhat.com sgotliv at redhat.com
Mon Dec 2 07:39:31 UTC 2013


Sergey Gotliv has posted comments on this change.

Change subject: sdc: Change error messages to info
......................................................................


Patch Set 1:

(1 comment)

We all know that for each FC domain we'll see that message at least once and since this is expected behavior and won't be changed in the near future, why not debug or info?

....................................................
File vdsm/storage/sdc.py
Line 139: 
Line 140:         try:
Line 141:             dom = findMethod(sdUUID)
Line 142:         except se.StorageDomainDoesNotExist:
Line 143:             self.log.info("domain %s not found", sdUUID, exc_info=True)
This error is real.
Line 144:             raise
Line 145:         else:
Line 146:             return dom
Line 147: 


-- 
To view, visit http://gerrit.ovirt.org/21597
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1aaf82604beef7ec9abbeb83b6a0c2b0b708853b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list