- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Remove calls to getImageDomainsList API
by amureini＠redhat.com 14 Oct '13

14 Oct '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Remove calls to getImageDomainsList API ...................................................................... engine: Remove calls to getImageDomainsList API This API is deprecated in VDSM. Use getImagesList instead. Change-Id: I33069871db34e8b129c4d80098523654cd2a0629 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommandTestAbstract.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/GetImageDomainsListVDSCommandParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/GetImageDomainsListVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerWrapper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 13 files changed, 182 insertions(+), 127 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15951 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33069871db34e8b129c4d80098523654cd2a0629 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eduardo <ewarszaw(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: liron aravot <liron.aravot(a)gmail.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: agent: use vdsm domain monitor
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: use vdsm domain monitor ...................................................................... agent: use vdsm domain monitor Ensure that the storage domain is monitored by VDSM using the repoStats and startMonitoringDomain APIs. Change-Id: I9bab369784fecbf972622922a925fd0b45de123f Requires: http://gerrit.ovirt.org/19762 Requires: http://gerrit.ovirt.org/20099 Bug-Url: https://bugzilla.redhat.com/1017406 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt-hosted-engine-ha.spec.in M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/hosted_engine.py 3 files changed, 82 insertions(+), 4 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20051 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bab369784fecbf972622922a925fd0b45de123f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(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 ovirt-hosted-engine-ha[master]: packaging: distribute ovirt-hosted-engine-ha.spec.in in tarball
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: packaging: distribute ovirt-hosted-engine-ha.spec.in in tarball ...................................................................... packaging: distribute ovirt-hosted-engine-ha.spec.in in tarball .spec.in file is needed for a complete rebuild from the tarball. Change-Id: I86c804f39a23b4cc905d52f74fb7621bf9e80bc4 Bug-Url: https://bugzilla.redhat.com/1018278 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M Makefile.am 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20120 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86c804f39a23b4cc905d52f74fb7621bf9e80bc4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: packaging: fixed sudoers generation
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: packaging: fixed sudoers generation ...................................................................... packaging: fixed sudoers generation sudoers is now generated at build time. Bug-Url: https://bugzilla.redhat.com/1013563 Change-Id: I1e069cefbd55a4aa7a85abd48beded57846ce78f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M sudoers/Makefile.am 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20164 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e069cefbd55a4aa7a85abd48beded57846ce78f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(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 ovirt-hosted-engine-ha[master]: build: ovirt-hosted-engine-ha-0.1.0-0.3.beta1
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-0.1.0-0.3.beta1 ...................................................................... build: ovirt-hosted-engine-ha-0.1.0-0.3.beta1 Change-Id: I5dae97f99e60e5bafac49c89ba679b05846601f0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Greg Padgett: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20165 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5dae97f99e60e5bafac49c89ba679b05846601f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(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 ovirt-hosted-engine-setup[master]: vdsm: use startMonitoringDomain
by sbonazzo＠redhat.com 14 Oct '13

14 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: use startMonitoringDomain ...................................................................... vdsm: use startMonitoringDomain Use startMonitoringDomain VDSM API for monitoring hosted engine storage domain in order to support shared:exclusive images without a pool. Use getVdsStats for waiting the start of the domain monitor. Updated vdsm minimum version required according to VDSM API documentation for startMonitoringDomain. Change-Id: I3099081b79d461685901a1a32e9d526bd3bb758a Requires: http://gerrit.ovirt.org/19762 Requires: http://gerrit.ovirt.org/20099 Bug-Url: https://bugzilla.redhat.com/1015136 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/ovirt_hosted_engine_setup/tasks.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 3 files changed, 53 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/19867 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3099081b79d461685901a1a32e9d526bd3bb758a Gerrit-PatchSet: 2 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: Eduardo <ewarszaw(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: Martin Sivák <msivak(a)redhat.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 ovirt-engine[master]: packaging: setup: revert nfs config on cleanup
by sbonazzo＠redhat.com 14 Oct '13

14 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: revert nfs config on cleanup ...................................................................... packaging: setup: revert nfs config on cleanup Revert /etc/sysconfig/nfs to its original contents on cleanup. Added a function editConfigContent in util to generalize edit loops that read conf files and do some changes to them. Used it to edit /etc/sysconfig/nfs as well as /etc/httpd/conf.d/ssl.conf and /var/lib/pgsql/data/postgresql.conf. Bug-Url: https://bugzilla.redhat.com/1013347 Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/util.py M packaging/setup/plugins/ovirt-engine-setup/apache/ssl.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py M packaging/setup/plugins/ovirt-engine-setup/system/nfs.py 4 files changed, 203 insertions(+), 79 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19001 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74dd4c1556bd6479fcf10f85fcbe083f215e0854 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(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 ovirt-engine[ovirt-engine-3.3]: engine: Allow vlan to serve as a migration network
by iheim＠redhat.com 14 Oct '13

14 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Allow vlan to serve as a migration network ...................................................................... engine: Allow vlan to serve as a migration network In order for a nic device to be entitled to serve as the underlying interface for a migration network, it should be UP. The nics statuses are reported by the getVdsStats verb of VDSM which doesn't report it for vlan devices. Therefore, until the statistics will be collected for vlans, we'll allow migration to go over the vlan if its underlying nic is UP. Change-Id: Icdfcca442d40d8f2c3b6b28dc9f520e286b7e8f2 Bug-Url: https://bugzilla.redhat.com/1014013 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java 1 file changed, 15 insertions(+), 2 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20146 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icdfcca442d40d8f2c3b6b28dc9f520e286b7e8f2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Inconsistent schema implementation in PayloadFile #...
by mpastern＠redhat.com 14 Oct '13

14 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: Inconsistent schema implementation in PayloadFile #1018782 ...................................................................... restapi: Inconsistent schema implementation in PayloadFile #1018782 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1018782 Change-Id: I82ec264bb18798fd911143741b60a4e9469efd48 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 18 insertions(+), 28 deletions(-) Approvals: Michael Pasternak: Verified Ori Liel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20162 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82ec264bb18798fd911143741b60a4e9469efd48 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: Samples: score by similar os for ksm
by dfediuck＠redhat.com 14 Oct '13

14 Oct '13

Doron Fediuck has submitted this change and it was merged. Change subject: Samples: score by similar os for ksm ...................................................................... Samples: score by similar os for ksm Score each host by how many of its vms have the same os as the requested vm. Bug-Url: https://bugzilla.redhat.com/773338 Change-Id: Idfb359ecbd73a1f4a2520ac8a65b753449be55bd Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> --- A plugins/examples/ksm_same_os_score.py 1 file changed, 62 insertions(+), 0 deletions(-) Approvals: Doron Fediuck: Looks good to me, approved Noam Slomianko: Verified -- To view, visit http://gerrit.ovirt.org/20033 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfb359ecbd73a1f4a2520ac8a65b753449be55bd Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0