Change in ovirt-engine[master]: events: use separate queue for events
by masayag@redhat.com
Moti Asayag has submitted this change and it was merged.
Change subject: events: use separate queue for events
......................................................................
events: use separate queue for events
Change-Id: I2ff65e274d03a8059130fbd9baca1320a6524aa3
Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/TransportFactory.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcUtils.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcIntegrationTest.java
M ovirt-engine.spec.in
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
M pom.xml
10 files changed, 81 insertions(+), 13 deletions(-)
Approvals:
Piotr Kliczewski: Verified
Jenkins CI: Looks good to me, but someone else must approve
Moti Asayag: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/41796
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2ff65e274d03a8059130fbd9baca1320a6524aa3
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: core [DB]: Append database update script for KSM with NUMA
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: core [DB]: Append database update script for KSM with NUMA
......................................................................
core [DB]: Append database update script for KSM with NUMA
Fix broken patch that mistekenly deleted this script in
https://gerrit.ovirt.org/#/c/39864/
Change-Id: I1c716c866f41f139551e74b2748f12c326f67619
Bug-Url: https://bugzilla.redhat.com/840114
Signed-off-by: Dudi Maroshi <dudi(a)redhat.com>
---
A packaging/dbscripts/upgrade/03_06_1600_add_ksm_with_numa_awareness.sql
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Dudi Maroshi: Verified
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42382
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1c716c866f41f139551e74b2748f12c326f67619
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: webadmin: NPE in HostPopupView (getGroupedList) if cluster h...
by tjelinek@redhat.com
Tomas Jelinek has submitted this change and it was merged.
Change subject: webadmin: NPE in HostPopupView (getGroupedList) if cluster has no DC
......................................................................
webadmin: NPE in HostPopupView (getGroupedList) if cluster has no DC
If the cluster had no DC, HostPopupView threw an NPE on
result.put(currentDataCenter, currentClusterList);
This can happe during force remove of a DC which had some clusters.
Change-Id: Ife9d62ed325c35781bff5058564c326bfa45199c
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Verified
Alexander Wels: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42388
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ife9d62ed325c35781bff5058564c326bfa45199c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: core: support cephx authentication on run vm
by derez@redhat.com
Daniel Erez has submitted this change and it was merged.
Change subject: core: support cephx authentication on run vm
......................................................................
core: support cephx authentication on run vm
Adding 'Auth' element to disk upon running a VM
with a CinderDisk.
Change-Id: I38ec477991a5b6e1f1ebb444a87f40de445d4ef1
Bug-Url: https://bugzilla.redhat.com/1185826
Signed-off-by: Daniel Erez <derez(a)redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java
2 files changed, 20 insertions(+), 0 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/41566
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I38ec477991a5b6e1f1ebb444a87f40de445d4ef1
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: scheduling: Add HostDevice Policy Unit
by ofrenkel@redhat.com
Omer Frenkel has submitted this change and it was merged.
Change subject: scheduling: Add HostDevice Policy Unit
......................................................................
scheduling: Add HostDevice Policy Unit
Added policy unit to filter hosts based on the new 'hostdevPassthrough'
flag in cases when the VM needs host devices.
Change-Id: I3e048fe3a9fc47ddbff0a8eb9e7a195b20465205
Signed-off-by: Martin Betak <mbetak(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/PolicyUnitImpl.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/HostDeviceFilterPolicyUnit.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
A packaging/dbscripts/upgrade/03_06_1590_add_host_device_policy_unit.sql
8 files changed, 75 insertions(+), 0 deletions(-)
Approvals:
Martin Sivák: Looks good to me, but someone else must approve
Jenkins CI: Looks good to me, but someone else must approve
Martin Betak: Verified
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42077
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3e048fe3a9fc47ddbff0a8eb9e7a195b20465205
Gerrit-PatchSet: 15
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(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: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: core: Fix disk storage type in OVF
by mlipchuk@redhat.com
Maor Lipchuk has submitted this change and it was merged.
Change subject: core: Fix disk storage type in OVF
......................................................................
core: Fix disk storage type in OVF
Fix the read and the write in the snapshot's OVF.
Change-Id: If73ffd17db43239ebfbb25ffbaf0219a9dd40815
Bug-Url: https://bugzilla.redhat.com/1185826
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmWriter.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java
3 files changed, 4 insertions(+), 9 deletions(-)
Approvals:
Daniel Erez: Looks good to me, approved
Maor Lipchuk: Verified
--
To view, visit https://gerrit.ovirt.org/42374
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If73ffd17db43239ebfbb25ffbaf0219a9dd40815
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: engine: Add KSM policy to NUMA hosts
by rgolan@redhat.com
Roy Golan has submitted this change and it was merged.
Change subject: engine: Add KSM policy to NUMA hosts
......................................................................
engine: Add KSM policy to NUMA hosts
Fix skipped code, while rebase in patch https://gerrit.ovirt.org/#/c/39783/
from patch-set 11 to 12, patch-set 12 had an erronous rebase.
Bug-Url: https://bugzilla.redhat.com/840114
Change-Id: I23aee2305902c0dcfc4dbf89221e4c60a4acf602
Signed-off-by: Dudi Maroshi <dudi(a)redhat.com>
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
1 file changed, 7 insertions(+), 2 deletions(-)
Approvals:
Dudi Maroshi: Verified
Juan Hernandez: Looks good to me, but someone else must approve
Roy Golan: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42373
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I23aee2305902c0dcfc4dbf89221e4c60a4acf602
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: core: calculate console_client_resources_url if not provided
by tjelinek@redhat.com
Tomas Jelinek has submitted this change and it was merged.
Change subject: core: calculate console_client_resources_url if not provided
......................................................................
core: calculate console_client_resources_url if not provided
Change-Id: I41542d6653f3102cd3908bddeb25b3277dd219bc
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M backend/manager/modules/bll/pom.xml
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ConfigureConsoleOptionsQuery.java
M ear/src/main/application/META-INF/jboss-deployment-structure.xml
3 files changed, 17 insertions(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Verified
Martin Peřina: Looks good to me, but someone else must approve
Jenkins CI: Looks good to me, but someone else must approve
Arik Hadas: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/42273
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I41542d6653f3102cd3908bddeb25b3277dd219bc
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in ovirt-engine[master]: restapi: Avoid NPE when receiving NUMA node without cores
by juan.hernandez@redhat.com
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Avoid NPE when receiving NUMA node without cores
......................................................................
restapi: Avoid NPE when receiving NUMA node without cores
Currently the server assumes the presence of the "cores" element when
receiving a request to add a NUMA node and if they aren't present a NPE
is generated. To avoid this issue this patch modifies the server so that
it checks for the presence of this element.
Change-Id: I12bbdd066af9818dcb2ffad5df8ac3849e3efdd6
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NumaMapper.java
1 file changed, 11 insertions(+), 4 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved
Jenkins CI: Verified; Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/42039
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I12bbdd066af9818dcb2ffad5df8ac3849e3efdd6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Jason Liao <chuan.liao(a)hp.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
9 years, 6 months
Change in jenkins[master]: deploying new YAML conf: disable removed jobs before deletion
by eedri@redhat.com
Eyal Edri has submitted this change and it was merged.
Change subject: deploying new YAML conf: disable removed jobs before deletion
......................................................................
deploying new YAML conf: disable removed jobs before deletion
jobs that were removed from YAML conf would be:
- disabled
- renamed with "_archived_for_deletion"
allow dry run by setting params['ARCHIVE_JOBS'] to false
Change-Id: Idc64b899ba4fc7be80e26ef243a2ea1551bc0e5f
Signed-off-by: Paz Dangur <pdangur(a)redhat.com>
---
A jobs/confs/groovy-scripts/archive_jobs.groovy
A jobs/confs/groovy-scripts/archive_jobs.postbuild.groovy
A jobs/confs/shell-scripts/jenkins_cleanup_yaml.sh
M jobs/confs/yaml/jobs/jenkins/jenkins_deploy_yamls.yaml
4 files changed, 142 insertions(+), 0 deletions(-)
Approvals:
Eyal Edri: Looks good to me; Ready for merge; Passed CI tests
Paz Dangur: Verified
--
To view, visit https://gerrit.ovirt.org/40781
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idc64b899ba4fc7be80e26ef243a2ea1551bc0e5f
Gerrit-PatchSet: 22
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Paz Dangur <pdangur(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com>
Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com>
Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com>
9 years, 6 months