Ofer Schreiber has submitted this change and it was merged.
Change subject: packaging: setup: allow invalid fqdn in non interactive mode
......................................................................
packaging: setup: allow invalid fqdn in non interactive mode
in interactive mode we allow user to retry and correct invalid host
name, in non interactive mode we simply fail installation.
there are two reasons why we should not fail installation:
1. in upgrade we need to accept whatever host name existed during setup.
this is easy to solve by activating validation only for setup.
2. user who specify host name via cmd-line should know what he is doing,
we cannot enforce our decision over.
this change will produce error during non interactive setup, but will
allow setup to continue.
Change-Id: Ic6c80ac47d55ade01dd6a3472e950d2c0ffa0074
Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com>
Reported-By: Michael Pasternak <mpastern(a)redhat.com>
---
M packaging/setup/plugins/ovirt-engine-setup/config/hostname.py
1 file changed, 8 insertions(+), 8 deletions(-)
Approvals:
Ofer Schreiber: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/16543
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6c80ac47d55ade01dd6a3472e950d2c0ffa0074
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>