No option to configure multiple drectories in the [include] section. about puppet-supervisord HOT 7 CLOSED

This is a tricky one, given that most people will be configuring all aspects of supervisord through the module having multiple directories would only make configuration more complex.

I thought about having the config_include param as an array but this means I would need to pick one of those as the location for all the configs. I think I will make the config_include param a simple string rather than a absolute path so you can customise it with you own space seperated paths.

Let me know if this works for you

from puppet-supervisord.

Actually this will break things as I rely on this for config file paths in other resources, I will have a think about the best approach.

from puppet-supervisord.

Hey,

Thanks for the quick response. I understand what you mean and that it conflicts with how you use this parameter.

I played a bit with it and found a simple workaround. I added a parameter $file to the class definition and added it also to the template file. With this everything works as before.

Let me know if you need more details. I am currently on the road...

Best Hannes

from puppet-supervisord.

I've committed code to add this feature, see f863a94. You can now specific an array of include paths with $config_dirs. If this is not specified ${config_include}/*conf will be used, so make sure it includes this.

This is a bit of a work around and the name could be better but I want to avoid breaking changes for existing users of the module which this does.

Give it a try and let me know if this work, I'll look to get this out in a release shortly.

from puppet-supervisord.

Hi,

i'll give it a try as soon as possible and write back.

Thanks!
Hannes

from puppet-supervisord.

Hi,

i just gave it a try and it works perfectly. Thanks, a lot for your help. Looking forward to see it in the next release.

Regards
Hannes

from puppet-supervisord.

Great, I'll push out a release shortly.

Thanks for the input.

from puppet-supervisord.