Martin Sivák has submitted this change and it was merged. Change subject: storage: validating NFS/gluster paths ...................................................................... storage: validating NFS/gluster paths Validating NFS/gluster paths with os.path.normpath as VDSM does. Try to cleanly disconnect a previously connected storage server if the storage server is already mounted on a wrong path. Asking the user to reboot if disconnectStorageServer fails (it will clean the whole /var/run becouse is on tmpfs). Change-Id: Ie7c8daa814dfdc3aaf3bae382e90b8d166839f40 Bug-Url: https://bugzilla.redhat.com/1300749 Signed-off-by: Simone Tiraboschi <stirabos@redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/env/path.py M ovirt_hosted_engine_ha/lib/exceptions.py M ovirt_hosted_engine_ha/lib/storage_server.py M ovirt_hosted_engine_ha/lib/upgrade.py 5 files changed, 165 insertions(+), 24 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/53357 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7c8daa814dfdc3aaf3bae382e90b8d166839f40 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>