August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-scheduler-proxy[master]: Initial packaging
by lhornyak＠redhat.com 12 Aug '13

12 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: Initial packaging ...................................................................... Initial packaging This patch introduces the initial version of the packaging stuff. Change-Id: I8aa86f5ee80f40eb25a719fd72e75f7b275f38a0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M .gitignore M Makefile A ovirt-scheduler-proxy.spec A packaging/ovirt-scheduler-proxy.systemd A packaging/ovirt-scheduler-proxy.sysv A setup.py R src/ovirtscheduler/API.py R src/ovirtscheduler/__init__.py R src/ovirtscheduler/loader.py R src/ovirtscheduler/oschedproxyd.py R src/ovirtscheduler/request_handler.py R src/ovirtscheduler/request_handler_test.py R src/ovirtscheduler/runner.py R src/ovirtscheduler/runner_test.py R src/ovirtscheduler/utils.py 15 files changed, 333 insertions(+), 4 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17955 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8aa86f5ee80f40eb25a719fd72e75f7b275f38a0 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: fixed edit name availability in tree context
by ecohen＠redhat.com 12 Aug '13

12 Aug '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin: fixed edit name availability in tree context ...................................................................... webadmin: fixed edit name availability in tree context When a specific item (for example, an actual host -- not "Hosts") is selected in the left-pane tree, we need to disable editing the name of that item. Change-Id: Ie50a6fcfeacdc94fe900f6569180106c31f5e9c8 Bug-Url: https://bugzilla.redhat.com/858952 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 5 files changed, 32 insertions(+), 11 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17886 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie50a6fcfeacdc94fe900f6569180106c31f5e9c8 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: fixed action availability in tree context
by ecohen＠redhat.com 12 Aug '13

12 Aug '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin: fixed action availability in tree context ...................................................................... webadmin: fixed action availability in tree context When a specific item (for example, an actual host -- not "Hosts") is selected in the left-pane tree, we need to hide all flavors of the "new" and "remove" buttons / context menu items. Change-Id: I846346e683db7210753c4e85426a210195509835 Bug-Url: https://bugzilla.redhat.com/858952 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/networks/NetworkListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 4 files changed, 36 insertions(+), 18 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17882 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I846346e683db7210753c4e85426a210195509835 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: history: added boolean on guest logged in user
by emesika＠redhat.com 12 Aug '13

12 Aug '13

Eli Mesika has submitted this change and it was merged. Change subject: history: added boolean on guest logged in user ...................................................................... history: added boolean on guest logged in user Change-Id: Ifcf84fe6ce81b5772acc53a6f999f242dd68c2e3 Bug-Url: https://bugzilla.redhat.com/893810 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17935 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcf84fe6ce81b5772acc53a6f999f242dd68c2e3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: etl: added boolean on logged in guest user
by emesika＠redhat.com 12 Aug '13

12 Aug '13

Eli Mesika has submitted this change and it was merged. Change subject: etl: added boolean on logged in guest user ...................................................................... etl: added boolean on logged in guest user Change-Id: Idf274828f4aa33ba96f66656b526d0ef608a32ff Bug-Url: https://bugzilla.redhat.com/893810 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M data-warehouse/history_etl/etl_sources/historyETLProcedure/src/main/java/ovirt_engine_dwh/aggregationtodaily_3_3/AggregationToDaily.java M data-warehouse/history_etl/etl_sources/historyETLProcedure/src/main/java/ovirt_engine_dwh/aggregationtohourly_3_3/AggregationToHourly.java M data-warehouse/history_etl/etl_sources/historyETLProcedure/src/main/java/ovirt_engine_dwh/statisticssync_3_3/StatisticsSync.java A data-warehouse/history_etl/tos_project/configuration/ExpressionBuilder/AggregationToDaily.xml M data-warehouse/history_etl/tos_project/configuration/ExpressionBuilder/AggregationToHourly.xml M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.3.item M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.3.properties M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.3.item M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.3.properties M data-warehouse/history_etl/tos_project/process/StatisticsSync_3.3.item M data-warehouse/history_etl/tos_project/process/StatisticsSync_3.3.properties M data-warehouse/history_etl/tos_project/talend.project A data-warehouse/historydbscripts_postgres/upgrade/03_03_0040_add_vm_guest_user_login_info.sql 13 files changed, 2,446 insertions(+), 400 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Yaniv Dary: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17936 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf274828f4aa33ba96f66656b526d0ef608a32ff Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: set cd-rom path after vds is set on run vm
by ofrenkel＠redhat.com 12 Aug '13

