June 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6.7]: core: storage_domain_shared_status SQL calculation
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: storage_domain_shared_status SQL calculation ...................................................................... core: storage_domain_shared_status SQL calculation A storage domain's shared status was calculated by the fn_get_storage_domain_shared_status_by_domain_id function, which posed several performance issues, as it forced the switching between pgplsql and sql contexts several times, not to mention performing DML and even potentially DDL operations as part of the flow. Additionally, this function was phrased in a convoluted and hard to understand fashion, making it hard to maintain. The old logic, before this patch, was as follows: IF (domain.type != ISO) THEN IF (domain not attached to a pool) THEN RETURN Unattached ELSE IF (domain.status == Active) THEN RETURN Active ELSE RETURN Inactive ENDIF ELSE IF (num of distinct statuses == 0) THEN RETURN Unattached ELSE IF (num of distinct statuses == 1) THEN IF (domain.status == Active) THEN RETURN Active ELSE RETURN Inactive ENDIF ELSE IF (domain is active in at lest one pool) THEN RETURN Mixed ELSE RETURN Inactive END END END This logic can clearly be simplified. First and foremost, the special treatment for an ISO domain should be removed. Instead, a general treatment for a domain with multiple pools should be set in place, where ISO just happens to be the only type that supports this. The refactored logic should handle two variables: the number of distinct statuses a domain has and the number of active statuses it has: IF (num_distinct_statuses) == 0 THEN RETURN Unattached ELSE IF (num_active_statuses > 0) THEN IF (num_distinct_statuses > 1) THEN RETURN Mixed ELSE RETURN Active ELSE RETURN Inactive END This logic can easily be expressed as an SQL query which can be joined on and save the context switches associated with a pgplsql function. Bug-Url: https://bugzilla.redhat.com/1337257 Change-Id: I9c1695a41e7a9ca3fb32cbbbfc8fc8e4ae80f6ef Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDaoTest.java M packaging/dbscripts/create_dwh_views.sql M packaging/dbscripts/create_functions.sql M packaging/dbscripts/create_views.sql 4 files changed, 34 insertions(+), 95 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/58535 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c1695a41e7a9ca3fb32cbbbfc8fc8e4ae80f6ef Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.7 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.7]: core: Improve calculation of disk actual/committed size
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Improve calculation of disk actual/committed size ...................................................................... core: Improve calculation of disk actual/committed size The various views containing the disk actual and committed sizes on storage domains was done by functions which performed aggregate queries, per storage domain. By converting them to joins, the calculation can be done in parallel on all the domains, like databases are supposed to be used. This patch performs the following: 1. Creates the storage_domains_image_sizes view to encapsulate the logic previously performed by the fn_get_disk_commited_value_by_storage and fn_get_actual_images_size_by_storage stored functions. 2. Remove the aforementioned functions. 3. Add some assertions to the DAO test to ensure the functionality is preserved. Change-Id: I1c76870153d55c4a4259ecaf6e9b96297f16aa72 Bug-Url: https://bugzilla.redhat.com/1341661 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDaoTest.java M packaging/dbscripts/create_functions.sql M packaging/dbscripts/create_views.sql 3 files changed, 61 insertions(+), 67 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/58534 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c76870153d55c4a4259ecaf6e9b96297f16aa72 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.7 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.7]: core: StorageDomainDaoTest cleanups backport
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: StorageDomainDaoTest cleanups backport ...................................................................... core: StorageDomainDaoTest cleanups backport This patch backports a series of cleanups done to StorageDomainDaoTest in order to facilitate bugfixes in the following patches. It includes: - Fixing naming conventions of the class' methods - Removing a useless member - Using constants from the FixturesTool - Fixing the order the assertEquals parameters - Removing redundant whitespaces that hinder the code's readability - Removing some pointless assertions Note this patch should also have a [very slight] performance improvement, as it uses a pre-created Guids instead of invoking the constructor several times and removing some useless assertions. This patch contains the fixes introduced in patches 3158869..15d72f1 from the master branch. Related-To: https://bugzilla.redhat.com/1337257 Related-To: https://bugzilla.redhat.com/1341661 Change-Id: I8f25dfbd3530b555a0c2aa3ff992990274be6db8 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDaoTest.java 1 file changed, 42 insertions(+), 106 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/58533 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8f25dfbd3530b555a0c2aa3ff992990274be6db8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.7 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: move guest memory related fields to statistics
by ahadas＠redhat.com 02 Jun '16

