February 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.1]: dashboards: update dwh views version
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: update dwh views version ...................................................................... dashboards: update dwh views version Change-Id: I2d845ac01c6788aff5e1aaf6d60d83e5504b95a1 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/ClusterDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/StorageDomainDwhDAO.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/VmDwhDAO.properties 4 files changed, 52 insertions(+), 52 deletions(-) Approvals: Tal Nisan: Passed CI tests Shirly Radco: Looks good to me, but someone else must approve Alexander Wels: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/72239 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d845ac01c6788aff5e1aaf6d60d83e5504b95a1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: dashboards: Fixed storage domain dashboard queries
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed storage domain dashboard queries ...................................................................... dashboards: Fixed storage domain dashboard queries Fixed storage domain dasboard queries. storage.last24hours_average, storage.hourly_history, storage.last5_minutes_average Change-Id: I2b8e69209e3c782ed7fe6a05abdaf84f550cdf18 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> (cherry picked from commit f5dbf6d71a8e7aa9cfb237ec634b75856009f42b) --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/StorageDomainDwhDAO.properties 1 file changed, 30 insertions(+), 76 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72245 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b8e69209e3c782ed7fe6a05abdaf84f550cdf18 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed last_5_min_cpu_mem_average dashboard query
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed last_5_min_cpu_mem_average dashboard query ...................................................................... dashboards: Fixed last_5_min_cpu_mem_average dashboard query Change-Id: I42af9ae012435071a8957cf6dfb7fca60bb630e5 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties 1 file changed, 20 insertions(+), 17 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72248 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I42af9ae012435071a8957cf6dfb7fca60bb630e5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed host dashboard queries
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed host dashboard queries ...................................................................... dashboards: Fixed host dashboard queries Fixed host.cpu_host_utilization and host.mem_host_utilization dashboard queries. Change-Id: I4102104188fbd089366f4d4e69f92090a4957e46 Bug-Url: https://bugzilla.redhat.com/1401010 Signed-off-by: Shirly Radco <sradco(a)redhat.com> (cherry picked from commit 4697568a76c66467f895650e9b56f3859d9fb047) --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties 1 file changed, 78 insertions(+), 105 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72261 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4102104188fbd089366f4d4e69f92090a4957e46 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed vms utilization dashboard queries
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed vms utilization dashboard queries ...................................................................... dashboards: Fixed vms utilization dashboard queries The queries incluede max, which slowed down the queries. I updtaed the queries so that they will not use it. Change-Id: I8908e9144629e73f59588960b0af91711a2e3746 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> (cherry picked from commit a2777729d5b530f6228ee915f494adfd9a935f42) --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/VmDwhDAO.properties 1 file changed, 60 insertions(+), 70 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72246 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8908e9144629e73f59588960b0af91711a2e3746 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed host.total_cpu_memory_count dashboard query
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed host.total_cpu_memory_count dashboard query ...................................................................... dashboards: Fixed host.total_cpu_memory_count dashboard query Change-Id: I560f8b7fdb71e05162b1b472707cf7dfa7b77d2c Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties 1 file changed, 3 insertions(+), 12 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72247 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I560f8b7fdb71e05162b1b472707cf7dfa7b77d2c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed host.hourly_cpu_mem_history dashboard query
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed host.hourly_cpu_mem_history dashboard query ...................................................................... dashboards: Fixed host.hourly_cpu_mem_history dashboard query Change-Id: I3184f4d4860af8f73544ce836195ec0242ab75f2 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/HostDwhDAO.properties 1 file changed, 57 insertions(+), 59 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3184f4d4860af8f73544ce836195ec0242ab75f2 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.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-4.1]: dashboards: Fixed cluster.last24hours dashboard query
by Code Review 16 Feb '17

16 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: dashboards: Fixed cluster.last24hours dashboard query ...................................................................... dashboards: Fixed cluster.last24hours dashboard query Change-Id: I91595689d576227b4b6216ed0864731c9cacb576 Bug-Url: https://bugzilla.redhat.com/1421942 Signed-off-by: Shirly Radco <sradco(a)redhat.com> (cherry picked from commit 7b7048bf580304db5a5c0a880ad5377aedc18e7d) --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/ClusterDwhDAO.properties 1 file changed, 28 insertions(+), 16 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72244 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91595689d576227b4b6216ed0864731c9cacb576 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.1]: Don't fork the Maven source plugin
by Code Review 15 Feb '17

15 Feb '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Don't fork the Maven source plugin ...................................................................... Don't fork the Maven source plugin Currently the project runs the 'jar' goal of the Maven source plugin.u A side effect of this is that the plugin explicitly executes the 'generate-sources' phase of the build. As a result the code generation, which is attached to that 'generate-sources' phase, runs twice. That doesn't have a big impact, as the generated code is the same, but it wastes resources. This patch changes the build process so that it uses the 'jar-no-fork', which doesn't have that negative side effect. Change-Id: Ia8a7b8859caec444350460c90abfcd461dd443dc Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72387 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8a7b8859caec444350460c90abfcd461dd443dc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.1]: Unify automation '*.packages' files
by Code Review 15 Feb '17

15 Feb '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Unify automation '*.packages' files ...................................................................... Unify automation '*.packages' files Currentl there is one 'build.packages' file, and then multiple symbolic links, one for each distribution, except Fedora 25. Because of this the build in Fedora 25 fails. As all the currently supported distributions require the same packages, there is no need for multiple files, so this patch replaces the distribution specific files with distribution independent files. Change-Id: I08fc74a41ef09f9896f0636fd9cd4abf66da718c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit e8e7aa6bdbc49b413d8de0fdba1e88d24e4795e6) --- R automation/build-artifacts.packages D automation/build-artifacts.packages.fc24 C automation/check-patch.packages D automation/check-patch.packages.el7 D automation/check-patch.packages.fc24 5 files changed, 0 insertions(+), 3 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I08fc74a41ef09f9896f0636fd9cd4abf66da718c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0