July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: setup: firewall configuration
by sbonazzo＠redhat.com 23 Jul '13

23 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: firewall configuration ...................................................................... packaging: setup: firewall configuration Added firewall manager selection and configuration for enabling access to port 5900 for remote-viewer access to the hosted engine VM console. Change-Id: I6c3ac2ae97416f4a9b82f9a7c404350345a0c765 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/network/Makefile.am M src/plugins/ovirt-hosted-engine-setup/network/__init__.py A src/plugins/ovirt-hosted-engine-setup/network/firewall.py A src/plugins/ovirt-hosted-engine-setup/network/firewall_manager.py M templates/Makefile.am A templates/hosted-console.xml.in A templates/iptables.default.in 9 files changed, 423 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16877 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c3ac2ae97416f4a9b82f9a7c404350345a0c765 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(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>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: moving templates in their own dir
by sbonazzo＠redhat.com 23 Jul '13

23 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: moving templates in their own dir ...................................................................... packaging: moving templates in their own dir moving templates sources in their own directory. Change-Id: Ic6e165fa70e32ef942281f5a90542891d646a844 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M Makefile.am M configure.ac A templates/Makefile.am R templates/hosted-engine.conf.in R templates/vm.conf.in 5 files changed, 30 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16876 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6e165fa70e32ef942281f5a90542891d646a844 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(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>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: ensure engine is up
by sbonazzo＠redhat.com 23 Jul '13

23 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: ensure engine is up ...................................................................... packaging: setup: ensure engine is up Consider engine up only if the answer match db up response. Avoids to try adding the host if the engine is not fully up. Change-Id: I4726a868d81d7f316585d7554c32ee728611875a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/engine/health.py 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16757 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4726a868d81d7f316585d7554c32ee728611875a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(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>

1 0

Change in ovirt-hosted-engine-setup[master]: customization: ask for admin password
by sbonazzo＠redhat.com 23 Jul '13

23 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: customization: ask for admin password ...................................................................... customization: ask for admin password admin password was hardcoded, now it is asked at customization stage. Also avoid to fail the setup if the only missing step is adding the local host to the datacenter (for example for wrong password) Change-Id: I62d1e93ae6ff4a5eb32b243b2f846121ea84595c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 2 files changed, 93 insertions(+), 26 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16756 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62d1e93ae6ff4a5eb32b243b2f846121ea84595c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(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>

1 0

Change in ovirt-hosted-engine-setup[master]: refactored cdrom configuration
by sbonazzo＠redhat.com 23 Jul '13

23 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: refactored cdrom configuration ...................................................................... refactored cdrom configuration Extracted cdrom configuration in its own plugin. Will ease adding ovf boot option. Change-Id: I83688a74d1f478b733bcce8bb838d09256360b8a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/vm/Makefile.am M src/plugins/ovirt-hosted-engine-setup/vm/__init__.py A src/plugins/ovirt-hosted-engine-setup/vm/boot_cdrom.py M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py 5 files changed, 150 insertions(+), 84 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16755 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I83688a74d1f478b733bcce8bb838d09256360b8a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(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>

1 0

Change in ovirt-engine[master]: webadmin: Fix styling of ballooning checkbox
by gchaplik＠redhat.com 23 Jul '13

23 Jul '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Fix styling of ballooning checkbox ...................................................................... webadmin: Fix styling of ballooning checkbox Ballooning enable/disable checkbox in the cluster popup had the wrong width, which caused the text to break into a new line. Change-Id: I5a804619f5102a107a84c2e5ab09175f288870a1 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17237 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a804619f5102a107a84c2e5ab09175f288870a1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: add new signature for /api/hosts|rel=add meta
by mpastern＠redhat.com 23 Jul '13

23 Jul '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: add new signature for /api/hosts|rel=add meta ...................................................................... restapi: add new signature for /api/hosts|rel=add meta new signature adds 'ssh' config args and replaces deprecated root_password with host.ssh.password Change-Id: I63a5265e5e276196fb81757c82b5119a52b35085 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17236 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I63a5265e5e276196fb81757c82b5119a52b35085 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: vdsm: packages: Fix invoke libvirt configuration command fai...
by Alon Bar-Lev 23 Jul '13

23 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: packages: Fix invoke libvirt configuration command failure ...................................................................... vdsm: packages: Fix invoke libvirt configuration command failure In vdsm latest master branch, libvirt reconfiguration command is "vdsm-tool libvirt-configure", but in ovirt-hostp-deploy it calls "vdsm-tool libvirt-reconfigure". This patch fixes the problem. Change-Id: I40c52687f9c2572a30d718f7cd7a62bbc0261c26 Signed-off-by: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, approved Yaniv Bronhaim: Looks good to me, but someone else must approve Zhou Zheng Sheng: Verified -- To view, visit http://gerrit.ovirt.org/17184 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40c52687f9c2572a30d718f7cd7a62bbc0261c26 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: regenerate against the latest api
by mpastern＠redhat.com 23 Jul '13

23 Jul '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: regenerate against the latest api ...................................................................... sdk: regenerate against the latest api - to Cluster added: * [@param cluster.comment: string] * [@param cluster.ballooning_enabled: boolean] - to Host added: * [@param host.comment: string] * [@param host.ssh.port: int] * [@param host.ssh.fingerprint: string] * [@param host.ssh.authentication_type: string] * [@param host.ssh.password: string] * forceselectspm() action - to Network added: * [@param network.comment: string] - to StorageDomain * [@param storagedomain.comment: string] - to Template added: * [@param template.comment: string] * [@param template.permissions.clone: boolean] - to VM added: * [@param vm.comment: string] * [@param vm.permissions.clone: boolean] Change-Id: I4d13130879d3cdebe0540a7f8425d2ad769e7c78 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 2 files changed, 438 insertions(+), 20 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17233 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d13130879d3cdebe0540a7f8425d2ad769e7c78 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Fix ovirt-engine-notifier logging config
by yzaslavs＠redhat.com 23 Jul '13

23 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fix ovirt-engine-notifier logging config ...................................................................... tools: Fix ovirt-engine-notifier logging config Fixes ovirt-engine-noitifier logging config under JBoss EAP. The problem started when JBoss EAP reimplemented log4j classes in JBoss Log Manager, but logging configuration initialization from system property log4j.configuration has not been included in those reimplemented classes. This patch initializes log4j configuration for ovirt-engine-notifier. Change-Id: Iad46dc04853e87e280c2f60100a041d781809eb8 Bug-Url: https://bugzilla.redhat.com/948744 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/Notifier.java 1 file changed, 22 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17051 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad46dc04853e87e280c2f60100a041d781809eb8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0