Change in ovirt-engine[master]: packaging: setup: allow invalid fqdn in non interactive mode

[oschreib at redhat.com]

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 at redhat.com>
Reported-By: Michael Pasternak <mpastern at 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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>