Change in vdsm[master]: Refactor storage connection subsystem

danken at redhat.com danken at redhat.com
Tue Mar 6 21:45:13 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Refactor storage connection subsystem
......................................................................


Refactor storage connection subsystem

Currently connection management is an abysmal mess. This is a
transitional patch
retrofitting the old API to some of the concepts we intend to implement.

Notice:
Permissions are no longer checked on storage connect only on domain
initialization. This is good as the storage connection is far from an
appropriate place for that check.

I've removed stale root folder handling for NFS. I will add it back I
can't do everything at once.

There are also no proper engine acceptable exceptions yet

Change-Id: Ifc5c160aba9b4cc88225dde0c44f9b766c6a4c0d
---
M vdsm.spec.in
M vdsm/storage/Makefile.am
M vdsm/storage/fileSD.py
M vdsm/storage/hsm.py
M vdsm/storage/iscsiadm.py
M vdsm/storage/localFsSD.py
M vdsm/storage/nfsSD.py
A vdsm/storage/storageServer.py
D vdsm/storage/storage_connection.py
M vdsm/storage/storage_exception.py
10 files changed, 553 insertions(+), 433 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifc5c160aba9b4cc88225dde0c44f9b766c6a4c0d
Gerrit-PatchSet: 20
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud at kohlvanwijngaarden.nl>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: shu ming <sming56 at gmail.com>


More information about the vdsm-patches mailing list