June 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Added alert on number of LVs config value
by tnisan＠redhat.com 24 Jun '14

24 Jun '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Added alert on number of LVs config value ...................................................................... core: Added alert on number of LVs config value Added a config value for the max number of LVs in a storage domain from which every image creation will cause an alert to the user Change-Id: Ie5ed86e41cb20c819cf8466c386027d8e0738590 Related-To: https://bugzilla.redhat.com/1015186 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28959 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie5ed86e41cb20c819cf8466c386027d8e0738590 Gerrit-PatchSet: 5 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: kdump: Fix kexec-tools dependency for RHEL 7
by Alon Bar-Lev 24 Jun '14

24 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: kdump: Fix kexec-tools dependency for RHEL 7 ...................................................................... kdump: Fix kexec-tools dependency for RHEL 7 Fixes kexec-tools package dependency for RHEL 7. kexec-tools 2.0.4-32 is part of RHEL 7.0, but it doesn't contain support for fence_kdump config. kexec-tools 2.0.4-33, which contains support for fence_kdump config, will be part of RHEL 7.1. Change-Id: Id898ba2ef1f4ab38253892e78e83443e8bf36eac Bug-Url: https://bugzilla.redhat.com/1079821 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-host-deploy-offline.spec.in M src/plugins/ovirt-host-deploy/core/offlinepackager.py M src/plugins/ovirt-host-deploy/kdump/packages.py 3 files changed, 10 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Martin Peřina: Verified -- To view, visit http://gerrit.ovirt.org/29047 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id898ba2ef1f4ab38253892e78e83443e8bf36eac Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: use CDI version 1.0-SP4
by rgolan＠redhat.com 24 Jun '14

24 Jun '14

Roy Golan has submitted this change and it was merged. Change subject: core: use CDI version 1.0-SP4 ...................................................................... core: use CDI version 1.0-SP4 Change-Id: I86888b59a87e22c5ab0fa74a7ffd30941a2c955c Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29109 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86888b59a87e22c5ab0fa74a7ffd30941a2c955c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: Improve iSCSI Bond CDA validation
by oourfali＠redhat.com 24 Jun '14

24 Jun '14

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Improve iSCSI Bond CDA validation ...................................................................... engine: Improve iSCSI Bond CDA validation Add following CDA validations due to adding iSCSI Multipathing support through the REST API: 1. Validate that datacenter version is at least 3.4. 2. Validate that added logical networks belong to the same datacenter as the specified iSCSI Bond and that none of them is "required" network. 3. Validate that added storage targets are indeed iSCSI targets, they are "connectable" and they belong to the storage domain from the same datacenter as the specified iSCSI Bond. Change-Id: I1d0ac7e6af6cb87b500740d679af2f13e63dbeb5 Bug-Url: https://bugzilla.redhat.com/1108174 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddIscsiBondCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/EditIscsiBondCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/IscsiBondValidator.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/IscsiBondValidatorTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 9 files changed, 430 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28605 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d0ac7e6af6cb87b500740d679af2f13e63dbeb5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin:Tasks with longer description overlaps date/time fixed
by kmayilsa＠redhat.com 24 Jun '14

24 Jun '14

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin:Tasks with longer description overlaps date/time fixed ...................................................................... webadmin:Tasks with longer description overlaps date/time fixed Tasks having longer description overlaps with date/time column in tasks panel.The overlapping part of the text is now replaced with '...' with proper padding. Change-Id: I84795780163a8196825f66bde5f2394cbb82f871 Bug-Url: https://bugzilla.redhat.com/1023929 Signed-off-by: Karnan TC <kchidamb(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/footer/TasksTree.java 1 file changed, 12 insertions(+), 4 deletions(-) Approvals: Karnan t c: Verified Alexander Wels: Looks good to me, approved Kanagaraj M: Looks good to me, but someone else must approve Ramesh N: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28470 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84795780163a8196825f66bde5f2394cbb82f871 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: CDI workaournd jboss 7.1.1 bug
by rgolan＠redhat.com 24 Jun '14

24 Jun '14

Roy Golan has submitted this change and it was merged. Change subject: core: CDI workaournd jboss 7.1.1 bug ...................................................................... core: CDI workaournd jboss 7.1.1 bug To workaround jboss 7.1.1 bug the Injector bean must be moved to BLL module so it will be injected into Backend.java and initialize on startup. Till master will support wildfly or at least jboss version EAP 6.x or greater CDI beans can't be detected across modules. To overcome that one must create a producer method and put it under BllCDIAdapter and there to create or use a getInstance incase its a singleton Change-Id: If996ccbce18fcc5471acff49f5bef8c70269e0aa Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandsFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/BllCDIAdapter.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/Injector.java 4 files changed, 4 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Roy Golan: Verified Liran Zelkha: Looks good to me, but someone else must approve Yair Zaslavsky: Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If996ccbce18fcc5471acff49f5bef8c70269e0aa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in samples-portals[master]: New operating systems logos.
by shavivi＠redhat.com 24 Jun '14

24 Jun '14

Shahar Havivi has submitted this change and it was merged. Change subject: New operating systems logos. ...................................................................... New operating systems logos. Change-Id: Ibcb6c44d936b38252c44ea9c0a7a049d789a7d2e Signed-off-by: Amador Pahim <apahim(a)redhat.com> --- M python-gtk/images/linux.png M python-gtk/images/ovirt.png M python-gtk/images/rhel.png A python-gtk/images/sles.png M python-gtk/images/ubuntu.png M python-gtk/images/win.png M python-gtk/userportalgtk.py 7 files changed, 3 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved Amador Pahim: Verified -- To view, visit http://gerrit.ovirt.org/28192 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcb6c44d936b38252c44ea9c0a7a049d789a7d2e Gerrit-PatchSet: 1 Gerrit-Project: samples-portals Gerrit-Branch: master Gerrit-Owner: Amador Pahim <apahim(a)redhat.com> Gerrit-Reviewer: Amador Pahim <apahim(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: bll: introducing runInternalQuery at CommandBase
by yzaslavs＠redhat.com 24 Jun '14

24 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: bll: introducing runInternalQuery at CommandBase ...................................................................... bll: introducing runInternalQuery at CommandBase Change-Id: I756c5b67a376ca73c95f3fcb83bfe7a19125b200 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddWatchdogCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CloneVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmFromImportExportCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmTemplateFromImportExportCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmVersionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/AttachNetworkToVdsInterfaceCommand.java 13 files changed, 27 insertions(+), 19 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ravi Nori: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28936 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I756c5b67a376ca73c95f3fcb83bfe7a19125b200 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Changed some Backend.getInstance() to getBackend()
by yzaslavs＠redhat.com 24 Jun '14

24 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Changed some Backend.getInstance() to getBackend() ...................................................................... core: Changed some Backend.getInstance() to getBackend() Change-Id: I238ae8570a2c87410422315e215061e9e3e6a27a Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 3 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ravi Nori: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29053 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I238ae8570a2c87410422315e215061e9e3e6a27a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: ovirt-node-plugin-vdsm: Bump release to 0.1.2-5
by dougsland＠redhat.com 23 Jun '14

23 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: ovirt-node-plugin-vdsm: Bump release to 0.1.2-5 ...................................................................... ovirt-node-plugin-vdsm: Bump release to 0.1.2-5 - hooks: on-boot changed the path Change-Id: I74d74504629c4a51e338bf35d5e51e252e78b602 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29091 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74d74504629c4a51e338bf35d5e51e252e78b602 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0