January 2015 - Engine-commits - oVirt List Archives

Change in ovirt-live[ovirt-live-3.5]: build: post ovirt-live-3.5.1
by sbonazzo＠redhat.com 25 Jan '15

25 Jan '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-live-3.5.1 ...................................................................... build: post ovirt-live-3.5.1 Change-Id: I32e922f4c0fdd362e49beecde82a81e8d4053b83 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/Makefile M centos/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37137 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32e922f4c0fdd362e49beecde82a81e8d4053b83 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.5 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: remove PreparingToHibernate status
by ahadas＠redhat.com 25 Jan '15

25 Jan '15

Arik Hadas has submitted this change and it was merged. Change subject: core: remove PreparingToHibernate status ...................................................................... core: remove PreparingToHibernate status In aa548551 the only usage of the VM status PreparingToHibernate was removed. This patch removes this status completely. Change-Id: Id0a8671393ab731bda5e68d4c9fe80995308b031 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/LocalizedVmStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcActionUtils.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VMStatus.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 backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsListFetcher.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/widget/table/column/VmStatusColumn.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabVirtualMachineView.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties A packaging/dbscripts/upgrade/03_06_0790_change_prepare_to_hibernate_to_up.sql 14 files changed, 6 insertions(+), 34 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Eli Mesika: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/37176 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0a8671393ab731bda5e68d4c9fe80995308b031 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: Add support for --type-name options
by juan.hernandez＠redhat.com 24 Jan '15

24 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Add support for --type-name options ...................................................................... cli: Add support for --type-name options Currently when a parent object needs to be specified in a command the only alternative is to use the "--type-identifier" option. This option will locate the parent object that matches the given identifier if it is an UUID, or the name otherwise. The problem is that some parent objects have identifiers that aren't UUIDs, so this won't be able to locate them. To avoid this issue this patch introduces new "--type-name" options that can be used in the same places that the "--type-identifier" options are used. When using the existing "--type-identifier" option the parent object will be located only by id, and when using the "--type-name" option it will be located only by name, thus elimiating any potential ambiguity. Change-Id: I438e0d5e3c29302fda63e0f0ab01e87a6841e10e Bug-Url: https://bugzilla.redhat.com/1177367 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/cli/messages.py M src/ovirtcli/command/action.py M src/ovirtcli/command/add.py M src/ovirtcli/command/command.py M src/ovirtcli/command/remove.py M src/ovirtcli/command/show.py M src/ovirtcli/command/update.py M src/ovirtcli/utils/autocompletionhelper.py 8 files changed, 88 insertions(+), 74 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36535 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I438e0d5e3c29302fda63e0f0ab01e87a6841e10e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: Ondřej Macháček <machacek.ondra(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-cli[master]: cli: Try id, name and alias for show
by juan.hernandez＠redhat.com 24 Jan '15

24 Jan '15

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Try id, name and alias for show ...................................................................... cli: Try id, name and alias for show The fix for bug 1148941 (commit 4b654a2) introduced a regression in the way that the "show" command locates objects, so that objects can't be located by name. For example, the following command will fail to locate the a virtual machine, even if it exists: # show vm myvm This patch fixes that regression, so that when given an object identifier the CLI will try to locate it first by identifier, then by name and finally by alias (for disks). Change-Id: I8d197c042e83939470aea93405546a25bdf3f0c0 Related-To: https://bugzilla.redhat.com/1148941 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/ovirtcli/command/command.py 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36918 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d197c042e83939470aea93405546a25bdf3f0c0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_
by oourfali＠redhat.com 23 Jan '15

23 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ ...................................................................... aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ needed to keep the format shell compatible. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1184895 Change-Id: I3a06518ff7c96ec7483b26591408cd2977636e36 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java M packaging/services/ovirt-engine/ovirt-engine.conf.in 2 files changed, 22 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37238 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a06518ff7c96ec7483b26591408cd2977636e36 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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 jenkins[master]: Fixed triggers for ovirt-node
by dcaroest＠redhat.com 23 Jan '15

23 Jan '15

David Caro has submitted this change and it was merged. Change subject: Fixed triggers for ovirt-node ...................................................................... Fixed triggers for ovirt-node Change-Id: I65ca4b79624b4448a82304ad80f292911c235637 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml A jobs/confs/yaml/triggers/ovirt-node.yaml 3 files changed, 16 insertions(+), 3 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved Anatoly Litovsky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/37220 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65ca4b79624b4448a82304ad80f292911c235637 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Added 3.5 to sdk-java jobs
by dcaroest＠redhat.com 23 Jan '15

23 Jan '15

David Caro has submitted this change and it was merged. Change subject: Added 3.5 to sdk-java jobs ...................................................................... Added 3.5 to sdk-java jobs Change-Id: I445925c5042c804b1c2e536368390e965a0eed98 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37240 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I445925c5042c804b1c2e536368390e965a0eed98 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin : Geo-Replication action confirmation popup
by kmayilsa＠redhat.com 23 Jan '15

23 Jan '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Geo-Replication action confirmation popup ...................................................................... webadmin : Geo-Replication action confirmation popup Geo-Replication action confirmation popup Change-Id: Ib9bf8f15ca617de36b149b17b6ded7a81e52ab4b Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterVolumeGeoRepActionConfirmationModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/PresenterModule.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/gluster/GlusterVolumeGeoRepActionConfirmPopUpViewPresenterWidget.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GeoRepActionConfirmPopUpView.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GeoRepActionConfirmPopUpView.ui.xml 6 files changed, 304 insertions(+), 0 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35224 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib9bf8f15ca617de36b149b17b6ded7a81e52ab4b Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(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]: core: domain might remain locked on deactivation
by tnisan＠redhat.com 22 Jan '15

22 Jan '15

Tal Nisan has submitted this change and it was merged. Change subject: core: domain might remain locked on deactivation ...................................................................... core: domain might remain locked on deactivation When passing the compensation context to the child command the 'recorded' changes aren't reverted becasue when attempting to load the compensation data the load is by the command id which is different between the parent command and the child command. When no tasks are created when deactivating domain and the actual deactivation fails, the domain may remained locked because of the above reason in same usecases (if more updates using the compensation are done in the child command) Change-Id: I4a1bea6522ceb8c83be9466ed50748559745c3c8 Bug-Url: https://bugzilla.redhat.com/1185050 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainWithOvfUpdateCommand.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/37233 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a1bea6522ceb8c83be9466ed50748559745c3c8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(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]: aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_
by oourfali＠redhat.com 22 Jan '15

22 Jan '15

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ ...................................................................... aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ needed to keep the format shell compatible. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1184895 Change-Id: I3a06518ff7c96ec7483b26591408cd2977636e36 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java M packaging/services/ovirt-engine/ovirt-engine.conf.in 2 files changed, 22 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37221 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a06518ff7c96ec7483b26591408cd2977636e36 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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