July 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine - changed error msg when host hostname can not be res...
by masayag＠redhat.com 13 Jul '15

13 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: engine - changed error msg when host hostname can not be resolved ...................................................................... engine - changed error msg when host hostname can not be resolved Changed the error message displayed in the audit log when the ip of a host registered using a hostname can not be resolved. Change-Id: I6c177f14a63799d0805698fb9bbef8f6a4ea3776 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1145678 Signed-off-by: mmirecki <mmirecki(a)redhat.com> Signed-off-by: Marcin Mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.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/VdsManager.java 3 files changed, 9 insertions(+), 1 deletion(-) Approvals: Marcin Mirecki: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43198 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c177f14a63799d0805698fb9bbef8f6a4ea3776 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(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]: core: moved GraphicsParameters to common
by tjelinek＠redhat.com 13 Jul '15

13 Jul '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: moved GraphicsParameters to common ...................................................................... core: moved GraphicsParameters to common Needed to be able to call from REST Change-Id: I35fb9fea787b3b0fea81036f3a5d65cddb19de0d Bug-Url: https://bugzilla.redhat.com/43079 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AbstractGraphicsDeviceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddGraphicsDeviceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveGraphicsDeviceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateGraphicsDeviceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java R backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/GraphicsParameters.java 9 files changed, 12 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43191 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35fb9fea787b3b0fea81036f3a5d65cddb19de0d Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: scheduling: Fix AREM tests
by rgolan＠redhat.com 13 Jul '15

13 Jul '15

Roy Golan has submitted this change and it was merged. Change subject: scheduling: Fix AREM tests ...................................................................... scheduling: Fix AREM tests This patch adds some more mocking to the AREM unit test to prevent any actual real world action to be triggered. Change-Id: I0735578ea76c4b4f050838ac102d9e7b87205518 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcementPerCluster.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/AffinityRulesEnforcementManagerTest.java 3 files changed, 42 insertions(+), 5 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43500 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0735578ea76c4b4f050838ac102d9e7b87205518 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: setup: Fix db removal issue
by sbonazzo＠redhat.com 13 Jul '15

13 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: setup: Fix db removal issue ...................................................................... setup: Fix db removal issue Fixes issue when database content was not removed if remove all engine components options was selected in engine-cleanup. Change-Id: Ib2312ba9e87f4d3304623ef9a73d10df500a62a6 Bug-Url: https://bugzilla.redhat.com/1241445 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine/constants.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/db/clear.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/system/engine.py 3 files changed, 7 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Verified Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/43351 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2312ba9e87f4d3304623ef9a73d10df500a62a6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: change VdcFault to EngineFault
by emesika＠redhat.com 13 Jul '15

13 Jul '15

Eli Mesika has submitted this change and it was merged. Change subject: core: change VdcFault to EngineFault ...................................................................... core: change VdcFault to EngineFault This patch is changes VdcFault to EngineFault in the command_entities table. Since the class change was done in a former patch, there can be a case in which engine will have data stored in his table and will try to deserialized it to the wrong class and crash Change-Id: I96d9f3646b65a80abad40e9eca79965db7d56aa0 Signed-off-by: emesika <emesika(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_1730_rename_vdcfault_to_enginefault.sql 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43468 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96d9f3646b65a80abad40e9eca79965db7d56aa0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[master]: packaging: setup: Use own jboss home
by ydary＠redhat.com 13 Jul '15

13 Jul '15

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: Use own jboss home ...................................................................... packaging: setup: Use own jboss home Use our own jboss home instead of the one configured by engine_common plugin. This allows running on a single machine a different version of jboss for the engine and for reports. This is needed since currently the engine moved to Wildfly and reports remains with jboss-as 7.1.1. Change-Id: Ia4ba71741ec7d821d1c6d3e89c878a8e5af642eb Bug-Url: https://bugzilla.redhat.com/1209022 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M Makefile M ovirt-engine-reports.spec.in M packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.conf.in M packaging/services/ovirt-engine-reportsd/ovirt-engine-reportsd.py M packaging/setup/ovirt_engine_setup/reports/config.py.in M packaging/setup/ovirt_engine_setup/reports/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/jboss.py 7 files changed, 58 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved; Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/42944 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4ba71741ec7d821d1c6d3e89c878a8e5af642eb Gerrit-PatchSet: 6 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: scheduling: Support for getting pending Host
by rgolan＠redhat.com 13 Jul '15

13 Jul '15

Roy Golan has submitted this change and it was merged. Change subject: scheduling: Support for getting pending Host ...................................................................... scheduling: Support for getting pending Host Making it possible to retrieve the current pending Host for a VM Change-Id: Id30c79e1dece8f4e78307000df0f5f8b01206104 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/pending/PendingResourceManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/pending/PendingVM.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/pending/PendingResourceManagerTest.java 4 files changed, 74 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Tomer Saban: Verified; Looks good to me, but someone else must approve Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43109 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id30c79e1dece8f4e78307000df0f5f8b01206104 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: add vmware providers
by ahadas＠redhat.com 13 Jul '15

13 Jul '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: add vmware providers ...................................................................... webadmin: add vmware providers Add the ability to add/edit/remove external providers of type VMware that provide VMs. Change-Id: Ief6e8fcff0ed8f5160aeaece8f5ed6a6c3b97725 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/EditProviderModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/VmwarePropertiesModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderPopupView.ui.xml A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/VmwarePropertiesWidget.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/VmwarePropertiesWidget.ui.xml 11 files changed, 366 insertions(+), 8 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/29380 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief6e8fcff0ed8f5160aeaece8f5ed6a6c3b97725 Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: Adopting ci standards
by sbonazzo＠redhat.com 13 Jul '15

13 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Adopting ci standards ...................................................................... Adopting ci standards Created only the build-artifacts script, with it's deps. For more info see: http://www.ovirt.org/index.php?title=CI/Build_and_test_standards Change-Id: I0d326176387886ac8d1e28f96a35b3ffcfd2f9f8 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A automation/README.md A automation/build-artifacts.packages A automation/build-artifacts.packages.el6 A automation/build-artifacts.repos A automation/build-artifacts.sh 5 files changed, 102 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests David Caro: Verified -- To view, visit https://gerrit.ovirt.org/43272 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0d326176387886ac8d1e28f96a35b3ffcfd2f9f8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[master]: sdk: Regenerate against the latest API
by juan.hernandez＠redhat.com 13 Jul '15

13 Jul '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate against the latest API ...................................................................... sdk: Regenerate against the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit 9eca55a. Change-Id: I964a958cf363181e9ccab1ceff70bb284ea91472 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/resources/api.xsd M generator/src/main/resources/rsdl.xml M sdk/Makefile M sdk/src/main/java/org/ovirt/engine/sdk/decorators/Host.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/HostHostDevice.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/HostHostDevices.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/InstanceTypeNIC.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageConnection.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageConnections.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomain.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/StorageDomains.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/TemplateNIC.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VM.java M sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMCdRom.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMHostDevice.java A sdk/src/main/java/org/ovirt/engine/sdk/decorators/VMHostDevices.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Action.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/BaseDevice.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/BaseResource.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/BaseResources.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/Host.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/HostDevice.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/HostDevicePassthrough.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/HostDevices.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/ObjectFactory.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/Product.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/StorageDomain.java A sdk/src/main/java/org/ovirt/engine/sdk/entities/Vendor.java M sdk/src/main/java/org/ovirt/engine/sdk/entities/VmPlacementPolicy.java 29 files changed, 2,062 insertions(+), 484 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43471 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I964a958cf363181e9ccab1ceff70bb284ea91472 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0