October 2015 - Engine-commits - oVirt List Archives

Change in ovirt-reports[ovirt-engine-reports-3.5]: build: after ovirt-engine-reports-3.5.5 respin
by ydary＠redhat.com 11 Oct '15

11 Oct '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: after ovirt-engine-reports-3.5.5 respin ...................................................................... build: after ovirt-engine-reports-3.5.5 respin Change-Id: I67883e3dc4823edbae3ef090cd049d0c7dd9cfbd Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47187 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67883e3dc4823edbae3ef090cd049d0c7dd9cfbd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: build: ovirt-engine-reports-3.5.5 respin
by ydary＠redhat.com 11 Oct '15

11 Oct '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-reports-3.5.5 respin ...................................................................... build: ovirt-engine-reports-3.5.5 respin Change-Id: Ibe9a32e00f6acbe742f2bc74c5f1b61fa4f53f56 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M ovirt-engine-reports.spec.in M version.mak 2 files changed, 8 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47186 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe9a32e00f6acbe742f2bc74c5f1b61fa4f53f56 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: reports: fixed SQL error on pgsql 8.4
by ydary＠redhat.com 11 Oct '15

11 Oct '15

Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed SQL error on pgsql 8.4 ...................................................................... reports: fixed SQL error on pgsql 8.4 Change-Id: I0941626a3763565f141208cc56c91acc10104e07 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-reports/resources/Reports/Trend/Hosts/single_host_resource_br2a_files/single_host_resource_br2a_jrxml.data M packaging/ovirt-reports/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A_files/single_vm_resources_over_time_BR10A_jrxml.data 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47183 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0941626a3763565f141208cc56c91acc10104e07 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[master]: reports: fixed SQL error on pgsql 8.4
by ydary＠redhat.com 11 Oct '15

11 Oct '15

Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed SQL error on pgsql 8.4 ...................................................................... reports: fixed SQL error on pgsql 8.4 Change-Id: Iad20fc93cc6ac6d6dd0741dc8c1d855efbc1eefa Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-reports/resources/Reports/Trend/Hosts/single_host_resource_br2a_files/single_host_resource_br2a_jrxml.data M packaging/ovirt-reports/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A_files/single_vm_resources_over_time_BR10A_jrxml.data 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47184 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad20fc93cc6ac6d6dd0741dc8c1d855efbc1eefa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6.0]: engine: Add disk check specific LUN visibility
by tnisan＠redhat.com 11 Oct '15

11 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: engine: Add disk check specific LUN visibility ...................................................................... engine: Add disk check specific LUN visibility When adding a new direct LUN disk, the engine tests the visibility of the LUN if a host has been specified. In order to improve performance, the visibility test done via GetDeviceList will be performed only on the specific LUN. Change-Id: I39d0267fffd7e96c0d5297ac4792f632201ecdf1 Bug-Url: https://bugzilla.redhat.com/1217401 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/DiskValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddDiskCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/storage/DiskValidatorTest.java 3 files changed, 17 insertions(+), 7 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47157 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39d0267fffd7e96c0d5297ac4792f632201ecdf1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.0 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6.0]: engine: GetDeviceList without check status
by tnisan＠redhat.com 11 Oct '15

11 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: engine: GetDeviceList without check status ...................................................................... engine: GetDeviceList without check status When a user opens the Storage Domain or Disk dialog for block storage, a GetDeviceList verb is called on VDSM. In order to populate the 'status' field, the getDeviceList verb perform a create PV test. This operation is expensive, and in setups with large number of devices it will cause performance problem and even timeouts. The flow before this patch was: - Engine calls getDeviceList, VDSM performs status check for each device, possibly times out - User selects a few devices (typically one) to add/extend a SD or create a new LUN disk - GUI warns user about used devices The flow with this patch: - Engine calls getDeviceList, skipping status check - User selects a few devices (typically one) to add/extend a SD or create a new LUN disk - Engine calls getDeviceList, checking status only for selected - Engine warns user about used devices Change-Id: Ib4484307e3d4ac66a4370bcef3ac015ad0f2d084 Bug-Url: https://bugzilla.redhat.com/1217401 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetDeviceListQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetUnregisteredBlockStorageDomainsQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RefreshLunsSizeCommand.java 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/businessentities/storage/LunStatus.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetDeviceListQueryParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/GetDeviceListVDSCommandParameters.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostStorageResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetDeviceListVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 20 files changed, 238 insertions(+), 38 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47156 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4484307e3d4ac66a4370bcef3ac015ad0f2d084 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.0 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: update threadholds for warning/abandoning patches
by eedri＠redhat.com 11 Oct '15

11 Oct '15

Eyal Edri has submitted this change and it was merged. Change subject: update threadholds for warning/abandoning patches ...................................................................... update threadholds for warning/abandoning patches - now will auto-abandon 300 days - now will warn on 200 days Change-Id: Ic8b7ee6f3a0132d640ed903e8666c32a483b6241 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/yaml/jobs/system/system_gerrit-alert-old-patches.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47180 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8b7ee6f3a0132d640ed903e8666c32a483b6241 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Add tests for storage server connection extension
by tnisan＠redhat.com 11 Oct '15

11 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Add tests for storage server connection extension ...................................................................... core: Add tests for storage server connection extension Change-Id: I00ff096f9ae67e507605855c8f19858318645db9 Bug-Url: https://bugzilla.redhat.com/1265133 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageServerConnectionExtensionValidator.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionExtensionCommandTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/storage/StorageServerConnectionExtensionValidatorTest.java 3 files changed, 121 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47175 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I00ff096f9ae67e507605855c8f19858318645db9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Validate inexistance of connection extension upon update
by tnisan＠redhat.com 11 Oct '15

11 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Validate inexistance of connection extension upon update ...................................................................... core: Validate inexistance of connection extension upon update When updating a storage server connection extension, make sure that another connection for the same host and target combination does not exist Change-Id: I90c5f56356e8b1ad75861760b6375cd23869db35 Bug-Url: https://bugzilla.redhat.com/1265133 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageServerConnectionExtensionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RemoveStorageServerConnectionExtensionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageServerConnectionExtensionCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionExtensionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageServerConnectionExtensionValidator.java 5 files changed, 28 insertions(+), 14 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47174 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I90c5f56356e8b1ad75861760b6375cd23869db35 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: restapi: Remove disks and storage domain option from add VM ...
by tnisan＠redhat.com 11 Oct '15

11 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Remove disks and storage domain option from add VM from config ...................................................................... restapi: Remove disks and storage domain option from add VM from config Change-Id: I0fdc659be984dd74ad19f904b884d6ebdc9cdb34 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1082977 --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47118 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0fdc659be984dd74ad19f904b884d6ebdc9cdb34 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0