June 2014 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Added vdsm-jsonrpc-java
by dcaroest＠redhat.com 27 Jun '14

27 Jun '14

David Caro has submitted this change and it was merged. Change subject: Added vdsm-jsonrpc-java ...................................................................... Added vdsm-jsonrpc-java Change-Id: Ib7512d99d1dade65a0401df9ee237720fbb876d2 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml A jobs/confs/yaml/scms/vdsm-jsonrpc-java.yaml 2 files changed, 34 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29357 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib7512d99d1dade65a0401df9ee237720fbb876d2 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: CentOS7 default dist is not valid
by dcaroest＠redhat.com 27 Jun '14

27 Jun '14

David Caro has submitted this change and it was merged. Change subject: CentOS7 default dist is not valid ...................................................................... CentOS7 default dist is not valid Injecting the dist into rpmmacros for manual builds to use it Change-Id: I856148c384b1ffc60a571a1e821d899ede3f89bf Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh M jobs/confs/shell-scripts/mock_build_onlyrpm.sh M mock_configs/epel-7-x86_64.cfg 3 files changed, 23 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29354 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I856148c384b1ffc60a571a1e821d899ede3f89bf Gerrit-PatchSet: 7 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: adding a ctr with CommandContext for internal command
by derez＠redhat.com 27 Jun '14

27 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: core: adding a ctr with CommandContext for internal command ...................................................................... core: adding a ctr with CommandContext for internal command Added a constructor that accepts CommandContext for SyncLunsInfoForBlockStorageDomainCommand. Change-Id: I1685cfbbce8bb276b8282cec47395cf7b283eb5d Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForBlockStorageDomainCommand.java 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29351 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1685cfbbce8bb276b8282cec47395cf7b283eb5d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: setup: using a longer delay waiting for the API
by sbonazzo＠redhat.com 27 Jun '14

27 Jun '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: using a longer delay waiting for the API ...................................................................... packaging: setup: using a longer delay waiting for the API Using a longer sleep time waiting for API to become ready Change-Id: I2658ec1656792fd0e92bec5e4b02fb2587038519 Bug-url: https://bugzilla.redhat.com/1113882 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/all-in-one/vdsm.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29348 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2658ec1656792fd0e92bec5e4b02fb2587038519 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine:fail stop volume if remove brick is running
by kmayilsa＠redhat.com 27 Jun '14

27 Jun '14

Kanagaraj M has submitted this change and it was merged. Change subject: engine:fail stop volume if remove brick is running ...................................................................... engine:fail stop volume if remove brick is running 'stop' gluster volume is not allowed if there is a remove brick operation is running on that volume Change-Id: Ic7f227b086e78374b385ebd2f4d8f8591825c7fe Bug-Url: https://bugzilla.redhat.com/1032020 Signed-off-by: Karnan TC <kchidamb(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/StopGlusterVolumeCommand.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/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 5 files changed, 11 insertions(+), 0 deletions(-) Approvals: Karnan t c: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7f227b086e78374b385ebd2f4d8f8591825c7fe Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Volume rebalance status popup button caption changed
by kmayilsa＠redhat.com 27 Jun '14

27 Jun '14

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: Volume rebalance status popup button caption changed ...................................................................... webadmin: Volume rebalance status popup button caption changed Changed the button caption of volume rebalance status popup from 'Stop Rebalance' to 'Stop' to maintain consistency along the button names in the popups. Change-Id: I907ba9b53dbff901cb0964ea90d36f0e66b7fedc Bug-Url: https://bugzilla.redhat.com/1029484 Signed-off-by: Karnan TC <kchidamb(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Karnan t c: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28471 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I907ba9b53dbff901cb0964ea90d36f0e66b7fedc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: modify ctr to be compatible with caller method
by yzaslavs＠redhat.com 27 Jun '14

27 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: modify ctr to be compatible with caller method ...................................................................... core: modify ctr to be compatible with caller method SyncLunsInfoForBlockStorageDomainCommand is being called without context although its constructor is only supported with context parameter, so we always get a runtime exception when the command is being called. The proposed fix is to remove the context from the constructor of SyncLunsInfoForBlockStorageDomainCommand. Change-Id: Ib06a348beb907da5e1b0c172a36a01fde09bbcbb Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SyncLunsInfoForBlockStorageDomainCommand.java 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29350 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib06a348beb907da5e1b0c172a36a01fde09bbcbb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin, userportal: Fix client-side sort issue
by vszocs＠redhat.com 27 Jun '14

27 Jun '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin,userportal: Fix client-side sort issue ...................................................................... webadmin,userportal: Fix client-side sort issue This patch fixes an issue where items would disappear from grid when activating client-side column sorting. This patch does not attempt to guarantee stability of sorting behavior in relation to fresh items retrieved from server. In other words, following edge case: a, apply client-side sorting on column A b, apply client-side sorting on column B c, refresh items, some items might switch positions In future, we can improve sorting logic to guarantee stability of sorting vs. refresh behavior. Change-Id: I437e9dd7ff2afdb2d87fb0ba6c17dcb081cd965d Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SortedListModel.java A frontend/webadmin/modules/uicommonweb/src/test/java/org/ovirt/engine/ui/uicommonweb/models/SortedListModelTest.java 2 files changed, 153 insertions(+), 8 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Vojtech Szocs: Verified; Looks good to me, approved Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28392 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I437e9dd7ff2afdb2d87fb0ba6c17dcb081cd965d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Adding kexec-tools jobs
by dcaroest＠redhat.com 27 Jun '14

27 Jun '14

David Caro has submitted this change and it was merged. Change subject: Adding kexec-tools jobs ...................................................................... Adding kexec-tools jobs Added also epel7 mock config Change-Id: I20f8097a1b7f9faa581c28f9f030bb00c85134a2 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh A jobs/confs/yaml/jobs/kexec-tools/kexec-tools_create-rpms.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml A mock_configs/epel-7-x86_64.cfg 4 files changed, 80 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29335 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20f8097a1b7f9faa581c28f9f030bb00c85134a2 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: kdump: Fix version comparisons
by Alon Bar-Lev 27 Jun '14

27 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: kdump: Fix version comparisons ...................................................................... kdump: Fix version comparisons Use rpmUtils.miscutils.compareEVR for correct package version comparisons. This should work: '2.0.0-273.el6' < '2.0.0-273.1' '2.0.0-273.1.el6' >= '2.0.0-273.1' '2.0.0-274.el6' >= '2.0.0-273.1' Change-Id: Ica14bd688c73a511ad8d7557231cff2e184830a9 Bug-Url: https://bugzilla.redhat.com/1079821 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M src/plugins/ovirt-host-deploy/kdump/packages.py 1 file changed, 7 insertions(+), 6 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Martin Peřina: Verified -- To view, visit http://gerrit.ovirt.org/29317 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica14bd688c73a511ad8d7557231cff2e184830a9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0