On 06/17/2015 07:20 AM, Jakub Cajka wrote: > Hello, > > Latest release of iprutils is shipping sos plugin, witch seems unnecessary > as sos (upstream) already > includes iprconfig plugin > (https://github.com/sosreport/sos/commit/ad464fcc0882f8764e8bbb091a91eeeb4a804ff3, > CC'ing author). > So all linux distribution including sos(report) should ship it(if not I > would consider it a bug/RFE). > Also I have contacted sos maintainer in fedora(also upstream dev) bmr > (https://admin.fedoraproject.org/pkgdb/packager/bmr/) > for advice about packaging sos plugins and correct way is to make it part > of upstream sos, > i.e. no packaging/shipping of sos plugins outside sos. > > As mentioned above sos should already contain iprconfig.py(iprutils.py > equivalent) and any sos plugin should be included > in sos upstream, so I propose dropping sos plugin code from iprutils code > base. > > I will prepare patch with drop if my proposal is accepted and push > eventually missing pieces in to > the sos upstream(in upstream sos plugin, some information were/are not > gathered > I have already opened PR https://github.com/sosreport/sos/pull/575, hope I > haven't missed anything).
Jakub, thanks for reporting this. I was unaware of the sosreport plugin that Aruna authored. Since it was committed in December of last year, that would explain why I've yet to see it show up in any distributions. The sosreport plugin included in the iprutils repository logs more information than the one in the sosreport repository, so I'd prefer to update the one in the sosreport repository to collect the same data we are collecting with the iprutils upstream plugin before we pull the plugin from the iprutils repository. Please cc me on any patches to the sosreport plugin. Thanks, Brian -- Brian King Power Linux I/O IBM Linux Technology Center ------------------------------------------------------------------------------ _______________________________________________ Iprdd-devel mailing list Iprdd-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/iprdd-devel
