- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin, engine: Gluster domain edit enhancement.
by tnisan＠redhat.com 25 Feb '15

25 Feb '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin, engine: Gluster domain edit enhancement. ...................................................................... webadmin, engine: Gluster domain edit enhancement. Gluster domain's connection and mount options editing was disabled in webadmin and engine. This patch adds the possibility to edit it. Change-Id: I7320520a15fe804a3a560bf1b8223bfee5b1b46a Bug-Url: https://bugzilla.redhat.com/1168686 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommand.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 2 files changed, 3 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Amit Aviram: Verified -- To view, visit https://gerrit.ovirt.org/37894 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7320520a15fe804a3a560bf1b8223bfee5b1b46a Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Refactoring GlusterModel to inherit from PosixModel.
by tnisan＠redhat.com 25 Feb '15

25 Feb '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Refactoring GlusterModel to inherit from PosixModel. ...................................................................... webadmin: Refactoring GlusterModel to inherit from PosixModel. Since GlusterStorageModel duplicates most of PosixStorageModel's content, and Gluster domain is actually a Posix storage domain, it is now inherits from PosixStorageModel. Change-Id: I7a355956f0f30a79f13d0bd72ec7a4d65cc29d1a Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/GlusterStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/PosixStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 3 files changed, 35 insertions(+), 273 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Amit Aviram: Verified -- To view, visit https://gerrit.ovirt.org/37893 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a355956f0f30a79f13d0bd72ec7a4d65cc29d1a Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(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]: engine: Correct user connected/disconnected messages
by ahadas＠redhat.com 25 Feb '15

25 Feb '15

Arik Hadas has submitted this change and it was merged. Change subject: engine: Correct user connected/disconnected messages ...................................................................... engine: Correct user connected/disconnected messages Message 'User disconnected' was added to Action Log when user had connected to a VM and vice versa. The bug was in VdsEventListener.processOnClientIpChange() method called from VmsMonitoring.afterVMsRefreshTreatment() method. processOnClientIpChange() was reading the client IP from DB, but IP address of the currently connecting client is written to DB after calling this method (saveVmsToDb() call at the end of afterVMsRefreshTreatment() method). To correct this additional parameter was added to processOnClientIpChange() method to pass IP address of the currently connecting client. Change-Id: Icce32619e3cc88f12e339b4eef5717f3d279e232 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java 3 files changed, 10 insertions(+), 6 deletions(-) Approvals: Shmuel Leib Melamud: Verified Omer Frenkel: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38070 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icce32619e3cc88f12e339b4eef5717f3d279e232 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Changing ovirt-node scms to work on git protocol
by tlitovsk＠redhat.com 25 Feb '15

25 Feb '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Changing ovirt-node scms to work on git protocol ...................................................................... Changing ovirt-node scms to work on git protocol Change-Id: If38b9d369a3b0c597002cd07cd0063ccd9e75188 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/yaml/scms/ovirt-node-scms.yaml 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: David Caro: Looks good to me, but someone else must approve Anatoly Litovsky: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38144 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If38b9d369a3b0c597002cd07cd0063ccd9e75188 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: host-deploy: simplify remote bootstarp command
by oourfali＠redhat.com 25 Feb '15

25 Feb '15

