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

Feature/firewall #581

Merged
merged 26 commits into from
Jun 9, 2022
Merged

Conversation

pneerincx
Copy link
Contributor

New iptables role:

  • This is work in progress....

Dependency for new iptables role: When a machine has multiple network interfaces, we need to be able to create rules for specific network interfaces as opposed to filtering all of them with the same rules. Therefore:

  • Added support for multiples network addresses/interfaces per host/machine/network
    • Refactored structure used in ip_addresses.yml files.
    • When ip_addresses.yml files are generated they are automatically sorted on the keys used in the dicts.
    • Create new include_vars_from_other_groups role, which fetches the info from ip_addresses.yml files and stores it into 2 dict that can be used by other roles.
    • Updated all roles that use the info from the ip_addresses.yml files, removed code to parse ip_addresses.yml files and declared a dependency on the new include_vars_from_other_groups` role:
      • sshd
      • static_hostname_lookup
      • ssh_host_signer
    • Updated all plays that use the info from the ip_addresses.yml files:
      • create-docs-server.yml
      • create-jenkins-server.yml
      • deploy-os_servers.yml
    • Updated group_vars for all stacks to work with the new structure in the ip_addresses.yml files.
  • Other minor updates to group_vars:
    • Removed unused variables mailhub and rewrite_domain.
    • Removed entries for old docs server, which no longer exists.
    • Renamed slurm_cluster_domain to stack_domain.
    • Removed public_ip_addresses variables, which is no longer needed.
    • Renamed network_private_management_id, network_private_management_id_13 and network_private_management_id for Nibbler to include stack_prefix in their names. E.g. "{{ stack_prefix }}_internal_management"
      Also renamed these networks on the Merlin cloud.
    • Added umcg_net4 to totp for Winged Helix.
    • Deleted group_vars/marvin_cluster/ip_addresses.yml as cluster is offline: these machines currently do not exist.

Bugfixes for issues that popped up during testing:

  • Added new single_role_playbooks/irods_davrods.yml
  • Re-added static_inventories/calculon_cluster.yml, which was lost in translation from *.ini to *.yml inventories.
  • Remove trailing $ (dollar sign) in network share names. This is used by SMB/CIFS shares to make them "hidden", but only complicates use of paths on Linux.

Regular merge to get latest updates from develop branch.
Regular merge from develop into feature branch.
Regular merge of develop into feature branch.
…r multiples network addresses/interfaces per host/machine/network.
…d by SMB/CIFS shares to make them "hidden", but only complicates use of paths on Linux.
… requirements file) to prevent the linter from interpreting it as a playbook.
@marieke-bijlsma marieke-bijlsma merged commit b0dd6f7 into rug-cit-hpc:develop Jun 9, 2022
@pneerincx pneerincx deleted the feature/firewall branch June 9, 2022 15:06
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.

2 participants