- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: setup: detect display port at runtime
by sbonazzo＠redhat.com 07 Oct '13

07 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: detect display port at runtime ...................................................................... packaging: setup: detect display port at runtime previously the tool wrongly assumed that the SPICE and the VNC port was always 5900. It may happen however that secureDisplayPort is set to 5901 instead of 5900. Now displayPort and secureDisplayPort are detected at runtime querying the status of the VM. Firewall ports 5900 and 5901 are now both open. Change-Id: I8ae46b061b2a3ad0190f798cb141a41431120839 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py M templates/hosted-console.xml.in 2 files changed, 29 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19869 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ae46b061b2a3ad0190f798cb141a41431120839 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: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in otopi[otopi-1.1]: packaging: post otopi-1.1.2
by Alon Bar-Lev 07 Oct '13

07 Oct '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.1.2 ...................................................................... packaging: post otopi-1.1.2 Change-Id: I4478425dde697e1bd467c6f8ff47724b5880671f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19924 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4478425dde697e1bd467c6f8ff47724b5880671f Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: otopi-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[otopi-1.1]: packaging: otopi-1.1.2
by Alon Bar-Lev 07 Oct '13

07 Oct '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.1.2 ...................................................................... packaging: otopi-1.1.2 Change-Id: Ia7793d1f9dfeeaca18c964aab581c49805a1c14d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19923 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7793d1f9dfeeaca18c964aab581c49805a1c14d Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: otopi-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: Update sysconfig/nfs contents
by sbonazzo＠redhat.com 07 Oct '13

07 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Update sysconfig/nfs contents ...................................................................... packaging: setup: Update sysconfig/nfs contents conf/nfs.sysconfig.in, which is used to update /etc/sysconfig/nfs, sets variables that are no longer used in recent fedora nfs-utils init scripts. This change sets instead variables that are used by the init system of both new fedora and old fedora/rhel/centos. Bug-Url: https://bugzilla.redhat.com/1011853 Change-Id: Ibc144c85a14e967c1994d35d36c3b3e16aa5ff77 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/conf/nfs.sysconfig.in 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc144c85a14e967c1994d35d36c3b3e16aa5ff77 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(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 otopi[otopi-1.1]: dialog: cli: fix exception show
by Alon Bar-Lev 07 Oct '13

07 Oct '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: dialog: cli: fix exception show ...................................................................... dialog: cli: fix exception show Change-Id: I8d834ec974bdf7a0a40888da077f1d46cb2b03a9 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/otopi/dialog/cli.py 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19920 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d834ec974bdf7a0a40888da077f1d46cb2b03a9 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: dialog: cli: fix exception show
by Alon Bar-Lev 07 Oct '13

07 Oct '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: dialog: cli: fix exception show ...................................................................... dialog: cli: fix exception show Change-Id: I8d834ec974bdf7a0a40888da077f1d46cb2b03a9 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/otopi/dialog/cli.py 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19919 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d834ec974bdf7a0a40888da077f1d46cb2b03a9 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: Do not create a new db on upgrade
by sbonazzo＠redhat.com 07 Oct '13

07 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Do not create a new db on upgrade ...................................................................... packaging: setup: Do not create a new db on upgrade Before this change, the following scenario made engine-setup emit a warning and create a new db: 1. engine-setup 2. service ovirt-engine stop; service postgresql stop 3. engine-setup This change changes the warning to an error (exception) unless it's during cleanup. Bug-Url: https://bugzilla.redhat.com/1014012 Change-Id: Ia336e951860f6e1a6a829cabf5e6b5e10592089b Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/db/connection.py 1 file changed, 14 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia336e951860f6e1a6a829cabf5e6b5e10592089b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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]: packaging: setup: weird message on cleanup if no db
by sbonazzo＠redhat.com 07 Oct '13

07 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: weird message on cleanup if no db ...................................................................... packaging: setup: weird message on cleanup if no db Make engine-cleanup continue gracefully if the database was manually deleted. Change-Id: Iaba0ba1563acbf8c0ef900b304aa577a3dd1f615 Bug-Url: https://bugzilla.redhat.com/1015037 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/db/pgpass.py 1 file changed, 20 insertions(+), 15 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19886 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaba0ba1563acbf8c0ef900b304aa577a3dd1f615 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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]: core: Support JSON serialization in ExternalId
by yzaslavs＠redhat.com 07 Oct '13

07 Oct '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Support JSON serialization in ExternalId ...................................................................... core: Support JSON serialization in ExternalId This patch adds a default constructor to the ExternalId class in order to support JSON serialization as required by the compensation mechanism. Change-Id: I51e643893b9e59cb0cacd1779398946f1babe694 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/ExternalId.java A backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/ExternalIdTest.java 2 files changed, 34 insertions(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19916 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51e643893b9e59cb0cacd1779398946f1babe694 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: frontend: Adding FQDN column to VmViews
by tjelinek＠redhat.com 07 Oct '13

07 Oct '13

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Adding FQDN column to VmViews ...................................................................... frontend: Adding FQDN column to VmViews The VM Views now also display the FQDN of the Virtual Machine Change-Id: If84c4d2a8c1f11e09f3748477a230d7d3188482b Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVirtualMachineView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/disk/SubTabDiskVmView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostVmView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkVmView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/pool/SubTabPoolVmView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/template/SubTabTemplateVmView.java 6 files changed, 52 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17865 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If84c4d2a8c1f11e09f3748477a230d7d3188482b Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0