- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Fixing correlationId in MultipleActionRunner
by mkublin＠redhat.com 01 May '13

01 May '13

Michael Kublin has submitted this change and it was merged. Change subject: engine: Fixing correlationId in MultipleActionRunner ...................................................................... engine: Fixing correlationId in MultipleActionRunner The following patch should connect canDoAction with execute part of command by correlationId. >From now, all loggs related to command which was run via MultipleactionRunner will have same correlationId. Removed some code which is checking if command has correlationId as irrelavant Change-Id: I2e0ce3452af69cfde3bc5f6b1075589455ce4cce Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MultipleActionsRunner.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/job/ExecutionHandler.java 3 files changed, 8 insertions(+), 18 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14329 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e0ce3452af69cfde3bc5f6b1075589455ce4cce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: initVdsOnUp- remove reconstruct and host status.
by mkublin＠redhat.com 01 May '13

01 May '13

Michael Kublin has submitted this change and it was merged. Change subject: core: initVdsOnUp- remove reconstruct and host status. ...................................................................... core: initVdsOnUp- remove reconstruct and host status. *Host that runs through init vds on up shouldn't attempt to reconstruct. *In case of failure during ConnectStoragePool, the host should fail in initalizeStorage() only if the master domain isn't in inactive/ unknown status and the exception wasn't XmlRpcRunTimeException (which is represented by VdcBllErrors.VDS_NETWORK_ERROR. Change-Id: I6418b9d2826146e1e4ceff4341c6d7cd3a0024af Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/dbscripts/storages_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDAOTest.java 5 files changed, 63 insertions(+), 24 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved Liron Ar: Verified -- To view, visit http://gerrit.ovirt.org/13709 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6418b9d2826146e1e4ceff4341c6d7cd3a0024af Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: initialize vds - select random host
by mkublin＠redhat.com 01 May '13

01 May '13

Michael Kublin has submitted this change and it was merged. Change subject: core: initialize vds - select random host ...................................................................... core: initialize vds - select random host when attempting to initialize vds, attempt to perform it through random host to not try always through the same one. Change-Id: If636850ba7d13b9a0be146059a9a964bb881abde Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBase.java 1 file changed, 9 insertions(+), 7 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved Liron Ar: Verified -- To view, visit http://gerrit.ovirt.org/13881 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If636850ba7d13b9a0be146059a9a964bb881abde Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>

1 0

Change in ovirt-engine[master]: findbugs: revert unnecessary Serializable
by mpastern＠redhat.com 01 May '13

01 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: findbugs: revert unnecessary Serializable ...................................................................... findbugs: revert unnecessary Serializable patch http://gerrit.ovirt.org/#/c/13993/ added Serializable to BackendAssignedPermissionsResource by mistake, this patch remove it. Change-Id: I659aa7f13b51a2c94dbf53f70704014a183ed0d7 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAssignedPermissionsResource.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14365 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I659aa7f13b51a2c94dbf53f70704014a183ed0d7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:ovirt-engine-backend: engine.log is flooded..
by yzaslavs＠redhat.com 01 May '13

01 May '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core:ovirt-engine-backend: engine.log is flooded.. ...................................................................... core:ovirt-engine-backend: engine.log is flooded.. ovirt-engine-backend: engine.log is flooded by Failed to decrypt Data Excluding the key store password (keystorePass) out of the list, so it will not be encrypted. Change-Id: Ife6005688a06d62a740970ea14b617010d6b9b2d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=880978 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/encryptionvalidator.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Eli Mesika: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14364 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife6005688a06d62a740970ea14b617010d6b9b2d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(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: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: prepare for the next development iteration
by mpastern＠redhat.com 01 May '13

01 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: prepare for the next development iteration ...................................................................... cli: prepare for the next development iteration Change-Id: Ia2748b9b417921a65e41268bbc0abc8c0274f507 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M setup.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14362 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2748b9b417921a65e41268bbc0abc8c0274f507 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>

1 0

Change in ovirt-engine-cli[master]: cli: release 3.3.0.2-1
by mpastern＠redhat.com 01 May '13

01 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: release 3.3.0.2-1 ...................................................................... cli: release 3.3.0.2-1 Change-Id: I15251afcf565d45da3ecb4bd696f6295cf5f5d68 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 16 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14361 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15251afcf565d45da3ecb4bd696f6295cf5f5d68 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>

1 0

Change in ovirt-engine-sdk[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 01 May '13

01 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I220b6df5c38d3272955fc4f61f2156e234c32e16 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M setup.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14360 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I220b6df5c38d3272955fc4f61f2156e234c32e16 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-sdk[master]: sdk: release 3.3.0.2-1
by mpastern＠redhat.com 01 May '13

01 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: release 3.3.0.2-1 ...................................................................... sdk: release 3.3.0.2-1 Change-Id: Ia63370843a58995d2a4fb1b0d4a5675c1ee7b2c9 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-sdk.spec.in M setup.py 3 files changed, 11 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14359 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia63370843a58995d2a4fb1b0d4a5675c1ee7b2c9 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-sdk[master]: sdk: regenerate against the latest api
by mpastern＠redhat.com 01 May '13

01 May '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 - in ClusterGlusterVolumeBricks fixed add() parameters - to StorageDomainVM.delete() added async parameter - fixed parent tag parameter for the tags.add()/.update() - nic.network is no longer mandatory parameter on vnic creation Change-Id: Ic9ea6a14aca1515a7eec55902ab6db85a21583e1 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/codegen/main.py M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 3 files changed, 7,412 insertions(+), 5,188 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14358 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic9ea6a14aca1515a7eec55902ab6db85a21583e1 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