Change in mom[master]: Add missing log for getStatistics in XMLRPC interface
by alitke＠redhat.com
Adam Litke has submitted this change and it was merged.
Change subject: Add missing log for getStatistics in XMLRPC interface
......................................................................
Add missing log for getStatistics in XMLRPC interface
All methods exported to XMLRPC interface has own info log, when is invoked.
Only getStatistics was missing this log info.
Change-Id: I0021fae11cb9c9efcd2c59e567ec3263a3693d21
Signed-off-by: Martin Pavlasek <mpavlase(a)redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34712
Reviewed-by: Adam Litke <alitke(a)redhat.com>
---
M mom/MOMFuncs.py
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Adam Litke: Looks good to me, approved
Martin Pavlásek: Verified
--
To view, visit http://gerrit.ovirt.org/34712
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0021fae11cb9c9efcd2c59e567ec3263a3693d21
Gerrit-PatchSet: 2
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Martin Pavlásek <mpavlase(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Martin Pavlásek <mpavlase(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years
Change in ovirt-dwh[ovirt-engine-dwh-3.5]: packaging: setup: pep8 fixes
by ydary＠redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: setup: pep8 fixes
......................................................................
packaging: setup: pep8 fixes
One is minor, other is real - missing import. Affects only certain kinds
of failures, therefore not noticed in testing.
Bug-Url: https://bugzilla.redhat.com/1162186
Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
(cherry picked from commit 72bba548ef8a6f30cbfb6d60b1581779420e2b9a)
---
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
2 files changed, 3 insertions(+), 2 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/35001
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: ovirt-engine-dwh-3.5
Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
10 years
Change in ovirt-dwh[master]: packaging: setup: pep8 fixes
by ydary＠redhat.com
Yaniv Dary has submitted this change and it was merged.
Change subject: packaging: setup: pep8 fixes
......................................................................
packaging: setup: pep8 fixes
One is minor, other is real - missing import. Affects only certain kinds
of failures, therefore not noticed in testing.
Bug-Url: https://bugzilla.redhat.com/1162186
Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py
M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py
2 files changed, 3 insertions(+), 2 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Simone Tiraboschi: Looks good to me, but someone else must approve
Yedidyah Bar David: Verified
Yaniv Dary: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/34992
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-release[master]: bump version / release
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: bump version / release
......................................................................
bump version / release
bump version release for allowing people
to get latest changes including deps for EL7
Change-Id: Ib2361d328d372d024700a1631d6b4e381333ebd3
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M ovirt-release-3.5/configure.ac
M ovirt-release-3.5/ovirt-release35.spec.in
M ovirt-release-master/configure.ac
M ovirt-release-master/ovirt-release-master.spec.in
4 files changed, 10 insertions(+), 4 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
Yedidyah Bar David: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/34996
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2361d328d372d024700a1631d6b4e381333ebd3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-release[master]: packaging: add missing dependencies for engine on el7
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: add missing dependencies for engine on el7
......................................................................
packaging: add missing dependencies for engine on el7
Change-Id: I77367f4540ddfdacf1d65328b7d8f757ed1efc06
Signed-off-by: Yedidyah Bar David <didi(a)redhat.com>
---
M ovirt-release-3.5/ovirt-el7-deps.repo.in
M ovirt-release-master/ovirt-el7-deps.repo.in
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Yedidyah Bar David: Verified
--
To view, visit http://gerrit.ovirt.org/34993
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I77367f4540ddfdacf1d65328b7d8f757ed1efc06
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-release
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-engine[master]: webadmin: CPU Type and CPU Architecture fields in Cluster di...
by tjelinek＠redhat.com
Tomas Jelinek has submitted this change and it was merged.
Change subject: webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix
......................................................................
webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix
* CPU Architecture listbox is placed right above the CPU Type because
they are related. (ClusterPopupView.ui.xml)
* CPU Architecture field doesn't disappear when CPU Type is selected.
(Bug fix). (ClusterPopupView.java)
* CPU Architecture filters content of CPU type listbox.
(ClusterModel.java, FilteredListModel.java)
Bug-Url: https://bugzilla.redhat.com/1159777
Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e
Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com>
---
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/FilteredListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
4 files changed, 123 insertions(+), 17 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Jakub Niedermertl: Verified
--
To view, visit http://gerrit.ovirt.org/34816
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-engine[master]: Adding host to DB before calling provision
by yzaslavs＠redhat.com
Yair Zaslavsky has submitted this change and it was merged.
Change subject: Adding host to DB before calling provision
......................................................................
Adding host to DB before calling provision
When calling provision foreman expect the engine's host-id to allow calling
the reinstallation after provisioning. Engine generates host-id while saving
host info to db, therefore this patch switches between the DB.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1158588
Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java
1 file changed, 11 insertions(+), 11 deletions(-)
Approvals:
Yaniv Bronhaim: Verified
Yair Zaslavsky: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/34860
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-engine[ovirt-engine-3.5]: packaging:[ver2] fail the build if osinfo-defaults.propertie...
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse
......................................................................
packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse
check the properties file we ship with the exact code that checks it
at runtime when the engine loads.
this test is hooked into the packaging process and must fail it.
Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9
Bug-Url: https://bugzilla.redhat.com/1056965
Signed-off-by: Roy Golan <rgolan(a)redhat.com>
---
M build/validations/pom.xml
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Roy Golan: Verified
--
To view, visit http://gerrit.ovirt.org/34959
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in ovirt-engine[master]: webadmin, userportal: Remove explicit include for generated s...
by vszocs＠redhat.com
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: Remove explicit include for generated sources
......................................................................
webadmin,userportal: Remove explicit include for generated sources
This patch builds on top of http://gerrit.ovirt.org/#/c/34402/
and ensures that everything under target/generated-sources/gwt
directory is added to project's resources.
Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae
Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com>
---
M frontend/webadmin/modules/frontend/pom.xml
1 file changed, 0 insertions(+), 3 deletions(-)
Approvals:
Alexander Wels: Verified; Looks good to me, approved
Vojtech Szocs: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/34708
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years
Change in jenkins[master]: build vdsm-jsonrpc-java 3.5 on el7
by dcaroest＠redhat.com
David Caro has submitted this change and it was merged.
Change subject: build vdsm-jsonrpc-java 3.5 on el7
......................................................................
build vdsm-jsonrpc-java 3.5 on el7
Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e
Bug-Url: https://bugzilla.redhat.com/1159293
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Eyal Edri: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/34923
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
10 years