Skip to content

Updated to allow for multiple 'applications' to add users to cron.allow and cron.deny#56

Open
ljkimmel wants to merge 3 commits into
Ericsson:masterfrom
ljkimmel:master
Open

Updated to allow for multiple 'applications' to add users to cron.allow and cron.deny#56
ljkimmel wants to merge 3 commits into
Ericsson:masterfrom
ljkimmel:master

Conversation

@ljkimmel
Copy link
Copy Markdown

Updated to allow for multiple 'applications' to add users to cron.allow and cron.deny via a defined type (cron::allow_deny_fragment).

@Phil-Friderici
Copy link
Copy Markdown
Contributor

Hej @ljkimmel,
thanks for your contribution ! I am working on this in my repository here:
https://github.com/Phil-Friderici/puppet-module-cron/tree/allow_deny_fragment

I'll do some minor refactoring and add spec tests for the new cron::allow_deny_fragment define.
Once finished, I'll come back to you and ask you to verify my changes.

Thanks & phile grüße

@Phil-Friderici
Copy link
Copy Markdown
Contributor

Hej @ljkimmel,

I have finished my work and would like you to give it a test run in your environment.
Please use my feature branch that you can find here: https://github.com/Phil-Friderici/puppet-module-cron/tree/allow_deny_fragment

If you are also satisfied, I will merge it :)

Thanks & phile grüße

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants