- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: wrong disk_profile when cloning to another SD
by gchaplik＠redhat.com 08 Jan '15

08 Jan '15

Gilad Chaplik has submitted this change and it was merged. Change subject: restapi: wrong disk_profile when cloning to another SD ...................................................................... restapi: wrong disk_profile when cloning to another SD When creating VM from template and cloning the disks to a different storage domain (not the original template's disks), causes wrong disk_profile error. Change-Id: I28f59d31d963e226d2da21751b185714fd57763f Bug-Url: https://bugzilla.redhat.com/1177940 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Gilad Chaplik: Verified -- To view, visit http://gerrit.ovirt.org/36591 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28f59d31d963e226d2da21751b185714fd57763f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-iso-uploader[master]: storage: fix nfs uploading
by sbonazzo＠redhat.com 08 Jan '15

08 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: fix nfs uploading ...................................................................... storage: fix nfs uploading Change-Id: Ie443e260a2773b5cc68512a6c2247dd00e99f368 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/__main__.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36499 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie443e260a2773b5cc68512a6c2247dd00e99f368 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: engine-backup: Do not backup jboss_runtime
by sbonazzo＠redhat.com 08 Jan '15

08 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: engine-backup: Do not backup jboss_runtime ...................................................................... packaging: engine-backup: Do not backup jboss_runtime Do not backup /var/lib/ovirt-engine/jboss_runtime . It's recreated on engine service start and needlessly takes space in the backup. Change-Id: I60a4779ed77fc370ddf1279c3267275d2d338372 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/bin/engine-backup.sh 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36500 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60a4779ed77fc370ddf1279c3267275d2d338372 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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

1 0

Change in ovirt-engine[master]: core: rm AddVmPoolWithVmsCommand.checkDestDomains
by tnisan＠redhat.com 08 Jan '15

08 Jan '15

Tal Nisan has submitted this change and it was merged. Change subject: core: rm AddVmPoolWithVmsCommand.checkDestDomains ...................................................................... core: rm AddVmPoolWithVmsCommand.checkDestDomains Remove the redundant implementation of AddVmPoolWithVmsCommand.checkDestDomains() which did nothing but call its super's implementation. Change-Id: Ib46d484611e41d6328c94426a577c63a10f6f656 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmPoolWithVmsCommand.java 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified Vered Volansky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/36687 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib46d484611e41d6328c94426a577c63a10f6f656 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: setup: Allow upgrading from 3.6
by sbonazzo＠redhat.com 08 Jan '15

08 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Allow upgrading from 3.6 ...................................................................... packaging: setup: Allow upgrading from 3.6 Change-Id: If8e5e87f107248dba09daade09a8167e470c6222 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/base/core/misc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/36646 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8e5e87f107248dba09daade09a8167e470c6222 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine 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

1 0

Change in ovirt-engine[ovirt-engine-3.5]: Override firewall configurations on deploy for provisioned h...
by oourfali＠redhat.com 08 Jan '15

08 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: Override firewall configurations on deploy for provisioned hosts ...................................................................... Override firewall configurations on deploy for provisioned hosts Currently we don't have parameter for override-firewall configurations in hosts\[id]\install restapi. Therefore we cannot pass this parameter along the way from foreman to engine. Until adding the parameter to the API this patch hacks the code to always override the firewall configurations for vdsm deploy for each foreman installation when updateVdsCommand is initiated by the reinstall request. This is monitored by checking that the host in InstallingOS status, which can be only if the host is firstly being provisioned by foreman. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1177126 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36632 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: Update only name and address fields when picking foreman par...
by oourfali＠redhat.com 08 Jan '15

08 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: Update only name and address fields when picking foreman parameters ...................................................................... Update only name and address fields when picking foreman parameters With updateModelFromVds we change other fields with no intention. Change-Id: Ia7eadb533dd02698f07103fa61e076b46d054522 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1178009 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java 1 file changed, 8 insertions(+), 11 deletions(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36631 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7eadb533dd02698f07103fa61e076b46d054522 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Override firewall configurations on deploy for provisioned h...
by oourfali＠redhat.com 08 Jan '15

08 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: Override firewall configurations on deploy for provisioned hosts ...................................................................... Override firewall configurations on deploy for provisioned hosts Currently we don't have parameter for override-firewall configurations in hosts\[id]\install restapi. Therefore we cannot pass this parameter along the way from foreman to engine. Until adding the parameter to the API this patch hacks the code to always override the firewall configurations for vdsm deploy for each foreman installation when updateVdsCommand is initiated by the reinstall request. This is monitored by checking that the host in InstallingOS status, which can be only if the host is firstly being provisioned by foreman. https://bugzilla.redhat.com/show_bug.cgi?id=1177126 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsCommand.java 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36634 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2634c33d87e9ef8ef41deeac28287b3b7ae1344a Gerrit-PatchSet: 4 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: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Update only name and address fields when picking foreman par...
by oourfali＠redhat.com 08 Jan '15

08 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: Update only name and address fields when picking foreman parameters ...................................................................... Update only name and address fields when picking foreman parameters With updateModelFromVds we change other fields with no intention. Change-Id: Ia7eadb533dd02698f07103fa61e076b46d054522 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1178009 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java 1 file changed, 8 insertions(+), 11 deletions(-) Approvals: Yaniv Bronhaim: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36550 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7eadb533dd02698f07103fa61e076b46d054522 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Removing redundant method midifier
by mlipchuk＠redhat.com 08 Jan '15

08 Jan '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Removing redundant method midifier ...................................................................... core: Removing redundant method midifier Removing public modifier in the StorageServerConnectionDAO interface. Quoting from the JLS: It is permitted, but discouraged as a matter of style, to redundantly specify the public and/or abstract modifier for a method declared in an interface. Change-Id: Ia0b950da2434379b2c23c6dc878bd613fbbef4b5 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageServerConnectionDAO.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit http://gerrit.ovirt.org/36666 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0b950da2434379b2c23c6dc878bd613fbbef4b5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0