Change in ovirt-engine[master]: tools: Adds notifier config validation prior to service startup
by emesika@redhat.com
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
10 years, 11 months
Change in ovirt-engine[master]: tools: Adds validate only argument to notifier
by emesika@redhat.com
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
10 years, 11 months
Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: rollback legacy sysconfig
by iheim@redhat.com
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
10 years, 11 months
Change in ovirt-engine[master]: Fixing cleandb.sh logging
by emesika@redhat.com
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>
10 years, 11 months
Change in ovirt-node-plugin-vdsm[node-3.0]: ovirt-node-plugin-vdsm.spec: increase the release
by fabiand@fedoraproject.org
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>
10 years, 11 months
Change in ovirt-engine[master]: packaging: setup: rollback legacy sysconfig
by sbonazzo@redhat.com
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
10 years, 11 months
Change in ovirt-node-plugin-vdsm[node-3.0]: Use Management class to pass informations
by dougsland@redhat.com
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>
10 years, 11 months
Change in ovirt-dwh[master]: packaging: services: systemd: convert deamons into notify type
by ydary@redhat.com
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>
10 years, 11 months
Change in ovirt-dwh[master]: packaging: services: remove pid when service start fails
by ydary@redhat.com
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>
10 years, 11 months
Change in ovirt-reports[master]: packaging: added override file for application context files
by ydary@redhat.com
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>
10 years, 11 months