Description
SNAPTELD documentation page lacks proper explanation of --auto-discover option. The option description is "Auto discover paths separated by colons", but doesn't say anything about the purpose.
There is a single example below with "snapteld --auto-discover /opt/snap/plugins/" line suggesting this option might be used to automatically load plugins, but that's all.
In the SNAPTELD_CONFIGURATION page at the end there is a comment on restarting snapteld process and it says that not only plugins in the auto_discover_path will be reloaded, but also tasks! (which wasn't mentioned anywhere else).
And there is the TASKS page which doesn't say anything about auto-discovery and auto-loading tasks at all.
I guess that auto-loading tasks feature was added later and documentation wasn't properly updated...
Anyway, I suggest adding an explanation of --auto-discover option on the SNAPTELD page with additional examples (with comments) showing clearly it migt be also used for auto-loading tasks.
Additionally, the TASKS page should also be updated with the proper information regarding auto-loading tasks.
Activity
nanliu commentedon Jan 31, 2017
Yeah this isn't documented very well. We should change the default config to use two directories: one for tasks, the other for plugins:
nanliu commentedon Feb 1, 2017
We also could use some community contributions deploying Snap with configuration management tools. I feel like this would avoid having everyone reinvent/rediscover good defaults/best practices.
For example, this could be one way take advantage of this setting to load tasks/plugins (with purging support of obsolete tasks/plugins) in Puppet:
jcooklin commentedon Feb 1, 2017
When the documentation is updated in regards to the auto load path it should mention that plugins will need to have the proper permissions (including execute bit set).
Relates to: #1501
Fix intelsdi-x#1499 update explaination to auto-discovery plugin.
Fix intelsdi-x#1499 update explaination to auto-discovery plugin.
Merge pull request #1510 from nanliu/autodis