June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Deprecate SetupNetworks VDS related classes
by alkaplan＠redhat.com 22 Jun '15

22 Jun '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Deprecate SetupNetworks VDS related classes ...................................................................... engine: Deprecate SetupNetworks VDS related classes Since SetupNetworks is replaced by HostSetupNetworks, the former should be deprecated. Change-Id: I46c0261162bd5b91e2d1cb9b41e963ba18729db3 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/FutureVDSCommandType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/SetupNetworksVdsCommandParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SetupNetworksVDSCommand.java 3 files changed, 10 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/33325 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46c0261162bd5b91e2d1cb9b41e963ba18729db3 Gerrit-PatchSet: 34 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Re-define HostSetupNetworks based on network attachm...
by alkaplan＠redhat.com 22 Jun '15

22 Jun '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Re-define HostSetupNetworks based on network attachments ...................................................................... engine: Re-define HostSetupNetworks based on network attachments The same old api is being redefined by network attachment and by externalizing the business logic out from the VDS command into the producer of the parameters (will be introduced in a latter patch). Change-Id: I75d02e4be71f00510bc8815c555a94d7efc9252e Signed-off-by: Moti Asayag <masayag(a)redhat.com> Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/FutureVDSCommandType.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/HostNetwork.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/HostSetupNetworksVdsCommandParameters.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HostSetupNetworksVDSCommand.java 4 files changed, 382 insertions(+), 1 deletion(-) Approvals: Martin Mucha: Verified Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/32910 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I75d02e4be71f00510bc8815c555a94d7efc9252e Gerrit-PatchSet: 38 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Changed log messages in AREM
by ofrenkel＠redhat.com 22 Jun '15

22 Jun '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Changed log messages in AREM ...................................................................... core: Changed log messages in AREM Several log messages were changed to debug in Affinity Rules Enforcement Manager in order not to make to much noise on the logs. Change-Id: I6e0584fff2e1ef8999449ef696f8d141a4b98c5a Signed-off-by: Tomer Saban <tsaban(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 2 files changed, 9 insertions(+), 10 deletions(-) Approvals: Jenkins CI: Passed CI tests Tomer Saban: Verified Roy Golan: Looks good to me, approved Omer Frenkel: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/42607 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e0584fff2e1ef8999449ef696f8d141a4b98c5a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(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]: gluster: Dont list striped volumes types while volume creation
by sabose＠redhat.com 22 Jun '15

22 Jun '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Dont list striped volumes types while volume creation ...................................................................... gluster: Dont list striped volumes types while volume creation Change-Id: Ib493645662f493a271635613264c8dff1e33c339 Bug-Url: https://bugzilla.redhat.com/1233621 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42665 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib493645662f493a271635613264c8dff1e33c339 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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]: webadmin, userportal: translations update (Korean only)
by gshereme＠redhat.com 22 Jun '15

22 Jun '15

Greg Sheremeta has submitted this change and it was merged. Change subject: webadmin, userportal: translations update (Korean only) ...................................................................... webadmin, userportal: translations update (Korean only) pulled translation from Zanata oVirt project, master version (Korean only) Change-Id: I8c579f5a63b8f347e3d27135f6eff677e0a0f673 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1234005 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- A backend/manager/modules/docs/src/main/resources/messages_ko_KR.properties A backend/manager/modules/welcome/src/main/resources/messages_ko_KR.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_ko_KR.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums_ko_KR.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants_ko_KR.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationMessages_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationMessages_ko_KR.properties A packaging/branding/ovirt.brand/messages_ko_KR.properties 17 files changed, 3,827 insertions(+), 1,275 deletions(-) Approvals: Jenkins CI: Passed CI tests Einav Cohen: Verified Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42590 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c579f5a63b8f347e3d27135f6eff677e0a0f673 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: add snapshot mount point to filter list
by emesika＠redhat.com 22 Jun '15

22 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: engine: add snapshot mount point to filter list ...................................................................... engine: add snapshot mount point to filter list Add '/var/run/gluster/snaps/' to the list of mount points ignored for storage devices. lvm snapshots will be mouted at /var/run/gluster/snaps in older versions of Fedora and RHEL-6. With recent version of Fedora and RHEL-7 it will be mounted at /run/gluster/snaps/. Change-Id: If9dabf1966a82d54a3ae5cddfb0c7f5df6b09429 Bug-Url: https://bugzilla.redhat.com/1227885 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved; Passed CI tests Shubhendu Tripathi: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, but someone else must approve Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/42656 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9dabf1966a82d54a3ae5cddfb0c7f5df6b09429 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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]: tools: crypto-tool: initial add
by masayag＠redhat.com 22 Jun '15

22 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: tools: crypto-tool: initial add ...................................................................... tools: crypto-tool: initial add Change-Id: I54dfbc65a06d5c3e6aade916eae2831bbf8231c0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- A backend/manager/tools/src/main/java/org/ovirt/engine/core/cryptotool/Main.java A backend/manager/tools/src/main/resources/org/ovirt/engine/core/cryptotool/arguments.properties M ovirt-engine.spec.in A packaging/bin/ovirt-engine-crypto-tool.sh 4 files changed, 435 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42564 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54dfbc65a06d5c3e6aade916eae2831bbf8231c0 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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]: packaging: s/usage:/Usage:/
by sbonazzo＠redhat.com 22 Jun '15

22 Jun '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: s/usage:/Usage:/ ...................................................................... packaging: s/usage:/Usage:/ Change-Id: I8a6f5c92061dd415262d72c70d38e2bac593be9c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/core/ExtensionsToolExecutor.java M backend/manager/extension-tool/src/main/java/org/ovirt/engine/exttool/logger/LoggerServiceImpl.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomainsExecutor.java M backend/manager/modules/builtin-extensions/src/main/resources/manage-domains-help.properties M packaging/man/man8/engine-manage-domains.8 M packaging/pythonlib/ovirt_engine/service.py M packaging/setup/bin/ovirt-engine-upgrade-check 7 files changed, 8 insertions(+), 8 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42663 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a6f5c92061dd415262d72c70d38e2bac593be9c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Fix broken DAO test
by oourfali＠redhat.com 22 Jun '15

22 Jun '15

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Fix broken DAO test ...................................................................... engine: Fix broken DAO test Change-Id: I0bd5528127540fb3f9559a01be3199f3fd09686f Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/dal/src/test/resources/fixtures.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Moti Asayag: Verified Oved Ourfali: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42666 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0bd5528127540fb3f9559a01be3199f3fd09686f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(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]: webadmin: reorganize sub-tabs under VMs main-tab
by derez＠redhat.com 22 Jun '15

22 Jun '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: reorganize sub-tabs under VMs main-tab ...................................................................... webadmin: reorganize sub-tabs under VMs main-tab Organize the sub-tabs under VMs main-tab to keep consistency and avoid conflict in order. Change-Id: I0dba1f12c69ad45e7b0b18f84cc30e775ff39504 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineApplicationPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineEventPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineGeneralPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineGuestInfoPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineHostDevicePresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineNetworkInterfacePresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachinePermissionPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineSnapshotPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/virtualMachine/SubTabVirtualMachineVirtualDiskPresenter.java 9 files changed, 9 insertions(+), 9 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42599 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0dba1f12c69ad45e7b0b18f84cc30e775ff39504 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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