Doron Fediuck has submitted this change and it was merged. Change subject: tools: engine-notifier accept special chars ......................................................................
tools: engine-notifier accept special chars Due to the way configuration was loaded in the bash script special characters would create some unwanted behaviors. Now the configuration file is read and the variables are added using "declare" statement It is expected that the config file is of the form key=value Change-Id: I5e6419a225bd02edaf1db38a2025dee74761ddf2 Bug-Url: https://bugzilla.redhat.com/883301 Signed-off-by: Noam Slomianko <nslom...@redhat.com> --- M backend/manager/tools/src/main/shell/engine-notifier.sh 1 file changed, 15 insertions(+), 2 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10704 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e6419a225bd02edaf1db38a2025dee74761ddf2 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslom...@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Noam Slomianko <nslom...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: ofri masad <oma...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
