September 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: engine: Adding 'DefaultMtu' to engine-config
by tnisan＠redhat.com 07 Sep '14

07 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: engine: Adding 'DefaultMtu' to engine-config ...................................................................... engine: Adding 'DefaultMtu' to engine-config Change-Id: Ie45576e4baa5f69ba3fd20ae9a0cad637b7e7b7d Bug-Url: https://bugzilla.redhat.com/1125175 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- A backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MtuValueHelper.java A backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/MtuValueHelperTest.java M packaging/etc/engine-config/engine-config.properties 3 files changed, 79 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30874 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie45576e4baa5f69ba3fd20ae9a0cad637b7e7b7d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: renaming 'DefaultMtu' config value to 'DefaultMTU'
by tnisan＠redhat.com 07 Sep '14

07 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: core: renaming 'DefaultMtu' config value to 'DefaultMTU' ...................................................................... core: renaming 'DefaultMtu' config value to 'DefaultMTU' Change-Id: Ic57d502a8894757cccbd87df3760ccac852a3ad9 Bug-Url: https://bugzilla.redhat.com/1125175 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelperTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkUtils.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NetworkUtilsTest.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SetupNetworksVDSCommandTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/ItemInfoPopup.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/renderer/MtuRenderer.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 10 files changed, 11 insertions(+), 10 deletions(-) Approvals: Alona Kaplan: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32559 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic57d502a8894757cccbd87df3760ccac852a3ad9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: engine: Add separate validation for multicast MAC address
by tnisan＠redhat.com 07 Sep '14

07 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: engine: Add separate validation for multicast MAC address ...................................................................... engine: Add separate validation for multicast MAC address Split MAC address format validation into two separate ones: 1. Valid MAC address 2. Unicast MAC address Change-Id: I4077a14fb61c2e082db9f81f1ded3b25ca98fc3f Bug-Url: https://bugzilla.redhat.com/1118385 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> (cherry picked from commit 526727ce47d578f958a6f58fdad461c9b55d4c10) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/VmNic.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/MacAddressValidationPatterns.java M backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/MacAddressPatternTest.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/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 8 files changed, 83 insertions(+), 32 deletions(-) Approvals: Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit http://gerrit.ovirt.org/32564 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4077a14fb61c2e082db9f81f1ded3b25ca98fc3f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: build: post ovirt-engine-dwh-3.5_rc2 respin
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: build: post ovirt-engine-dwh-3.5_rc2 respin ...................................................................... build: post ovirt-engine-dwh-3.5_rc2 respin Change-Id: I83c549bdeb7edd9bac6c838c8f63a32d457ab950 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32568 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I83c549bdeb7edd9bac6c838c8f63a32d457ab950 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: build: ovirt-engine-dwh-3.5_rc2 respin
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-dwh-3.5_rc2 respin ...................................................................... build: ovirt-engine-dwh-3.5_rc2 respin Change-Id: I65cd098c6441564a395d01b289e9d12eaf677aa8 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-dwh.spec.in M version.mak 2 files changed, 8 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32567 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65cd098c6441564a395d01b289e9d12eaf677aa8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: build: post ovirt-engine-reports-3.5_rc respin
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: build: post ovirt-engine-reports-3.5_rc respin ...................................................................... build: post ovirt-engine-reports-3.5_rc respin Change-Id: I51b4cd243647f4b032026d89f705dec3823fa04c Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32570 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51b4cd243647f4b032026d89f705dec3823fa04c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[master]: build: ovirt-engine-reports-3.5_rc2 respin
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-reports-3.5_rc2 respin ...................................................................... build: ovirt-engine-reports-3.5_rc2 respin Change-Id: Ia81d0d401fbf7efa7d97a37ec58b18d7495cecdc Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in M version.mak 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32561 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia81d0d401fbf7efa7d97a37ec58b18d7495cecdc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[master]: packaging: setup: Fix sso for separate host
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: Fix sso for separate host ...................................................................... packaging: setup: Fix sso for separate host Change-Id: I3e21448eedc78ce40018748719d23cd237928c51 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/reports/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/sso.py 2 files changed, 66 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29887 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e21448eedc78ce40018748719d23cd237928c51 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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-dwh[master]: history: updated etl to check valid installation
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: history: updated etl to check valid installation ...................................................................... history: updated etl to check valid installation I added a check to support seperate hosts. This check makes sure the current installation is the one that is configured on the engine side. If not it will fail the service. Change-Id: I4fdc850aed9bf1a7020eccfd5d26e547ca43f0bd Bug-Url: https://bugzilla.redhat.com/1118350 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/ovirt_engine_dwh/historyetl_3_5/contexts/Default.properties A etl_export/ovirt_engine_dwh/validateinstallation_3_5/contexts/Default.properties M etl_export/src/ovirt_engine_dwh/historyetl_3_5/HistoryETL.java A etl_export/src/ovirt_engine_dwh/validateinstallation_3_5/ValidateInstallation.java M packaging/conf/settings.properties.in M tos_project/context/connectionJDBC_3.5.item M tos_project/context/connectionJDBC_3.5.properties M tos_project/process/HistoryETL_3.5.item M tos_project/process/HistoryETL_3.5.properties A tos_project/process/ValidateInstallation_3.5.item A tos_project/process/ValidateInstallation_3.5.properties M tos_project/talend.project 12 files changed, 5,492 insertions(+), 595 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31871 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4fdc850aed9bf1a7020eccfd5d26e547ca43f0bd Gerrit-PatchSet: 15 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: packaging: setup: Prevent more than one dwh per engine
by ydary＠redhat.com 07 Sep '14

07 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: Prevent more than one dwh per engine ...................................................................... packaging: setup: Prevent more than one dwh per engine Change-Id: I0c0bbeef7fbd809895f4f0c1addd36e13f4894a6 Bug-Url: https://bugzilla.redhat.com/1118350 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dwh/constants.py A packaging/setup/ovirt_engine_setup/dwh/engine_db_timekeeping.py A packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/config/__init__.py A packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/config/dwh_uuid.py M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/__init__.py A packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/__init__.py A packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/__init__.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/core/single_etl.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py 11 files changed, 575 insertions(+), 33 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31325 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c0bbeef7fbd809895f4f0c1addd36e13f4894a6 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0