January 2015 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: post otopi-1.3.1
by Alon Bar-Lev 15 Jan '15

15 Jan '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.3.1 ...................................................................... packaging: post otopi-1.3.1 Change-Id: I1ac4f72d55c62018df478cf79eab1f33e7e03afd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36960 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ac4f72d55c62018df478cf79eab1f33e7e03afd Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-host-deploy[master]: packaging: ovirt-host-deploy-1.3.1
by Alon Bar-Lev 15 Jan '15

15 Jan '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: ovirt-host-deploy-1.3.1 ...................................................................... packaging: ovirt-host-deploy-1.3.1 Change-Id: I9b3b37c11e814a74ab901b2e2e96b3016fa743d5 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M ovirt-host-deploy-offline.spec.in M ovirt-host-deploy.spec.in 4 files changed, 9 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36957 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b3b37c11e814a74ab901b2e2e96b3016fa743d5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-host-deploy[master]: layout: update .gitignore
by Alon Bar-Lev 15 Jan '15

15 Jan '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: layout: update .gitignore ...................................................................... layout: update .gitignore Change-Id: I7e5eb17ccc8ee336311294c34dd510259f42225e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36956 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e5eb17ccc8ee336311294c34dd510259f42225e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-host-deploy[master]: packaging: post ovirt-host-deploy-1.3.1
by Alon Bar-Lev 15 Jan '15

15 Jan '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post ovirt-host-deploy-1.3.1 ...................................................................... packaging: post ovirt-host-deploy-1.3.1 Change-Id: I947072c0d21889168ace9e436c9fd5aca57d750c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36958 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I947072c0d21889168ace9e436c9fd5aca57d750c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Temporary fix to avoid filling up slaves
by dcaroest＠redhat.com 15 Jan '15

15 Jan '15

David Caro has submitted this change and it was merged. Change subject: Temporary fix to avoid filling up slaves ...................................................................... Temporary fix to avoid filling up slaves It should be removed as soon as the makefiles are properly fixed to use only the job's workspace Change-Id: Ib6b3e06fa38873cf0a79f0128b72a773d9f75655 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/ovirt_node_cleanup.sh 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: David Caro: Verified; Looks good to me, approved Anatoly Litovsky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/36941 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6b3e06fa38873cf0a79f0128b72a773d9f75655 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Added node el7 build
by dcaroest＠redhat.com 15 Jan '15

15 Jan '15

David Caro has submitted this change and it was merged. Change subject: Added node el7 build ...................................................................... Added node el7 build Requires to have merged the patch: http://gerrit.ovirt.org/#/c/36943 Change-Id: I41b7d67d2ce64ac960b4f7a80be41428c96a7daa Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36945 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41b7d67d2ce64ac960b4f7a80be41428c96a7daa Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Storage QoS popup fixups
by gchaplik＠redhat.com 15 Jan '15

15 Jan '15

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Storage QoS popup fixups ...................................................................... webadmin: Storage QoS popup fixups The storage qos popup is now more intutive and explains better how to use the ui(Total and read/write can't be put together). Change-Id: Ia294f95e13440456798600c34eca392c0213d3e7 Bug-Url: https://bugzilla.redhat.com/1168558 Signed-off-by: Tomer Saban <tsaban(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/qos/StorageQosMetricParametersModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/qos/StorageQosParametersModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/qos/StorageQosWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/qos/StorageQosWidget.ui.xml 5 files changed, 193 insertions(+), 93 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36115 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia294f95e13440456798600c34eca392c0213d3e7 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> 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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: Prompt if host non-operational
by sbonazzo＠redhat.com 15 Jan '15

15 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Prompt if host non-operational ...................................................................... packaging: setup: Prompt if host non-operational If user set up required networks but did not press 'activate', host will remain non-operational. Prompt if non-operational and suggest to activate. This patch assumes that "non-operational" should not be possible as a state the host can be in during deployment. No harm in prompting anyway. Change-Id: I481cb082adef25238bc06280e8d274b6d4cd8c9e Related-To: https://bugzilla.redhat.com/1178535 Bug-Url: https://bugzilla.redhat.com/1086032 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit fa9ac121c4bc0abca1d401f2eaa46a1e2cbbb816) --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 2 files changed, 50 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36926 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I481cb082adef25238bc06280e8d274b6d4cd8c9e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Fixed error message in new Storage QoS.
by gchaplik＠redhat.com 15 Jan '15

15 Jan '15

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Fixed error message in new Storage QoS. ...................................................................... webadmin: Fixed error message in new Storage QoS. When giving some natural number value to each textbox in the new Stoage QoS, wrong error message appears that says missing values. Now the error message says that a non-zero values was entered for total along with a non-zero value for read/write (which is invalid). Change-Id: I85412ffc86be8f903b9a3f07fbc3a10641be8e71 Bug-Url: https://bugzilla.redhat.com/1163805 Signed-off-by: Tomer Saban <tsaban(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageQosValidator.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, 7 insertions(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35601 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85412ffc86be8f903b9a3f07fbc3a10641be8e71 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> 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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Added range values to QoS error message.
by gchaplik＠redhat.com 15 Jan '15

15 Jan '15

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Added range values to QoS error message. ...................................................................... webadmin: Added range values to QoS error message. On the Data Center tab, When adding a QoS to the cpu with limit which is out of range. The error message doesn't state what is the legal limit range. Now the range is added to the error message(The range is 1-100). Also, changed the name of the variable: ACTION_TYPE_FAILED_QOS_OUT_OF_RANGE_VALUES -> ACTION_TYPE_FAILED_QOS_CPU_OUT_OF_RANGE_VALUES A new variable for the "out of range" error was added for the storage which is named: ACTION_TYPE_FAILED_QOS_STORAGE_OUT_OF_RANGE_VALUES Change-Id: Ia0e1b2ef97624b6b0f3c9c95e7a09e15561a458b Bug-Url: https://bugzilla.redhat.com/1142333 Signed-off-by: Tomer Saban <tsaban(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/qos/CpuQos.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/qos/StorageQos.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 6 files changed, 20 insertions(+), 10 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35093 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0e1b2ef97624b6b0f3c9c95e7a09e15561a458b Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> 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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0