02 Jun '16

Arik Hadas has submitted this change and it was merged. Change subject: core: move guest memory related fields to statistics ...................................................................... core: move guest memory related fields to statistics The information related to the memory within the guest used to be stored in VmDynamic. This is wrong. First, conceptually it is more 'statistics' then 'runtime' info. Second, it is updated frequenty and therefore leads to many updates of VM dynamic data by the VMs monitoring. This patch moves these fields to VmStatistics instead. Change-Id: If9944576a3283d77ad6c46c14f366efdf536b816 Bug-Url: https://bugzilla.redhat.com/1340722 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatistics.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDaoImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStatisticsDaoImpl.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java M packaging/dbscripts/create_dwh_views.sql M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/04_00_0760_move_guest_mem_fields_to_statistics.sql M packaging/dbscripts/vms_sp.sql 10 files changed, 117 insertions(+), 91 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/58305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9944576a3283d77ad6c46c14f366efdf536b816 Gerrit-PatchSet: 7 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: fix typos in WSP constants
by sbonazzo＠redhat.com 02 Jun '16

02 Jun '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix typos in WSP constants ...................................................................... packaging: setup: fix typos in WSP constants Fix typo in the names of OVESETUP_WSP_RPMDISTRO_PACKAGES and OVESETUP_WSP_RPMDISTRO_PACKAGES_SETUP Change-Id: I76fac1e9d6169edfcf117279a81e81eb3450135e Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1241869 --- M packaging/setup/ovirt_engine_setup/websocket_proxy/constants.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54867 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76fac1e9d6169edfcf117279a81e81eb3450135e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Improve calculation of disk actual/committed size
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Improve calculation of disk actual/committed size ...................................................................... core: Improve calculation of disk actual/committed size The various views containing the disk actual and committed sizes on storage domains was done by functions which performed aggregate queries, per storage domain. By converting them to joins, the calculation can be done in parallel on all the domains, like databases are supposed to be used. This patch performs the following: 1. Creates the storage_domains_image_sizes view to encapsulate the logic previously performed by the fn_get_disk_commited_value_by_storage and fn_get_actual_images_size_by_storage stored functions. 2. Remove the aforementioned functions. 3. Add some assertions to the DAO test to ensure the functionality is preserved. Change-Id: I1c76870153d55c4a4259ecaf6e9b96297f16aa72 Bug-Url: https://bugzilla.redhat.com/1341661 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDaoTest.java M packaging/dbscripts/create_functions.sql M packaging/dbscripts/create_views.sql 3 files changed, 61 insertions(+), 67 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/57684 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c76870153d55c4a4259ecaf6e9b96297f16aa72 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eldad Marciano <emarcian(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: storage_domain_shared_status SQL calculation
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: storage_domain_shared_status SQL calculation ...................................................................... core: storage_domain_shared_status SQL calculation A storage domain's shared status was calculated by the fn_get_storage_domain_shared_status_by_domain_id function, which posed several performance issues, as it forced the switching between pgplsql and sql contexts several times, not to mention performing DML and even potentially DDL operations as part of the flow. Additionally, this function was phrased in a convoluted and hard to understand fashion, making it hard to maintain. The old logic, before this patch, was as follows: IF (domain.type != ISO) THEN IF (domain not attached to a pool) THEN RETURN Unattached ELSE IF (domain.status == Active) THEN RETURN Active ELSE RETURN Inactive ENDIF ELSE IF (num of distinct statuses == 0) THEN RETURN Unattached ELSE IF (num of distinct statuses == 1) THEN IF (domain.status == Active) THEN RETURN Active ELSE RETURN Inactive ENDIF ELSE IF (domain is active in at lest one pool) THEN RETURN Mixed ELSE RETURN Inactive END END END This logic can clearly be simplified. First and foremost, the special treatment for an ISO domain should be removed. Instead, a general treatment for a domain with multiple pools should be set in place, where ISO just happens to be the only type that supports this. The refactored logic should handle two variables: the number of distinct statuses a domain has and the number of active statuses it has: IF (num_distinct_statuses) == 0 THEN RETURN Unattached ELSE IF (num_active_statuses > 0) THEN IF (num_distinct_statuses > 1) THEN RETURN Mixed ELSE RETURN Active ELSE RETURN Inactive END This logic can easily be expressed as an SQL query which can be joined on and save the context switches associated with a pgplsql function. Bug-Url: https://bugzilla.redhat.com/1337257 Change-Id: I9c1695a41e7a9ca3fb32cbbbfc8fc8e4ae80f6ef Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDaoTest.java M packaging/dbscripts/create_dwh_views.sql M packaging/dbscripts/create_functions.sql M packaging/dbscripts/create_views.sql 4 files changed, 34 insertions(+), 95 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58451 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c1695a41e7a9ca3fb32cbbbfc8fc8e4ae80f6ef Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eldad Marciano <emarcian(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Make it easier to debug master and 4 upgrade jobs
by eedri＠redhat.com 02 Jun '16

