Change in ovirt-engine[ovirt-engine-4.0]: Dashboard: no hosts
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[ovirt-engine-4.0]: core: Add tests to OvfManagerTest
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[ovirt-engine-4.0]: core: Add a method to generate random MAC addresses to Rando...
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-dwh[master]: history: fixed a bug when removing dc statistics
by Shirly Radco
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>
7 years, 11 months
Change in ovirt-engine[master]: core: Add a method to generate random MAC addresses to Rando...
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[master]: core: Add tests to OvfManagerTest
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Dashboard storage calculation
by tnisan＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[master]: Dashboard: no hosts
by awels＠redhat.com
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>
7 years, 11 months
Change in ovirt-engine[master]: webadmin: Dashboard storage calculation
by awels＠redhat.com
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>
7 years, 11 months
Change in gerrit-admin[master]: adding vdsm & engine config for gerrit hooks
by eedri＠redhat.com
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>
7 years, 11 months