Change in ovirt-engine[master]: core: Show only assignable host devices in frontend
by ahadas＠redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: Show only assignable host devices in frontend
......................................................................
core: Show only assignable host devices in frontend
VDSM now reports which host devices can be used for hostdev passthrough
purposes via the "is_assignable" flag. This patch adds parsing of this
field and uses it on the frontend to filter devices that will be offered
to the user for assignment.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1303160
Change-Id: I7d7b52ca10368a8fe0802d0ececfb8f1c12ebbe5
Signed-off-by: Martin Betak <mbetak(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/HostDevice.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/HostDeviceDaoImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/HostDeviceDaoTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/hostdev/AddVmHostDevicesModel.java
M packaging/dbscripts/host_device_sp.sql
A packaging/dbscripts/upgrade/04_00_0550_add_host_device_is_assignable.sql
9 files changed, 73 insertions(+), 11 deletions(-)
Approvals:
Martin Betak: Verified
Arik Hadas: Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/56349
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7d7b52ca10368a8fe0802d0ececfb8f1c12ebbe5
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in jenkins[master]: dropping httpcomponents from 3.6 publisher
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: dropping httpcomponents from 3.6 publisher
......................................................................
dropping httpcomponents from 3.6 publisher
Change-Id: I9c73b14d72b37cfde186022f6afa7cd113ed3aae
Signed-off-by: ngoldin <ngoldin(a)redhat.com>
---
M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml
1 file changed, 0 insertions(+), 4 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me; Ready for merge
Nadav Goldin: Verified; Ready for review
Eyal Edri: Looks good to me
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/56577
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9c73b14d72b37cfde186022f6afa7cd113ed3aae
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Nadav Goldin <ngoldin(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: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[master]: core: VNC/VGA allowed for 3.6 VMs on PPC
by ahadas＠redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: VNC/VGA allowed for 3.6 VMs on PPC
......................................................................
core: VNC/VGA allowed for 3.6 VMs on PPC
VNC/VGA is the only console which is enabled for 3.6 VMs
running on PPC.
Change-Id: I806d02d85aa1d55dcfa20ff29a18e0238da29656
Bug-Url: https://bugzilla.redhat.com/1302657
Signed-off-by: Marek Libra <mlibra(a)redhat.com>
---
M packaging/conf/osinfo-defaults.properties
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Marek Libra: Verified
Arik Hadas: Looks good to me, approved
Milan Zamazal: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/56623
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I806d02d85aa1d55dcfa20ff29a18e0238da29656
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Marek Libra <mlibra(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Enable tooltip on greyed out LUNs
by tnisan＠redhat.com
Tal Nisan has submitted this change and it was merged.
Change subject: webadmin: Enable tooltip on greyed out LUNs
......................................................................
webadmin: Enable tooltip on greyed out LUNs
When editing an ISCSI/FC storage domain, if a LUN is
already in use it will be greyed out.
If the user hovers over the LUN a tooltip appears
with the reason why it is disabled.
In this patch, the tooltip is added to the whole row,
not only the checkbox cell.
It will always appear on the left of the row.
Bug-Url: https://bugzilla.redhat.com/1330483
Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b
Signed-off-by: Fred Rolland <frolland(a)redhat.com>
Signed-off-by: Daniel Erez <derez(a)redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/AbstractSanStorageList.java
1 file changed, 4 insertions(+), 4 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Jenkins CI: Passed CI tests
Freddy Rolland: Verified
--
To view, visit https://gerrit.ovirt.org/56610
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[master]: engine: Create default network QoS for ovirtmgmt upon DC cre...
by alkaplan＠redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: engine: Create default network QoS for ovirtmgmt upon DC creation
......................................................................
engine: Create default network QoS for ovirtmgmt upon DC creation
Change-Id: I3551563903ca5a4548c08f8b774c7bd82ce7874a
Bug-Url: https://bugzilla.redhat.com/1255474
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/AddEmptyStoragePoolCommand.java
1 file changed, 39 insertions(+), 1 deletion(-)
Approvals:
Alona Kaplan: Looks good to me, approved; Passed CI tests
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/56429
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3551563903ca5a4548c08f8b774c7bd82ce7874a
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in jenkins[master]: [ovirt-scheduler] dropping el6, fc22 on master jobs
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: [ovirt-scheduler] dropping el6, fc22 on master jobs
......................................................................
[ovirt-scheduler] dropping el6, fc22 on master jobs
dropping ovirt-scheduler-proxy-master_build-artifacts-{el6, fc22} jobs
Change-Id: I89d973666d5505af87eb9018f31ce3e7dc983c55
Signed-off-by: ngoldin <ngoldin(a)redhat.com>
---
M jobs/confs/projects/ovirt-scheduler-proxy/ovirt-scheduler-proxy_standard.yaml
1 file changed, 0 insertions(+), 4 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me; Ready for merge
Nadav Goldin: Verified; Ready for review
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/56624
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I89d973666d5505af87eb9018f31ce3e7dc983c55
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[master]: engine: Send IPv6 address to VDSM as a migration destination.
by alkaplan＠redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: engine: Send IPv6 address to VDSM as a migration destination.
......................................................................
engine: Send IPv6 address to VDSM as a migration destination.
Send IPV6 address as the destination migration address
if IPv4 isn't configured on both sides and IPv6 is.
Change-Id: I751d540c0ebd3f0db57d1ca5809f661c0c713fdd
Bug-Url: https://bugzilla.redhat.com/1080725
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
1 file changed, 47 insertions(+), 11 deletions(-)
Approvals:
Alona Kaplan: Looks good to me, approved; Passed CI tests
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/56540
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I751d540c0ebd3f0db57d1ca5809f661c0c713fdd
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[master]: core: minimize access to database during vm analysis
by ahadas＠redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: minimize access to database during vm analysis
......................................................................
core: minimize access to database during vm analysis
VM fetchers provide 3 types of pairs to VmAnalyzer:
1. pair of reported VM from VDSM and VM from database
2. pair of VM from database and NULL
3. pair of VM from VDSM and NULL
1) means that the VM is 'stable', that is it is reported from
the host that it is expected to run on.
2) means that the VM disappeared, that is no longer reported
from the host that it is expected to run on.
3) means that the VM is reported from host that the VM was
not running on.
To resolve these cases the fetchers get the reported VM from
VDSM and the ones in the database that are running on the
monitored host. In case of #3, VmAnalyzer needs to query the
database again.
This patch queries the VM from the database, if needed, in the
factory of VmAnalyzer. That way, we reduce the access to the
database inside VmAnalyzer and replace unclear null-checks with
more declarative checks.
Change-Id: I5d6b83c4f4a2d8ee6d1bb3f7f69bebeeec0675cb
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerFactory.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmTestPairs.java
5 files changed, 25 insertions(+), 83 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Arik Hadas: Verified
--
To view, visit https://gerrit.ovirt.org/56566
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5d6b83c4f4a2d8ee6d1bb3f7f69bebeeec0675cb
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-engine[master]: engine: allow QoS and non-QoS networks to be attached to a nic
by alkaplan＠redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: engine: allow QoS and non-QoS networks to be attached to a nic
......................................................................
engine: allow QoS and non-QoS networks to be attached to a nic
Change-Id: Id83fa947c76fbea4cf313ef50487c2505507d3f3
Bug-Url: https://bugzilla.redhat.com/1271094
Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidatorTest.java
2 files changed, 1 insertion(+), 98 deletions(-)
Approvals:
Alona Kaplan: Looks good to me, approved; Passed CI tests
Yevgeny Zaspitsky: Verified
--
To view, visit https://gerrit.ovirt.org/56470
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id83fa947c76fbea4cf313ef50487c2505507d3f3
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years
Change in ovirt-live[ovirt-live-3.6]: oVirt Live 3.6.6 rc1
by sbonazzo＠redhat.com
Sandro Bonazzola has submitted this change and it was merged.
Change subject: oVirt Live 3.6.6 rc1
......................................................................
oVirt Live 3.6.6 rc1
Change-Id: Ife1eb5064420e037fb52c3eba68c7c6c1bc6e0ce
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/Makefile
M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/56621
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ife1eb5064420e037fb52c3eba68c7c6c1bc6e0ce
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: ovirt-live-3.6
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years