[sos-devel] Updated gluster plugin

Bryn M. Reeves bmr at redhat.com
Thu Nov 10 14:41:00 UTC 2011


On 11/10/2011 01:41 PM, Keith Robertson wrote:
> Previous one didn't work on RHEL 6.1 due to a missing addCopySpecs
> method. I just broke it out into individual calls so that it will work

Ah, yeah, good point.

> import sos.plugintools
>
> class gluster(sos.plugintools.PluginBase):
>     '''gluster related information'''

Also probably should implement checkenabled() here, something like:

     def checkenabled(self):
         return os.path.exists("/etc/glusterd")

So we don't log errors if the file paths don't exist.

>     def setup(self):
>         self.addCopySpec("/etc/glusterd/")
>         self.addCopySpec("/var/log/glusterfs/")

Cheers,
Bryn.




More information about the sos-devel mailing list