Change in vdsm[master]: sp: remove automatic storage pool reconnection

danken at redhat.com danken at redhat.com
Mon Dec 23 17:44:46 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: sp: remove automatic storage pool reconnection
......................................................................


sp: remove automatic storage pool reconnection

On startup the storage pool directory (/rhev/data-center/<spUUID>)
is removed by __cleanStorageRepository preventing the automatic
reconnection (_connectStoragePool) from actually taking place.
Since vdsm has been running for quite some time (years) without the
automatic reconnection we can safely remove it.

Change-Id: Ia57afc04db02b6c15633d09349a55b3ff5ae7fda
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21424
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M build-aux/Makefile.subs
M configure.ac
M debian/vdsm.dirs
M debian/vdsm.postinst
M init/vdsmd_init_common.sh.in
M lib/vdsm/config.py.in
M vdsm.spec.in
M vdsm/Makefile.am
M vdsm/storage/hsm.py
M vdsm/storage/sp.py
10 files changed, 1 insertion(+), 70 deletions(-)

Approvals:
  Federico Simoncelli: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia57afc04db02b6c15633d09349a55b3ff5ae7fda
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce 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: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk 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


More information about the vdsm-patches mailing list