Allon Mureinik has submitted this change and it was merged.

Change subject: backend: don't cache isoPrefix when empty
......................................................................


backend: don't cache isoPrefix when empty

In rare occasions (e.g. when an iso domain was unreachable at
connectStoragePool time), the isoPrefix for the pool might be
reported empty by vdsm and eventually the value is cached (if
a VM using a cdrom is started before the iso domain becomes
available again).

This patch avoids to cache empty (faulty) isoPrefix values.

Change-Id: I6f77f92c58d9e9336e15302809195928a1ac84a6
Signed-off-by: Federico Simoncelli <fsimo...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IsoPrefixVDSCommand.java
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6f77f92c58d9e9336e15302809195928a1ac84a6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to