September 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Avoid starting vm failed if pin to host is removed
by rgolan＠redhat.com 22 Sep '14

22 Sep '14

Roy Golan has submitted this change and it was merged. Change subject: core: Avoid starting vm failed if pin to host is removed ...................................................................... core: Avoid starting vm failed if pin to host is removed If the vm configured vm numa nodes and pinned them to host numa nodes, when the host was removed,the pinned host numa nodes would be removed also. Now if you try to run the vm, it would fail. This patch will avoid this situation. Change-Id: Icbbfd981d0996689cdc818bdd02e31155799d55c Bug-Url: https://bugzilla.redhat.com/1101387 Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NumaUtils.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, but someone else must approve Roy Golan: Looks good to me, approved Omer Frenkel: Looks good to me, but someone else must approve Xiaolei Shi: Verified -- To view, visit http://gerrit.ovirt.org/28166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbbfd981d0996689cdc818bdd02e31155799d55c Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Collect vm numa node runtime information
by rgolan＠redhat.com 22 Sep '14

22 Sep '14

Roy Golan has submitted this change and it was merged. Change subject: core: Collect vm numa node runtime information ...................................................................... core: Collect vm numa node runtime information Collect vm numa node runtime pinning to which host numa nodes data. This information is calculated in vdsm according to each vcpu pinning to which physical cpu core and the process(vcpu) consumed memory mapping to which host numa nodes. The reported message format from vdsm is: 'vNodeRuntimeInfo': {'<vmNodeIndex>': [<hostNodeIndex>], ...} Change-Id: I9d2c1dd03e0cc2f7d0ed9593e6ff889712894ac7 Bug-Url: https://bugzilla.redhat.com/1100202 Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatistics.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsNumaNodeDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmNumaNodeDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmNumaNodeDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmNumaNodeDAOTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M packaging/dbscripts/numa_sp.sql 11 files changed, 239 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified Roy Golan: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28096 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d2c1dd03e0cc2f7d0ed9593e6ff889712894ac7 Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: jsonrpc version update
by oourfali＠redhat.com 22 Sep '14

22 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: jsonrpc version update ...................................................................... packaging: jsonrpc version update Bug-Url: https://bugzilla.redhat.com/1143860 Change-Id: I3abe90cf2201e87ba3869c6c2a7abd2a30152723 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33175 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3abe90cf2201e87ba3869c6c2a7abd2a30152723 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Update AddStorageDomainCommonTest to succeed when stor...
by tnisan＠redhat.com 22 Sep '14

22 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Update AddStorageDomainCommonTest to succeed when storage pool is null ...................................................................... core: Update AddStorageDomainCommonTest to succeed when storage pool is null Change-Id: I5ba3a0f2c7c08c1592a5791784ca5023b61a156f Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommonTest.java 1 file changed, 2 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33173 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ba3a0f2c7c08c1592a5791784ca5023b61a156f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: version: add FEATURES
by fabiand＠redhat.com 22 Sep '14

22 Sep '14

Fabian Deutsch has submitted this change and it was merged. Change subject: version: add FEATURES ...................................................................... version: add FEATURES adds ability to detect specific features by other components. this will enable simple enable/disable of features at node level, and applies into the upper layers. Change-Id: Ic8c22deb05b76cc588c1279700a6e0151ad0042f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M version.ovirt-node-plugin-vdsm.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Alon Bar-Lev: Verified -- To view, visit http://gerrit.ovirt.org/33067 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8c22deb05b76cc588c1279700a6e0151ad0042f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: jsonrpc version update
by oourfali＠redhat.com 22 Sep '14

22 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: jsonrpc version update ...................................................................... packaging: jsonrpc version update Bug-Url: https://bugzilla.redhat.com/1143860 Change-Id: I3abe90cf2201e87ba3869c6c2a7abd2a30152723 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33169 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3abe90cf2201e87ba3869c6c2a7abd2a30152723 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Remove storage format validation when adding unattache...
by tnisan＠redhat.com 22 Sep '14

22 Sep '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove storage format validation when adding unattached data domain ...................................................................... core: Remove storage format validation when adding unattached data domain Remove storage format validation when adding an unattached data domain with a host that doesn't have a data center Change-Id: I4feea2d476e89e89d2b1ad572ce64904063a0914 Bug-Url: https://bugzilla.redhat.com/1141784 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33140 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4feea2d476e89e89d2b1ad572ce64904063a0914 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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]: core: allow 2% gap in host low swap
by oourfali＠redhat.com 22 Sep '14

22 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: allow 2% gap in host low swap ...................................................................... core: allow 2% gap in host low swap This patch change the computation of low swap memory such that it allows a 2% gap between the configured low swap value (LogPhysicalMemoryThresholdInMB) and the actual one in order to prevent th engine log to be flooded with warning when the actual value is OK. Change-Id: Ia3f5da5398141eeba83549a7e425e9f6b9f8673e Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33164 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3f5da5398141eeba83549a7e425e9f6b9f8673e 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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Adding possibility to specify multilevel options
by dcaroest＠redhat.com 22 Sep '14

22 Sep '14

David Caro has submitted this change and it was merged. Change subject: Adding possibility to specify multilevel options ...................................................................... Adding possibility to specify multilevel options For example environment vars with: --option environment.MYVAR=MYVALUE Change-Id: I660286c5fe592f8cda75b7880a16e0e4fc6f2003 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/mock_genconfig 1 file changed, 11 insertions(+), 3 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33160 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I660286c5fe592f8cda75b7880a16e0e4fc6f2003 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Add numa node index when create vm numa node
by rgolan＠redhat.com 22 Sep '14

22 Sep '14

Roy Golan has submitted this change and it was merged. Change subject: core: Add numa node index when create vm numa node ...................................................................... core: Add numa node index when create vm numa node Change-Id: I6a6f3f4c4f69dff73ef8df5dc8b6d9f4ed12ec28 Bug-Url: https://bugzilla.redhat.com/1099814 Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/NumaSettingFactory.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/NumaSettingFactoryTest.java 2 files changed, 3 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, but someone else must approve Roy Golan: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27988 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6a6f3f4c4f69dff73ef8df5dc8b6d9f4ed12ec28 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0