Tomas Jelinek has posted comments on this change.

Change subject: engine: custom properties: allow '=' in regex
......................................................................


Patch Set 3: Looks good to me, but someone else must approve

hi Gilad, after a quick look at the sources it seems that REST (VmMapper) and 
tools (CustomPropertiesValueHelper) have the same limitation. Do you think it 
should be part of this patch?

The FE seems to be OK for me, so giving +1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic845e2ab5a0f23d48c1506f4f834a851009ea253
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to