November 2013 - Engine-commits - oVirt List Archives

Change in ovirt-log-collector[ovirt-log-collector-3.3]: Remove limit of 1000 hosts, and make it use pagination
by sbonazzo＠redhat.com 14 Nov '13

14 Nov '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: Remove limit of 1000 hosts, and make it use pagination ...................................................................... Remove limit of 1000 hosts, and make it use pagination Change-Id: I28bdc52e838dc1e28a5965b3954556755d28fb82 Signed-off-by: Pablo <Pablo.Iranzo(a)gmail.com> --- M src/helper/hypervisors.py 1 file changed, 19 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Pablo Iranzo Gómez: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/21247 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28bdc52e838dc1e28a5965b3954556755d28fb82 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Pablo Iranzo Gómez <Pablo.Iranzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3.1]: packaging: setup: generate answerfile readable by user only
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: generate answerfile readable by user only ...................................................................... packaging: setup: generate answerfile readable by user only fix file mode of previous stored answer files at state. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1028748 Change-Id: I3c1ca00b7a56216d7051c7625c5e983d422caab9 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/answerfile.py M packaging/setup/plugins/ovirt-engine-setup/upgrade/__init__.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/answerfile_fixup.py 3 files changed, 65 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21249 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c1ca00b7a56216d7051c7625c5e983d422caab9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3.1]: packaging: setup: properly support relative answer file
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: properly support relative answer file ...................................................................... packaging: setup: properly support relative answer file Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1025313 Change-Id: I72d8e8c46528e492aa3339ff83227104bfb3ba9b Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/answerfile.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21248 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72d8e8c46528e492aa3339ff83227104bfb3ba9b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Consider external networks for VM scheduling
by masayag＠redhat.com 14 Nov '13

14 Nov '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Consider external networks for VM scheduling ...................................................................... engine: Consider external networks for VM scheduling External networks are assumed to be present in the entire cluster that they're attached to, so if an external network is attached to a vNIC it shouldn't be required by the scheduler. Change-Id: I0ea9bf9fadfa9954ee856a38779ed5a627c6165f Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/NetworkPolicyUnit.java 1 file changed, 8 insertions(+), 5 deletions(-) Approvals: Mike Kolesnik: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21245 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ea9bf9fadfa9954ee856a38779ed5a627c6165f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[master]: Add .gitreview to the ignore list
by sbonazzo＠redhat.com 14 Nov '13

14 Nov '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add .gitreview to the ignore list ...................................................................... Add .gitreview to the ignore list Change-Id: Ie7cb7da0681a8477b780fb11b581ffe0f62d0e37 Signed-off-by: Pablo <Pablo.Iranzo(a)gmail.com> --- M .gitignore 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21237 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7cb7da0681a8477b780fb11b581ffe0f62d0e37 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Pablo Iranzo Gómez <Pablo.Iranzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Set min connection pool size to 1
by emesika＠redhat.com 13 Nov '13

13 Nov '13

Eli Mesika has submitted this change and it was merged. Change subject: core: Set min connection pool size to 1 ...................................................................... core: Set min connection pool size to 1 In a previous patch we changed the default minimum database connection pool size to 75 (it used to be 1). This was done in order to have database connections ready from the very beginning and thus improve performance. But in practice this means that 75 PostgreSQL processes are created wether they are needed or not. These consume approx 2.1 GiB of RAM. In machines with a total of 4 GiB this evantually triggers the out of memory killer of the kernel, which will in turn select the process that is using more memory, which usually happens to be the engine itself, so it is killed. This patch changes the default minimum size back to 1 to avoid this issue. Change-Id: I9de2c9e6999cb22a4be99eabb66a7216e673d0d2 Bug-Url: https://bugzilla.redhat.com/1026100 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21188 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9de2c9e6999cb22a4be99eabb66a7216e673d0d2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: fix log message about rerun a vm
by ofrenkel＠redhat.com 13 Nov '13

13 Nov '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: fix log message about rerun a vm ...................................................................... core: fix log message about rerun a vm We used to have log message saying "vm {0} running in db and not running in vds - add to rerun treatment" for each vm that was set as running in the engine and reported as one that is not running by vdsm, even in cases where we didn't add it to the 'rerun treatment' later on. This patch change this logging message - now we log a message that just say that the vm is running in db and not in vds in that case, and we log another message saying that the vm is added to rerun treatment when it is actually added to the 'rerun treatment'. Change-Id: Icc2940845d150fcd41e7e327769f50c8bfffce5e Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/20883 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc2940845d150fcd41e7e327769f50c8bfffce5e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-3.3.1]: packaging: updated customizations to match 5.5.0 GA
by ydary＠redhat.com 13 Nov '13

13 Nov '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: updated customizations to match 5.5.0 GA ...................................................................... packaging: updated customizations to match 5.5.0 GA Change-Id: Id055510466b857998e22076fa9bfd79f5d51425a Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M server-customizations/WEB-INF/applicationContext-security-web.xml M server-customizations/WEB-INF/applicationContext.xml M server-customizations/WEB-INF/classes/jasperreports.properties M server-customizations/WEB-INF/decorators/decorator.jsp M server-customizations/WEB-INF/decorators/minimalDecorator.jsp M server-customizations/WEB-INF/web.xml 6 files changed, 195 insertions(+), 69 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21236 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id055510466b857998e22076fa9bfd79f5d51425a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3.1 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[ovirt-3.3.1]: packaging: removed a temp fix for logging in JRS
by ydary＠redhat.com 13 Nov '13

13 Nov '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: removed a temp fix for logging in JRS ...................................................................... packaging: removed a temp fix for logging in JRS We no longer need to copy any jars to make logging work. Change-Id: I3a0a6bc65040fe51ec92f754c749be4c6354ef90 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py 1 file changed, 0 insertions(+), 20 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21235 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a0a6bc65040fe51ec92f754c749be4c6354ef90 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3.1 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: updated customizations to match 5.5.0 GA
by ydary＠redhat.com 13 Nov '13

13 Nov '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: updated customizations to match 5.5.0 GA ...................................................................... packaging: updated customizations to match 5.5.0 GA Change-Id: Id055510466b857998e22076fa9bfd79f5d51425a Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M server-customizations/WEB-INF/applicationContext-security-web.xml M server-customizations/WEB-INF/applicationContext.xml M server-customizations/WEB-INF/classes/jasperreports.properties M server-customizations/WEB-INF/decorators/decorator.jsp M server-customizations/WEB-INF/decorators/minimalDecorator.jsp M server-customizations/WEB-INF/web.xml 6 files changed, 195 insertions(+), 69 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21234 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id055510466b857998e22076fa9bfd79f5d51425a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0