Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: validate host id ...................................................................... packaging: setup: validate host id Added host id validation on additional hosts setup. Requires: http://gerrit.ovirt.org/19140 Rebased the code for being compatible with http://gerrit.ovirt.org/19140 Bug-Url: https://bugzilla.redhat.com/1012417 Change-Id: I2487abfe1f722cb60e6f7c2a8daa4a81fb654bfd Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; --- M src/bin/hosted-engine.in M src/ovirt_hosted_engine_setup/constants.py M src/ovirt_hosted_engine_setup/util.py M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 5 files changed, 124 insertions(+), 33 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve Greg Padgett: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/19171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2487abfe1f722cb60e6f7c2a8daa4a81fb654bfd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com&gt; Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com&gt; Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com&gt; Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: oVirt Jenkins CI Server