Change in ovirt-engine[master]: core: Avoid starting vm failed if pin to host is removed
by rgolan@redhat.com
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
10 years, 7 months
Change in ovirt-engine[master]: core: Collect vm numa node runtime information
by rgolan@redhat.com
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
10 years, 7 months
Change in ovirt-engine[ovirt-engine-3.5]: packaging: jsonrpc version update
by oourfali@redhat.com
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
10 years, 7 months
Change in ovirt-engine[master]: core: Update AddStorageDomainCommonTest to succeed when stor...
by tnisan@redhat.com
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
10 years, 7 months
Change in ovirt-node-plugin-vdsm[master]: version: add FEATURES
by fabiand@redhat.com
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
10 years, 7 months
Change in ovirt-engine[master]: packaging: jsonrpc version update
by oourfali@redhat.com
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
10 years, 7 months
Change in ovirt-engine[master]: core: Remove storage format validation when adding unattache...
by tnisan@redhat.com
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
10 years, 7 months
Change in ovirt-engine[master]: core: allow 2% gap in host low swap
by oourfali@redhat.com
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
10 years, 7 months
Change in jenkins[master]: Adding possibility to specify multilevel options
by dcaroest@redhat.com
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>
10 years, 7 months
Change in ovirt-engine[master]: core: Add numa node index when create vm numa node
by rgolan@redhat.com
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
10 years, 7 months