Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

satellite_firewall role #21

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rdrazny
Copy link

@rdrazny rdrazny commented Aug 21, 2018

Initial version of satellite_firewall role. Opens network ports required by Satellite 6 to work correctly, both using firewalld (RHEL7+) or iptables (RHEL6 and lower).

- {proto: 'tcp', port: 5671}
- {proto: 'tcp', port: 2375}
- {proto: 'tcp', port: 5000}
- {proto: 'tcp', port: 16500}
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo? 16509

Copy link
Author

Choose a reason for hiding this comment

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

Typo. Fixed.

@@ -0,0 +1,66 @@
Role Name
Copy link
Member

Choose a reason for hiding this comment

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

Fill a role name

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

roles/satellite_firewall/README.md Outdated Show resolved Hide resolved
@ogajduse
Copy link
Member

ogajduse commented Sep 3, 2018

I was thinking about using one of the OASIS roles named firewalld for RHEL7 only. Our satellite_firewall role will be some kind of meta role if in this case.

Possible usage:

- name: Open Satellite ports using firewalld (role)
  include_role:
    name: oasis-roles.firewalld
  with_items:
    - "{{ common_ports }}"
    - "{{ satellite_ports }}"
    - "{{ capsule_ports }}"
  when:
    - ansible_distribution == "RedHat"
    - ansible_distribution_major_version|int >= 7

@ogajduse
Copy link
Member

ogajduse commented Sep 7, 2018

Is the capsule_firewall role worth creating at the moment when we can use this satellite_firewall role which enables all needed ports for capsule?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants