From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Adding abrt dependency and introduce configurator for it
......................................................................
Adding abrt dependency and introduce configurator for it
- Adding spec requirement for abrt addons to replace current coredump
kernel reports (see RFE for more info)
- Adding abrt conf files
- Removing logrotate config for coredump folder - assuming that abrt
takes care for not flooding in reports (need to be verified)
- Adding tests
Change-Id: I1ca5e66c9f029be75483b86414e328d074c7e454
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=917062
Signed-off-by: Yeela Kaplan <ykaplan(a)redhat.com>
Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
---
M init/systemd/85-vdsmd.preset
M init/vdsmd_init_common.sh.in
M lib/vdsm/tool/configurators/Makefile.am
A lib/vdsm/tool/configurators/abrt.py
M static/usr/lib/systemd/system/vdsmd.service.in
M tests/Makefile.am
A tests/toolTests_CCPP.conf
A tests/toolTests_abrt-action-save-package-data.conf
A tests/toolTests_abrt.conf
A tests/toolTests_vmcore.conf
M tests/tool_test.py
M vdsm.spec.in
M vdsm/vdsm-logrotate
13 files changed, 272 insertions(+), 68 deletions(-)
Approvals:
Piotr Kliczewski: Looks good to me, approved
Yaniv Bronhaim: Verified
Jenkins CI: Passed CI tests
Irit Goihman: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/44287
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1ca5e66c9f029be75483b86414e328d074c7e454
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>