- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Gluster peer probe and status on same server
by kmayilsa＠redhat.com 19 Dec '13

19 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Gluster peer probe and status on same server ...................................................................... engine: Gluster peer probe and status on same server Gluster peer status and volume info results are served from the local db of the node, and hence the results are not consistent if the commands are executed on different nodes. - Server is set to UP after a gluster peer probe only when it is returned in the peer list from another node. - Changing the up server selection to return the same server to avoid inconsistent results for non distributed gluster commands. Using getRandomUpServer while calling distributed gluster commands. Change-Id: I1359a0c8f270b70a6afd3fd1490e5bb602d9b03f Bug-Url: https://bugzilla.redhat.com/1043032 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterVolumeAdvancedDetailsQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterQueriesCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/tasks/GlusterTasksService.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/ClusterUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GetGlusterVolumeAdvancedDetailsQueryTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJobTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/dbscripts/vds_sp.sql 12 files changed, 88 insertions(+), 11 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/22448 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1359a0c8f270b70a6afd3fd1490e5bb602d9b03f Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Fixed redundant null check find bug error
by yzaslavs＠redhat.com 18 Dec '13

18 Dec '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: Fixed redundant null check find bug error ...................................................................... engine: Fixed redundant null check find bug error Redundant nullcheck of glusterServer, fixed Change-Id: I9652c52e2886a4c225b5a79985f18f7f372dd4e4 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22405 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9652c52e2886a4c225b5a79985f18f7f372dd4e4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-3.3]: packaging: setup: update reports DB schema on major release ...
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: update reports DB schema on major release upgrade ...................................................................... packaging: setup: update reports DB schema on major release upgrade This addition is required because there is a major schema change introduced by Jasper Server in 5.5 compared to 4.7; this is needed for correct operation of export/import functionality. Change-Id: I0f2557aae870c9f9f3c8076abd67b5c72120b279 Bug-Url: https://bugzilla.redhat.com/1043530 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 35 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22549 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f2557aae870c9f9f3c8076abd67b5c72120b279 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[ovirt-3.3]: packaging: setup: update error handling in remote DB flow
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: update error handling in remote DB flow ...................................................................... packaging: setup: update error handling in remote DB flow Change-Id: I60af89aee018eb44a63d38de42f2643c0d850d3e Bug-Url: https://bugzilla.redhat.com/1039946 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 2 insertions(+), 5 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22550 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60af89aee018eb44a63d38de42f2643c0d850d3e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: setup: update error handling in remote DB flow
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: update error handling in remote DB flow ...................................................................... packaging: setup: update error handling in remote DB flow Change-Id: I60af89aee018eb44a63d38de42f2643c0d850d3e Bug-Url: https://bugzilla.redhat.com/1039946 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 2 insertions(+), 5 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22235 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60af89aee018eb44a63d38de42f2643c0d850d3e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: setup: update reports DB schema on major release ...
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: update reports DB schema on major release upgrade ...................................................................... packaging: setup: update reports DB schema on major release upgrade This addition is required because there is a major schema change introduced by Jasper Server in 5.5 compared to 4.7; this is needed for correct operation of export/import functionality. Change-Id: I0f2557aae870c9f9f3c8076abd67b5c72120b279 Bug-Url: https://bugzilla.redhat.com/1043530 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 35 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22548 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f2557aae870c9f9f3c8076abd67b5c72120b279 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-dwh[ovirt-3.3]: packaging: setup: detect existing DB in remote flows
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: detect existing DB in remote flows ...................................................................... packaging: setup: detect existing DB in remote flows Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Bug-Url: https://bugzilla.redhat.com/1026947 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 60 insertions(+), 18 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22546 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: setup: detect existing DB in remote flows
by ydary＠redhat.com 18 Dec '13

18 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: detect existing DB in remote flows ...................................................................... packaging: setup: detect existing DB in remote flows Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Bug-Url: https://bugzilla.redhat.com/1026947 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 60 insertions(+), 18 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22538 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee813572cab7e0afcf5f5dfa97e4121334c3aa5b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Edit Network and apply to hosts at once
by masayag＠redhat.com 18 Dec '13

18 Dec '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Edit Network and apply to hosts at once ...................................................................... engine: Edit Network and apply to hosts at once Editing a network and applying its changes to all of the eligible hosts via a single action is enabled by sync-ing the network on the hosts (if the network wasn't renamed) See: http://www.ovirt.org/Features/MultiHostNetworkConfiguration Bug-Url: http://bugzilla.redhat.com/1040580 Change-Id: Iec04c7fb0c29ba6f61b7d788a9c3917f5d46554e Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/NetworkValidator.java 2 files changed, 148 insertions(+), 5 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22053 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec04c7fb0c29ba6f61b7d788a9c3917f5d46554e Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: When all filters fail, return all hosts
by msivak＠redhat.com 18 Dec '13

18 Dec '13

Martin Sivák has submitted this change and it was merged. Change subject: When all filters fail, return all hosts ...................................................................... When all filters fail, return all hosts Change-Id: I3e88e22795d17dbd6c9dd03e7cc84da31feb483e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1002444 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- A plugins/test_failing_plugin.py M src/ovirtscheduler/request_handler.py M src/ovirtscheduler/request_handler_test.py M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 4 files changed, 64 insertions(+), 22 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved Jiří Moskovčák: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e88e22795d17dbd6c9dd03e7cc84da31feb483e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0