May 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: CDI Transaction Interceptor
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: core: CDI Transaction Interceptor ...................................................................... core: CDI Transaction Interceptor Allow developers to mark CDI beans and their methods as transactional, and make sure that methods will run in a transactional context Change-Id: I94bf3b41764b137e5456995d393f023d3564e301 Bug-Url: https://bugzilla.redhat.com/1159621 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/ModuleConfigurationExtension.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/transaction/Transactional.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/transaction/TransactionalInterceptor.java M backend/manager/modules/utils/src/main/modules/org/ovirt/engine/core/utils/main/module.xml 4 files changed, 99 insertions(+), 47 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Liran Zelkha: Verified -- To view, visit https://gerrit.ovirt.org/39790 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94bf3b41764b137e5456995d393f023d3564e301 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Added canDoAction message for gluster volume deletion
by kmayilsa＠redhat.com 13 May '15

13 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Added canDoAction message for gluster volume deletion ...................................................................... gluster: Added canDoAction message for gluster volume deletion Added a canDoAction check to not to allow gluster volume deletion if it has snapshots associated with it. Change-Id: I80f0bd8719cc90ff2f8ee478d830c1a1a67e4cdf Bug-Url: https://bugzilla.redhat.com/1215876 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGlusterVolumeCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 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 5 files changed, 14 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved Sahina Bose: Looks good to me, but someone else must approve Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40414 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80f0bd8719cc90ff2f8ee478d830c1a1a67e4cdf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Save VM custom properties in OVF
by ofrenkel＠redhat.com 13 May '15

13 May '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Save VM custom properties in OVF ...................................................................... core: Save VM custom properties in OVF Added new tags and code to save/restore VM custom properties in OVF. This makes possible editing VM custom properties when VM is running. Change-Id: I985e2b5153f1531f98d167c97ff69c70cb9c4196 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1218973 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfProperties.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java 3 files changed, 28 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Looks good to me, but someone else must approve Shmuel Leib Melamud: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40829 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I985e2b5153f1531f98d167c97ff69c70cb9c4196 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Corrected warning message
by kmayilsa＠redhat.com 13 May '15

13 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: Corrected warning message ...................................................................... webadmin: Corrected warning message Corrected the warning message shown while updation of gluster volume snapshot options. Change-Id: I4576afa6bae06e268f507048ba0a6ab91f1e4f35 Bug-Url: https://bugzilla.redhat.com/1215928 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40808 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4576afa6bae06e268f507048ba0a6ab91f1e4f35 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Report updates availability for host
by masayag＠redhat.com 13 May '15

13 May '15

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Report updates availability for host ...................................................................... restapi: Report updates availability for host A new property is exposed to indicate if a new update is available for a specific host. A host update stands for a newer VDSM package, or any other package listed by the user in configuration value 'UserPackageNamesForCheckUpdate'. <host> ... <update_available>true</update_available> </host> Feature-page: http://www.ovirt.org/index.php?title=Home/Features/UpgradeManager Change-Id: I8fdab13d0195425212d135840da2cf37ae65e9b4 Bug-Url: https://bugzilla.redhat.com/1157368 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40461 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8fdab13d0195425212d135840da2cf37ae65e9b4 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Schedule a job for monitoring host's updates
by oourfali＠redhat.com 13 May '15

13 May '15

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Schedule a job for monitoring host's updates ...................................................................... engine: Schedule a job for monitoring host's updates The job will query the host for available updates for the specified packages. The monitored packages are the same as required for the host upgrade process. Change-Id: I2114d00e626c09cd7b8be1f9a4564a0f3c0ba700 Bug-Url: https://bugzilla.redhat.com/1157368 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/AvailableUpdatesFinder.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/HostUpgradeManager.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/OvirtNodeUpgradeManager.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/UpdateAvailable.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/Config.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 14 files changed, 253 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40460 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2114d00e626c09cd7b8be1f9a4564a0f3c0ba700 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Long response time when using list API
by juan.hernandez＠redhat.com 13 May '15

13 May '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Long response time when using list API ...................................................................... restapi: Long response time when using list API Cache some reflection work so that list API serialization will be faster. According to the profiler, this should save ~33% of serialization time. Change-Id: I1801e5589d8cec1474e32e495b35b6c150e101ca Bug-Url: https://bugzilla.redhat.com/1216023 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/LinkHelper.java 1 file changed, 44 insertions(+), 20 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve Liran Zelkha: Verified -- To view, visit https://gerrit.ovirt.org/40809 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1801e5589d8cec1474e32e495b35b6c150e101ca Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix typo for rebooting a VM
by amureini＠redhat.com 13 May '15

13 May '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix typo for rebooting a VM ...................................................................... core: Fix typo for rebooting a VM Fixed typo: Reebot -> Reboot Change-Id: Ibd85439417ad87940c38dca4d25101a248ecad7c Bug-Url: https://bugzilla.redhat.com/1221058 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40857 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd85439417ad87940c38dca4d25101a248ecad7c Gerrit-PatchSet: 2 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix typo in Network cluster messages
by amureini＠redhat.com 13 May '15

13 May '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix typo in Network cluster messages ...................................................................... core: Fix typo in Network cluster messages Fixed grammar mistake "more then" to "more than". Change-Id: I9791b66a9b57e519f42aa80ff76c98f5ddb76515 Bug-Url: https://bugzilla.redhat.com/1221055 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/job/ExecutionMessageDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 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 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/40855 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9791b66a9b57e519f42aa80ff76c98f5ddb76515 Gerrit-PatchSet: 2 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: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix typos in Gluster geo-replication
by sabose＠redhat.com 13 May '15

13 May '15

Sahina Bose has submitted this change and it was merged. Change subject: core: Fix typos in Gluster geo-replication ...................................................................... core: Fix typos in Gluster geo-replication Fixed typos: - brocker -> broker - inetrevene -> intervene Change-Id: I426d42c9c24e106aa482768d8e2a187ee5d4ae06 Bug-Url: https://bugzilla.redhat.com/1221052 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties 4 files changed, 6 insertions(+), 6 deletions(-) Approvals: Allon Mureinik: Verified Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40856 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I426d42c9c24e106aa482768d8e2a187ee5d4ae06 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0