[node-patches] Change in ovirt-node[master]: init: Fix the HOSTNAME check

dougsland at redhat.com dougsland at redhat.com
Fri Jan 29 16:46:51 UTC 2016


Douglas Schilling Landgraf has submitted this change and it was merged.

Change subject: init: Fix the HOSTNAME check
......................................................................


init: Fix the HOSTNAME check

HOSTNAME is checkd initially in the init script when the hostname arge
is read from the cmdline.
This breaks the HOSTNAME check further down, when it is read from the
sysconfig/network file.
The fix is to make the variale local before perfoming the check, if it's
local then it will only have the value from the sourced sysconfig/network
file, and not from further up.

Change-Id: Icac879a8c1f3e2269aa42c6e7c4268d82e4f03e3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1254580
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/ovirt-init-functions.sh.in
1 file changed, 12 insertions(+), 4 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests
  Fabian Deutsch: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/52879
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icac879a8c1f3e2269aa42c6e7c4268d82e4f03e3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the node-patches mailing list