From Yaniv Bronhaim <[email protected]>: Yaniv Bronhaim has posted comments on this change.
Change subject: (wip) Adding abrt dependency and introduce configurator for it ...................................................................... Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/44287/5/lib/vdsm/tool/configurators/abrt.py File lib/vdsm/tool/configurators/abrt.py: Line 62: 'fragments': [ Line 63: { Line 64: 'conditions': {}, Line 65: 'content': { Line 66: 'DumpLocation': '/var/tmp/abrt' > it's not included, it should be added yes its part of the action items of the feature to add abrt output to sos plugins. see https://trello.com/c/aRx6a6UN/6-bug-917062-rfe-add-abrt-integration Line 67: }, Line 68: }, Line 69: ] Line 70: }, -- To view, visit https://gerrit.ovirt.org/44287 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1ca5e66c9f029be75483b86414e328d074c7e454 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Irit Goihman <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenkins CI RO Gerrit-Reviewer: Martin Peřina <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
