From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: sos: fix broken vdsm plugin
......................................................................
sos: fix broken vdsm plugin
sos report 3.4 leads to this failure:
plugin vdsm does not install, skipping: type object 'Plugin' has no
attribute 'add_copy_spec_limit' a non-existing plugin (vdsm) was
specified in the command line.
add_copy_spec_limit was removed from sosreport Plugin code and replaced
by add_copy_spec function in commit
https://github.com/sosreport/sos/commit/c39a3df3ee1380fc6f2b394912923780f...
This fix replaced add_copy_spec_limit call with add_copy_spec and
removed function __addCopySpecLogLimit since it's not needed anymore.
Change-Id: Ibabbb801a82b985e81c8a5af9ce986ed0b31f019
Signed-off-by: Irit Goihman <igoihman(a)redhat.com>
Bug-Url:
https://bugzilla.redhat.com/1460732
---
M vdsm/sos/vdsm.py.in
1 file changed, 1 insertion(+), 14 deletions(-)
Approvals:
Yaniv Bronhaim: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Irit Goihman: Verified
Dan Kenigsberg: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/78197
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibabbb801a82b985e81c8a5af9ce986ed0b31f019
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoihman(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: Martin Peřina <mperina(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>