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

Use the firewall role and the selinux role from the postfix role #56

Merged
merged 2 commits into from Sep 27, 2022

Conversation

nhosoi
Copy link
Contributor

@nhosoi nhosoi commented Sep 23, 2022

  • Introduce postfix_manage_firewall to use the firewall role to manage the smtp services. Default to false - means the firewall role is not used.

  • Introduce postfix_manage_selinux to use the selinux role to manage the ports in the smtp services. Assign smtp_port_t to the smtp service ports. Default to false - means the selinux role is not used.

  • Add the test check task tasks/check_firewall_selinux.yml for verify the ports status.

  • Add meta/collection-requirements.yml.

- Introduce postfix_manage_firewall to use the firewall role to
  manage the smtp services.
  Default to false - means the firewall role is not used.

- Introduce postfix_manage_selinux to use the selinux role to
  manage the ports in the smtp services.
  Assign smtp_port_t to the smtp service ports.
  Default to false - means the selinux role is not used.

- Add the test check task tasks/check_firewall_selinux.yml for
  verify the ports status.

- Add meta/collection-requirements.yml.
@nhosoi
Copy link
Contributor Author

nhosoi commented Sep 23, 2022

[citest]

README.md Outdated

Otherwise, please run the following command line to install the collection.
```
ansible-galaxy collection install -r meta/requirements.yml
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ansible-galaxy collection install -r meta/requirements.yml
ansible-galaxy collection install -r meta/collection-requirements.yml

@richm
Copy link
Contributor

richm commented Sep 26, 2022

[citest]

@nhosoi
Copy link
Contributor Author

nhosoi commented Sep 27, 2022

@richm, all the CI tests passed. Is it ready to merge?
If the answer is yes, may I squash the two commits into one?

@richm richm merged commit 8348660 into linux-system-roles:master Sep 27, 2022
richm added a commit to richm/linux-system-roles-postfix that referenced this pull request Nov 1, 2022
[1.3.0] - 2022-11-01
--------------------

### New Features

- Use the firewall role and the selinux role from the postfix role (linux-system-roles#56)

- Introduce postfix_manage_firewall to use the firewall role to
  manage the smtp services.
  Default to false - means the firewall role is not used.

- Introduce postfix_manage_selinux to use the selinux role to
  manage the ports in the smtp services.
  Assign smtp_port_t to the smtp service ports.
  Default to false - means the selinux role is not used.

- Add the test check task tasks/check_firewall_selinux.yml for
  verify the ports status.

- Add meta/collection-requirements.yml.

### Bug Fixes

- none

### Other Changes

- none

Signed-off-by: Rich Megginson <rmeggins@redhat.com>
richm added a commit that referenced this pull request Nov 1, 2022
[1.3.0] - 2022-11-01
--------------------

### New Features

- Use the firewall role and the selinux role from the postfix role (#56)

- Introduce postfix_manage_firewall to use the firewall role to
  manage the smtp services.
  Default to false - means the firewall role is not used.

- Introduce postfix_manage_selinux to use the selinux role to
  manage the ports in the smtp services.
  Assign smtp_port_t to the smtp service ports.
  Default to false - means the selinux role is not used.

- Add the test check task tasks/check_firewall_selinux.yml for
  verify the ports status.

- Add meta/collection-requirements.yml.

### Bug Fixes

- none

### Other Changes

- none

Signed-off-by: Rich Megginson <rmeggins@redhat.com>

Signed-off-by: Rich Megginson <rmeggins@redhat.com>
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.

None yet

2 participants