02 Jun '16

Eyal Edri has submitted this change and it was merged. Change subject: Make it easier to debug master and 4 upgrade jobs ...................................................................... Make it easier to debug master and 4 upgrade jobs Change-Id: I448fb6a534c9eef18e51e19c0ac6eeb7929dcfdf Signed-off-by: Barak Korren <bkorren(a)redhat.com> --- M jobs/ovirt-engine_upgrade_to_4.0/build_engine_rpm.sh M jobs/ovirt-engine_upgrade_to_4.0/cleanup.sh M jobs/ovirt-engine_upgrade_to_4.0/upgrade.sh M jobs/ovirt-engine_upgrade_to_master/build_engine_rpm.sh M jobs/ovirt-engine_upgrade_to_master/cleanup.sh M jobs/ovirt-engine_upgrade_to_master/upgrade.sh 6 files changed, 62 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me; Passed CI tests; Ready for merge Barak Korren: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/58216 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I448fb6a534c9eef18e51e19c0ac6eeb7929dcfdf Gerrit-PatchSet: 6 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pzhukov(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: build: ovirt-engine-4.0.0.2
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: build: ovirt-engine-4.0.0.2 ...................................................................... build: ovirt-engine-4.0.0.2 Change-Id: Ic243ee6d238c0b6a5c2a6e026314778b7ae16397 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M automation/build-artifacts.sh M backend/manager/dependencies/common/pom.xml M backend/manager/dependencies/pom.xml M backend/manager/dependencies/tools/pom.xml M backend/manager/extensions-tool/pom.xml M backend/manager/modules/aaa/pom.xml M backend/manager/modules/auth-plugin/pom.xml M backend/manager/modules/bll/pom.xml M backend/manager/modules/branding/pom.xml M backend/manager/modules/builtin-extensions/pom.xml M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/dal/pom.xml M backend/manager/modules/docs/pom.xml M backend/manager/modules/enginesso/pom.xml M backend/manager/modules/extensions-api-root/extensions-api/pom.xml M backend/manager/modules/extensions-api-root/pom.xml M backend/manager/modules/extensions-manager/pom.xml M backend/manager/modules/logger/pom.xml M backend/manager/modules/pom.xml M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml M backend/manager/modules/restapi/interface/common/pom.xml M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/restapi/interface/pom.xml M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/pom.xml M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/webapp/pom.xml M backend/manager/modules/root/pom.xml M backend/manager/modules/scheduler/pom.xml M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/services/pom.xml M backend/manager/modules/utils/pom.xml M backend/manager/modules/uutils/pom.xml M backend/manager/modules/vdsbroker/pom.xml M backend/manager/modules/welcome/pom.xml M backend/manager/pom.xml M backend/manager/tools/pom.xml M backend/pom.xml M build-tools-root/checkstyles/pom.xml M build-tools-root/ovirt-checkstyle-extension/pom.xml M build-tools-root/ovirt-findbugs-filters/pom.xml M build-tools-root/pom.xml M build/validations/pom.xml M ear/pom.xml M frontend/pom.xml M frontend/webadmin/modules/frontend-symbols/pom.xml M frontend/webadmin/modules/frontend/pom.xml M frontend/webadmin/modules/gwt-common/pom.xml M frontend/webadmin/modules/gwt-extension/pom.xml M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/uicommonweb/pom.xml M frontend/webadmin/modules/uicompat/pom.xml M frontend/webadmin/modules/userportal-gwtp/pom.xml M frontend/webadmin/modules/webadmin/pom.xml M frontend/webadmin/pom.xml M mavenmake/pom.xml M ovirt-engine.spec.in M pom.xml M version.mak 60 files changed, 66 insertions(+), 66 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58521 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic243ee6d238c0b6a5c2a6e026314778b7ae16397 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: build: post ovirt-engine-4.0.0.2
by tnisan＠redhat.com 02 Jun '16

02 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: build: post ovirt-engine-4.0.0.2 ...................................................................... build: post ovirt-engine-4.0.0.2 Change-Id: I5c7590f64f64a738052a8c904be62bcebdcb32aa Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M automation/build-artifacts.sh M backend/manager/dependencies/common/pom.xml M backend/manager/dependencies/pom.xml M backend/manager/dependencies/tools/pom.xml M backend/manager/extensions-tool/pom.xml M backend/manager/modules/aaa/pom.xml M backend/manager/modules/auth-plugin/pom.xml M backend/manager/modules/bll/pom.xml M backend/manager/modules/branding/pom.xml M backend/manager/modules/builtin-extensions/pom.xml M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/dal/pom.xml M backend/manager/modules/docs/pom.xml M backend/manager/modules/enginesso/pom.xml M backend/manager/modules/extensions-api-root/extensions-api/pom.xml M backend/manager/modules/extensions-api-root/pom.xml M backend/manager/modules/extensions-manager/pom.xml M backend/manager/modules/logger/pom.xml M backend/manager/modules/pom.xml M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml M backend/manager/modules/restapi/interface/common/pom.xml M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/restapi/interface/pom.xml M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/pom.xml M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/webapp/pom.xml M backend/manager/modules/root/pom.xml M backend/manager/modules/scheduler/pom.xml M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/services/pom.xml M backend/manager/modules/utils/pom.xml M backend/manager/modules/uutils/pom.xml M backend/manager/modules/vdsbroker/pom.xml M backend/manager/modules/welcome/pom.xml M backend/manager/pom.xml M backend/manager/tools/pom.xml M backend/pom.xml M build-tools-root/checkstyles/pom.xml M build-tools-root/ovirt-checkstyle-extension/pom.xml M build-tools-root/ovirt-findbugs-filters/pom.xml M build-tools-root/pom.xml M build/validations/pom.xml M ear/pom.xml M frontend/pom.xml M frontend/webadmin/modules/frontend-symbols/pom.xml M frontend/webadmin/modules/frontend/pom.xml M frontend/webadmin/modules/gwt-common/pom.xml M frontend/webadmin/modules/gwt-extension/pom.xml M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/uicommonweb/pom.xml M frontend/webadmin/modules/uicompat/pom.xml M frontend/webadmin/modules/userportal-gwtp/pom.xml M frontend/webadmin/modules/webadmin/pom.xml M frontend/webadmin/pom.xml M mavenmake/pom.xml M pom.xml M version.mak 59 files changed, 67 insertions(+), 64 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58524 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c7590f64f64a738052a8c904be62bcebdcb32aa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0