Tal Nisan has submitted this change and it was merged. Change subject: core: fixing FenceAgent toString ...................................................................... core: fixing FenceAgent toString core: fixing FenceAgent toString to display options correctly The protocol between engine and VDSM requires that the separator between fence agent options will be a NEWLINE. However we are implementing the toString method that is using the logger to write log messages, we should replace the NEWLINE with COMMA. Change-Id: Ic117ee404fb909515d5a3d18c6e2c1ba8fa1a9b0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1457814 Signed-off-by: emesika <emesika(a)redhat.com&gt; --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/pm/FenceAgent.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/78695 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic117ee404fb909515d5a3d18c6e2c1ba8fa1a9b0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com&gt; Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;