Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#856167 - Parse correctly the management_server parameter ......................................................................
BZ#856167 - Parse correctly the management_server parameter When doing automatic installations we parse the management_server parameter to find the engine address and port number, but we don't parse it correctly if the port number is missing, in this case we end up using the address as both address and port number. This patch fixes that so when the port number is missing the default 443 will be used. Change-Id: I74d658902d3421c0ac554c5c2634ddc8c6a28b21 Signed-off-by: Juan Hernandez <[email protected]> --- M vdsm_reg/vdsm-config 1 file changed, 10 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Looks good to me, but someone else must approve Juan Hernandez: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8022 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74d658902d3421c0ac554c5c2634ddc8c6a28b21 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Michael Burns <[email protected]> Gerrit-Reviewer: Ryan Harper <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
