June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: events: use separate queue for events
by masayag＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: core [DB]: Append database update script for KSM with NUMA
by rgolan＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: webadmin: NPE in HostPopupView (getGroupedList) if cluster h...
by tjelinek＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: core: support cephx authentication on run vm
by derez＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: scheduling: Add HostDevice Policy Unit
by ofrenkel＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: core: Fix disk storage type in OVF
by mlipchuk＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: engine: Add KSM policy to NUMA hosts
by rgolan＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: core: calculate console_client_resources_url if not provided
by tjelinek＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in ovirt-engine[master]: restapi: Avoid NPE when receiving NUMA node without cores
by juan.hernandez＠redhat.com 15 Jun '15

15 Jun '15

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

1 0

Change in jenkins[master]: deploying new YAML conf: disable removed jobs before deletion
by eedri＠redhat.com 15 Jun '15

15 Jun '15

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>

1 0