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

Add support for Debian #65

Closed
kienanstewart opened this issue Jan 18, 2021 · 2 comments
Closed

Add support for Debian #65

kienanstewart opened this issue Jan 18, 2021 · 2 comments

Comments

@kienanstewart
Copy link
Contributor

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 5
  • Ruby: 2.5
  • Distribution: Debian Buster
  • Module version: 1.0.0

Description

There are a few changes to make the module work on Debian

  • the nftables configuration file should /etc/nftables.conf instead of /etc/sysconfig/nftables.conf
  • the directory /etc/nftables is not created by the Debian package for nftables (managing it in the module would work?)
  • firewalld is not installed by default, so should be false instead of mask on Debian systems by default
  • (optional) for Debian Stretch, the version of facter shipped w/ the OS does not use structured facts. If it is desired to extend support to Debian Stretch, that case should be handled as well.

I have a WIP branch dev for which I will open a PR.

thanks

@duritong
Copy link
Collaborator

duritong commented Mar 9, 2022

When people continue to push things on the outstanding pull request and make it possible to merge: #66 . This is a free software project and not a product, if you want something in either contribute or wait (patiently). Thank you.

@geor-g
Copy link

geor-g commented Sep 23, 2022

I believe that's done via #134.

@smortex smortex closed this as completed Sep 23, 2022
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 a pull request may close this issue.

4 participants