You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would argue that adding a crond service resource (in response to #7) to this module may not be the best the best thing.
You should/could either...
A) make it controlled via parameter
B) at least attempt to see if has been defined somewhere and not maybe not manage it here if so, using the "defined()" parser function
Personally, I think this is best handled with roles/profiles, allowing the dependency to be defined at that level if needed. Right now, with that cron resource, this module is managing resources that belong to another technology (cron). That makes it potentially hard to use some other module that explicitly is about managing cron for example.
Just some food for thought, perhaps a refactor should come from this. Otherwise awesome and well thought out module, thanks!
The text was updated successfully, but these errors were encountered:
Ginja
added a commit
to sfu-rcg/sssd
that referenced
this issue
Dec 1, 2014
I would argue that adding a crond service resource (in response to #7) to this module may not be the best the best thing.
You should/could either...
A) make it controlled via parameter
B) at least attempt to see if has been defined somewhere and not maybe not manage it here if so, using the "defined()" parser function
Personally, I think this is best handled with roles/profiles, allowing the dependency to be defined at that level if needed. Right now, with that cron resource, this module is managing resources that belong to another technology (cron). That makes it potentially hard to use some other module that explicitly is about managing cron for example.
Just some food for thought, perhaps a refactor should come from this. Otherwise awesome and well thought out module, thanks!
The text was updated successfully, but these errors were encountered: