Change in ovirt-engine[master]: tools: Fix engine-config description processing

yzaslavs at redhat.com yzaslavs at redhat.com
Tue Jul 9 09:40:57 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: tools: Fix engine-config description processing
......................................................................


tools: Fix engine-config description processing

The "description" field was being processed by calling "getString" which
would split the string by the delimiter character behind the scenes,
causing only the first string to return.

Now, the string will be rejoined using that delimiter character (padding
is added because the split string is trimmed).

Change-Id: Ieb6bf225b06d0ed8491e3c725daef7bd500b1ae1
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
---
M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/ConfigKeyFactory.java
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb6bf225b06d0ed8491e3c725daef7bd500b1ae1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list