12 Aug '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: set cd-rom path after vds is set on run vm ...................................................................... core: set cd-rom path after vds is set on run vm As a preliminary step in making IsoPrefixVDSCommand a VdsBrokerCommand instead of IrsBrokerCommand, the cd-rom path setting on run VM is moved to be made after the destination host on which the VM will be run on is set, so that we'll be able to query the iso prefix (which is needed for cd-rom path setting) from that host instead of from the SPM. This patch also changes the way the ISO path is determined: instead of going over the possible values from the one with the lowest priotity to the one with the highest priority and override the previous value each time we find a valid value, we'll go over the possible values in the reverse order and when finding a valid value we'll set the ISO path to it and stop - it simplifies the code and it is more efficient that way. Change-Id: Ic7919e224c093d86a2b1ebf564f0307d5dd813a7 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 33 insertions(+), 27 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17746 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7919e224c093d86a2b1ebf564f0307d5dd813a7 Gerrit-PatchSet: 4 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: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix compareTo for SRV records
by yzaslavs＠redhat.com 12 Aug '13

12 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fix compareTo for SRV records ...................................................................... core: Fix compareTo for SRV records This suggested implemenation is more natural as it is based on substraction of two values which is commonly used in comparisson Bug-Url: https://bugzilla.redhat.com/985623 Change-Id: Iff93a85f529633934515015653c517500bf51c6b Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/dns/DnsSRVLocator.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/utils/dns/DnsSRVLocatorTest.java 2 files changed, 3 insertions(+), 8 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff93a85f529633934515015653c517500bf51c6b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Adding unit test for DnsLocator
by yzaslavs＠redhat.com 12 Aug '13

12 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Adding unit test for DnsLocator ...................................................................... core: Adding unit test for DnsLocator The purpose of the second test is to cause contract breakage of compareTo, based on data attached to the bug Bug-Url: https://bugzilla.redhat.com/985623 Change-Id: Ied8ca585b38d624c04b3f232833e1b34ad5dfab5 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- A backend/manager/tools/src/test/java/org/ovirt/engine/core/utils/dns/DnsSRVLocatorTest.java 1 file changed, 84 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17048 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied8ca585b38d624c04b3f232833e1b34ad5dfab5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: refactor DnsSrvLocator to allow mocking
by yzaslavs＠redhat.com 12 Aug '13

12 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: refactor DnsSrvLocator to allow mocking ...................................................................... core: refactor DnsSrvLocator to allow mocking This patch refactors DnsSrvLocator in order to allow a mocking in a way that a list of SRV records can be passed to the mock. Change-Id: Ic1f04c62ff4206aed0ef5b40ff14163be7b145f3 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/dns/DnsSRVLocator.java 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17047 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1f04c62ff4206aed0ef5b40ff14163be7b145f3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Revert "restapi: RSDL - add 'deprecated' option"
by oliel＠redhat.com 12 Aug '13

12 Aug '13

Ori Liel has submitted this change and it was merged. Change subject: Revert "restapi: RSDL - add 'deprecated' option" ...................................................................... Revert "restapi: RSDL - add 'deprecated' option" This reverts commit 116071a9549101f9f29f650998fcbcfaded02d82. Reason for revert: The patch will be resubmitted shortly with a small addition. I prefer it this way than to have the change split across two patches. Change-Id: Ie4c0b203204eaaa2a1a90f166f769c55c8d861f5 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/rsdl/ParamData.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/rsdl/RsdlBuilder.java 3 files changed, 12 insertions(+), 40 deletions(-) Approvals: Ori Liel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17954 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4c0b203204eaaa2a1a90f166f769c55c8d861f5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0