June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-dwh[master]: history: Collect additional host hardware details
by sradco＠redhat.com 10 Jun '15

10 Jun '15

Shirly Radco has submitted this change and it was merged. Change subject: history: Collect additional host hardware details ...................................................................... history: Collect additional host hardware details Change-Id: I9a1c44f8870dafe7aa03c681cd097b9e7b17972b Bug-Url:https://bugzilla.redhat.com/1209931 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_views_3_6.sql A packaging/dbscripts/upgrade/03_06_0030_added_host_hardware_details.sql M tos_project/process/ConfigurationSync_3.6.item M tos_project/process/ConfigurationSync_3.6.properties M tos_project/process/HostConfigCheck_3.6.item M tos_project/process/HostConfigCheck_3.6.properties 6 files changed, 390 insertions(+), 115 deletions(-) Approvals: Shirly Radco: 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/41641 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a1c44f8870dafe7aa03c681cd097b9e7b17972b Gerrit-PatchSet: 10 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: history: Added vm chached/buffered memory to dwh views
by oourfali＠redhat.com 10 Jun '15

10 Jun '15

Oved Ourfali has submitted this change and it was merged. Change subject: history: Added vm chached/buffered memory to dwh views ...................................................................... history: Added vm chached/buffered memory to dwh views Change-Id: I592a31eb1abe6adc4e2edf200cfba04470962219 Bug-Url:https://bugzilla.redhat.com/1209910 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Shirly Radco: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41503 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I592a31eb1abe6adc4e2edf200cfba04470962219 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: pki: add pki-enroll-openssh-cert functionality
by sbonazzo＠redhat.com 10 Jun '15

10 Jun '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: pki: add pki-enroll-openssh-cert functionality ...................................................................... pki: add pki-enroll-openssh-cert functionality requires for future usage of openssh certificates. Change-Id: Ide34d4f0ffadf70e6ce42ae57328e262a832a3e4 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in A packaging/bin/pki-enroll-openssh-cert.sh 2 files changed, 111 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/38094 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide34d4f0ffadf70e6ce42ae57328e262a832a3e4 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core : Support tokens with special characters
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core : Support tokens with special characters ...................................................................... core : Support tokens with special characters Get rid of splitting search expr by [@: ] which does not support namespaces that have a space (in dc def for example dc=My Company) This patch gets the correct search string to be sent to the AD by: a. Remove ADUSER@/ADGROUP@ from the search string b. For each existing profile try to test if the search string starts with it. If so then remove 'PROFILE_NAME:' from the search string c. For existing namespaces in selected profile try to test if search string starts with it. If so then remove 'NAMESPACE:' from search string d. Send the query Also this patch allows to use "#" in string values. Change-Id: I2464227376eb2e6ee0b5ada2ced21278675b7572 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1096175 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/BaseConditionFieldAutoCompleter.java 2 files changed, 53 insertions(+), 15 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41832 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2464227376eb2e6ee0b5ada2ced21278675b7572 Gerrit-PatchSet: 6 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: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondřej Macháček <machacek.ondra(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: fixed visibility
by ahadas＠redhat.com 10 Jun '15

10 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: core: fixed visibility ...................................................................... core: fixed visibility Bug-Url: https://bugzilla.redhat.com/1200952 Change-Id: Ia31d63a9281b337ad9f6e2c7fd09ce375b76992c Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/RepoFileMetaDataDAO.java 2 files changed, 5 insertions(+), 5 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41277 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia31d63a9281b337ad9f6e2c7fd09ce375b76992c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: altered IsoDomainListSyncronizer to refresh and return...
by ahadas＠redhat.com 10 Jun '15

10 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: core: altered IsoDomainListSyncronizer to refresh and return really *all* files if asked for it ...................................................................... core: altered IsoDomainListSyncronizer to refresh and return really *all* files if asked for it • formerly when asked to refresh resp. get files using org.ovirt.engine.core.common.businessentities.storage.ImageFileType#All type, only 'ISO' and 'Floppy' were refreshed resp. obtained. After this change all files are listed by vdsm and stored in db upon such request. • having no way here to identify type of files, IsoDomainListSyncronizer will store them into db with type 'Unknown' assigned. Bug-Url: https://bugzilla.redhat.com/1200952 Change-Id: I85f93246f903c16e8fdf9054ab54b9d9a3184eed Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsoDomainListSyncronizer.java 1 file changed, 91 insertions(+), 31 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41276 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85f93246f903c16e8fdf9054ab54b9d9a3184eed Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Adding external status to UI
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: webadmin: Adding external status to UI ...................................................................... webadmin: Adding external status to UI This patch actually displays the external status icon in the host main view Change-Id: I99e7066176e70ec94f543aa1120bb355acad9b0a Signed-off-by: emesika <emesika(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationResources.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabHostView.java D frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/HostAdditionalStatusCell.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/EntityAdditionalStatusColumn.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/HostAdditionalStatusColumn.java A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/icn_ext_error.png A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/icn_ext_failure.png A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/icn_ext_info.png A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/icn_ext_warning.png 11 files changed, 118 insertions(+), 56 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Objections: Jenkins CI: I would prefer that you didn't submit this -- To view, visit https://gerrit.ovirt.org/40999 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99e7066176e70ec94f543aa1120bb355acad9b0a Gerrit-PatchSet: 27 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: rename NetworkRoleColumnHelper
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: webadmin: rename NetworkRoleColumnHelper ...................................................................... webadmin: rename NetworkRoleColumnHelper Rename webadmin: rename NetworkRoleColumnHelper to MultiImageColumnHelper Change-Id: I380248c9694ddfe57052a41b9b991aa67830ba4d Signed-off-by: emesika <emesika(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterManageNetworkPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabNetworkView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterNetworkView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java R frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/MultiImageColumnHelper.java 5 files changed, 17 insertions(+), 15 deletions(-) Approvals: Eli Mesika: Verified Alona Kaplan: Looks good to me, approved Jenkins CI: Verified -- To view, visit https://gerrit.ovirt.org/40998 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I380248c9694ddfe57052a41b9b991aa67830ba4d Gerrit-PatchSet: 24 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: REST API: Adding external status to API
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: REST API: Adding external status to API ...................................................................... REST API: Adding external status to API This patch adds handling the external_status field from the external event API call Change-Id: I833f8e607fac7107e9352a547b3e36cbf9000f01 Signed-off-by: emesika <emesika(a)redhat.com> --- A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/EntityExternalStatus.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/EventValidator.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/HostValidator.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java 8 files changed, 122 insertions(+), 7 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Jenkins CI: Verified -- To view, visit https://gerrit.ovirt.org/40997 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I833f8e607fac7107e9352a547b3e36cbf9000f01 Gerrit-PatchSet: 25 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: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core[Search]:Add search support to external_status
by emesika＠redhat.com 10 Jun '15

10 Jun '15

Eli Mesika has submitted this change and it was merged. Change subject: core[Search]:Add search support to external_status ...................................................................... core[Search]:Add search support to external_status This patch adds ability to search in the host view by the new external_status field Change-Id: Ic86129a29f62643d206001882f42b1bbb28dbb88 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/VdsConditionFieldAutoCompleter.java 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified Jenkins CI: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40996 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic86129a29f62643d206001882f42b1bbb28dbb88 Gerrit-PatchSet: 22 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: Jenkins CI Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0