October 2014 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: otopi-1.3.0
by Alon Bar-Lev 06 Oct '14

06 Oct '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.3.0 ...................................................................... packaging: otopi-1.3.0 Change-Id: Iff6a6c0b92309c61d6d4ffd691a08c7d5af2c506 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff6a6c0b92309c61d6d4ffd691a08c7d5af2c506 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in otopi[master]: packaging: post otopi-1.3.0
by Alon Bar-Lev 06 Oct '14

06 Oct '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.3.0 ...................................................................... packaging: post otopi-1.3.0 Change-Id: Idb55ffce40037323f658929b685c3f97aeea3a03 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33841 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb55ffce40037323f658929b685c3f97aeea3a03 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: config: Expose customizable console keyboard shortcuts
by tnisan＠redhat.com 06 Oct '14

06 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: config: Expose customizable console keyboard shortcuts ...................................................................... config: Expose customizable console keyboard shortcuts Change Icb37fe653f007307686765f6dc089206559cb7e3 generalized naming of configurable console shortcuts. However, exposing to the config was forgotten. This patch fixes the issue. Change-Id: I4b2a9b00ede59c9b57180a1f2f25a08fcaeb2ec8 Bug-Url: https://bugzilla.redhat.com/1141224 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M packaging/etc/engine-config/engine-config.properties 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Frank Kobzik: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33765 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b2a9b00ede59c9b57180a1f2f25a08fcaeb2ec8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: config: Expose customizable console keyboard shortcuts
by ofrenkel＠redhat.com 06 Oct '14

06 Oct '14

Omer Frenkel has submitted this change and it was merged. Change subject: config: Expose customizable console keyboard shortcuts ...................................................................... config: Expose customizable console keyboard shortcuts Change Icb37fe653f007307686765f6dc089206559cb7e3 generalized naming of configurable console shortcuts. However, exposing to the config was forgotten. This patch fixes the issue. Change-Id: I4b2a9b00ede59c9b57180a1f2f25a08fcaeb2ec8 Bug-Url: https://bugzilla.redhat.com/1141224 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M packaging/etc/engine-config/engine-config.properties 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: oVirt Jenkins CI Server: Looks good to me, but someone else must approve Frank Kobzik: Verified Omer Frenkel: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33763 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b2a9b00ede59c9b57180a1f2f25a08fcaeb2ec8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(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]: restapi: Make mapper test more generic
by juan.hernandez＠redhat.com 06 Oct '14

06 Oct '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Make mapper test more generic ...................................................................... restapi: Make mapper test more generic The base class for mapper tests has specific bound for the "from" type parameter: class AbstractInvertibleMappingTest<F extends BaseResource, T, I> ... But this bound isn't used, and it makes this class unusable for things other than resources. This patch removes that bound, so the class can be used with resources and also with other simpler types: class AbstractInvertibleMappingTest<F, T, I> ... Change-Id: I4176cd3a5d7b55ab7cbf00389d5b30a720da66f4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/AbstractInvertibleMappingTest.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33831 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4176cd3a5d7b55ab7cbf00389d5b30a720da66f4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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-engine[master]: core: Changed the PhysDevInitializationError msg
by amureini＠redhat.com 06 Oct '14

06 Oct '14

Allon Mureinik has submitted this change and it was merged. Change subject: core: Changed the PhysDevInitializationError msg ...................................................................... core: Changed the PhysDevInitializationError msg Previously, the vdsm PhysDevInitializationError error message did not include the case that the device is not accessible by the host. The error message was changed to include this case. Change-Id: I528544cffa7c82958a18912461840ae276619bfc Bug-Url: https://bugzilla.redhat.com/974859 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Idan Shaby: Verified -- To view, visit http://gerrit.ovirt.org/33824 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I528544cffa7c82958a18912461840ae276619bfc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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[ovirt-engine-3.5]: restapi: Fix use of "unregistered" parameter
by tnisan＠redhat.com 06 Oct '14

06 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Fix use of "unregistered" parameter ...................................................................... restapi: Fix use of "unregistered" parameter The "unregister" matrix parameter is used to include unregistered objects (disks, templates and VMs) when listing or adding them. But the paremeter is currently documented incorrectly, as it is missing from some operations where it is used and included in other operations where it isn't used. As a result the RSDL is incorrect and the Python and Java SDKs don't implement it correctly. This patch fixes the documentation of the parameter. Change-Id: Ifccc9082d71cc6bb14ae9dfc7efd3c024a82639f Bug-Url: https://bugzilla.redhat.com/1148715 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 29012b25c07d964d8f7aeeba7b6703bf452ae6b9) --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33827 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifccc9082d71cc6bb14ae9dfc7efd3c024a82639f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: retry fencing operation in case of failure
by emesika＠redhat.com 06 Oct '14

06 Oct '14

Eli Mesika has submitted this change and it was merged. Change subject: core: retry fencing operation in case of failure ...................................................................... core: retry fencing operation in case of failure retry fencing operation in case of a temporary failure Up to now, if 'unknown' was returned while waiting for a certain status ('on' / 'off') , the while loop was exited with a proper message, breaking the reboot sequence. It seems that there are some cases in which a fencing operation may fail temporarily and succeeded after a while. This patch retries to get to the desired status 3 times before giving up with the configured delay between each status check. Change-Id: Iab5dadd5cb07f2c61dfdf6d75e9fccca790ba483 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1140098 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java 1 file changed, 13 insertions(+), 3 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33486 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab5dadd5cb07f2c61dfdf6d75e9fccca790ba483 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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-release[master]: build: ovirt-release35-001
by sbonazzo＠redhat.com 06 Oct '14

06 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release35-001 ...................................................................... build: ovirt-release35-001 Change-Id: I72b090ce8b5d6fb19bd5122292b4ccdfe5a8e0b0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-3.5/configure.ac M ovirt-release-3.5/ovirt-release35.spec.in M ovirt-release-3.5/ovirt.repo.in 3 files changed, 9 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33397 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72b090ce8b5d6fb19bd5122292b4ccdfe5a8e0b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: 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-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: build: ovirt-hosted-engine-ha-1.2.3
by msivak＠redhat.com 06 Oct '14

06 Oct '14

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.2.3 ...................................................................... build: ovirt-hosted-engine-ha-1.2.3 Change-Id: Ibdbbbdf5e50ab441c544374a5de416a70affcb2b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 8 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33830 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibdbbbdf5e50ab441c544374a5de416a70affcb2b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0