[node-patches] Change in ovirt-node[master]: init: Split management_server if user used "addr:port"

fabiand at redhat.com fabiand at redhat.com
Thu Sep 11 14:01:09 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: init: Split management_server if user used "addr:port"
......................................................................


init: Split management_server if user used "addr:port"

If user specify in the autoinstall management_server="addr:port"
it will set the variable OVIRT_MANAGEMENT_SERVER="addr:port" in
/etc/default/ovirt which is incorrect. We should split it
and set OVIRT_MANAGEMENT_SERVER="addr" and OVIRT_MANAGEMENT_PORT="port".
This patch adds a validation into the boot schema to split the
management_server data correctly into /etc/default/ovirt schema.

Change-Id: I8b0d4055a8fd66c761a977d160422befc42019f9
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1065401
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M scripts/ovirt-init-functions.sh.in
1 file changed, 11 insertions(+), 2 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved
  Douglas Schilling Landgraf: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8b0d4055a8fd66c761a977d160422befc42019f9
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the node-patches mailing list