June 2014 - Engine-commits - oVirt List Archives

Change in otopi[otopi-1.2]: packagers: yum: disable self if not active
by Alon Bar-Lev 23 Jun '14

23 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packagers: yum: disable self if not active ...................................................................... packagers: yum: disable self if not active this breaks offline packager if exists, introduced at: 2b4f0bdccf9 since 2013-16-07, not sure how it passed all QA cycles. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1112370 Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/otopi/packagers/yumpackager.py 2 files changed, 10 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29090 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.2 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in otopi[master]: packagers: yum: disable self if not active
by Alon Bar-Lev 23 Jun '14

23 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packagers: yum: disable self if not active ...................................................................... packagers: yum: disable self if not active this breaks offline packager if exists, introduced at: 2b4f0bdccf9 since 2013-16-07, not sure how it passed all QA cycles. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1112370 Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/otopi/packagers/yumpackager.py 2 files changed, 10 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29089 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7 Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: hooks: on-boot changed the path
by dougsland＠redhat.com 23 Jun '14

23 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: hooks: on-boot changed the path ...................................................................... hooks: on-boot changed the path On-boot hook changed the path. This patch adds the new path. Change-Id: I4b5332a6a85bc7b8d8ec3f3cf615b8716a5ec32a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101215 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M hooks/Makefile.am M ovirt-node-plugin-vdsm.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29087 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b5332a6a85bc7b8d8ec3f3cf615b8716a5ec32a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: hooks: on-boot changed the path
by dougsland＠redhat.com 23 Jun '14

23 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: hooks: on-boot changed the path ...................................................................... hooks: on-boot changed the path On-boot hook changed the path. This patch adds the new path. Change-Id: I4b5332a6a85bc7b8d8ec3f3cf615b8716a5ec32a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101215 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M hooks/Makefile.am M ovirt-node-plugin-vdsm.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified; Looks good to me, approved Joey Boggs: Looks good to me, but someone else must approve Ryan Barry: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29085 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b5332a6a85bc7b8d8ec3f3cf615b8716a5ec32a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Joey Boggs <jboggs(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Added first mock job and some improvements
by dcaroest＠redhat.com 23 Jun '14

23 Jun '14

David Caro has submitted this change and it was merged. Change subject: Added first mock job and some improvements ...................................................................... Added first mock job and some improvements * ovirt-hosted-engine-ha create-rpms jobs for merged patches * Added README with some info on the organization * Generic mock build scripts and yamls for other jobs to reuse Change-Id: Id77cc184adadbd3b8daa54b3b9428da511496340 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M .gitignore A jobs/confs/README A jobs/confs/shell-scripts/mock_build_onlyrpm.sh.inc A jobs/confs/shell-scripts/mock_cleanup.sh A jobs/confs/shell-scripts/mock_setup.sh A jobs/confs/yaml/builders/rpm_mock.yaml M jobs/confs/yaml/defaults/defaults.yaml A jobs/confs/yaml/jobs/create-rpms_mock.yaml R jobs/confs/yaml/jobs/jenkins/jenkins_check_yaml.yaml R jobs/confs/yaml/jobs/jenkins/jenkins_deploy_yamls.yaml A jobs/confs/yaml/jobs/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml R jobs/confs/yaml/jobs/system/system_gerrit-garbage-collect.yaml M jobs/confs/yaml/parmeters/gerrit.yaml A jobs/confs/yaml/publishers/exported-artifacts.yaml M jobs/confs/yaml/scms/jenkins.yaml A jobs/confs/yaml/scms/ovirt-hosted-engine-ha.yaml A jobs/confs/yaml/triggers/gerrit.yaml 17 files changed, 337 insertions(+), 7 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/29001 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id77cc184adadbd3b8daa54b3b9428da511496340 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Add filter to ignore findbugs complaint
by masayag＠redhat.com 23 Jun '14

23 Jun '14

Moti Asayag has submitted this change and it was merged. Change subject: engine: Add filter to ignore findbugs complaint ...................................................................... engine: Add filter to ignore findbugs complaint The dbFacade member accessed by CDI should not be treated as unused code. Change-Id: I2b72d39e09739c911747de8811de5c7818cb8850 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/exclude-filters.xml 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Roy Golan: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/29073 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b72d39e09739c911747de8811de5c7818cb8850 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: fix typo cause invalid condition at fence ...
by sbonazzo＠redhat.com 23 Jun '14

23 Jun '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix typo cause invalid condition at fence removal ...................................................................... packaging: setup: fix typo cause invalid condition at fence removal Change-Id: I252d8520c451e19e6b6bca20b2c91e3dc89657fe Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/system/fence_kdump_listener.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yaniv Bronhaim: Verified; Looks good to me, but someone else must approve oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29045 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I252d8520c451e19e6b6bca20b2c91e3dc89657fe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-engine-reports-3.4]: reports: remove war file for new db
by ydary＠redhat.com 23 Jun '14

23 Jun '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: remove war file for new db ...................................................................... reports: remove war file for new db updated abort, when clean install will remove deployed files. Change-Id: Id4a88458e583f93874ef6fc6a7f683c0e1760e40 Bug-Url: https://bugzilla.redhat.com/1072357 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4a88458e583f93874ef6fc6a7f683c0e1760e40 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.4 Gerrit-Owner: 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]: engine: Fix coverity complaints
by masayag＠redhat.com 23 Jun '14

23 Jun '14

Moti Asayag has submitted this change and it was merged. Change subject: engine: Fix coverity complaints ...................................................................... engine: Fix coverity complaints Change-Id: I88c2d7a1c4aab7261e9ddf5b69de127353f4a410 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/DuplicateKeysCheck.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NoDuplicateProperties.java 2 files changed, 2 insertions(+), 3 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/29072 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88c2d7a1c4aab7261e9ddf5b69de127353f4a410 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Display network updated messages titles were rephr...
by lvernia＠redhat.com 23 Jun '14

23 Jun '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Display network updated messages titles were rephrased ...................................................................... webadmin: Display network updated messages titles were rephrased Display network updated messages titles were rephrased in order to reflect the past tense of the operation. Change-Id: I66a22881e1b5aaf147a3ef9018d7ac74e5d58454 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Lior Vernia: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit http://gerrit.ovirt.org/29029 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66a22881e1b5aaf147a3ef9018d7ac74e5d58454 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0