- Engine-commits - oVirt List Archives

Change in ovirt-log-collector[master]: sos: ovirt: add rhevm packages
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: sos: ovirt: add rhevm packages ...................................................................... sos: ovirt: add rhevm packages Align with upstream sos master patches. Add support for downstream packages and ovirt-scheduler-proxy Change-Id: I7e41635ffbb937390804e05961f49c687798cba2 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/sos/plugins/ovirt.py 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31356 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e41635ffbb937390804e05961f49c687798cba2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(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]: packaging: setup: Ask to remove engine only if enabled
by tnisan＠redhat.com 12 Aug '14

12 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Ask to remove engine only if enabled ...................................................................... packaging: setup: Ask to remove engine only if enabled Change-Id: I8d6b938b0635e30eeb10c4e10371eccf07987ecd Bug-Url: https://bugzilla.redhat.com/1060529 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit b84f6880e800b444c62d93311168b2d0085a6c40) --- M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/system/engine.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/31399 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d6b938b0635e30eeb10c4e10371eccf07987ecd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: iscsi: add multiple lun per target support
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: iscsi: add multiple lun per target support ...................................................................... packaging: setup: iscsi: add multiple lun per target support Add support for multiple lun on the same target. It allows to choose the lun to be used when more than one is available on the target. Change-Id: I0b00cc09754a8b95bc1497e0f769ae2a9c64f1b5 Bug-Url: https://bugzilla.redhat.com/1125315 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 027953923be247ac7f2a2c0e9dfe3f49ed2667bb) --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py 2 files changed, 123 insertions(+), 34 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31406 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b00cc09754a8b95bc1497e0f769ae2a9c64f1b5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: iscsi: add multiple lun per target support
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: iscsi: add multiple lun per target support ...................................................................... packaging: setup: iscsi: add multiple lun per target support Add support for multiple lun on the same target. It allows to choose the lun to be used when more than one is available on the target. Change-Id: I0b00cc09754a8b95bc1497e0f769ae2a9c64f1b5 Bug-Url: https://bugzilla.redhat.com/1125315 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py 2 files changed, 123 insertions(+), 34 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jiří Moskovčák: Looks good to me, but someone else must approve Greg Padgett: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/31398 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b00cc09754a8b95bc1497e0f769ae2a9c64f1b5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(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-iso-uploader[ovirt-iso-uploader-3.5]: error handling: handle invalid host
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: error handling: handle invalid host ...................................................................... error handling: handle invalid host previously if an invalid host was specified an error like ERROR: 'NoneType' object is not iterable was printed. Now a more explanatory error is showed: ERROR: Unable to connect to REST API. Message: The host name "localhost" contained in the URL doesn't match any of the names in the server certificate. ERROR: Unable to get ISO domain data Change-Id: I86ff1d2bcff923c76176554ae73beb47caa3d633 Bug-Url: https://bugzilla.redhat.com/1110756 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit a6b721e0e9164dfadccd3961467d1b65c8994554) --- M src/__main__.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31404 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86ff1d2bcff923c76176554ae73beb47caa3d633 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[master]: error handling: handle invalid host
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: error handling: handle invalid host ...................................................................... error handling: handle invalid host previously if an invalid host was specified an error like ERROR: 'NoneType' object is not iterable was printed. Now a more explanatory error is showed: ERROR: Unable to connect to REST API. Message: The host name "localhost" contained in the URL doesn't match any of the names in the server certificate. ERROR: Unable to get ISO domain data Change-Id: I86ff1d2bcff923c76176554ae73beb47caa3d633 Bug-Url: https://bugzilla.redhat.com/1110756 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31383 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86ff1d2bcff923c76176554ae73beb47caa3d633 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-release[master]: packaging: add ioprocess from EPEL
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: add ioprocess from EPEL ...................................................................... packaging: add ioprocess from EPEL Change-Id: I8a41d09c771229b586232769fef62da9aeb10e25 Bug-Url: https://bugzilla.redhat.com/1129236 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-3.5/configure.ac M ovirt-release-3.5/ovirt-el6-deps.repo.in M ovirt-release-3.5/ovirt-el7-deps.repo.in M ovirt-release-3.5/ovirt-release35.spec.in M ovirt-release-master/ovirt-el6-deps.repo.in M ovirt-release-master/ovirt-el7-deps.repo.in 6 files changed, 9 insertions(+), 6 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/31401 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a41d09c771229b586232769fef62da9aeb10e25 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: Ask to remove engine only if enabled
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Ask to remove engine only if enabled ...................................................................... packaging: setup: Ask to remove engine only if enabled Change-Id: I8d6b938b0635e30eeb10c4e10371eccf07987ecd Bug-Url: https://bugzilla.redhat.com/1060529 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/system/engine.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/31397 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d6b938b0635e30eeb10c4e10371eccf07987ecd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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

1 0

Change in ovirt-engine[master]: core: move provider+storage related classes to a dedicated p...
by ahadas＠redhat.com 12 Aug '14

12 Aug '14

Arik Hadas has submitted this change and it was merged. Change subject: core: move provider+storage related classes to a dedicated package ...................................................................... core: move provider+storage related classes to a dedicated package Change-Id: I7574d5110d2a1ce2e6b82d977c0b8cb964ff0242 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportRepoImageCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/ProviderProxyFactory.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageException.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackImageProviderProxy.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ImportExportRepoImageCommandTest.java 8 files changed, 12 insertions(+), 8 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29378 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7574d5110d2a1ce2e6b82d977c0b8cb964ff0242 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: all-in-one: postpone import for vdscli
by sbonazzo＠redhat.com 12 Aug '14

12 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: all-in-one: postpone import for vdscli ...................................................................... packaging: setup: all-in-one: postpone import for vdscli Postponed import of vdscli in order to not break development installation. Change-Id: I1fa236b356002a913a505e81766564c4ea1524cd Bug-Url: https://bugzilla.redhat.com/1100236 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/all-in-one/vdsmi.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/31395 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1fa236b356002a913a505e81766564c4ea1524cd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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