Change in ovirt-engine[ovirt-engine-4.0]: packaging: Remove dependency on fop on F23+
by oourfali＠redhat.com
Oved Ourfali has submitted this change and it was merged.
Change subject: packaging: Remove dependency on fop on F23+
......................................................................
packaging: Remove dependency on fop on F23+
Removes dependency on fop on F23+, because only fop 2.0 is available and
its API is incompatible with fop 1.1 we need (adn which is available on
EL7).
Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a
Bug-Url: https://bugzilla.redhat.com/1345986
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M ovirt-engine.spec.in
1 file changed, 23 insertions(+), 13 deletions(-)
Approvals:
Martin Peřina: Verified
Jenkins CI: Passed CI tests
Oved Ourfali: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/59184
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: Adding dao tests to std ci
by mperina＠redhat.com
Martin Peřina has submitted this change and it was merged.
Change subject: Adding dao tests to std ci
......................................................................
Adding dao tests to std ci
Basically this patch is all about running dao tests
inside a mock. For this, a postgresql DB should be
installed and run inside the mock.
Added postgres packages to .packages files
Created a new script, dao-tests.sh. This script is
being called only if files that are relevant to DB
had been changed. The script will pick a port
randomly and will update pg_hba.conf file to allow
DB access.
All the rest is the same as how the tests were
running when not in std ci.
Change-Id: If01639ebd0e346e0bd724a3b9ec68b44e7e9e765
Signed-off-by: Gil Shinar <gshinar(a)redhat.com>
---
M automation/check-merged.packages
M automation/check-merged.sh
M automation/check-patch.packages
A automation/check-patch.packages.el6
M automation/check-patch.sh
A automation/dao-tests.sh
6 files changed, 119 insertions(+), 0 deletions(-)
Approvals:
Martin Peřina: Looks good to me, approved
Eyal Edri: Looks good to me, but someone else must approve
Gil Shinar: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58633
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If01639ebd0e346e0bd724a3b9ec68b44e7e9e765
Gerrit-PatchSet: 18
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: packaging: Remove dependency on fop on F23+
by mperina＠redhat.com
Martin Peřina has submitted this change and it was merged.
Change subject: packaging: Remove dependency on fop on F23+
......................................................................
packaging: Remove dependency on fop on F23+
Removes dependency on fop on F23+, because only fop 2.0 is available and
its API is incompatible with fop 1.1 we need (adn which is available on
EL7).
Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a
Bug-Url: https://bugzilla.redhat.com/1345986
Signed-off-by: Martin Perina <mperina(a)redhat.com>
---
M ovirt-engine.spec.in
1 file changed, 23 insertions(+), 13 deletions(-)
Approvals:
Martin Peřina: Verified; Looks good to me, approved
Juan Hernandez: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/59084
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: restapi: Don't figure out NIC profile during V3 update
by juan.hernandez＠redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Don't figure out NIC profile during V3 update
......................................................................
restapi: Don't figure out NIC profile during V3 update
Currently the V3 compatibility layer tries to figure out an appropriate
NIC profile when a VM NIC is added or updated and no NIC profile is
explicitly provided. This works fine for adding, but during updates it
may select the wrong profile, thus randomly changing the network that
the NIC is attached to. To avoid that this patch changes the V3 adapter
so that it only tries to figure out the profile when the "network" or
"port_mirroring" attributes are included in the update, as any change to
those attributes requires a change of the VNIC profile.
In addition, to make things less random, the patch also changes the
logic to select the profile so that it sorts the candidates
alphabetically by name and chooses the first one.
Change-Id: Iaa953fcc200a1e175975e3dade9dc3a67d234f27
Bug-Url: https://bugzilla.redhat.com/1342054
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/helpers/V3NICHelper.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmNicServer.java
2 files changed, 8 insertions(+), 2 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58529
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa953fcc200a1e175975e3dade9dc3a67d234f27
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0]: core: fix serialization of pool related parameter classes
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: core: fix serialization of pool related parameter classes
......................................................................
core: fix serialization of pool related parameter classes
Fields in the parameters class should have setters and getters.
Change-Id: Iae19a2056ba31164c65744ea016a00329e6abcf2
Bug-Url: https://bugzilla.redhat.com/1345853
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmPoolWithVmsParameters.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmPoolOperationParameters.java
2 files changed, 25 insertions(+), 16 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit https://gerrit.ovirt.org/59144
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iae19a2056ba31164c65744ea016a00329e6abcf2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in jenkins[master]: fixing 4.0 publisher with 4.0 vdsm rpms
by eedri＠redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: fixing 4.0 publisher with 4.0 vdsm rpms
......................................................................
fixing 4.0 publisher with 4.0 vdsm rpms
Change-Id: Ie6746b9d9561d2467fdcc8b677f2d421cd39cbe3
Signed-off-by: Eyal Edri <eedri(a)redhat.com>
---
M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Eyal Edri: Verified; Ready for merge
Jenkins CI: Passed CI tests
Francesco Romani: Looks good to me
--
To view, visit https://gerrit.ovirt.org/59168
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie6746b9d9561d2467fdcc8b677f2d421cd39cbe3
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[ovirt-engine-4.0]: restapi: Add model HTML
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: restapi: Add model HTML
......................................................................
restapi: Add model HTML
This patch modifies the build process of the API so that it also builds
the HTML documentation of the model. The documentation will be available
from the API server, in the "/ovirt-engine/api/model" URL.
Change-Id: I869e6f2b8a6366d9f07231b74f6fe03427a93f08
Bug-Url: https://bugzilla.redhat.com/1346218
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
(cherry picked from commit d3022cc3126bb24afbe0fbe0af88d46426c62e73)
---
M backend/manager/modules/restapi/interface/definition/pom.xml
M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml
2 files changed, 38 insertions(+), 1 deletion(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/59129
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I869e6f2b8a6366d9f07231b74f6fe03427a93f08
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-hosted-engine-setup[master]: build: version bump after branching
by stirabos＠redhat.com
Simone Tiraboschi has submitted this change and it was merged.
Change subject: build: version bump after branching
......................................................................
build: version bump after branching
Change-Id: I7f57d4d4d970f041d7a27f1afc704cd55f13059e
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M configure.ac
M ovirt-hosted-engine-setup.spec.in
2 files changed, 5 insertions(+), 2 deletions(-)
Approvals:
Simone Tiraboschi: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Yedidyah Bar David: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/59120
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7f57d4d4d970f041d7a27f1afc704cd55f13059e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
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: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in jenkins[master]: Add ovirt-hosted-engine-setup stable branch for 4.0
by dcaro＠redhat.com
David Caro has submitted this change and it was merged.
Change subject: Add ovirt-hosted-engine-setup stable branch for 4.0
......................................................................
Add ovirt-hosted-engine-setup stable branch for 4.0
Configure ovirt-hosted-engine_stardard job
to build also from a stable branch for 4.0.
Adding it to the 4.0 nightly repo composer.
Change-Id: I1955bf3913e0856b3ffb2c9bf81d00cfb4fe9d37
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M jobs/confs/projects/ovirt-hosted-engine-setup/ovirt-hosted-engine_stardard.yaml
M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml
2 files changed, 3 insertions(+), 1 deletion(-)
Approvals:
Simone Tiraboschi: Verified; Ready for review
Jenkins CI: Passed CI tests
David Caro: Looks good to me; Ready for merge
--
To view, visit https://gerrit.ovirt.org/59150
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1955bf3913e0856b3ffb2c9bf81d00cfb4fe9d37
Gerrit-PatchSet: 1
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaro(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months
Change in ovirt-engine[master]: webadmin: Add label comboboxes to VM & host dialogs
by awels＠redhat.com
Alexander Wels has submitted this change and it was merged.
Change subject: webadmin: Add label comboboxes to VM & host dialogs
......................................................................
webadmin: Add label comboboxes to VM & host dialogs
This patch adds read-only comboboxes to the general tab of the
add/edit dialog boxes for VMs and hosts. The boxes are
populated with all available affinity labels. For existing VMs
and hosts, labels that have been set through the REST API are
shown as selected.
This is phase 1 of this feature. Phase 2 will introduce the
ability to change selected labels via the UI. Until that time,
labels can be modified via the REST API.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1338799
Change-Id: Iad36512b423a20b3ad7ad9e01ff43882555fcdb6
Signed-off-by: Phillip Bailey <phbailey(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants.properties
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.ui.xml
10 files changed, 165 insertions(+), 4 deletions(-)
Approvals:
Alexander Wels: Looks good to me, approved
Jenkins CI: Passed CI tests
Phillip Bailey: Verified
--
To view, visit https://gerrit.ovirt.org/57903
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iad36512b423a20b3ad7ad9e01ff43882555fcdb6
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 3 months