Change in ovirt-hosted-engine-setup[master]: storage: avoid tracebacks mixing storage domains

[sbonazzo at redhat.com]

Sandro Bonazzola has submitted this change and it was merged.

Change subject: storage: avoid tracebacks mixing storage domains
......................................................................


storage: avoid tracebacks mixing storage domains

Avoid traceback like:
"/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/ovirt-hosted-engine-setup/storage/storage.py",
line 340, in _getExistingDomain
    domain_info['remotePath']
  File
"/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/ovirt-hosted-engine-setup/storage/storage.py",
line 283, in _removeNFSTrailingSlash
    if len(nfspath[1]) > 1:
IndexError: list index out of range

when checking NFS storage domains on a VDSM instance running with a
non NFS/Gluster domain already attached.

Change-Id: I4d653a3ab0b6bb324a9380ee351307ca7ebb34bc
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d653a3ab0b6bb324a9380ee351307ca7ebb34bc
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server