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

Allow adding a select interface to a selected zone. #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

flybyray
Copy link

@flybyray flybyray commented Mar 31, 2022

Fixes #4


name: Pull request
about: Describe the proposed change


Describe the change
This patch allows to add an interface to a selected zone only on operating systems with firewalld. There is a new list variable firewall_interfaces. Example usage:

- role: ansible-role-firewall
  firewall_interfaces:
    - interface: eth0
      zone: trusted

Testing
Tests where done in a new molecule scenario called physical (maybe rename or remove it if there are better options) only on selected operating systems which might be used with firewalld service.

@dgibbs64
Copy link

This is exactly the feature I am after with this role +1 on this feature

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.

Allow adding a select interface to a selected zone.
2 participants