May 2015 - Engine-commits - oVirt List Archives

Change in ovirt-log-collector[ovirt-log-collector-3.5]: build: post ovirt-log-collector-3.5.3
by sbonazzo＠redhat.com 22 May '15

22 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-log-collector-3.5.3 ...................................................................... build: post ovirt-log-collector-3.5.3 Change-Id: Ie24fb8e106ffc47bc34676ead7402a0ad0227e13 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41335 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie24fb8e106ffc47bc34676ead7402a0ad0227e13 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.5]: build: ovirt-log-collector-3.5.3
by sbonazzo＠redhat.com 22 May '15

22 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-3.5.3 ...................................................................... build: ovirt-log-collector-3.5.3 Change-Id: I5347593b5e38317cc1e5e63b108ff791afbad57b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41334 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5347593b5e38317cc1e5e63b108ff791afbad57b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: spec: pull otopi-1.3.2
by piotr.kliczewski＠gmail.com 22 May '15

22 May '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: spec: pull otopi-1.3.2 ...................................................................... packaging: spec: pull otopi-1.3.2 Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41309 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5.3]: packaging: spec: pull otopi-1.3.2
by piotr.kliczewski＠gmail.com 22 May '15

22 May '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: spec: pull otopi-1.3.2 ...................................................................... packaging: spec: pull otopi-1.3.2 Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: spec: pull otopi-1.3.2
by sbonazzo＠redhat.com 22 May '15

22 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: pull otopi-1.3.2 ...................................................................... packaging: spec: pull otopi-1.3.2 Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41308 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f8b4eb35af6e273df7d3d2cc32083af5cd0d3c8 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Corrected setting of gluster volume snapshot config
by kmayilsa＠redhat.com 22 May '15

22 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Corrected setting of gluster volume snapshot config ...................................................................... gluster: Corrected setting of gluster volume snapshot config The gluster CLI command for setting gluster volume snapshot option snap-max-soft-limit does not accept '%' sign in the last. Corrected the code to remove the same if passed from UI. Change-Id: I9c5f425e9acd63646aad5ed4535cbc334c7b41aa Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeSnapshotConfigVDSCommand.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41324 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c5f425e9acd63646aad5ed4535cbc334c7b41aa Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: gluster: Corrected setting of gluster volume snapshot config
by kmayilsa＠redhat.com 22 May '15

22 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Corrected setting of gluster volume snapshot config ...................................................................... gluster: Corrected setting of gluster volume snapshot config The gluster CLI command for setting gluster volume snapshot option snap-max-soft-limit does not accept '%' sign in the last. Corrected the code to remove the same if passed from UI. Change-Id: I9c5f425e9acd63646aad5ed4535cbc334c7b41aa Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/SetGlusterVolumeSnapshotConfigVDSCommand.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Shubhendu Tripathi: Verified Jenkins CI: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41113 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c5f425e9acd63646aad5ed4535cbc334c7b41aa Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: webadmin, userportal: Optimize syntax checker handling
by piotr.kliczewski＠gmail.com 22 May '15

22 May '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin,userportal: Optimize syntax checker handling ...................................................................... webadmin,userportal: Optimize syntax checker handling 1. access to (search) syntax checker instance is now provided through Configurator#getSyntaxChecker * UserPortalConfigurator#getSyntaxChecker returns null (not available in UserPortal context) * WebAdminConfigurator#getSyntaxChecker returns instance obtained via SyntaxCheckerFactory 2. removed SearchableListModel#syntaxChecker field along with related code (instance initialization, getter) 3. modified SearchableListModel#isSearchValidForServerSideSorting to access syntax checker through Configurator * in UserPortal context, syntax checker will be null and above method will simply return true * in WebAdmin context, syntax checker will be non-null and existing behavior will be retained 4. modified SearchSuggestModel to access syntax checker through Configurator 5. removed unused SyntaxCheckerFactory#getUISyntaxChecker This patch improves performance of UserPortal while keeping existing behavior of WebAdmin (where syntax checker is used in SearchableListModel#isSearchValidForServerSideSorting). Change-Id: Iab7f369fa26452fe49c57f99dc53919a2bad0412 Bug-Url: https://bugzilla.redhat.com/1223836 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerFactory.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Configurator.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/autocomplete/SearchSuggestModel.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/uicommon/UserPortalConfigurator.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uicommon/WebAdminConfigurator.java 6 files changed, 30 insertions(+), 22 deletions(-) Approvals: Jenkins CI: Looks good to me, but someone else must approve Greg Sheremeta: Looks good to me, approved Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/41284 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab7f369fa26452fe49c57f99dc53919a2bad0412 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: dropping packages that requires vdsm on FC20.
by dcaroest＠redhat.com 22 May '15

22 May '15

David Caro has submitted this change and it was merged. Change subject: dropping packages that requires vdsm on FC20. ...................................................................... dropping packages that requires vdsm on FC20. Since VDSM is not available anymore on master FC20, dropping packages requiring it. Change-Id: I56f36311ac517b2d9959fc0ad72a74d36ef9b916 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml 3 files changed, 2 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Looks good to me; Passed CI tests David Caro: Ready for merge -- To view, visit https://gerrit.ovirt.org/41307 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56f36311ac517b2d9959fc0ad72a74d36ef9b916 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.5]: packaging: Updated AUTHORS
by sbonazzo＠redhat.com 22 May '15

22 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Updated AUTHORS ...................................................................... packaging: Updated AUTHORS Change-Id: I2e8f0b41a86b74614f2fab4ae80773c0dd61000f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 9a7712febebeb1b201db1a096c512b3f76ec49d5) --- M AUTHORS 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41312 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e8f0b41a86b74614f2fab4ae80773c0dd61000f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0