Change in vdsm[master]: Fix mbx Exception type.

ewarszaw at redhat.com ewarszaw at redhat.com
Sun Mar 3 12:02:31 UTC 2013


Eduardo has posted comments on this change.

Change subject: Fix mbx Exception type.
......................................................................


Patch Set 4: (1 inline comment)

....................................................
File vdsm/storage/storage_mailbox.py
Line 726:             #               "command: " + str(cmd))
Line 727:             (rc, in_mail, err) = misc.execCmd(cmd, sudo=False, raw=True)
Line 728:             if rc:
Line 729:                 raise IOError(errno.EIO, "_handleRequests._checkForMail - "
Line 730:                               "Could not read mailbox: %s", self._inbox)
This is not the _inbox content
Line 731: 
Line 732:             if (len(in_mail) != (self._outMailLen)):
Line 733:                 self.log.error('SPM_MailMonitor: _checkForMail - dd succeeded '
Line 734:                                'but read %d bytes instead of %d, cannot check '


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id8baf1121aaf351b029945972d5e87dbfa6a0bd5
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw 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: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list