- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: virtio -> VirtIO
by tjelinek＠redhat.com 12 Sep '13

12 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine: virtio -> VirtIO ...................................................................... engine: virtio -> VirtIO This patch correct wrong VirtIO case. Change-Id: Ifc2b02e02f1656b63dba84ffbe998f4a1db8a0a3 Bug-Url: https://bugzilla.redhat.com/999977 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18658 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifc2b02e02f1656b63dba84ffbe998f4a1db8a0a3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: VM NewPool-1 was restarted on Host <UNKNOWN>
by tjelinek＠redhat.com 12 Sep '13

12 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine: VM NewPool-1 was restarted on Host <UNKNOWN> ...................................................................... engine: VM NewPool-1 was restarted on Host <UNKNOWN> If the host has been started as prestarted the following event log has been logged: VM NewPool-1 was restarted on Host <UNKNOWN> The problem was that the Vm has been started as stateless. This first triggers the statelessVmTreatment() which does not schedule the host but will already be logged (e.g. the host will be unknown). Fixed by checking if the host has already been scheduled and if not than a different message will be logged (a similar to the one when the VM is started as stateless by the user). Change-Id: I60a74762984dc82154d0f8e073c60f631da229f0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=966619 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 4 files changed, 17 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18726 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60a74762984dc82154d0f8e073c60f631da229f0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: engine-backup - support old md5sum
by oschreib＠redhat.com 12 Sep '13

12 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: engine-backup - support old md5sum ...................................................................... packaging: engine-backup - support old md5sum Removed '--strict' md5sum option, which isn't supported on earlier versions of md5sum. Change-Id: I723bc81e84be05aa2fb3ca2ceb8210a2bfe7adc9 Signed-off-by: Ofer Schreiber <oschreib(a)redhat.com> Bug-URL: https://bugzilla.redhat.com/1003869 --- M packaging/bin/engine-backup.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I723bc81e84be05aa2fb3ca2ceb8210a2bfe7adc9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: build: commons-lang is available at dependencies module
by oschreib＠redhat.com 12 Sep '13

12 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: build: commons-lang is available at dependencies module ...................................................................... build: commons-lang is available at dependencies module Change-Id: Id8b531d2cf770d1efb130852373bb20a254e02a0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/root/pom.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19125 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8b531d2cf770d1efb130852373bb20a254e02a0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar
by amureini＠redhat.com 12 Sep '13

12 Sep '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar ...................................................................... core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar Fixed messages text. Change-Id: Ie898ba54ab5b673eefb594350df5ee22cf145de0 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19179 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie898ba54ab5b673eefb594350df5ee22cf145de0 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: setting singleQxlPci to false when importing external VMs
by oourfali＠redhat.com 12 Sep '13

12 Sep '13

Oved Ourfali has submitted this change and it was merged. Change subject: core: setting singleQxlPci to false when importing external VMs ...................................................................... core: setting singleQxlPci to false when importing external VMs Change-Id: I8ef10def74461d4fdd0e4c596eea1a157e6463e8 Bug-Url: https://bugzilla.redhat.com/978268 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/19180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ef10def74461d4fdd0e4c596eea1a157e6463e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Add subcollections for network resource
by masayag＠redhat.com 12 Sep '13

12 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Add subcollections for network resource ...................................................................... restapi: Add subcollections for network resource The patch adds the vnicprofiles and permissions subcollections for a single network resource. Change-Id: I8b8d8b144dcac69d5355bd26f8ffacb9e0fdfc62 Bug-Url: https://bugzilla.redhat.com/1003881 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendNetworkResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNetworkResource.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/18839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b8d8b144dcac69d5355bd26f8ffacb9e0fdfc62 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Add error message for ifup failure
by masayag＠redhat.com 12 Sep '13

12 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Add error message for ifup failure ...................................................................... engine: Add error message for ifup failure The patch adds a missing error code from vdsm for a failure to bring an interface up. Change-Id: I07adde3368c31319bed48839d089ba3ac48cd424 Bug-Url: https://bugzilla.redhat.com/990938 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties 5 files changed, 6 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19111 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I07adde3368c31319bed48839d089ba3ac48cd424 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Network interface name is mandatory
by masayag＠redhat.com 12 Sep '13

12 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Network interface name is mandatory ...................................................................... engine: Network interface name is mandatory The network interface name is used to identify the nic, therefore it cannot be omitted. The patch adds a NotNull constraint for the interface name. Change-Id: I26f33bf12e9b53c3e31719b93a689e11e974acca Bug-Url: https://bugzilla.redhat.com/979751 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SetupNetworksParametersTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/VdsNetworkInterface.java 2 files changed, 3 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26f33bf12e9b53c3e31719b93a689e11e974acca Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: correctly parse stroage domain info
by sbonazzo＠redhat.com 12 Sep '13

12 Sep '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: correctly parse stroage domain info ...................................................................... packaging: setup: correctly parse stroage domain info previously the getStorageDomainInfo parser assumed that remotePath was always defined in the dictionary returned by the call to getStorageDomainInfo. The key is not defined for iSCSI domains so now the parser check for remotePath availability before trying to use its value. Change-Id: I7e8db0f31dadd7072680fb1564514be065ea5431 Bug-Url: https://bugzilla.redhat.com/1005767 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19115 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e8db0f31dadd7072680fb1564514be065ea5431 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: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(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