May 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: Dashboard: no hosts
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: Dashboard: no hosts ...................................................................... Dashboard: no hosts - When the system doesn't have any hosts/storage domains defined yet, the donuts were showing invalid values instead of 0, this patch fixes the data generated by the servlet to return 0 when no hosts or storage domains are available. Change-Id: Ic925633eabdd3875a144bda05523fa4990028c7d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1337606 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/dashboard/GlobalUtilizationCpuSummary.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58097 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic925633eabdd3875a144bda05523fa4990028c7d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Add tests to OvfManagerTest
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add tests to OvfManagerTest ...................................................................... core: Add tests to OvfManagerTest Change-Id: Ie47ce56efa5fd18b22fd702f5fa3c093e0068f9f Bug-Url: https://bugzilla.redhat.com/1339658 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfManagerTest.java 1 file changed, 147 insertions(+), 31 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58073 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie47ce56efa5fd18b22fd702f5fa3c093e0068f9f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(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-4.0]: core: Add a method to generate random MAC addresses to Rando...
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add a method to generate random MAC addresses to RandomUtils ...................................................................... core: Add a method to generate random MAC addresses to RandomUtils Change-Id: I7b9947588698cdb7375c539e2f969fec67e32ae8 Bug-Url: https://bugzilla.redhat.com/1339658 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/RandomUtils.java 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58072 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b9947588698cdb7375c539e2f969fec67e32ae8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(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-dwh[master]: history: fixed a bug when removing dc statistics
by Shirly Radco 25 May '16

25 May '16

Shirly Radco has submitted this change and it was merged. Change subject: history: fixed a bug when removing dc statistics ...................................................................... history: fixed a bug when removing dc statistics removed datacenter_samples_history from the DeleteTimeKeepingJob. Change-Id: Ibbd6151c0b9f0e371c982702aca51373e689a6bb Bug-Url: https://bugzilla.redhat.com/1339597 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/src/main/java/ovirt_engine_dwh/deletetimekeepingjob_4_0/DeleteTimeKeepingJob.java M tos_project/process/DeleteTimeKeepingJob_4.0.item M tos_project/process/DeleteTimeKeepingJob_4.0.properties 3 files changed, 12 insertions(+), 708 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58042 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibbd6151c0b9f0e371c982702aca51373e689a6bb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Add a method to generate random MAC addresses to Rando...
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add a method to generate random MAC addresses to RandomUtils ...................................................................... core: Add a method to generate random MAC addresses to RandomUtils Change-Id: I7b9947588698cdb7375c539e2f969fec67e32ae8 Bug-Url: https://bugzilla.redhat.com/1339658 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/RandomUtils.java 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58062 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b9947588698cdb7375c539e2f969fec67e32ae8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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[master]: core: Add tests to OvfManagerTest
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add tests to OvfManagerTest ...................................................................... core: Add tests to OvfManagerTest Change-Id: Ie47ce56efa5fd18b22fd702f5fa3c093e0068f9f Bug-Url: https://bugzilla.redhat.com/1339658 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfManagerTest.java 1 file changed, 147 insertions(+), 31 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/57851 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie47ce56efa5fd18b22fd702f5fa3c093e0068f9f Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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-4.0]: webadmin: Dashboard storage calculation
by tnisan＠redhat.com 25 May '16

25 May '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Dashboard storage calculation ...................................................................... webadmin: Dashboard storage calculation - The dashboard storage queries where not taking the status of the storage into account. So it would count detached storage domains in the calculations. This patch makes sure that the detached storage domains are not counted. Change-Id: Ifdfb68afd0f8aa1dd06a15de8c469b4cae5363e8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1336896 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/StorageDomainDwhDAO.properties 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58095 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdfb68afd0f8aa1dd06a15de8c469b4cae5363e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: Dashboard: no hosts
by awels＠redhat.com 25 May '16

25 May '16

Alexander Wels has submitted this change and it was merged. Change subject: Dashboard: no hosts ...................................................................... Dashboard: no hosts - When the system doesn't have any hosts/storage domains defined yet, the donuts were showing invalid values instead of 0, this patch fixes the data generated by the servlet to return 0 when no hosts or storage domains are available. Change-Id: Ic925633eabdd3875a144bda05523fa4990028c7d Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1337606 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/dashboard/GlobalUtilizationCpuSummary.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58093 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic925633eabdd3875a144bda05523fa4990028c7d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Dashboard storage calculation
by awels＠redhat.com 25 May '16

25 May '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Dashboard storage calculation ...................................................................... webadmin: Dashboard storage calculation - The dashboard storage queries where not taking the status of the storage into account. So it would count detached storage domains in the calculations. This patch makes sure that the detached storage domains are not counted. Change-Id: Ifdfb68afd0f8aa1dd06a15de8c469b4cae5363e8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1336896 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/server/dashboard/dao/StorageDomainDwhDAO.properties 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57974 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdfb68afd0f8aa1dd06a15de8c469b4cae5363e8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in gerrit-admin[master]: adding vdsm & engine config for gerrit hooks
by eedri＠redhat.com 25 May '16

25 May '16

Eyal Edri has submitted this change and it was merged. Change subject: adding vdsm & engine config for gerrit hooks ...................................................................... adding vdsm & engine config for gerrit hooks - This is not auto-deployed to Gerrit! - For now its a backup for the config files which are on the gerrit server and should be used for tracking changes until we'll move it to a proper repo we can use to deploy safely to gerrit Change-Id: Id784c5c3851c0bb83d157e5c1b5b86c38d30aa8e Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- A hooks/config/README A hooks/config/ovirt-engine.config A hooks/config/vdsm.config 3 files changed, 37 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me, approved Jenkins CI: Passed CI tests David Caro: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id784c5c3851c0bb83d157e5c1b5b86c38d30aa8e Gerrit-PatchSet: 2 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0