March 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.4]: restapi: add support for import image as template
by iheim＠redhat.com 14 Mar '14

14 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: restapi: add support for import image as template ...................................................................... restapi: add support for import image as template Change-Id: I5efa82616c3fa3944800542aafce14d550bd74d9 Bug-Url: https://bugzilla.redhat.com/1075682 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCommand.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendActionableResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainImageResource.java 5 files changed, 27 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/25731 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5efa82616c3fa3944800542aafce14d550bd74d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: packaging: setup: Create /etc/pki/libvirt if not exists
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Create /etc/pki/libvirt if not exists ...................................................................... packaging: setup: Create /etc/pki/libvirt if not exists Bug-Url: https://bugzilla.redhat.com/1073446 Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/pki/vdsmpki.py 2 files changed, 27 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/25750 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 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: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.1]: packaging: setup: Create /etc/pki/libvirt if not exists
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Create /etc/pki/libvirt if not exists ...................................................................... packaging: setup: Create /etc/pki/libvirt if not exists Bug-Url: https://bugzilla.redhat.com/1034634 Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/pki/vdsmpki.py 2 files changed, 27 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/25751 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.1 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: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: Create /etc/pki/libvirt if not exists
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Create /etc/pki/libvirt if not exists ...................................................................... packaging: setup: Create /etc/pki/libvirt if not exists Bug-Url: https://bugzilla.redhat.com/1034634 Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/pki/vdsmpki.py 2 files changed, 27 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/25747 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5611532e2a2b2a98a662bc1549ed64a275f105fd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Remove error message when reports.xml doesn't exist
by emesika＠redhat.com 13 Mar '14

13 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Remove error message when reports.xml doesn't exist ...................................................................... core: Remove error message when reports.xml doesn't exist WebAdmin requests the reports.xml file after login, and ServletUtils prints an error message to the log if the file is not found. This patch provides an option to hide error messages on unneeded files that don't exist. Change-Id: Iaf6b075eb4754c5bb4153da8c6032a01bf801516 Bug-Url: https://bugzilla.redhat.com/1060242 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/FileServlet.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/ServletUtils.java 3 files changed, 27 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25324 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf6b075eb4754c5bb4153da8c6032a01bf801516 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Allow changing CSV params in vdc_options
by emesika＠redhat.com 13 Mar '14

13 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Allow changing CSV params in vdc_options ...................................................................... core: Allow changing CSV params in vdc_options In case vdc_option contain several values seperated by commas, this patch will give a nice stored procedure to remove a value from such a param. Change-Id: Ic25172a8b44e2f0aec81fe48808803c31b5f327b Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M packaging/dbscripts/common_sp.sql 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25744 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic25172a8b44e2f0aec81fe48808803c31b5f327b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.1]: packaging: setup: handle dummy network devices
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: handle dummy network devices ...................................................................... packaging: setup: handle dummy network devices Handle as unsupported devices any dummy device. Change-Id: I1159762e955c0c0cd9b00c5842077496025cba6d Bug-Url: https://bugzilla.redhat.com/1058438 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit c51096b909ed1b46f847abf4279c549f87fa55f7) --- M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 1 file changed, 38 insertions(+), 31 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25746 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1159762e955c0c0cd9b00c5842077496025cba6d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: handle dummy network devices
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: handle dummy network devices ...................................................................... packaging: setup: handle dummy network devices Handle as unsupported devices any dummy device. Change-Id: I1159762e955c0c0cd9b00c5842077496025cba6d Bug-Url: https://bugzilla.redhat.com/1058438 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 1 file changed, 38 insertions(+), 31 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25495 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1159762e955c0c0cd9b00c5842077496025cba6d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(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: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: don't ask about cdrom using answer file
by sbonazzo＠redhat.com 13 Mar '14

13 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: don't ask about cdrom using answer file ...................................................................... packaging: setup: don't ask about cdrom using answer file Previously if the cdrom was passed using answer file the value was used as default for the question about which image to use. Now it's used as is without asking anything else in non interactive mode. Change-Id: Idf4e2e46d63f876c1988238ad3d22e560c1c2fc4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/boot_cdrom.py 1 file changed, 35 insertions(+), 33 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25487 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf4e2e46d63f876c1988238ad3d22e560c1c2fc4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Do not show domain name for groups
by emesika＠redhat.com 13 Mar '14

13 Mar '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Do not show domain name for groups ...................................................................... core: Do not show domain name for groups The UI show user name for group, which is just its domain name. This patch solves this issue. Change-Id: Ife3bf9e19862ae0cc4462375032a24c62f44b6dc Bug-Url: https://bugzilla.redhat.com/1072918 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAODbFacadeImpl.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25735 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife3bf9e19862ae0cc4462375032a24c62f44b6dc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0