Maor Lipchuk has posted comments on this change.

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


Patch Set 1: Code-Review+1

Acttually, the problem which the patch solves here is much more worst.
Empty prefix will be set in the cache map every time the DC will not have ISO 
domain attached to it.
If user will attach an ISO domain after the cache map has been already 
initialized with the empty string, then the user will not be able to attach CD 
to its VMs

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f77f92c58d9e9336e15302809195928a1ac84a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to