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

List more autorequired resources in firewalld_rich_rule doc (and readme) #370

Open
qha opened this issue May 22, 2024 · 0 comments
Open

Comments

@qha
Copy link
Contributor

qha commented May 22, 2024

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: -
  • Ruby: -
  • Distribution: -
  • Module version: 5.0.0

How to reproduce (e.g Puppet code you use)

Visual inspection of puppet-firewalld/lib/puppet/type/firewalld_rich_rule.rb, puppet-firewalld/README.md and published readme/reference on https://forge.puppet.com/modules/puppet/firewalld

What are you seeing

Helpful statements that claim "firewalld_rich_rules will autorequire the firewalld_zone specified in the zone parameter or the firewalld_policy specified in the policy parameter so there is no need to add dependencies for this" along with even more helpful code that autorequires ipset and service resources in addition to the already mentioned zone and policy resources.

What behaviour did you expect instead

Documentation and code in sync.

Output log

Any additional information you'd like to impart

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

No branches or pull requests

1 participant