- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.3]: host-deploy: configure ssh port per port used to actual comm...
by iheim＠redhat.com 22 Oct '13

22 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: host-deploy: configure ssh port per port used to actual communications ...................................................................... host-deploy: configure ssh port per port used to actual communications Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1018948 Change-Id: I1b581f81ad3f11b22eb20b72d34a41fae47f18bc Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20309 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b581f81ad3f11b22eb20b72d34a41fae47f18bc Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Remove non-existent Storage Domain will be rolled forw...
by iheim＠redhat.com 22 Oct '13

22 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: core: Remove non-existent Storage Domain will be rolled forward. ...................................................................... core: Remove non-existent Storage Domain will be rolled forward. Removal of a Storage Domain which no longer exists causes a failure in the engine and blocks the user from removing the storage. The following patch validates the storage domain was not removed due to the reason of non existence, and if so it will roll-forward and delete it any how. Change-Id: I9515a5ddfcaa965c2bad1883f0543494915b8c7d Bug-Url: https://bugzilla.redhat.com/1001626 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStorageDomainCommand.java 1 file changed, 15 insertions(+), 3 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20333 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9515a5ddfcaa965c2bad1883f0543494915b8c7d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine-config: Add EncryptHostCommunication
by iheim＠redhat.com 22 Oct '13

22 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: engine-config: Add EncryptHostCommunication ...................................................................... engine-config: Add EncryptHostCommunication Make accessible EncryptHostCommunication via engine-config tool. Change-Id: I9ba5e548d031658089641a5442b5068515b61676 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1003117 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M packaging/etc/engine-config/engine-config.properties 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19861 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9ba5e548d031658089641a5442b5068515b61676 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Rename UseSecureConnectionWithServers
by iheim＠redhat.com 22 Oct '13

22 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Rename UseSecureConnectionWithServers ...................................................................... engine: Rename UseSecureConnectionWithServers Rename UseSecureConnectionWithServers to EncryptHostCommunication to be more descriptive. Change-Id: Ia25ed8e39a1a0bc737fb198e90d92489a8fe346c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1003117 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVdsCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/VdsManagerTest.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/setup/plugins/ovirt-engine-setup/config/options.py 10 files changed, 16 insertions(+), 14 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia25ed8e39a1a0bc737fb198e90d92489a8fe346c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: fix typeo in commit 36dcd16a0c64
by oschreib＠redhat.com 22 Oct '13

22 Oct '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fix typeo in commit 36dcd16a0c64 ...................................................................... packaging: setup: fix typeo in commit 36dcd16a0c64 Change-Id: I1a3e6029874e52de6de5c82be953401699d3be9c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/misc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20378 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1a3e6029874e52de6de5c82be953401699d3be9c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(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>

1 0

Change in ovirt-engine-cli[master]: cli: cleanup
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: cleanup ...................................................................... cli: cleanup Change-Id: I051dd8ee06bbbc115ca963cbc1bc1be2c816ae6c Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/action.py M src/ovirtcli/command/add.py M src/ovirtcli/command/connect.py M src/ovirtcli/command/console.py M src/ovirtcli/command/disconnect.py M src/ovirtcli/command/info.py M src/ovirtcli/command/list.py M src/ovirtcli/command/ping.py M src/ovirtcli/command/remove.py M src/ovirtcli/command/show.py M src/ovirtcli/command/update.py M src/ovirtcli/utils/colorhelper.py 12 files changed, 380 insertions(+), 184 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20355 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I051dd8ee06bbbc115ca963cbc1bc1be2c816ae6c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: limit history length to 3000
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: limit history length to 3000 ...................................................................... cli: limit history length to 3000 Change-Id: Iab4633d3ea9640eabf357d19ff93190688ae187f Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/historymanager.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20353 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab4633d3ea9640eabf357d19ff93190688ae187f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: raise CommandError on generic errors
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: raise CommandError on generic errors ...................................................................... cli: raise CommandError on generic errors Change-Id: I7bf173b4a1ab112d4fbc30fbfe797b9fe4929800 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/shell/engineshell.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20328 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7bf173b4a1ab112d4fbc30fbfe797b9fe4929800 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: implement /exit events listener
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement /exit events listener ...................................................................... cli: implement /exit events listener ExitListener is listens for the /exit events and triggers /disconnect command, this way even if shell will be killed via ctrl-d event (not via standard way of exiting - /exit command), /disconnect command still will be triggered releasing local and remote resources. Change-Id: If5dcd1a7433f71f4b4321641b168e72659f13058 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/disconnect.py A src/ovirtcli/listeners/exitlistener.py M src/ovirtcli/shell/engineshell.py 3 files changed, 48 insertions(+), 4 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20323 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5dcd1a7433f71f4b4321641b168e72659f13058 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: introducing event driven messaging
by mpastern＠redhat.com 22 Oct '13

22 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: introducing event driven messaging ...................................................................... cli: introducing event driven messaging Change-Id: I110a2b7d25a03ed1f256f24058f3c54535e27100 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- A src/ovirtcli/annotations/__init__.py A src/ovirtcli/annotations/requires.py A src/ovirtcli/events/__init__.py A src/ovirtcli/events/abstractevent.py A src/ovirtcli/events/event.py A src/ovirtcli/events/ievent.py A src/ovirtcli/listeners/__init__.py A src/ovirtcli/listeners/abstractlistener.py A src/ovirtcli/listeners/errorlistener.py A src/ovirtcli/listeners/ilistener.py M src/ovirtcli/shell/cmdshell.py M src/ovirtcli/shell/engineshell.py 12 files changed, 391 insertions(+), 31 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20321 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I110a2b7d25a03ed1f256f24058f3c54535e27100 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0