Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(#86) Enforce permissions on created files #87

Merged
merged 1 commit into from
May 3, 2022

Conversation

silug
Copy link
Contributor

@silug silug commented Apr 27, 2022

  • Add attributes for enforcing ownership and permissions on created files
  • Add data types to all class and defined type parameters
  • Move documentation to Puppet Strings
  • Add generated REFERENCE.md
  • Add rspec tests for kmod::option, new parameters, etc.
  • Bump stdlib dependency to 5.0.0 (for Stdlib::Filemode type)

Fixes #86

@silug silug force-pushed the issue_86 branch 2 times, most recently from 90802c4 to 43a6be6 Compare April 27, 2022 19:28
manifests/init.pp Outdated Show resolved Hide resolved
manifests/install.pp Outdated Show resolved Hide resolved
@silug silug force-pushed the issue_86 branch 4 times, most recently from ef91e77 to a3e7ffd Compare April 28, 2022 18:49
@silug silug marked this pull request as ready for review April 28, 2022 18:51
manifests/init.pp Outdated Show resolved Hide resolved
* Add attributes for enforcing ownership and permissions on created files
* Add data types to all class and defined type parameters
* Move documentation to Puppet Strings
* Add generated REFERENCE.md
* Add rspec tests for `kmod::option`, new parameters, etc.
* Bump stdlib dependency to 5.0.0 (for `Stdlib::Filemode` type)

Fixes voxpupuli#86
) {
include kmod
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why did you add the include here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need that to make sure that /etc/modprobe.d is managed.

@bastelfreak bastelfreak merged commit 30e2a42 into voxpupuli:master May 3, 2022
@bastelfreak bastelfreak added the enhancement New feature or request label May 3, 2022
@silug silug deleted the issue_86 branch May 3, 2022 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need a way to enforce permissions on created files
2 participants