Change in ovirt-engine[master]: core: fixed missing mac_pool_id in LabelTestDao
by alkaplan@redhat.com
Alona Kaplan has submitted this change and it was merged.
Change subject: core: fixed missing mac_pool_id in LabelTestDao
......................................................................
core: fixed missing mac_pool_id in LabelTestDao
Change-Id: Ibe197bcd632a23a3bb2e58ec197de2b65ab82659
Signed-off-by: Martin Mucha <mmucha(a)redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/LabelDaoTest.java
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Martin Mucha: Verified
Alona Kaplan: Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58328
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe197bcd632a23a3bb2e58ec197de2b65ab82659
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[master]: centos-7: Added auto accept defaults option
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: centos-7: Added auto accept defaults option
......................................................................
centos-7: Added auto accept defaults option
An option to automatically accept defaults was added to the answer file.
Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/ovirt-answer
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Lev Veyde: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58335
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[master]: centos-7: Update VDSM bridge setup script for the new API
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: centos-7: Update VDSM bridge setup script for the new API
......................................................................
centos-7: Update VDSM bridge setup script for the new API
The bridge setup script was updated to use the new API, as there
is a change both in the module name and the parameter types of
some of the functions used.
Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/oVirtLiveFiles/root/usr/libexec/vdsm/hooks/before_vdsm_start/20-setupBridge.py
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Lev Veyde: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58334
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[master]: centos-7: Remove nfs-server workaround
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: centos-7: Remove nfs-server workaround
......................................................................
centos-7: Remove nfs-server workaround
This seems no longer to be required in 4.0, and actually
causes the nfs-server to fail to start.
Change-Id: Id70d15f1a185c5279471528ba63a953444995af0
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh
1 file changed, 0 insertions(+), 3 deletions(-)
Approvals:
Lev Veyde: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58333
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id70d15f1a185c5279471528ba63a953444995af0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[master]: kickstart: remove kernel plugin
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: kickstart: remove kernel plugin
......................................................................
kickstart: remove kernel plugin
Remove ovirt-host-deploy kernel plugins, as it fails to run under
ovirt-live, and causes the host-deploy to fail.
Change-Id: I387c50ff241df0b00f33b094f47a72584122501f
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Lev Veyde: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58332
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I387c50ff241df0b00f33b094f47a72584122501f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-iso-uploader[master]: ovirt-iso-uploader: port to SDK version 4
by rmartins@redhat.com
Rafael Martins has submitted this change and it was merged.
Change subject: ovirt-iso-uploader: port to SDK version 4
......................................................................
ovirt-iso-uploader: port to SDK version 4
Please note that 'list' action behaviour changed, due to some
limitations of ovirt-engine-sdk4:
- Not showing the datacenter where the storage domain belongs
- Status column isn't very reliable.
Change-Id: I94e22eea5dcde995f4f109e90dd9921143cdab55
Signed-off-by: Rafael Martins <rmartins(a)redhat.com>
---
M configure.ac
M ovirt-iso-uploader.spec.in
M src/__main__.py
3 files changed, 64 insertions(+), 119 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Jenkins CI: Passed CI tests
Yedidyah Bar David: Looks good to me, but someone else must approve
Rafael Martins: Verified
--
To view, visit https://gerrit.ovirt.org/58201
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I94e22eea5dcde995f4f109e90dd9921143cdab55
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-iso-uploader
Gerrit-Branch: master
Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Rafael Martins <rmartins(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: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-engine[master]: core: do not compare hash of guest agent nics in vms monitoring
by ahadas@redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: do not compare hash of guest agent nics in vms monitoring
......................................................................
core: do not compare hash of guest agent nics in vms monitoring
The hash of the network interfaces reported by the guest agent is computed
by the engine (rather than reported by VDSM) and therefore should not be a
part of the comparison of the dynamic data from the database and the data
reported by VDSM in VMs monitoring that leads to redundant updates of the
dynamic data of VMs.
Change-Id: I01629b006525cd30be5d7cdf6e6ccd398131d50a
Bug-Url: https://bugzilla.redhat.com/1340722
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmDynamic.java
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Vinzenz Feenstra: Looks good to me, but someone else must approve
Arik Hadas: Verified; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58302
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I01629b006525cd30be5d7cdf6e6ccd398131d50a
Gerrit-PatchSet: 3
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: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-engine[master]: core: minor cleanup in VmHandler
by ahadas@redhat.com
Arik Hadas has submitted this change and it was merged.
Change subject: core: minor cleanup in VmHandler
......................................................................
core: minor cleanup in VmHandler
1. Remove unused method
2. Add SupressWarning for unchecked cast
Change-Id: I77d4493d7cd5e2f98e53359dd91bf8c049f398ec
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
1 file changed, 1 insertion(+), 5 deletions(-)
Approvals:
Shahar Havivi: Looks good to me, approved
Jenkins CI: Passed CI tests
Vinzenz Feenstra: Looks good to me, but someone else must approve
Arik Hadas: Verified
--
To view, visit https://gerrit.ovirt.org/58301
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I77d4493d7cd5e2f98e53359dd91bf8c049f398ec
Gerrit-PatchSet: 3
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: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[ovirt-live-4.0]: centos-7: Added auto accept defaults option
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: centos-7: Added auto accept defaults option
......................................................................
centos-7: Added auto accept defaults option
An option to automatically accept defaults was added to the answer file.
Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/ovirt-answer
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Lev Veyde: Verified
Sandro Bonazzola: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58290
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7d3b8a1dcd8df4ce7f64973eba2078002a77579f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: ovirt-live-4.0
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months
Change in ovirt-live[ovirt-live-4.0]: centos-7: Update VDSM bridge setup script for the new API
by lveyde@redhat.com
Lev Veyde has submitted this change and it was merged.
Change subject: centos-7: Update VDSM bridge setup script for the new API
......................................................................
centos-7: Update VDSM bridge setup script for the new API
The bridge setup script was updated to use the new API, as there
is a change both in the module name and the parameter types of
some of the functions used.
Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M centos-7/oVirtLiveFiles/root/usr/libexec/vdsm/hooks/before_vdsm_start/20-setupBridge.py
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Lev Veyde: Verified
Sandro Bonazzola: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/58287
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I498c379ac36a9fe17a238276fb9efcc0cdfa97f0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-live
Gerrit-Branch: ovirt-live-4.0
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
8 years, 5 months