Oved Ourfali has submitted this change and it was merged. Change subject: host-deploy: simplify remote bootstarp command ...................................................................... host-deploy: simplify remote bootstarp command + add -t parameter dropped in I9c780005a4. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1195672 Change-Id: I0c3260f61b0acbd914d587a59e610a9d627227b3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38125 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c3260f61b0acbd914d587a59e610a9d627227b3 Gerrit-PatchSet: 2 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: Moti Asayag <masayag(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 a couple typos
by dcaroest＠redhat.com 25 Feb '15

25 Feb '15

David Caro has submitted this change and it was merged. Change subject: Fixed a couple typos ...................................................................... Fixed a couple typos Change-Id: Iba640b28e7558ebd14d97ddee50c210b5e00e710 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/README 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Eyal Edri: Looks good to me, approved David Caro: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38110 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba640b28e7558ebd14d97ddee50c210b5e00e710 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Quartz cron scheduler to support startAt and endBy
by kmayilsa＠redhat.com 25 Feb '15

25 Feb '15

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Quartz cron scheduler to support startAt and endBy ...................................................................... engine: Quartz cron scheduler to support startAt and endBy Modified the Quartz scheduler to support start at and end by dates for cron based scheduler Change-Id: I5c3372098177dabbbe886660c9d7a586784b99be Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/scheduler/src/main/java/org/ovirt/engine/core/utils/timer/DBSchedulerUtilQuartzImpl.java M backend/manager/modules/scheduler/src/main/java/org/ovirt/engine/core/utils/timer/SchedulerUtil.java M backend/manager/modules/scheduler/src/main/java/org/ovirt/engine/core/utils/timer/SchedulerUtilBaseImpl.java 3 files changed, 90 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/37533 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5c3372098177dabbbe886660c9d7a586784b99be Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5.2]: setup: checking if command_entities table exist
by oourfali＠redhat.com 25 Feb '15

25 Feb '15

Oved Ourfali has submitted this change and it was merged. Change subject: setup: checking if command_entities table exist ...................................................................... setup: checking if command_entities table exist oVirt 3.5 has introduced command_entities table. In addition to adding some new functions related to this table to the task clenaer utility, some functions got changed, but as it is run during setup and might run on top of version which preceeds oVirt 3.5, it should be checked in the relevant SPs that the table exists. Bug-Url: https://bugzilla.redhat.com/1161012 Bug-Url: https://bugzilla.redhat.com/1164771 Change-Id: I3152c4d3f6e02915053ada5019abeea7cb356751 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> (cherry picked from commit 3ba84df3d9e6597e131c3c9882e6d7136a54e296) --- M packaging/setup/dbutils/taskcleaner.sh M packaging/setup/dbutils/taskcleaner_sp.sql A packaging/setup/dbutils/taskcleaner_sp_3_5.sql 3 files changed, 107 insertions(+), 81 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38157 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3152c4d3f6e02915053ada5019abeea7cb356751 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5.2 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: setup: checking if command_entities table exist
by oourfali＠redhat.com 25 Feb '15

25 Feb '15

Oved Ourfali has submitted this change and it was merged. Change subject: setup: checking if command_entities table exist ...................................................................... setup: checking if command_entities table exist oVirt 3.5 has introduced command_entities table. In addition to adding some new functions related to this table to the task clenaer utility, some functions got changed, but as it is run during setup and might run on top of version which preceeds oVirt 3.5, it should be checked in the relevant SPs that the table exists. Bug-Url: https://bugzilla.redhat.com/1161012 Bug-Url: https://bugzilla.redhat.com/1164771 Change-Id: I3152c4d3f6e02915053ada5019abeea7cb356751 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M packaging/setup/dbutils/taskcleaner.sh M packaging/setup/dbutils/taskcleaner_sp.sql A packaging/setup/dbutils/taskcleaner_sp_3_5.sql 3 files changed, 107 insertions(+), 81 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Looks good to me, but someone else must approve oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yair Zaslavsky: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/37847 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3152c4d3f6e02915053ada5019abeea7cb356751 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[master]: sdk: Regenerate against the latest API
by juan.hernandez＠redhat.com 25 Feb '15

25 Feb '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate against the latest API ...................................................................... sdk: Regenerate against the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit 7984e91. Change-Id: Ieabdb1749dd2d894743e48b04ca687f216a91d0f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M generator/src/main/resources/api.xsd M generator/src/main/resources/rsdl.xml M setup.py M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 7 files changed, 11,537 insertions(+), 2,774 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/37869 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieabdb1749dd2d894743e48b04ca687f216a91d0f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0