ansible-role-proxmox icon indicating copy to clipboard operation
ansible-role-proxmox copied to clipboard

Optimize sshd compatibility with other roles

Open alexschomb opened this issue 3 years ago • 1 comments

Hi,

Thanks for your great Ansible role, which is really helpful in centralizing/backup the configuration of multiple PVE server hosts in cooperation with PBS for VM backups.

You already stated in the README that compatibility of the sshd_config rules could be conflicting with other Ansible roles or applications managing the same file. Turns out that this is especially true for the popular geerlingguy/ansible-role-security, which conflicts with the Match block created by your role. As a result, I made use of pve_manage_ssh: false and manually moved the initial Match block from /etc/ssh/sshd_config to /etc/ssh/sshd_config.d/proxmox-cluster.conf, making it fully compatible to geerlingguy`s and other applications changes to sshd_config. Wouldn't it be a better practice for your role to write changes to this (or a similar named) separate config file? Should be a simple fix here: https://github.com/lae/ansible-role-proxmox/blob/f3bcd26309c8952ccee0a32184ce86936f6400af/tasks/ssh_cluster_config.yml#L42

The conflicting tasks of geerlingguy/ansible-role-security can be found in /tasks/ssh.yml#L7

alexschomb avatar Jan 17 '22 14:01 alexschomb

I guess that would work. We'd need to change the task to a copy or template task, and probably also keep the blockinfile task but with state: absent for removing it from existing clusters.

(Sorry about the deleted message, was reading all over the place and missed one line)

lae avatar Jan 17 '22 14:01 lae