From Dan Kenigsberg <[email protected]>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: exception: Add context to some virt errors
......................................................................


exception: Add context to some virt errors

Change NoSuchVM and MigrationInProgress to inherit from
ContextException, so we can raise these errors with context in new code
for plugging and unplugging leases.

Nobody is raising these errors yet, since virt code is still using the
define module, using same instance of the exception for all errors.

Change-Id: I32ec3f86f8d53f8412f4c0526fc85e2a42e30ea5
Bug-Url: https://bugzilla.redhat.com/1408825
Signed-off-by: Nir Soffer <[email protected]>
---
M lib/vdsm/common/exception.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Martin Polednik: Looks good to me, but someone else must approve
  Milan Zamazal: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/69401
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I32ec3f86f8d53f8412f4c0526fc85e2a42e30ea5
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Milan Zamazal <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to