January 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: tools: Adds notifier config validation prior to service startup
by emesika＠redhat.com 09 Jan '14

09 Jan '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: Adds notifier config validation prior to service startup ...................................................................... tools: Adds notifier config validation prior to service startup Executes notifier config validation prior service startup. If validation fails, service returns error exit code. Change-Id: I11ad77a33697f254e60e1d0b12e343900e5b0b34 Bug-Url: https://bugzilla.redhat.com/989382 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.py 1 file changed, 44 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22629 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11ad77a33697f254e60e1d0b12e343900e5b0b34 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Adds validate only argument to notifier
by emesika＠redhat.com 09 Jan '14

09 Jan '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: Adds validate only argument to notifier ...................................................................... tools: Adds validate only argument to notifier Adds command line argument to notifier, that executes only validation of notification properties and exits after. Change-Id: I2d95f2f4225652d47732c12e5e0716b7f625bbe3 Bug-Url: https://bugzilla.redhat.com/989382 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/Notifier.java 1 file changed, 23 insertions(+), 4 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22628 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d95f2f4225652d47732c12e5e0716b7f625bbe3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: rollback legacy sysconfig
by iheim＠redhat.com 09 Jan '14

09 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: rollback legacy sysconfig ...................................................................... packaging: setup: rollback legacy sysconfig Rolling back also /etc/sysconfig/ovirt-engine on cleanup after failure. Change-Id: Ie9e4a90a7e0e47aeb88a938fedb4f2088d533a70 Bug-Url: https://bugzilla.redhat.com/1049622 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/legacy/config.py 1 file changed, 52 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23092 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9e4a90a7e0e47aeb88a938fedb4f2088d533a70 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: Fixing cleandb.sh logging
by emesika＠redhat.com 09 Jan '14

09 Jan '14

Eli Mesika has submitted this change and it was merged. Change subject: Fixing cleandb.sh logging ...................................................................... Fixing cleandb.sh logging Adding ":" after the l option in getopts to make th logging work. Change-Id: I1ae7beb0d4bbfcf37c83896a379ffc40bb6608d0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1050954 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/cleandb.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/23094 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ae7beb0d4bbfcf37c83896a379ffc40bb6608d0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: ovirt-node-plugin-vdsm.spec: increase the release
by fabiand＠fedoraproject.org 09 Jan '14

09 Jan '14

Fabian Deutsch has submitted this change and it was merged. Change subject: ovirt-node-plugin-vdsm.spec: increase the release ...................................................................... ovirt-node-plugin-vdsm.spec: increase the release New build to include: - Use Management class to pass informations (BZ#1048929) Change-Id: I6035307637ef3b793f23044f103498a1bcae0855 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Douglas Schilling Landgraf: Verified -- To view, visit http://gerrit.ovirt.org/23093 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6035307637ef3b793f23044f103498a1bcae0855 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>

1 0

Change in ovirt-engine[master]: packaging: setup: rollback legacy sysconfig
by sbonazzo＠redhat.com 09 Jan '14

09 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: rollback legacy sysconfig ...................................................................... packaging: setup: rollback legacy sysconfig Rolling back also /etc/sysconfig/ovirt-engine on cleanup after failure. Change-Id: Ie9e4a90a7e0e47aeb88a938fedb4f2088d533a70 Bug-Url: https://bugzilla.redhat.com/1049622 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/legacy/config.py 1 file changed, 52 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9e4a90a7e0e47aeb88a938fedb4f2088d533a70 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: Use Management class to pass informations
by dougsland＠redhat.com 09 Jan '14

09 Jan '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: Use Management class to pass informations ...................................................................... Use Management class to pass informations Use Management class from ovirt node to provide information to users. Change-Id: I441375ea4f7e18b55a7e4d8a7f661e067598de30 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1048929 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 17 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23081 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I441375ea4f7e18b55a7e4d8a7f661e067598de30 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: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org>

1 0

Change in ovirt-dwh[master]: packaging: services: systemd: convert deamons into notify type
by ydary＠redhat.com 09 Jan '14

09 Jan '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: services: systemd: convert deamons into notify type ...................................................................... packaging: services: systemd: convert deamons into notify type notify supports validation pre-start. Change-Id: Ic806da35ecb2ca5d40e03e0981840272173978d2 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine-dwhd/ovirt-engine-dwhd.systemd.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23075 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic806da35ecb2ca5d40e03e0981840272173978d2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: services: remove pid when service start fails
by ydary＠redhat.com 09 Jan '14

09 Jan '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: services: remove pid when service start fails ...................................................................... packaging: services: remove pid when service start fails the service is running under non privileged user, so it cannot remove the pid file only clear it. if service is dead at validation stage its status is printed as: dead but pid file exists to remedy that, if service fails to start we remove the pid if service start fails and pid file is zero length. Change-Id: I11a29e58e0fd673eb1df79983b0c09799f9ef0f3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine-dwhd/ovirt-engine-dwhd.sysv.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23074 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11a29e58e0fd673eb1df79983b0c09799f9ef0f3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: packaging: added override file for application context files
by ydary＠redhat.com 09 Jan '14

09 Jan '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: added override file for application context files ...................................................................... packaging: added override file for application context files Change-Id: I1f9e637dd385dfd6928a511ec801759616746782 Bug-Url: https://bugzilla.redhat.com/1023751 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-engine-reports-setup.py A server-customizations/WEB-INF/applicationContext-ovirt-override.xml D server-customizations/WEB-INF/applicationContext-security-web.xml D server-customizations/WEB-INF/applicationContext.xml 4 files changed, 67 insertions(+), 2,010 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20833 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f9e637dd385dfd6928a511ec801759616746782 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0