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(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: shu ming <sming56(a)gmail.com>